docs(roadmap): error-code consolidation and snapshot store landed - #1031
Conversation
Mark "One error-code scheme instead of four" and "Store each contract snapshot once" as landed (TML-3067/3070/3075/3071, TML-3059/3072), refresh the header counts (3 done / 11 in flight / 20 not started), and add both to Recently landed. MD and HTML updated together. Signed-off-by: willbot <w.a.madden+machine@gmail.com> Signed-off-by: Will Madden <madden@prisma.io>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughWalkthroughThe roadmap documentation reflects July 23 progress, updates v8-RC1 release status counts, and records the unified error-code scheme and content-addressed migration snapshot store as landed in both HTML and Markdown formats. ChangesRoadmap status refresh
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
size-limit report 📦
|
@prisma-next/extension-author-tools
@prisma-next/mongo-runtime
@prisma-next/family-mongo
@prisma-next/sql-runtime
@prisma-next/family-sql
@prisma-next/extension-arktype-json
@prisma-next/middleware-cache
@prisma-next/mongo
@prisma-next/extension-paradedb
@prisma-next/extension-pgvector
@prisma-next/extension-postgis
@prisma-next/postgres
@prisma-next/sql-orm-client
@prisma-next/sqlite
@prisma-next/extension-supabase
@prisma-next/target-mongo
@prisma-next/adapter-mongo
@prisma-next/driver-mongo
@prisma-next/contract
@prisma-next/utils
@prisma-next/config
@prisma-next/errors
@prisma-next/framework-components
@prisma-next/operations
@prisma-next/ts-render
@prisma-next/contract-authoring
@prisma-next/ids
@prisma-next/psl-parser
@prisma-next/psl-printer
@prisma-next/cli
@prisma-next/cli-telemetry
@prisma-next/config-loader
@prisma-next/emitter
@prisma-next/language-server
@prisma-next/migration-tools
prisma-next
@prisma-next/vite-plugin-contract-emit
@prisma-next/mongo-codec
@prisma-next/mongo-contract
@prisma-next/mongo-value
@prisma-next/mongo-contract-psl
@prisma-next/mongo-contract-ts
@prisma-next/mongo-emitter
@prisma-next/mongo-schema-ir
@prisma-next/mongo-query-ast
@prisma-next/mongo-orm
@prisma-next/mongo-query-builder
@prisma-next/mongo-lowering
@prisma-next/mongo-wire
@prisma-next/sql-contract
@prisma-next/sql-errors
@prisma-next/sql-operations
@prisma-next/sql-schema-ir
@prisma-next/sql-contract-psl
@prisma-next/sql-contract-ts
@prisma-next/sql-contract-emitter
@prisma-next/sql-lane-query-builder
@prisma-next/sql-relational-core
@prisma-next/sql-builder
@prisma-next/target-postgres
@prisma-next/target-sqlite
@prisma-next/adapter-postgres
@prisma-next/adapter-sqlite
@prisma-next/driver-postgres
@prisma-next/driver-sqlite
commit: |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@ROADMAP.html`:
- Around line 416-418: Update the snapshot-store wording in ROADMAP.html lines
416-418 and ROADMAP.md lines 130-132 to describe one content-addressed snapshot
entry per distinct contract, replacing “one file per distinct contract.” Keep
the surrounding migration and hashing details unchanged.
- Line 276: Reconcile the July 22 minimum-supported-Postgres gate in the
release-gating content: update ROADMAP.html lines 276-276 and ROADMAP.md lines
18-18 consistently by either removing the gate if resolved or explicitly marking
it overdue/currently blocking; preserve the July 24 polymorphism and
scoreboard-freeze details.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yml
Review profile: CHILL
Plan: Pro Plus
Run ID: af8ae74f-ca0a-4621-9b7e-0dfd92853363
📒 Files selected for processing (2)
ROADMAP.htmlROADMAP.md
Address review: the min-Postgres decision (July 22) is now past its date on a doc updated July 23, so present it as overdue-and-still-blocking rather than an upcoming gate; describe the snapshot store as one content-addressed entry per distinct contract (it holds contract.json + contract.d.ts per hash), not one file. Signed-off-by: willbot <w.a.madden+machine@gmail.com> Signed-off-by: Will Madden <madden@prisma.io>
Linked issue
Progress update to ROADMAP.md/ROADMAP.html — no ticket; reflects merged work from TML-3067, TML-3070, TML-3075, TML-3071, TML-3059, TML-3072.
At a glance
Section 3 flips two items to ✅ Landed: One error-code scheme instead of four (all published errors are dotted structural envelopes; 221-code reference page with a per-PR CI completeness check) and Store each contract snapshot once (content-addressed
migrations/snapshots/store). Header counts move to 3 done / 11 in flight / 20 not started; both items join Recently landed. MD and HTML updated together; HTML render verified.Testing performed
Checklist
git commit -s) per the DCO.🤖 Generated with Claude Code
Summary by CodeRabbit
NAMESPACE.SUBCODEscheme and the included 221-code reference with automated validation.