Skip to content

docs(skills): drop the RLS-testing guidance — testing a Supabase project is not our surface - #1002

Merged
wmadden merged 1 commit into
mainfrom
chore-drop-supabase-testing-guidance
Jul 17, 2026
Merged

docs(skills): drop the RLS-testing guidance — testing a Supabase project is not our surface#1002
wmadden merged 1 commit into
mainfrom
chore-drop-supabase-testing-guidance

Conversation

@wmadden-electric

@wmadden-electric wmadden-electric commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Decision

How users test their Supabase project is Supabase's concern, not Prisma Next's. The prisma-next-supabase skill had grown a full "Testing your RLS policies" workflow — a pgTAP tutorial with a worked SQL example — and the 0.15→0.16 upgrade recipes and extension README echoed the same supabase test db endorsement. None of that is our product surface, nothing in this repo executes any of it (documented-but-unexecuted guidance is exactly the defect class TML-3035 was about), and the endorsement traces back to a forwarded external-agent suggestion that was mislabeled as an operator decision during shaping — not to anyone deciding "our skills teach pgTAP."

What changed

  • skills/prisma-next-supabase/SKILL.md — the "Workflow — Testing your RLS policies" section is deleted. The skill now covers only what is ours: policy authoring, migrations, role binding, JWT config, connections, and the one grant our admin-read feature needs. Also fixes a stale line from TML-3035: a first Supabase project works first-try — RLS migrations, verify, migrate, JWKS, grants #997: the example ships acceptance tests only (the hermetic suites moved into the extension package).
  • Both 0.15→0.16 upgrade recipes — the bootstrapSupabaseShim removal entries keep their real remediation (delete the dead import; keep hermetic helpers package-internal) and drop the trailing "use supabase test db (pgTAP)" advice.
  • packages/3-extensions/supabase/README.md — drops the closing "user-facing RLS testing is Supabase's own tooling" sentence.

Repo-wide sweep confirms zero remaining pgTAP / supabase test db mentions. Prose only — no code paths touched; pnpm lint:skills green.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated Supabase extension and upgrade guidance to remove references to Supabase CLI and pgTAP testing workflows.
    • Simplified instructions for removing deprecated Supabase test utilities during the 0.15-to-0.16 upgrade.
    • Removed the dedicated RLS testing workflow section and streamlined related reference descriptions.

… is not our surface

The Supabase skill taught pgTAP syntax and a supabase test db workflow;
the upgrade recipes and extension README echoed it. None of it is
Prisma Next product surface, nothing in the repo runs it, and the
endorsement traces to a forwarded suggestion, not a product decision.
The skill now covers only what is ours: policy authoring, migrations,
role binding, JWT config, connections, grants.

Signed-off-by: willbot <w.a.madden+machine@gmail.com>
Signed-off-by: Will Madden <madden@prisma.io>
@wmadden-electric
wmadden-electric requested a review from a team as a code owner July 17, 2026 09:55
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: aa939a00-10cc-4be7-b412-17acdc180f56

📥 Commits

Reviewing files that changed from the base of the PR and between 0df4f7a and cf2a100.

📒 Files selected for processing (4)
  • packages/3-extensions/supabase/README.md
  • skills/extension-author/prisma-next-extension-upgrade/upgrades/0.15-to-0.16/instructions.md
  • skills/prisma-next-supabase/SKILL.md
  • skills/upgrade/prisma-next-upgrade/upgrades/0.15-to-0.16/instructions.md

📝 Walkthrough

Walkthrough

Supabase documentation and 0.15→0.16 upgrade instructions remove Supabase CLI-based RLS testing guidance, including local-stack and pgTAP examples. The Supabase reference-files description is also revised.

Changes

Supabase testing documentation

Layer / File(s) Summary
Align Supabase testing guidance
packages/3-extensions/supabase/README.md, skills/.../instructions.md, skills/prisma-next-supabase/SKILL.md
Removes Supabase CLI and pgTAP testing references from Supabase responsibilities, workflows, and upgrade guidance, and updates the examples/supabase reference description.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers: wmadden

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the PR’s main doc change: removing Supabase RLS-testing guidance from Prisma Next docs.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore-drop-supabase-testing-guidance

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@wmadden
wmadden enabled auto-merge July 17, 2026 09:57
@github-actions

Copy link
Copy Markdown

size-limit report 📦

Path Size
postgres / no-emit 158.95 KB (0%)
postgres / emit 132.46 KB (0%)
mongo / no-emit 98.71 KB (0%)
mongo / emit 89.43 KB (0%)
cf-worker / no-emit 185.12 KB (0%)
cf-worker / emit 155.88 KB (0%)

@pkg-pr-new

pkg-pr-new Bot commented Jul 17, 2026

Copy link
Copy Markdown

Open in StackBlitz

@prisma-next/extension-author-tools

npm i https://pkg.pr.new/@prisma-next/extension-author-tools@1002

@prisma-next/mongo-runtime

npm i https://pkg.pr.new/@prisma-next/mongo-runtime@1002

@prisma-next/family-mongo

npm i https://pkg.pr.new/@prisma-next/family-mongo@1002

@prisma-next/sql-runtime

npm i https://pkg.pr.new/@prisma-next/sql-runtime@1002

@prisma-next/family-sql

npm i https://pkg.pr.new/@prisma-next/family-sql@1002

@prisma-next/extension-arktype-json

npm i https://pkg.pr.new/@prisma-next/extension-arktype-json@1002

@prisma-next/middleware-cache

npm i https://pkg.pr.new/@prisma-next/middleware-cache@1002

@prisma-next/mongo

npm i https://pkg.pr.new/@prisma-next/mongo@1002

@prisma-next/extension-paradedb

npm i https://pkg.pr.new/@prisma-next/extension-paradedb@1002

@prisma-next/extension-pgvector

npm i https://pkg.pr.new/@prisma-next/extension-pgvector@1002

@prisma-next/extension-postgis

npm i https://pkg.pr.new/@prisma-next/extension-postgis@1002

@prisma-next/postgres

npm i https://pkg.pr.new/@prisma-next/postgres@1002

@prisma-next/sql-orm-client

npm i https://pkg.pr.new/@prisma-next/sql-orm-client@1002

@prisma-next/sqlite

npm i https://pkg.pr.new/@prisma-next/sqlite@1002

@prisma-next/extension-supabase

npm i https://pkg.pr.new/@prisma-next/extension-supabase@1002

@prisma-next/target-mongo

npm i https://pkg.pr.new/@prisma-next/target-mongo@1002

@prisma-next/adapter-mongo

npm i https://pkg.pr.new/@prisma-next/adapter-mongo@1002

@prisma-next/driver-mongo

npm i https://pkg.pr.new/@prisma-next/driver-mongo@1002

@prisma-next/contract

npm i https://pkg.pr.new/@prisma-next/contract@1002

@prisma-next/utils

npm i https://pkg.pr.new/@prisma-next/utils@1002

@prisma-next/config

npm i https://pkg.pr.new/@prisma-next/config@1002

@prisma-next/errors

npm i https://pkg.pr.new/@prisma-next/errors@1002

@prisma-next/framework-components

npm i https://pkg.pr.new/@prisma-next/framework-components@1002

@prisma-next/operations

npm i https://pkg.pr.new/@prisma-next/operations@1002

@prisma-next/ts-render

npm i https://pkg.pr.new/@prisma-next/ts-render@1002

@prisma-next/contract-authoring

npm i https://pkg.pr.new/@prisma-next/contract-authoring@1002

@prisma-next/ids

npm i https://pkg.pr.new/@prisma-next/ids@1002

@prisma-next/psl-parser

npm i https://pkg.pr.new/@prisma-next/psl-parser@1002

@prisma-next/psl-printer

npm i https://pkg.pr.new/@prisma-next/psl-printer@1002

@prisma-next/cli

npm i https://pkg.pr.new/@prisma-next/cli@1002

@prisma-next/cli-telemetry

npm i https://pkg.pr.new/@prisma-next/cli-telemetry@1002

@prisma-next/config-loader

npm i https://pkg.pr.new/@prisma-next/config-loader@1002

@prisma-next/emitter

npm i https://pkg.pr.new/@prisma-next/emitter@1002

@prisma-next/language-server

npm i https://pkg.pr.new/@prisma-next/language-server@1002

@prisma-next/migration-tools

npm i https://pkg.pr.new/@prisma-next/migration-tools@1002

prisma-next

npm i https://pkg.pr.new/prisma-next@1002

@prisma-next/vite-plugin-contract-emit

npm i https://pkg.pr.new/@prisma-next/vite-plugin-contract-emit@1002

@prisma-next/mongo-codec

npm i https://pkg.pr.new/@prisma-next/mongo-codec@1002

@prisma-next/mongo-contract

npm i https://pkg.pr.new/@prisma-next/mongo-contract@1002

@prisma-next/mongo-value

npm i https://pkg.pr.new/@prisma-next/mongo-value@1002

@prisma-next/mongo-contract-psl

npm i https://pkg.pr.new/@prisma-next/mongo-contract-psl@1002

@prisma-next/mongo-contract-ts

npm i https://pkg.pr.new/@prisma-next/mongo-contract-ts@1002

@prisma-next/mongo-emitter

npm i https://pkg.pr.new/@prisma-next/mongo-emitter@1002

@prisma-next/mongo-schema-ir

npm i https://pkg.pr.new/@prisma-next/mongo-schema-ir@1002

@prisma-next/mongo-query-ast

npm i https://pkg.pr.new/@prisma-next/mongo-query-ast@1002

@prisma-next/mongo-orm

npm i https://pkg.pr.new/@prisma-next/mongo-orm@1002

@prisma-next/mongo-query-builder

npm i https://pkg.pr.new/@prisma-next/mongo-query-builder@1002

@prisma-next/mongo-lowering

npm i https://pkg.pr.new/@prisma-next/mongo-lowering@1002

@prisma-next/mongo-wire

npm i https://pkg.pr.new/@prisma-next/mongo-wire@1002

@prisma-next/sql-contract

npm i https://pkg.pr.new/@prisma-next/sql-contract@1002

@prisma-next/sql-errors

npm i https://pkg.pr.new/@prisma-next/sql-errors@1002

@prisma-next/sql-operations

npm i https://pkg.pr.new/@prisma-next/sql-operations@1002

@prisma-next/sql-schema-ir

npm i https://pkg.pr.new/@prisma-next/sql-schema-ir@1002

@prisma-next/sql-contract-psl

npm i https://pkg.pr.new/@prisma-next/sql-contract-psl@1002

@prisma-next/sql-contract-ts

npm i https://pkg.pr.new/@prisma-next/sql-contract-ts@1002

@prisma-next/sql-contract-emitter

npm i https://pkg.pr.new/@prisma-next/sql-contract-emitter@1002

@prisma-next/sql-lane-query-builder

npm i https://pkg.pr.new/@prisma-next/sql-lane-query-builder@1002

@prisma-next/sql-relational-core

npm i https://pkg.pr.new/@prisma-next/sql-relational-core@1002

@prisma-next/sql-builder

npm i https://pkg.pr.new/@prisma-next/sql-builder@1002

@prisma-next/target-postgres

npm i https://pkg.pr.new/@prisma-next/target-postgres@1002

@prisma-next/target-sqlite

npm i https://pkg.pr.new/@prisma-next/target-sqlite@1002

@prisma-next/adapter-postgres

npm i https://pkg.pr.new/@prisma-next/adapter-postgres@1002

@prisma-next/adapter-sqlite

npm i https://pkg.pr.new/@prisma-next/adapter-sqlite@1002

@prisma-next/driver-postgres

npm i https://pkg.pr.new/@prisma-next/driver-postgres@1002

@prisma-next/driver-sqlite

npm i https://pkg.pr.new/@prisma-next/driver-sqlite@1002

commit: cf2a100

@wmadden
wmadden added this pull request to the merge queue Jul 17, 2026
Merged via the queue into main with commit 383163d Jul 17, 2026
22 checks passed
@wmadden
wmadden deleted the chore-drop-supabase-testing-guidance branch July 17, 2026 10:22
@coderabbitai coderabbitai Bot mentioned this pull request Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants