codecartographer-pi 0.20.0 → 0.21.0

package/dist/core/utils.d.ts

@@ -77,3 +77,10 @@ export declare function formatMillis(ms: number): string;
  * hand-edited markers) so callers can fall back to string equality.
  */
 export declare function compareDottedVersions(a: string, b: string): number | null;
+/**
+ * Every git subprocess gets this. A hung git — a credential helper waiting
+ * on a prompt, a slow filesystem — used to hang a publish or a Broad-Side
+ * submit indefinitely while every fetch carried a 30 s timeout (self-audit
+ * sem 3.12).
+ */
+export declare const GIT_TIMEOUT_MS = 30000;

package/dist/core/utils.js

@@ -206,3 +206,10 @@ export function compareDottedVersions(a, b) {
     }
     return 0;
 }
+/**
+ * Every git subprocess gets this. A hung git — a credential helper waiting
+ * on a prompt, a slow filesystem — used to hang a publish or a Broad-Side
+ * submit indefinitely while every fetch carried a 30 s timeout (self-audit
+ * sem 3.12).
+ */
+export const GIT_TIMEOUT_MS = 30_000;

package/dist/core/workspace.js

@@ -7,7 +7,7 @@ import { appendFile, copyFile, cp, mkdir, readFile, readdir, rename } from "node
 import { basename, dirname, join, relative } from "node:path";
 import { fileURLToPath } from "node:url";
 import { getPipelineLabel, recomputeCursor } from "./pipeline.js";
-import { acquireLock, applyHandoff, autoAssignIds, createEmptyStatus, normalizeStatus, parseHandoff } from "./status.js";
+import { acquireLock, applyHandoff, autoAssignIds, createEmptyStatus, normalizeStatus, parseHandoff, textOf } from "./status.js";
 import { atomicWriteFile, compareDottedVersions, newlineIfUnterminated, pathExists } from "./utils.js";
 import { loadYamlFile, stringifySimpleYaml } from "./yaml.js";
 // Walk up from the current file to find the package root. Needed because the
@@ -68,7 +68,7 @@ export async function getWorkspaceState(cwd) {
     if (!(await pathExists(statusPath)))
         return null;
     const rawStatus = await loadYamlFile(statusPath);
-    const pipelineRelativePath = rawStatus.pipeline?.trim();
+    const pipelineRelativePath = (textOf(rawStatus.pipeline) ?? "").trim();
     if (!pipelineRelativePath) {
         throw new Error(`Missing pipeline in ${relative(cwd, statusPath) || statusPath}`);
     }

package/dist/core/yaml.js

@@ -497,8 +497,15 @@ export function formatYamlScalar(value) {
     const stringValue = String(value);
     if (stringValue === "")
         return '""';
-    if (/^[A-Za-z0-9_./-]+$/.test(stringValue))
+    // Bare only when the reader would hand the same string back. The charset
+    // keeps out spaces, quotes, `#` and `:`; the round-trip check keeps out the
+    // strings the reader coerces — "2048", "true", "null", "1.5" — which used
+    // to go out bare and come back as a number, a boolean, or nothing, until a
+    // repository named `2048` bricked its workspace on `project_name.trim`
+    // (#225). A lone `-` is a sequence marker inside a list, so it is quoted too.
+    if (/^[A-Za-z0-9_./-]+$/.test(stringValue) && stringValue !== "-" && parseYamlScalar(stringValue) === stringValue) {
         return stringValue;
+    }
     return JSON.stringify(stringValue);
 }
 export function stringifySimpleYaml(value, indent = 0) {

package/dist/extensions/codecarto/auto-runner.d.ts

@@ -45,7 +45,7 @@ export interface AutoCompleteResult {
     warnings: string[];
 }
 export declare function autoCompletePhase(cwd: string, validation: ValidationResult): Promise<AutoCompleteResult>;
-export type AutoOutcome = "complete" | "stopped" | "aborted";
+export type AutoOutcome = "complete" | "stopped" | "aborted" | "stuck";
 export interface AutoRunOptions {
     strict: boolean;
     llmSteerOverride?: boolean;

package/dist/extensions/codecarto/auto-runner.js

@@ -17,7 +17,7 @@ import { clearPhase, finishPhase, getPhaseActivity, startPhase } from "./agent-s
 import { buildSteeringMessage, rewritePhasePrompt } from "./agent-rewriter.js";
 import { buildPhaseSummary } from "./agent-summary.js";
 import { getAgentsWidget } from "./agent-widget.js";
-import { appendUsageRun, buildPhasePrompt, buildValidationSummary, completeValidatedPhase, formatMillis, formatTokenCount, getNextEligiblePhase, getWorkspaceState, describeConfigProblems, loadCodecartoConfig, PACKAGE_VERSION, PhasePreflightError, runPhasePreflight, validatePhaseOutput, writeDashboard, } from "../../core/index.js";
+import { appendUsageRun, buildPhasePrompt, buildValidationSummary, completeValidatedPhase, formatMillis, formatTokenCount, describeStuckPipeline, resolvePipelineOutcome, getWorkspaceState, describeConfigProblems, loadCodecartoConfig, PACKAGE_VERSION, PhasePreflightError, runPhasePreflight, validatePhaseOutput, writeDashboard, } from "../../core/index.js";
 /**
  * Run one phase end to end: optional LLM-steered rewrite, spawn the sub-agent,
  * wait for it, then emit the side effects the historical /codecarto-next chain
@@ -203,13 +203,22 @@ export async function runAuto(ctx, pi, initialState, options) {
                 reason: "User aborted the auto run.",
             });
         }
-        const phase = getNextEligiblePhase(state);
-        if (!phase) {
+        const outcome = resolvePipelineOutcome(state);
+        if (outcome.kind === "stuck") {
+            // Not "complete": the loop ending because nothing can run is the
+            // case that used to read as success (#228).
+            return finish({
+                outcome: "stuck",
+                reason: describeStuckPipeline(outcome.blocked),
+            });
+        }
+        if (outcome.kind === "complete") {
             return finish({
                 outcome: "complete",
                 reason: "Pipeline complete.",
             });
         }
+        const phase = outcome.phase;
         let preflight;
         try {
             preflight = await runPhasePreflight(state, phase);
@@ -308,6 +317,8 @@ export function buildAutoSummary(result, availableSkills = []) {
                 return `**Auto pipeline stopped at \`${result.stoppedAt?.phaseId ?? "?"}\`.**`;
             case "aborted":
                 return `**Auto pipeline aborted${result.stoppedAt?.phaseId ? ` during \`${result.stoppedAt.phaseId}\`` : ""}.**`;
+            case "stuck":
+                return `**Auto pipeline stuck: no phase can run.**`;
         }
     })();
     const statsLine = `_⟳ ${ranOf} · ${tokensStr} tokens · ${wallTime}_`;
@@ -321,6 +332,10 @@ export function buildAutoSummary(result, availableSkills = []) {
         lines.push("", result.reason);
         lines.push("", recoveryHint(result));
     }
+    if (result.outcome === "stuck") {
+        // The reason is the stuck sentence, which already says what to fix.
+        lines.push("", result.reason);
+    }
     if (result.outcome === "complete") {
         lines.push("", "Dashboard: `.codecarto/dashboard.html`");
         if (availableSkills.length > 0) {

package/dist/extensions/codecarto/broadside-flags.d.ts

@@ -16,11 +16,13 @@ export interface BroadsideFlags {
     /** Undefined means "use the repository's config default". */
     maxCost?: number;
     waitSeconds?: number;
+    /** For collect: the run to collect instead of the most recent (#268). */
+    runId?: string;
     benchmarks: boolean;
     unknown: string[];
     /** Set on an invalid combination. The caller surfaces it as an error. */
     error?: string;
 }
 /** Every token the completer offers, in the order it offers them. */
-export declare const KNOWN_BROADSIDE_TOKENS: readonly ["submit", "collect", "status", "models", "architecture", "api", "security", "defect", "conventions", "porting", "--incremental", "--no-incremental", "--max-cost=", "--wait=", "--no-synthesis", "--no-triage", "--no-retry-truncated", "--benchmarks"];
+export declare const KNOWN_BROADSIDE_TOKENS: readonly ["submit", "collect", "status", "models", "architecture", "api", "security", "defect", "conventions", "porting", "--incremental", "--no-incremental", "--max-cost=", "--wait=", "--run=", "--no-synthesis", "--no-triage", "--no-retry-truncated", "--benchmarks"];
 export declare function parseBroadsideFlags(args: string): BroadsideFlags;

package/dist/extensions/codecarto/broadside-flags.js

@@ -13,6 +13,7 @@
 //   --no-incremental       --no-triage
 //   --max-cost=N           --no-retry-truncated
 //   --wait=SECONDS         --benchmarks (models only)
+//   --run=ID (collect only: an older run, as listed by status)
 //
 // --incremental has a spelled-out negative because the value is tri-state:
 // absent defers to config.yaml, so a repository that set `incremental: true`
@@ -34,6 +35,7 @@ export const KNOWN_BROADSIDE_TOKENS = [
     "--no-incremental",
     "--max-cost=",
     "--wait=",
+    "--run=",
     "--no-synthesis",
     "--no-triage",
     "--no-retry-truncated",
@@ -106,6 +108,13 @@ export function parseBroadsideFlags(args) {
             result.waitSeconds = parseNumeric(token, "--wait", result);
             continue;
         }
+        if (token.startsWith("--run=")) {
+            const value = token.slice("--run=".length).trim();
+            if (!value)
+                result.error ??= "--run= needs a run id (see /codecarto-broadside status).";
+            result.runId = value || undefined;
+            continue;
+        }
         result.unknown.push(token);
     }
     // Flags that only mean something for one action are refused rather than
@@ -125,5 +134,8 @@ export function parseBroadsideFlags(args) {
     if (result.action === "status" && result.waitSeconds !== undefined) {
         result.error ??= "--wait is only meaningful for submit and collect; status reads recorded state.";
     }
+    if (result.runId !== undefined && result.action !== "collect") {
+        result.error ??= `--run is only meaningful for collect (got action "${result.action}").`;
+    }
     return result;
 }

package/dist/extensions/codecarto/index.js

@@ -1,5 +1,4 @@
 import { mkdir, readFile, rename, writeFile } from "node:fs/promises";
-import { homedir } from "node:os";
 import { basename, dirname, join, resolve } from "node:path";
 import { autoCompletePhase, buildAutoSummary, isPhaseRunning, runAuto, runSinglePhase } from "./auto-runner.js";
 import { disposeAgentsWidget } from "./agent-widget.js";
@@ -10,7 +9,7 @@ import { parseNextFlags } from "./next-flags.js";
 import { buildPiGuideMessage } from "./guide-framing.js";
 import { isCtxLive, notifyCtx } from "./notify.js";
 import { phaseCompactionExtension } from "./phase-compaction.js";
-import { applyAmendment, buildPhasePrompt, buildSkillPrompt, buildValidationSummary, backupWorkspaceState, canonicalPath, copyPackagedWorkspace, computePerPhaseTotals, computeTotals, ConfidentialityMismatchError, createEmptyStatus, DEFAULT_PIPELINE_PATH, describeScaffoldStaleness, deriveSlug, discoverLibrary, describeDanglingCarryForward, describeMissingCompletedOutputs, getNextEligiblePhase, getPipelineLabel, getWorkspaceState, isWithinPath, resolveExistingPrefix, BROADSIDE_LENS_IDS, BROADSIDE_SKILL_NAME, BroadsideCancelledError, broadsideDirFor, collectResultText, estimateSubmitText, getLens, listAmendmentNames, listBatchModels, listGuideTopics, listScaffoldRefreshFiles, listMissingCompletedOutputs, listSkillNames, resolveSkillName, loadAmendmentFile, loadBroadsideConfig, modelsText, runBroadsideCollect, runBroadsideStatus, runBroadsideSubmit, statusText, describeConfigProblems, loadCodecartoConfig, loadUsage, loadYamlFile, normalizeForComparison, packagedWorkspaceDir, pathExists, PACKAGE_VERSION, readBroadsideSkill, readGuide, refreshScaffold, PhasePreflightError, PIPELINE_ALIASES, publishEntry, resolvePhase, resolvePipelineChoice, resolvePublishSourceRepo, SourceRepoMismatchError, runPhasePreflight, SCAFFOLD_REFRESH_PROTECTED, seedOrchestratorFiles, stringifySimpleYaml, switchPipeline, validatePhaseOutput, writeLibraryConfig, writeDashboard, } from "../../core/index.js";
+import { applyAmendment, buildPhasePrompt, buildSkillPrompt, buildValidationSummary, backupWorkspaceState, canonicalPath, copyPackagedWorkspace, computePerPhaseTotals, computeTotals, ConfidentialityMismatchError, createEmptyStatus, DEFAULT_PIPELINE_PATH, describeScaffoldStaleness, deriveSlug, discoverLibrary, describeDanglingCarryForward, describeMissingCompletedOutputs, describeStuckPipeline, expandTilde, getPipelineLabel, getWorkspaceState, isWithinPath, resolveExistingPrefix, BROADSIDE_LENS_IDS, BROADSIDE_SKILL_NAME, BroadsideConfigError, defaultBroadsideConfig, BroadsideCancelledError, broadsideDirFor, collectResultText, estimateSubmitText, getLens, listAmendmentNames, listBatchModels, listGuideTopics, listScaffoldRefreshFiles, listMissingCompletedOutputs, listSkillNames, resolvePipelineOutcome, resolveSkillName, loadAmendmentFile, loadBroadsideConfig, modelsText, runBroadsideCollect, runBroadsideStatus, runBroadsideSubmit, statusText, describeConfigProblems, loadCodecartoConfig, loadUsage, loadYamlFile, normalizeForComparison, packagedWorkspaceDir, pathExists, PACKAGE_VERSION, readBroadsideSkill, readGuide, refreshScaffold, PhasePreflightError, PIPELINE_ALIASES, publishEntry, resolvePhase, resolvePipelineChoice, resolvePublishSourceRepo, SourceRepoMismatchError, runPhasePreflight, SCAFFOLD_REFRESH_PROTECTED, seedOrchestratorFiles, stringifySimpleYaml, switchPipeline, validatePhaseOutput, writeLibraryConfig, writeDashboard, } from "../../core/index.js";
 import { initLibrary } from "../../core/library.js";
 import { resolveUserConfigPath } from "../../core/orchestrator-config.js";
 const STATUS_WIDGET_ID = "codecarto-widget";
@@ -56,9 +55,23 @@ function formatUsageDuration(ms) {
     const seconds = Math.floor((ms % 60_000) / 1000);
     return `${minutes}m${seconds.toString().padStart(2, "0")}s`;
 }
+/**
+ * The phase the status line and session name show: the eligible phase, the
+ * first blocked phase when the pipeline is stuck, or "complete" — never
+ * "complete" for a pipeline that cannot finish (#228).
+ */
+function currentPhaseLabel(state) {
+    const outcome = resolvePipelineOutcome(state);
+    if (outcome.kind === "eligible")
+        return outcome.phase.id;
+    if (outcome.kind === "stuck")
+        return outcome.blocked[0]?.phaseId ?? "stuck";
+    return "complete";
+}
 function buildStatusLines(state, extraLines = []) {
-    const nextPhase = getNextEligiblePhase(state);
-    const currentPhase = nextPhase?.id ?? state.status.current_phase ?? "complete";
+    const outcome = resolvePipelineOutcome(state);
+    const nextPhase = outcome.kind === "eligible" ? outcome.phase : null;
+    const currentPhase = currentPhaseLabel(state);
     const pipelineLabel = getPipelineLabel(state.status.pipeline);
     const completedCount = state.pipeline.phase_order.filter((phaseId) => state.status.phases[phaseId]?.status === "complete").length;
     const terminalOpenQuestions = Object.values(state.status.phases).reduce((sum, phase) => sum + (phase.open_questions?.length ?? 0), 0);
@@ -68,7 +81,7 @@ function buildStatusLines(state, extraLines = []) {
     const lines = [
         "CodeCartographer",
         `Phase: ${currentPhase}`,
-        `Pipeline state: ${currentPhase === "complete" ? "complete" : "in progress"}`,
+        `Pipeline state: ${outcome.kind === "eligible" ? "in progress" : outcome.kind}`,
         `Pipeline: ${pipelineLabel}`,
         `Progress: ${completedCount}/${state.pipeline.phase_order.length} complete`,
         `Open questions (terminal unresolved): ${terminalOpenQuestions}`,
@@ -76,6 +89,8 @@ function buildStatusLines(state, extraLines = []) {
         `Post-pipeline work: ${postPipelinePending} pending`,
         `Next: ${nextAction}`,
     ];
+    if (outcome.kind === "stuck")
+        lines.push(describeStuckPipeline(outcome.blocked));
     const scaffoldNotice = describeScaffoldStaleness(state);
     if (scaffoldNotice)
         lines.push(`Scaffold: ${scaffoldNotice}`);
@@ -93,7 +108,7 @@ function setUiState(ctx, state, extraLines = []) {
         return;
     }
     const theme = ctx.ui.theme;
-    const currentPhase = getNextEligiblePhase(state)?.id ?? state.status.current_phase ?? "complete";
+    const currentPhase = currentPhaseLabel(state);
     ctx.ui.setStatus(STATUS_LINE_ID, `${theme.fg("accent", "CC")} ${theme.fg("dim", currentPhase)}`);
     ctx.ui.setWidget(STATUS_WIDGET_ID, buildStatusLines(state, extraLines));
 }
@@ -291,7 +306,7 @@ export default function codeCartographerExtension(pi) {
         const state = await readWorkspaceState(ctx, false);
         setUiState(ctx, state, extraLines ?? lastFeedbackLines);
         if (state) {
-            const phaseId = getNextEligiblePhase(state)?.id ?? state.status.current_phase;
+            const phaseId = currentPhaseLabel(state);
             if (phaseId)
                 pi.setSessionName(`CodeCartographer: ${phaseId}`);
         }
@@ -498,13 +513,17 @@ export default function codeCartographerExtension(pi) {
             const state = await ensureWorkspaceState(ctx);
             if (!state)
                 return;
-            const nextPhase = getNextEligiblePhase(state)?.id ?? "complete";
+            const outcome = resolvePipelineOutcome(state);
+            const nextPhase = currentPhaseLabel(state);
             // Same check codecarto_status makes: a phase complete in status.yaml
             // whose report is not on disk (#259).
             const missingOutputs = describeMissingCompletedOutputs(await listMissingCompletedOutputs(state));
             lastFeedbackLines = [`Current phase: ${nextPhase}`, `Pipeline: ${getPipelineLabel(state.status.pipeline)}`, ...missingOutputs];
             setUiState(ctx, state, lastFeedbackLines);
-            if (missingOutputs.length > 0) {
+            if (outcome.kind === "stuck") {
+                notifyCtx(ctx, `CodeCartographer phase: ${nextPhase}. ${describeStuckPipeline(outcome.blocked)}`, "error");
+            }
+            else if (missingOutputs.length > 0) {
                 notifyCtx(ctx, `CodeCartographer phase: ${nextPhase}. ${missingOutputs[0]}`, "warning");
             }
             else {
@@ -608,7 +627,7 @@ export default function codeCartographerExtension(pi) {
                         // the readout tracks progress live instead of staying frozen
                         // at the initial phase until the whole auto run finishes.
                         setUiState(ctx, advancedState, [`Auto pipeline${flags.strict ? " (strict)" : ""} running…`]);
-                        const phaseId = getNextEligiblePhase(advancedState)?.id ?? advancedState.status.current_phase;
+                        const phaseId = currentPhaseLabel(advancedState);
                         if (phaseId)
                             pi.setSessionName(`CodeCartographer: ${phaseId}`);
                     },
@@ -624,13 +643,21 @@ export default function codeCartographerExtension(pi) {
                 notifyCtx(ctx, `Auto pipeline ${result.outcome}: ${result.phasesRun.length}/${result.totalPhases} phases.`, result.outcome === "complete" ? "info" : "warning");
                 return;
             }
-            const phase = getNextEligiblePhase(state);
-            if (!phase) {
+            const outcome = resolvePipelineOutcome(state);
+            if (outcome.kind === "stuck") {
+                const message = describeStuckPipeline(outcome.blocked);
+                lastFeedbackLines = [message];
+                setUiState(ctx, state, lastFeedbackLines);
+                notifyCtx(ctx, message, "error");
+                return;
+            }
+            if (outcome.kind === "complete") {
                 lastFeedbackLines = ["All phases complete."];
                 setUiState(ctx, state, lastFeedbackLines);
                 notifyCtx(ctx, "All CodeCartographer phases are complete.", "info");
                 return;
             }
+            const phase = outcome.phase;
             let preflight;
             try {
                 preflight = await runPhasePreflight(state, phase);
@@ -868,9 +895,13 @@ export default function codeCartographerExtension(pi) {
             const state = await ensureWorkspaceState(ctx);
             if (!state)
                 return;
-            const nextPhase = getNextEligiblePhase(state);
-            if (nextPhase) {
-                notifyCtx(ctx, `Cannot run skill: pipeline is not complete (next phase: ${nextPhase.id}). Finish the pipeline before running post-pipeline skills.`, "error");
+            const outcome = resolvePipelineOutcome(state);
+            if (outcome.kind === "eligible") {
+                notifyCtx(ctx, `Cannot run skill: pipeline is not complete (next phase: ${outcome.phase.id}). Finish the pipeline before running post-pipeline skills.`, "error");
+                return;
+            }
+            if (outcome.kind === "stuck") {
+                notifyCtx(ctx, `Cannot run skill: the pipeline is not complete. ${describeStuckPipeline(outcome.blocked)}`, "error");
                 return;
             }
             // Resolve against the installed list only: the name is never joined
@@ -908,11 +939,13 @@ export default function codeCartographerExtension(pi) {
             const lines = skills.length > 0
                 ? [`Available skills (${skills.length}):`, ...skills.map((name) => `  - ${name}`)]
                 : ["No skills installed."];
-            const nextPhase = getNextEligiblePhase(state);
+            const outcome = resolvePipelineOutcome(state);
             if (skills.length > 0) {
-                lines.push(nextPhase
-                    ? `Post-pipeline skills unlock when the pipeline completes (next phase: ${nextPhase.id}).`
-                    : "Run one with /codecarto-skill <name>.");
+                lines.push(outcome.kind === "eligible"
+                    ? `Post-pipeline skills unlock when the pipeline completes (next phase: ${outcome.phase.id}).`
+                    : outcome.kind === "stuck"
+                        ? `Post-pipeline skills unlock when the pipeline completes, and it cannot: ${describeStuckPipeline(outcome.blocked)}`
+                        : "Run one with /codecarto-skill <name>.");
             }
             // Broad-Side is listed apart from the post-pipeline set because it
             // answers to /codecarto-skill without the completion gate.
@@ -988,7 +1021,23 @@ export default function codeCartographerExtension(pi) {
             // Broad-Side runs on any git repository, with or without a workspace —
             // so this command never goes through ensureWorkspaceState.
             const broadsideDir = broadsideDirFor(ctx.cwd);
-            const config = await loadBroadsideConfig(broadsideDir);
+            // A config.yaml that exists but cannot be read refuses every action
+            // that would act on it (#232); status only reads recorded runs, so it
+            // answers and says the file is unreadable.
+            let config;
+            let configWarning = null;
+            try {
+                config = await loadBroadsideConfig(broadsideDir);
+            }
+            catch (error) {
+                const message = error instanceof Error ? error.message : String(error);
+                if (!(error instanceof BroadsideConfigError) || flags.action !== "status") {
+                    notifyCtx(ctx, `Broad-Side ${flags.action} refused: ${message}`, "error");
+                    return;
+                }
+                config = defaultBroadsideConfig();
+                configWarning = message;
+            }
             const finish = (lines, notice, level = "info") => {
                 lastFeedbackLines = lines;
                 if (codecartoModeActive) {
@@ -1005,9 +1054,18 @@ export default function codeCartographerExtension(pi) {
                 notifyCtx(ctx, notice, level);
             };
             if (flags.action === "status") {
-                const { state } = await runBroadsideStatus(ctx.cwd);
+                let state;
+                try {
+                    ({ state } = await runBroadsideStatus(ctx.cwd));
+                }
+                catch (error) {
+                    // A corrupt state.json: nothing trustworthy to report, and the
+                    // error names the preserved copy (#233).
+                    notifyCtx(ctx, `Broad-Side status failed: ${error instanceof Error ? error.message : String(error)}`, "error");
+                    return;
+                }
                 const runs = state.runs.length;
-                finish(statusText(state).split("\n"), runs > 0 ? `Broad-Side: ${runs} recorded run${runs === 1 ? "" : "s"}` : "Broad-Side: no runs recorded yet");
+                finish([...statusText(state).split("\n"), ...(configWarning ? [`Warning: ${configWarning}`] : [])], runs > 0 ? `Broad-Side: ${runs} recorded run${runs === 1 ? "" : "s"}` : "Broad-Side: no runs recorded yet", configWarning ? "warning" : "info");
                 return;
             }
             const apiKey = resolveBroadsideKey(config.apiKey);
@@ -1045,8 +1103,10 @@ export default function codeCartographerExtension(pi) {
                 progress.set(lensId, `${status} (${counts.completed ?? 0}/${counts.total ?? "?"})`);
                 renderProgress("Polling batches…");
             };
+            // An explicit --wait=0 polls once and returns; it used to become
+            // undefined and, in core, the 25-minute budget (#230).
             const waitSeconds = flags.waitSeconds ?? config.waitSeconds;
-            const waitMs = waitSeconds > 0 ? waitSeconds * 1000 : undefined;
+            const waitMs = waitSeconds * 1000;
             const includeSynthesis = flags.includeSynthesis ?? config.includeSynthesis;
             const includeTriage = flags.includeTriage ?? config.includeTriage;
             const retryTruncated = flags.retryTruncated ?? config.retryTruncated;
@@ -1113,6 +1173,7 @@ export default function codeCartographerExtension(pi) {
                     includeTriage,
                     retryTruncated,
                     onStatus,
+                    ...(flags.runId && { runId: flags.runId }),
                 });
                 const lines = collectResultText(collect).split("\n");
                 const done = collect.status === "completed";
@@ -1255,12 +1316,11 @@ export default function codeCartographerExtension(pi) {
                 notifyCtx(ctx, "Usage: /codecarto-library-init <path> [--namespace <name>]", "warning");
                 return;
             }
-            const libraryPath = pathArg.startsWith("~") ? join(homedir(), pathArg.slice(1)) : resolve(pathArg);
+            // The same expansion the config loader applies, so `~user/x` and a
+            // bare `~` mean the same thing everywhere (self-audit mech 6.14).
+            const libraryPath = resolve(expandTilde(pathArg));
             try {
-                const result = await initLibrary(libraryPath, {
-                    namespaced: !!namespace,
-                    ...(namespace ? {} : {}),
-                });
+                const result = await initLibrary(libraryPath, { namespaced: !!namespace });
                 // Write library.path (and library.namespace when given) to the
                 // user-global config and nothing else — not publish_confirm, so
                 // initializing a library does not change how publish behaves on
@@ -1459,9 +1519,13 @@ export default function codeCartographerExtension(pi) {
                 notifyCtx(ctx, error instanceof Error ? error.message : String(error), "error");
                 return;
             }
-            const nextPhase = getNextEligiblePhase(state);
-            if (nextPhase) {
-                notifyCtx(ctx, `Cannot amend: the pipeline is not complete (next phase: ${nextPhase.id}). `
+            const outcome = resolvePipelineOutcome(state);
+            if (outcome.kind === "stuck") {
+                notifyCtx(ctx, `Cannot amend: the pipeline is not complete. ${describeStuckPipeline(outcome.blocked)}`, "error");
+                return;
+            }
+            if (outcome.kind === "eligible") {
+                notifyCtx(ctx, `Cannot amend: the pipeline is not complete (next phase: ${outcome.phase.id}). `
                     + "Resolve open questions and routed items through that phase's handoff (open_question_closures / carry_forward_closures) instead.", "error");
                 return;
             }

package/dist/extensions/codecarto/phase-compaction.js

@@ -7,7 +7,11 @@ export function phaseIdFromSessionName(sessionName) {
     if (!sessionName?.startsWith(PHASE_SESSION_PREFIX))
         return null;
     const phaseId = sessionName.slice(PHASE_SESSION_PREFIX.length).trim();
-    return /^[a-z0-9][a-z0-9-]*$/.test(phaseId) ? phaseId : null;
+    // The same alphabet assertSafePhaseId admits (core/status.ts). A narrower
+    // match here silently dropped the bash block, write confinement, and
+    // checkpointing for a custom pipeline whose ids used `.`, `_`, or a capital
+    // (self-audit mech 6.11).
+    return /^[A-Za-z0-9][A-Za-z0-9._-]*$/.test(phaseId) ? phaseId : null;
 }
 export function buildPhaseCompactionInstructions(phaseId, primaryOutput) {
     const output = primaryOutput ? `.codecarto/${primaryOutput}` : "the phase's declared primary output";

package/dist/mcp-server/server.d.ts

@@ -37,6 +37,7 @@ export declare function handleSwitchPipeline(args: {
 }>;
 export declare function handleNext(args: {
     cwd: string;
+    unattended?: boolean;
 }): Promise<{
     content: {
         type: "text";
@@ -49,6 +50,7 @@ export declare function handleNext(args: {
 export declare function handlePhase(args: {
     cwd: string;
     phase: string;
+    unattended?: boolean;
 }): Promise<{
     content: {
         type: "text";
@@ -130,7 +132,6 @@ export declare function handleLibraryInit(args: {
     library_path: string;
     name?: string;
     namespace?: string;
-    cwd?: string;
 }): Promise<{
     content: {
         type: "text";
@@ -236,6 +237,7 @@ export declare function handleBroadside(args: {
     lenses?: string[];
     api_key?: string;
     wait_seconds?: number;
+    run_id?: string;
     include_synthesis?: boolean;
     include_triage?: boolean;
     retry_truncated?: boolean;