Skip to content

Commit adf5243

Browse files
committed
docs(cli): correct stale --prisma copy now that prisma-next v3 has shipped
Dan's review on #691: the `stash eql migration --prisma` copy said the flag "ships with prisma-next EQL v3 support" / "not available yet" — but that support has shipped (#655/#683/#685), and Prisma Next installs EQL v3 through its OWN migration system (`prisma-next migration apply`), not through this command. Reframed all five spots to current reality (not "coming soon", but "use prisma-next migration apply"): the user-facing message + its doc comment, the `eqlMigrationCommand` code comment, the `--prisma` flag help in the registry, the changeset, and the stash-cli skill (which ships to customers). No behaviour change — `--prisma` still fails with a pointer. Build + 566 tests green. Claude-Session: https://claude.ai/code/session_01MxTTPaPP16m6br7Hoab94w
1 parent 6381171 commit adf5243

5 files changed

Lines changed: 13 additions & 12 deletions

File tree

.changeset/eql-migration-command.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,5 +17,6 @@ the `cs_migrations` tracking schema, so a single `drizzle-kit migrate` covers
1717
everything `stash encrypt …` needs. `--supabase` appends the `eql_v3` +
1818
`eql_v3_internal` role grants for PostgREST/RLS access.
1919

20-
`--prisma` is registered but not available yet — it ships with prisma-next EQL v3
21-
support and fails with a pointer until then.
20+
`--prisma` is registered but not supported — Prisma Next installs EQL v3 through
21+
its own migration system (`prisma-next migration apply`), so the command points
22+
users there instead of emitting a migration.

packages/cli/src/cli/registry.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -394,7 +394,7 @@ export const registry: CommandGroup[] = [
394394
{
395395
name: '--prisma',
396396
description:
397-
'Emit a Prisma Next migration (ships with prisma-next EQL v3 support; not available yet).',
397+
'Not supported — Prisma Next installs EQL v3 via its own migration system (`prisma-next migration apply`); fails with a pointer.',
398398
},
399399
{
400400
name: '--supabase',

packages/cli/src/commands/eql/migration.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -94,9 +94,9 @@ export async function eqlMigrationCommand(
9494
}
9595

9696
if (options.prisma) {
97-
// The prisma-next emitter ships stacked on the prisma-next EQL v3 work
98-
// (PR #655); it can't install a v3 schema that doesn't exist on that
99-
// surface yet. Fail loudly with a pointer rather than emit a broken file.
97+
// Prisma Next installs EQL v3 through its own migration system
98+
// (`prisma-next migration apply`, shipped in #655), so this command has
99+
// nothing to emit for it. Fail loudly with a pointer rather than a no-op.
100100
p.log.error(messages.eql.migrationPrismaUnavailable)
101101
throw new CliExit(1)
102102
}

packages/cli/src/messages.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -73,13 +73,13 @@ export const messages = {
7373
migrationOneTarget:
7474
'Pass exactly one target: `--drizzle` or `--prisma`, not both.',
7575
/**
76-
* `--prisma` isn't wired yet — Prisma Next installs EQL v3 through its own
77-
* migration system (`prisma-next migration apply`), so the CLI emitter is
78-
* not needed there today. Points at the tracking issue so the failure is
79-
* actionable.
76+
* `--prisma` is registered but has nothing to emit: Prisma Next installs
77+
* EQL v3 through its own migration system (`prisma-next migration apply`),
78+
* so this command doesn't generate a Prisma migration. The message points
79+
* users at the path that does the work.
8080
*/
8181
migrationPrismaUnavailable:
82-
'`stash eql migration --prisma` is not available yet — Prisma Next installs EQL v3 via its own migration system (`prisma-next migration apply`, see cipherstash/stack#690). Use `--drizzle` today.',
82+
'`stash eql migration --prisma` is not supported — Prisma Next installs EQL v3 through its own migration system. Run `prisma-next migration apply` instead. Use `--drizzle` for a Drizzle project.',
8383
/** `--name` carried characters outside `[A-Za-z0-9_-]`. */
8484
migrationBadName:
8585
'Migration name must contain only letters, numbers, dashes, and underscores.',

skills/stash-cli/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -380,7 +380,7 @@ stash eql migration --drizzle --supabase # also grant eql_v3 to anon/authentic
380380
| Flag | Description |
381381
|---|---|
382382
| `--drizzle` | Emit a Drizzle custom migration (via `drizzle-kit generate --custom`, then inject the SQL). Requires `drizzle-kit`. |
383-
| `--prisma` | Emit a Prisma Next migration. **Not available yet** — ships with prisma-next EQL v3 support; fails with a pointer until then. |
383+
| `--prisma` | **Not supported.** Prisma Next installs EQL v3 through its own migration system — run `prisma-next migration apply` instead. The flag fails with a pointer. |
384384
| `--supabase` | Append the Supabase role grants (`eql_v3` + `eql_v3_internal``anon`, `authenticated`, `service_role`). Harmless when you connect directly as `postgres`; needed when the same tables are reached via PostgREST/RLS. |
385385
| `--name <name>` | Migration name (Drizzle). Default `install-eql`. |
386386
| `--out <path>` | Output directory (Drizzle). Default `drizzle`. |

0 commit comments

Comments
 (0)