Skip to content

Conversation

@marc-romu
Copy link
Member

No description provided.

marc-romu and others added 8 commits March 3, 2025 18:49
…hanges (#55)

## Description

This PR fixes an issue with the AI Chat component where it wouldn't open
the chat dialog when the Run parameter was set to true but no input
values had changed. The component should always run when the Run
parameter is set to true, regardless of input changes.

The fix adds a new RunOnlyOnInputChanges property to the
StatefulAsyncComponentBase class that controls whether a component
should only run when inputs change. This property is set to true by
default for most components, but is set to false for the AI Chat
component, allowing it to run every time the Run parameter is set to
true.

Key changes:

- Added RunOnlyOnInputChanges property to StatefulAsyncComponentBase
- Modified StatefulAsyncComponentBase.SolveInstance to check this
property
- Set RunOnlyOnInputChanges = false in the AIChatComponent constructor

## Breaking Changes

No breaking changes. All existing components will maintain their current
behavior with the default value of RunOnlyOnInputChanges = true.

## Testing Done

- Verified that the AI Chat component now runs every time the Run
parameter is set to true
- Confirmed that other components still only run when their inputs
change

## 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)
Automated PR to update version badge in README.md for dev branch

  Changes:
  - Updated version badge to 0.1.2-dev.250308
  - Updated status and color based on version type
…con display (#57)

## Description

This PR introduces a centralized default provider selection and improves
the component icon display when using the default provider. The changes
include:

- Added a dropdown in the settings dialog to select the global default
AI provider
- Modified AIComponentAttributes to display the correct provider icon
when "Default" is selected
- Updated CHANGELOG.md to document these changes
- These changes provide a more intuitive user experience by allowing
centralized configuration of the default provider and ensuring visual
consistency in component display.

Fixes #41

## Breaking Changes

There are no breaking changes in this PR. Existing functionality remains
the same, with added features for better provider management.

## Testing Done

- Tested default provider selection in settings dialog
- Verified correct icon display when "Default" is selected in components
- Checked persistence of default provider setting across sessions

## 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)
@marc-romu marc-romu merged commit 8f8645a into marc-romu/chat Mar 8, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants