feat(smus): Merge Unified Studio feature enhancements#8311
Merged
Will-ShaoHua merged 141 commits intoaws:feature/smusfrom Nov 18, 2025
Merged
feat(smus): Merge Unified Studio feature enhancements#8311Will-ShaoHua merged 141 commits intoaws:feature/smusfrom
Will-ShaoHua merged 141 commits intoaws:feature/smusfrom
Conversation
fix(smus): dispose smus auth provider upon cred expiry
…ic error (aws#2290) ## Problem - User dont see connection type specific error - SMUS iam creds does not stored with correct conn type ## Solution - Add correct conn type when storing creds in memory - Surface conn type specific error when cred expires --- - Treat all work as PUBLIC. Private `feature/x` branches will not be squash-merged at release time. - Your code changes must meet the guidelines in [CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines). - License: I confirm that my contribution is made under the terms of the Apache 2.0 license.
Merge staging into feature/smus-m2
**Description** Updating some text to maken things more explicit and clear for users for the login mode **Motivation** Clarity for users - Treat all work as PUBLIC. Private `feature/x` branches will not be squash-merged at release time. - Your code changes must meet the guidelines in [CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines). - License: I confirm that my contribution is made under the terms of the Apache 2.0 license. --------- Co-authored-by: Bhargava Varadharajan <vabharga@amazon.com>
…hide error on screen (aws#2301) ## Problem - when creds are expired, refresh project does not cause the auth node show cred expiry - When creds are expired the error window better match the error on auth node ## Solution - on creds expiry refresh project triggers invalid conn - resurface cred expiry error window --- - Treat all work as PUBLIC. Private `feature/x` branches will not be squash-merged at release time. - Your code changes must meet the guidelines in [CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines). - License: I confirm that my contribution is made under the terms of the Apache 2.0 license.
## Problem
- manual addition of SMUS to logger
## Solution
- Added 'smus' to LogTopic type in logger.ts
- Updated all SMUS-related files to use getLogger('smus') instead of
getLogger()
- Removed manual 'SMUS:' prefixes from all log messages
- Logger now automatically prepends 'smus:' to all SMUS log messages
verified Changes
```
2025-11-17 09:12:55.014 [info] auth: Handling validation error of connection: 77fd26fb-ec96-4e54-8171-703530254b3f
2025-11-17 09:12:55.014 [info] auth: Updating connection state of 77fd26fb-ec96-4e54-8171-703530254b3f to invalid
2025-11-17 09:12:55.023 [info] smus: SMUS Connection: Clearing all cached clients
2025-11-17 09:12:55.029 [info] smus: Not in SageMaker Unified Studio remote environment, skipping user activity monitoring
2025-11-17 09:12:55.033 [info] secondaryAuth: connectionId=77fd26fb-ec96-4e54-8171-703530254b3f, connectionStatus=invalid
```
---
- Treat all work as PUBLIC. Private `feature/x` branches will not be
squash-merged at release time.
- Your code changes must meet the guidelines in
[CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines).
- License: I confirm that my contribution is made under the terms of the
Apache 2.0 license.
**Description** Change adds auth mode as a telemetry property. **Motivation** This will help distinguish traffic across the two supported auth modes. **Testing Done** Unit tests, tested locally. Shared results with team.
telemetry(smus): add authmode to telemetry
vpbhargav
changed the title
ashishrp-aws
approved these changes
Nov 18, 2025
laileni-aws
changed the title
Will-ShaoHua
approved these changes
Nov 18, 2025
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
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.
Problem
Merging some staged feature enhancements for SMUS support in AWS Toolkit.
Solution
feature/xbranches will not be squash-merged at release time.