-
Notifications
You must be signed in to change notification settings - Fork 94
[DevTools] PR2: Backend Services, Console Viewer, and Resources #2879
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
megha-narayanan
merged 81 commits into
aws-amplify:feature/dev-tools
from
megha-narayanan:pr-2-backend-services
Jul 18, 2025
Merged
Changes from 3 commits
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
e994339
PR1: Dependencies, Basic React App Facade, and Barebones Devtools Com…
02a7289
made timeout longer AND temporary workaround for license issues
776fd8a
PR2: Basic Sandbox Manager and Resource Console PRELIM
7bd6b1b
Fixing Lint and Tests
093c130
fixed lint issues and tests
c9d0917
updating for structure changes, printer
ce447ae
merging with pr-1 as base
8ff3baf
update API
3085faf
updating package_lock (based on pr_1)
d9d10c9
update default_packages
9955896
added changeset
1a0849a
fix lint
ce3a461
reconcile status types
134e83a
removed profile from start sandbox modal as it can't do anything
52d92e0
fixed loading of start sandbox when modal dismissed
77b7f60
removed aws-sdk upgrade, removed dependencies in root package
87fa132
got rid of cleanAnsiCodes (and duplicate), added stripAnsi instead, a…
740271c
moved server starting from portchecker to devtools command
68c9ab7
fix lint
73f32e0
pull out normalizeCDKConstructPath utility and refactor usage in CfnD…
dc33887
fix api for cli-core (added normalizeCDK constructPath)
4f2ed23
Refactor socket communication and service structure
11c9684
make AWS console links visible, remove unused log note
fa14390
Update sandbox state management, add initialization error handling
0d1c15f
get resources if sandbox is stopped
e5c9925
fix API for initialization error
61e3a47
inject dependencies with sandbox devtools command factory
1e47844
making socket handlers use resource service (eliminating duplicate code)
732c99d
Implement socket event constants instead of strings
3f95603
add changeset
126366d
add socket_handler tests
152f5c4
add unit tests for resource console functions
b3455b3
refactor ResourceService to use backendClient and RegionFetcher throu…
724ff1a
add resource_service tests
740e08c
update api for regionfetcher
050928b
move shutdown_service import in socket handlers
7d12cb5
added auto-scroll disable if you scroll up in console logs
e289446
enhance friendly name generation and add template metadata fetching
97c3b99
changeset
e53ec95
fixed search bar disappearing issue.
0fdfae9
refactor shutdown process to remove unnecessary client notifications …
66e7255
api update for metadata
4708052
updated deployed_resources_enumerator test to include metadata
af2e6ac
Implement logging functionality without global printer override, fixe…
33570fd
manually tested additional resources, added them to tests
b96a7d1
more friendly name updates + testing
72b1a55
updated resource service test for new friendly name
ead3ee3
Add random delay on initial load to prevent thundering herd problem; …
41ba348
replace local type definitions with centralized SandboxStatus type fr…
219e3cd
api update for sandbox status
1671e6d
refactored sandbox devtools command to make it more testable, and fix…
88ed395
fixed SandboxStatus updates
75e5b1d
misc comment updates
ede0865
remove unused index.css file (which was added for PR1, and visually d…
df5e7b5
Remove useEffect in ConsoleViewer
megha-narayanan a47c32d
Minor output and naming changes in sandbox_devtools command
megha-narayanan 6c2a488
finish fixing renaming in sandbox_devtools_command
9c49385
remove aws-sdk upgrades
1089978
remove deprecated private method in cfn_deployment_progress_logger
9153373
remove unneeded variable in Console_Viewer
417b73e
Made confirmation dialogues a custom modal instead of browser window
24c704c
Fixed potential double space in Header status
76b21a3
combined useEffects in App.tsx
64c0c4d
remove resourceConfigChanged
109f154
added error for failed emit
b8fc07b
added unit tests for state management
c9d2288
Update packages/deployed-backend-client/src/deployed-backend-client/d…
megha-narayanan 9416275
get rid of unused fields in StartSandboxModal, fix internal typing fo…
9b8693a
Merge branch 'pr-2-backend-services' of https://github.com/megha-nara…
8db171f
expand issue comment
b271bad
fix error in deployed_resources_enumerator
f569654
throw error on null socket
1004c78
inject logger factory as dependency
5786485
header status text fix
4861a0d
fix lint
1c6d9c0
fixed unsubscribe naming.
e5d1da0
handle undefined response on describestacks command
422058c
rate limiting comment
e3c1c9b
header minor fixes
aaa7a5a
Update packages/cli/src/commands/sandbox/sandbox-devtools/react-app/s…
megha-narayanan f55e43c
change fn name
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be in a follow up PR but I believe this useEffect can be removed since this component rerenders when sandboxStatus changes (since it is a prop) and you already initialize the state as
false
in line 35.