lsd-pi 1.0.0 → 1.1.0

package/src/resources/extensions/gsd/preferences.ts

@@ -1,539 +0,0 @@
-/**
- * GSD Preferences -- loading, merging, and rendering.
- *
- * This module is the primary entry point for preference operations.
- * Type definitions live in ./preferences-types.js, validation in
- * ./preferences-validation.js, skill logic in ./preferences-skills.js,
- * and model logic in ./preferences-models.js.
- *
- * All symbols are re-exported here so that existing `import { ... } from "./preferences.js"`
- * statements continue to work without modification.
- */
-
-import { existsSync, readFileSync } from "node:fs";
-import { homedir } from "node:os";
-import { join } from "node:path";
-
-import { gsdRoot } from "./paths.js";
-import { parse as parseYaml } from "yaml";
-import type { PostUnitHookConfig, PreDispatchHookConfig, TokenProfile } from "./types.js";
-import type { DynamicRoutingConfig } from "./model-router.js";
-import { normalizeStringArray } from "../shared/format-utils.js";
-import { resolveProfileDefaults as _resolveProfileDefaults } from "./preferences-models.js";
-
-import {
-  KNOWN_PREFERENCE_KEYS,
-  MODE_DEFAULTS,
-  type WorkflowMode,
-  type GSDPreferences,
-  type LoadedGSDPreferences,
-  type SkillResolution,
-} from "./preferences-types.js";
-import { validatePreferences } from "./preferences-validation.js";
-import { formatSkillRef } from "./preferences-skills.js";
-
-// ─── Re-exports: types ──────────────────────────────────────────────────────
-// Every type/interface that was previously exported from this file is
-// re-exported so that downstream `import { Foo } from "./preferences.js"`
-// statements keep compiling.
-
-export type {
-  WorkflowMode,
-  GSDSkillRule,
-  GSDPhaseModelConfig,
-  GSDModelConfig,
-  GSDModelConfigV2,
-  ResolvedModelConfig,
-  SkillDiscoveryMode,
-  AutoSupervisorConfig,
-  RemoteQuestionsConfig,
-  CmuxPreferences,
-  GSDPreferences,
-  LoadedGSDPreferences,
-  SkillResolution,
-  SkillResolutionReport,
-} from "./preferences-types.js";
-
-// ─── Re-exports: validation ─────────────────────────────────────────────────
-export { validatePreferences } from "./preferences-validation.js";
-
-// ─── Re-exports: skills ─────────────────────────────────────────────────────
-export {
-  resolveAllSkillReferences,
-  resolveSkillDiscoveryMode,
-  resolveSkillStalenessDays,
-} from "./preferences-skills.js";
-
-// ─── Re-exports: models ─────────────────────────────────────────────────────
-export {
-  resolveModelForUnit,
-  resolveModelWithFallbacksForUnit,
-  getNextFallbackModel,
-  isTransientNetworkError,
-  validateModelId,
-  updatePreferencesModels,
-  resolveDynamicRoutingConfig,
-  resolveAutoSupervisorConfig,
-  resolveProfileDefaults,
-  resolveEffectiveProfile,
-  resolveInlineLevel,
-  resolveContextSelection,
-  resolveSearchProviderFromPreferences,
-} from "./preferences-models.js";
-
-// ─── Path Constants & Getters ───────────────────────────────────────────────
-
-function gsdHome(): string {
-  return process.env.LSD_HOME || join(homedir(), ".lsd");
-}
-
-function globalPreferencesPath(): string {
-  return join(gsdHome(), "preferences.md");
-}
-
-function legacyGlobalPreferencesPath(): string {
-  return join(homedir(), ".pi", "agent", "gsd-preferences.md");
-}
-
-function projectPreferencesPath(): string {
-  return join(gsdRoot(process.cwd()), "preferences.md");
-}
-// Bootstrap in gitignore.ts historically created PREFERENCES.md (uppercase) by mistake.
-// Check uppercase as a fallback so those files aren't silently ignored.
-function globalPreferencesPathUppercase(): string {
-  return join(gsdHome(), "PREFERENCES.md");
-}
-function projectPreferencesPathUppercase(): string {
-  return join(gsdRoot(process.cwd()), "PREFERENCES.md");
-}
-
-export function getGlobalGSDPreferencesPath(): string {
-  return globalPreferencesPath();
-}
-
-export function getLegacyGlobalGSDPreferencesPath(): string {
-  return legacyGlobalPreferencesPath();
-}
-
-export function getProjectGSDPreferencesPath(): string {
-  return projectPreferencesPath();
-}
-
-// ─── Loading ────────────────────────────────────────────────────────────────
-
-export function loadGlobalGSDPreferences(): LoadedGSDPreferences | null {
-  return loadPreferencesFile(globalPreferencesPath(), "global")
-    ?? loadPreferencesFile(globalPreferencesPathUppercase(), "global")
-    ?? loadPreferencesFile(legacyGlobalPreferencesPath(), "global");
-}
-
-export function loadProjectGSDPreferences(): LoadedGSDPreferences | null {
-  return loadPreferencesFile(projectPreferencesPath(), "project")
-    ?? loadPreferencesFile(projectPreferencesPathUppercase(), "project");
-}
-
-export function loadEffectiveGSDPreferences(): LoadedGSDPreferences | null {
-  const globalPreferences = loadGlobalGSDPreferences();
-  const projectPreferences = loadProjectGSDPreferences();
-
-  if (!globalPreferences && !projectPreferences) return null;
-
-  let result: LoadedGSDPreferences;
-  if (!globalPreferences) {
-    result = projectPreferences!;
-  } else if (!projectPreferences) {
-    result = globalPreferences;
-  } else {
-    const mergedWarnings = [
-      ...(globalPreferences.warnings ?? []),
-      ...(projectPreferences.warnings ?? []),
-    ];
-    result = {
-      path: projectPreferences.path,
-      scope: "project",
-      preferences: mergePreferences(globalPreferences.preferences, projectPreferences.preferences),
-      ...(mergedWarnings.length > 0 ? { warnings: mergedWarnings } : {}),
-    };
-  }
-
-  // Apply token-profile defaults as the lowest-priority layer so that
-  // `token_profile: budget` sets models and phase-skips automatically.
-  // Explicit user preferences always override profile defaults.
-  const profile = result.preferences.token_profile as TokenProfile | undefined;
-  if (profile) {
-    const profileDefaults = _resolveProfileDefaults(profile);
-    result = {
-      ...result,
-      preferences: mergePreferences(profileDefaults as GSDPreferences, result.preferences),
-    };
-  }
-
-  // Apply mode defaults as the lowest-priority layer
-  if (result.preferences.mode) {
-    result = {
-      ...result,
-      preferences: applyModeDefaults(result.preferences.mode, result.preferences),
-    };
-  }
-
-  return result;
-}
-
-function loadPreferencesFile(path: string, scope: "global" | "project"): LoadedGSDPreferences | null {
-  if (!existsSync(path)) return null;
-
-  const raw = readFileSync(path, "utf-8");
-  const preferences = parsePreferencesMarkdown(raw);
-  if (!preferences) return null;
-
-  const validation = validatePreferences(preferences);
-  const allWarnings = [...validation.warnings, ...validation.errors];
-
-  return {
-    path,
-    scope,
-    preferences: validation.preferences,
-    ...(allWarnings.length > 0 ? { warnings: allWarnings } : {}),
-  };
-}
-
-let _warnedUnrecognizedFormat = false;
-
-/** @internal Reset the warn-once flag — exported for testing only. */
-export function _resetParseWarningFlag(): void {
-  _warnedUnrecognizedFormat = false;
-}
-
-/** @internal Exported for testing only */
-export function parsePreferencesMarkdown(content: string): GSDPreferences | null {
-  // Use indexOf instead of [\s\S]*? regex to avoid backtracking (#468)
-  const startMarker = content.startsWith('---\r\n') ? '---\r\n' : '---\n';
-  if (content.startsWith(startMarker)) {
-    const searchStart = startMarker.length;
-    const endIdx = content.indexOf('\n---', searchStart);
-    if (endIdx === -1) return null;
-    const block = content.slice(searchStart, endIdx);
-    return parseFrontmatterBlock(block.replace(/\r/g, ''));
-  }
-
-  // Fallback: heading+list format (e.g. "## Git\n- isolation: none") (#2036)
-  // GSD agents may write preferences files without frontmatter delimiters.
-  if (/^##\s+\w/m.test(content)) {
-    return parseHeadingListFormat(content);
-  }
-
-  if (!_warnedUnrecognizedFormat) {
-    _warnedUnrecognizedFormat = true;
-    console.warn("[parsePreferencesMarkdown] preferences.md exists but uses an unrecognized format — skipping.");
-  }
-  return null;
-}
-
-function parseFrontmatterBlock(frontmatter: string): GSDPreferences {
-  try {
-    const parsed = parseYaml(frontmatter);
-    if (typeof parsed !== 'object' || parsed === null) {
-      return {} as GSDPreferences;
-    }
-    return parsed as GSDPreferences;
-  } catch (e) {
-    console.error("[parseFrontmatterBlock] YAML parse error:", e);
-    return {} as GSDPreferences;
-  }
-}
-
-/**
- * Parse heading+list format into a nested object, then cast to GSDPreferences.
- * Handles markdown like:
- *   ## Git
- *   - isolation: none
- *   - commit_docs: true
- *   ## Models
- *   - planner: sonnet
- */
-function parseHeadingListFormat(content: string): GSDPreferences {
-  const result: Record<string, string[]> = {};
-  let currentSection: string | null = null;
-
-  for (const rawLine of content.split('\n')) {
-    const line = rawLine.replace(/\r$/, '');
-    const headingMatch = line.match(/^##\s+(.+)$/);
-    if (headingMatch) {
-      currentSection = headingMatch[1].trim().toLowerCase().replace(/\s+/g, '_');
-      if (!result[currentSection]) result[currentSection] = [];
-      continue;
-    }
-    if (currentSection && line.trim() && !line.trimStart().startsWith('#')) {
-      result[currentSection].push(line);
-    }
-  }
-
-  const typed: Record<string, unknown> = {};
-  for (const [section, lines] of Object.entries(result)) {
-    if (lines.length === 0) continue;
-
-    const usesLegacyListItems = lines.every((line) => /^\s*-\s+[^:]+:\s*.*$/.test(line));
-    const yamlBlock = usesLegacyListItems
-      ? lines.map((line) => line.replace(/^\s*-\s+/, '')).join('\n')
-      : lines.join('\n');
-
-    try {
-      const parsed = parseYaml(yamlBlock);
-      if (typeof parsed !== 'object' || parsed === null) continue;
-
-      let targetSection = section;
-      let value: unknown = parsed;
-
-      if (!Array.isArray(parsed)) {
-        const keys = Object.keys(parsed);
-        if (keys.length === 1) {
-          const [onlyKey] = keys;
-          if (onlyKey === section || (!KNOWN_PREFERENCE_KEYS.has(section) && KNOWN_PREFERENCE_KEYS.has(onlyKey))) {
-            targetSection = onlyKey;
-            value = (parsed as Record<string, unknown>)[onlyKey];
-          }
-        }
-      }
-
-      typed[targetSection] = value;
-    } catch {
-      /* malformed section — skip */
-    }
-  }
-
-  return typed as GSDPreferences;
-}
-
-// ─── Merging ────────────────────────────────────────────────────────────────
-
-/**
- * Apply mode defaults as the lowest-priority layer.
- * Mode defaults fill in undefined fields; any explicit user value wins.
- */
-export function applyModeDefaults(mode: WorkflowMode, prefs: GSDPreferences): GSDPreferences {
-  const defaults = MODE_DEFAULTS[mode];
-  if (!defaults) return prefs;
-  return mergePreferences(defaults, prefs);
-}
-
-function mergePreferences(base: GSDPreferences, override: GSDPreferences): GSDPreferences {
-  return {
-    version: override.version ?? base.version,
-    mode: override.mode ?? base.mode,
-    always_use_skills: mergeStringLists(base.always_use_skills, override.always_use_skills),
-    prefer_skills: mergeStringLists(base.prefer_skills, override.prefer_skills),
-    avoid_skills: mergeStringLists(base.avoid_skills, override.avoid_skills),
-    skill_rules: [...(base.skill_rules ?? []), ...(override.skill_rules ?? [])],
-    custom_instructions: mergeStringLists(base.custom_instructions, override.custom_instructions),
-    models: { ...(base.models ?? {}), ...(override.models ?? {}) },
-    skill_discovery: override.skill_discovery ?? base.skill_discovery,
-    skill_staleness_days: override.skill_staleness_days ?? base.skill_staleness_days,
-    auto_supervisor: { ...(base.auto_supervisor ?? {}), ...(override.auto_supervisor ?? {}) },
-    uat_dispatch: override.uat_dispatch ?? base.uat_dispatch,
-    unique_milestone_ids: override.unique_milestone_ids ?? base.unique_milestone_ids,
-    budget_ceiling: override.budget_ceiling ?? base.budget_ceiling,
-    budget_enforcement: override.budget_enforcement ?? base.budget_enforcement,
-    context_pause_threshold: override.context_pause_threshold ?? base.context_pause_threshold,
-    notifications: (base.notifications || override.notifications)
-      ? { ...(base.notifications ?? {}), ...(override.notifications ?? {}) }
-      : undefined,
-    cmux: (base.cmux || override.cmux)
-      ? { ...(base.cmux ?? {}), ...(override.cmux ?? {}) }
-      : undefined,
-    remote_questions: override.remote_questions
-      ? { ...(base.remote_questions ?? {}), ...override.remote_questions }
-      : base.remote_questions,
-    git: (base.git || override.git)
-      ? { ...(base.git ?? {}), ...(override.git ?? {}) }
-      : undefined,
-    post_unit_hooks: mergePostUnitHooks(base.post_unit_hooks, override.post_unit_hooks),
-    pre_dispatch_hooks: mergePreDispatchHooks(base.pre_dispatch_hooks, override.pre_dispatch_hooks),
-    dynamic_routing: (base.dynamic_routing || override.dynamic_routing)
-      ? { ...(base.dynamic_routing ?? {}), ...(override.dynamic_routing ?? {}) } as DynamicRoutingConfig
-      : undefined,
-    token_profile: override.token_profile ?? base.token_profile,
-    phases: (base.phases || override.phases)
-      ? { ...(base.phases ?? {}), ...(override.phases ?? {}) }
-      : undefined,
-    parallel: (base.parallel || override.parallel)
-      ? { ...(base.parallel ?? {}), ...(override.parallel ?? {}) } as import("./types.js").ParallelConfig
-      : undefined,
-    verification_commands: mergeStringLists(base.verification_commands, override.verification_commands),
-    verification_auto_fix: override.verification_auto_fix ?? base.verification_auto_fix,
-    verification_max_retries: override.verification_max_retries ?? base.verification_max_retries,
-    search_provider: override.search_provider ?? base.search_provider,
-    context_selection: override.context_selection ?? base.context_selection,
-    auto_visualize: override.auto_visualize ?? base.auto_visualize,
-    auto_report: override.auto_report ?? base.auto_report,
-    github: (base.github || override.github)
-      ? { ...(base.github ?? {}), ...(override.github ?? {}) } as import("../github-sync/types.js").GitHubSyncConfig
-      : undefined,
-    service_tier: override.service_tier ?? base.service_tier,
-    forensics_dedup: override.forensics_dedup ?? base.forensics_dedup,
-    show_token_cost: override.show_token_cost ?? base.show_token_cost,
-  };
-}
-
-function mergeStringLists(base?: unknown, override?: unknown): string[] | undefined {
-  const merged = [
-    ...normalizeStringArray(base),
-    ...normalizeStringArray(override),
-  ]
-    .map((item) => item.trim())
-    .filter(Boolean);
-  return merged.length > 0 ? Array.from(new Set(merged)) : undefined;
-}
-
-function mergePostUnitHooks(
-  base?: PostUnitHookConfig[],
-  override?: PostUnitHookConfig[],
-): PostUnitHookConfig[] | undefined {
-  if (!base?.length && !override?.length) return undefined;
-  const merged = [...(base ?? [])];
-  for (const hook of override ?? []) {
-    // Override hooks with same name replace base hooks
-    const idx = merged.findIndex(h => h.name === hook.name);
-    if (idx >= 0) {
-      merged[idx] = hook;
-    } else {
-      merged.push(hook);
-    }
-  }
-  return merged.length > 0 ? merged : undefined;
-}
-
-function mergePreDispatchHooks(
-  base?: PreDispatchHookConfig[],
-  override?: PreDispatchHookConfig[],
-): PreDispatchHookConfig[] | undefined {
-  if (!base?.length && !override?.length) return undefined;
-  const merged = [...(base ?? [])];
-  for (const hook of override ?? []) {
-    const idx = merged.findIndex(h => h.name === hook.name);
-    if (idx >= 0) {
-      merged[idx] = hook;
-    } else {
-      merged.push(hook);
-    }
-  }
-  return merged.length > 0 ? merged : undefined;
-}
-
-// ─── System Prompt Rendering ──────────────────────────────────────────────────
-
-export function renderPreferencesForSystemPrompt(preferences: GSDPreferences, resolutions?: Map<string, SkillResolution>): string {
-  const validated = validatePreferences(preferences);
-  const lines: string[] = ["## GSD Skill Preferences"];
-
-  if (validated.errors.length > 0) {
-    lines.push("- Validation: some preference values were ignored because they were invalid.");
-  }
-  for (const warning of validated.warnings) {
-    lines.push(`- Deprecation: ${warning}`);
-  }
-
-  preferences = validated.preferences;
-
-  lines.push(
-    "- Treat these as explicit skill-selection policy for GSD work.",
-    "- If a listed skill exists and is relevant, load and follow it instead of treating it as a vague suggestion.",
-    "- Current user instructions still override these defaults.",
-  );
-
-  const fmt = (ref: string) => resolutions ? formatSkillRef(ref, resolutions) : ref;
-
-  if (preferences.always_use_skills && preferences.always_use_skills.length > 0) {
-    lines.push("- Always use these skills when relevant:");
-    for (const skill of preferences.always_use_skills) {
-      lines.push(`  - ${fmt(skill)}`);
-    }
-  }
-
-  if (preferences.prefer_skills && preferences.prefer_skills.length > 0) {
-    lines.push("- Prefer these skills when relevant:");
-    for (const skill of preferences.prefer_skills) {
-      lines.push(`  - ${fmt(skill)}`);
-    }
-  }
-
-  if (preferences.avoid_skills && preferences.avoid_skills.length > 0) {
-    lines.push("- Avoid these skills unless clearly needed:");
-    for (const skill of preferences.avoid_skills) {
-      lines.push(`  - ${fmt(skill)}`);
-    }
-  }
-
-  if (preferences.skill_rules && preferences.skill_rules.length > 0) {
-    lines.push("- Situational rules:");
-    for (const rule of preferences.skill_rules) {
-      lines.push(`  - When ${rule.when}:`);
-      if (rule.use && rule.use.length > 0) {
-        lines.push(`    - use: ${rule.use.map(fmt).join(", ")}`);
-      }
-      if (rule.prefer && rule.prefer.length > 0) {
-        lines.push(`    - prefer: ${rule.prefer.map(fmt).join(", ")}`);
-      }
-      if (rule.avoid && rule.avoid.length > 0) {
-        lines.push(`    - avoid: ${rule.avoid.map(fmt).join(", ")}`);
-      }
-    }
-  }
-
-  if (preferences.custom_instructions && preferences.custom_instructions.length > 0) {
-    lines.push("- Additional instructions:");
-    for (const instruction of preferences.custom_instructions) {
-      lines.push(`  - ${instruction}`);
-    }
-  }
-
-  return lines.join("\n");
-}
-
-// ─── Hook Resolution ──────────────────────────────────────────────────────────
-
-/**
- * Resolve enabled post-unit hooks from effective preferences.
- * Returns an empty array when no hooks are configured.
- */
-export function resolvePostUnitHooks(): PostUnitHookConfig[] {
-  const prefs = loadEffectiveGSDPreferences();
-  return (prefs?.preferences.post_unit_hooks ?? [])
-    .filter(h => h.enabled !== false);
-}
-
-/**
- * Resolve enabled pre-dispatch hooks from effective preferences.
- * Returns an empty array when no hooks are configured.
- */
-export function resolvePreDispatchHooks(): PreDispatchHookConfig[] {
-  const prefs = loadEffectiveGSDPreferences();
-  return (prefs?.preferences.pre_dispatch_hooks ?? [])
-    .filter(h => h.enabled !== false);
-}
-
-// ─── Isolation & Parallel ─────────────────────────────────────────────────────
-
-/**
- * Resolve the effective git isolation mode from preferences.
- * Returns "none" (default), "worktree", or "branch".
- *
- * Default is "none" so GSD works out of the box without preferences.md.
- * Worktree isolation requires explicit opt-in because it depends on git
- * branch infrastructure that must be set up before use.
- */
-export function getIsolationMode(): "none" | "worktree" | "branch" {
-  const prefs = loadEffectiveGSDPreferences()?.preferences?.git;
-  if (prefs?.isolation === "worktree") return "worktree";
-  if (prefs?.isolation === "branch") return "branch";
-  return "none"; // default — no isolation, work on current branch
-}
-
-export function resolveParallelConfig(prefs: GSDPreferences | undefined): import("./types.js").ParallelConfig {
-  return {
-    enabled: prefs?.parallel?.enabled ?? false,
-    max_workers: Math.max(1, Math.min(4, prefs?.parallel?.max_workers ?? 2)),
-    budget_ceiling: prefs?.parallel?.budget_ceiling,
-    merge_strategy: prefs?.parallel?.merge_strategy ?? "per-milestone",
-    auto_merge: prefs?.parallel?.auto_merge ?? "confirm",
-  };
-}

package/src/resources/extensions/gsd/progress-score.ts

@@ -1,161 +0,0 @@
-/**
- * GSD Progress Score — Traffic Light Status Indicator (#1221)
- *
- * Combines existing health signals into a single at-a-glance status:
- *   - Green: progressing well
- *   - Yellow: struggling (retries, warnings)
- *   - Red: stuck (loops, persistent errors, no activity)
- *
- * Purely derived — no stored state. Reads from doctor-proactive health
- * tracking, stuck detection counters, and working-tree activity.
- */
-
-import {
-  getHealthTrend,
-  getConsecutiveErrorUnits,
-  getHealthHistory,
-  getLatestHealthIssues,
-  getLatestHealthFixes,
-  type HealthSnapshot,
-} from "./doctor-proactive.js";
-
-// ── Types ──────────────────────────────────────────────────────────────────
-
-export type ProgressLevel = "green" | "yellow" | "red";
-
-export interface ProgressScore {
-  level: ProgressLevel;
-  summary: string;
-  signals: ProgressSignal[];
-}
-
-export interface ProgressSignal {
-  kind: "positive" | "negative" | "neutral";
-  label: string;
-}
-
-function escalateLevel(level: ProgressLevel, next: ProgressLevel): ProgressLevel {
-  const ranks: Record<ProgressLevel, number> = {
-    green: 0,
-    yellow: 1,
-    red: 2,
-  };
-  return ranks[next] > ranks[level] ? next : level;
-}
-
-// ── Public API ──────────────────────────────────────────────────────────────
-
-/**
- * Compute the current progress score from health signals.
- */
-export function computeProgressScore(): ProgressScore {
-  const signals: ProgressSignal[] = [];
-  let level: ProgressLevel = "green";
-
-  // Check consecutive errors
-  const consecutiveErrors = getConsecutiveErrorUnits();
-  if (consecutiveErrors >= 3) {
-    signals.push({ kind: "negative", label: `${consecutiveErrors} consecutive error units` });
-    level = escalateLevel(level, "red");
-  } else if (consecutiveErrors >= 1) {
-    signals.push({ kind: "negative", label: `${consecutiveErrors} consecutive error unit(s)` });
-    level = escalateLevel(level, "yellow");
-  }
-
-  // Check health trend
-  const trend = getHealthTrend();
-  if (trend === "degrading") {
-    signals.push({ kind: "negative", label: "Health trend declining" });
-    level = escalateLevel(level, "yellow");
-  } else if (trend === "improving") {
-    signals.push({ kind: "positive", label: "Health trend improving" });
-  } else if (trend === "stable") {
-    signals.push({ kind: "neutral", label: "Health trend stable" });
-  }
-
-  // Check recent history
-  const history = getHealthHistory();
-  if (history.length === 0) {
-    signals.push({ kind: "neutral", label: "No health data yet" });
-  }
-
-  // Surface actual doctor issue details when degraded
-  if (level !== "green") {
-    const latestIssues = getLatestHealthIssues();
-    // Show up to 5 most relevant issues (errors first, then warnings)
-    const sorted = [...latestIssues].sort((a, b) => {
-      const rank = { error: 0, warning: 1, info: 2 };
-      return rank[a.severity] - rank[b.severity];
-    });
-    for (const issue of sorted.slice(0, 5)) {
-      signals.push({
-        kind: issue.severity === "error" ? "negative" : "neutral",
-        label: issue.message,
-      });
-    }
-
-    const latestFixes = getLatestHealthFixes();
-    for (const fix of latestFixes.slice(0, 3)) {
-      signals.push({ kind: "positive", label: `Fixed: ${fix}` });
-    }
-  }
-
-  const summary = level === "green"
-    ? "Progressing well"
-    : level === "yellow"
-      ? "Some issues detected"
-      : "Stuck or erroring";
-
-  return { level, summary, signals };
-}
-
-/**
- * Compute progress score with additional context for dashboard display.
- */
-export function computeProgressScoreWithContext(context: {
-  sameUnitCount?: number;
-  recoveryCount?: number;
-  completedCount?: number;
-}): ProgressScore {
-  const base = computeProgressScore();
-
-  if (context.sameUnitCount && context.sameUnitCount >= 3) {
-    base.signals.push({ kind: "negative", label: `Same unit dispatched ${context.sameUnitCount}× consecutively` });
-    base.level = escalateLevel(base.level, "red");
-    base.summary = "Stuck on same unit";
-  } else if (context.sameUnitCount && context.sameUnitCount >= 2) {
-    base.signals.push({ kind: "negative", label: `Same unit dispatched ${context.sameUnitCount}×` });
-    base.level = escalateLevel(base.level, "yellow");
-  }
-
-  if (context.recoveryCount && context.recoveryCount > 0) {
-    base.signals.push({ kind: "negative", label: `${context.recoveryCount} recovery attempts` });
-    base.level = escalateLevel(base.level, "yellow");
-  }
-
-  if (context.completedCount && context.completedCount > 0) {
-    base.signals.push({ kind: "positive", label: `${context.completedCount} units completed` });
-  }
-
-  return base;
-}
-
-/**
- * Format a one-line progress indicator for dashboard/status display.
- */
-export function formatProgressLine(score: ProgressScore): string {
-  const icon = score.level === "green" ? "●" : score.level === "yellow" ? "◐" : "○";
-  return `${icon} ${score.summary}`;
-}
-
-/**
- * Format a multi-line progress report.
- */
-export function formatProgressReport(score: ProgressScore): string {
-  const lines = [formatProgressLine(score)];
-  for (const signal of score.signals) {
-    const prefix = signal.kind === "positive" ? "  ✓" : signal.kind === "negative" ? "  ✗" : "  ·";
-    lines.push(`${prefix} ${signal.label}`);
-  }
-  return lines.join("\n");
-}