feat(lambda): Add checkbox in local invoke webview for debugger

[mbfreder]

Problem

Solution

Add checkbox in local invoke webview so customers can choose to locally invoke with or without debugger

Note:

No tests added since the functionality is already tested here:

aws-toolkit-vscode/packages/core/src/test/shared/sam/debugger/samDebugConfigProvider.test.ts

Line 111 in a7732b9

async function assertEqualNoDebugTemplateTarget(

---

• 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.

[github-actions]

• This pull request modifies code in src/* but no tests were added/updated.
  • Confirm whether tests should be added or ensure the PR description explains why tests are not required.
• This pull request implements a feat or fix, so it must include a changelog entry (unless the fix is for an unreleased feature). Review the changelog guidelines.
  • Note: beta or "experiment" features that have active users should announce fixes in the changelog.
  • If this is not a feature or fix, use an appropriate type from the title guidelines. For example, telemetry-only changes should use the telemetry type.

[justinmk3]

? This is already supported by the existing (inherited from vscode) noDebug field:

aws-toolkit-vscode/packages/core/src/shared/sam/debugger/awsSamDebugger.ts

Lines 179 to 183 in 70ba83f

	//
	// Debug properties (when user runs with debugging enabled).
	//
	/** vscode implicit field, set if user invokes "Run (Start Without Debugging)". */
	noDebug?: boolean

[justinmk3]

This is already supported when the user invokes a sam launch config via Debug: Start Without Debugging :

However, it may still be desired to make that more obvious via this new checkbox.

[mbfreder]

Test failures are unrelated to the changes.

[justinmk3]

this probably isn't needed either, because launchConfig on line 33 should already have noDebug field.

oh but if this is "binding" to the vue, maybe this is needed easier/clearer.

[mbfreder]

Yes, it is binding to the vue.