ai-foreman 1.7.11 → 1.7.13

package/dist/cli/status.js

@@ -3,7 +3,8 @@ import { resolve, join } from "node:path";
 import { existsSync, readdirSync, readFileSync } from "node:fs";
 import { findResumableBranchSessions, formatBranchContinueCommand } from "../branch/resume.js";
 import { recoverableBuildRuns } from "../buildRuns.js";
-import { WorkflowDb, WORKFLOW_DB_FILE } from "../workflowDb.js";
+import { WORKFLOW_DB_FILE } from "../workflowDb.js";
+import { WorkflowReader } from "../workflowReader.js";
 import { formatWorkflowIssue } from "../outcomes.js";
 function fail(message) {
     console.error(`foreman: ${message}`);
@@ -21,7 +22,7 @@ export function runStatus(project) {
     const dir = join(projectDir, ".foreman");
     const recoverable = recoverableBuildRuns(projectDir);
     const workflowPath = join(projectDir, WORKFLOW_DB_FILE);
-    const workflow = existsSync(workflowPath) ? new WorkflowDb(projectDir, workflowPath) : undefined;
+    const workflow = existsSync(workflowPath) ? new WorkflowReader(projectDir, workflowPath) : undefined;
     const activeWorkflows = workflow?.activeRuns() ?? [];
     if (!existsSync(dir) && activeWorkflows.length === 0) {
         workflow?.close();
@@ -101,6 +102,16 @@ function printWorkflowIssues(workflow, runs) {
         console.log(`  ${run.runId} ${run.kind} ${run.status} checkpoint=${run.checkpoint}`);
         for (const value of workflow.issues(run.runId))
             console.log(`    ${formatWorkflowIssue(value).replace(/\n/g, " | ")}`);
+        if (workflow instanceof WorkflowReader) {
+            const supervisor = workflow.supervisorState(run.runId);
+            if (supervisor)
+                console.log(`    supervisor=${supervisor.status} generation=${supervisor.generation} worker=${supervisor.workerPid ?? "none"} restarts=${supervisor.runRestarts}`);
+            const attempts = workflow.recoveryAttempts(run.runId);
+            if (attempts.length)
+                console.log(`    recovery attempts=${attempts.length}; latest=${attempts.at(-1)?.action}:${attempts.at(-1)?.outcome}`);
+            for (const decision of workflow.pendingHumanDecisions(run.runId))
+                console.log(`    pending decision ${decision.decisionId}: ${decision.choices.map((choice) => choice.id).join(" | ")}`);
+        }
     }
 }
 function printRecoverable(projectDir, runs) {

package/dist/cli/tickets.js

@@ -20,7 +20,8 @@ import { checkpointInterview, completeInterview, createInterviewRecord, failInte
 import { applyTicketPopulation, authorizeTicketRetirements, extractTicketPopulationProposal, materializeTicketPopulation, recoverTicketPublications, TICKET_POPULATION_PROPOSAL_END, TICKET_POPULATION_PROPOSAL_START } from "../ticketPopulation.js";
 import { loadTickets } from "../tickets/ticketLoader.js";
 import { resolveTicketPaths } from "../tickets/config.js";
-import { formatTicketDetails, getTicketDetails } from "../tickets/details.js";
+import { formatAllTicketDetails, formatTicketDetails, getAllTicketDetails, getTicketDetails } from "../tickets/details.js";
+import { appendTicketOutput } from "../tickets/output.js";
 import { applyResolvedTicketReset, previewTicketReset, recoverTicketResetPublications, resetTickets, resolveTicketResetSelection, TicketResetDependencyConflictError } from "../tickets/reset.js";
 import { validateBranchPrefix } from "../branch/prefix.js";
 import { listTicketGroups, previewTicketGroupRepair, repairTicketGroups } from "../tickets/groups.js";
@@ -1026,7 +1027,7 @@ export async function cmdPopulateCli(opts) {
         }));
         builder = roleBuilder.builder;
         viewer = printEvents(builder.events());
-        const foreman = new Foreman(builder, log, config.notifications.enabled, false, 3, dir);
+        const foreman = new Foreman(builder, log, { desktop: config.notifications.enabled, terminalBell: config.notifications.terminal_bell }, false, 3, dir);
         console.log(`foreman tickets: populating tickets with ${roleBuilder.runtime}`);
         console.log(`foreman tickets: project ${dir}`);
         console.log(`foreman tickets: role ${TICKET_POPULATE_ROLE}`);
@@ -1375,18 +1376,35 @@ export function buildTicketsCommand(options = {}) {
     });
     // ── update ──────────────────────────────────────────────────────────────────
     tickets
-        .command("show <ticketId>")
-        .description("Show the complete canonical ticket, active state, validation, delivery, and history.")
+        .command("show [ticketId]")
+        .description("Show one or all complete canonical tickets, including state, validation, delivery, and history.")
         .option("-p, --project <dir>", "project directory (default: cwd)")
+        .option("--all", "show every canonical ticket, including terminal tickets")
         .option("--json", "write the complete stable record as JSON")
+        .option("--output <file>", "append the rendered output to a file")
         .action((ticketId, opts) => {
         try {
-            const details = getTicketDetails(cwd(opts), ticketId);
-            if (opts.json)
-                process.stdout.write(`${JSON.stringify(details, null, 2)}\n`);
-            else
-                for (const line of formatTicketDetails(details))
+            if (ticketId && opts.all)
+                throw new Error("choose either a ticket ID or --all, not both");
+            if (!ticketId && !opts.all) {
+                throw new Error("select one ticket: `rafi tickets show T123` or `rafi tickets show --all`");
+            }
+            const projectDir = cwd(opts);
+            const details = opts.all ? getAllTicketDetails(projectDir) : getTicketDetails(projectDir, ticketId);
+            const lines = opts.all ? formatAllTicketDetails(details) : formatTicketDetails(details);
+            const payload = opts.json ? `${JSON.stringify(details, null, 2)}\n` : `${lines.join("\n")}\n`;
+            if (opts.output) {
+                const result = appendTicketOutput(projectDir, resolve(String(opts.output)), payload);
+                const count = opts.all ? details.ticket_count : 1;
+                console.log(`rafi tickets show: ${result.disposition} ${count} ${count === 1 ? "ticket" : "tickets"} as ${opts.json ? "json" : "text"} to ${result.path}`);
+            }
+            else if (opts.json) {
+                process.stdout.write(payload);
+            }
+            else {
+                for (const line of lines)
                     console.log(line);
+            }
         }
         catch (err) {
             fail(String(err instanceof Error ? err.message : err));

package/dist/config.d.ts

@@ -1,3 +1,4 @@
+import { type ObservabilityConfig } from "./observability.js";
 export interface PermissionConfig {
     /** Bash command prefixes that are auto-approved when every segment matches. */
     allowBash: string[];
@@ -13,6 +14,8 @@ export interface PermissionConfig {
 export interface NotificationsConfig {
     /** Fire a desktop notification when the builder needs user input. Default: false. */
     enabled: boolean;
+    /** Ring the terminal bell when user attention is required or a run finishes. Default: true. */
+    terminal_bell: boolean;
 }
 export interface QaConfig {
     /** Run a QA review pass after each completed ticket or step. Default: true. */
@@ -22,6 +25,7 @@ export interface ForemanConfig {
     permissions: PermissionConfig;
     notifications: NotificationsConfig;
     qa: QaConfig;
+    observability: ObservabilityConfig;
 }
 /** Built-in defaults; foreman.yaml overrides any field present. */
 export declare const DEFAULT_CONFIG: ForemanConfig;

package/dist/config.js

@@ -1,5 +1,6 @@
 import { readFileSync, existsSync } from "node:fs";
 import { parse } from "yaml";
+import { DEFAULT_OBSERVABILITY_CONFIG } from "./observability.js";
 /** Built-in defaults; foreman.yaml overrides any field present. */
 export const DEFAULT_CONFIG = {
     permissions: {
@@ -65,8 +66,9 @@ export const DEFAULT_CONFIG = {
         ],
         escalateTools: ["WebFetch", "WebSearch"],
     },
-    notifications: { enabled: false },
+    notifications: { enabled: false, terminal_bell: true },
     qa: { enabled: true },
+    observability: DEFAULT_OBSERVABILITY_CONFIG,
 };
 /** Load foreman.yaml if present and deep-merge it over the defaults. */
 export function loadConfig(path = "foreman.yaml") {
@@ -81,6 +83,7 @@ export function loadConfig(path = "foreman.yaml") {
             ...(raw.notifications ?? {}),
         },
         qa: { ...DEFAULT_CONFIG.qa, ...(raw.qa ?? {}) },
+        observability: { ...DEFAULT_CONFIG.observability, ...(raw.observability ?? {}) },
     };
 }
 function validateConfig(raw, path) {
@@ -104,10 +107,34 @@ function validateConfig(raw, path) {
     }
     if (cfg.notifications !== undefined) {
         validateBooleanObject(cfg.notifications, "notifications", path);
+        const terminalBell = cfg.notifications.terminal_bell;
+        if (terminalBell !== undefined && typeof terminalBell !== "boolean") {
+            throw new Error(`${path}: notifications.terminal_bell must be a boolean`);
+        }
     }
     if (cfg.qa !== undefined) {
         validateBooleanObject(cfg.qa, "qa", path);
     }
+    if (cfg.observability !== undefined)
+        validateObservability(cfg.observability, path);
+}
+function validateObservability(value, path) {
+    if (!value || typeof value !== "object" || Array.isArray(value))
+        throw new Error(`${path}: observability must be an object`);
+    const input = value;
+    const keys = ["enabled", "sample_interval_seconds", "detail_retention_days", "log_retention_days", "detail_soft_limit_mb", "detail_hard_limit_mb", "log_limit_mb"];
+    for (const key of Object.keys(input))
+        if (!keys.includes(key))
+            throw new Error(`${path}: unknown observability setting ${key}`);
+    if (input.enabled !== undefined && typeof input.enabled !== "boolean")
+        throw new Error(`${path}: observability.enabled must be a boolean`);
+    for (const key of keys.filter((key) => key !== "enabled")) {
+        if (input[key] !== undefined && (!Number.isInteger(input[key]) || Number(input[key]) <= 0))
+            throw new Error(`${path}: observability.${key} must be a positive integer`);
+    }
+    const merged = { ...DEFAULT_OBSERVABILITY_CONFIG, ...input };
+    if (merged.detail_soft_limit_mb >= merged.detail_hard_limit_mb)
+        throw new Error(`${path}: observability.detail_soft_limit_mb must be less than detail_hard_limit_mb`);
 }
 function validateBooleanObject(value, name, path) {
     if (!value || typeof value !== "object" || Array.isArray(value)) {

package/dist/continuity.js

@@ -138,7 +138,8 @@ export class ContinuityAdapter {
         finally {
             db.close();
         }
-        const original = await this.adapter.sendTurn(`${instruction}\n\n${continuityInstruction()}`);
+        const providerInstruction = `${instruction}\n\n${continuityInstruction()}`;
+        const original = await this.adapter.sendTurn(providerInstruction);
         const activeSession = this.adapter.sessionRef?.() ?? this.adapter.sessionId();
         if (activeSession) {
             const leaseDb = new WorkflowDb(this.options.projectDir);
@@ -165,7 +166,8 @@ export class ContinuityAdapter {
                     instruction,
                 ].join("\n\n"));
             }
-            return { ...original, text: parsed.cleanText };
+            return { ...original, text: parsed.cleanText, hostInstruction: instruction, providerInstruction,
+                rawResponse: original.rawResponse ?? original.text, cleanedResponse: parsed.cleanText };
         }
         const repair = await this.adapter.sendTurn([
             "Continuity protocol repair only. Do not run tools, repeat work, or change files.",
@@ -177,7 +179,8 @@ export class ContinuityAdapter {
         if (repaired.delta) {
             this.publish(repaired.delta, "turn_completed_after_repair", original);
             this.moveRecoveryLeaseAfterCheckpoint();
-            return { ...original, text: parsed.cleanText };
+            return { ...original, text: parsed.cleanText, hostInstruction: instruction, providerInstruction,
+                rawResponse: original.rawResponse ?? original.text, cleanedResponse: parsed.cleanText };
         }
         const invalidDb = new WorkflowDb(this.options.projectDir);
         try {
@@ -205,7 +208,8 @@ export class ContinuityAdapter {
                 throw new ContinuityRecoveryRequiredError(this.options.runId, this.options.role, "validated recovery callback did not return a genuinely fresh successor");
             }
             await this.adoptValidatedSuccessor(successor);
-            return { ...original, text: parsed.cleanText };
+            return { ...original, text: parsed.cleanText, hostInstruction: instruction, providerInstruction,
+                rawResponse: original.rawResponse ?? original.text, cleanedResponse: parsed.cleanText };
         }
         if (!this.options.createSuccessor)
             throw new ContinuityRecoveryRequiredError(this.options.runId, this.options.role, "continuity repair failed in the original session and no validated successor is configured");
@@ -225,7 +229,8 @@ export class ContinuityAdapter {
         finally {
             leaseDb.close();
         }
-        return { ...original, text: parsed.cleanText };
+        return { ...original, text: parsed.cleanText, hostInstruction: instruction, providerInstruction,
+            rawResponse: original.rawResponse ?? original.text, cleanedResponse: parsed.cleanText };
     }
     sessionId() { return this.adapter.sessionId(); }
     sessionRef() { return this.adapter.sessionRef?.(); }
@@ -327,12 +332,12 @@ export class ContinuityAdapter {
             try {
                 for await (const event of source.events()) {
                     this.queue.push(event);
-                    if (event.kind === "tool" || event.kind === "session-transition" || event.kind === "context-usage") {
+                    // Continuity stores only recovery-relevant transitions. High-rate tool
+                    // and context telemetry belongs exclusively to observability.
+                    if (event.kind === "session-transition") {
                         const db = new WorkflowDb(this.options.projectDir);
                         try {
-                            const payload = event.kind === "tool"
-                                ? { name: event.name, inputDigest: sha(JSON.stringify(event.input ?? null)) }
-                                : event;
+                            const payload = event;
                             db.appendContinuityEvent({ runId: this.options.runId, role: "host", kind: `provider_${event.kind.replace(/-/g, "_")}`, payload, authoritativeStateRevision: this.revision() });
                         }
                         finally {

package/dist/diagnostics.d.ts

@@ -0,0 +1,25 @@
+import type { BuildRunRecordV2, ManagerDiagnosticReportV1 } from "rafi-spec";
+export type ExternalDiagnosticMode = "auto" | "on" | "off";
+export interface DiagnosticCommandResult {
+    ok: boolean;
+    stdout: string;
+    stderr?: string;
+    timedOut?: boolean;
+}
+export type DiagnosticCommandRunner = (command: string, args: readonly string[], options: {
+    cwd: string;
+    timeoutMs: number;
+}) => DiagnosticCommandResult;
+export interface CollectManagerDiagnosticsOptions {
+    runId?: string;
+    question?: string;
+    external?: ExternalDiagnosticMode;
+    now?: Date;
+    commandRunner?: DiagnosticCommandRunner;
+    maxDetailSpans?: number;
+}
+export declare function selectDiagnosticRun(projectDir: string, requestedRunId?: string): BuildRunRecordV2;
+/** Collects and calculates a canonical report. The provider receives this report, never raw database rows. */
+export declare function collectManagerDiagnostics(projectDir: string, options?: CollectManagerDiagnosticsOptions): ManagerDiagnosticReportV1;
+export declare function unionDuration(intervals: ReadonlyArray<readonly [number, number]>): number;
+export declare function ticketSizeBucketForRun(projectDir: string, ticketIds: readonly string[]): string | undefined;