Skip to content

sync main#2207

Merged
laileni-aws merged 25 commits into
feature/server-side-semantic-searchfrom
main
Sep 3, 2025
Merged

sync main#2207
laileni-aws merged 25 commits into
feature/server-side-semantic-searchfrom
main

Conversation

@liumofei-amazon

Copy link
Copy Markdown
Contributor

Problem

Solution

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

andrewyuq and others added 25 commits August 26, 2025 15:51
* Updating the model of listAvailableModels (#2064)

* fix(amazonq): fix flickering issue for model selection dropdown and agenticCoding toggle (#2065)

* fix(amazonq): fix flickering issue for modelId and agenticCoding

* fix(amazonq): Fixing flaky tests

* feat(amazonq): Fetching models from backend and adding cache implementation. (#2075)

* fix: removing and refactoring legacy code before implementing model selection

* feat(amazonq): adding cache implementation and fetching models from listAvailableModels api

* feat(amazonq): adding selected model in error case

* feat(amazonq): adding test cases

* fix: addressing comments

* fix: fixing test cases and adding modelName to models

* fix: minor edits

* fix: minor edits

* fix: minor modifications in logs

* fix: adding default model if api throws any errors

* fix: refactoring code

* fix:  Improve model selection fallback logic when user's preferred model is unavailable (#2089)

* fix: if user preferred model does not exist, fall back to default model

* fix: minor test changes

* fix: to support backward compatibility for vs and eclipse, adding back modelSelection in chat-client (#2095)

* fix: check available models from backend before selecting default model from fallback models (#2102)

* feat(amazonq): use model display names (#2123)

* fix: cached model list should be invalidated on sign out (#2131)

* fix: cached model list should be invalidated on sign out

* fix test

* avoid throwing error

* fix: adding default modelId and setting cache ttl to 30 minutes (#2161)

* fix: adding defaultmodelId and setting cache ttl to 30 minutes

* fix: fixing tests

* fix: updating comments

* Updating the model of listAvailableModels (#2064)

* fix(amazonq): fix flickering issue for model selection dropdown and agenticCoding toggle (#2065)

* fix(amazonq): fix flickering issue for modelId and agenticCoding

* fix(amazonq): Fixing flaky tests

* feat(amazonq): Fetching models from backend and adding cache implementation. (#2075)

* fix: removing and refactoring legacy code before implementing model selection

* feat(amazonq): adding cache implementation and fetching models from listAvailableModels api

* feat(amazonq): adding selected model in error case

* feat(amazonq): adding test cases

* fix: addressing comments

* fix: fixing test cases and adding modelName to models

* fix: minor edits

* fix: minor edits

* fix: minor modifications in logs

* fix: adding default model if api throws any errors

* fix: refactoring code

* fix:  Improve model selection fallback logic when user's preferred model is unavailable (#2089)

* fix: if user preferred model does not exist, fall back to default model

* fix: minor test changes

* fix: to support backward compatibility for vs and eclipse, adding back modelSelection in chat-client (#2095)

* fix: check available models from backend before selecting default model from fallback models (#2102)

* feat(amazonq): use model display names (#2123)

* fix: cached model list should be invalidated on sign out (#2131)

* fix: cached model list should be invalidated on sign out

* fix test

* avoid throwing error

* fix: adding default modelId and setting cache ttl to 30 minutes (#2161)

* fix: adding defaultmodelId and setting cache ttl to 30 minutes

* fix: fixing tests

* fix: updating comments

* fix: lint issue while resolving merge conflicts

---------

Co-authored-by: Tai Lai <ctlai95@gmail.com>
…r card (#2176)

Co-authored-by: Laxman Reddy <141967714+laileni-aws@users.noreply.github.com>
042c98e6: optimizing memory usage in Indexing
Need to emit a metric for each issue found by the agentic reviewer to reach parity with old behavior.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This reverts commit 28567e3.

indexing bundle contains a crypto polyfill that is needed for some reason
* emit errorMessage in addMessage

* fix: emit errorMessage in addMessage
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* feat: enabling model selection for code review tool

* feat: added unit test for model selection for code review tool

* fix: making modelId required for codeReviewTool in input schema validation

* fix: fixing unit tests

---------

Co-authored-by: Laxman Reddy <141967714+laileni-aws@users.noreply.github.com>
@liumofei-amazon
liumofei-amazon requested a review from a team as a code owner September 3, 2025 22:54

@laileni-aws laileni-aws left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please opt from Merge commit before merging this PR or else git history won't align.

@laileni-aws

Copy link
Copy Markdown
Contributor

Please opt from Merge commit before merging this PR or else git history won't align.

Actually this is not enabled in Settings so even though PR gets merged but still we can sync history of both branches.

Only option is to enable the Merge Commit option from settings.

@laileni-aws
laileni-aws merged commit df0f7d3 into feature/server-side-semantic-search Sep 3, 2025
35 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.