claude-code-session-manager 0.98.0 → 0.101.0

package/src/main/__tests__/scheduler-autofix-select.test.cjs

@@ -1,439 +0,0 @@
-/**
- * scheduler-autofix-select.test.cjs — unit tests for selectAutoFixTargets.
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/scheduler-autofix-select.test.cjs
- */
-
-'use strict';
-
-import { test } from 'vitest';
-const assert = require('node:assert/strict');
-const {
-  selectAutoFixTargets, isUnresolvableNeedsReview, isRescanCandidate,
-  healTargetForFix, isFixPlanBeyondDepthCap, MAX_INVESTIGATION_DEPTH,
-  isEligibleForImmediateAutoFix, isExhaustedAutoFix, isPlanUnqueued, fixSlugFor,
-  applyRcaClassification,
-} = require('../scheduler.cjs');
-
-const noSiblingOnDisk = () => false;
-const noRunDir = () => null;
-
-function makeJob(overrides = {}) {
-  return {
-    slug: '05-my-feature',
-    status: 'needs_review',
-    runId: '2026-06-16T10-00-00-000Z',
-    parallelGroup: 5,
-    ...overrides,
-  };
-}
-
-test('selects a fresh needs_review job', () => {
-  const jobs = [makeJob()];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 1);
-  assert.strictEqual(result[0].slug, '05-my-feature');
-});
-
-test('excludes job with autoFixAttempted: true whose retry is already spent', () => {
-  const jobs = [makeJob({ autoFixAttempted: true, autoFixRetries: 1 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('excludes a fix-plan slug (05-fix-foo)', () => {
-  const jobs = [makeJob({ slug: '05-fix-foo' })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('excludes a failed job', () => {
-  const jobs = [makeJob({ status: 'failed' })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('excludes a completed job', () => {
-  const jobs = [makeJob({ status: 'completed' })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('excludes a job missing runId when no run dir resolves', () => {
-  const jobs = [makeJob({ runId: null })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk, resolveJobRunId: noRunDir });
-  assert.strictEqual(result.length, 0);
-});
-
-test('selects a job missing runId when a run dir resolves (gap 1: runId backfill)', () => {
-  const jobs = [makeJob({ runId: null })];
-  const result = selectAutoFixTargets(jobs, {
-    fixSlugExists: noSiblingOnDisk,
-    resolveJobRunId: () => '2026-06-16T10-00-00-000Z',
-  });
-  assert.strictEqual(result.length, 1);
-});
-
-test('isUnresolvableNeedsReview: needs_review + no runId + no run dir → true', () => {
-  const job = makeJob({ runId: null });
-  assert.strictEqual(isUnresolvableNeedsReview(job, { hasRunDir: false }), true);
-});
-
-test('isUnresolvableNeedsReview: needs_review + no runId + has run dir → false', () => {
-  const job = makeJob({ runId: null });
-  assert.strictEqual(isUnresolvableNeedsReview(job, { hasRunDir: true }), false);
-});
-
-test('isUnresolvableNeedsReview: needs_review + runId present → false', () => {
-  const job = makeJob();
-  assert.strictEqual(isUnresolvableNeedsReview(job, { hasRunDir: false }), false);
-});
-
-test('isUnresolvableNeedsReview: non needs_review status → false', () => {
-  const job = makeJob({ runId: null, status: 'failed' });
-  assert.strictEqual(isUnresolvableNeedsReview(job, { hasRunDir: false }), false);
-});
-
-test('autoFixOutcome no-plan with retries=0 is retried', () => {
-  const jobs = [makeJob({ autoFixAttempted: true, autoFixOutcome: 'no-plan', autoFixRetries: 0 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 1);
-});
-
-test('autoFixOutcome no-plan with retries=1 is exhausted (not selected)', () => {
-  const jobs = [makeJob({ autoFixAttempted: true, autoFixOutcome: 'no-plan', autoFixRetries: 1 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-// ---- PRD: unstamped/error outcomes must retry once, not dead-end forever ----
-
-test('autoFixAttempted true with NO outcome recorded (unstamped — spawn threw before onExit wired) is retry-eligible once', () => {
-  const jobs = [makeJob({ autoFixAttempted: true, autoFixRetries: 0 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 1);
-});
-
-test('autoFixAttempted true with no outcome AND retries=1 is exhausted (not selected)', () => {
-  const jobs = [makeJob({ autoFixAttempted: true, autoFixRetries: 1 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('autoFixOutcome error with retries=0 is retry-eligible', () => {
-  const jobs = [makeJob({ autoFixAttempted: true, autoFixOutcome: 'error', autoFixRetries: 0 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 1);
-});
-
-test('autoFixOutcome error with retries=1 is exhausted (not selected)', () => {
-  const jobs = [makeJob({ autoFixAttempted: true, autoFixOutcome: 'error', autoFixRetries: 1 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('autoFixOutcome plan is NEVER retried, regardless of autoFixRetries', () => {
-  const jobs = [makeJob({ autoFixAttempted: true, autoFixOutcome: 'plan', autoFixRetries: 0 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-// ---- isExhaustedAutoFix / isPlanUnqueued (reverifyNeedsReview annotation) ----
-
-test('isExhaustedAutoFix: no-plan outcome with retries spent → true', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixOutcome: 'no-plan', autoFixRetries: 1 });
-  assert.strictEqual(isExhaustedAutoFix(job), true);
-});
-
-test('isExhaustedAutoFix: error outcome with retries spent → true', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixOutcome: 'error', autoFixRetries: 1 });
-  assert.strictEqual(isExhaustedAutoFix(job), true);
-});
-
-test('isExhaustedAutoFix: unstamped outcome with retries spent → true', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixRetries: 1 });
-  assert.strictEqual(isExhaustedAutoFix(job), true);
-});
-
-test('isExhaustedAutoFix: retries not yet spent → false (still eligible for its one retry)', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixOutcome: 'no-plan', autoFixRetries: 0 });
-  assert.strictEqual(isExhaustedAutoFix(job), false);
-});
-
-test('isExhaustedAutoFix: plan outcome is never "exhausted" here — isPlanUnqueued\'s job instead', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixOutcome: 'plan', autoFixRetries: 1 });
-  assert.strictEqual(isExhaustedAutoFix(job), false);
-});
-
-test('isPlanUnqueued: plan outcome, fix slug absent from queue → true', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixOutcome: 'plan' });
-  assert.strictEqual(isPlanUnqueued(job, new Set()), true);
-});
-
-test('isPlanUnqueued: plan outcome, fix slug present in queue → false', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixOutcome: 'plan' });
-  assert.strictEqual(isPlanUnqueued(job, new Set([fixSlugFor(job)])), false);
-});
-
-test('isPlanUnqueued: no-plan outcome is never "plan unqueued"', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixOutcome: 'no-plan' });
-  assert.strictEqual(isPlanUnqueued(job, new Set()), false);
-});
-
-// ---- invariant: every needs_review job is either an auto-fix target, or
-// classified by one of the annotation predicates (i.e. would receive a
-// non-null verifierVerdict) — no job may fall through both. ----
-
-test('invariant: attempted+no-outcome, retries spent → not a target, but IS annotatable', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixRetries: 1 });
-  const isTarget = selectAutoFixTargets([job], { fixSlugExists: noSiblingOnDisk }).length === 1;
-  const isAnnotatable = isExhaustedAutoFix(job) || isPlanUnqueued(job, new Set())
-    || isUnresolvableNeedsReview(job, { hasRunDir: true });
-  assert.strictEqual(isTarget, false);
-  assert.strictEqual(isAnnotatable, true);
-});
-
-test('invariant: plan outcome, no queue row → not a target, but IS annotatable (autofix_plan_unqueued)', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixOutcome: 'plan' });
-  const isTarget = selectAutoFixTargets([job], { fixSlugExists: noSiblingOnDisk }).length === 1;
-  const isAnnotatable = isExhaustedAutoFix(job) || isPlanUnqueued(job, new Set())
-    || isUnresolvableNeedsReview(job, { hasRunDir: true });
-  assert.strictEqual(isTarget, false);
-  assert.strictEqual(isAnnotatable, true);
-});
-
-test('invariant: plan outcome WITH a queue row → not a target and NOT annotated (waiting on the fix-plan job itself)', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixOutcome: 'plan' });
-  const queuedSlugs = new Set([fixSlugFor(job)]);
-  const isTarget = selectAutoFixTargets([job], { fixSlugExists: noSiblingOnDisk }).length === 1;
-  const isAnnotatable = isExhaustedAutoFix(job) || isPlanUnqueued(job, queuedSlugs)
-    || isUnresolvableNeedsReview(job, { hasRunDir: true });
-  assert.strictEqual(isTarget, false);
-  assert.strictEqual(isAnnotatable, false);
-});
-
-test('excludes a job whose rcaRecoveryAction is archive (already-shipped stale re-run must never buy a fix-plan)', () => {
-  const jobs = [makeJob({ rcaRecoveryAction: 'archive', rcaFailureClass: 'already-shipped' })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('excludes when fix sibling exists on disk', () => {
-  const jobs = [makeJob()];
-  // fixSlug = '05-fix-my-feature'
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: (s) => s === '05-fix-my-feature' });
-  assert.strictEqual(result.length, 0);
-});
-
-test('excludes when fix sibling already in the queue', () => {
-  const sibling = { slug: '05-fix-my-feature', status: 'pending', runId: null };
-  const jobs = [makeJob(), sibling];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('fixSlug uses padded parallelGroup and strips leading digits from slug', () => {
-  const jobs = [makeJob({ slug: '07-some-task', parallelGroup: 7 })];
-  // Expected fixSlug: '07-fix-some-task'
-  const seen = [];
-  selectAutoFixTargets(jobs, {
-    fixSlugExists: (s) => { seen.push(s); return false; },
-  });
-  assert.strictEqual(seen[0], '07-fix-some-task');
-});
-
-test('defaults parallelGroup to 99 when absent', () => {
-  const jobs = [makeJob({ slug: '05-my-feature', parallelGroup: undefined })];
-  const seen = [];
-  selectAutoFixTargets(jobs, {
-    fixSlugExists: (s) => { seen.push(s); return false; },
-  });
-  assert.strictEqual(seen[0], '99-fix-my-feature');
-});
-
-test('isRescanCandidate: needs_review + runId + no_verdict_sentinel → true (RESCANNABLE_VERDICTS includes it)', () => {
-  const job = makeJob({ verifierVerdict: 'no_verdict_sentinel' });
-  assert.strictEqual(isRescanCandidate(job), true);
-});
-
-test('isRescanCandidate: needs_review + runId + uncommitted_changes → false (git commit-guard verdict stays excluded)', () => {
-  const job = makeJob({ verifierVerdict: 'uncommitted_changes' });
-  assert.strictEqual(isRescanCandidate(job), false);
-});
-
-test('isRescanCandidate: needs_review + runId + pass_no_commit → true (fix-plan exemption can flip the verdict on rescan)', () => {
-  const job = makeJob({ verifierVerdict: 'pass_no_commit' });
-  assert.strictEqual(isRescanCandidate(job), true);
-});
-
-// ---- investigationDepth bound (fix-plan-of-a-fix-plan recursion cap) ----
-
-test('depth-1 (ordinary, no investigationDepth field) needs_review job is eligible', () => {
-  const jobs = [makeJob({ slug: '05-my-feature' })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 1);
-});
-
-test('fix-plan job with investigationDepth: 1 is still eligible (its first -fix follow-up)', () => {
-  const jobs = [makeJob({ slug: '05-fix-foo', investigationDepth: 1 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 1);
-});
-
-test('fix-plan job with investigationDepth: 2 is now NOT eligible (cap lowered to permit at most -fix-fix)', () => {
-  const jobs = [makeJob({ slug: '05-fix-foo', investigationDepth: 2 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('fix-plan job with investigationDepth: 3 is NOT eligible (bound holds)', () => {
-  const jobs = [makeJob({ slug: '05-fix-foo', investigationDepth: 3 })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('legacy fix-plan job with no investigationDepth field defaults to excluded (back-compat)', () => {
-  const jobs = [makeJob({ slug: '05-fix-foo' })];
-  const result = selectAutoFixTargets(jobs, { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-test('isFixPlanBeyondDepthCap: non-fix-plan slug is never capped regardless of depth', () => {
-  assert.strictEqual(isFixPlanBeyondDepthCap('05-my-feature', 99), false);
-});
-
-test('isFixPlanBeyondDepthCap: fix-plan at depth 1 is not capped, depth 2 is', () => {
-  assert.strictEqual(isFixPlanBeyondDepthCap('05-fix-foo', 1), false);
-  assert.strictEqual(isFixPlanBeyondDepthCap('05-fix-foo', MAX_INVESTIGATION_DEPTH), false);
-  assert.strictEqual(isFixPlanBeyondDepthCap('05-fix-foo', MAX_INVESTIGATION_DEPTH + 1), true);
-});
-
-// PRD 1131: classification is now provenance-aware (isFixPlan), not slug-only.
-// A legacy row with no `isFixPlan` argument at all still falls back to the
-// slug-only heuristic above (no re-classification of live history).
-
-test('isFixPlanBeyondDepthCap: a scheduler-authored fix plan (isFixPlan:true) is still classified as one and capped at the same depth', () => {
-  assert.strictEqual(isFixPlanBeyondDepthCap('05-fix-foo', MAX_INVESTIGATION_DEPTH, true), false);
-  assert.strictEqual(isFixPlanBeyondDepthCap('05-fix-foo', MAX_INVESTIGATION_DEPTH + 1, true), true);
-});
-
-test('isFixPlanBeyondDepthCap: a human-authored PRD (isFixPlan:false) whose slug merely matches "fix-" is never capped, even with no investigationDepth (PRD 1126 regression)', () => {
-  assert.strictEqual(isFixPlanBeyondDepthCap('126-fix-plan-death-reopens-parent', undefined, false), false);
-});
-
-test('isFixPlanBeyondDepthCap: a legacy row (isFixPlan argument omitted/undefined) with a fix- slug and no depth field is still treated as a fix plan via the slug fallback', () => {
-  assert.strictEqual(isFixPlanBeyondDepthCap('05-fix-foo', undefined, undefined), true);
-});
-
-test('healTargetForFix: depth-2 fix-of-a-fix slug resolves back to the depth-1 fix job (521-fix-recorder-export-footer-redesign)', () => {
-  // Depth-1 fix job: originally authored to fix job 521, itself in needs_review.
-  const depth1FixJob = {
-    slug: '521-fix-recorder-export-footer-redesign',
-    status: 'needs_review',
-    parallelGroup: 521,
-    investigationDepth: 2,
-  };
-  const jobs = [depth1FixJob];
-  // spawnInvestigation's fix-slug formula: baseSlug strips only the leading
-  // numeric prefix (not "fix-"), so investigating this depth-1 fix job
-  // produces a depth-2 fix slug of the form "<group>-fix-fix-<rest>".
-  const baseSlug = depth1FixJob.slug.replace(/^\d+-/, '');
-  const depth2FixSlug = `${String(depth1FixJob.parallelGroup).padStart(2, '0')}-fix-${baseSlug}`;
-  assert.strictEqual(depth2FixSlug, '521-fix-fix-recorder-export-footer-redesign');
-  const resolved = healTargetForFix(depth2FixSlug, jobs);
-  assert.strictEqual(resolved, depth1FixJob);
-});
-
-// ---- isEligibleForImmediateAutoFix (spawnJob's same-tick auto-fix trigger,
-// PRD 2026-07-12: needs_review jobs no longer wait up to 10 min for
-// reverifyNeedsReview()'s periodic pass) ----
-
-test('isEligibleForImmediateAutoFix: fresh needs_review job (fresh from spawnJob, not reverifyNeedsReview) is eligible', () => {
-  const job = makeJob();
-  const result = isEligibleForImmediateAutoFix(job, [job], noSiblingOnDisk);
-  assert.strictEqual(result, true);
-});
-
-test('isEligibleForImmediateAutoFix: attempted job with retry already spent is excluded (prevents double-spawn with the periodic pass)', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixRetries: 1 });
-  const result = isEligibleForImmediateAutoFix(job, [job], noSiblingOnDisk);
-  assert.strictEqual(result, false);
-});
-
-test('isEligibleForImmediateAutoFix: attempted job with an unstamped outcome and its retry still unspent IS eligible (bounded once)', () => {
-  const job = makeJob({ autoFixAttempted: true, autoFixRetries: 0 });
-  const result = isEligibleForImmediateAutoFix(job, [job], noSiblingOnDisk);
-  assert.strictEqual(result, true);
-});
-
-test('isEligibleForImmediateAutoFix: fix sibling already on disk excludes the job', () => {
-  const job = makeJob();
-  // fixSlug = '05-fix-my-feature'
-  const result = isEligibleForImmediateAutoFix(job, [job], (s) => s === '05-fix-my-feature');
-  assert.strictEqual(result, false);
-});
-
-test('isEligibleForImmediateAutoFix: fix-plan job beyond the depth cap is excluded', () => {
-  const job = makeJob({ slug: '05-fix-foo', investigationDepth: 3 });
-  const result = isEligibleForImmediateAutoFix(job, [job], noSiblingOnDisk);
-  assert.strictEqual(result, false);
-});
-
-test('isEligibleForImmediateAutoFix: fix-plan job at investigationDepth 2 is now excluded (previously eligible under the old cap)', () => {
-  const job = makeJob({ slug: '05-fix-foo', investigationDepth: 2 });
-  const result = isEligibleForImmediateAutoFix(job, [job], noSiblingOnDisk);
-  assert.strictEqual(result, false);
-});
-
-test('isEligibleForImmediateAutoFix: a different job in the same queue does not affect this job\'s eligibility', () => {
-  const job = makeJob();
-  const other = { slug: '05-fix-my-feature', status: 'pending', runId: null };
-  const result = isEligibleForImmediateAutoFix(job, [job, other], noSiblingOnDisk);
-  // The fix sibling is already queued, so this job is excluded too (matches
-  // selectAutoFixTargets's "no fix sibling in queue" rule).
-  assert.strictEqual(result, false);
-});
-
-test('isEligibleForImmediateAutoFix: a job whose status is not needs_review (e.g. still pending) is excluded', () => {
-  const job = makeJob({ status: 'pending' });
-  const result = isEligibleForImmediateAutoFix(job, [job], noSiblingOnDisk);
-  assert.strictEqual(result, false);
-});
-
-// ---- applyRcaClassification (persists writeRcaReport's verdict onto the job row) ----
-
-test('applyRcaClassification: persists failureClass + recoveryAction onto a needs_review job', () => {
-  const job = makeJob({ status: 'needs_review' });
-  const applied = applyRcaClassification(job, { filed: true, failureClass: 'already-shipped', recoveryAction: 'archive' });
-  assert.strictEqual(applied, true);
-  assert.strictEqual(job.rcaFailureClass, 'already-shipped');
-  assert.strictEqual(job.rcaRecoveryAction, 'archive');
-});
-
-test('applyRcaClassification: no-op when the report was never filed', () => {
-  const job = makeJob({ status: 'needs_review' });
-  const applied = applyRcaClassification(job, { filed: false, reason: 'disabled' });
-  assert.strictEqual(applied, false);
-  assert.strictEqual(job.rcaFailureClass, undefined);
-});
-
-test('applyRcaClassification: no-op when the job has moved off needs_review', () => {
-  const job = makeJob({ status: 'completed' });
-  const applied = applyRcaClassification(job, { filed: true, failureClass: 'unknown', recoveryAction: 'investigate' });
-  assert.strictEqual(applied, false);
-  assert.strictEqual(job.rcaFailureClass, undefined);
-});
-
-test('applyRcaClassification: no-op when the job is missing (e.g. slug lookup miss)', () => {
-  const applied = applyRcaClassification(undefined, { filed: true, failureClass: 'unknown', recoveryAction: 'investigate' });
-  assert.strictEqual(applied, false);
-});
-
-test('applyRcaClassification result feeds selectAutoFixTargets: an archive-classified job is excluded', () => {
-  const job = makeJob({ status: 'needs_review' });
-  applyRcaClassification(job, { filed: true, failureClass: 'already-shipped', recoveryAction: 'archive' });
-  const result = selectAutoFixTargets([job], { fixSlugExists: noSiblingOnDisk });
-  assert.strictEqual(result.length, 0);
-});
-
-console.log('scheduler-autofix-select tests: PASS');

package/src/main/__tests__/scheduler-autopromote.test.cjs

@@ -1,51 +0,0 @@
-/**
- * scheduler-autopromote.test.cjs — unit tests for isPromotableOriginal.
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/scheduler-autopromote.test.cjs
- */
-
-'use strict';
-
-import { test } from 'vitest';
-const assert = require('node:assert/strict');
-const { isPromotableOriginal, healTargetForFix } = require('../scheduler.cjs');
-
-test('isPromotableOriginal: failed → true', () => {
-  assert.strictEqual(isPromotableOriginal('failed'), true);
-});
-
-test('isPromotableOriginal: needs_review → true', () => {
-  assert.strictEqual(isPromotableOriginal('needs_review'), true);
-});
-
-test('isPromotableOriginal: completed → false', () => {
-  assert.strictEqual(isPromotableOriginal('completed'), false);
-});
-
-test('isPromotableOriginal: running → false', () => {
-  assert.strictEqual(isPromotableOriginal('running'), false);
-});
-
-test('isPromotableOriginal: pending → false', () => {
-  assert.strictEqual(isPromotableOriginal('pending'), false);
-});
-
-test('healTargetForFix: matches the specific original by full numeric-prefixed slug, not just base', () => {
-  const jobs = [
-    { slug: '451-foo', status: 'needs_review' },
-    { slug: '453-foo', status: 'needs_review' },
-  ];
-  const target = healTargetForFix('451-fix-foo', jobs);
-  assert.strictEqual(target.slug, '451-foo');
-});
-
-test('healTargetForFix: returns null when no promotable original matches', () => {
-  const jobs = [
-    { slug: '451-foo', status: 'completed' },
-    { slug: '453-foo', status: 'needs_review' },
-  ];
-  const target = healTargetForFix('451-fix-foo', jobs);
-  assert.strictEqual(target, null);
-});
-
-console.log('scheduler-autopromote tests: PASS');

package/src/main/__tests__/scheduler-bash-timeout-env.test.cjs

@@ -1,101 +0,0 @@
-/**
- * scheduler-bash-timeout-env.test.cjs — PRD 1097.
- *
- * A headless `claude -p` job inherits the harness's 120s default foreground
- * Bash timeout. Any legit gate command (a test suite, a build) that runs
- * long gets auto-backgrounded, and the tool result promises a notification
- * that a single-shot headless run can never receive — the run dead-ends
- * with no commit and no verdict. The fix is to raise
- * BASH_DEFAULT_TIMEOUT_MS / BASH_MAX_TIMEOUT_MS on the spawned child's env.
- *
- * This proves:
- *   1. executeJob's real spawned child actually receives both env vars, with
- *      the exported constant values (not just that the constants exist).
- *   2. BASH_MAX_TIMEOUT_MS stays strictly below IDLE_OUTPUT_KILL_MS — the
- *      coupling that makes this more than a one-liner: a long foreground
- *      Bash emits no stream-json events, so the log mtime stalls and the
- *      idle-tail watchdog would SIGTERM the job mid-gate if the two ever
- *      crossed.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/scheduler-bash-timeout-env.test.cjs
- */
-
-'use strict';
-
-import { test, expect, beforeAll, afterAll, afterEach } from 'vitest';
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-const claudeStub = require('../../../tests/helpers/claudeStub.cjs');
-
-let tmpHome;
-let originalHome;
-let executeJob;
-let BASH_DEFAULT_TIMEOUT_MS;
-let BASH_MAX_TIMEOUT_MS;
-let IDLE_OUTPUT_KILL_MS;
-
-beforeAll(() => {
-  originalHome = process.env.HOME;
-  tmpHome = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-bash-timeout-env-'));
-  process.env.HOME = tmpHome;
-  ({ executeJob, BASH_DEFAULT_TIMEOUT_MS, BASH_MAX_TIMEOUT_MS, IDLE_OUTPUT_KILL_MS } = require('../scheduler.cjs'));
-});
-
-afterAll(() => {
-  process.env.HOME = originalHome;
-  fs.rmSync(tmpHome, { recursive: true, force: true });
-});
-
-afterEach(() => {
-  delete process.env.SM_CLAUDE_BIN;
-});
-
-// Stub `claude` binary: dumps the two env vars it received into a marker
-// file, then emits a stream-json success result and exits 0.
-function writeClaudeStub() {
-  return claudeStub.writeClaudeStub({ body: `
-    const fs = require('fs');
-    const path = require('path');
-    fs.writeFileSync(path.join(process.cwd(), 'env.marker'), JSON.stringify({
-      BASH_DEFAULT_TIMEOUT_MS: process.env.BASH_DEFAULT_TIMEOUT_MS,
-      BASH_MAX_TIMEOUT_MS: process.env.BASH_MAX_TIMEOUT_MS,
-    }), 'utf8');
-    process.stdout.write(JSON.stringify({ type: 'result', subtype: 'success', result: 'ok' }) + '\\n');
-    process.exit(0);
-  ` });
-}
-
-test('BASH_MAX_TIMEOUT_MS stays strictly below IDLE_OUTPUT_KILL_MS with real margin', () => {
-  expect(typeof BASH_DEFAULT_TIMEOUT_MS).toBe('number');
-  expect(typeof BASH_MAX_TIMEOUT_MS).toBe('number');
-  expect(typeof IDLE_OUTPUT_KILL_MS).toBe('number');
-  expect(BASH_DEFAULT_TIMEOUT_MS).toBeLessThanOrEqual(BASH_MAX_TIMEOUT_MS);
-  expect(BASH_MAX_TIMEOUT_MS).toBeLessThan(IDLE_OUTPUT_KILL_MS);
-  // Real margin, not a razor's edge.
-  expect(IDLE_OUTPUT_KILL_MS - BASH_MAX_TIMEOUT_MS).toBeGreaterThanOrEqual(60_000);
-});
-
-test('executeJob spawns the executor child with BASH_DEFAULT_TIMEOUT_MS and BASH_MAX_TIMEOUT_MS set', async () => {
-  const mainCwd = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-bash-timeout-main-'));
-  fs.mkdirSync(path.join(tmpHome, '.claude'), { recursive: true });
-  const runDir = fs.mkdtempSync(path.join(tmpHome, '.claude', 'sm-bash-timeout-run-'));
-  try {
-    const slug = `1097-test-${process.pid}-${Math.floor(Math.random() * 1e6)}`;
-    const prdsDir = path.join(mainCwd, 'session-manager-operations', 'scheduler', 'prds');
-    fs.mkdirSync(prdsDir, { recursive: true });
-    fs.writeFileSync(path.join(prdsDir, `${slug}.md`), 'Verify the env vars are set.', 'utf8');
-    process.env.SM_CLAUDE_BIN = writeClaudeStub();
-
-    const job = { slug, cwd: mainCwd };
-    const result = await executeJob(job, runDir, mainCwd, () => Promise.resolve());
-
-    expect(result.exitCode).toBe(0);
-    const marker = JSON.parse(fs.readFileSync(path.join(mainCwd, 'env.marker'), 'utf8'));
-    expect(marker.BASH_DEFAULT_TIMEOUT_MS).toBe(String(BASH_DEFAULT_TIMEOUT_MS));
-    expect(marker.BASH_MAX_TIMEOUT_MS).toBe(String(BASH_MAX_TIMEOUT_MS));
-  } finally {
-    fs.rmSync(mainCwd, { recursive: true, force: true });
-    fs.rmSync(runDir, { recursive: true, force: true });
-  }
-});