Feature to add IAM inline suggestion support in CodeWhispererService #2223

parameja1 · 2025-09-09T16:35:25Z

Problem

Currently IAM Auth support is missing in flare for InlineSuggestions, which is mainly used in Sagemaker instances

Solution

Changes are to add IAM request and response processing in flare to handle InlineSuggestions
Update to use dynamic CodeWhisperer client in CodeReview tool
Add unit tests to cover critical areas of dynamic service manager selection and iam auth check util

-- Tested in local with debug CodeWhispererServer configuration and working

-- Will update with test screen recordings for SMUS and Standalone VSCode instances
-- DO NOT MERGE yet, raising PR for comments/review meanwhile

License

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

* fix(amazonq): leverage lcs to find the chars added and removed (aws#2092) * fix: update client name to support Sagemaker AI origin for agentic chat (aws#2093) Co-authored-by: chungjac <chungjac@amazon.com> * chore(release): release packages from branch main (aws#2073) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * chore: format version.json after incrementing (aws#2068) * chore: merge agentic version 1.27.0 (aws#2107) * Bump agentic version: 1.27.0 * Revert "feat(amazonq): read tool ui revamp" This reverts commit c65428b. * fix: the style in version json (aws#2106) --------- Co-authored-by: aws-toolkit-automation <> Co-authored-by: Tai Lai <tlaiamz@amazon.com> Co-authored-by: Christopher Christou <39839589+awschristou@users.noreply.github.com> * fix(amazonq): persist mcp configs in agent json on start-up (aws#2112) --------- Co-authored-by: andrewyuq <89420755+andrewyuq@users.noreply.github.com> Co-authored-by: chungjac <chungjac@amazon.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Christopher Christou <39839589+awschristou@users.noreply.github.com> Co-authored-by: Sherry Lu <75588211+XiaoxuanLu@users.noreply.github.com> Co-authored-by: Tai Lai <tlaiamz@amazon.com> Co-authored-by: Dung Dong <dungdong@amazon.com>

…are (aws#2134) * Chore: Sync test smus branch with main branch commits (aws#2097) * fix(amazonq): skips continuous monitoring when WCS sees workspace as idle (aws#2066) * fix(amazonq): skips continuous monitoring when WCS sees workspace as idle * fix(amazonq): skips creating remote workspace at the start --------- Co-authored-by: Jiatong Li <jiatonli@amazon.com> * fix: sessionManager misused because there are 2 types of manager now (aws#2090) * chore: mapping adt plugin to abap to let inline suggestion work (aws#2085) * chore: support inline suggestion in adt plugin for eclipse * fix: add import URI * chore: add detailed error description for uri is empty for language id * feat(amazonq): read tool ui revamp * feat(amazonq): read tool message revamp (aws#2049) * feat(amazonq): read tool message revamp * fix tests * feat: file search ui (aws#2078) * feat: file search ui * fix tests * fix integration tests * remove unnecessary type check * fix: use quotes instead of backticks * fix: creating a new sesion for Edits trigger with next token (aws#2094) * chore: bump @aws/mynah-ui to 4.36.4 (aws#2096) --------- Co-authored-by: Jiatong Li <lijiatong1997@outlook.com> Co-authored-by: Jiatong Li <jiatonli@amazon.com> Co-authored-by: Will Lo <96078566+Will-ShaoHua@users.noreply.github.com> Co-authored-by: Sherry Lu <75588211+XiaoxuanLu@users.noreply.github.com> Co-authored-by: Tai Lai <ctlai95@gmail.com> Co-authored-by: atontb <104926752+atonaamz@users.noreply.github.com> * feat(amazonq): add inline suggestion support for iam auth client * fix(amazonq): skips continuous monitoring when WCS sees workspace as idle (aws#2066) * fix(amazonq): skips continuous monitoring when WCS sees workspace as idle * fix(amazonq): skips creating remote workspace at the start --------- Co-authored-by: Jiatong Li <jiatonli@amazon.com> * fix: sessionManager misused because there are 2 types of manager now (aws#2090) * chore: mapping adt plugin to abap to let inline suggestion work (aws#2085) * fix(amazonq): leverage lcs to find the chars added and removed (aws#2092) * fix: update client name to support Sagemaker AI origin for agentic chat (aws#2093) Co-authored-by: chungjac <chungjac@amazon.com> * chore(release): release packages from branch main (aws#2073) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * chore: fix test failures in merge due to missing init of IdleWorkspaceManager --------- Co-authored-by: Jiatong Li <lijiatong1997@outlook.com> Co-authored-by: Jiatong Li <jiatonli@amazon.com> Co-authored-by: Will Lo <96078566+Will-ShaoHua@users.noreply.github.com> Co-authored-by: Sherry Lu <75588211+XiaoxuanLu@users.noreply.github.com> Co-authored-by: Tai Lai <ctlai95@gmail.com> Co-authored-by: atontb <104926752+atonaamz@users.noreply.github.com> Co-authored-by: andrewyuq <89420755+andrewyuq@users.noreply.github.com> Co-authored-by: chungjac <chungjac@amazon.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: invictus <149003065+ashishrp-aws@users.noreply.github.com>

…ting

…uth logic check

…anager logic and add unit tests

codecov-commenter · 2025-09-09T16:40:14Z

Codecov Report

❌ Patch coverage is 44.51220% with 91 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.71%. Comparing base (470492c) to head (7fa59aa).

Files with missing lines	Patch %	Lines
...p-codewhisperer/src/shared/codeWhispererService.ts	39.50%	49 Missing ⚠️
...rc/language-server/agenticChat/tools/toolServer.ts	0.00%	20 Missing ⚠️
server/aws-lsp-codewhisperer/src/shared/utils.ts	19.04%	17 Missing ⚠️
...ge-server/inline-completion/codeWhispererServer.ts	88.09%	5 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2223      +/-   ##
==========================================
- Coverage   59.73%   59.71%   -0.03%     
==========================================
  Files         254      254              
  Lines       56125    56245     +120     
  Branches     3326     3337      +11     
==========================================
+ Hits        33527    33584      +57     
- Misses      22534    22597      +63     
  Partials       64       64

Flag	Coverage Δ
unittests	`59.71% <44.51%> (-0.03%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

leigaol · 2025-09-09T17:29:49Z

server/aws-lsp-codewhisperer/src/language-server/inline-completion/codeWhispererServer.ts

        const onInitializedHandler = async () => {
-            amazonQServiceManager = serviceManager()
+            try {
+                amazonQServiceManager = serviceManager(credentialsProvider)


do you really need L758?

Is the question about passing credentialsProvider? Currently not being used, added it as a future-proofing case to use credsProvider to determine auth

what is your suggestion?

why do you need amazonQServiceManager = serviceManager(credentialsProvider) when you can just do

amazonQServiceManager = isUsingIAMAuth(credentialsProvider) ? getOrThrowBaseIAMServiceManager() : getOrThrowBaseTokenServiceManager()

If overridden it would break the factory pattern and also unit tests for CWServer needs to be rewritten
@leigaol - As discussed let me remove the fallback service manager logic here and update, thanks

leigaol · 2025-09-09T17:30:28Z

server/aws-lsp-codewhisperer/src/shared/codeWhispererService.ts

    requestId: string
    codewhispererSessionId: string
    nextToken?: string
+    serviceType?: 'iam' | 'token'


nit: authType?

Ack, will update

sdharani91 · 2025-09-09T18:11:16Z

app/aws-lsp-codewhisperer-runtimes/src/agent-standalone.ts

    AmazonQServiceServerToken,
    CodeWhispererSecurityScanServerTokenProxy,
-    CodeWhispererServerTokenProxy,
+    CodeWhispererServer,


Don't you need to create a proxy for CodeWhispererServer?

Not needed since its a dynamic server instance already created, i see adding proxy would just be redundant wrapper in this case - is there a reason that you see to add proxy?

Will-ShaoHua · 2025-09-09T19:54:36Z

server/aws-lsp-codewhisperer/src/language-server/inline-completion/codeWhispererServer.ts

+                }
+
+                // Log detailed information about the request context
+                logging.error(`[IAM AUTH ERROR] Request context: ${JSON.stringify(session.requestContext)}`)


i believe this part of log already cover what you are looking for here
https://github.com/aws/language-servers/blob/main/server/aws-lsp-codewhisperer/src/shared/codeWhispererService.ts#L443-L478

Ack, addressing in next commit

Will-ShaoHua · 2025-09-09T19:56:19Z

server/aws-lsp-codewhisperer/src/language-server/inline-completion/codeWhispererServer.ts

+                if (error.message?.includes('not authorized') || error.message?.includes('AccessDenied')) {
+                    logging.error(
+                        `[IAM AUTH ERROR] IAM authentication error: User not authorized. Check IAM permissions for CodeWhisperer.`
+                    )
+                } else if (error.message?.includes('invalid parameter')) {
+                    logging.error(
+                        `[IAM AUTH ERROR] IAM authentication error: Invalid request parameters for IAM client`
+                    )
+                } else if (error.message?.includes('credentials')) {
+                    logging.error(`[IAM AUTH ERROR] IAM authentication error: Invalid or missing credentials`)
+                }


i dont get why we need such different log string

it makes sense if it's an UI which will be surfaced to users but they are not.

Ack, updating the error logs

…er logic

leigaol

Please do a lot of regression test with IAM and Bearer Token. We don't have a comprehensive E2E test, so please do it manually

* fix: Use file context override in the inline completion params for Jupyter Notebook (#2114) * feat(amazonq): read tool ui revamp (#2113) (#2121) * feat(amazonq): read tool ui revamp * feat(amazonq): read tool message revamp (#2049) * feat(amazonq): read tool message revamp * fix tests * feat: file search ui (#2078) * feat: file search ui * fix tests * fix integration tests * remove unnecessary type check * fix: use quotes instead of backticks * fix header update issue * fix integration test * refactor: improve generateCompletion logging format (#2125) * fix(amazonq): handle case where multiple rules are provided with the same name (#2118) * fix(amazonq): handle case where multiple rules are provided with the same name * fix(amazonq): add unit test for duplicate custom guidelines * fix(amazonq): add unit test for processToolUses * fix(amazonq): set context type for AmazonQ.md to rule * fix(amazonq): add README.md back as rule context type --------- Co-authored-by: Blake Lazarine <blakelaz@amazon.com> * perf: remove edit completion retry mechanism on document change (#2124) * ci: remove need for custom PAT for release branch workflow (#2126) The token from GitHub Actions is sufficient * fix: proper path handling for additional context (#2129) * fix: proper pathing for additonal context * fix: update existing tests to also mock path.join() * fix(amazonq): fix regression of mcp config in agent config (#2101) * fix(amazonq): update process-permission-updates to be the same as previous behavior * fix(chat-client): update package.json chat-client to prod * fix(amazonq): retain mcp permissions after disable/enable server * fix: dont call initOneServer on Built-in * fix: deny permission does not persist after restart IDE --------- Co-authored-by: Boyu <bywang@amazon.com> * feat: remove project type validation from LSP layer (#2103) * feat: remove project type validation from LSP layer * refactor: remove unused validation functions from LSP layer * fix: fix for button text and remove profilearn caching (#2137) * chore: bump runtimes to 0.2.126 (#2138) * fix: fix to add disk caching for mcp admin state (#2139) * fix: fix to add disk caching for mcp admin state * fix: fix to add logging * fix: fix to initialize mcpManager in any case and discover servers based on mcpState * fix: fix for unit test failure * refactor: cleanup old nep code path which is no longered being used in codewhispererServer.ts (#2141) * fix(amazonq): add server side control for WCS features (#2128) Co-authored-by: Jiatong Li <jiatonli@amazon.com> * chore: update STE userContext version metadata (#2142) * fix: fix to turn on and off MCP servers incase of error based on last state (#2143) Co-authored-by: Laxman Reddy <141967714+laileni-aws@users.noreply.github.com> * fix: empty userTriggerDecision not being sent for NEP code path (#2140) * refactor: clean up old NEP code path which is no longer used * fix: empty userTriggerDecision not being sent for NEP code path * test: disable telemetry test temporarily * fix: patch * revert: a * revert: a * chore(release): release packages from branch main (#2117) * fix(amazonq): don't let flare send discard for the still valid suggestion in JB (#2145) * fix(amazonq): don't let flare send discard for the still valid suggestion in JB if a valid sessionId is returned back to JB, JB will eventually send a decision for it, but when user types a character to reject the current suggestion, on JB side it will 1) send reject for the current one 2) send a new trigger for the latest context. 2) will happen before 1) which will discard the current active session on flare's end. We don't want flare to do that for JB. * fix(amazonq): createSession doesn't need to close the previous session it should be handled together with telemetry reporting * fix(amazonq): fix test * fix(amazonq): test fix attempt * fix(amazonq): test fix attempt * chore(release): release packages from branch main (#2149) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * chore: merge agentic version 1.29.0 (#2151) * chore: bump agentic version: 1.29.0 * chore: empty commit to start Github Action (#2150) --------- Co-authored-by: aws-toolkit-automation <> * feat: add basic OAuth client for remote MCP (#2136) * feat: add basic mcp oauth client * fix: relax condition for triggering oauth flow; add resilience checks for as availability * feat: add unit test for auth client * fix: multiple fixes for remote mcp error and timeout * fix: adding streakTracker to track streakLength across Completions and Edits (#2147) * feat: disable pkce flow during plugin load (#2153) * chore: bump runtimes to 0.2.127 (#2156) * fix: multiple fixes on auth flow edge cases (#2155) * feat: update MCP manager and utilities (#2158) Co-authored-by: Boyu <bywang@amazon.com> * fix(amazonq): disable typewriter animation (#2160) * ci: generation of builds action can be triggered manually (#2157) Co-authored-by: Laxman Reddy <141967714+laileni-aws@users.noreply.github.com> * fix(amazonq): fix for mcp servers operations to edit server config only (#2165) * fix(amazonq): fix for mcp servers operations to edit server specific config * fix(amazonq): additional mcp server config fixes * fix: resolve test failures * fix: update MCP manager configuration * fix: allow ci to run on release branches (#2159) Currently release/agentic/* branches used in our release process have rules that require CI to be run. However the corresponding workflows do not actually trigger for release branches. This change ensures no manual overrides are required in the release process and CI mapped with the branch actually runs. * fix: reduce auto trigger frequency for VSC (#2168) * fix: reduce auto trigger frequency for VSC * fix: skip one unit test * fix: skip unit test --------- Co-authored-by: Laxman Reddy <141967714+laileni-aws@users.noreply.github.com> * fix(amazonq): dedupe openTabs supplemental contexts (#2172) * fix: fix pkce windows url path (#2173) * feat: Auto fetch models from listAvailableModels API (#2171) * 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> * fix(amazonq): fix to add mcp server tool error handling and status for card (#2176) Co-authored-by: Laxman Reddy <141967714+laileni-aws@users.noreply.github.com> * revert: reduce auto trigger frequency for VSC (#2168)" (#2177) This reverts commit 00e11ff. * fix(amazonq): status message update for mcp tool permission accpetance (#2178) * deps: update indexing bundle to 042c98e6 (#2174) 042c98e6: optimizing memory usage in Indexing * feat(amazonq): emit metric for each issue (#2179) Need to emit a metric for each issue found by the agentic reviewer to reach parity with old behavior. * chore(release): release packages from branch main (#2152) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * chore: bump agentic version: 1.30.0 * chore: empty commit to trigger workflow * revert: deps: update indexing bundle to 042c98e6 (#2174) (#2183) This reverts commit 28567e3. indexing bundle contains a crypto polyfill that is needed for some reason * fix: emit acceptedLineCount metric and AgenticCodeAccepted interaction type (#2167) * fix: auto trigger should only respect previous decisions in the past 2mins (#2189) * chore: supplemental context log (#2185) * fix: should send classifier score after taking sigmoid (#2188) * perf: only process edit requests 1 at a time (#2187) * feat: passing suggestionTypes and pluginVersion/lspVersion to STE (#2180) * fix: compact UI is not updated correctly when multiple nudges are displayed (#2192) * fix: fix calculation for num-lines contributed by the LLM (#2191) * revert: PR 2172 dedupe openTabs supplemental contexts (#2194) This reverts commit aa87ae2. * fix: emit errorMessage in addMessage (#2197) * emit errorMessage in addMessage * fix: emit errorMessage in addMessage * chore(release): release packages from branch main (#2190) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * feat: model selection for code review tool (#2196) * 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> * fix(amazonq): fix to update MCP servers list when last server is removed from agent config (#2206) * chore: merge agentic version 1.31.0 (#2205) * chore: bump agentic version: 1.31.0 * chore: empty commit to trigger release workflow (#2203) --------- Co-authored-by: aws-toolkit-automation <> * fix(amazonq): add IntelliSense autotriggerType (#2199) * fix(amazonq): fix to correct the client for getProfile request (#2211) * fix(amazonq): fix to correct the client for getProfile request * fix(amazonq): removed unneccessary constructor objects * fix: fix to add assigned VARs in unit tests * feat: add support for getSupplementalContext LSP API (#2212) * feat: add supplemental context API * feat: add supplemental context API support * feat: add custom_transformation folder support to artifact.zip (#2201) * feat: add custom_transformation folder support to artifact.zip * feat: add custom transformation folder support with async file operations * fix: improve custom transformation folder access logging message --------- Co-authored-by: huawenm <72728725+huawenm@users.noreply.github.com> * feat(amazonq): default to diff-based scans (#2195) * feat(amazonq): default to diff-based scans * feat(amazonq): make /review behavior clearly show it is a diff scan * fix(amazonq): clean up commented lines * fix(amazonq): fix maximum finding count in CodeReview tool description * fix(amazonq): improve messaging for diff vs full scans * feat(amazonq): have codeReview tool ask for clarification if it is not sure what scope to review * feat(amazonq): when there is no diff, fall back to a full review * Update chat-client/src/client/mynahUi.ts Co-authored-by: Tai Lai <ctlai95@gmail.com> * fix(amazonq): fix test * fix(amazonq): fix another test --------- Co-authored-by: Blake Lazarine <blakelaz@amazon.com> Co-authored-by: Tai Lai <ctlai95@gmail.com> * fix: potential xss issue reported in `mynah-ui` (#2209) * fix: potential xss issue * fix: addressing comments * fix: unescaping the history message while opening history tab * fix(amazonq): update to the agent config format to bring parity with Q CLI (#2202) * fix(amazonq): add arbitrary files to clients.zip (#2217) * chore(release): release packages from branch main (#2204) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: chungjac <chungjac@amazon.com> * feat: feature to add iam inline suggestion support in codeWhispererservice (#2223) * Sync feature smus inline suggestions branch from main commits (#2109) * fix(amazonq): leverage lcs to find the chars added and removed (#2092) * fix: update client name to support Sagemaker AI origin for agentic chat (#2093) Co-authored-by: chungjac <chungjac@amazon.com> * chore(release): release packages from branch main (#2073) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * chore: format version.json after incrementing (#2068) * chore: merge agentic version 1.27.0 (#2107) * Bump agentic version: 1.27.0 * Revert "feat(amazonq): read tool ui revamp" This reverts commit c65428b. * fix: the style in version json (#2106) --------- Co-authored-by: aws-toolkit-automation <> Co-authored-by: Tai Lai <tlaiamz@amazon.com> Co-authored-by: Christopher Christou <39839589+awschristou@users.noreply.github.com> * fix(amazonq): persist mcp configs in agent json on start-up (#2112) --------- Co-authored-by: andrewyuq <89420755+andrewyuq@users.noreply.github.com> Co-authored-by: chungjac <chungjac@amazon.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Christopher Christou <39839589+awschristou@users.noreply.github.com> Co-authored-by: Sherry Lu <75588211+XiaoxuanLu@users.noreply.github.com> Co-authored-by: Tai Lai <tlaiamz@amazon.com> Co-authored-by: Dung Dong <dungdong@amazon.com> * feat(amazonq): Integrate IAM Auth support for InlineSuggestions in Flare (#2134) * Chore: Sync test smus branch with main branch commits (#2097) * fix(amazonq): skips continuous monitoring when WCS sees workspace as idle (#2066) * fix(amazonq): skips continuous monitoring when WCS sees workspace as idle * fix(amazonq): skips creating remote workspace at the start --------- Co-authored-by: Jiatong Li <jiatonli@amazon.com> * fix: sessionManager misused because there are 2 types of manager now (#2090) * chore: mapping adt plugin to abap to let inline suggestion work (#2085) * chore: support inline suggestion in adt plugin for eclipse * fix: add import URI * chore: add detailed error description for uri is empty for language id * feat(amazonq): read tool ui revamp * feat(amazonq): read tool message revamp (#2049) * feat(amazonq): read tool message revamp * fix tests * feat: file search ui (#2078) * feat: file search ui * fix tests * fix integration tests * remove unnecessary type check * fix: use quotes instead of backticks * fix: creating a new sesion for Edits trigger with next token (#2094) * chore: bump @aws/mynah-ui to 4.36.4 (#2096) --------- Co-authored-by: Jiatong Li <lijiatong1997@outlook.com> Co-authored-by: Jiatong Li <jiatonli@amazon.com> Co-authored-by: Will Lo <96078566+Will-ShaoHua@users.noreply.github.com> Co-authored-by: Sherry Lu <75588211+XiaoxuanLu@users.noreply.github.com> Co-authored-by: Tai Lai <ctlai95@gmail.com> Co-authored-by: atontb <104926752+atonaamz@users.noreply.github.com> * feat(amazonq): add inline suggestion support for iam auth client * fix(amazonq): skips continuous monitoring when WCS sees workspace as idle (#2066) * fix(amazonq): skips continuous monitoring when WCS sees workspace as idle * fix(amazonq): skips creating remote workspace at the start --------- Co-authored-by: Jiatong Li <jiatonli@amazon.com> * fix: sessionManager misused because there are 2 types of manager now (#2090) * chore: mapping adt plugin to abap to let inline suggestion work (#2085) * fix(amazonq): leverage lcs to find the chars added and removed (#2092) * fix: update client name to support Sagemaker AI origin for agentic chat (#2093) Co-authored-by: chungjac <chungjac@amazon.com> * chore(release): release packages from branch main (#2073) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * chore: fix test failures in merge due to missing init of IdleWorkspaceManager --------- Co-authored-by: Jiatong Li <lijiatong1997@outlook.com> Co-authored-by: Jiatong Li <jiatonli@amazon.com> Co-authored-by: Will Lo <96078566+Will-ShaoHua@users.noreply.github.com> Co-authored-by: Sherry Lu <75588211+XiaoxuanLu@users.noreply.github.com> Co-authored-by: Tai Lai <ctlai95@gmail.com> Co-authored-by: atontb <104926752+atonaamz@users.noreply.github.com> Co-authored-by: andrewyuq <89420755+andrewyuq@users.noreply.github.com> Co-authored-by: chungjac <chungjac@amazon.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: invictus <149003065+ashishrp-aws@users.noreply.github.com> * fix: add unit tests for IAM flow, remove redundant logs added for testing * fix: file formatting * fix: rebase from main, clean up redundant logs * fix: remove old code refs and debug logs * feat: use cw proxy server based on iam auth logic check * fix: todo comment and unused imports * fix: update codewhisperer server to use proper manager based on iam auth logic check * feat: create new codewhisperer server instance with dynamic service manager logic and add unit tests * fix: address pr comments, fix unit tests as per updated service manager logic --------- Co-authored-by: andrewyuq <89420755+andrewyuq@users.noreply.github.com> Co-authored-by: chungjac <chungjac@amazon.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Christopher Christou <39839589+awschristou@users.noreply.github.com> Co-authored-by: Sherry Lu <75588211+XiaoxuanLu@users.noreply.github.com> Co-authored-by: Tai Lai <tlaiamz@amazon.com> Co-authored-by: Dung Dong <dungdong@amazon.com> Co-authored-by: Jiatong Li <lijiatong1997@outlook.com> Co-authored-by: Jiatong Li <jiatonli@amazon.com> Co-authored-by: Will Lo <96078566+Will-ShaoHua@users.noreply.github.com> Co-authored-by: Tai Lai <ctlai95@gmail.com> Co-authored-by: atontb <104926752+atonaamz@users.noreply.github.com> Co-authored-by: invictus <149003065+ashishrp-aws@users.noreply.github.com> Co-authored-by: Laxman Reddy <141967714+laileni-aws@users.noreply.github.com> * feat: upgrading node version from 18 to 24 (#2226) * feat: upgrade node version to 24 * fix: adding node 24 to overrides.json and using curl instead wget * fix: update to package-lock.json --------- Co-authored-by: Lei Gao <97199248+leigaol@users.noreply.github.com> Co-authored-by: Tai Lai <ctlai95@gmail.com> Co-authored-by: Will Lo <96078566+Will-ShaoHua@users.noreply.github.com> Co-authored-by: BlakeLazarine <blake.l.lazarine@gmail.com> Co-authored-by: Blake Lazarine <blakelaz@amazon.com> Co-authored-by: Richard Li <742829+rli@users.noreply.github.com> Co-authored-by: chungjac <chungjac@amazon.com> Co-authored-by: Dung Dong <dungdong@amazon.com> Co-authored-by: Boyu <bywang@amazon.com> Co-authored-by: Rajanna-Karthik <karsraja@amazon.com> Co-authored-by: tsmithsz <84354541+tsmithsz@users.noreply.github.com> Co-authored-by: Jiatong Li <lijiatong1997@outlook.com> Co-authored-by: Jiatong Li <jiatonli@amazon.com> Co-authored-by: Laxman Reddy <141967714+laileni-aws@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: andrewyuq <89420755+andrewyuq@users.noreply.github.com> Co-authored-by: manodnyab <66754471+manodnyab@users.noreply.github.com> Co-authored-by: atontb <104926752+atonaamz@users.noreply.github.com> Co-authored-by: Shruti Sinha <44882001+shruti0085@users.noreply.github.com> Co-authored-by: aws-toolkit-automation <> Co-authored-by: Richard Li <richali@amazon.com> Co-authored-by: Jason Guo <81202082+jguoamz@users.noreply.github.com> Co-authored-by: mkovelam <mkovelam@amazon.com> Co-authored-by: huawenm <72728725+huawenm@users.noreply.github.com> Co-authored-by: Jayakrishna P <parameja@amazon.com> Co-authored-by: Christopher Christou <39839589+awschristou@users.noreply.github.com> Co-authored-by: Sherry Lu <75588211+XiaoxuanLu@users.noreply.github.com> Co-authored-by: Tai Lai <tlaiamz@amazon.com>

parameja1 and others added 10 commits September 5, 2025 16:57

fix: add unit tests for IAM flow, remove redundant logs added for tes…

d3d6044

…ting

fix: file formatting

c554eb2

fix: rebase from main, clean up redundant logs

15190f0

fix: remove old code refs and debug logs

371d79d

feat: use cw proxy server based on iam auth logic check

b2783a1

fix: todo comment and unused imports

16d71c0

fix: update codewhisperer server to use proper manager based on iam a…

18524fd

…uth logic check

feat: create new codewhisperer server instance with dynamic service m…

fec1526

…anager logic and add unit tests

parameja1 requested a review from a team as a code owner September 9, 2025 16:35

leigaol reviewed Sep 9, 2025

View reviewed changes

sdharani91 reviewed Sep 9, 2025

View reviewed changes

Will-ShaoHua reviewed Sep 9, 2025

View reviewed changes

sdharani91 approved these changes Sep 9, 2025

View reviewed changes

Will-ShaoHua self-requested a review September 9, 2025 20:21

Will-ShaoHua approved these changes Sep 9, 2025

View reviewed changes

fix: address pr comments, fix unit tests as per updated service manag…

559b0fe

…er logic

leigaol approved these changes Sep 9, 2025

View reviewed changes

Merge branch 'main' into feature/smus-iam-inlinesuggestion

7fa59aa

laileni-aws merged commit 8e19f19 into aws:main Sep 9, 2025
6 checks passed

github-actions bot mentioned this pull request Sep 9, 2025

chore(release): release packages from branch main #2225

Merged

Feature to add IAM inline suggestion support in CodeWhispererService #2223

Feature to add IAM inline suggestion support in CodeWhispererService #2223

Uh oh!

Conversation

parameja1 commented Sep 9, 2025

Problem

Solution

License

Uh oh!

codecov-commenter commented Sep 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Will-ShaoHua Sep 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

leigaol left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

codecov-commenter commented Sep 9, 2025 •

edited

Loading

Will-ShaoHua Sep 9, 2025 •

edited

Loading

leigaol left a comment •

edited

Loading