Skip to content

Conversation

l0minous
Copy link
Contributor

@l0minous l0minous commented Jul 14, 2025

Problem

View Execution Details by Execution ARN command currently only supports Standard Executions

Solution

  • Extending Lambda Client to support getFunctionConfiguration (used when viewing executions that have a Lambda state)
  • Extending activation.ts to differentiate between standard and express executions, express executions prompt for a second input box to pass in startTime
  • Calling CloudWatchLogsClient to be used alongside lambda client in ApiHandler
  • Passing startTime through meta tag

Verification

test7


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

@l0minous l0minous requested a review from a team as a code owner July 14, 2025 20:32
Copy link

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

@l0minous l0minous force-pushed the ExpressExecutions branch 2 times, most recently from 5330f3d to 9a1617d Compare July 18, 2025 21:55
@l0minous l0minous force-pushed the ExpressExecutions branch from 9a1617d to fd656bb Compare July 18, 2025 22:59
* @param executionArn The ARN of the execution to display
*/
public async initializePanel(panel: vscode.WebviewPanel, executionArn: string): Promise<void> {
public async initializePanel(panel: vscode.WebviewPanel, executionArn: string, startTime?: string): Promise<void> {
Copy link
Contributor

Choose a reason for hiding this comment

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

nit:
Do we have dispose events for this Webview?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There arent any currently but maybe later if something that needs to be disposed is added

@laileni-aws laileni-aws merged commit 68253c3 into aws:feature/stepfunctions-execution Jul 22, 2025
34 of 37 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