principles-disciple 1.127.0 → 1.129.0

package/tests/hooks/prompt-diet.test.ts

@@ -1,335 +0,0 @@
-/**
- * Prompt diet tests — PRI-291
- *
- * Verifies that the default prompt contains only MVP-required sections
- * and does NOT contain sections removed/default-disabled by the MVP diet.
- *
- * Acceptance criteria covered:
- * 1. Default prompt does NOT contain Thinking OS text
- * 2. Default prompt does NOT contain <routing_guidance>
- * 3. Default prompt does NOT contain GFI attitude/personality directive text
- * 4. Runtime V2 activation still injects validated directives into prependSystemContext
- * 5. runtime_v2_prompt_activations_injected event still emits
- * 6. GFI scoring/empathy evidence path still records friction and can emit pain
- */
-
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-
-// ─── Mock dependencies ───────────────────────────────────────────────────────
-
-beforeEach(() => {
-  vi.clearAllMocks();
-  process.env.PD_LEGACY_PROMPT_DIAGNOSTICIAN_ENABLED = 'true';
-  process.env.PD_EMPATHY_API_KEY_ENV = 'PD_TEST_DISABLED_EMPATHY_API_KEY';
-});
-
-afterEach(() => {
-  process.env.PD_LEGACY_PROMPT_DIAGNOSTICIAN_ENABLED = '';
-  delete process.env.PD_EMPATHY_API_KEY_ENV;
-});
-
-const mockGetPendingDiagnosticianTasks = vi.fn<(stateDir: string) => unknown[]>();
-
-vi.mock('../../src/core/diagnostician-task-store.js', async () => ({
-  getPendingDiagnosticianTasks: (...args: unknown[]) =>
-    mockGetPendingDiagnosticianTasks(...args),
-}));
-
-vi.mock('../../src/core/event-log.js', () => ({
-  EventLogService: {
-    get: vi.fn().mockReturnValue({
-      recordHeartbeatDiagnosis: vi.fn(),
-      recordRuntimeV2ActivationsInjected: vi.fn(),
-    }),
-  },
-}));
-
-vi.mock('../../src/core/workspace-context.js', () => {
-  const mockWctx = {
-    workspaceDir: '/fake/workspace',
-    stateDir: '/fake/state',
-    resolve: (key: string) => `/fake/${key}`,
-    trajectory: { recordSession: vi.fn(), recordUserTurn: vi.fn() },
-    config: { get: vi.fn() },
-    eventLog: {
-      recordRuntimeV2ActivationsInjected: vi.fn(),
-    },
-    evolutionReducer: {
-      getActivePrinciples: vi.fn().mockReturnValue([]),
-      getProbationPrinciples: vi.fn().mockReturnValue([]),
-    },
-  };
-  return {
-    WorkspaceContext: {
-      fromHookContext: vi.fn().mockReturnValue(mockWctx),
-      fromHookContextExplicit: vi.fn().mockReturnValue(mockWctx),
-    },
-  };
-});
-
-let sessionGfiValue = 20;
-vi.mock('../../src/core/session-tracker.js', () => ({
-  getSession: vi.fn().mockImplementation(() => ({ currentGfi: sessionGfiValue })),
-  resetFriction: vi.fn(),
-  trackFriction: vi.fn(),
-  setInjectedProbationIds: vi.fn(),
-  clearInjectedProbationIds: vi.fn(),
-  decayGfi: vi.fn(),
-  getGfiDecayElapsed: vi.fn().mockReturnValue(0),
-}));
-
-vi.mock('../../src/core/path-resolver.js', () => ({
-  PathResolver: { getExtensionRoot: vi.fn().mockReturnValue('/fake/extension') },
-}));
-
-vi.mock('../../src/core/principle-injection.js', () => ({
-  selectPrinciplesForInjection: vi.fn().mockReturnValue({
-    selected: [],
-    wasTruncated: false,
-    breakdown: { p0: 0, p1: 0, p2: 0 },
-    totalChars: 0,
-  }),
-  DEFAULT_PRINCIPLE_BUDGET: 3000,
-}));
-
-vi.mock('../../src/core/empathy-keyword-matcher.js', () => ({
-  matchEmpathyKeywords: vi.fn().mockReturnValue({ score: 0, matched: null, severity: 'none', matchedTerms: [] }),
-  loadKeywordStore: vi.fn().mockReturnValue({ terms: {}, stats: { totalHits: 0 } }),
-  saveKeywordStore: vi.fn(),
-  shouldTriggerOptimization: vi.fn().mockReturnValue(false),
-  getKeywordStoreSummary: vi.fn().mockReturnValue({ totalTerms: 0, highFalsePositiveTerms: [] }),
-}));
-
-vi.mock('../../src/core/empathy-types.js', () => ({
-  severityToPenalty: vi.fn().mockReturnValue(5),
-  DEFAULT_EMPATHY_KEYWORD_CONFIG: {},
-}));
-
-vi.mock('../../src/core/correction-cue-learner.js', () => ({
-  CorrectionCueLearner: {
-    get: vi.fn().mockReturnValue({
-      match: vi.fn().mockReturnValue({ matched: null, matchedTerms: [], confidence: 0 }),
-      recordHits: vi.fn(),
-      recordTruePositive: vi.fn(),
-      flush: vi.fn(),
-    }),
-  },
-}));
-
-vi.mock('../../src/core/focus-history.js', () => ({
-  extractSummary: vi.fn().mockReturnValue(''),
-  getHistoryVersions: vi.fn().mockResolvedValue([]),
-  parseWorkingMemorySection: vi.fn().mockReturnValue(null),
-  workingMemoryToInjection: vi.fn().mockReturnValue(''),
-  autoCompressFocus: vi.fn().mockReturnValue({ compressed: false, reason: 'not_needed' }),
-  safeReadCurrentFocus: vi.fn().mockReturnValue({ content: '', recovered: false, validationErrors: [] }),
-}));
-
-vi.mock('../../src/core/local-worker-routing.js', () => ({
-  classifyTask: vi.fn().mockReturnValue({
-    decision: 'stay_main',
-    classification: 'unknown',
-    reason: 'mocked',
-    blockers: [],
-  }),
-}));
-
-// ─── Helpers ─────────────────────────────────────────────────────────────────
-
-function makeMinimalEvent(overrides: {
-  trigger?: string;
-  sessionId?: string;
-} = {}) {
-  const { trigger = 'user', sessionId = 'test-session-diet' } = overrides;
-  return {
-    prompt: 'hello world',
-    messages: [],
-    trigger,
-    sessionId,
-  } as unknown as Parameters<typeof import('../../src/hooks/prompt.js').handleBeforePromptBuild>[0];
-}
-
-function makeCtx(overrides: {
-  sessionGfi?: number;
-  trigger?: string;
-  sessionId?: string;
-} = {}) {
-  const { sessionGfi = 20, trigger = 'user', sessionId = 'test-session-diet' } = overrides;
-  sessionGfiValue = sessionGfi;
-  return {
-    workspaceDir: '/fake/workspace',
-    trigger,
-    sessionId,
-    api: {
-      logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() },
-      runtime: {},
-      config: {},
-    },
-  } as unknown as Parameters<typeof import('../../src/hooks/prompt.js').handleBeforePromptBuild>[1];
-}
-
-async function getPromptOutput(overrides: { sessionGfi?: number } = {}) {
-  const { handleBeforePromptBuild } = await import('../../src/hooks/prompt.js');
-  const result = await handleBeforePromptBuild(makeMinimalEvent(), makeCtx(overrides));
-  return {
-    prepend: result?.prependSystemContext ?? '',
-    append: result?.appendSystemContext ?? '',
-    context: result?.prependContext ?? '',
-  };
-}
-
-// ─── Tests: MVP Diet — sections that must NOT appear by default ────────────
-
-describe('PRI-291 Prompt Diet: default prompt excludes non-MVP sections', { timeout: 15000 }, () => {
-  it('default prompt does NOT contain Thinking OS text', async () => {
-    const { append } = await getPromptOutput();
-    expect(append).not.toContain('<thinking_os>');
-  });
-
-  it('default prompt does NOT contain <routing_guidance>', async () => {
-    const { append } = await getPromptOutput();
-    expect(append).not.toContain('<routing_guidance>');
-    expect(append).not.toContain('DELEGATION SUGGESTION');
-    expect(append).not.toContain('ROUTING GUIDANCE');
-  });
-
-  it('default prompt does NOT contain GFI attitude/personality text', async () => {
-    const { prepend, append } = await getPromptOutput();
-    const combined = prepend + append;
-    expect(combined).not.toContain('HUMBLE_RECOVERY');
-    expect(combined).not.toContain('CONCILIATORY');
-    expect(combined).not.toContain('EFFICIENT');
-    expect(combined).not.toContain('Spicy Evolver');
-    expect(combined).not.toContain('despise entropy');
-    expect(combined).not.toContain('evolve through pain');
-  });
-
-  it('default prompt does NOT contain <project_context> (default is off)', async () => {
-    const { append } = await getPromptOutput();
-    // project_context tag should not appear as a content block.
-    // Note: the EXECUTION RULES section may list it as a priority description,
-    // but the actual <project_context>...</project_context> content block must be absent.
-    const hasProjectContextBlock = append.includes('<project_context>\n');
-    expect(hasProjectContextBlock).toBe(false);
-  });
-
-  it('EXECUTION RULES does not list removed sections', async () => {
-    const { append } = await getPromptOutput();
-    // Only present when appendParts is non-empty (principles exist)
-    // But even when present, removed sections should not be listed
-    if (append.includes('EXECUTION RULES')) {
-      expect(append).not.toContain('<thinking_os>');
-      expect(append).not.toContain('<routing_guidance>');
-      expect(append).not.toContain('<reflection_log>');
-    }
-  });
-});
-
-// ─── Tests: MVP Diet — sections that MUST still appear ─────────────────────
-
-describe('PRI-291 Prompt Diet: MVP sections preserved', () => {
-  it('AGENT IDENTITY is still injected in prependSystemContext', async () => {
-    const { prepend } = await getPromptOutput();
-    expect(prepend).toContain('AGENT IDENTITY');
-  });
-
-  it('evolution principles can still be injected when active', async () => {
-    const { WorkspaceContext } = await import('../../src/core/workspace-context.js');
-    (WorkspaceContext.fromHookContext as ReturnType<typeof vi.fn>).mockReturnValueOnce({
-      workspaceDir: '/fake/workspace',
-      stateDir: '/fake/state',
-      resolve: (key: string) => `/fake/${key}`,
-      trajectory: { recordSession: vi.fn(), recordUserTurn: vi.fn() },
-      config: { get: vi.fn() },
-      eventLog: { recordRuntimeV2ActivationsInjected: vi.fn() },
-      evolutionReducer: {
-        getActivePrinciples: vi.fn().mockReturnValue([
-          { id: 'P1', text: 'Evolution principle still works' },
-        ]),
-        getProbationPrinciples: vi.fn().mockReturnValue([]),
-      },
-    });
-
-    const { handleBeforePromptBuild } = await import('../../src/hooks/prompt.js');
-    const result = await handleBeforePromptBuild(makeMinimalEvent(), makeCtx());
-
-    expect(result?.appendSystemContext).toContain('<evolution_principles>');
-    expect(result?.appendSystemContext).toContain('Evolution principle still works');
-  });
-
-  it('core principles can still be injected', async () => {
-    const { WorkspaceContext } = await import('../../src/core/workspace-context.js');
-    (WorkspaceContext.fromHookContext as ReturnType<typeof vi.fn>).mockReturnValueOnce({
-      workspaceDir: '/fake/workspace',
-      stateDir: '/fake/state',
-      resolve: (key: string) => `/fake/${key}`,
-      trajectory: { recordSession: vi.fn(), recordUserTurn: vi.fn() },
-      config: { get: vi.fn() },
-      eventLog: { recordRuntimeV2ActivationsInjected: vi.fn() },
-      evolutionReducer: {
-        getActivePrinciples: vi.fn().mockReturnValue([
-          { id: 'CP1', text: 'Core principle preserved' },
-        ]),
-        getProbationPrinciples: vi.fn().mockReturnValue([]),
-      },
-    });
-
-    const { handleBeforePromptBuild } = await import('../../src/hooks/prompt.js');
-    const result = await handleBeforePromptBuild(makeMinimalEvent(), makeCtx());
-
-    expect(result?.appendSystemContext).toContain('<core_principles>');
-  });
-
-  it('GFI scoring still runs — trackFriction called on empathy match', async () => {
-    const { matchEmpathyKeywords } = await import('../../src/core/empathy-keyword-matcher.js');
-    (matchEmpathyKeywords as ReturnType<typeof vi.fn>).mockReturnValueOnce({
-      score: 0.8,
-      matched: true,
-      severity: 'moderate',
-      matchedTerms: ['frustrated'],
-    });
-
-    const { WorkspaceContext } = await import('../../src/core/workspace-context.js');
-    (WorkspaceContext.fromHookContext as ReturnType<typeof vi.fn>).mockReturnValueOnce({
-      workspaceDir: '/fake/workspace',
-      stateDir: '/fake/state',
-      resolve: (key: string) => `/fake/${key}`,
-      trajectory: { recordSession: vi.fn(), recordUserTurn: vi.fn(), recordPainEvent: vi.fn() },
-      config: { get: vi.fn().mockImplementation((k: string) => {
-        if (k === 'thresholds.pain_trigger') return 100;
-        if (k === 'severity_thresholds.high') return 70;
-        if (k === 'language') return 'en';
-        return undefined;
-      }) },
-      eventLog: { recordPainSignal: vi.fn() },
-      evolutionReducer: {
-        getActivePrinciples: vi.fn().mockReturnValue([]),
-        getProbationPrinciples: vi.fn().mockReturnValue([]),
-      },
-    });
-
-    const { handleBeforePromptBuild } = await import('../../src/hooks/prompt.js');
-    const event = {
-      prompt: 'I am frustrated with this result',
-      messages: [{ role: 'user', content: 'I am frustrated' }],
-      trigger: 'user',
-      sessionId: 'gfi-test-session',
-    } as unknown as Parameters<typeof import('../../src/hooks/prompt.js').handleBeforePromptBuild>[0];
-
-    await handleBeforePromptBuild(event, makeCtx({ sessionGfi: 80 }));
-
-    const { trackFriction } = await import('../../src/core/session-tracker.js');
-    expect(trackFriction).toHaveBeenCalled();
-  });
-
-  it('size guard still works — total injection under 9000', async () => {
-    const { handleBeforePromptBuild } = await import('../../src/hooks/prompt.js');
-    const result = await handleBeforePromptBuild(makeMinimalEvent(), makeCtx());
-
-    const total =
-      (result?.prependSystemContext?.length ?? 0) +
-      (result?.prependContext?.length ?? 0) +
-      (result?.appendSystemContext?.length ?? 0);
-    expect(total).toBeLessThanOrEqual(9000);
-  });
-});

package/tests/hooks/prompt-size-guard.test.ts

@@ -1,319 +0,0 @@
-/**
- * Tests for prompt.ts diagnostician fixes (Phase A: Immediate Hemorrhage Control)
- *
- * Covers:
- * 1. Compact diagnostician task injection block format
- * 2. Size guard: injection stays under MAX_INJECTION_SIZE (9000)
- * 3. Diagnostician priority mode: low-priority blocks stripped when tasks pending
- * 4. Fail-closed: never returns injection over limit
- */
-
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import * as fs from 'fs';
-import * as os from 'os';
-import * as path from 'path';
-
-// ─── Mock dependencies ───────────────────────────────────────────────────────
-
-const mockGetPendingDiagnosticianTasks = vi.fn<(stateDir: string) => unknown[]>();
-
-beforeEach(() => {
-  vi.clearAllMocks();
-  mockGetPendingDiagnosticianTasks.mockReturnValue([]);
-  // Enable legacy diagnostician injection — tests verify the compact block format
-  process.env.PD_LEGACY_PROMPT_DIAGNOSTICIAN_ENABLED = 'true';
-});
-
-afterEach(() => {
-  process.env.PD_LEGACY_PROMPT_DIAGNOSTICIAN_ENABLED = '';
-});
-
-vi.mock('../../src/core/diagnostician-task-store.js', async () => ({
-  getPendingDiagnosticianTasks: (...args: unknown[]) =>
-    mockGetPendingDiagnosticianTasks(...args),
-}));
-
-vi.mock('../../src/core/event-log.js', () => ({
-  EventLogService: {
-    get: vi.fn().mockReturnValue({
-      recordHeartbeatDiagnosis: vi.fn(),
-    }),
-  },
-}));
-
-vi.mock('../../src/core/workspace-context.js', () => {
-  const mockWctx = {
-    workspaceDir: '/fake/workspace',
-    stateDir: '/fake/state',
-    resolve: (key: string) => `/fake/${key}`,
-    trajectory: { recordSession: vi.fn(), recordUserTurn: vi.fn() },
-    config: { get: vi.fn() },
-    evolutionReducer: {
-      getActivePrinciples: vi.fn().mockReturnValue([]),
-      getProbationPrinciples: vi.fn().mockReturnValue([]),
-    },
-  };
-  return {
-    WorkspaceContext: {
-      fromHookContext: vi.fn().mockReturnValue(mockWctx),
-      fromHookContextExplicit: vi.fn().mockReturnValue(mockWctx),
-    },
-  };
-});
-
-vi.mock('../../src/core/session-tracker.js', () => ({
-  getSession: vi.fn().mockReturnValue({ currentGfi: 20 }),
-  resetFriction: vi.fn(),
-  trackFriction: vi.fn(),
-  setInjectedProbationIds: vi.fn(),
-  clearInjectedProbationIds: vi.fn(),
-  decayGfi: vi.fn(),
-  getGfiDecayElapsed: vi.fn().mockReturnValue(0),
-}));
-
-vi.mock('../../src/core/path-resolver.js', () => ({
-  PathResolver: { getExtensionRoot: vi.fn().mockReturnValue('/fake/extension') },
-}));
-
-vi.mock('../../src/core/principle-injection.js', () => ({
-  selectPrinciplesForInjection: vi.fn().mockReturnValue({
-    selected: [],
-    wasTruncated: false,
-    breakdown: { p0: 0, p1: 0, p2: 0 },
-    totalChars: 0,
-  }),
-  DEFAULT_PRINCIPLE_BUDGET: 3000,
-}));
-
-vi.mock('../../src/core/empathy-keyword-matcher.js', () => ({
-  matchEmpathyKeywords: vi.fn().mockReturnValue({ score: 0, matched: null, severity: 'none', matchedTerms: [] }),
-  loadKeywordStore: vi.fn().mockReturnValue({ terms: {}, stats: { totalHits: 0 } }),
-  saveKeywordStore: vi.fn(),
-  shouldTriggerOptimization: vi.fn().mockReturnValue(false),
-  getKeywordStoreSummary: vi.fn().mockReturnValue({ totalTerms: 0, highFalsePositiveTerms: [] }),
-}));
-
-vi.mock('../../src/core/empathy-types.js', () => ({
-  severityToPenalty: vi.fn().mockReturnValue(5),
-  DEFAULT_EMPATHY_KEYWORD_CONFIG: {},
-}));
-
-vi.mock('../../src/core/correction-cue-learner.js', () => ({
-  CorrectionCueLearner: {
-    get: vi.fn().mockReturnValue({
-      match: vi.fn().mockReturnValue({ matched: null, matchedTerms: [], confidence: 0 }),
-      recordHits: vi.fn(),
-      recordTruePositive: vi.fn(),
-      flush: vi.fn(),
-    }),
-  },
-}));
-
-vi.mock('../../src/core/focus-history.js', () => ({
-  extractSummary: vi.fn().mockReturnValue(''),
-  getHistoryVersions: vi.fn().mockResolvedValue([]),
-  parseWorkingMemorySection: vi.fn().mockReturnValue(null),
-  workingMemoryToInjection: vi.fn().mockReturnValue(''),
-  autoCompressFocus: vi.fn().mockReturnValue({ compressed: false, reason: 'not_needed' }),
-  safeReadCurrentFocus: vi.fn().mockReturnValue({ content: '', recovered: false, validationErrors: [] }),
-}));
-
-vi.mock('../../src/core/local-worker-routing.js', () => ({
-  classifyTask: vi.fn().mockReturnValue({
-    decision: 'stay_main',
-    classification: 'unknown',
-    reason: 'mocked',
-    blockers: [],
-  }),
-}));
-
-// ─── Helpers ─────────────────────────────────────────────────────────────────
-
-function fakeTask(overrides: Partial<{
-  id: string; prompt: string; createdAt: string; status: string;
-}> = {}): { id: string; task: { prompt: string; createdAt: string; status: 'pending' } } {
-  return {
-    id: overrides.id ?? 'test-task-1',
-    task: {
-      prompt: overrides.prompt ?? 'Diagnose pain signal: source=tool_failure score=75 reason=Command failed',
-      createdAt: overrides.createdAt ?? '2026-04-21T10:00:00.000Z',
-      status: 'pending',
-    },
-  };
-}
-
-function makeMinimalEvent(): Parameters<typeof import('../../src/hooks/prompt.js').handleBeforePromptBuild>[0] {
-  return {
-    prompt: 'hello world',
-    messages: [],
-    trigger: 'heartbeat',
-    sessionId: 'test-session-123',
-  } as unknown as Parameters<typeof import('../../src/hooks/prompt.js').handleBeforePromptBuild>[0];
-}
-
-// ─── Tests ───────────────────────────────────────────────────────────────────
-
-// M8: Legacy diagnostician task block injection removed — single-path pain→ledger pipeline
-// no longer injects diagnostician tasks into prompts. These tests verify the old behavior
-// and are kept as regression guards (skipped), not as M8 requirements.
-describe.skip('Diagnostician compact task injection', () => {
-  it('injects a compact block containing task_id, reason, marker and report paths', async () => {
-    const { handleBeforePromptBuild } = await import('../../src/hooks/prompt.js');
-
-    mockGetPendingDiagnosticianTasks.mockReturnValueOnce([fakeTask({
-      id: 'task-abc',
-      prompt:
-        'Pain signal: source=tool_failure\nscore=75\nreason=Command npm test failed with exit code 1\nsession_id=sess-123',
-    })]);
-
-    const ctx = {
-      workspaceDir: '/fake/workspace',
-      trigger: 'heartbeat',
-      sessionId: 'test-session-123',
-      api: {
-        logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() },
-        runtime: {},
-        config: {},
-      },
-    } as unknown as Parameters<typeof handleBeforePromptBuild>[1];
-
-    const result = await handleBeforePromptBuild(makeMinimalEvent(), ctx);
-
-    const combined = (result?.prependContext ?? '') + (result?.appendSystemContext ?? '');
-
-    // Must contain structural fields
-    expect(combined).toContain('task_id: task-abc');
-    expect(combined).toContain('.evolution_complete_task-abc');
-    expect(combined).toContain('.diagnostician_report_task-abc.json');
-
-    // Must NOT contain the full raw prompt (which could be 2-4 KB)
-    // The compact diagnostician block is small; the heartbeat checklist adds to total
-    expect(combined.length).toBeLessThan(2000);
-  });
-
-  it('injects exactly one task per heartbeat regardless of queue depth', async () => {
-    const { handleBeforePromptBuild } = await import('../../src/hooks/prompt.js');
-
-    const tasks = [
-      fakeTask({ id: 'task-1' }),
-      fakeTask({ id: 'task-2' }),
-      fakeTask({ id: 'task-3' }),
-    ];
-    mockGetPendingDiagnosticianTasks.mockReturnValueOnce(tasks);
-
-    const ctx = {
-      workspaceDir: '/fake/workspace',
-      trigger: 'heartbeat',
-      sessionId: 'test-session-123',
-      api: {
-        logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() },
-        runtime: {},
-        config: {},
-      },
-    } as unknown as Parameters<typeof handleBeforePromptBuild>[1];
-
-    const result = await handleBeforePromptBuild(makeMinimalEvent(), ctx);
-    const combined = (result?.prependContext ?? '') + (result?.appendSystemContext ?? '');
-
-    // Only the first task ID appears in the block
-    expect(combined).toContain('task-1');
-    expect(combined).not.toContain('task-2');
-    expect(combined).not.toContain('task-3');
-    // Note mentions remaining count
-    expect(combined).toContain('2 more task(s) are queued');
-  });
-});
-
-describe('Size guard: fail-closed', () => {
-  it('never returns a combined injection that exceeds MAX_INJECTION_SIZE (9000)', async () => {
-    const { handleBeforePromptBuild } = await import('../../src/hooks/prompt.js');
-
-    // One large pending task to trigger diagnostician priority mode
-    const largePrompt = 'Pain signal: ' + 'x'.repeat(5000);
-    mockGetPendingDiagnosticianTasks.mockReturnValueOnce([fakeTask({ prompt: largePrompt })]);
-
-    const ctx = {
-      workspaceDir: '/fake/workspace',
-      trigger: 'heartbeat',
-      sessionId: 'test-session-123',
-      api: {
-        logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() },
-        runtime: {},
-        config: {},
-      },
-    } as unknown as Parameters<typeof handleBeforePromptBuild>[1];
-
-    const result = await handleBeforePromptBuild(makeMinimalEvent(), ctx);
-
-    const totalSize =
-      (result?.prependSystemContext?.length ?? 0) +
-      (result?.prependContext?.length ?? 0) +
-      (result?.appendSystemContext?.length ?? 0);
-
-    expect(totalSize).toBeLessThanOrEqual(9000);
-  });
-
-  it('strips project_context and strips thinking_os/evolution_principles when inDiagMode and over limit', async () => {
-    const { handleBeforePromptBuild } = await import('../../src/hooks/prompt.js');
-
-    // A long reason string in the task so prependContext itself is large
-    const longReason = 'x'.repeat(500);
-    mockGetPendingDiagnosticianTasks.mockReturnValueOnce([
-      fakeTask({ prompt: `Pain signal: reason=${longReason} source=tool_failure score=85` }),
-    ]);
-
-    const ctx = {
-      workspaceDir: '/fake/workspace',
-      trigger: 'heartbeat',
-      sessionId: 'test-session-123',
-      api: {
-        logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() },
-        runtime: {},
-        config: {},
-      },
-    } as unknown as Parameters<typeof handleBeforePromptBuild>[1];
-
-    const result = await handleBeforePromptBuild(makeMinimalEvent(), ctx);
-
-    // The size guard must never throw — result must be defined
-    expect(result).toBeDefined();
-    const combined =
-      (result?.prependSystemContext?.length ?? 0) +
-      (result?.prependContext?.length ?? 0) +
-      (result?.appendSystemContext?.length ?? 0);
-
-    // Must stay within MAX_INJECTION_SIZE (9000)
-    expect(combined).toBeLessThanOrEqual(9000);
-  });
-});
-
-// M8: Legacy diagnostician prompt injection removed — single-path pain→ledger pipeline
-describe.skip('Diagnostician priority mode', () => {
-  it('sets pendingDiagTaskCount > 0 so size guard knows to strip low-priority blocks', async () => {
-    const { handleBeforePromptBuild } = await import('../../src/hooks/prompt.js');
-
-    mockGetPendingDiagnosticianTasks.mockReturnValueOnce([fakeTask()]);
-
-    const infoLogger = vi.fn();
-    const ctx = {
-      workspaceDir: '/fake/workspace',
-      trigger: 'heartbeat',
-      sessionId: 'test-session-123',
-      api: {
-        logger: { info: infoLogger, warn: vi.fn(), error: vi.fn() },
-        runtime: {},
-        config: {},
-      },
-    } as unknown as Parameters<typeof handleBeforePromptBuild>[1];
-
-    const result = await handleBeforePromptBuild(makeMinimalEvent(), ctx);
-
-    // Should have logged task injection
-    expect(infoLogger).toHaveBeenCalledWith(
-      expect.stringContaining('Injected compact diagnostician task block')
-    );
-
-    // Result must be valid
-    expect(result?.prependContext).toBeDefined();
-  });
-});