@productbrain/cli 0.1.0-beta.77 → 0.1.0-beta.7700

package/dist/scoreboard/localDrift.test.js

@@ -0,0 +1,82 @@
+/**
+ * M5 local-pull drift unit tests (A5, DEC-1284).
+ *
+ * Exercises computeLocalDrift over a real temp `.productbrain/` so the stat path is genuinely
+ * driven (not mocked): a config-bearing root (so findProjectRoot resolves) + a projection file
+ * whose mtime is the local pull. Asserts the lag/age math, the clamp, and the fail-soft branches.
+ */
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { mkdtempSync, mkdirSync, writeFileSync, utimesSync, rmSync } from 'fs';
+import { tmpdir } from 'os';
+import { resolve } from 'path';
+import { computeLocalDrift } from './localDrift.js';
+let root;
+function setMtime(path, ms) {
+    const secs = ms / 1000;
+    utimesSync(path, secs, secs);
+}
+beforeEach(() => {
+    root = mkdtempSync(resolve(tmpdir(), 'pb-m5-'));
+    const pb = resolve(root, '.productbrain');
+    mkdirSync(pb, { recursive: true });
+    // config.json makes this a CLI project root (findProjectRoot requires it).
+    writeFileSync(resolve(pb, 'config.json'), JSON.stringify({ siteUrl: 'https://example.test' }));
+});
+afterEach(() => {
+    rmSync(root, { recursive: true, force: true });
+});
+describe('computeLocalDrift', () => {
+    it('fails soft to unavailable when no projection files exist', () => {
+        const d = computeLocalDrift(5_000, 10_000, root);
+        expect(d.localProjectionMtime).toBeNull();
+        expect(d.localLagMs).toBeNull();
+        expect(d.localAgeMs).toBeNull();
+        expect(d.unavailableReason).toMatch(/projection files/);
+    });
+    it('computes lag (server − local) and age (now − local) from the newest projection mtime', () => {
+        const pb = resolve(root, '.productbrain');
+        const ctx = resolve(pb, 'context.md');
+        writeFileSync(ctx, '# context');
+        setMtime(ctx, 1_000);
+        // Also write a newer briefing — newest mtime should win.
+        const brief = resolve(pb, 'briefing.md');
+        writeFileSync(brief, '# briefing');
+        setMtime(brief, 3_000);
+        const d = computeLocalDrift(5_000 /* server */, 10_000 /* now */, root);
+        expect(d.localProjectionMtime).toBe(3_000);
+        expect(d.serverMaterializedAt).toBe(5_000);
+        expect(d.localLagMs).toBe(2_000); // 5000 − 3000
+        expect(d.localAgeMs).toBe(7_000); // 10000 − 3000
+        expect(d.unavailableReason).toBeUndefined();
+    });
+    it('clamps lag at 0 when the local pull is newer than the server materialization', () => {
+        const pb = resolve(root, '.productbrain');
+        const ctx = resolve(pb, 'context.md');
+        writeFileSync(ctx, '# context');
+        setMtime(ctx, 9_000);
+        const d = computeLocalDrift(5_000, 10_000, root);
+        expect(d.localLagMs).toBe(0); // max(0, 5000 − 9000)
+        expect(d.localAgeMs).toBe(1_000);
+    });
+    it('returns null lag when the server never materialized', () => {
+        const pb = resolve(root, '.productbrain');
+        const ctx = resolve(pb, 'context.md');
+        writeFileSync(ctx, '# context');
+        setMtime(ctx, 2_000);
+        const d = computeLocalDrift(null, 10_000, root);
+        expect(d.localLagMs).toBeNull();
+        expect(d.localAgeMs).toBe(8_000);
+    });
+    it('reports unavailable when no project root is found', () => {
+        const bare = mkdtempSync(resolve(tmpdir(), 'pb-noroot-'));
+        try {
+            const d = computeLocalDrift(5_000, 10_000, bare);
+            expect(d.localProjectionMtime).toBeNull();
+            expect(d.unavailableReason).toMatch(/project root|missing/i);
+        }
+        finally {
+            rmSync(bare, { recursive: true, force: true });
+        }
+    });
+});
+//# sourceMappingURL=localDrift.test.js.map

package/dist/scoreboard/localDrift.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"localDrift.test.js","sourceRoot":"","sources":["../../src/scoreboard/localDrift.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACrE,OAAO,EAAE,WAAW,EAAE,SAAS,EAAE,aAAa,EAAE,UAAU,EAAE,MAAM,EAAE,MAAM,IAAI,CAAC;AAC/E,OAAO,EAAE,MAAM,EAAE,MAAM,IAAI,CAAC;AAC5B,OAAO,EAAE,OAAO,EAAE,MAAM,MAAM,CAAC;AAC/B,OAAO,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAC;AAEpD,IAAI,IAAY,CAAC;AAEjB,SAAS,QAAQ,CAAC,IAAY,EAAE,EAAU;IACxC,MAAM,IAAI,GAAG,EAAE,GAAG,IAAI,CAAC;IACvB,UAAU,CAAC,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC,CAAC;AAC/B,CAAC;AAED,UAAU,CAAC,GAAG,EAAE;IACd,IAAI,GAAG,WAAW,CAAC,OAAO,CAAC,MAAM,EAAE,EAAE,QAAQ,CAAC,CAAC,CAAC;IAChD,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,EAAE,eAAe,CAAC,CAAC;IAC1C,SAAS,CAAC,EAAE,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IACnC,2EAA2E;IAC3E,aAAa,CAAC,OAAO,CAAC,EAAE,EAAE,aAAa,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,OAAO,EAAE,sBAAsB,EAAE,CAAC,CAAC,CAAC;AACjG,CAAC,CAAC,CAAC;AAEH,SAAS,CAAC,GAAG,EAAE;IACb,MAAM,CAAC,IAAI,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;AACjD,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QAClE,MAAM,CAAC,GAAG,iBAAiB,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,CAAC,CAAC;QACjD,MAAM,CAAC,CAAC,CAAC,oBAAoB,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC1C,MAAM,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,QAAQ,EAAE,CAAC;QAChC,MAAM,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,QAAQ,EAAE,CAAC;QAChC,MAAM,CAAC,CAAC,CAAC,iBAAiB,CAAC,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sFAAsF,EAAE,GAAG,EAAE;QAC9F,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,EAAE,eAAe,CAAC,CAAC;QAC1C,MAAM,GAAG,GAAG,OAAO,CAAC,EAAE,EAAE,YAAY,CAAC,CAAC;QACtC,aAAa,CAAC,GAAG,EAAE,WAAW,CAAC,CAAC;QAChC,QAAQ,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;QACrB,yDAAyD;QACzD,MAAM,KAAK,GAAG,OAAO,CAAC,EAAE,EAAE,aAAa,CAAC,CAAC;QACzC,aAAa,CAAC,KAAK,EAAE,YAAY,CAAC,CAAC;QACnC,QAAQ,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC;QAEvB,MAAM,CAAC,GAAG,iBAAiB,CAAC,KAAK,CAAC,YAAY,EAAE,MAAM,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;QACxE,MAAM,CAAC,CAAC,CAAC,oBAAoB,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC3C,MAAM,CAAC,CAAC,CAAC,oBAAoB,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC3C,MAAM,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,cAAc;QAChD,MAAM,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,eAAe;QACjD,MAAM,CAAC,CAAC,CAAC,iBAAiB,CAAC,CAAC,aAAa,EAAE,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8EAA8E,EAAE,GAAG,EAAE;QACtF,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,EAAE,eAAe,CAAC,CAAC;QAC1C,MAAM,GAAG,GAAG,OAAO,CAAC,EAAE,EAAE,YAAY,CAAC,CAAC;QACtC,aAAa,CAAC,GAAG,EAAE,WAAW,CAAC,CAAC;QAChC,QAAQ,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;QACrB,MAAM,CAAC,GAAG,iBAAiB,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,CAAC,CAAC;QACjD,MAAM,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,sBAAsB;QACpD,MAAM,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAC7D,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,EAAE,eAAe,CAAC,CAAC;QAC1C,MAAM,GAAG,GAAG,OAAO,CAAC,EAAE,EAAE,YAAY,CAAC,CAAC;QACtC,aAAa,CAAC,GAAG,EAAE,WAAW,CAAC,CAAC;QAChC,QAAQ,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;QACrB,MAAM,CAAC,GAAG,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,CAAC,CAAC;QAChD,MAAM,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,QAAQ,EAAE,CAAC;QAChC,MAAM,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC3D,MAAM,IAAI,GAAG,WAAW,CAAC,OAAO,CAAC,MAAM,EAAE,EAAE,YAAY,CAAC,CAAC,CAAC;QAC1D,IAAI,CAAC;YACH,MAAM,CAAC,GAAG,iBAAiB,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,CAAC,CAAC;YACjD,MAAM,CAAC,CAAC,CAAC,oBAAoB,CAAC,CAAC,QAAQ,EAAE,CAAC;YAC1C,MAAM,CAAC,CAAC,CAAC,iBAAiB,CAAC,CAAC,OAAO,CAAC,uBAAuB,CAAC,CAAC;QAC/D,CAAC;gBAAS,CAAC;YACT,MAAM,CAAC,IAAI,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACjD,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/surfaces/__tests__/adapter.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=adapter.test.d.ts.map

package/dist/surfaces/__tests__/adapter.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"adapter.test.d.ts","sourceRoot":"","sources":["../../../src/surfaces/__tests__/adapter.test.ts"],"names":[],"mappings":""}

package/dist/surfaces/__tests__/adapter.test.js

@@ -0,0 +1,86 @@
+/**
+ * SurfaceAdapter contract tests — WP-421 S2c (DEC-952).
+ *
+ * Verifies every adapter:
+ *   - has identity, displayName, detect, paths, transform, goldens, canonicalPathFor.
+ *   - has at least one golden fixture and transform(canonical) === expected.
+ *   - canonicalPathFor returns null for paths not belonging to that surface.
+ */
+import { describe, expect, it } from 'vitest';
+import { SURFACE_ADAPTERS, SURFACE_ADAPTER_LIST } from '../registry.js';
+import { claudeAdapter } from '../adapters/claude.js';
+import { cursorAdapter } from '../adapters/cursor.js';
+import { codexAdapter } from '../adapters/codex.js';
+import { copilotAdapter } from '../adapters/copilot.js';
+describe('SurfaceAdapter contract', () => {
+    it('registers all four adapters', () => {
+        expect(SURFACE_ADAPTER_LIST).toHaveLength(4);
+        expect(Object.keys(SURFACE_ADAPTERS).sort()).toEqual(['claude', 'codex', 'copilot', 'cursor']);
+    });
+    it.each(SURFACE_ADAPTER_LIST.map((a) => [a.name, a]))('%s adapter exposes the full interface', (_name, adapter) => {
+        expect(typeof adapter.name).toBe('string');
+        expect(typeof adapter.displayName).toBe('string');
+        expect(typeof adapter.detect).toBe('function');
+        expect(typeof adapter.paths).toBe('function');
+        expect(typeof adapter.transform).toBe('function');
+        expect(typeof adapter.canonicalPathFor).toBe('function');
+        expect(Array.isArray(adapter.goldens)).toBe(true);
+        expect(adapter.goldens.length).toBeGreaterThan(0);
+    });
+    it.each(SURFACE_ADAPTER_LIST.map((a) => [a.name, a]))('%s goldens round-trip transform === expected', (_name, adapter) => {
+        for (const golden of adapter.goldens) {
+            const out = adapter.transform({ canonical: golden.canonical });
+            expect(out).toBe(golden.expected);
+        }
+    });
+});
+describe('canonicalPathFor — per-surface mapping', () => {
+    it('claude maps .claude/skills/orient.md → .productbrain/skills/orient.md', () => {
+        expect(claudeAdapter.canonicalPathFor('.claude/skills/orient.md')).toBe('.productbrain/skills/orient.md');
+    });
+    it('claude returns null for the rule kind (DEC-1736 — no claude rule projection) and non-claude paths', () => {
+        expect(claudeAdapter.canonicalPathFor('.claude/rules/foo.md')).toBeNull();
+        expect(claudeAdapter.canonicalPathFor('.cursor/rules/foo.mdc')).toBeNull();
+        expect(claudeAdapter.canonicalPathFor('.github/copilot-instructions.md')).toBeNull();
+        expect(claudeAdapter.canonicalPathFor('AGENTS.md')).toBeNull();
+    });
+    it('cursor flips .mdc → .md (the headline cursor signature)', () => {
+        expect(cursorAdapter.canonicalPathFor('.cursor/rules/foo.mdc')).toBe('.productbrain/rules/foo.md');
+    });
+    it('cursor maps .cursor/skills/<name>/SKILL.md → .productbrain/skills/<name>.md', () => {
+        expect(cursorAdapter.canonicalPathFor('.cursor/skills/sample/SKILL.md')).toBe('.productbrain/skills/sample.md');
+    });
+    it('cursor returns null for non-cursor paths and the wrong extension', () => {
+        expect(cursorAdapter.canonicalPathFor('.cursor/rules/foo.md')).toBeNull();
+        expect(cursorAdapter.canonicalPathFor('.claude/rules/foo.md')).toBeNull();
+    });
+    it('codex maps .codex/skills/orient.md → .productbrain/skills/orient.md', () => {
+        expect(codexAdapter.canonicalPathFor('.codex/skills/orient.md')).toBe('.productbrain/skills/orient.md');
+    });
+    it('codex returns null for AGENTS.md and non-codex paths', () => {
+        expect(codexAdapter.canonicalPathFor('AGENTS.md')).toBeNull();
+        expect(codexAdapter.canonicalPathFor('.cursor/rules/foo.mdc')).toBeNull();
+    });
+    it('copilot returns null for its aggregate file (no defensible 1:1 reverse map)', () => {
+        expect(copilotAdapter.canonicalPathFor('.github/copilot-instructions.md')).toBeNull();
+        expect(copilotAdapter.canonicalPathFor('.claude/rules/foo.md')).toBeNull();
+    });
+});
+describe('canonicalPathFor — forward project then reverse map round-trips', () => {
+    it.each([
+        [cursorAdapter, 'rule', 'foo'],
+        [cursorAdapter, 'skill', 'sample'],
+        [codexAdapter, 'skill', 'orient'],
+    ])('%s adapter %s "%s" projects forward then reverse-maps back to .productbrain', (adapter, kind, basename) => {
+        const projection = adapter.paths({ kind, basename }).projectionPath;
+        const reverse = adapter.canonicalPathFor(projection);
+        const expected = `.productbrain/${kind === 'rule' ? 'rules' : 'skills'}/${basename}.md`;
+        expect(reverse).toBe(expected);
+    });
+    it('copilot forward-projects to the aggregate file but reverse-maps to null (telemetry fallback)', () => {
+        const projection = copilotAdapter.paths({ kind: 'rule', basename: 'foo' }).projectionPath;
+        expect(projection).toBe('.github/copilot-instructions.md');
+        expect(copilotAdapter.canonicalPathFor(projection)).toBeNull();
+    });
+});
+//# sourceMappingURL=adapter.test.js.map

package/dist/surfaces/__tests__/adapter.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"adapter.test.js","sourceRoot":"","sources":["../../../src/surfaces/__tests__/adapter.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AACH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,gBAAgB,EAAE,oBAAoB,EAAE,MAAM,gBAAgB,CAAC;AACxE,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;AAExD,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;IACvC,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,CAAC,oBAAoB,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7C,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAClD,CAAC,QAAQ,EAAE,OAAO,EAAE,SAAS,EAAE,QAAQ,CAAC,CACzC,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,IAAI,CAAC,oBAAoB,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,CACnD,uCAAuC,EACvC,CAAC,KAAK,EAAE,OAAO,EAAE,EAAE;QACjB,MAAM,CAAC,OAAO,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAC3C,MAAM,CAAC,OAAO,OAAO,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC/C,MAAM,CAAC,OAAO,OAAO,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC9C,MAAM,CAAC,OAAO,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,OAAO,CAAC,gBAAgB,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACzD,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;IACpD,CAAC,CACF,CAAC;IAEF,EAAE,CAAC,IAAI,CAAC,oBAAoB,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,CACnD,8CAA8C,EAC9C,CAAC,KAAK,EAAE,OAAO,EAAE,EAAE;QACjB,KAAK,MAAM,MAAM,IAAI,OAAO,CAAC,OAAO,EAAE,CAAC;YACrC,MAAM,GAAG,GAAG,OAAO,CAAC,SAAS,CAAC,EAAE,SAAS,EAAE,MAAM,CAAC,SAAS,EAAE,CAAC,CAAC;YAC/D,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC;QACpC,CAAC;IACH,CAAC,CACF,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,wCAAwC,EAAE,GAAG,EAAE;IACtD,EAAE,CAAC,uEAAuE,EAAE,GAAG,EAAE;QAC/E,MAAM,CAAC,aAAa,CAAC,gBAAgB,CAAC,0BAA0B,CAAC,CAAC,CAAC,IAAI,CACrE,gCAAgC,CACjC,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mGAAmG,EAAE,GAAG,EAAE;QAC3G,MAAM,CAAC,aAAa,CAAC,gBAAgB,CAAC,sBAAsB,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC1E,MAAM,CAAC,aAAa,CAAC,gBAAgB,CAAC,uBAAuB,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC3E,MAAM,CAAC,aAAa,CAAC,gBAAgB,CAAC,iCAAiC,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;QACrF,MAAM,CAAC,aAAa,CAAC,gBAAgB,CAAC,WAAW,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,GAAG,EAAE;QACjE,MAAM,CAAC,aAAa,CAAC,gBAAgB,CAAC,uBAAuB,CAAC,CAAC,CAAC,IAAI,CAClE,4BAA4B,CAC7B,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6EAA6E,EAAE,GAAG,EAAE;QACrF,MAAM,CAAC,aAAa,CAAC,gBAAgB,CAAC,gCAAgC,CAAC,CAAC,CAAC,IAAI,CAC3E,gCAAgC,CACjC,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kEAAkE,EAAE,GAAG,EAAE;QAC1E,MAAM,CAAC,aAAa,CAAC,gBAAgB,CAAC,sBAAsB,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC1E,MAAM,CAAC,aAAa,CAAC,gBAAgB,CAAC,sBAAsB,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC7E,MAAM,CAAC,YAAY,CAAC,gBAAgB,CAAC,yBAAyB,CAAC,CAAC,CAAC,IAAI,CACnE,gCAAgC,CACjC,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,MAAM,CAAC,YAAY,CAAC,gBAAgB,CAAC,WAAW,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC9D,MAAM,CAAC,YAAY,CAAC,gBAAgB,CAAC,uBAAuB,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6EAA6E,EAAE,GAAG,EAAE;QACrF,MAAM,CACJ,cAAc,CAAC,gBAAgB,CAAC,iCAAiC,CAAC,CACnE,CAAC,QAAQ,EAAE,CAAC;QACb,MAAM,CAAC,cAAc,CAAC,gBAAgB,CAAC,sBAAsB,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;IAC7E,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,iEAAiE,EAAE,GAAG,EAAE;IAC/E,EAAE,CAAC,IAAI,CAAC;QACN,CAAC,aAAa,EAAE,MAAM,EAAE,KAAK,CAAC;QAC9B,CAAC,aAAa,EAAE,OAAO,EAAE,QAAQ,CAAC;QAClC,CAAC,YAAY,EAAE,OAAO,EAAE,QAAQ,CAAC;KACzB,CAAC,CACT,6EAA6E,EAC7E,CAAC,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE;QAC1B,MAAM,UAAU,GAAG,OAAO,CAAC,KAAK,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,CAAC,cAAc,CAAC;QACpE,MAAM,OAAO,GAAG,OAAO,CAAC,gBAAgB,CAAC,UAAU,CAAC,CAAC;QACrD,MAAM,QAAQ,GAAG,iBAAiB,IAAI,KAAK,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,IAAI,QAAQ,KAAK,CAAC;QACxF,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IACjC,CAAC,CACF,CAAC;IAEF,EAAE,CAAC,8FAA8F,EAAE,GAAG,EAAE;QACtG,MAAM,UAAU,GAAG,cAAc,CAAC,KAAK,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,KAAK,EAAE,CAAC,CAAC,cAAc,CAAC;QAC1F,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,iCAAiC,CAAC,CAAC;QAC3D,MAAM,CAAC,cAAc,CAAC,gBAAgB,CAAC,UAAU,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;IACjE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/surfaces/__tests__/claude-skill-pointer-drift.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=claude-skill-pointer-drift.test.d.ts.map

package/dist/surfaces/__tests__/claude-skill-pointer-drift.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"claude-skill-pointer-drift.test.d.ts","sourceRoot":"","sources":["../../../src/surfaces/__tests__/claude-skill-pointer-drift.test.ts"],"names":[],"mappings":""}

package/dist/surfaces/__tests__/claude-skill-pointer-drift.test.js

@@ -0,0 +1,179 @@
+/**
+ * Claude skill pointer drift test — WP-761 E4a (TASK-251).
+ *
+ * PR #1039 review round 2 found three canonical skill bodies wrong or
+ * dangerous to invoke (git-cleanup, create-blog, prototype). Round 3 found
+ * three more (flag-cleanup, community-post, retro) plus a fourth
+ * (triage) already fixed content-wise but never load-bearing for this WP,
+ * plus a real gap in this test's own body-match assertion. The mechanism
+ * that kept drawing these findings — "expose every canonical skill as a
+ * Claude pointer, unaudited" — is cut here, not patched again: **a delivery
+ * route ships ONLY the skills WP-761 itself makes load-bearing.**
+ *
+ * INCLUDED is the whole population. Four skills today — `shaping` (WP-761
+ * TASK-245 / E1 edits this file directly), `defect-driven-review` (WP-761
+ * TASK-242 / E2 edits this file directly), and `after-merge` / `handover`
+ * (WP-761 E0 moved both out of `.claude/rules/` and into
+ * `.productbrain/skills/` as on-demand skills — landed). No other canonical
+ * skill gets a pointer from this test's population, regardless of how good
+ * its content is — being exposed here is not "this skill happens to be
+ * fine," it's "this WP depends on it," and nothing else has been audited
+ * well enough yet to make that claim.
+ *
+ * EXCLUDED_SKILLS is kept as the historical record of skills that were
+ * exposed and then found broken (round 2's three) — not consulted by the
+ * population logic below (INCLUDED already leaves them out), but still
+ * asserted to have no pointer, so a future edit can't silently re-expose
+ * a known-broken skill by, say, copy-pasting an old INCLUDED entry.
+ *
+ * Each pointer's frontmatter `name` and `description`, and its body's
+ * target line, are a deliberate denormalized copy of the canonical's
+ * (constitution §12):
+ *
+ *   - Source of truth: .productbrain/skills/<name>.md's `name:`/`description:`
+ *     and its being the file the pointer's body names.
+ *   - Query need: the loader must see a name and description in the pointer
+ *     to auto-match a request, and it cannot read a file it has not loaded.
+ *   - Sync contract: this test. It parses both frontmatter blocks and
+ *     asserts the name and description strings are equal, and that the
+ *     pointer body (frontmatter stripped) is EXACTLY the expected one-line
+ *     target — not merely contains it, so nothing can be prepended,
+ *     appended, or hidden in frontmatter — in both directions, naming the
+ *     skill on drift.
+ *
+ * Never imports the production frontmatter parser (WP-431 S4 precedent, see
+ * pb-setup-passthrough.test.ts) — an independent reader means this test
+ * doesn't share a blind spot with whatever a future generator produces.
+ * It does use the generic `yaml` library (already a repo dependency,
+ * root package.json, resolved via workspace hoisting) to parse the
+ * frontmatter block itself — that's a third-party parser, not production
+ * code, so it doesn't reintroduce the shared-blind-spot risk WP-431 S4
+ * guarded against; see readFrontmatter below (TEN-3784 finding 4: the old
+ * whitespace-joining reader couldn't tell a folded (`>-`) description from
+ * a literal (`|-`) one, or preserve a paragraph break).
+ */
+import { describe, expect, it } from 'vitest';
+import { existsSync, readFileSync, readdirSync } from 'fs';
+import { dirname, join } from 'path';
+import { fileURLToPath } from 'url';
+import { parse as parseYaml } from 'yaml';
+const __dirname = dirname(fileURLToPath(import.meta.url));
+// repo root: packages/cli/src/surfaces/__tests__ -> ../../../../..
+const REPO_ROOT = join(__dirname, '..', '..', '..', '..', '..');
+const CANONICAL_DIR = join(REPO_ROOT, '.productbrain', 'skills');
+const POINTER_ROOT = join(REPO_ROOT, '.claude', 'skills');
+/**
+ * The whole pointer population: name -> which WP-761 slice makes it
+ * load-bearing. Add an entry only when a slice of THIS work package edits
+ * or depends on the file directly — never for "this skill looks fine."
+ */
+const INCLUDED_SKILLS = {
+    shaping: 'WP-761 TASK-245 (E1) edits this file directly — adds a canonical subtraction step and RULE-288/RULE-419 citations.',
+    'defect-driven-review': 'WP-761 TASK-242 (E2) edits this file directly — adds Reviewer 2 item 9 and the Artifact clause.',
+    'after-merge': 'WP-761 E0 moves this rule out of .claude/rules/ into .productbrain/skills/ as an on-demand skill — the largest single subtraction of always-loaded bytes available to this WP.',
+    handover: 'WP-761 E0 moves this rule out of .claude/rules/ into .productbrain/skills/ as an on-demand skill, alongside after-merge.',
+};
+/**
+ * Historical record only (PR #1039 review round 2) — these were exposed,
+ * found wrong or dangerous to invoke, and removed. Not consulted by
+ * INCLUDED_SKILLS above (which already excludes them by omission); asserted
+ * below so a future edit can't silently re-expose one.
+ *
+ * `prototype` (round 2's third finding) is NOT here: E4a-2 deleted the
+ * canonical skill outright (no scaffold/manifest/sync-script has ever
+ * existed in this repo — confirmed live), so there is no file left to
+ * exclude a pointer for. https://github.com/ProductBrain/Product-OS/pull/1039#discussion_r4070378892
+ */
+const EXCLUDED_SKILLS = {
+    'git-cleanup': 'Step 7 pushes `git push origin main` directly — the forbidden direct-to-main path (worktree contract, .claude/rules/git-workflow.md). https://github.com/ProductBrain/Product-OS/pull/1039#discussion_r4070378871',
+    'create-blog': 'Step 3 hardcodes an absolute Mac path ("/Users/randyhereman/Coding/Product-OS/docs/blog/readme.md") for the mandatory voice-guide gate — fails on any other checkout. https://github.com/ProductBrain/Product-OS/pull/1039#discussion_r4070378884',
+};
+/**
+ * Parses one canonical/pointer file's frontmatter block via the `yaml`
+ * library (TEN-3784 finding 4) and returns it as a plain object. Folded
+ * (`>-`) vs literal (`|-`) block scalars and paragraph breaks are resolved
+ * per real YAML semantics, unlike the old whitespace-joining reader this
+ * replaces.
+ */
+function readFrontmatter(raw) {
+    const fmMatch = raw.match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n?/);
+    if (!fmMatch)
+        return {};
+    const parsed = parseYaml(fmMatch[1]);
+    return parsed && typeof parsed === 'object' ? parsed : {};
+}
+/** Strips the frontmatter block, returning only the body, trimmed. */
+function stripFrontmatter(raw) {
+    const fmMatch = raw.match(/^---\r?\n[\s\S]*?\r?\n---\r?\n?/);
+    return (fmMatch ? raw.slice(fmMatch[0].length) : raw).trim();
+}
+describe('Claude skill pointer drift (WP-761 E4a)', () => {
+    it('INCLUDED_SKILLS <-> pointer: every listed skill has a pointer with matching name, description, and exact body', () => {
+        const names = Object.keys(INCLUDED_SKILLS).sort();
+        const mismatches = [];
+        const failedNames = new Set();
+        for (const name of names) {
+            const canonicalPath = join(CANONICAL_DIR, `${name}.md`);
+            const pointerPath = join(POINTER_ROOT, name, 'SKILL.md');
+            if (!existsSync(canonicalPath)) {
+                mismatches.push(`${name}: INCLUDED but no canonical file at .productbrain/skills/${name}.md`);
+                failedNames.add(name);
+                continue;
+            }
+            if (!existsSync(pointerPath)) {
+                mismatches.push(`${name}: INCLUDED but no pointer at .claude/skills/${name}/SKILL.md`);
+                failedNames.add(name);
+                continue;
+            }
+            const canonicalRaw = readFileSync(canonicalPath, 'utf8');
+            const pointerRaw = readFileSync(pointerPath, 'utf8');
+            const canonicalFm = readFrontmatter(canonicalRaw);
+            const pointerFm = readFrontmatter(pointerRaw);
+            const canonicalName = canonicalFm.name;
+            const pointerName = pointerFm.name;
+            if (pointerName !== name) {
+                mismatches.push(`${name}: pointer name field is "${String(pointerName)}", expected "${name}"`);
+                failedNames.add(name);
+            }
+            if (canonicalName !== name) {
+                mismatches.push(`${name}: canonical name field is "${String(canonicalName)}", expected "${name}"`);
+                failedNames.add(name);
+            }
+            const canonicalDesc = canonicalFm.description;
+            const pointerDesc = pointerFm.description;
+            if (canonicalDesc !== pointerDesc) {
+                mismatches.push(`${name}: description drift — canonical="${canonicalDesc}" pointer="${pointerDesc}"`);
+                failedNames.add(name);
+            }
+            // Exact body match (frontmatter stripped) — not `.includes()`. A
+            // pointer that prepends/appends stale or unsafe instructions, or
+            // hides the target line inside frontmatter, must go red here.
+            const expectedBody = `Read and follow .productbrain/skills/${name}.md`;
+            const actualBody = stripFrontmatter(pointerRaw);
+            if (actualBody !== expectedBody) {
+                mismatches.push(`${name}: pointer body is "${actualBody}", expected exactly "${expectedBody}"`);
+                failedNames.add(name);
+            }
+        }
+        const matched = names.length - failedNames.size;
+        expect(mismatches, `${matched}/${names.length} INCLUDED skills have a fully matching pointer`).toEqual([]);
+        expect(names.length).toBeGreaterThan(0);
+    });
+    it('pointer -> INCLUDED_SKILLS: every pointer directory is a declared INCLUDED entry (no undeclared exposure)', () => {
+        const includedNames = new Set(Object.keys(INCLUDED_SKILLS));
+        const pointerNames = existsSync(POINTER_ROOT)
+            ? readdirSync(POINTER_ROOT, { withFileTypes: true })
+                .filter((e) => e.isDirectory())
+                .map((e) => e.name)
+                .sort()
+            : [];
+        const undeclared = pointerNames.filter((name) => !includedNames.has(name));
+        expect(undeclared, `${pointerNames.length - undeclared.length}/${pointerNames.length} pointers are declared in INCLUDED_SKILLS`).toEqual([]);
+        expect(pointerNames.length, `pointer count (${pointerNames.length}) vs INCLUDED count (${includedNames.size})`).toBe(includedNames.size);
+    });
+    it('EXCLUDED_SKILLS stay excluded: no pointer directory exists for any historical-record entry', () => {
+        const stillExposed = Object.keys(EXCLUDED_SKILLS).filter((name) => existsSync(join(POINTER_ROOT, name, 'SKILL.md')));
+        expect(stillExposed, `${Object.keys(EXCLUDED_SKILLS).length - stillExposed.length}/${Object.keys(EXCLUDED_SKILLS).length} excluded skills correctly have no pointer`).toEqual([]);
+    });
+});
+//# sourceMappingURL=claude-skill-pointer-drift.test.js.map

package/dist/surfaces/__tests__/claude-skill-pointer-drift.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"claude-skill-pointer-drift.test.js","sourceRoot":"","sources":["../../../src/surfaces/__tests__/claude-skill-pointer-drift.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAqDG;AACH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,UAAU,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,IAAI,CAAC;AAC3D,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAC;AACrC,OAAO,EAAE,aAAa,EAAE,MAAM,KAAK,CAAC;AACpC,OAAO,EAAE,KAAK,IAAI,SAAS,EAAE,MAAM,MAAM,CAAC;AAE1C,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,aAAa,GAAG,IAAI,CAAC,SAAS,EAAE,eAAe,EAAE,QAAQ,CAAC,CAAC;AACjE,MAAM,YAAY,GAAG,IAAI,CAAC,SAAS,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC;AAE1D;;;;GAIG;AACH,MAAM,eAAe,GAA2B;IAC9C,OAAO,EACL,oHAAoH;IACtH,sBAAsB,EACpB,iGAAiG;IACnG,aAAa,EACX,gLAAgL;IAClL,QAAQ,EACN,0HAA0H;CAC7H,CAAC;AAEF;;;;;;;;;;GAUG;AACH,MAAM,eAAe,GAA2B;IAC9C,aAAa,EACX,mNAAmN;IACrN,aAAa,EACX,mPAAmP;CACtP,CAAC;AAEF;;;;;;GAMG;AACH,SAAS,eAAe,CAAC,GAAW;IAClC,MAAM,OAAO,GAAG,GAAG,CAAC,KAAK,CAAC,mCAAmC,CAAC,CAAC;IAC/D,IAAI,CAAC,OAAO;QAAE,OAAO,EAAE,CAAC;IACxB,MAAM,MAAM,GAAY,SAAS,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC;IAC9C,OAAO,MAAM,IAAI,OAAO,MAAM,KAAK,QAAQ,CAAC,CAAC,CAAE,MAAkC,CAAC,CAAC,CAAC,EAAE,CAAC;AACzF,CAAC;AAED,sEAAsE;AACtE,SAAS,gBAAgB,CAAC,GAAW;IACnC,MAAM,OAAO,GAAG,GAAG,CAAC,KAAK,CAAC,iCAAiC,CAAC,CAAC;IAC7D,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;AAC/D,CAAC;AAED,QAAQ,CAAC,yCAAyC,EAAE,GAAG,EAAE;IACvD,EAAE,CAAC,+GAA+G,EAAE,GAAG,EAAE;QACvH,MAAM,KAAK,GAAG,MAAM,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC,IAAI,EAAE,CAAC;QAClD,MAAM,UAAU,GAAa,EAAE,CAAC;QAChC,MAAM,WAAW,GAAG,IAAI,GAAG,EAAU,CAAC;QACtC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,MAAM,aAAa,GAAG,IAAI,CAAC,aAAa,EAAE,GAAG,IAAI,KAAK,CAAC,CAAC;YACxD,MAAM,WAAW,GAAG,IAAI,CAAC,YAAY,EAAE,IAAI,EAAE,UAAU,CAAC,CAAC;YACzD,IAAI,CAAC,UAAU,CAAC,aAAa,CAAC,EAAE,CAAC;gBAC/B,UAAU,CAAC,IAAI,CAAC,GAAG,IAAI,4DAA4D,IAAI,KAAK,CAAC,CAAC;gBAC9F,WAAW,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;gBACtB,SAAS;YACX,CAAC;YACD,IAAI,CAAC,UAAU,CAAC,WAAW,CAAC,EAAE,CAAC;gBAC7B,UAAU,CAAC,IAAI,CAAC,GAAG,IAAI,+CAA+C,IAAI,WAAW,CAAC,CAAC;gBACvF,WAAW,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;gBACtB,SAAS;YACX,CAAC;YACD,MAAM,YAAY,GAAG,YAAY,CAAC,aAAa,EAAE,MAAM,CAAC,CAAC;YACzD,MAAM,UAAU,GAAG,YAAY,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;YAErD,MAAM,WAAW,GAAG,eAAe,CAAC,YAAY,CAAC,CAAC;YAClD,MAAM,SAAS,GAAG,eAAe,CAAC,UAAU,CAAC,CAAC;YAE9C,MAAM,aAAa,GAAG,WAAW,CAAC,IAAI,CAAC;YACvC,MAAM,WAAW,GAAG,SAAS,CAAC,IAAI,CAAC;YACnC,IAAI,WAAW,KAAK,IAAI,EAAE,CAAC;gBACzB,UAAU,CAAC,IAAI,CAAC,GAAG,IAAI,4BAA4B,MAAM,CAAC,WAAW,CAAC,gBAAgB,IAAI,GAAG,CAAC,CAAC;gBAC/F,WAAW,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YACxB,CAAC;YACD,IAAI,aAAa,KAAK,IAAI,EAAE,CAAC;gBAC3B,UAAU,CAAC,IAAI,CAAC,GAAG,IAAI,8BAA8B,MAAM,CAAC,aAAa,CAAC,gBAAgB,IAAI,GAAG,CAAC,CAAC;gBACnG,WAAW,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YACxB,CAAC;YAED,MAAM,aAAa,GAAG,WAAW,CAAC,WAAW,CAAC;YAC9C,MAAM,WAAW,GAAG,SAAS,CAAC,WAAW,CAAC;YAC1C,IAAI,aAAa,KAAK,WAAW,EAAE,CAAC;gBAClC,UAAU,CAAC,IAAI,CACb,GAAG,IAAI,oCAAoC,aAAa,cAAc,WAAW,GAAG,CACrF,CAAC;gBACF,WAAW,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YACxB,CAAC;YAED,iEAAiE;YACjE,iEAAiE;YACjE,8DAA8D;YAC9D,MAAM,YAAY,GAAG,wCAAwC,IAAI,KAAK,CAAC;YACvE,MAAM,UAAU,GAAG,gBAAgB,CAAC,UAAU,CAAC,CAAC;YAChD,IAAI,UAAU,KAAK,YAAY,EAAE,CAAC;gBAChC,UAAU,CAAC,IAAI,CACb,GAAG,IAAI,sBAAsB,UAAU,wBAAwB,YAAY,GAAG,CAC/E,CAAC;gBACF,WAAW,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YACxB,CAAC;QACH,CAAC;QACD,MAAM,OAAO,GAAG,KAAK,CAAC,MAAM,GAAG,WAAW,CAAC,IAAI,CAAC;QAChD,MAAM,CAAC,UAAU,EAAE,GAAG,OAAO,IAAI,KAAK,CAAC,MAAM,gDAAgD,CAAC,CAAC,OAAO,CACpG,EAAE,CACH,CAAC;QACF,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2GAA2G,EAAE,GAAG,EAAE;QACnH,MAAM,aAAa,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC,CAAC;QAC5D,MAAM,YAAY,GAAG,UAAU,CAAC,YAAY,CAAC;YAC3C,CAAC,CAAC,WAAW,CAAC,YAAY,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC;iBAC/C,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;iBAC9B,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC;iBAClB,IAAI,EAAE;YACX,CAAC,CAAC,EAAE,CAAC;QACP,MAAM,UAAU,GAAG,YAAY,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3E,MAAM,CACJ,UAAU,EACV,GAAG,YAAY,CAAC,MAAM,GAAG,UAAU,CAAC,MAAM,IAAI,YAAY,CAAC,MAAM,2CAA2C,CAC7G,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QACd,MAAM,CAAC,YAAY,CAAC,MAAM,EAAE,kBAAkB,YAAY,CAAC,MAAM,wBAAwB,aAAa,CAAC,IAAI,GAAG,CAAC,CAAC,IAAI,CAClH,aAAa,CAAC,IAAI,CACnB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4FAA4F,EAAE,GAAG,EAAE;QACpG,MAAM,YAAY,GAAG,MAAM,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAChE,UAAU,CAAC,IAAI,CAAC,YAAY,EAAE,IAAI,EAAE,UAAU,CAAC,CAAC,CACjD,CAAC;QACF,MAAM,CACJ,YAAY,EACZ,GAAG,MAAM,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC,MAAM,GAAG,YAAY,CAAC,MAAM,IAAI,MAAM,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC,MAAM,4CAA4C,CAChJ,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAChB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/surfaces/__tests__/telemetry.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=telemetry.test.d.ts.map

package/dist/surfaces/__tests__/telemetry.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"telemetry.test.d.ts","sourceRoot":"","sources":["../../../src/surfaces/__tests__/telemetry.test.ts"],"names":[],"mappings":""}

package/dist/surfaces/__tests__/telemetry.test.js

@@ -0,0 +1,55 @@
+/**
+ * Reverse-map telemetry fallback — WP-421 S2c.
+ * When canonicalPathFor returns null on a known surface, the consumer must
+ * emit `surface-reverse-map-missing` telemetry and surface the generic upstream
+ * message ("edit upstream in .productbrain/skills|rules|hooks/").
+ */
+import { describe, expect, it, vi, beforeEach } from 'vitest';
+const trackEventMock = vi.fn();
+vi.mock('../../lib/telemetry.js', () => ({
+    trackEvent: (...args) => trackEventMock(...args),
+}));
+import { reportReverseMapMissing, getReverseMapFallbackMessage, } from '../telemetry.js';
+import { canonicalPathForAnySurface } from '../registry.js';
+describe('reportReverseMapMissing', () => {
+    beforeEach(() => {
+        trackEventMock.mockClear();
+    });
+    it('emits surface-reverse-map-missing with surface + projectionPath', () => {
+        const warnings = [];
+        reportReverseMapMissing({ surface: 'copilot', projectionPath: '.github/copilot-instructions.md' }, (msg) => warnings.push(msg));
+        expect(trackEventMock).toHaveBeenCalledWith('surface-reverse-map-missing', {
+            surface: 'copilot',
+            projectionPath: '.github/copilot-instructions.md',
+        });
+        expect(warnings).toHaveLength(1);
+        expect(warnings[0]).toContain('[surface-reverse-map-missing]');
+        expect(warnings[0]).toContain('copilot');
+        expect(warnings[0]).toContain(getReverseMapFallbackMessage());
+    });
+    it('exposes the canonical fallback message', () => {
+        expect(getReverseMapFallbackMessage()).toContain('.productbrain/skills');
+        expect(getReverseMapFallbackMessage()).toContain('rules');
+        expect(getReverseMapFallbackMessage()).toContain('hooks');
+    });
+});
+describe('canonicalPathForAnySurface', () => {
+    it('returns the first matching adapter for a known projection path', () => {
+        const cursorMatch = canonicalPathForAnySurface('.cursor/rules/foo.mdc');
+        expect(cursorMatch).toEqual({
+            surface: 'cursor',
+            canonicalPath: '.productbrain/rules/foo.md',
+        });
+        const claudeMatch = canonicalPathForAnySurface('.claude/skills/orient.md');
+        expect(claudeMatch).toEqual({
+            surface: 'claude',
+            canonicalPath: '.productbrain/skills/orient.md',
+        });
+    });
+    it('returns null for an unknown projection path (caller emits telemetry)', () => {
+        expect(canonicalPathForAnySurface('some/unknown/path.txt')).toBeNull();
+        // Copilot's aggregate file legitimately has no reverse map → null.
+        expect(canonicalPathForAnySurface('.github/copilot-instructions.md')).toBeNull();
+    });
+});
+//# sourceMappingURL=telemetry.test.js.map

package/dist/surfaces/__tests__/telemetry.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"telemetry.test.js","sourceRoot":"","sources":["../../../src/surfaces/__tests__/telemetry.test.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AACH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAE9D,MAAM,cAAc,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAC/B,EAAE,CAAC,IAAI,CAAC,wBAAwB,EAAE,GAAG,EAAE,CAAC,CAAC;IACvC,UAAU,EAAE,CAAC,GAAG,IAAe,EAAE,EAAE,CAAC,cAAc,CAAC,GAAG,IAAI,CAAC;CAC5D,CAAC,CAAC,CAAC;AAEJ,OAAO,EACL,uBAAuB,EACvB,4BAA4B,GAC7B,MAAM,iBAAiB,CAAC;AACzB,OAAO,EAAE,0BAA0B,EAAE,MAAM,gBAAgB,CAAC;AAE5D,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;IACvC,UAAU,CAAC,GAAG,EAAE;QACd,cAAc,CAAC,SAAS,EAAE,CAAC;IAC7B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QACzE,MAAM,QAAQ,GAAa,EAAE,CAAC;QAC9B,uBAAuB,CACrB,EAAE,OAAO,EAAE,SAAS,EAAE,cAAc,EAAE,iCAAiC,EAAE,EACzE,CAAC,GAAG,EAAE,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,GAAG,CAAC,CAC5B,CAAC;QAEF,MAAM,CAAC,cAAc,CAAC,CAAC,oBAAoB,CAAC,6BAA6B,EAAE;YACzE,OAAO,EAAE,SAAS;YAClB,cAAc,EAAE,iCAAiC;SAClD,CAAC,CAAC;QACH,MAAM,CAAC,QAAQ,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACjC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,+BAA+B,CAAC,CAAC;QAC/D,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;QACzC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,4BAA4B,EAAE,CAAC,CAAC;IAChE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,CAAC,4BAA4B,EAAE,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;QACzE,MAAM,CAAC,4BAA4B,EAAE,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;QAC1D,MAAM,CAAC,4BAA4B,EAAE,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,4BAA4B,EAAE,GAAG,EAAE;IAC1C,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;QACxE,MAAM,WAAW,GAAG,0BAA0B,CAAC,uBAAuB,CAAC,CAAC;QACxE,MAAM,CAAC,WAAW,CAAC,CAAC,OAAO,CAAC;YAC1B,OAAO,EAAE,QAAQ;YACjB,aAAa,EAAE,4BAA4B;SAC5C,CAAC,CAAC;QAEH,MAAM,WAAW,GAAG,0BAA0B,CAAC,0BAA0B,CAAC,CAAC;QAC3E,MAAM,CAAC,WAAW,CAAC,CAAC,OAAO,CAAC;YAC1B,OAAO,EAAE,QAAQ;YACjB,aAAa,EAAE,gCAAgC;SAChD,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sEAAsE,EAAE,GAAG,EAAE;QAC9E,MAAM,CAAC,0BAA0B,CAAC,uBAAuB,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;QACvE,mEAAmE;QACnE,MAAM,CAAC,0BAA0B,CAAC,iCAAiC,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;IACnF,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/surfaces/adapter.d.ts

@@ -0,0 +1,70 @@
+/**
+ * SurfaceAdapter — WP-421 S2c (DEC-952).
+ *
+ * One canonical asset in `.productbrain/{skills,rules,hooks}/`; per-surface adapters
+ * detect/paths/transform/goldens. New surface = adapter file + registry line.
+ *
+ * canonicalPathFor is the REVERSE map: given a projection path on a surface
+ * (e.g. `.cursor/rules/foo.mdc`), return the canonical authoring path
+ * (e.g. `.productbrain/rules/foo.md`). Returns null when the path does not
+ * belong to this adapter — callers should use the telemetry fallback in
+ * `surfaces/telemetry.ts` to emit `surface-reverse-map-missing`.
+ */
+import type { CanonicalRule, CanonicalSkill } from '../generators/portable-knowledge.js';
+/** Discriminated union covering canonical asset bodies the adapters consume. */
+export type CanonicalAssetBody = ({
+    kind: 'skill';
+} & CanonicalSkill) | ({
+    kind: 'rule';
+} & CanonicalRule);
+/** Optional Chain grounding context the transform may consult when shaping output. */
+export interface ChainGroundingContext {
+    workspaceStage?: string;
+    totalRelations?: number;
+    detectedStack?: string[];
+    /** Free-form context bag for future grounding signals — adapters MUST tolerate unknown keys. */
+    [key: string]: unknown;
+}
+export interface SurfaceAssetRef {
+    kind: 'skill' | 'rule' | 'hook';
+    basename: string;
+}
+export interface SurfacePaths {
+    /** Path relative to repoRoot where the projection is written. */
+    projectionPath: string;
+}
+export interface SurfaceAdapter {
+    /** Stable machine identity (matches SurfaceName in registry.ts). */
+    name: string;
+    /** Human-friendly display label. */
+    displayName: string;
+    /**
+     * Does this surface exist in the project? Used to decide whether to project at all.
+     * Pure: only inspects repoRoot — no network, no Chain calls.
+     */
+    detect(repoRoot: string): Promise<boolean> | boolean;
+    /** Where the projection is written for a given canonical asset. */
+    paths(asset: SurfaceAssetRef): SurfacePaths;
+    /** Take a canonical asset (+ optional grounding) and produce surface-formatted content. */
+    transform(input: {
+        canonical: CanonicalAssetBody;
+        grounding?: ChainGroundingContext;
+    }): string;
+    /**
+     * Round-trip golden fixtures: pinned (canonical, expected) pairs.
+     * Each adapter MUST define at least one. Tests assert transform(canonical) === expected.
+     */
+    goldens: SurfaceGolden[];
+    /**
+     * REVERSE map from a projection path back to the canonical authoring path
+     * in `.productbrain/`. Used by tampered-bucket prompts to point users at the
+     * upstream source. Returns null when the path is unknown to this adapter.
+     */
+    canonicalPathFor(projectionPath: string): string | null;
+}
+export interface SurfaceGolden {
+    name: string;
+    canonical: CanonicalAssetBody;
+    expected: string;
+}
+//# sourceMappingURL=adapter.d.ts.map

package/dist/surfaces/adapter.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"adapter.d.ts","sourceRoot":"","sources":["../../src/surfaces/adapter.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AACH,OAAO,KAAK,EAAE,aAAa,EAAE,cAAc,EAAE,MAAM,qCAAqC,CAAC;AAEzF,gFAAgF;AAChF,MAAM,MAAM,kBAAkB,GAC1B,CAAC;IAAE,IAAI,EAAE,OAAO,CAAA;CAAE,GAAG,cAAc,CAAC,GACpC,CAAC;IAAE,IAAI,EAAE,MAAM,CAAA;CAAE,GAAG,aAAa,CAAC,CAAC;AAEvC,sFAAsF;AACtF,MAAM,WAAW,qBAAqB;IACpC,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,aAAa,CAAC,EAAE,MAAM,EAAE,CAAC;IACzB,gGAAgG;IAChG,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC;CACxB;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,OAAO,GAAG,MAAM,GAAG,MAAM,CAAC;IAChC,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,YAAY;IAC3B,iEAAiE;IACjE,cAAc,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,WAAW,cAAc;IAC7B,oEAAoE;IACpE,IAAI,EAAE,MAAM,CAAC;IACb,oCAAoC;IACpC,WAAW,EAAE,MAAM,CAAC;IAEpB;;;OAGG;IACH,MAAM,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,GAAG,OAAO,CAAC;IAErD,mEAAmE;IACnE,KAAK,CAAC,KAAK,EAAE,eAAe,GAAG,YAAY,CAAC;IAE5C,2FAA2F;IAC3F,SAAS,CAAC,KAAK,EAAE;QAAE,SAAS,EAAE,kBAAkB,CAAC;QAAC,SAAS,CAAC,EAAE,qBAAqB,CAAA;KAAE,GAAG,MAAM,CAAC;IAE/F;;;OAGG;IACH,OAAO,EAAE,aAAa,EAAE,CAAC;IAEzB;;;;OAIG;IACH,gBAAgB,CAAC,cAAc,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAAC;CACzD;AAED,MAAM,WAAW,aAAa;IAC5B,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,kBAAkB,CAAC;IAC9B,QAAQ,EAAE,MAAM,CAAC;CAClB"}

package/dist/surfaces/adapter.js

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=adapter.js.map

package/dist/surfaces/adapter.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"adapter.js","sourceRoot":"","sources":["../../src/surfaces/adapter.ts"],"names":[],"mappings":""}

package/dist/surfaces/adapters/claude.d.ts

@@ -0,0 +1,3 @@
+import type { SurfaceAdapter } from '../adapter.js';
+export declare const claudeAdapter: SurfaceAdapter;
+//# sourceMappingURL=claude.d.ts.map

package/dist/surfaces/adapters/claude.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"claude.d.ts","sourceRoot":"","sources":["../../../src/surfaces/adapters/claude.ts"],"names":[],"mappings":"AAiBA,OAAO,KAAK,EAGV,cAAc,EAIf,MAAM,eAAe,CAAC;AAoDvB,eAAO,MAAM,aAAa,EAAE,cAQ3B,CAAC"}

package/dist/surfaces/adapters/claude.js

@@ -0,0 +1,72 @@
+/**
+ * Claude Code SurfaceAdapter — WP-421 S2c (DEC-952).
+ *
+ * Projects:
+ *   - hooks → .claude/settings.json (handled outside the per-asset transform path)
+ *
+ * The skill kind has no claude WRITE projection (E4b, WP-761/TASK-249): the repo's one
+ * statement of where a Claude skill is written is the directory-form pointer file
+ * (.claude/skills/<name>/SKILL.md, E4a); canonicalPathFor below still reverse-maps the
+ * legacy flat form for orphan detection.
+ *
+ * Rules (`.claude/rules/<name>.md`) and the skill router (`.claude/rules/skill-router.md`)
+ * are NOT projected here or anywhere else — DEC-1736 (WP-759/TASK-234): `pb handshake`
+ * stops writing `.claude/rules/*`.
+ */
+import { existsSync } from 'fs';
+import { join } from 'path';
+function transform(input) {
+    const { canonical } = input;
+    if (canonical.kind === 'rule') {
+        // The rule kind has no claude projection (DEC-1736, WP-759/TASK-234) — an unhandled kind
+        // here is a caller bug, not a silent no-op.
+        throw new Error('claude adapter does not project the rule kind');
+    }
+    // Skill kind: transform still returns the canonical body verbatim (used by goldens and by
+    // callers projecting through other means); there is no claude WRITE path for it (see paths()).
+    return canonical.body;
+}
+function paths(asset) {
+    switch (asset.kind) {
+        case 'hook':
+            return { projectionPath: '.claude/settings.json' };
+        default:
+            // DEC-1736: the rule kind has no claude projection path.
+            // E4b (WP-761/TASK-249): the skill kind has no claude WRITE path either — the
+            // flat .claude/skills/<name>.md declaration is deleted so the repo holds exactly
+            // one statement of where a Claude skill is written (the E4a directory-form pointer).
+            throw new Error(`claude adapter has no projection path for asset kind "${asset.kind}"`);
+    }
+}
+const goldenSkill = {
+    name: 'sample-skill',
+    description: 'Sample skill for golden test',
+    triggers: ['/sample'],
+    body: 'Reference .productbrain/skills/other.md inside the body.\n',
+    sourcePath: '/abs/.productbrain/skills/sample-skill.md',
+};
+const goldens = [
+    {
+        name: 'skill-verbatim-body',
+        canonical: { kind: 'skill', ...goldenSkill },
+        expected: goldenSkill.body,
+    },
+];
+function canonicalPathFor(projectionPath) {
+    // Normalize: accept paths with or without leading "./".
+    const p = projectionPath.replace(/^\.\//, '');
+    const m = p.match(/^\.claude\/skills\/(.+)\.md$/);
+    if (m)
+        return `.productbrain/skills/${m[1]}.md`;
+    return null;
+}
+export const claudeAdapter = {
+    name: 'claude',
+    displayName: 'Claude Code',
+    detect: (repoRoot) => existsSync(join(repoRoot, '.claude')),
+    paths,
+    transform,
+    goldens,
+    canonicalPathFor,
+};
+//# sourceMappingURL=claude.js.map