Repository navigation
Project shaping: remove dbgenerated(...) from Prisma 8 (spec, plan, slice specs) - #30327
Merged
Merged
Conversation
Contributor
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (6)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
wmadden-electric
enabled auto-merge
September 17, 2026 07:38
4 of 5 tasks
wmadden
approved these changes
Sep 17, 2026
Project spec, plan, deferred decisions, and three slice specs: the sql tagged literal for raw SQL defaults (ADR 129), codec-owned PSL literals (the PSL half of ADR 184), and the removal itself with the Supabase contract regeneration and upgrade instruction. Slices A and B are independent and run in parallel; C depends on both. Signed-off-by: willbot <w.a.madden+machine@gmail.com> Signed-off-by: Will Madden <madden@prisma.io> Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…ify comparison plan for raw defaults Signed-off-by: willbot <w.a.madden+machine@gmail.com> Signed-off-by: Will Madden <madden@prisma.io> Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…t ADR 129 does and does not cover Signed-off-by: willbot <w.a.madden+machine@gmail.com> Signed-off-by: Will Madden <madden@prisma.io> Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Signed-off-by: willbot <w.a.madden+machine@gmail.com> Signed-off-by: Will Madden <madden@prisma.io> Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
wmadden
disabled auto-merge
September 17, 2026 07:39
wmadden
force-pushed
the
remove-dbgenerated-plan
branch
from
September 17, 2026 07:39
67f6477 to
aa5603e
Compare
RyanGarber
pushed a commit
to RyanGarber/prisma-orm
that referenced
this pull request
Sep 20, 2026
…lice specs) (prisma#30327) ## Linked issue n/a — project shaping artifacts. Follow-up PRs: slice A prisma#30325, slice B prisma#30324; slice C follows both. ## Summary Adds the project folder for removing `@default(dbgenerated("..."))` from Prisma 8: the project spec with its decisions, the plan splitting the work into three slices, the deferred-decision list, and the three slice specs. Nothing outside `projects/remove-dbgenerated/` changes. Merging this first lets both slice PRs rebase to pure code diffs instead of each replaying the same docs commit. ## Testing performed - n/a — docs only; `pnpm lint:docs` covers the markdown. ## Skill update n/a — internal only. ## Checklist - [x] All commits are signed off (`git commit -s`) per the [DCO](../CONTRIBUTING.md#developer-certificate-of-origin-dco). - [x] I read [CONTRIBUTING.md](../CONTRIBUTING.md) and the change is scoped to one logical concern. - [x] Tests are updated (n/a, doc-only). - [ ] The PR title is in `TML-NNNN: <sentence-case title>` form (no ticket; the project spec records that Linear is not yet created). - [x] The **Skill update** section above is filled in. ## Notes for the reviewer The folder is transient: the project's close-out step moves long-lived content to `docs/` and deletes it. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Signed-off-by: willbot <w.a.madden+machine@gmail.com> Signed-off-by: Will Madden <madden@prisma.io> Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
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
2 participants
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.
Linked issue
n/a — project shaping artifacts. Follow-up PRs: slice A #30325, slice B #30324; slice C follows both.
Summary
Adds the project folder for removing
@default(dbgenerated("..."))from Prisma 8: the project spec with its decisions, the plan splitting the work into three slices, the deferred-decision list, and the three slice specs. Nothing outsideprojects/remove-dbgenerated/changes. Merging this first lets both slice PRs rebase to pure code diffs instead of each replaying the same docs commit.Testing performed
pnpm lint:docscovers the markdown.Skill update
n/a — internal only.
Checklist
git commit -s) per the DCO.TML-NNNN: <sentence-case title>form (no ticket; the project spec records that Linear is not yet created).Notes for the reviewer
The folder is transient: the project's close-out step moves long-lived content to
docs/and deletes it.🤖 Generated with Claude Code