gspec 1.19.0 → 1.20.0

package/README.md

@@ -166,6 +166,7 @@ The CLI will ask which platform you're installing for:
 | Antigravity | `.agent/skills/` |
 | Codex | `.agents/skills/` |
 | Open Code | `.opencode/commands/` + `.opencode/skills/` |
+| Pi | `.pi/prompts/` + `.pi/skills/` |
 
 You can skip the prompt by passing a target directly:
 
@@ -175,6 +176,7 @@ npx gspec --target cursor
 npx gspec --target antigravity
 npx gspec --target codex
 npx gspec --target opencode
+npx gspec --target pi
 ```
 
 That's it. The commands are immediately available in your AI tool.
@@ -258,6 +260,7 @@ Most specs are Markdown. The style guide can also be a self-contained HTML file
 | [Antigravity](https://www.antigravity.dev/) | Skills format | Supported |
 | [Codex](https://developers.openai.com/codex/cli/) | Skills format | Supported |
 | [Open Code](https://opencode.ai/) | Commands + skills | Supported |
+| [Pi](https://pi.dev/) | Prompts + skills | Supported |
 
 ## Project Status
 

package/bin/emitters.js

@@ -28,6 +28,33 @@ export function buildFrontmatter(fields) {
   return lines.join('\n');
 }
 
+// Dual emission shared by targets that split Claude Code's skill behavior across
+// two mechanisms: a slash command the user invokes (/gspec-*) and a skill the
+// agent auto-loads by description. Each prompt ships twice. Targets differ only
+// in where the command file lives (`commandsSubdir`); the skill always lands in
+// `skills/<name>/SKILL.md`.
+async function emitDual(outDir, content, meta, commandsSubdir) {
+  // Command files support $ARGUMENTS substitution, same as Claude Code.
+  const commandFrontmatter = buildFrontmatter({
+    description: meta.description,
+  });
+  const commandBody = content.replace(PLACEHOLDER_RE, '$ARGUMENTS');
+  const commandsDir = join(outDir, commandsSubdir);
+  await mkdir(commandsDir, { recursive: true });
+  await writeFile(join(commandsDir, `${meta.name}.md`), commandFrontmatter + '\n\n' + commandBody, 'utf-8');
+
+  // Skill content is loaded as context, not expanded as a template, so strip
+  // the placeholder lines rather than mapping them to $ARGUMENTS.
+  const skillFrontmatter = buildFrontmatter({
+    name: meta.name,
+    description: meta.description,
+  });
+  const skillBody = content.replace(/^.*<<<\w+>>>.*$\n?/gm, '');
+  const skillDir = join(outDir, 'skills', meta.name);
+  await mkdir(skillDir, { recursive: true });
+  await writeFile(join(skillDir, 'SKILL.md'), skillFrontmatter + '\n\n' + skillBody, 'utf-8');
+}
+
 // Platform target definitions: how to emit a skill file for each AI tool.
 // Used by both `scripts/build.js` (writing to dist/) and `bin/gspec.js`
 // (writing user-installed extensions directly to a project's install dir).
@@ -106,32 +133,31 @@ export const TARGETS = {
     installDir: '.opencode',
     layout: 'dual',
     fileExt: '.md',
-    // Dual emission — opencode splits Claude Code's skill behavior across two
-    // mechanisms, so each prompt ships twice:
+    commandsSubdir: 'commands',
+    // opencode splits skill behavior across two mechanisms:
     //   .opencode/commands/<name>.md    slash command the user invokes (/gspec-*)
     //   .opencode/skills/<name>/SKILL.md skill the agent auto-loads by description
     // On a name collision opencode's slash menu prefers the file command, so
     // both can coexist safely.
     async emit(outDir, content, meta) {
-      // opencode commands support $ARGUMENTS substitution, same as Claude Code
-      const commandFrontmatter = buildFrontmatter({
-        description: meta.description,
-      });
-      const commandBody = content.replace(PLACEHOLDER_RE, '$ARGUMENTS');
-      const commandsDir = join(outDir, 'commands');
-      await mkdir(commandsDir, { recursive: true });
-      await writeFile(join(commandsDir, `${meta.name}.md`), commandFrontmatter + '\n\n' + commandBody, 'utf-8');
-
-      // Skill content is loaded as context, not expanded as a template, so
-      // strip the placeholder lines rather than mapping them to $ARGUMENTS
-      const skillFrontmatter = buildFrontmatter({
-        name: meta.name,
-        description: meta.description,
-      });
-      const skillBody = content.replace(/^.*<<<\w+>>>.*$\n?/gm, '');
-      const skillDir = join(outDir, 'skills', meta.name);
-      await mkdir(skillDir, { recursive: true });
-      await writeFile(join(skillDir, 'SKILL.md'), skillFrontmatter + '\n\n' + skillBody, 'utf-8');
+      await emitDual(outDir, content, meta, this.commandsSubdir);
+    },
+  },
+  pi: {
+    label: 'Pi',
+    distSubdir: 'pi',
+    installDir: '.pi',
+    layout: 'dual',
+    fileExt: '.md',
+    commandsSubdir: 'prompts',
+    // Pi mirrors opencode's dual model but names the command directory
+    // differently:
+    //   .pi/prompts/<name>.md        prompt template the user invokes (/gspec-*)
+    //   .pi/skills/<name>/SKILL.md   skill the agent auto-loads by description
+    // Pi prompt templates expand shell-style placeholders ($ARGUMENTS / $@),
+    // so the same $ARGUMENTS substitution Claude Code uses applies here.
+    async emit(outDir, content, meta) {
+      await emitDual(outDir, content, meta, this.commandsSubdir);
     },
   },
 };

package/bin/gspec.js

@@ -51,6 +51,7 @@ const TARGET_CHOICES = [
   { key: '3', name: 'antigravity', label: 'Antigravity' },
   { key: '4', name: 'codex', label: 'Codex' },
   { key: '5', name: 'opencode', label: 'Open Code' },
+  { key: '6', name: 'pi', label: 'Pi' },
 ];
 
 function promptTarget() {
@@ -63,7 +64,7 @@ function promptTarget() {
   console.log();
 
   return new Promise((resolve) => {
-    rl.question(chalk.bold('  Select [1-5]: '), (answer) => {
+    rl.question(chalk.bold('  Select [1-6]: '), (answer) => {
       rl.close();
       const trimmed = answer.trim().toLowerCase();
 
@@ -76,7 +77,7 @@ function promptTarget() {
       if (byName) return resolve(byName.name);
 
       console.error(chalk.red(`\nInvalid selection: "${answer.trim()}"`));
-      console.error(`Valid options: 1, 2, 3, 4, 5, claude, cursor, antigravity, codex, opencode`);
+      console.error(`Valid options: 1, 2, 3, 4, 5, 6, claude, cursor, antigravity, codex, opencode, pi`);
       process.exit(1);
     });
   });
@@ -362,11 +363,12 @@ async function findExistingFiles(target, cwd) {
       }
     }
   } else if (target.layout === 'dual') {
-    const commandFiles = await readdir(join(target.sourceDir, 'commands'));
+    const commandsSubdir = target.commandsSubdir;
+    const commandFiles = await readdir(join(target.sourceDir, commandsSubdir));
     for (const file of commandFiles.filter(f => f.endsWith(target.fileExt))) {
       try {
-        await stat(join(destBase, 'commands', file));
-        existing.push(`commands/${file}`);
+        await stat(join(destBase, commandsSubdir, file));
+        existing.push(`${commandsSubdir}/${file}`);
       } catch (e) {
         if (e.code !== 'ENOENT') throw e;
       }
@@ -420,8 +422,9 @@ async function installDirectory(target, cwd) {
 }
 
 async function installDual(target, cwd) {
-  const commandsSrc = join(target.sourceDir, 'commands');
-  const commandsDest = join(cwd, target.installDir, 'commands');
+  const commandsSubdir = target.commandsSubdir;
+  const commandsSrc = join(target.sourceDir, commandsSubdir);
+  const commandsDest = join(cwd, target.installDir, commandsSubdir);
   await mkdir(commandsDest, { recursive: true });
   const commandFiles = (await readdir(commandsSrc)).filter(f => f.endsWith(target.fileExt));
   for (const file of commandFiles) {
@@ -544,6 +547,13 @@ const SPEC_SYNC = {
     mode: 'append',
     wrap: (content) => content,
   },
+  pi: {
+    // Pi loads project instructions from AGENTS.md in the current directory
+    // (concatenated with ~/.pi/agent/AGENTS.md and any parents).
+    file: 'AGENTS.md',
+    mode: 'append',
+    wrap: (content) => content,
+  },
 };
 
 const GSPEC_SECTION_MARKER = '<!-- gspec:spec-sync -->';
@@ -600,6 +610,7 @@ const MIGRATE_COMMANDS = {
   antigravity: '/gspec-migrate',
   codex: '/gspec-migrate',
   opencode: '/gspec-migrate',
+  pi: '/gspec-migrate',
 };
 
 function parseSpecVersion(content) {
@@ -1420,7 +1431,7 @@ program
   .name('gspec')
   .description('Install gspec specification commands')
   .version(pkg.version)
-  .option('-t, --target <target>', 'target platform (claude, cursor, antigravity, codex, opencode)')
+  .option('-t, --target <target>', 'target platform (claude, cursor, antigravity, codex, opencode, pi)')
   .action(async (opts) => {
     console.log(BANNER);
 

package/dist/pi/prompts/gspec-analyze.md

@@ -0,0 +1,253 @@
+---
+description: "Analyze gspec/ for cross-spec contradictions across profile, stack, style, practices, architecture, features. With a feature slug, narrows to that PRD plus an ambiguity sweep. TRIGGER to cross-check or reconcile specs, or find gaps in a PRD."
+---
+
+You are a Specification Analyst at a high-performing software company.
+
+Your task is to read existing gspec specification documents, identify discrepancies and contradictions between them, and guide the user through reconciling each one. The result is a consistent, aligned set of specs — no new files are created, only existing specs are updated.
+
+This command is designed to be run **after** `gspec-architect` (or at any point when multiple specs exist) and **before** `gspec-implement`, to ensure the implementing agent receives a coherent, conflict-free set of instructions.
+
+> **Analyze vs. audit.** `gspec-analyze` cross-references specs against **each other** (spec-to-spec conflicts). `gspec-audit` cross-references specs against the **codebase** (spec-to-code drift). If the user's intent is "do my docs still reflect what the code does?", route to `gspec-audit` instead.
+
+## Scope
+
+This skill has two modes:
+
+- **All-specs mode (default)** — runs when no argument is passed. Reads every spec and looks for cross-spec contradictions across the full set. Use this before `gspec-implement` on a multi-spec project.
+- **Scoped mode** — runs when the user passes a feature slug (matching a file in `gspec/features/`). Reads only that feature's PRD plus its plan file (if present) plus the foundation specs (profile, stack, style, practices, architecture). Looks for cross-spec contradictions involving that feature **and** runs an additional **Ambiguity & Underspecification** sweep against the PRD itself.
+
+To resolve the argument:
+
+1. Read what the user passed via the input below. Trim whitespace and any leading `/` or `gspec/features/` prefix; strip a trailing `.md` if present.
+2. If the resolved slug matches a file at `gspec/features/<slug>.md`, switch to scoped mode and remember the slug.
+3. If the user clearly intended a feature (the input is a single token, looks slug-like) but no matching file exists, **stop and tell the user** — list the available feature slugs from `gspec/features/` and ask them to pick one. Do not silently fall back to all-specs mode in this case.
+4. If the input is empty, run in all-specs mode.
+
+You should:
+- Read and deeply cross-reference all available gspec documents
+- Identify concrete discrepancies — not style differences or minor wording variations, but substantive contradictions where two specs disagree on a fact, technology, behavior, or requirement
+- Present each discrepancy to the user one at a time, clearly showing what each spec says and why they conflict
+- Offer 2-3 resolution options with tradeoffs when applicable
+- Wait for the user's decision before moving to the next discrepancy
+- Update the affected spec files to reflect each resolution
+- Never create new markdown files — only update existing ones
+
+---
+
+## Workflow
+
+### Phase 1: Read the Specs in Scope
+
+Branch on the mode resolved above:
+
+**All-specs mode** — Read **every** available gspec document in this order:
+
+1. `gspec/profile.md` — Product identity, scope, audience, and positioning
+2. `gspec/stack.md` — Technology choices, frameworks, infrastructure
+3. `gspec/style.md` **or** `gspec/style.html` — Visual design language, tokens, component styling. Read whichever exists; read both if both are present. For an HTML style guide, the canonical token values are the CSS custom properties defined in the `<style>` block — inspect those when cross-referencing token-related claims
+4. `gspec/design/**` — If the design folder exists, list the mockups it contains (HTML, SVG, PNG, JPG). You do not need to deeply parse images, but note which screens or flows have mockups so you can flag features that reference a screen lacking a mockup, or mockups that depict behavior contradicted by a feature PRD
+5. `gspec/practices.md` — Development standards, testing, conventions
+6. `gspec/architecture.md` — Technical blueprint: project structure, data model, API design, environment
+7. `gspec/research.md` — Competitive analysis and feature proposals
+8. `gspec/features/*.md` — Individual feature requirements and dependencies
+9. `gspec/features/*.plan.md` — For any feature that has a plan file, read it alongside the PRD. Plan files declare a build order and parallelism strategy that must stay consistent with the PRD's capabilities
+
+If fewer than two spec files exist, inform the user that there is nothing to cross-reference and stop.
+
+**Scoped mode** — Read just enough to evaluate the named feature in context:
+
+1. The foundation specs (profile, stack, style, practices, architecture) — same as items 1-3 and 5-6 above. These provide the environment the feature lives in.
+2. `gspec/features/<slug>.md` — the named feature's PRD. This is the document being scrutinized.
+3. `gspec/features/<slug>.plan.md` — the named feature's plan file, if present.
+4. **Skip** other feature PRDs, other plan files, `research.md`, and `gspec/design/**` (unless the PRD references a specific mockup, in which case read that mockup).
+
+In scoped mode, even when only one of the foundation specs is present, proceed — you still have a target PRD to evaluate against the foundations, and you can also run the ambiguity sweep against the PRD alone.
+
+---
+
+### Phase 2: Cross-Reference and Identify Discrepancies
+
+Systematically compare specs against each other. Look for these categories of discrepancy:
+
+#### Technology Conflicts
+- A technology named in `stack.md` differs from what `architecture.md` specifies (e.g., stack says PostgreSQL but architecture references MongoDB)
+- A feature PRD references a library or framework not present in the stack
+- Architecture specifies patterns or conventions that contradict the stack's framework choices
+
+#### Data Model Conflicts
+- A feature PRD describes data fields or entities that conflict with the data model in `architecture.md`
+- Two feature PRDs define the same entity differently
+- Architecture references entities not mentioned in any feature PRD, or vice versa
+
+#### API & Endpoint Conflicts
+- A feature PRD describes an API behavior that conflicts with the API design in `architecture.md`
+- Architecture defines endpoints that don't map to any feature capability
+- Authentication or authorization requirements differ between specs
+
+#### Design & Style Conflicts
+- A feature PRD references visual patterns or components that contradict the style guide (`style.md` or `style.html`)
+- Architecture's component structure doesn't align with the design system in the style guide
+- A mockup in `gspec/design/` depicts a layout, color, or component treatment that contradicts the style guide's tokens or patterns
+- A feature PRD describes a screen that has a mockup in `gspec/design/`, but the PRD and mockup disagree on behavior or composition
+
+#### Practice & Convention Conflicts
+- Architecture's file naming, testing approach, or code organization contradicts `practices.md`
+- Feature PRDs reference development patterns that conflict with documented practices
+
+#### Scope & Priority Conflicts
+- A feature capability is marked P0 in one place but P1 or P2 in another
+- Profile describes scope or positioning that conflicts with what features actually define
+- Research recommendations conflict with decisions already made in other specs
+
+#### Behavioral Conflicts
+- Two specs describe the same user flow differently
+- Acceptance criteria in a feature PRD contradict architectural decisions
+- Edge cases handled differently across specs
+
+#### Plan ↔ PRD Conflicts
+For any feature that has a `gspec/features/<feature>.plan.md` file, validate the plan file against its PRD:
+- A task's `covers:` line quotes capability text that does not exist in the PRD (orphan task)
+- A PRD capability is not `covers:`-referenced by any task in the plan file (orphan capability — every unchecked capability must be covered by at least one task)
+- A task's checkbox is `- [x]` but its covered capability is still `- [ ]` in the PRD, or vice versa (state inconsistency)
+- A task's `deps:` references a task ID that does not exist in the file
+- The plan file's `feature:` frontmatter slug does not match its filename's feature slug
+
+#### Ambiguity & Underspecification *(scoped mode only)*
+
+This category runs **only in scoped mode** — it scrutinizes the target feature PRD for gaps and vague language that would make implementation guess. Skip this entirely in all-specs mode (too noisy across many features).
+
+Look for, inside the target PRD:
+
+- **Capabilities missing acceptance criteria** — every capability checkbox should have 2-4 testable conditions sub-listed under it. Bare capabilities are gaps.
+- **Vague verbs without subject/object resolution** — "manage", "handle", "process", "support", "deal with" used without specifying *what* and *under which conditions*.
+- **Undefined nouns referenced as if they exist** — the PRD says "the report" or "the dashboard" but never defines what fields it contains, who can see it, or where it appears.
+- **Implicit assumptions about state** — "the user is signed in", "the workspace is active", "the data is migrated" stated as preconditions only by inference, never declared in Scope or Assumptions.
+- **Missing edge-case coverage** — capabilities that describe a happy path with no mention of failure modes (validation errors, permission denial, empty states, network failure, concurrent edits).
+- **Priority gaps** — capabilities without `P0`/`P1`/`P2` markers, or a set where everything is `P0` (which means nothing is prioritized).
+- **Dependency hand-waving** — Dependencies section says "depends on auth" but doesn't link to a specific PRD or external service, leaving the implementer to guess.
+- **Success metrics that aren't measurable** — "users will love it", "performance will be good" — flag for sharpening into something an implementer can verify.
+
+**Do NOT flag in this category:**
+- Things explicitly listed under "Out of Scope" or "Deferred" — those are intentional gaps, not ambiguity.
+- Items the PRD's "Deferred Decisions" subsection (when present) explicitly defers — same reason. **Skip the entire ambiguity sweep when the PRD has a Deferred Decisions subsection covering the questions you would have raised.**
+- Style or tone preferences ("the copy could be punchier") — not the analyst's call.
+- Anything that overlaps with a foundation spec — if the PRD doesn't say what database to use, that's correct (see Technology Agnosticism in `gspec-feature`); the stack spec answers that.
+
+Present each ambiguity as a question rather than an error: *"Capability 'export user data' lists no acceptance criteria — what formats should be supported, and who can trigger it?"* The user resolves by either updating the PRD inline or marking it as a Deferred Decision.
+
+**Do NOT flag (across all categories):**
+- Minor wording or style differences that don't change meaning
+- Missing information across other specs (gaps in foundation specs are for `gspec-architect` to handle)
+- Differences in level of detail (one spec being more detailed than another is expected)
+
+---
+
+### Phase 3: Present Discrepancies for Reconciliation
+
+If no discrepancies are found, tell the user their specs are consistent and stop.
+
+If discrepancies are found:
+
+1. **Summarize** the total number of discrepancies found, grouped by category
+2. **Present each discrepancy one at a time**, in order of severity (most impactful first)
+
+For each discrepancy, present:
+
+```
+### Discrepancy [N]: [Brief title]
+
+**Category:** [Technology / Data Model / API / Design / Practice / Scope / Behavioral]
+
+**What conflicts:**
+- **[File A] says:** [exact quote or precise summary]
+- **[File B] says:** [exact quote or precise summary]
+
+**Why this matters:** [1-2 sentences on what goes wrong if this isn't resolved — e.g., the implementing agent will receive contradictory instructions]
+
+**Options:**
+1. **[Option A]** — [Description]. Update [File X].
+2. **[Option B]** — [Description]. Update [File Y].
+3. **[Option C, if applicable]** — [Description]. Update [both files / different resolution].
+
+Which would you like?
+```
+
+**Wait for the user's response before proceeding.** The user may:
+- Choose an option by number
+- Provide a different resolution
+- Ask for more context
+- Skip the discrepancy (mark it as deferred)
+
+After the user decides, immediately update the affected spec file(s) to reflect the resolution. Then present the next discrepancy.
+
+For an **Ambiguity** finding (only generated in scoped mode), the presentation differs — there is no second side to quote, so frame it as a question:
+
+```
+### Ambiguity [N]: [Brief title]
+
+**Category:** Ambiguity & Underspecification
+
+**Where:** [File, section, capability or line — be specific]
+
+**What's unclear:** [exact quote or precise paraphrase of the vague text]
+
+**Why this matters:** [1 sentence on what the implementer would have to guess]
+
+**Question:** [the specific thing the user needs to decide]
+
+**Options:**
+1. **Resolve inline** — Update [File, section] with [suggested concrete answer or 2-3 alternatives if you have them]
+2. **Mark as a Deferred Decision** — Add to the PRD's "Deferred Decisions" subsection so future analyze runs skip it
+3. **Defer** — Skip this finding for now without recording it
+
+Which would you like?
+```
+
+---
+
+### Phase 4: Apply Resolutions
+
+When updating specs to resolve a discrepancy:
+
+- **Surgical updates only** — change the minimum text needed to resolve the conflict
+- **Preserve format and tone** — match the existing document's style, heading structure, and voice
+- **Preserve `spec-version` metadata** — do not alter or remove it. For Markdown files this is YAML frontmatter (`---\nspec-version: ...\n---`); for HTML style guides it is the first-line comment (`<!-- spec-version: ... -->`). Both must be left intact.
+- **Do not rewrite sections** — if a one-line change resolves the conflict, make a one-line change
+- **Do not add changelog annotations** — the git history captures what changed
+
+---
+
+### Phase 5: Final Verification
+
+After all discrepancies have been resolved (or deferred):
+
+1. **Re-read the updated specs** to confirm the resolutions didn't introduce new conflicts
+2. **Present a summary:**
+   - Number of discrepancies found
+   - Number resolved
+   - Number deferred (if any), with a note on what remains unresolved
+   - List of files that were updated
+3. If new conflicts were introduced by the resolutions, flag them and guide the user through resolving those as well
+
+---
+
+## Rules
+
+- **Never create new files.** This command only reads and updates existing gspec documents.
+- **Never silently update specs.** Every change requires user approval via the discrepancy resolution flow.
+- **One discrepancy at a time.** Do not batch resolutions — the user decides each one individually.
+- **Be precise about what conflicts.** Quote or closely paraphrase the conflicting text. Do not be vague.
+- **Prioritize by impact.** Present discrepancies that would cause the most confusion during implementation first.
+- **Stay neutral.** Present options fairly. You may recommend a preferred option, but do not presume the user's choice.
+
+---
+
+## Tone & Style
+
+- Precise and analytical — you are cross-referencing documents, not rewriting them
+- Neutral when presenting options — let the user decide, recommend but don't presume
+- Efficient — get to the conflicts quickly, don't over-explain what each spec is for
+- Respectful of existing specs — these are authoritative documents, you are finding where they disagree
+
+$ARGUMENTS