Skip to content

Conversation

@roger-zhangg
Copy link
Member

@roger-zhangg roger-zhangg commented Sep 17, 2025

Problem

Few UI feedback:

  • Payload is not intuitive
  • Open Handler should be a button
    Inconsistent layer behavior

UI before

image

Solution

This PR refactors the Lambda Remote Invoke UI to follow VSCode's native design patterns, optimizes the codebase by removing unused components, and enhances the payload management experience with integrated quickpick functionality for remote test events.

image

Update global layer version to 2

🎨 UI/UX Improvements

  • Redesigned Remote Debugging section to follow VSCode Settings UI pattern

    • Title with inline "Remove Debug Setup" button and timer info
    • Checkbox aligned with description on same line for better visual hierarchy
  • Redesigned Local Root Path section with VSCode Settings style

    • Added "Open Handler" button with disabled state and helpful tooltips
    • Shortened button labels to "Browse" and "Download" for cleaner interface
    • Enhanced descriptions with bold text to guide user actions
  • Completely redesigned Payload section

    • Removed radio buttons for cleaner, unified interface
    • Added button group: "Load sample event", "Load local file", "Load remote event", "Save as remote event"
    • Improved textarea with monospace font and better sizing

🚀 New Features

  • VSCode quickpick integration for remote test events

    • selectRemoteTestEvent(): Shows native quickpick for loading saved events
    • saveRemoteTestEvent(): Quickpick with options to create new or overwrite existing events
    • Input validation for event names
    • Confirmation dialogs for overwriting existing events
  • Enhanced error handling

    • Gracefully handles "lambda-testevent-schemas registry not found" error
    • Returns empty array instead of throwing when no test events exist
    • Helpful user messages when no events are found
  • Force flag support for overwriting existing test events

    • Added force parameter to SamCliRemoteTestEventsParameters
    • Automatically uses --force when user confirms overwrite


  • 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.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@roger-zhangg roger-zhangg requested a review from a team as a code owner September 17, 2025 19:14
@amazon-inspector-ohio
Copy link

⏳ I'm reviewing this pull request for security vulnerabilities and code quality issues. I'll provide an update when I'm done

@amazon-inspector-ohio
Copy link

✅ I finished the code review, and didn't find any security or code quality issues.

Copy link

@bnusunny bnusunny left a comment

Choose a reason for hiding this comment

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

LGTM

@ashishrp-aws ashishrp-aws merged commit 42a862a into aws:master Oct 1, 2025
42 of 46 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.

5 participants