-
Notifications
You must be signed in to change notification settings - Fork 15
refactor: simplify step input structure by removing run key and using context.flowInput #558
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
base: 12-23-migrate_48_files_from_input.run_to_asymmetric_handler_signatures
Are you sure you want to change the base?
Conversation
|
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
View your CI Pipeline Execution ↗ for commit 440342c
☁️ Nx Cloud last updated this comment at |
pkgs/core/supabase/migrations/20251223110504_pgflow_add_flow_input_column.sql
Show resolved
Hide resolved
1ed5f8d to
6090a08
Compare
7d18e09 to
697f365
Compare
697f365 to
a9a1da1
Compare
a9a1da1 to
440342c
Compare
🔍 Preview Deployment: Website✅ Deployment successful! 🔗 Preview URL: https://pr-558.pgflow.pages.dev 📝 Details:
_Last updated: _ |

Simplify Step Input Structure for Better Context Access
This PR changes how inputs are passed to steps in a workflow:
Removed the
runkey from step inputs:runkeycontext.flowInputinsteadSimplified input structure:
{}Updated documentation comments to reflect the new input structure:
flow_inputis available separately in the contextUpdated tests to verify the new input structure works correctly with:
This change makes the input structure cleaner and more intuitive while ensuring all steps still have access to the original flow input when needed.