@fitlab-ai/agent-infra 0.9.9 → 0.9.11

package/dist/lib/task/commands/log.js

@@ -3,15 +3,15 @@ import { classifyAgent } from '../../agent-clients/tokens.js';
 import { formatTable } from '../../table.js';
 import { parseTaskScope } from '../command-options.js';
 import { resolveTaskContext } from '../resolve-ref.js';
-import { isReviewStage, parseLedger } from '../ledger.js';
+import { LEDGER_SECTION_MISSING_CODE, LEDGER_SECTION_MISSING_MESSAGE, isReviewStage, parseLedgerDocument } from '../ledger.js';
 import { parseActivityLog, pairEntries } from '../activity-log.js';
-const USAGE = `Usage: ai task log [<N | TASK-id> | --task <ref> | -t <ref>]
+const USAGE = `Usage: ai task log [--task <ref> | -t <ref>]
 
 Renders a task's activity log as a per-step status table. A step's start and
 completion are paired onto one row: STARTED holds the start time, DONE the
 completion time (or '(in progress)' while still running).
-  Omit <ref>   Resolve the unique active task for the current branch.
-  <ref>   Bare numeric short id, or a full TASK-YYYYMMDD-HHMMSS id.
+  Omit the scope   Resolve the unique active task for the current branch.
+  --task/-t <ref> Bare numeric short id, or a full TASK-YYYYMMDD-HHMMSS id.
 
 Columns: # (row) / STEP / AGENT / STARTED / DONE / NOTE
   A human-executed step shows AGENT as 'human' and, when it has no start marker,
@@ -85,8 +85,8 @@ function isHumanCountField(field) {
     return fieldNumber(field, 'manual-validation') !== undefined || fieldNumber(field, 'human-decision') !== undefined;
 }
 // A step is human-executed when its agent token is not a known AI token.
-// Known long names (claude-code -> claude, gemini-cli -> gemini) classify as
-// AI; empty or unknown tokens classify as non-AI (human).
+// Known long names classify as AI; empty or unknown tokens classify as non-AI
+// (human).
 export function isHumanAgent(agent) {
     return classifyAgent(agent).status !== 'ai';
 }
@@ -118,12 +118,12 @@ function log(args = []) {
         process.exitCode = 1;
         return;
     }
-    if (scope.positionals.length > 1 || (scope.explicit && scope.positionals.length > 0)) {
-        process.stderr.write('ai task log: task ref must be provided once\n');
+    if (scope.positionals.length > 0) {
+        process.stderr.write('ai task log: positional task ref is not supported; use --task <ref> or -t <ref>\n');
         process.exitCode = 1;
         return;
     }
-    const resolved = resolveTaskContext(scope.taskRef ?? scope.positionals[0]);
+    const resolved = resolveTaskContext(scope.taskRef);
     if (!resolved.ok) {
         process.stderr.write(`ai task log: ${resolved.message}\n`);
         process.exitCode = 1;
@@ -142,7 +142,18 @@ function log(args = []) {
         return;
     }
     const steps = pairEntries(entries);
-    const humanDecisionCounts = countHumanDecisionsByStage(parseLedger(content));
+    let humanDecisionCounts = new Map();
+    let ledgerDiagnostic = null;
+    try {
+        const ledger = parseLedgerDocument(content);
+        if (ledger.present)
+            humanDecisionCounts = countHumanDecisionsByStage(ledger.rows);
+        else
+            ledgerDiagnostic = `${LEDGER_SECTION_MISSING_CODE}: ${LEDGER_SECTION_MISSING_MESSAGE}`;
+    }
+    catch (error) {
+        ledgerDiagnostic = `LEDGER_DOCUMENT_INVALID: ${error instanceof Error ? error.message : String(error)}`;
+    }
     const rows = steps.map((s, idx) => {
         const stage = reviewStageForStep(s.step);
         const note = stage
@@ -157,6 +168,10 @@ function log(args = []) {
     for (const line of formatTable(TABLE_HEADERS, rows, { zebra: Boolean(process.stdout.isTTY) })) {
         process.stdout.write(`${line}\n`);
     }
+    if (ledgerDiagnostic) {
+        process.stdout.write(`Ledger: unavailable [${ledgerDiagnostic}]\n`);
+        process.exitCode = 1;
+    }
     process.stdout.write(`Total: ${steps.length} steps\n`);
 }
 export { log };

package/dist/lib/task/commands/show.js

@@ -1,12 +1,11 @@
 import fs from 'node:fs';
 import { parseTaskScope } from '../command-options.js';
 import { resolveTaskContext } from '../resolve-ref.js';
-const USAGE = `Usage: ai task show [<N | TASK-id> | --task <ref> | -t <ref>]
+const USAGE = `Usage: ai task show [--task <ref> | -t <ref>]
 
 Prints the task.md content for the matching task.
   Omit the ref       Resolve the unique active task for the current branch.
-  N (bare numeric)   Recommended; resolves the active short id via the registry.
-  TASK-YYYYMMDD-HHMMSS  Locates a task in active / blocked / completed / archive.
+  --task/-t <ref>    Bare numeric short id, or a full TASK-YYYYMMDD-HHMMSS id.
 `;
 function show(args = []) {
     if (args[0] === '--help' || args[0] === '-h') {
@@ -22,12 +21,12 @@ function show(args = []) {
         process.exitCode = 1;
         return;
     }
-    if (scope.positionals.length > 1 || (scope.explicit && scope.positionals.length > 0)) {
-        process.stderr.write('ai task show: task ref must be provided once\n');
+    if (scope.positionals.length > 0) {
+        process.stderr.write('ai task show: positional task ref is not supported; use --task <ref> or -t <ref>\n');
         process.exitCode = 1;
         return;
     }
-    const resolved = resolveTaskContext(scope.taskRef ?? scope.positionals[0]);
+    const resolved = resolveTaskContext(scope.taskRef);
     if (!resolved.ok) {
         process.stderr.write(`ai task show: ${resolved.message}\n`);
         process.exitCode = 1;

package/dist/lib/task/commands/status.js

@@ -8,15 +8,15 @@ import { enumerateArtifacts } from '../artifacts.js';
 import { parseTaskFrontmatter, extractTitle } from '../frontmatter.js';
 import { loadShortIdByTaskId } from '../short-id.js';
 import { getOpenWorkflowWarnings, formatWorkflowWarningSummary } from '../workflow-warnings.js';
-import { parseActivityLog, pairEntries } from '../activity-log.js';
+import { parseActivityLog, pairEntries, startedBackedRows } from '../activity-log.js';
 import { readRun } from '../orchestration.js';
 import { statusCard } from '../../server/display.js';
-const USAGE = `Usage: ai task status [<N | TASK-id> | --task <ref> | -t <ref>]
+const USAGE = `Usage: ai task status [--task <ref> | -t <ref>]
 
 Prints an aggregated "health check" view for a task: header, metadata,
 artifacts, workflow/runtime execution state, and git branch state.
-  Omit <ref>   Resolve the unique active task for the current branch.
-  <ref>   Bare numeric short id, or a full TASK-YYYYMMDD-HHMMSS id.
+  Omit the scope   Resolve the unique active task for the current branch.
+  --task/-t <ref> Bare numeric short id, or a full TASK-YYYYMMDD-HHMMSS id.
 
 Git rows are best-effort: a failed git call degrades that row to '-' without
 failing the command.
@@ -155,7 +155,7 @@ function collectWorkflow(content, now = new Date()) {
     if (!parsed.sectionFound || parsed.entries.length === 0) {
         return { state: 'unknown', step: DASH, agent: DASH, startedAt: DASH, doneAt: DASH, stale: DASH };
     }
-    const rows = pairEntries(parsed.entries);
+    const rows = startedBackedRows(pairEntries(parsed.entries));
     const latest = rows.at(-1);
     if (!latest) {
         return { state: 'unknown', step: DASH, agent: DASH, startedAt: DASH, doneAt: DASH, stale: DASH };
@@ -431,10 +431,10 @@ function status(args = []) {
     let model;
     try {
         const scope = parseTaskScope(args);
-        if (scope.positionals.length > 1 || (scope.explicit && scope.positionals.length > 0)) {
-            throw new Error('task ref must be provided once');
+        if (scope.positionals.length > 0) {
+            throw new Error('positional task ref is not supported; use --task <ref> or -t <ref>');
         }
-        const resolved = resolveTaskContext(scope.taskRef ?? scope.positionals[0]);
+        const resolved = resolveTaskContext(scope.taskRef);
         if (!resolved.ok)
             throw new Error(resolved.message);
         model = buildFromResolved({

package/dist/lib/task/commit-operation.js

@@ -82,12 +82,12 @@ function boundTask(input, taskId, taskDir, taskMdPath, repoRoot, state) {
 function validateCommon(input, branch) {
     if (!input.message.trim())
         return { code: 'GIT_COMMIT_INPUT_INVALID', message: 'Commit message is required' };
-    if (input.paths.length === 0 && !input.push)
-        return { code: 'GIT_COMMIT_INPUT_INVALID', message: 'At least one explicit path is required unless this is a push-only retry' };
     if (!/^[a-f0-9]{40,64}$/.test(input.expectedHead))
         return { code: 'GIT_HEAD_EXPECTATION_REQUIRED', message: 'A valid expected HEAD is required' };
     if (!/^[a-f0-9]{40,64}$/.test(input.expectedTree))
         return { code: 'GIT_TREE_EXPECTATION_REQUIRED', message: 'A valid expected tree is required' };
+    if (!input.push)
+        return { code: 'GIT_PUSH_INPUT_INVALID', message: 'Commit delivery requires a push policy' };
     if (input.paths.some((candidate) => !candidate || candidate.startsWith('/') || candidate.split(/[\\/]/).includes('..'))) {
         return { code: 'GIT_COMMIT_INPUT_INVALID', message: 'Commit paths must stay inside the repository' };
     }
@@ -96,13 +96,11 @@ function validateCommon(input, branch) {
     }
     if (branch !== null && branch === '')
         return { code: 'GIT_BRANCH_INVALID', message: 'A named branch is required' };
-    if (input.push) {
-        if (!input.push.remote || input.push.refs.length !== 1)
-            return { code: 'GIT_PUSH_INPUT_INVALID', message: 'Push requires one remote and one ref' };
-        const ref = input.push.refs[0] ?? '';
-        if (!HEAD_REF.test(ref) || HEAD_REF.exec(ref)?.[1] !== branch) {
-            return { code: 'GIT_PUSH_INPUT_INVALID', message: 'Push ref must be the current full heads ref' };
-        }
+    if (!input.push.remote || input.push.refs.length !== 1)
+        return { code: 'GIT_PUSH_INPUT_INVALID', message: 'Push requires one remote and one ref' };
+    const ref = input.push.refs[0] ?? '';
+    if (!HEAD_REF.test(ref) || HEAD_REF.exec(ref)?.[1] !== branch) {
+        return { code: 'GIT_PUSH_INPUT_INVALID', message: 'Push ref must be the current full heads ref' };
     }
     return null;
 }
@@ -253,22 +251,20 @@ function executeUnlocked(input, task, mode) {
     }
     let status = committed.status;
     let result = committed.status === 'applied' ? 'committed' : 'no_op';
-    if (input.push) {
-        const target = `${input.push.remote}:${input.push.refs[0]}`;
-        const decision = commitPushDecision({ branch, automatic: input.push.automatic }, target);
-        if (!decision.shouldPush) {
-            warnings.push(decision.warning);
-        }
-        else {
-            const pushed = pushGitRefs({
-                cwd: input.cwd,
-                remote: input.push.remote,
-                refs: input.push.refs,
-                expectedSha: committed.snapshot?.head ?? gitText(input.cwd, ['rev-parse', 'HEAD'])
-            });
-            if (pushed.status !== 'applied')
-                warnings.push(warning('COMMIT_PUSH_FAILED', pushed.error?.message ?? 'Git push failed', true, 'push', target, 'ACTION_REQUIRED'));
-        }
+    const target = `${input.push.remote}:${input.push.refs[0]}`;
+    const decision = commitPushDecision({ branch }, target);
+    if (!decision.shouldPush) {
+        warnings.push(decision.warning);
+    }
+    else {
+        const pushed = pushGitRefs({
+            cwd: input.cwd,
+            remote: input.push.remote,
+            refs: input.push.refs,
+            expectedSha: committed.snapshot?.head ?? gitText(input.cwd, ['rev-parse', 'HEAD'])
+        });
+        if (pushed.status !== 'applied')
+            warnings.push(warning('COMMIT_PUSH_FAILED', pushed.error?.message ?? 'Git push failed', true, 'push', target, 'ACTION_REQUIRED'));
     }
     if (warnings.length > 0) {
         status = 'applied';

package/dist/lib/task/commit-policy.js

@@ -1,6 +1,6 @@
 function commitPushDecision(policy, target) {
     const branch = policy.branch.trim();
-    if (policy.automatic && (branch === 'main' || branch === 'master'))
+    if (branch === 'main' || branch === 'master')
         return {
             shouldPush: false,
             warning: {