0.2.0-alpha release #119
Merged
0.2.0-alpha release #119
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This PR updates the version badge in the README.md to match the current version in Solution.props. This is an automated PR created by the Update Version Badge workflow.
## Description This PR migrates AI providers (MistralAI and OpenAI) to separate projects to improve modularity and enable runtime provider discovery. Key changes include: - Migrating the provider-specific code from the core project to dedicated projects. - Introducing ProviderManager and IAIProviderFactory for dynamic provider registration and discovery. - Updating core components (e.g., AIStatefulAsyncComponentBase and SmartHopperSettings) to integrate with the new provider architecture. ## Breaking Changes None. ## Testing Done RH8.17 on windows. ## Checklist - [x] This PR is focused on a single feature or bug fix - [x] Version in Solution.props was updated, if necessary, and follows semantic versioning - [x] CHANGELOG.md has been updated - [x] PR title follows [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) format - [x] PR description follows [Pull Request Description Template](#pull-request-description-template)
## Branch Update This PR updates `dev` with changes from `main`. ### ℹ️ Protected Branch This PR was created because `dev` is a protected branch that requires changes through pull requests. ### Instructions 1. Review the changes 2. Approve and merge the changes This PR was automatically created by the Branch Update workflow.
…t/SmartHopper into marc-romu/chat
Fixes Bug: Settings menu hides sometimes #94
…#97) ## Description This pull request resolves issue #94 where the settings menu was hidden behind other windows by refactoring the settings dialog code into a dedicated dialog class. Key changes include: - Refactoring the settings dialog UI from SettingsMenuItem into a standalone SettingsDialog. ## Breaking Changes None detected. ## Testing Done RH8.17 on windows. ## Checklist - [x] This PR is focused on a single feature or bug fix - [x] Version in Solution.props was updated, if necessary, and follows semantic versioning - [ ] CHANGELOG.md has been updated - [x] PR title follows [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) format - [x] PR description follows [Pull Request Description Template](#pull-request-description-template)
Fixes Bug: AI Chat component freezes all Rhino! #85
Wrap the Task.Run call in a try-catch block to handle any exceptions from _htmlRenderer.GetInitialHtml() and prevent unhandled errors during initialization. Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
## Description This PR fixes two issues with the WebChatDialog: 1. Prevents UI freezing during initial chat window launch 2. Makes the chat window non-blocking so users can interact with other Rhino/Grasshopper tools while the chat is open Fixes #85 ## Changes - Moved HTML generation to a background thread before loading into WebView - Updated dialog initialization to prevent UI thread blocking ## Breaking Changes None detected. ## Testing Done RH8.17 on windows ## Checklist - [x] This PR is focused on a single feature or bug fix - [x] Version in Solution.props was updated, if necessary, and follows semantic versioning - [ ] CHANGELOG.md has been updated - [x] PR title follows [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) format - [x] PR description follows [Pull Request Description Template](#pull-request-description-template)
…-branch/marc-romu/chat-from-marc-romu/workflows-1743541548
This PR updates the development version date in Solution.props to reflect the latest changes. This is an automated PR created by the Update Development Version Date workflow.
…mu/chat-from-main-1743603229
This PR updates the version badge in the README.md to match the current version in Solution.props. This is an automated PR created by the Update Version Badge workflow.
…m-main-1743610147
## Branch Update This PR updates `dev` with changes from `main`. ### ℹ️ Protected Branch This PR was created because `dev` is a protected branch that requires changes through pull requests. ### Instructions 1. Review the changes 2. Approve and merge the changes This PR was automatically created by the Branch Update workflow.
…u/chat-from-dev-1743610230
…t/SmartHopper into marc-romu/chat
…out menu (#115) ## Description Improvements in the text and visualization of the about menu ## Breaking Changes None. ## Testing Done Rh8.17 on windows ## Checklist - [x] This PR is focused on a single feature or bug fix - [x] Version in Solution.props was updated, if necessary, and follows semantic versioning - [x] CHANGELOG.md has been updated - [x] PR title follows [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) format - [x] PR description follows [Pull Request Description Template](#pull-request-description-template)
This PR updates the version badge in the README.md to match the current version in Solution.props. This is an automated PR created by the Update Version Badge workflow.
## Description - [fix(aichat): allow only one open chat dialog per component](0da4d6e) - [fix(aichat): issue when chat dialog remained hidden behind other rhin…](b01f230) - [fix(aichat): eventual freezing of all Rhino-related dialogs on init t…](8273c6e) ## Breaking Changes AI Chat no longer opens multiple dialogs. ## Testing Done RH8.17 on windows ## Checklist - [ ] This PR is focused on a single feature or bug fix - [ ] Version in Solution.props was updated, if necessary, and follows semantic versioning - [ ] CHANGELOG.md has been updated - [ ] PR title follows [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) format - [ ] PR description follows [Pull Request Description Template](#pull-request-description-template)
This PR updates the version badge in the README.md to match the current version in Solution.props. This is an automated PR created by the Update Version Badge workflow.
Contributor
There was a problem hiding this comment.
Copilot reviewed 47 out of 52 changed files in this pull request and generated no comments.
Files not reviewed (5)
- .windsurfrules: Language not supported
- SmartHopper.sln: Language not supported
- Solution.props: Language not supported
- src/SmartHopper.Config/Properties/providersResources.Designer.cs: Language not supported
- src/SmartHopper.Config/Properties/providersResources.resx: Language not supported
Comments suppressed due to low confidence (2)
src/SmartHopper.Config/ProviderManager.cs:187
- Consider logging a warning or error when a duplicate provider (same name) is encountered instead of silently ignoring the duplicate registration.
if (!_providers.ContainsKey(providerName))
src/SmartHopper.Components/Text/AITextGenerate.cs:162
- [nitpick] Since the same context filter value is used in multiple components, consider defining a named constant or utility function for the filter string to improve maintainability and avoid potential typos.
messages => parent.GetResponse(messages, contextProviderFilter: "-environment,-time")
…loaded with the same name
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.