Skip to content

Commit 82796a7

Browse files
committed
docs(skills): fix three factual errors in stash-prisma-next flagged on #683
- init does NOT scaffold wiring for Prisma Next (it skips the encryption-client scaffold — schema is derived from contract.json); say so instead of 'scaffolds the wiring'. (Copilot) - eqlAsc/eqlDesc are free functions taking the column expression (eqlAsc(u.salary)), not zero-arg methods; fix the ORDER BY table row to match the code example. (Copilot) - install-skills.ts: the skillsFor regression guard is a test asserting SKILL_MAP has an entry for every value in a maintained ALL_INTEGRATIONS list, not one that reads the provider registry; correct the comment. (Copilot) Claude-Session: https://claude.ai/code/session_01MxTTPaPP16m6br7Hoab94w
1 parent 5a8e6da commit 82796a7

2 files changed

Lines changed: 7 additions & 4 deletions

File tree

packages/cli/src/commands/init/lib/install-skills.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ const BASE_SKILLS: readonly string[] = ['stash-encryption', 'stash-cli']
2727
* entry would ship as `undefined` and crash both consumers (`installSkills`,
2828
* the AGENTS.md builder) with "not iterable". Degrade to the base skill set
2929
* instead: the user still gets `stash-encryption` + `stash-cli`, never a
30-
* stack trace. (Regression-guarded by a test that pins SKILL_MAP's keys to
31-
* the init provider registry.)
30+
* stack trace. (Regression-guarded by a test asserting SKILL_MAP has a
31+
* non-empty entry for every value in a maintained `ALL_INTEGRATIONS` list.)
3232
*/
3333
export function skillsFor(integration: Integration): readonly string[] {
3434
return SKILL_MAP[integration] ?? BASE_SKILLS

skills/stash-prisma-next/SKILL.md

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,10 @@ npm install @cipherstash/stack @cipherstash/prisma-next
3636
```
3737

3838
Or run `npx stash init --prisma-next`, which detects Prisma Next, installs both
39-
packages pinned to the CLI release, authenticates, and scaffolds the wiring.
39+
packages pinned to the CLI release, and authenticates. It does **not** scaffold
40+
the wiring files — Prisma Next derives its schema from `contract.json`, so there
41+
is no encryption-client file to generate; init prints the next steps (declare
42+
encrypted columns, emit the contract, run the migration) instead.
4043

4144
## The three wiring points
4245

@@ -174,7 +177,7 @@ EQL-derived `eql*` vocabulary (the legacy v2 surface keeps `cipherstash*` names)
174177
| `eqlMatch(term)` | free-text token match (`eql_v3.contains`) | `EncryptedTextSearch` |
175178
| `eqlGt/eqlGte/eqlLt/eqlLte(v)` | range comparison | an `*Ord` domain |
176179
| `eqlBetween(lo,hi)` / `eqlNotBetween(lo,hi)` | range window | an `*Ord` domain |
177-
| `eqlAsc()` / `eqlDesc()` | ORDER BY | an `*Ord` or `TextSearch` domain |
180+
| `eqlAsc(col)` / `eqlDesc(col)` | ORDER BY (free functions, take the column) | an `*Ord` or `TextSearch` domain |
178181
| `eqlJsonContains(obj)` | encrypted JSON containment (`@>`) | `EncryptedJson` |
179182

180183
```typescript

0 commit comments

Comments
 (0)