@productbrain/cli 0.1.0-beta.7395 → 0.1.0-beta.7409

package/dist/__tests__/handshake-preview.test.js

@@ -3,7 +3,6 @@
  * STD-197: preview is the new default (no --apply flag = no writes).
  */
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import { createHash } from 'crypto';
 const { vfs, MOCK_MARKER, MOCK_CWD, MOCK_HOME } = vi.hoisted(() => ({
     vfs: {},
     MOCK_MARKER: 'auto-generated by pb handshake',
@@ -126,28 +125,15 @@ vi.mock('../lib/method-registry.js', () => ({
 }));
 import { readdirSync, writeFileSync } from 'fs';
 import { formatHandshakeReport } from '../formatters/handshake.js';
-import { kernelCall, kernelCallWithSession } from '../lib/client.js';
-import { readSession } from '../lib/session.js';
+import { kernelCall } from '../lib/client.js';
 import { detectRepo } from '../lib/repo-detect.js';
 import { readManifestStatus } from '../generators/manifest.js';
-import { generateCursorSkill, shouldEmitToTarget } from '../generators/portable-knowledge.js';
+import { generateCursorSkill, shouldEmitToTarget, readCanonicalSkills } from '../generators/portable-knowledge.js';
 import { runHandshake } from '../commands/handshake.js';
 function capturedReport() {
     const calls = vi.mocked(formatHandshakeReport).mock.calls;
     return calls.length > 0 ? calls[calls.length - 1][0] : null;
 }
-function setupAuthoringTestHash(asset) {
-    const canonical = {
-        entryId: asset.entryId,
-        name: asset.name,
-        description: asset.description,
-        assetKind: asset.assetKind,
-        triggers: asset.triggers ?? [],
-        semanticRefs: asset.semanticRefs ?? [],
-        body: asset.body.replace(/\r\n/g, '\n').replace(/\r/g, '\n').trimEnd(),
-    };
-    return `sha256:${createHash('sha256').update(JSON.stringify(canonical), 'utf8').digest('hex')}`;
-}
 describe('runHandshake — preview/apply/dryRun invariants (WP-332 S1)', () => {
     let cwdSpy;
     beforeEach(() => {
@@ -219,43 +205,17 @@ describe('runHandshake — preview/apply/dryRun invariants (WP-332 S1)', () => {
         const combined = chunks.join('').trim();
         expect(() => JSON.parse(combined)).not.toThrow();
     });
-    it('PR #912 round 12 (Codex P2): PB_JSON=1 does NOT suppress logErr — lenient warnings still reach stderr', async () => {
-        // Reuses the "DB baseline changed" fixture below, which reliably triggers a real logErr()
-        // warning ("Warning: authoring deletion ... was not applied ...") independent of stdout.
-        vfs[`${MOCK_CWD}/.productbrain/.authoring-sync.json`] = JSON.stringify({
-            version: 1,
-            assets: { 'SETUP-SKILL-STALE': { path: '.productbrain/skills/Stale Skill.md', hash: 'sha256:old-baseline' } },
-        });
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            if (tool === 'setup.listAssetsForUser') {
-                return {
-                    activeAssets: [{
-                            entryId: 'SETUP-SKILL-STALE',
-                            name: 'Stale Skill',
-                            description: 'DB changed after local sync.',
-                            body: '# Stale Skill\nUpdated DB body.',
-                            triggers: [],
-                            assetKind: 'skill',
-                            semanticRefs: [],
-                            disabledByOwner: false,
-                            lastProjectedHash: 'sha256:new-baseline',
-                            scope: 'personal',
-                        }],
-                    dormantAssets: [],
-                    hasAnyReceipt: true,
-                };
-            }
-            if (tool === 'setup.updateLastProjectedHash')
-                return { ok: true };
-            if (tool === 'setup.materializeSetup')
-                return { ok: true, assetCount: 1, receiptWritten: true };
-            return [];
+    it('PR #912 round 12 (Codex P2): PB_JSON=1 does NOT suppress logErr — warnings still reach stderr', async () => {
+        // WP-776 S3d/S3f: the original fixture ("DB baseline changed" during authoring sync) is
+        // deleted along with the setup-assets system. An unregistered manifest surface is a
+        // still-live, simpler logErr() trigger (surfaces/registry.ts's real, unmocked
+        // validateSurfacesForMode) that proves the same PB_JSON invariant.
+        vi.mocked(readManifestStatus).mockReturnValue({
+            mode: 'project',
+            surfaces: ['.cursor', '.not-a-real-surface'],
+            lock: {},
+            parseStatus: 'ok',
+            manifest: null,
         });
         const priorPbJson = process.env.PB_JSON;
         process.env.PB_JSON = '1';
@@ -274,7 +234,7 @@ describe('runHandshake — preview/apply/dryRun invariants (WP-332 S1)', () => {
             stdoutSpy.mockRestore();
             stderrSpy.mockRestore();
         }
-        expect(stderrChunks.some((c) => c.includes('Warning: authoring deletion for SETUP-SKILL-STALE was not applied'))).toBe(true);
+        expect(stderrChunks.some((c) => c.includes('manifest surface ".not-a-real-surface" is not registered'))).toBe(true);
     });
     it('PR #912 round 13 (Codex P1, security): pb --json handshake never leaks an HTTPS git-remote credential or raw Convex fields', async () => {
         vfs[`${MOCK_CWD}/.productbrain/briefing.md`] = 'Driving entry: WP-462.';
@@ -422,7 +382,11 @@ describe('runHandshake — preview/apply/dryRun invariants (WP-332 S1)', () => {
         expect(capturedReport()?.filesWritten).toEqual([]);
         expect(capturedReport()?.filesSkipped?.some((f) => f.reason === 'preview-only (materialize: observe)')).toBe(true);
     });
-    it('apply + govern authority writes files and invokes materializeSetup (OBJ-24 complement to observe)', async () => {
+    it('apply + govern authority writes files (OBJ-24 complement to observe)', async () => {
+        // WP-776 S3d/S3f: the setup.materializeSetup assertion is dropped — that receipt write is
+        // deleted along with the rest of the setup-assets system (see handshake.ts's own comment
+        // at the old step 8b). The write-mode invariant itself (govern authority actually writes,
+        // unlike observe) still holds and is still worth guarding.
         vi.mocked(readManifestStatus).mockReturnValue({
             mode: 'govern',
             surfaces: ['.cursor'],
@@ -432,7 +396,6 @@ describe('runHandshake — preview/apply/dryRun invariants (WP-332 S1)', () => {
         });
         await runHandshake({ apply: true, surfaces: ['cursor'] });
         expect(writeFileSync).toHaveBeenCalled();
-        expect(vi.mocked(kernelCall).mock.calls.some(([tool]) => tool === 'setup.materializeSetup')).toBe(true);
         expect(capturedReport()?.filesSkipped?.some((f) => f.reason === 'preview-only (materialize: observe)')).toBeFalsy();
     });
     it('--surfaces is an intersection with manifest.surfaces, not a superset', async () => {
@@ -444,125 +407,18 @@ describe('runHandshake — preview/apply/dryRun invariants (WP-332 S1)', () => {
             manifest: null,
         });
         vi.mocked(shouldEmitToTarget).mockImplementation((_entry, target) => target === 'cursor');
-        vi.mocked(generateCursorSkill).mockImplementation((skill) => skill.body);
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            if (tool === 'setup.listAssetsForUser') {
-                return [{
-                        entryId: 'SETUP-SKILL-INTERSECTION',
-                        name: 'Intersection',
-                        description: 'Intersection',
-                        body: '# Intersection',
-                        triggers: [],
-                        assetKind: 'skill',
-                        semanticRefs: [],
-                        disabledByOwner: false,
-                        lastProjectedHash: null,
-                    }];
-            }
-            if (tool === 'setup.getCurrentSetupState')
-                return { effectiveMode: 'observe' };
-            if (tool === 'setup.recordTransition')
-                return { ok: true, receiptId: 'r1', deduped: false };
-            return [];
-        });
+        vi.mocked(generateCursorSkill).mockImplementation(() => '# Intersection');
+        vi.mocked(readCanonicalSkills).mockReturnValueOnce([
+            { name: 'Intersection', description: 'Intersection', triggers: [], body: '# Intersection', sourcePath: 'local' },
+        ]);
         await runHandshake({ apply: true, surfaces: ['claude'] });
         expect(writeFileSync).not.toHaveBeenCalledWith(expect.stringContaining('/.cursor/skills/Intersection/SKILL.md'), expect.any(String));
     });
-    it('apply records a mode-change transition when manifest mode differs from current setup state', async () => {
-        vi.mocked(readManifestStatus).mockReturnValue({
-            mode: 'project',
-            surfaces: ['.cursor'],
-            lock: { 'SETUP-SKILL-TRANSITION': 'workspace' },
-            parseStatus: 'ok',
-            manifest: null,
-        });
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            if (tool === 'setup.listAssetsForUser')
-                return [];
-            if (tool === 'setup.getCurrentSetupState')
-                return { effectiveMode: 'observe' };
-            if (tool === 'setup.recordTransition')
-                return { ok: true, receiptId: 'r1', deduped: false };
-            return [];
-        });
-        await runHandshake({ apply: true });
-        expect(kernelCall).toHaveBeenCalledWith('setup.recordTransition', {
-            fromMode: 'observe',
-            toMode: 'project',
-            parseStatus: 'ok',
-            surfaces: ['.cursor'],
-            lock: { 'SETUP-SKILL-TRANSITION': 'workspace' },
-        });
-    });
-    it('apply bulk-imports .productbrain authoring files with frontmatter id before DB projection', async () => {
-        vi.mocked(readManifestStatus).mockReturnValue({
-            mode: 'project',
-            surfaces: ['.cursor'],
-            lock: {},
-            parseStatus: 'ok',
-            manifest: null,
-        });
-        const skillPath = `${MOCK_CWD}/.productbrain/skills/local.md`;
-        vfs[`${MOCK_CWD}/.productbrain/skills`] = '';
-        vfs[skillPath] = [
-            '---',
-            'id: SETUP-SKILL-FRONTMATTER',
-            'name: Local Skill',
-            'description: Local description',
-            'triggers:',
-            '  - local',
-            'semanticRefs:',
-            '  - entry:WP-423',
-            '---',
-            '# Local Skill',
-            '',
-            'Body.',
-        ].join('\n');
-        vi.mocked(readdirSync).mockImplementation((path) => {
-            if (String(path).endsWith('/.productbrain/skills'))
-                return ['local.md'];
-            return [];
-        });
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            // WP-428 S2 (Critical #1): CLI now calls ingestSetupAssetWithBody per-asset.
-            if (tool === 'setup.ingestSetupAssetWithBody')
-                return { action: 'created', entryId: 'SETUP-SKILL-FRONTMATTER' };
-            if (tool === 'setup.listAssetsForUser')
-                return [];
-            if (tool === 'setup.getCurrentSetupState')
-                return { effectiveMode: 'project' };
-            return [];
-        });
-        await runHandshake({ apply: true });
-        // WP-428 S2: CLI now calls ingestSetupAssetWithBody per-asset (not ingestSetupAssetsBatch).
-        expect(kernelCall).toHaveBeenCalledWith('setup.ingestSetupAssetWithBody', expect.objectContaining({
-            entryId: 'SETUP-SKILL-LOCAL',
-            frontmatterId: 'SETUP-SKILL-FRONTMATTER',
-            name: 'Local Skill',
-            description: 'Local description',
-            assetKind: 'skill',
-            triggers: ['local'],
-            semanticRefs: ['entry:WP-423'],
-        }));
-    });
+    // WP-776 S3d/S3f: "apply records a mode-change transition..." and "apply bulk-imports
+    // .productbrain authoring files..." are deleted — both tested the DB-first setup-asset
+    // source (setup.recordTransition, setup.ingestSetupAssetWithBody), which is gone along with
+    // the rest of the setup-assets system. `pb handshake` now always reads skills/rules from the
+    // filesystem; there is no DB projection or transition receipt left to test.
     it('preview mode reports unchanged for files with identical content', async () => {
         const agentsPath = `${MOCK_CWD}/AGENTS.md`;
         vfs[agentsPath] = `<!-- ${MOCK_MARKER} -->\nagents content`;
@@ -609,484 +465,18 @@ describe('runHandshake — preview/apply/dryRun invariants (WP-332 S1)', () => {
         expect(vfs[claudePath]).toContain('<!-- pb:region:start');
         expect(vfs[agentsPath]).toContain('<!-- pb:region:start');
     });
-    it('apply: user-owned files never trigger a chain.createEntry drift TEN, even with an active session + receipts (TEN-2150)', async () => {
-        vfs[`${MOCK_CWD}/CLAUDE.md`] = '# My own CLAUDE.md\nNo auto-generated marker.';
-        vfs[`${MOCK_CWD}/AGENTS.md`] = '# My own AGENTS.md\nNo auto-generated marker.';
-        // Force the EXACT conditions under which the old code logged the spurious draft
-        // TEN: an active write session + a workspace that already has setup receipts
-        // (non-first-run). hasAnyReceipt is ignored post-fix but proves this is a real
-        // RED guard against the removed forked-adapter drift-logging block.
-        vi.mocked(readSession).mockReturnValue({ sessionId: 'test-session' });
-        vi.mocked(kernelCall).mockImplementation(async (tool) => tool === 'setup.listAssetsForUser'
-            ? { activeAssets: [], dormantAssets: [], hasAnyReceipt: true }
-            : []);
-        try {
-            await runHandshake({ apply: true });
-            // The session-bound path is genuinely exercised (materializeSetup runs through it)…
-            expect(kernelCallWithSession).toHaveBeenCalled();
-            // …but NO draft TEN is ever logged about the user's own files.
-            expect(kernelCallWithSession).not.toHaveBeenCalledWith('chain.createEntry', expect.anything());
-        }
-        finally {
-            // Restore defaults — beforeEach clears call history but not implementations.
-            vi.mocked(readSession).mockReturnValue(null);
-            vi.mocked(kernelCall).mockResolvedValue([]);
-        }
-    });
-    it('PR #912 round 14 (Codex P2): the empty-active-assets progress line describes report-only leftover listing, never cleanup/removal', async () => {
-        vi.mocked(kernelCall).mockImplementation(async (tool) => tool === 'setup.listAssetsForUser'
-            ? { activeAssets: [], dormantAssets: [], hasAnyReceipt: true }
-            : []);
-        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
-        let combined;
-        try {
-            await runHandshake({ apply: true });
-            combined = writeSpy.mock.calls.map(([chunk]) => String(chunk)).join('');
-        }
-        finally {
-            writeSpy.mockRestore();
-            vi.mocked(kernelCall).mockResolvedValue([]);
-        }
-        expect(combined).toContain('any existing projections on the selected surfaces will be listed as leftovers, not removed');
-        expect(combined).not.toMatch(/projections clear/i);
-        expect(combined).not.toMatch(/cleanup runs/i);
-    });
-    it('apply mode projects DB setup assets with body-derived refs, disabled filtering, and hash trailers', async () => {
-        vi.mocked(shouldEmitToTarget).mockImplementation((_entry, target) => target === 'cursor');
-        vi.mocked(generateCursorSkill).mockImplementation((skill) => skill.body);
-        vi.mocked(kernelCall).mockImplementation(async (tool, args) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            if (tool === 'setup.listAssetsForUser') {
-                return [
-                    {
-                        entryId: 'SETUP-SKILL-NO-REFS',
-                        name: 'No Refs',
-                        description: 'No ref skill',
-                        body: '# No refs\nPlain body.',
-                        triggers: [],
-                        assetKind: 'skill',
-                        semanticRefs: [],
-                        disabledByOwner: false,
-                        lastProjectedHash: null,
-                    },
-                    {
-                        entryId: 'SETUP-SKILL-WITH-REF',
-                        name: 'With Ref',
-                        description: 'Ref skill',
-                        body: '# With ref\nUse {{ref:entry:WP-345}}.',
-                        triggers: [],
-                        assetKind: 'skill',
-                        semanticRefs: ['entry:WP-345'],
-                        disabledByOwner: false,
-                        lastProjectedHash: null,
-                    },
-                    {
-                        entryId: 'SETUP-SKILL-DISABLED',
-                        name: 'Disabled Skill',
-                        description: 'Disabled',
-                        body: '# Disabled',
-                        triggers: [],
-                        assetKind: 'skill',
-                        semanticRefs: [],
-                        disabledByOwner: true,
-                        lastProjectedHash: null,
-                    },
-                    {
-                        entryId: 'SETUP-BUNDLE-DEMO',
-                        name: 'Bundle Demo',
-                        description: 'Bundle',
-                        body: '# Bundle',
-                        triggers: [],
-                        assetKind: 'bundle',
-                        semanticRefs: [],
-                        disabledByOwner: false,
-                        lastProjectedHash: null,
-                    },
-                ];
-            }
-            if (tool === 'setup.resolveSemanticRefs') {
-                expect(args?.semanticRefs).toEqual(['entry:WP-345']);
-                return [{ ref: 'entry:WP-345', resolved: { entryId: 'WP-345', name: 'Dynamic Skills' } }];
-            }
-            if (tool === 'setup.updateLastProjectedHash')
-                return { ok: true };
-            if (tool === 'setup.materializeSetup')
-                return { ok: true, assetCount: 2, receiptWritten: true };
-            return [];
-        });
-        await runHandshake({ apply: true, surfaces: ['cursor'] });
-        const writes = vi.mocked(writeFileSync).mock.calls.map(([path, content]) => ({
-            path: String(path),
-            content: String(content),
-        }));
-        const noRefs = writes.find((w) => w.path.endsWith('/.cursor/skills/No Refs/SKILL.md'));
-        const withRef = writes.find((w) => w.path.endsWith('/.cursor/skills/With Ref/SKILL.md'));
-        expect(noRefs?.content).toContain('Plain body.');
-        expect(noRefs?.content).toContain('<!-- pb-hash: sha256:');
-        expect(withRef?.content).toContain('Use {{ref:entry:WP-345}}.');
-        expect(withRef?.content).toContain('<!-- pb-hash: sha256:');
-        expect(writes.some((w) => w.path.includes('Disabled Skill'))).toBe(false);
-        expect(writes.some((w) => w.path.includes('Bundle Demo'))).toBe(false);
-        const hashUpdates = vi.mocked(kernelCall).mock.calls.filter(([tool]) => tool === 'setup.updateLastProjectedHash');
-        expect(hashUpdates).toHaveLength(2);
-        expect(hashUpdates.map(([, args]) => args.entryId).sort()).toEqual([
-            'SETUP-SKILL-NO-REFS',
-            'SETUP-SKILL-WITH-REF',
-        ]);
-    });
-    it('does not update lastProjectedHash when a DB asset projection is forked and skipped', async () => {
-        vi.mocked(shouldEmitToTarget).mockImplementation((_entry, target) => target === 'cursor');
-        vi.mocked(generateCursorSkill).mockImplementation((skill) => skill.body);
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            if (tool === 'setup.listAssetsForUser') {
-                return [
-                    {
-                        entryId: 'SETUP-SKILL-FORKED',
-                        name: 'Forked Skill',
-                        description: 'Forked',
-                        body: '# Forked\nBody.',
-                        triggers: [],
-                        assetKind: 'skill',
-                        semanticRefs: [],
-                        disabledByOwner: false,
-                        lastProjectedHash: null,
-                    },
-                ];
-            }
-            if (tool === 'setup.materializeSetup')
-                return { ok: true, assetCount: 1, receiptWritten: true };
-            return [];
-        });
-        vfs[`${MOCK_CWD}/.cursor/skills/Forked Skill/SKILL.md`] = '# User fork\nNo generated marker.';
-        await runHandshake({ apply: true, surfaces: ['cursor'] });
-        expect(vi.mocked(kernelCall).mock.calls.some(([tool]) => tool === 'setup.updateLastProjectedHash')).toBe(false);
-    });
-    it('writes missing caller-owned personal DB assets back to .productbrain authoring files', async () => {
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            if (tool === 'setup.listAssetsForUser') {
-                return {
-                    activeAssets: [
-                        {
-                            entryId: 'SETUP-SKILL-PERSONAL',
-                            name: 'Personal Skill',
-                            description: 'Personal authoring row.',
-                            body: '# Personal Skill\nBody.',
-                            triggers: ['personal'],
-                            assetKind: 'skill',
-                            semanticRefs: ['entry:WP-423'],
-                            disabledByOwner: false,
-                            lastProjectedHash: null,
-                            scope: 'personal',
-                        },
-                    ],
-                    dormantAssets: [],
-                    hasAnyReceipt: true,
-                };
-            }
-            if (tool === 'setup.updateLastProjectedHash')
-                return { ok: true };
-            if (tool === 'setup.materializeSetup')
-                return { ok: true, assetCount: 1, receiptWritten: true };
-            return [];
-        });
-        await runHandshake({ apply: true });
-        const path = `${MOCK_CWD}/.productbrain/skills/Personal Skill.md`;
-        expect(vfs[path]).toContain('id: SETUP-SKILL-PERSONAL');
-        expect(vfs[path]).toContain('# Personal Skill\nBody.');
-        expect(vi.mocked(kernelCall).mock.calls.some(([tool]) => tool === 'setup.updateLastProjectedHash')).toBe(true);
-    });
-    it('writes newer DB edits over a tracked authoring file unchanged from baseline', async () => {
-        const baselineHash = setupAuthoringTestHash({
-            entryId: 'SETUP-SKILL-DB-WINS',
-            name: 'DB Wins',
-            description: 'DB should win.',
-            assetKind: 'skill',
-            body: '# Original',
-        });
-        vfs[`${MOCK_CWD}/.productbrain/skills`] = '';
-        vfs[`${MOCK_CWD}/.productbrain/skills/DB Wins.md`] = [
-            '---',
-            'id: SETUP-SKILL-DB-WINS',
-            'name: "DB Wins"',
-            'description: "DB should win."',
-            'assetKind: skill',
-            '---',
-            '# Original',
-            '',
-        ].join('\n');
-        vfs[`${MOCK_CWD}/.productbrain/.authoring-sync.json`] = JSON.stringify({
-            version: 1,
-            assets: {
-                'SETUP-SKILL-DB-WINS': {
-                    path: '.productbrain/skills/DB Wins.md',
-                    hash: baselineHash,
-                },
-            },
-        });
-        vi.mocked(readdirSync).mockImplementation((path) => {
-            if (String(path).endsWith('/.productbrain/skills'))
-                return ['DB Wins.md'];
-            return [];
-        });
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            // WP-428 S2 (Critical #1): CLI now calls ingestSetupAssetWithBody per-asset.
-            if (tool === 'setup.ingestSetupAssetWithBody') {
-                return { action: 'unchanged', entryId: 'SETUP-SKILL-DB-WINS' };
-            }
-            if (tool === 'setup.listAssetsForUser') {
-                return {
-                    activeAssets: [
-                        {
-                            entryId: 'SETUP-SKILL-DB-WINS',
-                            name: 'DB Wins',
-                            description: 'DB should win.',
-                            body: '# DB edit',
-                            triggers: [],
-                            assetKind: 'skill',
-                            semanticRefs: [],
-                            disabledByOwner: false,
-                            lastProjectedHash: baselineHash,
-                            scope: 'personal',
-                        },
-                    ],
-                    dormantAssets: [],
-                    hasAnyReceipt: true,
-                };
-            }
-            if (tool === 'setup.updateLastProjectedHash')
-                return { ok: true };
-            if (tool === 'setup.materializeSetup')
-                return { ok: true, assetCount: 1, receiptWritten: true };
-            return [];
-        });
-        await runHandshake({ apply: true });
-        expect(vfs[`${MOCK_CWD}/.productbrain/skills/DB Wins.md`]).toContain('# DB edit');
-        expect(vi.mocked(kernelCall).mock.calls.some(([tool]) => tool === 'setup.updateLastProjectedHash')).toBe(true);
-    });
-    it('marks personal DB assets dormant when their tracked authoring file is deleted', async () => {
-        const syncedHash = setupAuthoringTestHash({
-            entryId: 'SETUP-SKILL-DELETED',
-            name: 'Deleted Skill',
-            description: 'Deleted authoring row.',
-            assetKind: 'skill',
-            body: '# Deleted Skill\nBody.',
-        });
-        vfs[`${MOCK_CWD}/.productbrain/.authoring-sync.json`] = JSON.stringify({
-            version: 1,
-            assets: {
-                'SETUP-SKILL-DELETED': {
-                    path: '.productbrain/skills/Deleted Skill.md',
-                    hash: syncedHash,
-                },
-            },
-        });
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            if (tool === 'setup.listAssetsForUser') {
-                return {
-                    activeAssets: [
-                        {
-                            entryId: 'SETUP-SKILL-DELETED',
-                            name: 'Deleted Skill',
-                            description: 'Deleted authoring row.',
-                            body: '# Deleted Skill\nBody.',
-                            triggers: [],
-                            assetKind: 'skill',
-                            semanticRefs: [],
-                            disabledByOwner: false,
-                            lastProjectedHash: syncedHash,
-                            scope: 'personal',
-                        },
-                    ],
-                    dormantAssets: [],
-                    hasAnyReceipt: true,
-                };
-            }
-            if (tool === 'setup.markPersonalSetupAssetDormantFromSync')
-                return { action: 'dormant', entryId: 'SETUP-SKILL-DELETED' };
-            if (tool === 'setup.materializeSetup')
-                return { ok: true, assetCount: 0, receiptWritten: true };
-            return [];
-        });
-        await runHandshake({ apply: true });
-        expect(vi.mocked(kernelCall).mock.calls).toContainEqual([
-            'setup.markPersonalSetupAssetDormantFromSync',
-            { entryId: 'SETUP-SKILL-DELETED', expectedLastProjectedHash: syncedHash },
-        ]);
-    });
-    it('does not mark a stale tracked deletion dormant when the DB baseline changed', async () => {
-        vfs[`${MOCK_CWD}/.productbrain/.authoring-sync.json`] = JSON.stringify({
-            version: 1,
-            assets: {
-                'SETUP-SKILL-STALE': {
-                    path: '.productbrain/skills/Stale Skill.md',
-                    hash: 'sha256:old-baseline',
-                },
-            },
-        });
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            if (tool === 'setup.listAssetsForUser') {
-                return {
-                    activeAssets: [
-                        {
-                            entryId: 'SETUP-SKILL-STALE',
-                            name: 'Stale Skill',
-                            description: 'DB changed after local sync.',
-                            body: '# Stale Skill\nUpdated DB body.',
-                            triggers: [],
-                            assetKind: 'skill',
-                            semanticRefs: [],
-                            disabledByOwner: false,
-                            lastProjectedHash: 'sha256:new-baseline',
-                            scope: 'personal',
-                        },
-                    ],
-                    dormantAssets: [],
-                    hasAnyReceipt: true,
-                };
-            }
-            if (tool === 'setup.updateLastProjectedHash')
-                return { ok: true };
-            if (tool === 'setup.materializeSetup')
-                return { ok: true, assetCount: 1, receiptWritten: true };
-            return [];
-        });
-        await runHandshake({ apply: true });
-        expect(vi.mocked(kernelCall).mock.calls.some(([tool]) => tool === 'setup.markPersonalSetupAssetDormantFromSync')).toBe(false);
-        expect(vfs[`${MOCK_CWD}/.productbrain/skills/Stale Skill.md`]).toContain('Updated DB body.');
-    });
-    it('does not write a stale snapshot back when dormant sync returns a server-side conflict', async () => {
-        const listedHash = setupAuthoringTestHash({
-            entryId: 'SETUP-SKILL-RACE',
-            name: 'Race Skill',
-            description: 'Race during sync.',
-            assetKind: 'skill',
-            body: '# Race Skill\nDB body.',
-        });
-        vfs[`${MOCK_CWD}/.productbrain/.authoring-sync.json`] = JSON.stringify({
-            version: 1,
-            assets: {
-                'SETUP-SKILL-RACE': {
-                    path: '.productbrain/skills/Race Skill.md',
-                    hash: listedHash,
-                },
-            },
-        });
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            if (tool === 'setup.listAssetsForUser') {
-                return {
-                    activeAssets: [
-                        {
-                            entryId: 'SETUP-SKILL-RACE',
-                            name: 'Race Skill',
-                            description: 'Race during sync.',
-                            body: '# Race Skill\nDB body.',
-                            triggers: [],
-                            assetKind: 'skill',
-                            semanticRefs: [],
-                            disabledByOwner: false,
-                            lastProjectedHash: listedHash,
-                            scope: 'personal',
-                        },
-                    ],
-                    dormantAssets: [],
-                    hasAnyReceipt: true,
-                };
-            }
-            if (tool === 'setup.markPersonalSetupAssetDormantFromSync')
-                return { action: 'conflict', entryId: 'SETUP-SKILL-RACE' };
-            if (tool === 'setup.updateLastProjectedHash')
-                return { ok: true };
-            if (tool === 'setup.materializeSetup')
-                return { ok: true, assetCount: 1, receiptWritten: true };
-            return [];
-        });
-        await runHandshake({ apply: true });
-        expect(vi.mocked(kernelCall).mock.calls).toContainEqual([
-            'setup.markPersonalSetupAssetDormantFromSync',
-            { entryId: 'SETUP-SKILL-RACE', expectedLastProjectedHash: listedHash },
-        ]);
-        expect(vfs[`${MOCK_CWD}/.productbrain/skills/Race Skill.md`]).toBeUndefined();
-        expect(vi.mocked(kernelCall).mock.calls.some(([tool, args]) => tool === 'setup.updateLastProjectedHash' &&
-            args?.entryId === 'SETUP-SKILL-RACE')).toBe(false);
-    });
-    it('does not mark a missing personal authoring file dormant on a fresh clone without local tracking state', async () => {
-        vi.mocked(kernelCall).mockImplementation(async (tool) => {
-            if (tool === 'chain.workspaceReadiness')
-                return null;
-            if (tool === 'chain.getOrientView')
-                return null;
-            if (tool === 'chain.searchEntries')
-                return [];
-            if (tool === 'setup.listAssetsForUser') {
-                return {
-                    activeAssets: [
-                        {
-                            entryId: 'SETUP-SKILL-FRESH',
-                            name: 'Fresh Skill',
-                            description: 'Fresh clone row.',
-                            body: '# Fresh Skill\nBody.',
-                            triggers: [],
-                            assetKind: 'skill',
-                            semanticRefs: [],
-                            disabledByOwner: false,
-                            lastProjectedHash: 'sha256:previous',
-                            scope: 'personal',
-                        },
-                    ],
-                    dormantAssets: [],
-                    hasAnyReceipt: true,
-                };
-            }
-            if (tool === 'setup.updateLastProjectedHash')
-                return { ok: true };
-            if (tool === 'setup.materializeSetup')
-                return { ok: true, assetCount: 1, receiptWritten: true };
-            return [];
-        });
-        await runHandshake({ apply: true });
-        expect(vi.mocked(kernelCall).mock.calls.some(([tool]) => tool === 'setup.markPersonalSetupAssetDormantFromSync')).toBe(false);
-        expect(vfs[`${MOCK_CWD}/.productbrain/skills/Fresh Skill.md`]).toContain('id: SETUP-SKILL-FRESH');
-    });
+    // WP-776 S3d/S3f: the remaining DB-first setup-asset projection/authoring-sync tests that
+    // used to live here (project DB setup assets with refs/hash trailers, skip updating
+    // lastProjectedHash on a forked write, write personal DB assets back to .productbrain
+    // authoring files, write newer DB edits over a tracked file, mark personal DB assets dormant
+    // on authoring-file deletion, refuse a stale dormant-mark when the DB baseline changed,
+    // refuse a stale snapshot write on a server-side sync conflict, refuse a dormant-mark with no
+    // local tracking state, and the 'user-owned files never trigger a drift TEN' regression guard)
+    // are all deleted — every one of them exercised setup.listAssetsForUser / setup.
+    // ingestSetupAssetWithBody / setup.markPersonalSetupAssetDormantFromSync / setup.
+    // updateLastProjectedHash / setup.materializeSetup, none of which exist any more.
+    // `pb handshake` reads skills/rules from the filesystem only now (see the '--surfaces is an
+    // intersection' test above); there is no DB-sourced asset, authoring sync, or dormant-marking
+    // behavior left to test.
 });
 //# sourceMappingURL=handshake-preview.test.js.map