Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Updates ROADMAP.md to reflect the current project state now that Phases 1–3 have shipped, and reframes Phase 4 as the remaining optional backlog.
Changes:
- Adds a “Current status” section marking Phases 1–3 completed and Phase 4 optional/not started.
- Adds outcome summaries for Phases 1–3 to document what shipped.
- Updates the suggested issue order and release strategy to align with the shipped phases and remaining optional work.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
Summary
Updates the roadmap now that Phases 1 through 3 have shipped.
The document previously still read like the first three phases were pending work. That was no longer accurate after the correctness, ergonomics, typing, portability, and maintenance changes were merged. This PR brings the roadmap back in sync with the actual project state.
User impact
There is no runtime behavior change in this PR. The effect is documentation accuracy.
Maintainers and contributors can now see at a glance which phases are complete, what each completed phase delivered, and what work is still intentionally deferred to the optional backlog. That should reduce duplicate planning and make future issue selection clearer.
Root cause
The roadmap was written as a forward-looking plan and was not updated after the Phase 1, Phase 2, and Phase 3 implementation PRs were merged. As a result, it overstated the amount of remaining work and no longer matched the repository history.
Fix
The roadmap now marks Phases 1 through 3 as completed, adds short outcome summaries for each completed phase, and narrows the remaining plan to Phase 4's optional expansion items.
The suggested issue order and release strategy were also rewritten so they reflect the current state instead of describing already-shipped work.
Validation
This is a documentation-only change, so I did not rerun PHPUnit, PHPStan, or PHP-CS-Fixer.