@deskwork/core 0.45.2 → 0.46.0

package/dist/doctor/rules/sites-to-lanes-migration.ts

@@ -0,0 +1,449 @@
+/**
+ * Rule: sites-to-lanes-migration (Phase 39b).
+ *
+ * Detects the pre-migration shape and migrates it under `--fix`. Per the
+ * sites→lanes retirement spec (`docs/superpowers/specs/2026-06-02-sites-
+ * to-lanes-retirement-design.md` §"Migration"):
+ *
+ *   Detect (audit): `config.sites` is present (legacy shape) OR any
+ *   artifact-bearing entry lacks `artifactPath`. The rule emits a
+ *   detection finding describing what `--fix` will do, PLUS one
+ *   `migration-ambiguous` finding per entry whose slug+stage resolves to
+ *   more than one candidate file (AUDIT-20260602-03 — see the backfiller).
+ *
+ *   Fix (apply): three steps, in order:
+ *     1. Lanes from sites — per legacy `site`, create/ensure a lane
+ *        whose `id` is the site slug, `pipelineTemplate: editorial`,
+ *        `host ← site.host` (when present), and `scaffoldDefaults` keyed
+ *        at minimum by `markdown → site.contentDir`.
+ *     2. Backfill `artifactPath` — stamp every UNAMBIGUOUS artifact-
+ *        bearing entry from its single on-disk candidate. Ambiguous
+ *        entries are REFUSED (the backfiller's ambiguity-halt).
+ *     3. Drop `sites` — remove the `sites` / `defaultSite` block from the
+ *        on-disk config.
+ *
+ * Project-scoped despite the runner's per-site loop: the runner calls
+ * each rule once per `Object.keys(config.sites)`. This rule is a
+ * whole-project operation (it rewrites the single config + every
+ * sidecar), so it acts ONLY on the runner's FIRST site pass and no-ops
+ * on the rest — guarded by `isLeadSite`. Its source of truth is the
+ * on-disk config read through the migration-only tolerant reader
+ * (`legacy-config.ts`), NOT `ctx.config`, so once `sites` is dropped a
+ * re-run sees nothing to migrate (idempotency).
+ *
+ * Sibling-relative imports per the doctor convention.
+ */
+
+import { existsSync, mkdirSync } from 'node:fs';
+import { appendJournalEvent } from '../../journal/append.ts';
+import { commitLaneConfig } from '../../lanes/operations/commit.ts';
+import { laneConfigPath, lanesDir, loadLaneConfig } from '../../lanes/loader.ts';
+import { LaneConfigSchema, type LaneConfig } from '../../lanes/types.ts';
+import {
+  dropSitesBlock,
+  readLegacySites,
+  type LegacySite,
+} from '../legacy-config.ts';
+import {
+  backfillFromLegacySites,
+  detectAmbiguousBackfills,
+  type AmbiguousBackfill,
+  type LaneBase,
+} from '../sites-migration-backfill.ts';
+import { readAllSidecars } from '../../sidecar/read-all.ts';
+import type {
+  DoctorContext,
+  DoctorRule,
+  Finding,
+  RepairPlan,
+  RepairResult,
+} from '../types.ts';
+
+const RULE_ID = 'sites-to-lanes-migration';
+const AMBIGUOUS_RULE_ID = 'migration-ambiguous';
+const MIGRATION_PIPELINE_TEMPLATE = 'editorial';
+
+/**
+ * The migration is whole-project. Phase 39c collapsed the runner's
+ * per-site loop into a single project pass (`runner.ts` `PROJECT_SCOPE`),
+ * so this rule now fires exactly once per run regardless. The guard is
+ * retained as a constant `true` (its source of truth is the on-disk
+ * config via `readLegacySites`, not `ctx.config`, so idempotency is
+ * unaffected).
+ */
+function isLeadSite(_ctx: DoctorContext): boolean {
+  return true;
+}
+
+/**
+ * Collect the legacy bases (one per legacy site) — each site's contentDir
+ * paired with the lane id it migrates to (the site slug). These are the
+ * bases the backfiller searches; the lane id lets it stamp the migrated
+ * entry's `lane` (AUDIT-20260603-12). Returns an empty array when there
+ * are no legacy sites (post-migration).
+ */
+function legacyLaneBases(sites: ReadonlyMap<string, LegacySite>): LaneBase[] {
+  return [...sites.entries()].map(([slug, s]) => ({
+    laneId: slug,
+    contentDir: s.contentDir,
+  }));
+}
+
+/**
+ * Whether any artifact-bearing entry still lacks `artifactPath`.
+ *
+ * Does NOT swallow read errors (AUDIT-20260603-14). A corrupt sidecar
+ * makes `readAllSidecars` throw; previously this was caught and turned
+ * into `false`, so `audit()` silently under-reported ("nothing missing")
+ * on the exact on-disk state that `apply()` rejects — an audit/apply
+ * asymmetry the project's "no swallowed exceptions" rule names as a bug
+ * factory. The throw now propagates to `audit()`, which converts it into
+ * an `error` finding (so audit and apply agree on corrupt input, and the
+ * run still completes — AUDIT-20260603-13).
+ */
+async function anyEntryMissingArtifactPath(projectRoot: string): Promise<boolean> {
+  const entries = await readAllSidecars(projectRoot);
+  return entries.some(
+    (e) => e.artifactPath === undefined || e.artifactPath === '',
+  );
+}
+
+/**
+ * Build the per-site lane config (id = slug). Carries the site's
+ * website-publishing metadata forward when present: `host` (Decision #2)
+ * and `redirectsPath` (Decision #23 / c4) re-home onto the lane as
+ * optional siblings; each is omitted entirely when the legacy site did
+ * not declare it (no empty-string writes). `scaffoldDefaults.markdown`
+ * captures the legacy contentDir as the add-time default — never
+ * identity or resolution.
+ */
+function laneFromSite(slug: string, site: LegacySite): LaneConfig {
+  const lane: LaneConfig = {
+    id: slug,
+    name: slug,
+    pipelineTemplate: MIGRATION_PIPELINE_TEMPLATE,
+    // Per Phase 39c the lane carries no `contentDir` — the legacy site's
+    // content directory becomes the lane's add-time `scaffoldDefaults`
+    // (the `markdown` kind, the editorial pipeline's artifact kind).
+    // Location info is the ENTRY's `artifactPath`, never the lane.
+    scaffoldDefaults: { markdown: site.contentDir },
+    ...(site.host !== undefined ? { host: site.host } : {}),
+    ...(site.redirectsPath !== undefined
+      ? { redirectsPath: site.redirectsPath }
+      : {}),
+  };
+  return lane;
+}
+
+/**
+ * Collect the migration's detection + ambiguity findings. Extracted from
+ * `audit()` so the latter can wrap it in a single try/catch that converts
+ * any read throw into an `error` finding (AUDIT-20260603-13/-14) without
+ * the detection logic itself growing a defensive nest. May throw —
+ * `audit()` is its only caller and handles the throw.
+ */
+async function collectFindings(ctx: DoctorContext): Promise<Finding[]> {
+  const { sites } = readLegacySites(ctx.projectRoot);
+  const sitesPresent = sites.size > 0;
+  const missingArtifactPath = await anyEntryMissingArtifactPath(ctx.projectRoot);
+
+  const findings: Finding[] = [];
+
+  if (sitesPresent || missingArtifactPath) {
+    const slugs = [...sites.keys()];
+    findings.push({
+      ruleId: RULE_ID,
+      site: ctx.site,
+      severity: 'warning',
+      message:
+        `Project uses the legacy "sites" model` +
+        (sitesPresent ? ` (${slugs.length} site(s): ${slugs.join(', ')})` : '') +
+        (missingArtifactPath ? ' and has entries missing artifactPath' : '') +
+        `. --fix migrates each site to a lane (host + scaffoldDefaults), ` +
+        `backfills each unambiguous entry's artifactPath, and drops the ` +
+        `sites block. Slug-collision entries (resolving to >1 file) are ` +
+        `refused and reported as migration-ambiguous.`,
+      details: {
+        siteSlugs: slugs,
+        sitesPresent,
+        missingArtifactPath,
+      },
+    });
+  }
+
+  // Surface ambiguity collisions as their own findings BEFORE any fix
+  // runs (AUDIT-20260602-03). These are report-only — the migration
+  // refuses to stamp them.
+  const bases = legacyLaneBases(sites);
+  if (bases.length > 0) {
+    const ambiguous = await detectAmbiguousBackfills(ctx.projectRoot, bases);
+    for (const amb of ambiguous) {
+      findings.push(ambiguousFinding(ctx.site, amb));
+    }
+  }
+
+  return findings;
+}
+
+const rule: DoctorRule = {
+  id: RULE_ID,
+  label: 'Migrate legacy sites to lanes (Phase 39 sites→lanes retirement)',
+
+  async audit(ctx: DoctorContext): Promise<Finding[]> {
+    if (!isLeadSite(ctx)) return [];
+
+    // The migration's reads can throw on a broken-but-present config (a
+    // legacy site missing `contentDir` — `readLegacySites`) or a corrupt
+    // sidecar (`readAllSidecars` via `anyEntryMissingArtifactPath` /
+    // `detectAmbiguousBackfills`). An uncaught throw here aborts the WHOLE
+    // doctor run, denying the operator every other rule's output
+    // (AUDIT-20260603-13). The runner does not guard `rule.audit()`, so
+    // the rule guards itself: any throw becomes an `error`-severity
+    // finding naming the problem, and the run continues. This also keeps
+    // audit and apply consistent on corrupt sidecars (AUDIT-20260603-14):
+    // both surface the corruption loudly rather than audit silently
+    // reporting clean while apply throws.
+    try {
+      return await collectFindings(ctx);
+    } catch (err) {
+      const reason = err instanceof Error ? err.message : String(err);
+      return [
+        {
+          ruleId: RULE_ID,
+          site: ctx.site,
+          severity: 'error',
+          message:
+            `sites-to-lanes migration could not inspect the project: ${reason}. ` +
+            `Repair the offending config or sidecar and re-run doctor.`,
+          details: { error: reason },
+        },
+      ];
+    }
+  },
+
+  async plan(_ctx: DoctorContext, finding: Finding): Promise<RepairPlan> {
+    if (finding.ruleId === AMBIGUOUS_RULE_ID) {
+      const rawCandidates = finding.details.candidates;
+      const candidates = Array.isArray(rawCandidates)
+        ? rawCandidates.map((c) => String(c))
+        : [];
+      return {
+        kind: 'report-only',
+        finding,
+        reason:
+          `Entry ${String(finding.details.entryUuid)} (slug=${String(finding.details.slug)}) ` +
+          `resolves to ${candidates.length} candidate files across legacy site ` +
+          `contentDirs: ${candidates.join(', ')}. The migration refuses to stamp ` +
+          `an ambiguous guess (it would launder a #394-class collision into ` +
+          `permanent data). Disambiguate manually — set the entry's artifactPath ` +
+          `explicitly, or remove the duplicate file — then re-run doctor --fix.`,
+      };
+    }
+    return {
+      kind: 'apply',
+      finding,
+      summary:
+        'migrate legacy sites → lanes: create lanes (host + scaffoldDefaults), ' +
+        'backfill unambiguous entry artifactPaths, drop the sites block',
+      payload: {},
+    };
+  },
+
+  async apply(ctx: DoctorContext, plan: RepairPlan): Promise<RepairResult> {
+    if (plan.kind !== 'apply') {
+      return {
+        finding: plan.finding,
+        applied: false,
+        message: 'plan is not directly appliable',
+        skipReason: 'apply-failed',
+      };
+    }
+
+    try {
+      const { sites } = readLegacySites(ctx.projectRoot);
+      const bases = legacyLaneBases(sites);
+
+      // No legacy sites to migrate, but the detection still fired — that
+      // means it fired on `missingArtifactPath` alone (AUDIT-20260603-15).
+      // Every repair action keys off legacy sites: lane creation iterates
+      // `sites`, and the backfiller only searches each legacy
+      // `site.contentDir`. With no bases there is nothing this migration
+      // can stamp, so claiming `applied: true` would advertise a
+      // successful fix for a run that changed nothing and did not converge
+      // (the next audit re-fires the same finding). Report honestly: the
+      // entries need the lane-native back-fill, not the sites→lanes path.
+      if (bases.length === 0) {
+        const stillMissing = await anyEntryMissingArtifactPath(ctx.projectRoot);
+        if (stillMissing) {
+          return {
+            finding: plan.finding,
+            applied: false,
+            message:
+              'sites-to-lanes: no legacy sites to migrate, but entries still ' +
+              'lack artifactPath. This migration only back-fills from legacy ' +
+              'site contentDirs, of which there are none — it cannot stamp ' +
+              'these entries. Back-fill via the lane-native path ' +
+              '(`migrateLaneMembership` / `/deskwork:lane move`) instead.',
+            skipReason: 'prerequisite-missing',
+          };
+        }
+        // No sites AND nothing missing → genuinely nothing to do.
+        return {
+          finding: plan.finding,
+          applied: false,
+          message:
+            'sites-to-lanes: no legacy sites and no entries missing ' +
+            'artifactPath — nothing to migrate.',
+          skipReason: 'no-action-needed',
+        };
+      }
+
+      // Step 1 — lanes from sites (idempotent: skip an existing lane file).
+      const lanesCreated: string[] = [];
+      if (sites.size > 0) {
+        mkdirSync(lanesDir(ctx.projectRoot), { recursive: true });
+      }
+      for (const [slug, site] of sites) {
+        const target = laneConfigPath(ctx.projectRoot, slug);
+        if (existsSync(target)) {
+          // AUDIT-20260604-10: a pre-existing lane file (re-run / partial
+          // migration) was previously skipped wholesale — then Step 3's
+          // dropSitesBlock removed the only remaining copy of the legacy
+          // `redirectsPath`, silently discarding it. Merge the legacy
+          // redirectsPath onto the existing lane when the lane lacks it.
+          //
+          // Faithfulness (AUDIT-20260604-08-followup / claude-02): reading
+          // via loadLaneConfig is faithful here because LaneConfigSchema is
+          // `.strict()` with NO `.default()` fields — a successfully-loaded
+          // lane carries exactly its on-disk valid fields ($rationale
+          // included), and an unknown key would have THROWN at load (strict),
+          // not been silently stripped. So `{ ...existing, redirectsPath }`
+          // adds one field and re-writes the rest verbatim; commitLaneConfig
+          // re-validates via the same LaneConfigSchema.safeParse gate the
+          // create branch uses below. Only the c4-introduced redirectsPath is
+          // reconciled; other fields are not re-derived (the lane already
+          // exists by operator/earlier intent).
+          if (site.redirectsPath !== undefined) {
+            const existing = loadLaneConfig(slug, ctx.projectRoot);
+            if (existing.redirectsPath === undefined) {
+              commitLaneConfig(
+                ctx.projectRoot,
+                slug,
+                { ...existing, redirectsPath: site.redirectsPath },
+                'merge-legacy-redirectsPath',
+              );
+              await appendJournalEvent(ctx.projectRoot, {
+                kind: 'lane-migration',
+                at: new Date().toISOString(),
+                migration: 'merge-redirectsPath-into-existing-lane',
+                source: `sites.${slug}`,
+                target: `lanes.${slug}`,
+                details: { legacySiteId: slug, redirectsPath: site.redirectsPath },
+              });
+            }
+          }
+          continue;
+        }
+        const lane = laneFromSite(slug, site);
+        const validated = LaneConfigSchema.safeParse(lane);
+        if (!validated.success) {
+          return {
+            finding: plan.finding,
+            applied: false,
+            message:
+              `sites-to-lanes: lane for site "${slug}" failed schema validation: ` +
+              validated.error.message,
+            skipReason: 'apply-failed',
+          };
+        }
+        commitLaneConfig(ctx.projectRoot, slug, validated.data, 'create');
+        await appendJournalEvent(ctx.projectRoot, {
+          kind: 'lane-migration',
+          at: new Date().toISOString(),
+          migration: 'lane-from-legacy-site',
+          source: `sites.${slug}`,
+          target: `lanes.${slug}`,
+          details: {
+            legacySiteId: slug,
+            scaffoldMarkdown: site.contentDir,
+            ...(site.host !== undefined ? { host: site.host } : {}),
+            ...(site.redirectsPath !== undefined
+              ? { redirectsPath: site.redirectsPath }
+              : {}),
+          },
+        });
+        lanesCreated.push(slug);
+      }
+
+      // Step 2 — backfill unambiguous entry artifactPaths (ambiguity-halt).
+      const backfill = await backfillFromLegacySites(ctx.projectRoot, bases);
+
+      // Step 3 — drop the sites block from the config.
+      const dropped = dropSitesBlock(ctx.projectRoot);
+      if (dropped) {
+        await appendJournalEvent(ctx.projectRoot, {
+          kind: 'lane-migration',
+          at: new Date().toISOString(),
+          migration: 'drop-sites-block',
+          source: '.deskwork/config.json#sites',
+          target: '.deskwork/config.json',
+          details: {
+            lanesCreated: lanesCreated.length,
+            entriesStamped: backfill.stamped.length,
+            entriesAmbiguous: backfill.ambiguous.length,
+          },
+        });
+      }
+
+      const ambiguousNote =
+        backfill.ambiguous.length > 0
+          ? ` ${backfill.ambiguous.length} entry(ies) refused as ambiguous ` +
+            `(reported separately as migration-ambiguous).`
+          : '';
+
+      return {
+        finding: plan.finding,
+        applied: true,
+        message:
+          `migrated: ${lanesCreated.length} lane(s) created, ` +
+          `${backfill.stamped.length} entry artifactPath(s) backfilled, ` +
+          `sites block ${dropped ? 'dropped' : 'already absent'}.${ambiguousNote}`,
+        details: {
+          lanesCreated,
+          entriesStamped: backfill.stamped,
+          entriesAmbiguous: backfill.ambiguous.map((a) => a.entryUuid),
+          sitesDropped: dropped,
+        },
+      };
+    } catch (err) {
+      const reason = err instanceof Error ? err.message : String(err);
+      return {
+        finding: plan.finding,
+        applied: false,
+        message: `sites-to-lanes migration failed: ${reason}`,
+        skipReason: 'apply-failed',
+      };
+    }
+  },
+};
+
+function ambiguousFinding(site: string, amb: AmbiguousBackfill): Finding {
+  return {
+    ruleId: AMBIGUOUS_RULE_ID,
+    site,
+    severity: 'error',
+    message:
+      `Entry ${amb.entryUuid} (slug=${amb.slug}, stage=${amb.stage}) resolves to ` +
+      `${amb.candidates.length} candidate files across legacy site contentDirs ` +
+      `(${amb.candidates.join(', ')}). The migration refuses to stamp an ` +
+      `ambiguous artifactPath — disambiguate manually and re-run.`,
+    details: {
+      entryUuid: amb.entryUuid,
+      slug: amb.slug,
+      stage: amb.stage,
+      candidates: amb.candidates,
+    },
+  };
+}
+
+export default rule;

package/dist/doctor/rules/workflow-stale.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"workflow-stale.d.ts","sourceRoot":"","sources":["../../../src/doctor/rules/workflow-stale.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAMH,OAAO,KAAK,EAEV,UAAU,EAIX,MAAM,aAAa,CAAC;AA6CrB,QAAA,MAAM,IAAI,EAAE,UA+EX,CAAC;AAEF,eAAe,IAAI,CAAC"}
+{"version":3,"file":"workflow-stale.d.ts","sourceRoot":"","sources":["../../../src/doctor/rules/workflow-stale.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAMH,OAAO,KAAK,EAEV,UAAU,EAIX,MAAM,aAAa,CAAC;AAiDrB,QAAA,MAAM,IAAI,EAAE,UA+EX,CAAC;AAEF,eAAe,IAAI,CAAC"}

package/dist/doctor/rules/workflow-stale.js

@@ -19,8 +19,11 @@ import { join } from 'node:path';
 import { pipelinePath } from "../../review/pipeline.js";
 const RULE_ID = 'workflow-stale';
 function isStale(workflow, ctx) {
-    if (workflow.site !== ctx.site)
-        return false;
+    // Phase 39c (sites→lanes retirement): the doctor runs a single
+    // project pass and reconciles every workflow against the single
+    // project calendar. The legacy per-site filter (`workflow.site !==
+    // ctx.site`) is retired — `ctx.site` is now the `PROJECT_SCOPE`
+    // sentinel, never a real site slug.
     if (workflow.state === 'applied' || workflow.state === 'cancelled') {
         return false;
     }

package/dist/doctor/rules/workflow-stale.js.map

@@ -1 +1 @@
-{"version":3,"file":"workflow-stale.js","sourceRoot":"","sources":["../../../src/doctor/rules/workflow-stale.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,OAAO,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AAClD,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAUxD,MAAM,OAAO,GAAG,gBAAgB,CAAC;AAEjC,SAAS,OAAO,CACd,QAA2B,EAC3B,GAAkB;IAElB,IAAI,QAAQ,CAAC,IAAI,KAAK,GAAG,CAAC,IAAI;QAAE,OAAO,KAAK,CAAC;IAC7C,IAAI,QAAQ,CAAC,KAAK,KAAK,SAAS,IAAI,QAAQ,CAAC,KAAK,KAAK,WAAW,EAAE,CAAC;QACnE,OAAO,KAAK,CAAC;IACf,CAAC;IACD,oEAAoE;IACpE,qEAAqE;IACrE,oEAAoE;IACpE,IAAI,QAAQ,CAAC,OAAO,EAAE,CAAC;QACrB,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,QAAQ,CAAC,OAAO,CAAC,CAAC;IACtE,CAAC;IACD,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,QAAQ,CAAC,IAAI,CAAC,CAAC;AACrE,CAAC;AAED;;;;GAIG;AACH,SAAS,gBAAgB,CACvB,WAAmB,EACnB,MAA+B,EAC/B,UAAkB;IAElB,MAAM,GAAG,GAAG,YAAY,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;IAC9C,IAAI,KAAe,CAAC;IACpB,IAAI,CAAC;QACH,KAAK,GAAG,WAAW,CAAC,GAAG,CAAC,CAAC;IAC3B,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;IACD,MAAM,MAAM,GAAG,IAAI,UAAU,OAAO,CAAC;IACrC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC;YAAE,OAAO,IAAI,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;IACpD,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,IAAI,GAAe;IACvB,EAAE,EAAE,OAAO;IACX,KAAK,EAAE,wDAAwD;IAE/D,KAAK,CAAC,KAAK,CAAC,GAAkB;QAC5B,MAAM,QAAQ,GAAc,EAAE,CAAC;QAC/B,KAAK,MAAM,CAAC,IAAI,GAAG,CAAC,SAAS,EAAE,CAAC;YAC9B,IAAI,CAAC,OAAO,CAAC,CAAC,EAAE,GAAG,CAAC;gBAAE,SAAS;YAC/B,QAAQ,CAAC,IAAI,CAAC;gBACZ,MAAM,EAAE,OAAO;gBACf,IAAI,EAAE,GAAG,CAAC,IAAI;gBACd,QAAQ,EAAE,SAAS;gBACnB,OAAO,EAAE,YAAY,CAAC,CAAC,EAAE,WAAW,CAAC,CAAC,IAAI,YAAY,CAAC,CAAC,KAAK,kCAAkC;gBAC/F,OAAO,EAAE;oBACP,UAAU,EAAE,CAAC,CAAC,EAAE;oBAChB,IAAI,EAAE,CAAC,CAAC,IAAI;oBACZ,KAAK,EAAE,CAAC,CAAC,KAAK;oBACd,OAAO,EAAE,CAAC,CAAC,OAAO,IAAI,IAAI;iBAC3B;aACF,CAAC,CAAC;QACL,CAAC;QACD,OAAO,QAAQ,CAAC;IAClB,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,IAAmB,EAAE,OAAgB;QAC9C,MAAM,UAAU,GAAG,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,UAAU,IAAI,EAAE,CAAC,CAAC;QAC5D,OAAO;YACL,IAAI,EAAE,OAAO;YACb,OAAO;YACP,OAAO,EAAE,8CAA8C,UAAU,8BAA8B;YAC/F,OAAO,EAAE,EAAE,UAAU,EAAE;SACxB,CAAC;IACJ,CAAC;IAED,KAAK,CAAC,KAAK,CAAC,GAAkB,EAAE,IAAgB;QAC9C,IAAI,IAAI,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;YAC1B,OAAO;gBACL,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,OAAO,EAAE,KAAK;gBACd,OAAO,EAAE,oEAAoE;gBAC7E,UAAU,EAAE,cAAc;aAC3B,CAAC;QACJ,CAAC;QACD,MAAM,UAAU,GAAG,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,IAAI,EAAE,CAAC,CAAC;QACzD,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,OAAO;gBACL,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,OAAO,EAAE,KAAK;gBACd,OAAO,EAAE,kCAAkC;gBAC3C,UAAU,EAAE,cAAc;aAC3B,CAAC;QACJ,CAAC;QACD,MAAM,IAAI,GAAG,gBAAgB,CAAC,GAAG,CAAC,WAAW,EAAE,GAAG,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC;QACvE,IAAI,CAAC,IAAI,EAAE,CAAC;YACV,OAAO;gBACL,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,OAAO,EAAE,KAAK;gBACd,OAAO,EAAE,uCAAuC,UAAU,EAAE;gBAC5D,UAAU,EAAE,cAAc;aAC3B,CAAC;QACJ,CAAC;QACD,IAAI,CAAC;YACH,UAAU,CAAC,IAAI,CAAC,CAAC;QACnB,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,MAAM,MAAM,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;YAChE,OAAO;gBACL,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,OAAO,EAAE,KAAK;gBACd,OAAO,EAAE,oBAAoB,IAAI,KAAK,MAAM,EAAE;gBAC9C,UAAU,EAAE,cAAc;aAC3B,CAAC;QACJ,CAAC;QACD,OAAO;YACL,OAAO,EAAE,IAAI,CAAC,OAAO;YACrB,OAAO,EAAE,IAAI;YACb,OAAO,EAAE,uCAAuC,UAAU,EAAE;YAC5D,OAAO,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;SAC9B,CAAC;IACJ,CAAC;CACF,CAAC;AAEF,eAAe,IAAI,CAAC"}
+{"version":3,"file":"workflow-stale.js","sourceRoot":"","sources":["../../../src/doctor/rules/workflow-stale.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,OAAO,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AAClD,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAUxD,MAAM,OAAO,GAAG,gBAAgB,CAAC;AAEjC,SAAS,OAAO,CACd,QAA2B,EAC3B,GAAkB;IAElB,+DAA+D;IAC/D,gEAAgE;IAChE,mEAAmE;IACnE,gEAAgE;IAChE,oCAAoC;IACpC,IAAI,QAAQ,CAAC,KAAK,KAAK,SAAS,IAAI,QAAQ,CAAC,KAAK,KAAK,WAAW,EAAE,CAAC;QACnE,OAAO,KAAK,CAAC;IACf,CAAC;IACD,oEAAoE;IACpE,qEAAqE;IACrE,oEAAoE;IACpE,IAAI,QAAQ,CAAC,OAAO,EAAE,CAAC;QACrB,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,QAAQ,CAAC,OAAO,CAAC,CAAC;IACtE,CAAC;IACD,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,QAAQ,CAAC,IAAI,CAAC,CAAC;AACrE,CAAC;AAED;;;;GAIG;AACH,SAAS,gBAAgB,CACvB,WAAmB,EACnB,MAA+B,EAC/B,UAAkB;IAElB,MAAM,GAAG,GAAG,YAAY,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;IAC9C,IAAI,KAAe,CAAC;IACpB,IAAI,CAAC;QACH,KAAK,GAAG,WAAW,CAAC,GAAG,CAAC,CAAC;IAC3B,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;IACD,MAAM,MAAM,GAAG,IAAI,UAAU,OAAO,CAAC;IACrC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC;YAAE,OAAO,IAAI,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;IACpD,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,IAAI,GAAe;IACvB,EAAE,EAAE,OAAO;IACX,KAAK,EAAE,wDAAwD;IAE/D,KAAK,CAAC,KAAK,CAAC,GAAkB;QAC5B,MAAM,QAAQ,GAAc,EAAE,CAAC;QAC/B,KAAK,MAAM,CAAC,IAAI,GAAG,CAAC,SAAS,EAAE,CAAC;YAC9B,IAAI,CAAC,OAAO,CAAC,CAAC,EAAE,GAAG,CAAC;gBAAE,SAAS;YAC/B,QAAQ,CAAC,IAAI,CAAC;gBACZ,MAAM,EAAE,OAAO;gBACf,IAAI,EAAE,GAAG,CAAC,IAAI;gBACd,QAAQ,EAAE,SAAS;gBACnB,OAAO,EAAE,YAAY,CAAC,CAAC,EAAE,WAAW,CAAC,CAAC,IAAI,YAAY,CAAC,CAAC,KAAK,kCAAkC;gBAC/F,OAAO,EAAE;oBACP,UAAU,EAAE,CAAC,CAAC,EAAE;oBAChB,IAAI,EAAE,CAAC,CAAC,IAAI;oBACZ,KAAK,EAAE,CAAC,CAAC,KAAK;oBACd,OAAO,EAAE,CAAC,CAAC,OAAO,IAAI,IAAI;iBAC3B;aACF,CAAC,CAAC;QACL,CAAC;QACD,OAAO,QAAQ,CAAC;IAClB,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,IAAmB,EAAE,OAAgB;QAC9C,MAAM,UAAU,GAAG,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,UAAU,IAAI,EAAE,CAAC,CAAC;QAC5D,OAAO;YACL,IAAI,EAAE,OAAO;YACb,OAAO;YACP,OAAO,EAAE,8CAA8C,UAAU,8BAA8B;YAC/F,OAAO,EAAE,EAAE,UAAU,EAAE;SACxB,CAAC;IACJ,CAAC;IAED,KAAK,CAAC,KAAK,CAAC,GAAkB,EAAE,IAAgB;QAC9C,IAAI,IAAI,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;YAC1B,OAAO;gBACL,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,OAAO,EAAE,KAAK;gBACd,OAAO,EAAE,oEAAoE;gBAC7E,UAAU,EAAE,cAAc;aAC3B,CAAC;QACJ,CAAC;QACD,MAAM,UAAU,GAAG,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,IAAI,EAAE,CAAC,CAAC;QACzD,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,OAAO;gBACL,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,OAAO,EAAE,KAAK;gBACd,OAAO,EAAE,kCAAkC;gBAC3C,UAAU,EAAE,cAAc;aAC3B,CAAC;QACJ,CAAC;QACD,MAAM,IAAI,GAAG,gBAAgB,CAAC,GAAG,CAAC,WAAW,EAAE,GAAG,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC;QACvE,IAAI,CAAC,IAAI,EAAE,CAAC;YACV,OAAO;gBACL,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,OAAO,EAAE,KAAK;gBACd,OAAO,EAAE,uCAAuC,UAAU,EAAE;gBAC5D,UAAU,EAAE,cAAc;aAC3B,CAAC;QACJ,CAAC;QACD,IAAI,CAAC;YACH,UAAU,CAAC,IAAI,CAAC,CAAC;QACnB,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,MAAM,MAAM,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;YAChE,OAAO;gBACL,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,OAAO,EAAE,KAAK;gBACd,OAAO,EAAE,oBAAoB,IAAI,KAAK,MAAM,EAAE;gBAC9C,UAAU,EAAE,cAAc;aAC3B,CAAC;QACJ,CAAC;QACD,OAAO;YACL,OAAO,EAAE,IAAI,CAAC,OAAO;YACrB,OAAO,EAAE,IAAI;YACb,OAAO,EAAE,uCAAuC,UAAU,EAAE;YAC5D,OAAO,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;SAC9B,CAAC;IACJ,CAAC;CACF,CAAC;AAEF,eAAe,IAAI,CAAC"}

package/dist/doctor/rules/workflow-stale.ts

@@ -33,7 +33,11 @@ function isStale(
   workflow: DraftWorkflowItem,
   ctx: DoctorContext,
 ): boolean {
-  if (workflow.site !== ctx.site) return false;
+  // Phase 39c (sites→lanes retirement): the doctor runs a single
+  // project pass and reconciles every workflow against the single
+  // project calendar. The legacy per-site filter (`workflow.site !==
+  // ctx.site`) is retired — `ctx.site` is now the `PROJECT_SCOPE`
+  // sentinel, never a real site slug.
   if (workflow.state === 'applied' || workflow.state === 'cancelled') {
     return false;
   }

package/dist/doctor/runner.d.ts

@@ -38,11 +38,25 @@ export declare function parseFixArgument(arg: string): string[];
 export interface DoctorRunOptions {
     projectRoot: string;
     config: DeskworkConfig;
-    /** Restrict to one site; undefined = run for every site in config. */
+    /**
+     * Phase 39c (sites→lanes retirement): the doctor runs a SINGLE
+     * project-scoped pass. `site` is retained on the options shape (the
+     * `--site` CLI flag still parses) but it no longer scopes the run —
+     * the doctor operates on the project's sidecar set, calendar, and
+     * workflow store as a whole.
+     */
     site?: string;
     /** Restrict the rule set; undefined = all rules. */
     ruleIds?: string[];
 }
+/**
+ * The single project scope label. Phase 39c collapsed the doctor's
+ * per-site loop (location is no longer the identifying axis — the
+ * sidecar set is the source of truth) into one project pass. Findings
+ * carry this as their `site` label so existing consumers that group by
+ * `finding.site` still see a stable key.
+ */
+export declare const PROJECT_SCOPE = "project";
 /**
  * Audit: collect findings without mutating the world. Returns a fully-
  * built report with empty `repairs`. Suitable for pre-commit hooks

package/dist/doctor/runner.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/doctor/runner.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAMH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAcnD,OAAO,KAAK,EAEV,iBAAiB,EACjB,YAAY,EACZ,UAAU,EAIX,MAAM,YAAY,CAAC;AAEpB;;;;;;;;;;GAUG;AACH,eAAO,MAAM,KAAK,EAAE,aAAa,CAAC,UAAU,CAa3C,CAAC;AAMF;;;;;;;;;;;GAWG;AACH,wBAAgB,gBAAgB,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,EAAE,CAiBtD;AAED,MAAM,WAAW,gBAAgB;IAC/B,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,cAAc,CAAC;IACvB,sEAAsE;IACtE,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,oDAAoD;IACpD,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC;CACpB;AAwED;;;;GAIG;AACH,wBAAsB,QAAQ,CAC5B,IAAI,EAAE,gBAAgB,EACtB,WAAW,EAAE,iBAAiB,GAC7B,OAAO,CAAC,YAAY,CAAC,CAavB;AAED;;;;;;;;GAQG;AACH,wBAAsB,SAAS,CAC7B,IAAI,EAAE,gBAAgB,EACtB,WAAW,EAAE,iBAAiB,GAC7B,OAAO,CAAC,YAAY,CAAC,CAoBvB;AA+FD;;;;;GAKG;AACH,eAAO,MAAM,cAAc,EAAE,iBAU5B,CAAC;AAEF;;;GAGG;AACH,eAAO,MAAM,kBAAkB,EAAE,iBAOhC,CAAC"}
+{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/doctor/runner.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAMH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAenD,OAAO,KAAK,EAEV,iBAAiB,EACjB,YAAY,EACZ,UAAU,EAIX,MAAM,YAAY,CAAC;AAEpB;;;;;;;;;;GAUG;AACH,eAAO,MAAM,KAAK,EAAE,aAAa,CAAC,UAAU,CAc3C,CAAC;AAMF;;;;;;;;;;;GAWG;AACH,wBAAgB,gBAAgB,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,EAAE,CAiBtD;AAED,MAAM,WAAW,gBAAgB;IAC/B,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,cAAc,CAAC;IACvB;;;;;;OAMG;IACH,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,oDAAoD;IACpD,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC;CACpB;AAED;;;;;;GAMG;AACH,eAAO,MAAM,aAAa,YAAY,CAAC;AA+EvC;;;;GAIG;AACH,wBAAsB,QAAQ,CAC5B,IAAI,EAAE,gBAAgB,EACtB,WAAW,EAAE,iBAAiB,GAC7B,OAAO,CAAC,YAAY,CAAC,CAQvB;AAED;;;;;;;;GAQG;AACH,wBAAsB,SAAS,CAC7B,IAAI,EAAE,gBAAgB,EACtB,WAAW,EAAE,iBAAiB,GAC7B,OAAO,CAAC,YAAY,CAAC,CAcvB;AA+FD;;;;;GAKG;AACH,eAAO,MAAM,cAAc,EAAE,iBAU5B,CAAC;AAEF;;;GAGG;AACH,eAAO,MAAM,kBAAkB,EAAE,iBAOhC,CAAC"}