Skip to content

Refactor Lambda workflows: extract filesystem step, improve error handling - #4

Merged
beatgeek merged 2 commits into
filesystem-supportfrom
copilot/sub-pr-3
Feb 2, 2026
Merged

Refactor Lambda workflows: extract filesystem step, improve error handling#4
beatgeek merged 2 commits into
filesystem-supportfrom
copilot/sub-pr-3

Conversation

Copilot AI commented Feb 2, 2026

Copy link
Copy Markdown

Summary

  • Addresses PR review feedback: extracts filesystem creation into dedicated step, enhances API error reporting, fixes deletion message timing

Changes

  • deploy-lambda.yml: Split "Launch instance" into "Create or select filesystem" (new) + "Launch instance" (refactored)
    • Filesystem ID now passed via step outputs: steps.filesystem.outputs.filesystem_id
    • Error handling extracts API error from .error // .errors[0].message // .message with fallback to raw response
  • teardown-lambda.yml: Deletion note only shows on curl failure (|| echo)

Testing

  • Not run (reason: workflow changes validated via YAML syntax check)

Review focus

  • Rust runtime paths
  • Helion sidecar/worker integration

Rollout / Notes

  • No functional changes to deployed resources, only workflow organization and diagnostics

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…ncerns

Co-authored-by: beatgeek <2343225+beatgeek@users.noreply.github.com>
Copilot AI changed the title [WIP] Add filesystem support and teardown workflow to Lambda Refactor Lambda workflows: extract filesystem step, improve error handling Feb 2, 2026
Copilot AI requested a review from beatgeek February 2, 2026 00:47
@beatgeek
beatgeek marked this pull request as ready for review February 2, 2026 16:07
@beatgeek
beatgeek merged commit bd4af54 into filesystem-support Feb 2, 2026
3 checks passed
@beatgeek
beatgeek deleted the copilot/sub-pr-3 branch February 2, 2026 16:07
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.

2 participants