fix: remove temp migrations and prepare stack to be merged #230
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
| Name | Type |
|---|---|
| @pgflow/dsl | Minor |
| @pgflow/core | Minor |
| pgflow | Minor |
| @pgflow/client | Minor |
| @pgflow/edge-worker | Minor |
| @pgflow/example-flows | Minor |
| @pgflow/website | Minor |
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
This was referenced Oct 6, 2025
jumski
changed the title
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
pkgs/core/supabase/migrations/20251006063001_pgflow_add_map_step_type.sql
Show resolved
Hide resolved
|
View your CI Pipeline Execution ↗ for commit 8dc0d99
☁️ Nx Cloud last updated this comment at |
jumski
force-pushed
the
10-06-chore-consolidate-migrations-and-finalize
branch
from
672447d to
d9bdc7c
Compare
jumski
force-pushed
the
09-26-docs-add-map-array-docs
branch
from
90350c4 to
a2f5600
Compare
jumski
force-pushed
the
10-06-chore-consolidate-migrations-and-finalize
branch
from
d9bdc7c to
8084414
Compare
… improve failure handling - Renamed existing changesets for clarity - Implemented map step type infrastructure in SQL core - Added .map() method to Flow DSL with compile-time validation and type safety - Enhanced failure handling to archive all pending messages on run failure - Fixed type contract violations to fail immediately without retries - Updated documentation and created detailed changeset - Removed obsolete plan files and consolidated migration strategy - Improved robustness and observability for map steps and failure scenarios
jumski
force-pushed
the
10-06-chore-consolidate-migrations-and-finalize
branch
from
8084414 to
8dc0d99
Compare
🔍 Preview Deployment: Website✅ Deployment successful! 🔗 Preview URL: https://pr-230.pgflow.pages.dev 📝 Details:
_Last updated: _ |
🔍 Preview Deployment: Playground❌ Deployment failed! Please check the workflow logs for details. |
Merge activity
|
graphite-app bot
pushed a commit
that referenced
this pull request
Oct 7, 2025
# Remove planning documents now that map infrastructure is complete This PR removes all planning documents that were used during the map infrastructure development. The map feature is now complete and fully integrated into the codebase, making these planning documents obsolete. Specifically: - Removed [PLAN.md](http://PLAN.md) which tracked the map infrastructure implementation progress - Removed PLAN_partial_completion.md which outlined failure handling strategies - Removed PLAN_step_output.md which detailed performance optimization plans - Removed PLAN_race_condition_testing.md which described testing approaches - Consolidated all temporary migrations into a single clean migration These documents served as implementation guides during development but are no longer needed now that the features have been fully implemented. The code itself now serves as the source of truth for how map steps, failure handling, and output aggregation work.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Remove planning documents now that map infrastructure is complete
This PR removes all planning documents that were used during the map infrastructure development. The map feature is now complete and fully integrated into the codebase, making these planning documents obsolete.
Specifically:
These documents served as implementation guides during development but are no longer needed now that the features have been fully implemented. The code itself now serves as the source of truth for how map steps, failure handling, and output aggregation work.