@principles/core 1.287.1 → 1.287.2

package/dist/runtime-v2/__tests__/artificer-runner-vslice.test.js

@@ -1,1923 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { ArtificerRunner } from '../internalization/artificer-runner.js';
-import { MemoryPIArtifactStore } from '../internalization/pi-artifact-store.js';
-import { DefaultArtificerValidator } from '../internalization/artificer-output.js';
-import { createPITaskDiagnosticJson, parsePITaskMetadata } from '../internalization/pitask-metadata.js';
-import { TestDoubleRuntimeAdapter } from '../adapter/test-double-runtime-adapter.js';
-import { FORMATION_CANDIDATE_LIMIT } from '../internalization/formation-context.js';
-import { PDRuntimeError } from '../error-categories.js';
-const SCRIBE_TASK_ID = 'scribe-001';
-const ARTIFICER_TASK_ID = 'artificer-001';
-function makeScribeTask(overrides = {}) {
-    return {
-        taskId: SCRIBE_TASK_ID,
-        taskKind: 'scribe',
-        status: 'succeeded',
-        attemptCount: 1,
-        maxAttempts: 3,
-        resultRef: 'scribe://run-001',
-        createdAt: new Date().toISOString(),
-        updatedAt: new Date().toISOString(),
-        diagnosticJson: createPITaskDiagnosticJson({
-            dependencyTaskIds: [],
-            channel: 'prompt',
-            timeoutMs: 300_000,
-            inputArtifactRefs: [],
-            outputArtifactRefs: [{ artifactType: 'principle', ref: 'pi-art-scribe-001-run-001' }],
-        }),
-        ...overrides,
-    };
-}
-function makeArtificerTask(overrides = {}) {
-    return {
-        taskId: ARTIFICER_TASK_ID,
-        taskKind: 'artificer',
-        status: 'pending',
-        attemptCount: 0,
-        maxAttempts: 3,
-        createdAt: new Date().toISOString(),
-        updatedAt: new Date().toISOString(),
-        diagnosticJson: createPITaskDiagnosticJson({
-            dependencyTaskIds: [SCRIBE_TASK_ID],
-            channel: 'prompt',
-            timeoutMs: 300_000,
-            inputArtifactRefs: [{ artifactType: 'principle', ref: 'pi-art-scribe-001-run-001' }],
-            outputArtifactRefs: [],
-        }),
-        ...overrides,
-    };
-}
-// PRI-780: v2 is the only generation contract — the runner-level fixtures
-// carry a valid Owner-labelled pack and v2-aligned outputs.
-const V2_RULE_CONTEXT = {
-    version: 2,
-    history: { status: 'available', truncated: false, calls: [] },
-    facts: { priorReadOfTarget: 'unknown', readCount: 0, writeCount: 0, uniqueWritePathCount: 0, sameActionBlockCount: null },
-};
-const V2_TEST_PACK = {
-    sourceNegativeCase: {
-        caseId: 'negative-1',
-        kind: 'negative',
-        toolName: 'write_file',
-        params: { path: '/etc/passwd' },
-        expectedDecision: 'block',
-        ruleContext: V2_RULE_CONTEXT,
-    },
-    ownerDesiredOutcome: 'block writes outside the workspace',
-    positiveCounterexamples: [{
-            caseId: 'positive-1',
-            kind: 'positive',
-            toolName: 'write_file',
-            params: { path: '/project/file.txt' },
-            expectedDecision: 'allow',
-            ruleContext: V2_RULE_CONTEXT,
-        }],
-    evidenceRefs: ['pain://1'],
-    redactionNotes: [],
-};
-function makeArtificerOutput() {
-    return {
-        taskId: ARTIFICER_TASK_ID,
-        sourceScribeArtifactId: 'pi-art-scribe-001-run-001',
-        requiresContextVersion: 2,
-        evidenceRefs: ['pain://1'],
-        implementationCode: 'function evaluate(input, helpers) { return { decision: "allow", matched: false, reason: "ok" }; }',
-        goldenTraceCases: [
-            { caseId: 'positive-1', kind: 'positive', toolName: 'write_file', params: { path: '/project/file.txt' }, expectedDecision: 'allow', ruleContext: V2_RULE_CONTEXT },
-            { caseId: 'negative-1', kind: 'negative', toolName: 'write_file', params: { path: '/etc/passwd' }, expectedDecision: 'block', ruleContext: V2_RULE_CONTEXT },
-        ],
-        affectedTools: ['write_file'],
-        implementationSummary: 'Add input validation to all async operations',
-        sourceTrace: {
-            scribeArtifactId: 'pi-art-scribe-001-run-001',
-        },
-        risks: ['May add latency from error checking'],
-        generatedAt: new Date().toISOString(),
-    };
-}
-function makeScribeArtifact() {
-    return {
-        artifactId: 'pi-art-scribe-001-run-001',
-        artifactKind: 'principle',
-        sourceTaskId: SCRIBE_TASK_ID,
-        lineageArtifactIds: [],
-        validationStatus: 'pending',
-        contentJson: JSON.stringify({
-            taskId: SCRIBE_TASK_ID,
-            sourcePhilosopherArtifactId: 'pi-art-philosopher-001',
-            principleDraft: {
-                title: 'Systematic Error Handling',
-                statement: 'All async operations must include explicit error handling',
-                rationale: 'Uncaught errors cascade into system instability',
-                applicability: ['All async operations'],
-                antiPatterns: ['Ignoring promise rejections'],
-                confidence: 0.9,
-            },
-            sourceTrace: {
-                philosopherArtifactId: 'pi-art-philosopher-001',
-            },
-            risks: [],
-            generatedAt: new Date().toISOString(),
-        }),
-        createdAt: new Date().toISOString(),
-        updatedAt: new Date().toISOString(),
-    };
-}
-describe('ArtificerRunner (PRI-111)', () => {
-    let artifactStore = new MemoryPIArtifactStore();
-    beforeEach(() => {
-        artifactStore = new MemoryPIArtifactStore();
-    });
-    function createMockDeps(overrides = {}) {
-        const artificerTask = makeArtificerTask();
-        const scribeTask = makeScribeTask();
-        const stateManager = {
-            acquireLease: vi.fn().mockResolvedValue(artificerTask),
-            getTask: vi.fn().mockImplementation((id) => {
-                if (id === ARTIFICER_TASK_ID)
-                    return Promise.resolve(artificerTask);
-                if (id === SCRIBE_TASK_ID)
-                    return Promise.resolve(scribeTask);
-                return Promise.resolve(null);
-            }),
-            getRunsByTask: vi.fn().mockResolvedValue([{
-                    runId: 'run-artificer-001',
-                    taskId: ARTIFICER_TASK_ID,
-                    runtimeKind: 'artificer',
-                    startedAt: new Date().toISOString(),
-                }]),
-            getValidRunsByTaskTolerant: vi.fn().mockResolvedValue({
-                runs: [{ runId: 'run-artificer-001', taskId: ARTIFICER_TASK_ID, runtimeKind: 'artificer', startedAt: new Date().toISOString() }],
-                degradedRuns: [],
-            }),
-            updateRunOutput: vi.fn().mockResolvedValue(undefined),
-            markTaskSucceeded: vi.fn().mockResolvedValue(undefined),
-            markTaskFailed: vi.fn().mockResolvedValue(undefined),
-            markTaskRetryWait: vi.fn().mockResolvedValue(undefined),
-            getRetryPolicy: vi.fn().mockReturnValue({ shouldRetry: () => false }),
-        };
-        const runHandle = { runId: 'run-artificer-001', runtimeKind: 'test-double', startedAt: new Date().toISOString() };
-        const succeededStatus = { status: 'succeeded', runId: 'run-artificer-001' };
-        const runtimeAdapter = {
-            startRun: vi.fn().mockResolvedValue(runHandle),
-            pollRun: vi.fn().mockResolvedValue(succeededStatus),
-            fetchOutput: vi.fn().mockResolvedValue({
-                payload: makeArtificerOutput(),
-            }),
-            cancelRun: vi.fn().mockResolvedValue(undefined),
-        };
-        const eventEmitter = {
-            emitTelemetry: vi.fn(),
-        };
-        const validator = new DefaultArtificerValidator();
-        return {
-            stateManager,
-            runtimeAdapter,
-            eventEmitter,
-            validator,
-            artifactStore,
-            behaviorExamplePack: V2_TEST_PACK,
-            ...overrides,
-        };
-    }
-    it('taskKind not artificer fails closed and releases lease', async () => {
-        const wrongKindTask = makeArtificerTask({ taskKind: 'dreamer' });
-        const deps = createMockDeps();
-        deps.stateManager.acquireLease = vi.fn().mockResolvedValue(wrongKindTask);
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('failed');
-        expect(result.errorCategory).toBe('input_invalid');
-        expect(result.failureReason).toContain("must be 'artificer'");
-        expect(deps.stateManager.markTaskFailed).toHaveBeenCalledWith(ARTIFICER_TASK_ID, 'input_invalid');
-    });
-    it('lease conflict is non-mutating', async () => {
-        const deps = createMockDeps();
-        const leaseError = new PDRuntimeError('lease_conflict', 'Another runner holds the lease');
-        deps.stateManager.acquireLease = vi.fn().mockRejectedValue(leaseError);
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('failed');
-        expect(result.errorCategory).toBe('lease_conflict');
-        expect(deps.stateManager.markTaskFailed).not.toHaveBeenCalled();
-        expect(deps.stateManager.markTaskRetryWait).not.toHaveBeenCalled();
-    });
-    it('missing scribe dependency blocked/failure', async () => {
-        const noDepTask = makeArtificerTask({
-            diagnosticJson: createPITaskDiagnosticJson({
-                dependencyTaskIds: [],
-                channel: 'prompt',
-                timeoutMs: 300_000,
-                inputArtifactRefs: [],
-                outputArtifactRefs: [],
-            }),
-        });
-        const deps = createMockDeps();
-        deps.stateManager.acquireLease = vi.fn().mockResolvedValue(noDepTask);
-        deps.stateManager.getTask = vi.fn().mockResolvedValue(noDepTask);
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('failed');
-        expect(result.errorCategory).toBe('input_invalid');
-    });
-    it('scribe dependency not succeeded cannot execute', async () => {
-        const pendingScribe = makeScribeTask({ status: 'pending' });
-        const deps = createMockDeps();
-        deps.stateManager.getTask = vi.fn().mockImplementation((id) => {
-            if (id === ARTIFICER_TASK_ID)
-                return Promise.resolve(makeArtificerTask());
-            if (id === SCRIBE_TASK_ID)
-                return Promise.resolve(pendingScribe);
-            return Promise.resolve(null);
-        });
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('failed');
-        expect(result.errorCategory).toBe('input_invalid');
-    });
-    it('scribe artifact missing goes to retry/fail', async () => {
-        const emptyArtifactStore = new MemoryPIArtifactStore();
-        const deps = createMockDeps({ artifactStore: emptyArtifactStore });
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('failed');
-        expect(result.errorCategory).toBe('input_invalid');
-    });
-    it('valid runtime output writes artificer PIArtifact', async () => {
-        const store = new MemoryPIArtifactStore();
-        await store.upsertArtifact(makeScribeArtifact());
-        const deps = createMockDeps({ artifactStore: store });
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('succeeded');
-        expect(result.artifactId).toBeDefined();
-        const artifacts = await store.listBySourceTaskId(ARTIFICER_TASK_ID);
-        expect(artifacts).toHaveLength(1);
-        expect(artifacts[0]?.artifactKind).toBe('principle');
-    });
-    it('PRI-780 B2: missing BehaviorExamplePack fails the attempt LOUD before any LLM call', async () => {
-        const store = new MemoryPIArtifactStore();
-        await store.upsertArtifact(makeScribeArtifact());
-        // Task-driven constructors (consumer cycle, run-once) have no pack channel:
-        // the runner must refuse at invokeRuntime (pre-LLM), never degrade to v1.
-        const deps = createMockDeps({ artifactStore: store, behaviorExamplePack: undefined });
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('failed');
-        expect(JSON.stringify(result)).toContain('behavior_example_pack_missing');
-        // Pre-LLM proof: the runtime adapter was never invoked — no token spend.
-        const { startRun } = deps.runtimeAdapter;
-        expect(startRun).not.toHaveBeenCalled();
-    });
-    it('valid runtime output marks task succeeded', async () => {
-        const store = new MemoryPIArtifactStore();
-        await store.upsertArtifact(makeScribeArtifact());
-        const deps = createMockDeps({ artifactStore: store });
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('succeeded');
-        expect(deps.stateManager.markTaskSucceeded).toHaveBeenCalledWith(ARTIFICER_TASK_ID, expect.stringContaining('artificer://'));
-    });
-    it('invalid output does not write artifact', async () => {
-        const store = new MemoryPIArtifactStore();
-        await store.upsertArtifact(makeScribeArtifact());
-        const deps = createMockDeps({ artifactStore: store });
-        const invalidOutput = {
-            taskId: 'wrong-task-id',
-            sourceScribeArtifactId: '',
-            implementationCode: '',
-            goldenTraceCases: [],
-            affectedTools: [],
-            implementationSummary: '',
-            sourceTrace: {
-                scribeArtifactId: '',
-            },
-            risks: 'not-array',
-            generatedAt: '',
-        };
-        deps.runtimeAdapter.fetchOutput = vi.fn().mockResolvedValue({
-            payload: invalidOutput,
-        });
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('failed');
-        const artifacts = await store.listBySourceTaskId(ARTIFICER_TASK_ID);
-        expect(artifacts).toHaveLength(0);
-    });
-    it('artifact write failure goes to retry/fail, not mark succeeded', async () => {
-        const failingStore = {
-            listBySourceTaskId: vi.fn().mockResolvedValue([makeScribeArtifact()]),
-            upsertArtifact: vi.fn().mockRejectedValue(new Error('Disk full')),
-        };
-        const deps = createMockDeps({ artifactStore: failingStore });
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('failed');
-        expect(deps.stateManager.markTaskSucceeded).not.toHaveBeenCalled();
-    });
-    it('mismatched sourceScribeArtifactId echo is reconciled before artifact commit (PRI-541)', async () => {
-        const store = new MemoryPIArtifactStore();
-        await store.upsertArtifact(makeScribeArtifact());
-        const deps = createMockDeps({ artifactStore: store });
-        const mismatchedOutput = makeArtificerOutput();
-        mismatchedOutput.sourceScribeArtifactId = 'wrong-artifact-id';
-        mismatchedOutput.sourceTrace.scribeArtifactId = 'wrong-artifact-id';
-        deps.runtimeAdapter.fetchOutput = vi.fn().mockResolvedValue({
-            payload: mismatchedOutput,
-        });
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        // Lineage is runner-owned (rc-6): the corrupted echo is corrected from the
-        // task record before validation/commit instead of dead-ending (PRI-541).
-        expect(result.status).toBe('succeeded');
-        const artifacts = await store.listBySourceTaskId(ARTIFICER_TASK_ID);
-        expect(artifacts).toHaveLength(1);
-        expect(artifacts[0]?.contentJson).toContain('pi-art-scribe-001-run-001');
-        expect(deps.stateManager.markTaskSucceeded).toHaveBeenCalled();
-        const telemetryCalls = deps.eventEmitter.emitTelemetry.mock.calls;
-        const correctedEvent = telemetryCalls
-            .map((call) => call[0])
-            .find((evt) => evt.eventType === 'artificer_lineage_echo_corrected');
-        expect(correctedEvent).toBeDefined();
-        expect(correctedEvent?.payload.correctedFields).toEqual(expect.arrayContaining(['sourceScribeArtifactId', 'sourceTrace.scribeArtifactId']));
-    });
-});
-describe('ArtificerRunner.validateOutput — v2 mode-error errorCategory (CodeRabbit PR2 outside-diff)', () => {
-    const pack = {
-        sourceNegativeCase: {
-            caseId: 'negative-1', kind: 'negative', toolName: 'write_file',
-            params: { path: '/etc/passwd' }, expectedDecision: 'block',
-        },
-        ownerDesiredOutcome: 'block writes outside the workspace',
-        positiveCounterexamples: [{
-                caseId: 'positive-1', kind: 'positive', toolName: 'write_file',
-                params: { path: '/project/file.txt' }, expectedDecision: 'allow',
-            }],
-        evidenceRefs: ['pain://1'],
-        redactionNotes: [],
-    };
-    it('classifies modeErrors as output_invalid when base validator passes', async () => {
-        const deps = {
-            stateManager: {},
-            runtimeAdapter: {},
-            eventEmitter: {},
-            artifactStore: new MemoryPIArtifactStore(),
-            validator: new DefaultArtificerValidator(),
-            behaviorExamplePack: pack,
-        };
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        // PRI-780: the base validator passes, but an output that omits
-        // requiresContextVersion: 2 violates the v2-only contract.
-        const output = { ...makeArtificerOutput(), requiresContextVersion: undefined };
-        const context = {
-            contextHash: 'test-hash',
-            scribeArtifact: null,
-            sourceScribeArtifactId: null,
-            adversarialFeedback: null,
-            repairFeedback: null,
-        };
-        const result = await runner.validateOutput(output, ARTIFICER_TASK_ID, context);
-        expect(result.valid).toBe(false);
-        expect(result.errorCategory).toBe('output_invalid');
-        expect(result.errors.some(e => e.includes('requiresContextVersion'))).toBe(true);
-    });
-});
-describe('DefaultArtificerValidator (PRI-111)', () => {
-    const validator = new DefaultArtificerValidator();
-    it('accepts valid Artificer output', async () => {
-        const result = await validator.validate(makeArtificerOutput(), ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(true);
-    });
-    it('rejects taskId mismatch', async () => {
-        const output = makeArtificerOutput();
-        output.taskId = 'wrong';
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('taskId mismatch'))).toBe(true);
-    });
-    it('rejects missing sourceScribeArtifactId', async () => {
-        const output = makeArtificerOutput();
-        output.sourceScribeArtifactId = '';
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('sourceScribeArtifactId'))).toBe(true);
-    });
-    it('rejects mismatched sourceScribeArtifactId when expected is provided', async () => {
-        const output = makeArtificerOutput();
-        output.sourceScribeArtifactId = 'wrong-artifact-id';
-        const result = await validator.validate(output, ARTIFICER_TASK_ID, 'pi-art-scribe-001-run-001');
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('sourceScribeArtifactId mismatch'))).toBe(true);
-    });
-    it('rejects mismatched sourceTrace.scribeArtifactId when expected is provided', async () => {
-        const output = makeArtificerOutput();
-        output.sourceTrace.scribeArtifactId = 'wrong-artifact-id';
-        const result = await validator.validate(output, ARTIFICER_TASK_ID, 'pi-art-scribe-001-run-001');
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('sourceTrace.scribeArtifactId mismatch'))).toBe(true);
-    });
-    it('rejects null output', async () => {
-        const result = await validator.validate(null, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-    });
-    it('rejects missing implementationCode (unified output requires code)', async () => {
-        const output = makeArtificerOutput();
-        output.implementationCode = '';
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('implementationCode'))).toBe(true);
-    });
-    it('rejects missing implementationSummary', async () => {
-        const output = makeArtificerOutput();
-        output.implementationSummary = '';
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('implementationSummary'))).toBe(true);
-    });
-    it('rejects empty affectedTools array', async () => {
-        const output = makeArtificerOutput();
-        output.affectedTools = [];
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('affectedTools'))).toBe(true);
-    });
-    it('rejects risks with non-string elements', async () => {
-        const output = makeArtificerOutput();
-        output.risks = [42];
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('risks must be an array of strings'))).toBe(true);
-    });
-    it('rejects missing sourceTrace.scribeArtifactId', async () => {
-        const output = makeArtificerOutput();
-        output.sourceTrace.scribeArtifactId = '';
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('sourceTrace.scribeArtifactId'))).toBe(true);
-    });
-    it('accepts valid output with matching expectedSourceScribeArtifactId', async () => {
-        const output = makeArtificerOutput();
-        const result = await validator.validate(output, ARTIFICER_TASK_ID, 'pi-art-scribe-001-run-001');
-        expect(result.valid).toBe(true);
-    });
-    it('rejects non-string philosopherArtifactId in sourceTrace', async () => {
-        const output = makeArtificerOutput();
-        output.sourceTrace.philosopherArtifactId = 42;
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('philosopherArtifactId'))).toBe(true);
-    });
-    it('rejects non-string dreamerArtifactId in sourceTrace', async () => {
-        const output = makeArtificerOutput();
-        output.sourceTrace.dreamerArtifactId = { evil: true };
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('dreamerArtifactId'))).toBe(true);
-    });
-    it('rejects mismatched sourceScribeArtifactId vs sourceTrace.scribeArtifactId', async () => {
-        const output = makeArtificerOutput();
-        output.sourceTrace.scribeArtifactId = 'different-id';
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('must match'))).toBe(true);
-    });
-    it('rejects prototype-inherited taskId (ERR-013)', async () => {
-        const proto = { taskId: ARTIFICER_TASK_ID };
-        const output = Object.create(proto);
-        // Copy all own properties from a valid output except taskId
-        const valid = makeArtificerOutput();
-        Object.assign(output, { ...valid, taskId: undefined });
-        delete output.taskId;
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('taskId'))).toBe(true);
-    });
-    it('rejects prototype-inherited sourceScribeArtifactId (ERR-013)', async () => {
-        const output = makeArtificerOutput();
-        const ownValue = output.sourceScribeArtifactId;
-        delete output.sourceScribeArtifactId;
-        Object.setPrototypeOf(output, { sourceScribeArtifactId: ownValue });
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('sourceScribeArtifactId'))).toBe(true);
-    });
-    it('rejects prototype-inherited sourceTrace.scribeArtifactId (ERR-013)', async () => {
-        const output = makeArtificerOutput();
-        const ownValue = output.sourceTrace.scribeArtifactId;
-        delete output.sourceTrace.scribeArtifactId;
-        Object.setPrototypeOf(output.sourceTrace, { scribeArtifactId: ownValue });
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('sourceTrace.scribeArtifactId'))).toBe(true);
-    });
-    it('rejects prototype-inherited implementationSummary (ERR-013)', async () => {
-        const output = makeArtificerOutput();
-        const ownValue = output.implementationSummary;
-        delete output.implementationSummary;
-        Object.setPrototypeOf(output, { implementationSummary: ownValue });
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('implementationSummary'))).toBe(true);
-    });
-    it('rejects empty string sourceTrace.philosopherArtifactId when present', async () => {
-        const output = makeArtificerOutput();
-        output.sourceTrace.philosopherArtifactId = '';
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('philosopherArtifactId'))).toBe(true);
-    });
-    it('rejects empty string sourceTrace.dreamerArtifactId when present', async () => {
-        const output = makeArtificerOutput();
-        output.sourceTrace.dreamerArtifactId = '';
-        const result = await validator.validate(output, ARTIFICER_TASK_ID);
-        expect(result.valid).toBe(false);
-        expect(result.errors.some(e => e.includes('dreamerArtifactId'))).toBe(true);
-    });
-});
-describe('ArtificerRunner integration: test-double captures sourceScribeArtifactId from prompt', () => {
-    it('seed scribe artifact -> run artificer with test-double -> succeeded with correct sourceScribeArtifactId', async () => {
-        const SCRIBE_ART_ID = 'pi-art-scribe-real-001';
-        const artifactStore = new MemoryPIArtifactStore();
-        const scribeArtifact = {
-            artifactId: SCRIBE_ART_ID,
-            artifactKind: 'principle',
-            sourceTaskId: SCRIBE_TASK_ID,
-            lineageArtifactIds: [],
-            validationStatus: 'pending',
-            contentJson: JSON.stringify({
-                taskId: SCRIBE_TASK_ID,
-                sourcePhilosopherArtifactId: 'pi-art-philosopher-001',
-                principleDraft: {
-                    title: 'Systematic Error Handling',
-                    statement: 'All async operations must include explicit error handling',
-                    rationale: 'Uncaught errors cascade into system instability',
-                    applicability: ['All async operations'],
-                    antiPatterns: ['Ignoring promise rejections'],
-                    confidence: 0.9,
-                },
-                sourceTrace: { philosopherArtifactId: 'pi-art-philosopher-001' },
-                risks: [],
-                generatedAt: new Date().toISOString(),
-            }),
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-        };
-        await artifactStore.upsertArtifact(scribeArtifact);
-        const artificerTask = makeArtificerTask();
-        let capturedSourceScribeArtifactId = SCRIBE_ART_ID;
-        const runtimeAdapter = new TestDoubleRuntimeAdapter({
-            onStartRun: (input) => {
-                try {
-                    const payloadStr = typeof input.inputPayload === 'string' ? input.inputPayload : JSON.stringify(input.inputPayload);
-                    const parsed = JSON.parse(payloadStr);
-                    if (typeof parsed.sourceScribeArtifactId === 'string' && parsed.sourceScribeArtifactId.trim() !== '') {
-                        capturedSourceScribeArtifactId = parsed.sourceScribeArtifactId;
-                    }
-                }
-                catch { /* ignore */ }
-                return { runId: 'run-integration-001', runtimeKind: 'test-double', startedAt: new Date().toISOString() };
-            },
-            onPollRun: (_runId) => ({
-                runId: _runId,
-                status: 'succeeded',
-                startedAt: new Date().toISOString(),
-                endedAt: new Date().toISOString(),
-            }),
-            onFetchOutput: (_runId) => ({
-                runId: _runId,
-                payload: {
-                    taskId: ARTIFICER_TASK_ID,
-                    sourceScribeArtifactId: capturedSourceScribeArtifactId,
-                    requiresContextVersion: 2,
-                    evidenceRefs: ['pain://1'],
-                    implementationCode: 'function evaluate(input, helpers) { return { decision: "allow", matched: false, reason: "ok" }; }',
-                    goldenTraceCases: [
-                        { caseId: 'positive-1', kind: 'positive', toolName: 'write_file', params: { path: '/project/file.txt' }, expectedDecision: 'allow', ruleContext: V2_RULE_CONTEXT },
-                        { caseId: 'negative-1', kind: 'negative', toolName: 'write_file', params: { path: '/etc/passwd' }, expectedDecision: 'block', ruleContext: V2_RULE_CONTEXT },
-                    ],
-                    affectedTools: ['write_file'],
-                    implementationSummary: 'Add input validation to all async operations',
-                    sourceTrace: {
-                        scribeArtifactId: capturedSourceScribeArtifactId,
-                    },
-                    risks: ['May add latency from error checking'],
-                    generatedAt: new Date().toISOString(),
-                },
-            }),
-        });
-        const stateManager = {
-            acquireLease: vi.fn().mockResolvedValue(artificerTask),
-            getTask: vi.fn().mockImplementation((id) => {
-                if (id === ARTIFICER_TASK_ID)
-                    return Promise.resolve(artificerTask);
-                if (id === SCRIBE_TASK_ID)
-                    return Promise.resolve(makeScribeTask());
-                return Promise.resolve(null);
-            }),
-            getRunsByTask: vi.fn().mockResolvedValue([{
-                    runId: 'run-integration-001',
-                    taskId: ARTIFICER_TASK_ID,
-                    runtimeKind: 'artificer',
-                    startedAt: new Date().toISOString(),
-                }]),
-            getValidRunsByTaskTolerant: vi.fn().mockResolvedValue({
-                runs: [{ runId: 'run-integration-001', taskId: ARTIFICER_TASK_ID, runtimeKind: 'artificer', startedAt: new Date().toISOString() }],
-                degradedRuns: [],
-            }),
-            updateRunOutput: vi.fn().mockResolvedValue(undefined),
-            markTaskSucceeded: vi.fn().mockResolvedValue(undefined),
-            markTaskFailed: vi.fn().mockResolvedValue(undefined),
-            markTaskRetryWait: vi.fn().mockResolvedValue(undefined),
-            getRetryPolicy: vi.fn().mockReturnValue({ shouldRetry: () => false }),
-        };
-        const eventEmitter = {
-            emitTelemetry: vi.fn(),
-        };
-        const deps = {
-            stateManager,
-            runtimeAdapter,
-            eventEmitter,
-            validator: new DefaultArtificerValidator(),
-            artifactStore,
-            behaviorExamplePack: V2_TEST_PACK,
-        };
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test-integration',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 5000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID);
-        expect(result.status).toBe('succeeded');
-        expect(capturedSourceScribeArtifactId).toBe(SCRIBE_ART_ID);
-        expect(result.output?.sourceScribeArtifactId).toBe(SCRIBE_ART_ID);
-        expect(result.output?.sourceTrace.scribeArtifactId).toBe(SCRIBE_ART_ID);
-        const artifacts = await artifactStore.listBySourceTaskId(ARTIFICER_TASK_ID);
-        expect(artifacts).toHaveLength(1);
-        expect(artifacts[0]?.artifactKind).toBe('principle');
-        const [storedArtifact] = artifacts;
-        expect(storedArtifact).toBeDefined();
-        if (!storedArtifact)
-            return;
-        const storedOutput = JSON.parse(storedArtifact.contentJson);
-        expect(storedOutput.sourceScribeArtifactId).toBe(SCRIBE_ART_ID);
-        expect(storedOutput.sourceTrace.scribeArtifactId).toBe(SCRIBE_ART_ID);
-    });
-});
-describe('PRI-508: ArtificerRunner.buildContext reads dreamer artifact via scribe.sourceTrace.dreamerArtifactId', () => {
-    // Vertical slice 4: end-to-end — dreamer artifact in store → prompt contains dreamerContext 5维字段
-    const SCRIBE_ART_ID = 'pi-art-scribe-pri508';
-    const DREAMER_ART_ID = 'pi-art-dreamer-pri508';
-    const SCRIBE_TASK_ID_PRI508 = 'scribe-pri508';
-    const ARTIFICER_TASK_ID_PRI508 = 'artificer-pri508';
-    function makeScribeArtifactWithDreamer() {
-        return {
-            artifactId: SCRIBE_ART_ID,
-            artifactKind: 'principle',
-            sourceTaskId: SCRIBE_TASK_ID_PRI508,
-            lineageArtifactIds: [DREAMER_ART_ID],
-            validationStatus: 'pending',
-            contentJson: JSON.stringify({
-                taskId: SCRIBE_TASK_ID_PRI508,
-                sourcePhilosopherArtifactId: 'pi-art-philosopher-pri508',
-                principleDraft: {
-                    title: 'Path Traversal Guard',
-                    statement: 'Validate parent path before write_file',
-                    rationale: 'Unchecked parent path enables path traversal',
-                    applicability: ['write_file'],
-                    antiPatterns: ['Trusting raw user input path'],
-                    confidence: 0.9,
-                },
-                sourceTrace: {
-                    philosopherArtifactId: 'pi-art-philosopher-pri508',
-                    dreamerArtifactId: DREAMER_ART_ID,
-                },
-                risks: [],
-                generatedAt: new Date().toISOString(),
-            }),
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-        };
-    }
-    function makeDreamerArtifact() {
-        return {
-            artifactId: DREAMER_ART_ID,
-            artifactKind: 'principle',
-            sourceTaskId: 'dreamer-task-pri508',
-            lineageArtifactIds: [],
-            validationStatus: 'pending',
-            contentJson: JSON.stringify({
-                valid: true,
-                taskId: 'dreamer-task-pri508',
-                candidates: [
-                    {
-                        candidateIndex: 0,
-                        badDecision: 'agent called write_file without resolving parent path',
-                        betterDecision: 'agent must resolve and validate parent path before write_file',
-                        rationale: 'parent path resolution prevents path traversal exploits',
-                        confidence: 0.85,
-                        riskLevel: 'medium',
-                        strategicPerspective: 'proactive validation beats reactive cleanup',
-                    },
-                ],
-                contextRefs: [],
-                generatedAt: new Date().toISOString(),
-            }),
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-        };
-    }
-    function makeScribeTaskPri508() {
-        return {
-            taskId: SCRIBE_TASK_ID_PRI508,
-            taskKind: 'scribe',
-            status: 'succeeded',
-            attemptCount: 1,
-            maxAttempts: 3,
-            resultRef: 'scribe://run-pri508',
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-            diagnosticJson: createPITaskDiagnosticJson({
-                dependencyTaskIds: [],
-                channel: 'prompt',
-                timeoutMs: 300_000,
-                inputArtifactRefs: [],
-                outputArtifactRefs: [{ artifactType: 'principle', ref: SCRIBE_ART_ID }],
-            }),
-        };
-    }
-    function makeArtificerTaskPri508() {
-        return {
-            taskId: ARTIFICER_TASK_ID_PRI508,
-            taskKind: 'artificer',
-            status: 'pending',
-            attemptCount: 0,
-            maxAttempts: 3,
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-            diagnosticJson: createPITaskDiagnosticJson({
-                dependencyTaskIds: [SCRIBE_TASK_ID_PRI508],
-                channel: 'prompt',
-                timeoutMs: 300_000,
-                inputArtifactRefs: [{ artifactType: 'principle', ref: SCRIBE_ART_ID }],
-                outputArtifactRefs: [],
-            }),
-        };
-    }
-    function makeArtificerOutputPri508() {
-        return {
-            taskId: ARTIFICER_TASK_ID_PRI508,
-            sourceScribeArtifactId: SCRIBE_ART_ID,
-            requiresContextVersion: 2,
-            evidenceRefs: ['pain://1'],
-            implementationCode: 'function evaluate(input, helpers) { return { decision: "allow", matched: false, reason: "ok" }; }',
-            goldenTraceCases: [
-                { caseId: 'positive-1', kind: 'positive', toolName: 'write_file', params: { path: '/project/file.txt' }, expectedDecision: 'allow', ruleContext: V2_RULE_CONTEXT },
-                { caseId: 'negative-1', kind: 'negative', toolName: 'write_file', params: { path: '/etc/passwd' }, expectedDecision: 'block', ruleContext: V2_RULE_CONTEXT },
-            ],
-            affectedTools: ['write_file'],
-            implementationSummary: 'Validate parent path before write',
-            sourceTrace: {
-                scribeArtifactId: SCRIBE_ART_ID,
-            },
-            risks: [],
-            generatedAt: new Date().toISOString(),
-        };
-    }
-    it('dreamerContext 5维字段透传到 artificer prompt (end-to-end)', async () => {
-        const artifactStore = new MemoryPIArtifactStore();
-        await artifactStore.upsertArtifact(makeScribeArtifactWithDreamer());
-        await artifactStore.upsertArtifact(makeDreamerArtifact());
-        const artificerTask = makeArtificerTaskPri508();
-        const scribeTask = makeScribeTaskPri508();
-        const stateManager = {
-            acquireLease: vi.fn().mockResolvedValue(artificerTask),
-            getTask: vi.fn().mockImplementation((id) => {
-                if (id === ARTIFICER_TASK_ID_PRI508)
-                    return Promise.resolve(artificerTask);
-                if (id === SCRIBE_TASK_ID_PRI508)
-                    return Promise.resolve(scribeTask);
-                return Promise.resolve(null);
-            }),
-            getRunsByTask: vi.fn().mockResolvedValue([{
-                    runId: 'run-pri508',
-                    taskId: ARTIFICER_TASK_ID_PRI508,
-                    runtimeKind: 'artificer',
-                    startedAt: new Date().toISOString(),
-                }]),
-            getValidRunsByTaskTolerant: vi.fn().mockResolvedValue({
-                runs: [{ runId: 'run-pri508', taskId: ARTIFICER_TASK_ID_PRI508, runtimeKind: 'artificer', startedAt: new Date().toISOString() }],
-                degradedRuns: [],
-            }),
-            updateRunOutput: vi.fn().mockResolvedValue(undefined),
-            markTaskSucceeded: vi.fn().mockResolvedValue(undefined),
-            markTaskFailed: vi.fn().mockResolvedValue(undefined),
-            markTaskRetryWait: vi.fn().mockResolvedValue(undefined),
-            getRetryPolicy: vi.fn().mockReturnValue({ shouldRetry: () => false }),
-        };
-        let capturedPrompt;
-        const runtimeAdapter = {
-            startRun: vi.fn().mockImplementation((input) => {
-                capturedPrompt = typeof input.inputPayload === 'string' ? input.inputPayload : JSON.stringify(input.inputPayload);
-                return Promise.resolve({ runId: 'run-pri508', runtimeKind: 'test-double', startedAt: new Date().toISOString() });
-            }),
-            pollRun: vi.fn().mockResolvedValue({ status: 'succeeded', runId: 'run-pri508' }),
-            fetchOutput: vi.fn().mockResolvedValue({ payload: makeArtificerOutputPri508() }),
-            cancelRun: vi.fn().mockResolvedValue(undefined),
-        };
-        const eventEmitter = { emitTelemetry: vi.fn() };
-        const validator = new DefaultArtificerValidator();
-        const deps = {
-            stateManager,
-            runtimeAdapter,
-            eventEmitter,
-            artifactStore,
-            validator,
-            behaviorExamplePack: V2_TEST_PACK,
-        };
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID_PRI508);
-        expect(result.status).toBe('succeeded');
-        // Assert the prompt captured at startRun contains the dreamer candidate set
-        expect(capturedPrompt).toBeDefined();
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.dreamerContext).toBeDefined();
-        expect(parsed.dreamerContext.candidates).toHaveLength(1);
-        expect(parsed.dreamerContext.candidates[0].badDecision).toBe('agent called write_file without resolving parent path');
-        expect(parsed.dreamerContext.candidates[0].betterDecision).toBe('agent must resolve and validate parent path before write_file');
-        expect(parsed.dreamerContext.candidates[0].rationale).toBe('parent path resolution prevents path traversal exploits');
-        expect(parsed.dreamerContext.candidates[0].riskLevel).toBe('medium');
-        expect(parsed.dreamerContext.candidates[0].strategicPerspective).toBe('proactive validation beats reactive cleanup');
-        expect(parsed.dreamerContext.omittedCandidateCount).toBe(0);
-    });
-    it('dreamerArtifactId 缺失时 dreamerContext undefined (向后兼容)', async () => {
-        // scribe artifact 不带 sourceTrace.dreamerArtifactId → dreamerContext 不出现在 prompt
-        const artifactStore = new MemoryPIArtifactStore();
-        const scribeArtifactNoDreamer = {
-            artifactId: SCRIBE_ART_ID,
-            artifactKind: 'principle',
-            sourceTaskId: SCRIBE_TASK_ID_PRI508,
-            lineageArtifactIds: [],
-            validationStatus: 'pending',
-            contentJson: JSON.stringify({
-                taskId: SCRIBE_TASK_ID_PRI508,
-                sourcePhilosopherArtifactId: 'pi-art-philosopher-pri508',
-                principleDraft: {
-                    title: 'Path Traversal Guard',
-                    statement: 'Validate parent path before write_file',
-                    rationale: 'Unchecked parent path enables path traversal',
-                    applicability: ['write_file'],
-                    antiPatterns: ['Trusting raw user input path'],
-                    confidence: 0.9,
-                },
-                sourceTrace: {
-                    philosopherArtifactId: 'pi-art-philosopher-pri508',
-                    // dreamerArtifactId 刻意缺失
-                },
-                risks: [],
-                generatedAt: new Date().toISOString(),
-            }),
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-        };
-        await artifactStore.upsertArtifact(scribeArtifactNoDreamer);
-        const artificerTask = makeArtificerTaskPri508();
-        const scribeTask = makeScribeTaskPri508();
-        const stateManager = {
-            acquireLease: vi.fn().mockResolvedValue(artificerTask),
-            getTask: vi.fn().mockImplementation((id) => {
-                if (id === ARTIFICER_TASK_ID_PRI508)
-                    return Promise.resolve(artificerTask);
-                if (id === SCRIBE_TASK_ID_PRI508)
-                    return Promise.resolve(scribeTask);
-                return Promise.resolve(null);
-            }),
-            getRunsByTask: vi.fn().mockResolvedValue([{ runId: 'run-pri508-nodreamer', taskId: ARTIFICER_TASK_ID_PRI508, runtimeKind: 'artificer', startedAt: new Date().toISOString() }]),
-            getValidRunsByTaskTolerant: vi.fn().mockResolvedValue({ runs: [{ runId: 'run-pri508-nodreamer', taskId: ARTIFICER_TASK_ID_PRI508, runtimeKind: 'artificer', startedAt: new Date().toISOString() }], degradedRuns: [] }),
-            updateRunOutput: vi.fn().mockResolvedValue(undefined),
-            markTaskSucceeded: vi.fn().mockResolvedValue(undefined),
-            markTaskFailed: vi.fn().mockResolvedValue(undefined),
-            markTaskRetryWait: vi.fn().mockResolvedValue(undefined),
-            getRetryPolicy: vi.fn().mockReturnValue({ shouldRetry: () => false }),
-        };
-        let capturedPrompt;
-        const runtimeAdapter = {
-            startRun: vi.fn().mockImplementation((input) => {
-                capturedPrompt = typeof input.inputPayload === 'string' ? input.inputPayload : JSON.stringify(input.inputPayload);
-                return Promise.resolve({ runId: 'run-pri508-nodreamer', runtimeKind: 'test-double', startedAt: new Date().toISOString() });
-            }),
-            pollRun: vi.fn().mockResolvedValue({ status: 'succeeded', runId: 'run-pri508-nodreamer' }),
-            fetchOutput: vi.fn().mockResolvedValue({ payload: makeArtificerOutputPri508() }),
-            cancelRun: vi.fn().mockResolvedValue(undefined),
-        };
-        const eventEmitter = { emitTelemetry: vi.fn() };
-        const validator = new DefaultArtificerValidator();
-        const deps = {
-            stateManager,
-            runtimeAdapter,
-            eventEmitter,
-            artifactStore,
-            validator,
-            behaviorExamplePack: V2_TEST_PACK,
-        };
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID_PRI508);
-        expect(result.status).toBe('succeeded');
-        expect(capturedPrompt).toBeDefined();
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.dreamerContext).toBeUndefined();
-    });
-    it('dreamerArtifactId 存在但 artifact 查不到时 emit event + dreamerContext undefined (rc-9)', async () => {
-        // scribe 带 dreamerArtifactId 但 store 里没有 dreamer artifact → 不能静默成功，要有可观察事件
-        const artifactStore = new MemoryPIArtifactStore();
-        await artifactStore.upsertArtifact(makeScribeArtifactWithDreamer());
-        // 故意不 upsert dreamer artifact
-        const artificerTask = makeArtificerTaskPri508();
-        const scribeTask = makeScribeTaskPri508();
-        const stateManager = {
-            acquireLease: vi.fn().mockResolvedValue(artificerTask),
-            getTask: vi.fn().mockImplementation((id) => {
-                if (id === ARTIFICER_TASK_ID_PRI508)
-                    return Promise.resolve(artificerTask);
-                if (id === SCRIBE_TASK_ID_PRI508)
-                    return Promise.resolve(scribeTask);
-                return Promise.resolve(null);
-            }),
-            getRunsByTask: vi.fn().mockResolvedValue([{ runId: 'run-pri508-missing', taskId: ARTIFICER_TASK_ID_PRI508, runtimeKind: 'artificer', startedAt: new Date().toISOString() }]),
-            getValidRunsByTaskTolerant: vi.fn().mockResolvedValue({ runs: [{ runId: 'run-pri508-missing', taskId: ARTIFICER_TASK_ID_PRI508, runtimeKind: 'artificer', startedAt: new Date().toISOString() }], degradedRuns: [] }),
-            updateRunOutput: vi.fn().mockResolvedValue(undefined),
-            markTaskSucceeded: vi.fn().mockResolvedValue(undefined),
-            markTaskFailed: vi.fn().mockResolvedValue(undefined),
-            markTaskRetryWait: vi.fn().mockResolvedValue(undefined),
-            getRetryPolicy: vi.fn().mockReturnValue({ shouldRetry: () => false }),
-        };
-        let capturedPrompt;
-        const runtimeAdapter = {
-            startRun: vi.fn().mockImplementation((input) => {
-                capturedPrompt = typeof input.inputPayload === 'string' ? input.inputPayload : JSON.stringify(input.inputPayload);
-                return Promise.resolve({ runId: 'run-pri508-missing', runtimeKind: 'test-double', startedAt: new Date().toISOString() });
-            }),
-            pollRun: vi.fn().mockResolvedValue({ status: 'succeeded', runId: 'run-pri508-missing' }),
-            fetchOutput: vi.fn().mockResolvedValue({ payload: makeArtificerOutputPri508() }),
-            cancelRun: vi.fn().mockResolvedValue(undefined),
-        };
-        const emitTelemetry = vi.fn();
-        const eventEmitter = { emitTelemetry };
-        const validator = new DefaultArtificerValidator();
-        const deps = {
-            stateManager,
-            runtimeAdapter,
-            eventEmitter,
-            artifactStore,
-            validator,
-            behaviorExamplePack: V2_TEST_PACK,
-        };
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID_PRI508);
-        // 仍应成功（dreamerContext 是 best-effort，不阻塞主流程）
-        expect(result.status).toBe('succeeded');
-        // rc-9: 不能静默 — 必须有可观察事件
-        // emitTelemetry 接收单个结构化对象 (eventType + traceId + payload + ...)
-        expect(emitTelemetry).toHaveBeenCalledWith(expect.objectContaining({
-            eventType: expect.stringContaining('dreamer'),
-            traceId: ARTIFICER_TASK_ID_PRI508,
-        }));
-        expect(capturedPrompt).toBeDefined();
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.dreamerContext).toBeUndefined();
-    });
-    // ── PRI-508 edge cases: cover resolveDreamerContext() failure branches (rc-9) ──
-    // Each case asserts: (a) runner still succeeds (best-effort, non-blocking),
-    // (b) dreamerContext undefined in prompt, (c) emitTelemetry called with the
-    // expected observable event name so the gap is not silent.
-    /**
-     * Shared harness for PRI-508 edge-case tests. Accepts a custom scribe artifact
-     * and optional dreamer artifact, runs the artificer end-to-end, and returns
-     * the captured prompt + telemetry spy so each case can assert behavior.
-     */
-    async function runPri508EdgeCase(params) {
-        const artifactStore = new MemoryPIArtifactStore();
-        await artifactStore.upsertArtifact(params.scribeArtifact);
-        if (params.dreamerArtifact) {
-            await artifactStore.upsertArtifact(params.dreamerArtifact);
-        }
-        const artificerTask = makeArtificerTaskPri508();
-        const scribeTask = makeScribeTaskPri508();
-        const stateManager = {
-            acquireLease: vi.fn().mockResolvedValue(artificerTask),
-            getTask: vi.fn().mockImplementation((id) => {
-                if (id === ARTIFICER_TASK_ID_PRI508)
-                    return Promise.resolve(artificerTask);
-                if (id === SCRIBE_TASK_ID_PRI508)
-                    return Promise.resolve(scribeTask);
-                return Promise.resolve(null);
-            }),
-            getRunsByTask: vi.fn().mockResolvedValue([{ runId: 'run-pri508-edge', taskId: ARTIFICER_TASK_ID_PRI508, runtimeKind: 'artificer', startedAt: new Date().toISOString() }]),
-            getValidRunsByTaskTolerant: vi.fn().mockResolvedValue({ runs: [{ runId: 'run-pri508-edge', taskId: ARTIFICER_TASK_ID_PRI508, runtimeKind: 'artificer', startedAt: new Date().toISOString() }], degradedRuns: [] }),
-            updateRunOutput: vi.fn().mockResolvedValue(undefined),
-            markTaskSucceeded: vi.fn().mockResolvedValue(undefined),
-            markTaskFailed: vi.fn().mockResolvedValue(undefined),
-            markTaskRetryWait: vi.fn().mockResolvedValue(undefined),
-            getRetryPolicy: vi.fn().mockReturnValue({ shouldRetry: () => false }),
-        };
-        let capturedPrompt;
-        const runtimeAdapter = {
-            startRun: vi.fn().mockImplementation((input) => {
-                capturedPrompt = typeof input.inputPayload === 'string' ? input.inputPayload : JSON.stringify(input.inputPayload);
-                return Promise.resolve({ runId: 'run-pri508-edge', runtimeKind: 'test-double', startedAt: new Date().toISOString() });
-            }),
-            pollRun: vi.fn().mockResolvedValue({ status: 'succeeded', runId: 'run-pri508-edge' }),
-            fetchOutput: vi.fn().mockResolvedValue({ payload: makeArtificerOutputPri508() }),
-            cancelRun: vi.fn().mockResolvedValue(undefined),
-        };
-        const emitTelemetry = vi.fn();
-        const eventEmitter = { emitTelemetry };
-        const validator = new DefaultArtificerValidator();
-        const deps = {
-            stateManager,
-            runtimeAdapter,
-            eventEmitter,
-            artifactStore,
-            validator,
-            behaviorExamplePack: V2_TEST_PACK,
-        };
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID_PRI508);
-        return { result, capturedPrompt, emitTelemetry };
-    }
-    /** Build a scribe artifact with a custom contentJson string. */
-    function makeScribeWithContentJson(contentJson) {
-        return {
-            artifactId: SCRIBE_ART_ID,
-            artifactKind: 'principle',
-            sourceTaskId: SCRIBE_TASK_ID_PRI508,
-            lineageArtifactIds: [],
-            validationStatus: 'pending',
-            contentJson,
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-        };
-    }
-    /** Build a scribe artifact whose sourceTrace.dreamerArtifactId points to DREAMER_ART_ID. */
-    function makeScribeWithDreamerRef() {
-        return makeScribeWithContentJson(JSON.stringify({
-            taskId: SCRIBE_TASK_ID_PRI508,
-            sourcePhilosopherArtifactId: 'pi-art-philosopher-pri508',
-            principleDraft: { title: 'T', statement: 'S', rationale: 'R', applicability: [], antiPatterns: [], confidence: 0.5 },
-            sourceTrace: { philosopherArtifactId: 'pi-art-philosopher-pri508', dreamerArtifactId: DREAMER_ART_ID },
-            risks: [],
-            generatedAt: new Date().toISOString(),
-        }));
-    }
-    /** Build a dreamer artifact with a custom contentJson string. */
-    function makeDreamerWithContentJson(contentJson) {
-        return {
-            artifactId: DREAMER_ART_ID,
-            artifactKind: 'principle',
-            sourceTaskId: 'dreamer-task-pri508',
-            lineageArtifactIds: [],
-            validationStatus: 'pending',
-            contentJson,
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-        };
-    }
-    it.each([
-        {
-            name: 'scribe contentJson unparseable → emit dreamer_context_skipped',
-            scribeContentJson: '{not valid json',
-            expectedEvent: 'dreamer_context_skipped',
-        },
-        {
-            name: 'scribe contentJson not a record (array) → emit dreamer_context_skipped (PRI-816 rc-9)',
-            scribeContentJson: JSON.stringify(['not', 'a', 'record']),
-            expectedEvent: 'dreamer_context_skipped',
-        },
-        {
-            name: 'scribe.sourceTrace missing → emit dreamer_context_skipped (PRI-816 rc-9; backward-compat skip is now observable)',
-            scribeContentJson: JSON.stringify({ taskId: SCRIBE_TASK_ID_PRI508, principleDraft: { statement: 'S' } }),
-            expectedEvent: 'dreamer_context_skipped',
-        },
-        {
-            name: 'scribe.sourceTrace not a record (string) → emit dreamer_context_skipped (PRI-816 rc-9)',
-            scribeContentJson: JSON.stringify({ sourceTrace: 'not-a-record' }),
-            expectedEvent: 'dreamer_context_skipped',
-        },
-        {
-            name: 'sourceTrace.dreamerArtifactId missing → emit dreamer_context_skipped (PRI-816 rc-9)',
-            scribeContentJson: JSON.stringify({ sourceTrace: { philosopherArtifactId: 'pi-art-philosopher-pri508' } }),
-            expectedEvent: 'dreamer_context_skipped',
-        },
-        {
-            name: 'sourceTrace.dreamerArtifactId is undefined → emit dreamer_context_skipped (PRI-816 rc-9)',
-            scribeContentJson: JSON.stringify({ sourceTrace: { dreamerArtifactId: undefined } }),
-            expectedEvent: 'dreamer_context_skipped',
-        },
-        {
-            name: 'sourceTrace.dreamerArtifactId not a string (object) → emit dreamer_context_skipped',
-            scribeContentJson: JSON.stringify({ sourceTrace: { dreamerArtifactId: { evil: true } } }),
-            expectedEvent: 'dreamer_context_skipped',
-        },
-        {
-            name: 'sourceTrace.dreamerArtifactId empty string → emit dreamer_context_skipped',
-            scribeContentJson: JSON.stringify({ sourceTrace: { dreamerArtifactId: '' } }),
-            expectedEvent: 'dreamer_context_skipped',
-        },
-    ])('edge: $name', async ({ scribeContentJson, expectedEvent }) => {
-        const { result, capturedPrompt, emitTelemetry } = await runPri508EdgeCase({
-            scribeArtifact: makeScribeWithContentJson(scribeContentJson),
-        });
-        // Best-effort: runner still succeeds even when dreamer context cannot be resolved.
-        expect(result.status).toBe('succeeded');
-        expect(capturedPrompt).toBeDefined();
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.dreamerContext).toBeUndefined();
-        if (expectedEvent !== null) {
-            // rc-9: failure must be observable, not silent.
-            expect(emitTelemetry).toHaveBeenCalledWith(expect.objectContaining({
-                eventType: expect.stringContaining(expectedEvent),
-                traceId: ARTIFICER_TASK_ID_PRI508,
-            }));
-        }
-    });
-    it.each([
-        {
-            name: 'dreamer contentJson unparseable → emit dreamer_context_invalid',
-            dreamerContentJson: '{broken json',
-            expectedEvent: 'dreamer_context_invalid',
-        },
-        {
-            name: 'dreamer contentJson not a record (array) → emit dreamer_context_invalid',
-            dreamerContentJson: JSON.stringify(['not', 'a', 'record']),
-            expectedEvent: 'dreamer_context_invalid',
-        },
-        {
-            name: 'dreamer candidates missing → emit dreamer_context_invalid',
-            dreamerContentJson: JSON.stringify({ valid: true, taskId: 'dreamer-task-pri508' }),
-            expectedEvent: 'dreamer_context_invalid',
-        },
-        {
-            name: 'dreamer candidates not an array (string) → emit dreamer_context_invalid',
-            dreamerContentJson: JSON.stringify({ candidates: 'not-an-array' }),
-            expectedEvent: 'dreamer_context_invalid',
-        },
-        {
-            name: 'dreamer candidates empty array → emit dreamer_context_invalid',
-            dreamerContentJson: JSON.stringify({ candidates: [] }),
-            expectedEvent: 'dreamer_context_invalid',
-        },
-        {
-            name: 'dreamer candidates[0] not a record (string) → emit dreamer_context_invalid',
-            dreamerContentJson: JSON.stringify({ candidates: ['not-a-record'] }),
-            expectedEvent: 'dreamer_context_invalid',
-        },
-        {
-            name: 'dreamer candidates[0] missing badDecision → emit dreamer_context_invalid',
-            dreamerContentJson: JSON.stringify({ candidates: [{ betterDecision: 'b', rationale: 'r' }] }),
-            expectedEvent: 'dreamer_context_invalid',
-        },
-        {
-            name: 'dreamer candidates[0] missing betterDecision → emit dreamer_context_invalid',
-            dreamerContentJson: JSON.stringify({ candidates: [{ badDecision: 'a', rationale: 'r' }] }),
-            expectedEvent: 'dreamer_context_invalid',
-        },
-        {
-            name: 'dreamer candidates[0] missing rationale → emit dreamer_context_invalid',
-            dreamerContentJson: JSON.stringify({ candidates: [{ badDecision: 'a', betterDecision: 'b' }] }),
-            expectedEvent: 'dreamer_context_invalid',
-        },
-        {
-            name: 'dreamer candidates[0] badDecision empty string → emit dreamer_context_invalid',
-            dreamerContentJson: JSON.stringify({ candidates: [{ badDecision: '   ', betterDecision: 'b', rationale: 'r' }] }),
-            expectedEvent: 'dreamer_context_invalid',
-        },
-        {
-            name: 'dreamer candidates[0] badDecision not a string (number) → emit dreamer_context_invalid',
-            dreamerContentJson: JSON.stringify({ candidates: [{ badDecision: 42, betterDecision: 'b', rationale: 'r' }] }),
-            expectedEvent: 'dreamer_context_invalid',
-        },
-    ])('edge: $name', async ({ dreamerContentJson, expectedEvent }) => {
-        const { result, capturedPrompt, emitTelemetry } = await runPri508EdgeCase({
-            scribeArtifact: makeScribeWithDreamerRef(),
-            dreamerArtifact: makeDreamerWithContentJson(dreamerContentJson),
-        });
-        // Best-effort: runner still succeeds even when dreamer context validation fails.
-        expect(result.status).toBe('succeeded');
-        expect(capturedPrompt).toBeDefined();
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.dreamerContext).toBeUndefined();
-        // rc-9: failure must be observable, not silent.
-        expect(emitTelemetry).toHaveBeenCalledWith(expect.objectContaining({
-            eventType: expect.stringContaining(expectedEvent),
-            traceId: ARTIFICER_TASK_ID_PRI508,
-        }));
-    });
-    it('edge: dreamer proposal with only required 3 fields + no optional → dreamerContext resolved (explicit nulls)', async () => {
-        // Validates the optional-field path: riskLevel/strategicPerspective/confidence
-        // absent should still yield a valid candidate, with the unresolvable signals
-        // carried as explicit nulls rather than silently dropped (rc-9).
-        const dreamerArtifact = makeDreamerWithContentJson(JSON.stringify({
-            candidates: [{
-                    badDecision: 'bad',
-                    betterDecision: 'better',
-                    rationale: 'why',
-                    // riskLevel and strategicPerspective intentionally absent
-                }],
-        }));
-        const { result, capturedPrompt } = await runPri508EdgeCase({
-            scribeArtifact: makeScribeWithDreamerRef(),
-            dreamerArtifact,
-        });
-        expect(result.status).toBe('succeeded');
-        expect(capturedPrompt).toBeDefined();
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.dreamerContext).toBeDefined();
-        const [candidate] = parsed.dreamerContext.candidates;
-        expect(candidate.badDecision).toBe('bad');
-        expect(candidate.betterDecision).toBe('better');
-        expect(candidate.rationale).toBe('why');
-        expect(candidate.riskLevel).toBeNull();
-        expect(candidate.strategicPerspective).toBeNull();
-        expect(candidate.confidence).toBeNull();
-        expect(parsed.dreamerContext.omittedCandidateCount).toBe(0);
-    });
-    // ── PRI-839: bounded candidate set (was candidates[0]-only) ──────────────
-    it('PRI-839: EVERY dreamer candidate reaches the prompt, priority-ranked by confidence', async () => {
-        const dreamerArtifact = makeDreamerWithContentJson(JSON.stringify({
-            candidates: [
-                { candidateIndex: 0, badDecision: 'bad-0', betterDecision: 'better-0', rationale: 'why-0', confidence: 0.4, riskLevel: 'high', strategicPerspective: 'lens-0' },
-                { candidateIndex: 1, badDecision: 'bad-1', betterDecision: 'better-1', rationale: 'why-1', confidence: 0.9, riskLevel: 'low', strategicPerspective: 'lens-1' },
-                { candidateIndex: 2, badDecision: 'bad-2', betterDecision: 'better-2', rationale: 'why-2', confidence: 0.6, riskLevel: 'medium', strategicPerspective: 'lens-2' },
-            ],
-        }));
-        const { result, capturedPrompt } = await runPri508EdgeCase({
-            scribeArtifact: makeScribeWithDreamerRef(),
-            dreamerArtifact,
-        });
-        expect(result.status).toBe('succeeded');
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.dreamerContext).toBeDefined();
-        // The DC-3 fix: all three alternatives survive — not just candidates[0].
-        expect(parsed.dreamerContext.candidates).toHaveLength(3);
-        // Deterministic priority: confidence DESC, then authored index.
-        expect(parsed.dreamerContext.candidates.map((c) => c.candidateIndex)).toEqual([1, 2, 0]);
-        expect(parsed.dreamerContext.candidates.map((c) => c.priorityRank)).toEqual([1, 2, 3]);
-        expect(parsed.dreamerContext.omittedCandidateCount).toBe(0);
-        expect(parsed.dreamerContext.differenceSummary).toContain('3 mutually distinct proposals');
-        expect(parsed.dreamerContext.differenceSummary).toContain('risk spread');
-    });
-    it('PRI-839: candidates beyond the bound are dropped, counted, and announced', async () => {
-        const candidates = Array.from({ length: FORMATION_CANDIDATE_LIMIT + 2 }, (_, index) => ({
-            candidateIndex: index,
-            badDecision: `bad-${index}`,
-            betterDecision: `better-${index}`,
-            rationale: `why-${index}`,
-            confidence: 0.5,
-            riskLevel: 'low',
-            strategicPerspective: `lens-${index}`,
-        }));
-        const { result, capturedPrompt, emitTelemetry } = await runPri508EdgeCase({
-            scribeArtifact: makeScribeWithDreamerRef(),
-            dreamerArtifact: makeDreamerWithContentJson(JSON.stringify({ candidates })),
-        });
-        expect(result.status).toBe('succeeded');
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.dreamerContext.candidates).toHaveLength(FORMATION_CANDIDATE_LIMIT);
-        expect(parsed.dreamerContext.omittedCandidateCount).toBe(2);
-        // rc-9: the truncation is observable, not silent.
-        expect(emitTelemetry).toHaveBeenCalledWith(expect.objectContaining({
-            eventType: expect.stringContaining('dreamer_context_partial'),
-            traceId: ARTIFICER_TASK_ID_PRI508,
-        }));
-    });
-    it('PRI-839 legacy: a malformed candidate is skipped without discarding the valid ones', async () => {
-        const dreamerArtifact = makeDreamerWithContentJson(JSON.stringify({
-            candidates: [
-                { badDecision: 'bad-0', betterDecision: 'better-0', rationale: 'why-0' },
-                { betterDecision: 'b', rationale: 'r' }, // malformed: no badDecision
-                'not-a-record', // malformed: not an object
-                { badDecision: 'bad-3', betterDecision: 'better-3', rationale: 'why-3' },
-            ],
-        }));
-        const { result, capturedPrompt } = await runPri508EdgeCase({
-            scribeArtifact: makeScribeWithDreamerRef(),
-            dreamerArtifact,
-        });
-        expect(result.status).toBe('succeeded');
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.dreamerContext.candidates).toHaveLength(2);
-        expect(parsed.dreamerContext.candidates.map((c) => c.candidateIndex)).toEqual([0, 3]);
-    });
-    it('PRI-839: single candidate still resolves (no behaviour change from the old path)', async () => {
-        const { result, capturedPrompt } = await runPri508EdgeCase({
-            scribeArtifact: makeScribeWithDreamerRef(),
-            dreamerArtifact: makeDreamerWithContentJson(JSON.stringify({
-                candidates: [{ badDecision: 'only-bad', betterDecision: 'only-better', rationale: 'only-why', confidence: 0.7, riskLevel: 'medium', strategicPerspective: 'lens' }],
-            })),
-        });
-        expect(result.status).toBe('succeeded');
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.dreamerContext.candidates).toHaveLength(1);
-        expect(parsed.dreamerContext.candidates[0].badDecision).toBe('only-bad');
-        expect(parsed.dreamerContext.differenceSummary).toContain('no alternatives were proposed');
-    });
-});
-// ── PRI-509 Slice 3: ArtificerRunner.buildContext reads repairPayload ────────
-// When evaluator returns needs_revision, it seeds an artificer repair task
-// whose diagnosticJson contains a repairPayload. The artificer-runner must
-// detect this payload, format it as a repairFeedback string, and forward it
-// to the prompt builder so the LLM addresses each requiredChange instead of
-// regenerating blind. Trust boundary (rc-1, rc-2): repairPayload originates
-// from evaluator LLM output persisted in diagnosticJson — already validated
-// by isValidRepairPayload in pitask-metadata.ts, but the runner must still
-// treat the hydrated value as opaque text when formatting the prompt.
-describe('PRI-509: ArtificerRunner.buildContext reads repairPayload → repairFeedback', () => {
-    const SCRIBE_ART_ID_PRI509 = 'pi-art-scribe-pri509';
-    const SCRIBE_TASK_ID_PRI509 = 'scribe-pri509';
-    const ARTIFICER_TASK_ID_PRI509 = 'artificer-repair-r1';
-    function makeScribeArtifactPri509() {
-        return {
-            artifactId: SCRIBE_ART_ID_PRI509,
-            artifactKind: 'principle',
-            sourceTaskId: SCRIBE_TASK_ID_PRI509,
-            lineageArtifactIds: [],
-            validationStatus: 'pending',
-            contentJson: JSON.stringify({
-                taskId: SCRIBE_TASK_ID_PRI509,
-                sourcePhilosopherArtifactId: 'pi-art-philosopher-pri509',
-                principleDraft: {
-                    title: 'Input Validation Discipline',
-                    statement: 'Validate tool inputs before execution',
-                    rationale: 'Unvalidated inputs cause downstream failures',
-                    applicability: ['all tools'],
-                    antiPatterns: ['Trusting raw input'],
-                    confidence: 0.85,
-                },
-                sourceTrace: { philosopherArtifactId: 'pi-art-philosopher-pri509' },
-                risks: [],
-                generatedAt: new Date().toISOString(),
-            }),
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-        };
-    }
-    function makeMetadata(overrides) {
-        return {
-            dependencyTaskIds: overrides?.dependencyTaskIds ?? [],
-            channel: overrides?.channel ?? 'prompt',
-            timeoutMs: overrides?.timeoutMs ?? 300_000,
-            inputArtifactRefs: overrides?.inputArtifactRefs ?? [],
-            outputArtifactRefs: overrides?.outputArtifactRefs ?? [],
-            parentTaskId: overrides?.parentTaskId,
-            correlationId: overrides?.correlationId,
-            rejectionCount: overrides?.rejectionCount,
-            adversarialFeedback: overrides?.adversarialFeedback,
-            repairPayload: overrides?.repairPayload,
-        };
-    }
-    function makeArtificerTaskPri509(repairPayload) {
-        const meta = makeMetadata({
-            dependencyTaskIds: [SCRIBE_TASK_ID_PRI509],
-            channel: 'prompt',
-            timeoutMs: 300_000,
-            inputArtifactRefs: [{ artifactType: 'principle', ref: SCRIBE_ART_ID_PRI509 }],
-            outputArtifactRefs: [],
-        });
-        if (repairPayload) {
-            meta.repairPayload = repairPayload;
-        }
-        return {
-            taskId: ARTIFICER_TASK_ID_PRI509,
-            taskKind: 'artificer',
-            status: 'pending',
-            attemptCount: 0,
-            maxAttempts: 3,
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-            diagnosticJson: createPITaskDiagnosticJson(meta),
-        };
-    }
-    function makeArtificerOutputPri509() {
-        return {
-            taskId: ARTIFICER_TASK_ID_PRI509,
-            sourceScribeArtifactId: SCRIBE_ART_ID_PRI509,
-            requiresContextVersion: 2,
-            evidenceRefs: ['pain://1'],
-            implementationCode: 'function evaluate(input, helpers) { return { decision: "allow", matched: false, reason: "ok" }; }',
-            goldenTraceCases: [
-                { caseId: 'positive-1', kind: 'positive', toolName: 'write_file', params: { path: '/project/file.txt' }, expectedDecision: 'allow', ruleContext: V2_RULE_CONTEXT },
-                { caseId: 'negative-1', kind: 'negative', toolName: 'write_file', params: { path: '/etc/passwd' }, expectedDecision: 'block', ruleContext: V2_RULE_CONTEXT },
-            ],
-            affectedTools: ['write_file'],
-            implementationSummary: 'Validate parent path before write',
-            sourceTrace: { scribeArtifactId: SCRIBE_ART_ID_PRI509 },
-            risks: [],
-            generatedAt: new Date().toISOString(),
-        };
-    }
-    function makeScribeTaskPri509() {
-        return {
-            taskId: SCRIBE_TASK_ID_PRI509,
-            taskKind: 'scribe',
-            status: 'succeeded',
-            attemptCount: 1,
-            maxAttempts: 3,
-            resultRef: 'scribe://run-pri509',
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-            diagnosticJson: createPITaskDiagnosticJson({
-                dependencyTaskIds: [],
-                channel: 'prompt',
-                timeoutMs: 300_000,
-                inputArtifactRefs: [],
-                outputArtifactRefs: [{ artifactType: 'principle', ref: SCRIBE_ART_ID_PRI509 }],
-            }),
-        };
-    }
-    async function runPri509RepairLoop(artificerTask) {
-        const store = new MemoryPIArtifactStore();
-        await store.upsertArtifact(makeScribeArtifactPri509());
-        const scribeTask = makeScribeTaskPri509();
-        const stateManager = {
-            acquireLease: vi.fn().mockResolvedValue(artificerTask),
-            getTask: vi.fn().mockImplementation((id) => {
-                if (id === ARTIFICER_TASK_ID_PRI509)
-                    return Promise.resolve(artificerTask);
-                if (id === SCRIBE_TASK_ID_PRI509)
-                    return Promise.resolve(scribeTask);
-                return Promise.resolve(null);
-            }),
-            getRunsByTask: vi.fn().mockResolvedValue([{ runId: 'run-pri509', taskId: ARTIFICER_TASK_ID_PRI509, runtimeKind: 'artificer', startedAt: new Date().toISOString() }]),
-            getValidRunsByTaskTolerant: vi.fn().mockResolvedValue({ runs: [{ runId: 'run-pri509', taskId: ARTIFICER_TASK_ID_PRI509, runtimeKind: 'artificer', startedAt: new Date().toISOString() }], degradedRuns: [] }),
-            updateRunOutput: vi.fn().mockResolvedValue(undefined),
-            markTaskSucceeded: vi.fn().mockResolvedValue(undefined),
-            markTaskFailed: vi.fn().mockResolvedValue(undefined),
-            markTaskRetryWait: vi.fn().mockResolvedValue(undefined),
-            getRetryPolicy: vi.fn().mockReturnValue({ shouldRetry: () => false }),
-        };
-        let capturedPrompt;
-        const runtimeAdapter = {
-            startRun: vi.fn().mockImplementation((input) => {
-                capturedPrompt = typeof input.inputPayload === 'string' ? input.inputPayload : JSON.stringify(input.inputPayload);
-                return Promise.resolve({ runId: 'run-pri509', runtimeKind: 'test-double', startedAt: new Date().toISOString() });
-            }),
-            pollRun: vi.fn().mockResolvedValue({ status: 'succeeded', runId: 'run-pri509' }),
-            fetchOutput: vi.fn().mockResolvedValue({ payload: makeArtificerOutputPri509() }),
-            cancelRun: vi.fn().mockResolvedValue(undefined),
-        };
-        const eventEmitter = { emitTelemetry: vi.fn() };
-        const validator = new DefaultArtificerValidator();
-        const deps = {
-            stateManager,
-            runtimeAdapter,
-            eventEmitter,
-            artifactStore: store,
-            validator,
-            behaviorExamplePack: V2_TEST_PACK,
-        };
-        const runner = new ArtificerRunner(deps, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        const result = await runner.run(ARTIFICER_TASK_ID_PRI509);
-        return { result, capturedPrompt };
-    }
-    it('repairPayload present → repairFeedback serialized into prompt message (with requiredChanges text + previousScore)', async () => {
-        const artificerTask = makeArtificerTaskPri509({
-            requiredChanges: ['add path validation', 'fix case sensitivity'],
-            concerns: ['code quality is poor', 'missing error handling'],
-            previousScore: 0.65,
-            repairIteration: 1,
-            sourceArtificerArtifactId: 'pi-art-artificer-original',
-            sourceEvaluatorTaskId: 'evaluator-r0',
-        });
-        const { result, capturedPrompt } = await runPri509RepairLoop(artificerTask);
-        expect(result.status).toBe('succeeded');
-        expect(capturedPrompt).toBeDefined();
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.repairFeedback).toBeDefined();
-        // requiredChanges text must be present so artificer can address each one
-        expect(parsed.repairFeedback).toContain('add path validation');
-        expect(parsed.repairFeedback).toContain('fix case sensitivity');
-        // previousScore must be present so artificer knows the prior score
-        expect(parsed.repairFeedback).toContain('0.65');
-        // concerns text must be present (PoC-validated format)
-        expect(parsed.repairFeedback).toContain('code quality is poor');
-        // Must signal this is a needs_revision retry (not a fresh attempt)
-        expect(parsed.repairFeedback).toMatch(/needs_revision|prior attempt/i);
-    });
-    it('repairPayload absent → repairFeedback undefined in prompt (backward compat)', async () => {
-        // Round-1 artificer task without repairPayload — repairFeedback must NOT
-        // appear in the prompt (backward compatible with pre-PRI-509 flows).
-        const artificerTask = makeArtificerTaskPri509(); // no repairPayload
-        const { result, capturedPrompt } = await runPri509RepairLoop(artificerTask);
-        expect(result.status).toBe('succeeded');
-        expect(capturedPrompt).toBeDefined();
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.repairFeedback).toBeUndefined();
-    });
-    it('repairPayload present → repairFeedback appears in promptInput (typed surface)', async () => {
-        const artificerTask = makeArtificerTaskPri509({
-            requiredChanges: ['add input validation'],
-            concerns: [],
-            previousScore: 0.55,
-            repairIteration: 2,
-            sourceArtificerArtifactId: 'pi-art-artificer-r1',
-            sourceEvaluatorTaskId: 'evaluator-r1',
-        });
-        const { result } = await runPri509RepairLoop(artificerTask);
-        expect(result.status).toBe('succeeded');
-        // The typed promptInput surface must also carry repairFeedback ( Slice 2 contract).
-        // We can't directly inspect promptInput here — it's internal to the runner.
-        // The prompt message assertion above already covers the e2e path. This test
-        // exists to assert the runner doesn't throw when repairIteration=2.
-    });
-    it('repairPayload + adversarialFeedback both present → both forwarded (orthogonal)', async () => {
-        // PRI-428 (adversarialFeedback) and PRI-509 (repairPayload) are orthogonal
-        // signals — adversarialFeedback comes from RuleHost probe failures, while
-        // repairPayload comes from evaluator needs_revision. Both can be present on
-        // the same task and must both reach the prompt.
-        const meta = makeMetadata({
-            dependencyTaskIds: [SCRIBE_TASK_ID_PRI509],
-            channel: 'prompt',
-            timeoutMs: 300_000,
-            inputArtifactRefs: [{ artifactType: 'principle', ref: SCRIBE_ART_ID_PRI509 }],
-            outputArtifactRefs: [],
-        });
-        meta.adversarialFeedback = 'Adversarial probe failed: tool=test_tool, case=neg-1, expected=block, got=allow';
-        meta.repairPayload = {
-            requiredChanges: ['add input validation'],
-            concerns: ['missing error handling'],
-            previousScore: 0.6,
-            repairIteration: 1,
-            sourceArtificerArtifactId: 'pi-art-artificer-original',
-            sourceEvaluatorTaskId: 'evaluator-r0',
-        };
-        const artificerTask = {
-            taskId: ARTIFICER_TASK_ID_PRI509,
-            taskKind: 'artificer',
-            status: 'pending',
-            attemptCount: 0,
-            maxAttempts: 3,
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-            diagnosticJson: createPITaskDiagnosticJson(meta),
-        };
-        const { result, capturedPrompt } = await runPri509RepairLoop(artificerTask);
-        expect(result.status).toBe('succeeded');
-        expect(capturedPrompt).toBeDefined();
-        const parsed = JSON.parse(capturedPrompt);
-        // Both signals must be present
-        expect(parsed.adversarialFeedback).toBeDefined();
-        expect(parsed.adversarialFeedback).toContain('Adversarial probe failed');
-        expect(parsed.repairFeedback).toBeDefined();
-        expect(parsed.repairFeedback).toContain('add input validation');
-        expect(parsed.repairFeedback).toContain('0.6');
-    });
-    it('repairPayload with diagnosticReplay → replay evidence rendered in repairFeedback (PRI-634 R4 P1-2)', async () => {
-        // The evaluator's needs_revision round ran the diagnostic adversarial
-        // replay; its evidence must reach the next Artificer via repairFeedback.
-        const artificerTask = makeArtificerTaskPri509({
-            requiredChanges: ['fix matcher semantics'],
-            concerns: ['intent mismatch'],
-            previousScore: 0.5,
-            repairIteration: 1,
-            sourceArtificerArtifactId: 'pi-art-artificer-original',
-            sourceEvaluatorTaskId: 'evaluator-r0',
-            diagnosticReplay: { ran: true, passed: true, failedCaseCount: 0 },
-        });
-        const { result, capturedPrompt } = await runPri509RepairLoop(artificerTask);
-        expect(result.status).toBe('succeeded');
-        expect(capturedPrompt).toBeDefined();
-        const parsed = JSON.parse(capturedPrompt);
-        expect(parsed.repairFeedback).toBeDefined();
-        expect(parsed.repairFeedback).toContain('Deterministic adversarial replay');
-        expect(parsed.repairFeedback).toContain('PASSED');
-        // The diagnostic note must be explicit that it does NOT change the verdict
-        expect(parsed.repairFeedback).toContain('needs_revision');
-    });
-    it('isValidRepairPayload rejects malformed diagnosticReplay (fail-closed hydrate, PRI-634 R4)', () => {
-        // diagnosticJson round-trip: a repairPayload whose diagnosticReplay has a
-        // non-boolean `ran` must be rejected at the trust boundary (rc-1/rc-3) —
-        // hydrate yields repairPayload=undefined rather than trusting the shape.
-        const base = {
-            requiredChanges: ['fix matcher'],
-            concerns: [],
-            previousScore: 0.5,
-            repairIteration: 1,
-            sourceArtificerArtifactId: 'pi-art-artificer-original',
-            sourceEvaluatorTaskId: 'evaluator-r0',
-        };
-        const malformed = [
-            { ran: 'yes', passed: true, failedCaseCount: 0 }, // ran not boolean
-            { ran: true, passed: 1, failedCaseCount: 0 }, // passed not boolean
-            { ran: true, passed: true, failedCaseCount: -1 }, // negative count
-            { ran: true, passed: true, failedCaseCount: 1.5 }, // non-integer count
-            'not-an-object', // whole field wrong type
-        ];
-        for (const bad of malformed) {
-            const meta = makeMetadata({ dependencyTaskIds: [SCRIBE_TASK_ID_PRI509] });
-            meta.repairPayload = { ...base, diagnosticReplay: bad };
-            const taskJson = createPITaskDiagnosticJson(meta);
-            const hydrated = parsePITaskMetadata(taskJson);
-            expect(hydrated?.repairPayload, `expected rejection for ${JSON.stringify(bad)}`).toBeUndefined();
-        }
-        // Valid diagnosticReplay passes the trust boundary
-        const metaOk = makeMetadata({ dependencyTaskIds: [SCRIBE_TASK_ID_PRI509] });
-        metaOk.repairPayload = { ...base, diagnosticReplay: { ran: true, passed: false, failedCaseCount: 2 } };
-        const hydratedOk = parsePITaskMetadata(createPITaskDiagnosticJson(metaOk));
-        expect(hydratedOk?.repairPayload?.diagnosticReplay).toEqual({ ran: true, passed: false, failedCaseCount: 2 });
-    });
-});
-// ── PRI-859 (CIL-001): contextHash covers the dreamer candidate lineage ─────
-describe('PRI-859: Artificer contextHash covers the dreamer candidate lineage', () => {
-    const SCRIBE_TASK_859 = 'scribe-pri859';
-    const ARTIFICER_TASK_859 = 'artificer-pri859';
-    const SCRIBE_ART_859 = 'pi-art-scribe-pri859';
-    const DREAMER_ART_A = 'pi-art-dreamer-pri859-A';
-    const DREAMER_ART_B = 'pi-art-dreamer-pri859-B';
-    function scribeTask859() {
-        return {
-            taskId: SCRIBE_TASK_859,
-            taskKind: 'scribe',
-            status: 'succeeded',
-            attemptCount: 1,
-            maxAttempts: 3,
-            resultRef: 'scribe://run-pri859',
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-            diagnosticJson: createPITaskDiagnosticJson({
-                dependencyTaskIds: [],
-                channel: 'prompt',
-                timeoutMs: 300_000,
-                inputArtifactRefs: [],
-                outputArtifactRefs: [{ artifactType: 'principle', ref: SCRIBE_ART_859 }],
-            }),
-        };
-    }
-    function artificerTask859() {
-        return {
-            taskId: ARTIFICER_TASK_859,
-            taskKind: 'artificer',
-            status: 'pending',
-            attemptCount: 0,
-            maxAttempts: 3,
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-            diagnosticJson: createPITaskDiagnosticJson({
-                dependencyTaskIds: [SCRIBE_TASK_859],
-                channel: 'prompt',
-                timeoutMs: 300_000,
-                inputArtifactRefs: [{ artifactType: 'principle', ref: SCRIBE_ART_859 }],
-                outputArtifactRefs: [],
-            }),
-        };
-    }
-    /** Scribe artifact whose sourceTrace optionally points at a dreamer artifact. */
-    function scribeArtifact859(dreamerArtifactId) {
-        return {
-            artifactId: SCRIBE_ART_859,
-            artifactKind: 'principle',
-            sourceTaskId: SCRIBE_TASK_859,
-            lineageArtifactIds: dreamerArtifactId ? [dreamerArtifactId] : [],
-            validationStatus: 'pending',
-            contentJson: JSON.stringify({
-                taskId: SCRIBE_TASK_859,
-                sourcePhilosopherArtifactId: 'pi-art-philosopher-pri859',
-                principleDraft: { title: 'T', statement: 'S', rationale: 'R', applicability: [], antiPatterns: [], confidence: 0.5 },
-                sourceTrace: {
-                    philosopherArtifactId: 'pi-art-philosopher-pri859',
-                    ...(dreamerArtifactId !== undefined ? { dreamerArtifactId } : {}),
-                },
-                risks: [],
-                generatedAt: new Date().toISOString(),
-            }),
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-        };
-    }
-    function dreamerArtifact859(artifactId, marker) {
-        return {
-            artifactId,
-            artifactKind: 'principle',
-            sourceTaskId: `dreamer-task-${marker}`,
-            lineageArtifactIds: [],
-            validationStatus: 'pending',
-            contentJson: JSON.stringify({
-                valid: true,
-                taskId: `dreamer-task-${marker}`,
-                candidates: [{
-                        candidateIndex: 0,
-                        badDecision: `bad decision ${marker}`,
-                        betterDecision: `better decision ${marker}`,
-                        rationale: `rationale ${marker}`,
-                        confidence: 0.8,
-                        riskLevel: 'medium',
-                        strategicPerspective: `strategy ${marker}`,
-                    }],
-                contextRefs: [],
-                generatedAt: new Date().toISOString(),
-            }),
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-        };
-    }
-    function runner859(artifactStore) {
-        const scribeTask = scribeTask859();
-        const artificerTask = artificerTask859();
-        const stateManager = {
-            getTask: vi.fn().mockImplementation((id) => {
-                if (id === ARTIFICER_TASK_859)
-                    return Promise.resolve(artificerTask);
-                if (id === SCRIBE_TASK_859)
-                    return Promise.resolve(scribeTask);
-                return Promise.resolve(null);
-            }),
-        };
-        let capturedInputPayload;
-        const runtimeAdapter = {
-            startRun: vi.fn().mockImplementation((input) => {
-                capturedInputPayload = input.inputPayload;
-                return Promise.resolve({ runId: 'run-pri859', runtimeKind: 'test-double', startedAt: new Date().toISOString() });
-            }),
-        };
-        const eventEmitter = { emitTelemetry: vi.fn() };
-        const runner = new ArtificerRunner({
-            stateManager,
-            runtimeAdapter,
-            eventEmitter,
-            artifactStore,
-            validator: new DefaultArtificerValidator(),
-            behaviorExamplePack: V2_TEST_PACK,
-        }, {
-            owner: 'test',
-            runtimeKind: 'artificer',
-            pollIntervalMs: 10,
-            timeoutMs: 1000,
-        });
-        return { runner, captured: () => capturedInputPayload };
-    }
-    it('same semantic input ⇒ same contextHash (deterministic identity)', async () => {
-        const store = new MemoryPIArtifactStore();
-        await store.upsertArtifact(scribeArtifact859(DREAMER_ART_A));
-        await store.upsertArtifact(dreamerArtifact859(DREAMER_ART_A, 'A'));
-        const { runner } = runner859(store);
-        const first = await runner.buildContext(ARTIFICER_TASK_859);
-        const second = await runner.buildContext(ARTIFICER_TASK_859);
-        expect(first.dreamerContext).toBeDefined();
-        expect(first.contextHash).toMatch(/^ctx-/);
-        expect(second.contextHash).toBe(first.contextHash);
-    });
-    it('different candidate context ⇒ different contextHash (CIL-001 core bug; pre-fix both worlds shared one identity)', async () => {
-        const storeA = new MemoryPIArtifactStore();
-        await storeA.upsertArtifact(scribeArtifact859(DREAMER_ART_A));
-        await storeA.upsertArtifact(dreamerArtifact859(DREAMER_ART_A, 'A'));
-        const storeB = new MemoryPIArtifactStore();
-        await storeB.upsertArtifact(scribeArtifact859(DREAMER_ART_B));
-        await storeB.upsertArtifact(dreamerArtifact859(DREAMER_ART_B, 'B'));
-        const ctxA = await runner859(storeA).runner.buildContext(ARTIFICER_TASK_859);
-        const ctxB = await runner859(storeB).runner.buildContext(ARTIFICER_TASK_859);
-        // Same scribe artifact id in both worlds — the ONLY difference is the
-        // dreamer candidate lineage the prompt visibly carries.
-        expect(ctxA.sourceScribeArtifactId).toBe(ctxB.sourceScribeArtifactId);
-        expect(ctxA.dreamerContext).toBeDefined();
-        expect(ctxB.dreamerContext).toBeDefined();
-        expect(ctxA.contextHash).not.toBe(ctxB.contextHash);
-    });
-    it('candidate content change with the SAME ids (dreamer re-upsert) still shifts identity when the lineage id moves', async () => {
-        // The impact scenario from CIL-001: a re-run writes a new dreamer artifact
-        // (fresh artifact id, upsert semantics). The identity that the prompt's
-        // candidate set is derived from must not stay on the stale hash.
-        const store = new MemoryPIArtifactStore();
-        await store.upsertArtifact(scribeArtifact859(DREAMER_ART_A));
-        await store.upsertArtifact(dreamerArtifact859(DREAMER_ART_A, 'A'));
-        const before = await runner859(store).runner.buildContext(ARTIFICER_TASK_859);
-        // Re-point the scribe lineage at a different dreamer artifact carrying a
-        // different candidate set (as a re-run + upsert produces).
-        await store.upsertArtifact(scribeArtifact859(DREAMER_ART_B));
-        await store.upsertArtifact(dreamerArtifact859(DREAMER_ART_B, 'B'));
-        const after = await runner859(store).runner.buildContext(ARTIFICER_TASK_859);
-        expect(after.contextHash).not.toBe(before.contextHash);
-    });
-    it('missing optional dreamer context preserves the fallback identity (refs = [scribeArtifactId] only)', async () => {
-        // No sourceTrace.dreamerArtifactId at all...
-        const storeNoRef = new MemoryPIArtifactStore();
-        await storeNoRef.upsertArtifact(scribeArtifact859(undefined));
-        const ctxNoRef = await runner859(storeNoRef).runner.buildContext(ARTIFICER_TASK_859);
-        expect(ctxNoRef.dreamerContext).toBeUndefined();
-        // ...and dreamer ref present but unresolvable (degraded, rc-9 observable).
-        const storeMissing = new MemoryPIArtifactStore();
-        await storeMissing.upsertArtifact(scribeArtifact859(DREAMER_ART_A));
-        const ctxMissing = await runner859(storeMissing).runner.buildContext(ARTIFICER_TASK_859);
-        expect(ctxMissing.dreamerContext).toBeUndefined();
-        // Both degrade to the pre-PRI-859 identity shape: hash of [artifactRef]
-        // only — backward compatible, and distinct from the dreamer-covered hash.
-        const storeWithDreamer = new MemoryPIArtifactStore();
-        await storeWithDreamer.upsertArtifact(scribeArtifact859(DREAMER_ART_A));
-        await storeWithDreamer.upsertArtifact(dreamerArtifact859(DREAMER_ART_A, 'A'));
-        const ctxWithDreamer = await runner859(storeWithDreamer).runner.buildContext(ARTIFICER_TASK_859);
-        expect(ctxNoRef.contextHash).toBe(ctxMissing.contextHash);
-        expect(ctxNoRef.contextHash).not.toBe(ctxWithDreamer.contextHash);
-    });
-    it('identity fix does NOT change the model-visible prompt shape (no lineage field leaks into dreamerContext)', async () => {
-        const store = new MemoryPIArtifactStore();
-        await store.upsertArtifact(scribeArtifact859(DREAMER_ART_A));
-        await store.upsertArtifact(dreamerArtifact859(DREAMER_ART_A, 'A'));
-        const { runner, captured } = runner859(store);
-        const ctx = await runner.buildContext(ARTIFICER_TASK_859);
-        await runner.invokeRuntime(ARTIFICER_TASK_859, ctx);
-        expect(captured()).toBeDefined();
-        const parsed = JSON.parse(captured());
-        expect(parsed.dreamerContext).toBeDefined();
-        // Exactly the pre-PRI-859 three-field projection — sourceDreamerArtifactId
-        // is hash input only and must never reach the model.
-        expect(Object.keys(parsed.dreamerContext).sort()).toEqual(['candidates', 'differenceSummary', 'omittedCandidateCount']);
-        expect(parsed.dreamerContext.sourceDreamerArtifactId).toBeUndefined();
-        expect(parsed.dreamerContext.dreamerArtifactId).toBeUndefined();
-        // The contextHash field itself is prompt-visible and carries the new identity.
-        expect(parsed.contextHash).toBe(ctx.contextHash);
-    });
-});
-//# sourceMappingURL=artificer-runner-vslice.test.js.map