@productbrain/cli 0.1.0-beta.973 → 0.1.0-beta.9754

package/dist/surfaces/__tests__/pb-setup-passthrough.test.js

@@ -1,132 +0,0 @@
-/**
- * pb-setup canonical skill — SurfaceAdapter passthrough goldens (WP-431 S4).
- *
- * doneWhen #1: the canonical .productbrain/skills/pb-setup.md projects through
- * each adapter's `transform` such that the body content is preserved verbatim
- * (no surface-specific rewriting of the body). Per DEC-997, the skill body is
- * the universal SSOT and uses the in-body surface-routing block to drive both
- * CLI and chat-only flows; the adapter's job is wrapping (frontmatter,
- * marker), not rewriting the body itself.
- *
- * Parity assertions per adapter:
- *   - claude:  body emitted verbatim (no wrapping). Path matches `.claude/skills/pb-setup.md`.
- *   - cursor:  body preserved inside a Cursor SKILL.md wrapper. Path matches
- *              `.cursor/skills/pb-setup/SKILL.md`. Triggers appear in the
- *              wrapper's description.
- *   - codex:   body preserved inside a Codex skill projection wrapper. Path
- *              matches `.codex/skills/pb-setup.md`. Triggers appear in the
- *              wrapper.
- *   - copilot: skills not projected (skillsFormat: 'none'). Transform returns
- *              empty string for skill kind (matches handshake.ts behavior).
- *
- * The test reads the canonical file from disk so a content drift in the
- * source markdown surfaces here.
- */
-import { describe, expect, it } from 'vitest';
-import { readFileSync, existsSync } from 'fs';
-import { join } from 'path';
-import { fileURLToPath } from 'url';
-import { dirname } from 'path';
-import { claudeAdapter } from '../adapters/claude.js';
-import { cursorAdapter } from '../adapters/cursor.js';
-import { codexAdapter } from '../adapters/codex.js';
-import { copilotAdapter } from '../adapters/copilot.js';
-const __dirname = dirname(fileURLToPath(import.meta.url));
-// repo root: packages/cli/src/surfaces/__tests__ -> ../../../../..
-const REPO_ROOT = join(__dirname, '..', '..', '..', '..', '..');
-const CANONICAL_PATH = join(REPO_ROOT, '.productbrain', 'skills', 'pb-setup.md');
-/**
- * Minimal frontmatter parser tuned to the pb-setup skill shape. We avoid
- * importing the production parser to keep this test independent of internal
- * helper drift (the goldens are about the `transform` contract, not the
- * parser).
- */
-function loadCanonicalPbSetup() {
-    if (!existsSync(CANONICAL_PATH)) {
-        throw new Error(`Canonical pb-setup.md not found at ${CANONICAL_PATH}`);
-    }
-    const raw = readFileSync(CANONICAL_PATH, 'utf8');
-    const match = raw.match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n?([\s\S]*)$/);
-    if (!match)
-        throw new Error('pb-setup.md missing frontmatter');
-    const yaml = match[1];
-    const body = match[2];
-    const nameMatch = yaml.match(/^name:\s*(.+)$/m);
-    const descMatch = yaml.match(/description:\s*>-\s*\n((?:\s+.+\n?)+?)(?=^\w|^---|\Z)/m);
-    const triggers = [];
-    const trigBlock = yaml.match(/^triggers:\s*\n((?:\s+-\s.+\n?)+)/m);
-    if (trigBlock) {
-        for (const line of trigBlock[1].split('\n')) {
-            const m = line.match(/^\s+-\s+(.+)$/);
-            if (m)
-                triggers.push(m[1].trim().replace(/^["']|["']$/g, ''));
-        }
-    }
-    return {
-        name: (nameMatch?.[1] ?? 'pb-setup').trim(),
-        description: descMatch
-            ? descMatch[1].split('\n').map((l) => l.trim()).filter(Boolean).join(' ')
-            : '',
-        triggers,
-        body,
-        sourcePath: CANONICAL_PATH,
-    };
-}
-describe('pb-setup canonical skill — SurfaceAdapter passthrough', () => {
-    const canonical = loadCanonicalPbSetup();
-    it('canonical file exists with expected name and triggers', () => {
-        expect(canonical.name).toBe('pb-setup');
-        expect(canonical.triggers).toContain('pb setup');
-        expect(canonical.body.length).toBeGreaterThan(500);
-    });
-    it('claude adapter: body verbatim, path .claude/skills/pb-setup.md', () => {
-        const out = claudeAdapter.transform({ canonical: { kind: 'skill', ...canonical } });
-        expect(out).toBe(canonical.body);
-        const path = claudeAdapter.paths({ kind: 'skill', basename: canonical.name }).projectionPath;
-        expect(path).toBe('.claude/skills/pb-setup.md');
-    });
-    it('cursor adapter: body content preserved inside wrapper, triggers injected, path .cursor/skills/pb-setup/SKILL.md', () => {
-        const out = cursorAdapter.transform({ canonical: { kind: 'skill', ...canonical } });
-        // Body content must appear verbatim (the cursor regex does not match
-        // `.productbrain/{skills,rules,hooks}/...md` — the body has no
-        // `.productbrain/skills/<name>.md` literal references that would be
-        // rewritten, so passthrough holds).
-        expect(out).toContain(canonical.body);
-        // Cursor wraps with its own frontmatter + marker comment.
-        expect(out.startsWith('---\nname: pb-setup')).toBe(true);
-        expect(out).toContain('source: .productbrain/skills/pb-setup.md');
-        // Triggers travel into the description per Cursor's convention.
-        expect(out).toContain('"pb setup"');
-        const path = cursorAdapter.paths({ kind: 'skill', basename: canonical.name }).projectionPath;
-        expect(path).toBe('.cursor/skills/pb-setup/SKILL.md');
-    });
-    it('codex adapter: body content preserved inside wrapper, triggers attached, path .codex/skills/pb-setup.md', () => {
-        const out = codexAdapter.transform({ canonical: { kind: 'skill', ...canonical } });
-        expect(out).toContain(canonical.body);
-        expect(out).toContain('source: .productbrain/skills/pb-setup.md');
-        // Codex wraps with a `# <name>` heading, description, and a Triggers line.
-        expect(out).toContain('# pb-setup');
-        expect(out).toContain('**Triggers:**');
-        expect(out).toContain('`pb setup`');
-        const path = codexAdapter.paths({ kind: 'skill', basename: canonical.name }).projectionPath;
-        expect(path).toBe('.codex/skills/pb-setup.md');
-    });
-    it('copilot adapter: skills not projected (skillsFormat: none), transform returns empty', () => {
-        const out = copilotAdapter.transform({ canonical: { kind: 'skill', ...canonical } });
-        expect(out).toBe('');
-        // Copilot's only target file is the aggregate copilot-instructions.md.
-        const path = copilotAdapter.paths({ kind: 'skill', basename: canonical.name }).projectionPath;
-        expect(path).toBe('.github/copilot-instructions.md');
-    });
-    it('canonical body is free of `.productbrain/{skills,rules}/<name>.md` literal refs that would trigger surface path rewrites', () => {
-        // Per DEC-997, the canonical skill body should not depend on adapter path
-        // rewriting — the in-body surface-routing block carries all surface logic.
-        // Guard: if a future edit adds such a literal reference, this test catches
-        // it BEFORE the cursor/codex transform mangles it through the regex.
-        const skillsRefs = canonical.body.match(/\.productbrain\/skills\/[\w.-]+\.md/g) ?? [];
-        const rulesRefs = canonical.body.match(/\.productbrain\/rules\/[\w.-]+\.md/g) ?? [];
-        expect(skillsRefs).toEqual([]);
-        expect(rulesRefs).toEqual([]);
-    });
-});
-//# sourceMappingURL=pb-setup-passthrough.test.js.map

package/dist/surfaces/__tests__/pb-setup-passthrough.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"pb-setup-passthrough.test.js","sourceRoot":"","sources":["../../../src/surfaces/__tests__/pb-setup-passthrough.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AACH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,MAAM,IAAI,CAAC;AAC9C,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAC;AAC5B,OAAO,EAAE,aAAa,EAAE,MAAM,KAAK,CAAC;AACpC,OAAO,EAAE,OAAO,EAAE,MAAM,MAAM,CAAC;AAC/B,OAAO,EAAE,aAAa,EAAE,MAAM,uBAAuB,CAAC;AACtD,OAAO,EAAE,aAAa,EAAE,MAAM,uBAAuB,CAAC;AACtD,OAAO,EAAE,YAAY,EAAE,MAAM,sBAAsB,CAAC;AACpD,OAAO,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAGxD,MAAM,SAAS,GAAG,OAAO,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;AAC1D,mEAAmE;AACnE,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC,CAAC;AAChE,MAAM,cAAc,GAAG,IAAI,CAAC,SAAS,EAAE,eAAe,EAAE,QAAQ,EAAE,aAAa,CAAC,CAAC;AAEjF;;;;;GAKG;AACH,SAAS,oBAAoB;IAC3B,IAAI,CAAC,UAAU,CAAC,cAAc,CAAC,EAAE,CAAC;QAChC,MAAM,IAAI,KAAK,CAAC,sCAAsC,cAAc,EAAE,CAAC,CAAC;IAC1E,CAAC;IACD,MAAM,GAAG,GAAG,YAAY,CAAC,cAAc,EAAE,MAAM,CAAC,CAAC;IACjD,MAAM,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC,6CAA6C,CAAC,CAAC;IACvE,IAAI,CAAC,KAAK;QAAE,MAAM,IAAI,KAAK,CAAC,iCAAiC,CAAC,CAAC;IAE/D,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;IACtB,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;IAEtB,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,iBAAiB,CAAC,CAAC;IAChD,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,wDAAwD,CAAC,CAAC;IACvF,MAAM,QAAQ,GAAa,EAAE,CAAC;IAC9B,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,oCAAoC,CAAC,CAAC;IACnE,IAAI,SAAS,EAAE,CAAC;QACd,KAAK,MAAM,IAAI,IAAI,SAAS,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;YAC5C,MAAM,CAAC,GAAG,IAAI,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;YACtC,IAAI,CAAC;gBAAE,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,OAAO,CAAC,cAAc,EAAE,EAAE,CAAC,CAAC,CAAC;QAChE,CAAC;IACH,CAAC;IAED,OAAO;QACL,IAAI,EAAE,CAAC,SAAS,EAAE,CAAC,CAAC,CAAC,IAAI,UAAU,CAAC,CAAC,IAAI,EAAE;QAC3C,WAAW,EAAE,SAAS;YACpB,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC;YACzE,CAAC,CAAC,EAAE;QACN,QAAQ;QACR,IAAI;QACJ,UAAU,EAAE,cAAc;KAC3B,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,uDAAuD,EAAE,GAAG,EAAE;IACrE,MAAM,SAAS,GAAG,oBAAoB,EAAE,CAAC;IAEzC,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;QAC/D,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACxC,MAAM,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACjD,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,GAAG,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;QACxE,MAAM,GAAG,GAAG,aAAa,CAAC,SAAS,CAAC,EAAE,SAAS,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,GAAG,SAAS,EAAE,EAAE,CAAC,CAAC;QACpF,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACjC,MAAM,IAAI,GAAG,aAAa,CAAC,KAAK,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,QAAQ,EAAE,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC,cAAc,CAAC;QAC7F,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iHAAiH,EAAE,GAAG,EAAE;QACzH,MAAM,GAAG,GAAG,aAAa,CAAC,SAAS,CAAC,EAAE,SAAS,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,GAAG,SAAS,EAAE,EAAE,CAAC,CAAC;QACpF,qEAAqE;QACrE,+DAA+D;QAC/D,oEAAoE;QACpE,oCAAoC;QACpC,MAAM,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACtC,0DAA0D;QAC1D,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,qBAAqB,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACzD,MAAM,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,0CAA0C,CAAC,CAAC;QAClE,gEAAgE;QAChE,MAAM,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QACpC,MAAM,IAAI,GAAG,aAAa,CAAC,KAAK,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,QAAQ,EAAE,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC,cAAc,CAAC;QAC7F,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,kCAAkC,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yGAAyG,EAAE,GAAG,EAAE;QACjH,MAAM,GAAG,GAAG,YAAY,CAAC,SAAS,CAAC,EAAE,SAAS,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,GAAG,SAAS,EAAE,EAAE,CAAC,CAAC;QACnF,MAAM,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACtC,MAAM,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,0CAA0C,CAAC,CAAC;QAClE,2EAA2E;QAC3E,MAAM,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QACpC,MAAM,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QACvC,MAAM,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QACpC,MAAM,IAAI,GAAG,YAAY,CAAC,KAAK,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,QAAQ,EAAE,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC,cAAc,CAAC;QAC5F,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,2BAA2B,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qFAAqF,EAAE,GAAG,EAAE;QAC7F,MAAM,GAAG,GAAG,cAAc,CAAC,SAAS,CAAC,EAAE,SAAS,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,GAAG,SAAS,EAAE,EAAE,CAAC,CAAC;QACrF,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACrB,uEAAuE;QACvE,MAAM,IAAI,GAAG,cAAc,CAAC,KAAK,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,QAAQ,EAAE,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC,cAAc,CAAC;QAC9F,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,iCAAiC,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0HAA0H,EAAE,GAAG,EAAE;QAClI,0EAA0E;QAC1E,2EAA2E;QAC3E,2EAA2E;QAC3E,qEAAqE;QACrE,MAAM,UAAU,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,sCAAsC,CAAC,IAAI,EAAE,CAAC;QACtF,MAAM,SAAS,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,qCAAqC,CAAC,IAAI,EAAE,CAAC;QACpF,MAAM,CAAC,UAAU,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAC/B,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAChC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/templates/archetypes/boundary.md

@@ -1,23 +0,0 @@
----
-name: chain-boundaries
-description: "{{entry_count}} system boundaries derived from Chain architecture entries"
-autoApply: true
-level: core
----
-
-# Chain-Derived System Boundaries
-
-These {{entry_count}} architecture and decision entries define system boundaries for this workspace. Do not cross a boundary without understanding the contract it defines.
-
-**Boundary entries:** {{entry_ids}}
-
-{{boundary_summary}}
-
-## What this means
-
-- Boundaries define what a domain owns and what it does not own.
-- Cross-domain changes require explicit negotiation — check the Team APIs in `docs/system/domains/`.
-- Run `pb get <ID>` to read the full boundary entry before touching cross-domain seams.
-- If your change crosses a domain boundary, identify the owning domain first (domain-boundaries rule).
-
-**Source entries (names):** {{entry_names}}

package/templates/archetypes/constraint.md

@@ -1,23 +0,0 @@
----
-name: chain-constraints
-description: "{{entry_count}} implementation constraints derived from Chain governance entries"
-autoApply: true
-level: core
----
-
-# Chain-Derived Constraints
-
-These {{entry_count}} governance entries constrain implementation in this workspace. Before writing code that touches any affected domain, check these entries on the Chain.
-
-**Constraint entries:** {{entry_ids}}
-
-{{constraint_summary}}
-
-## What this means
-
-- These standards and rules are non-negotiable. They constrain behavior across all slices.
-- Run `pb get <ID>` to read the full entry before implementing anything in its domain.
-- If your change conflicts with a constraint, capture a tension before proceeding.
-- "I didn't know" is not a valid defense when the constraint is on the Chain.
-
-**Source entries (names):** {{entry_names}}

package/templates/archetypes/convention.md

@@ -1,23 +0,0 @@
----
-name: chain-conventions
-description: "{{entry_count}} conventions derived from Chain decision and pattern entries"
-autoApply: true
-level: core
----
-
-# Chain-Derived Conventions
-
-These {{entry_count}} decision and pattern entries establish conventions for this workspace. Follow them unless you have a decision on the Chain that says otherwise.
-
-**Convention entries:** {{entry_ids}}
-
-{{convention_summary}}
-
-## What this means
-
-- Conventions are established patterns, not suggestions. They reduce decision fatigue.
-- Before inventing a new pattern, search the Chain — it may already be a convention.
-- Run `pb get <ID>` to understand the rationale behind a convention before deviating.
-- If you must deviate, capture a DEC on the Chain explaining the new direction.
-
-**Source entries (names):** {{entry_names}}

package/templates/archetypes/policy.md

@@ -1,23 +0,0 @@
----
-name: chain-policies
-description: "{{entry_count}} operational policies derived from Chain policy and business-rule entries"
-autoApply: true
-level: core
----
-
-# Chain-Derived Policies
-
-These {{entry_count}} policy and business-rule entries govern workflows and access in this workspace. Policies apply even when they are inconvenient.
-
-**Policy entries:** {{entry_ids}}
-
-{{policy_summary}}
-
-## What this means
-
-- Policies govern who can do what and under what conditions.
-- Security, auth, and access control policies are never optional.
-- Run `pb get <ID>` to read the full policy before implementing anything in its domain.
-- If a policy conflicts with a feature requirement, capture a TEN and escalate — do not quietly violate.
-
-**Source entries (names):** {{entry_names}}

package/templates/archetypes/quality-gate.md

@@ -1,23 +0,0 @@
----
-name: chain-quality-gates
-description: "{{entry_count}} quality gates derived from Chain standard and business-rule entries"
-autoApply: true
-level: core
----
-
-# Chain-Derived Quality Gates
-
-These {{entry_count}} standard and business-rule entries define quality gates that must pass before shipping. No exceptions without a documented reason on the Chain.
-
-**Quality gate entries:** {{entry_ids}}
-
-{{quality_summary}}
-
-## What this means
-
-- Quality gates are pass/fail. They are not approximate.
-- Run `npm run build` and `npm run test` before every push — always.
-- Run `pb get <ID>` to read the specific acceptance criteria for a quality gate.
-- If a gate cannot pass due to a legitimate constraint, capture a TEN before shipping.
-
-**Source entries (names):** {{entry_names}}

package/templates/archetypes/workflow.md

@@ -1,23 +0,0 @@
----
-name: chain-workflows
-description: "{{entry_count}} required process flows derived from Chain flow entries"
-autoApply: true
-level: core
----
-
-# Chain-Derived Workflows
-
-These {{entry_count}} flow entries define required processes for this workspace. If a workflow entry exists for a process you are about to execute, follow it.
-
-**Workflow entries:** {{entry_ids}}
-
-{{workflow_summary}}
-
-## What this means
-
-- Workflows define the correct sequence of steps for key processes.
-- Skipping a step is not the same as completing the workflow.
-- Run `pb get <ID>` to read the full workflow before executing the process it defines.
-- If the workflow is incomplete or outdated, capture a TEN — do not silently deviate.
-
-**Source entries (names):** {{entry_names}}