@formigio/fazemos-cli 0.10.12 → 0.10.13

package/dist/index.js

@@ -2469,14 +2469,58 @@ commitments
         process.exit(1);
     }
 });
+// [F24 §3.6 / R16-R20] cm reopen — Restore an open state on a closed
+// commitment. The close event stays in the audit trail (a new audit_log
+// row records the reopen with from_state and to_state='open'). Agents
+// cannot reopen — the API's permission predicate naturally excludes
+// service principals (requireMemberId throws when req.member.id is null).
+commitments
+    .command('reopen <id>')
+    .description('Restore an open state on a closed commitment. The close event stays in the audit trail.')
+    .option('--reason <text>', 'Optional human-readable note for the audit log (max 500 chars)')
+    .option('--json', 'Print the raw API response as JSON (machine-readable)')
+    .action(async (id, opts) => {
+    try {
+        const body = {};
+        if (opts.reason)
+            body.reason = opts.reason;
+        const data = await api('PATCH', `/api/commitments/${id}/reopen`, body);
+        if (opts.json) {
+            console.log(JSON.stringify(data, null, 2));
+            return;
+        }
+        if (data.noop) {
+            console.log(chalk.yellow('This commitment is already open. Nothing to reopen.'));
+            return;
+        }
+        const c = data.commitment;
+        const prev = data.previousCompletedAt;
+        const prevFmt = prev ? new Date(prev).toISOString().slice(0, 16).replace('T', ' ') : null;
+        const tail = prevFmt ? ` The close on ${prevFmt} stays in the audit trail.` : '';
+        console.log(chalk.green(`Reopened. Commitment ${c.id} is open again.${tail}`));
+    }
+    catch (err) {
+        console.error(chalk.red(err.message));
+        process.exit(1);
+    }
+});
+// [F24 §3.4 / R13-R14] cm execute — gained `--repos` option for
+// commitment-source clone overrides (mirrors the top-level execute flag).
+// The server-side endpoint now reads agent_config.repos as a fallback
+// when no body.repos is supplied; this CLI flag is the operator-side
+// override for `fazemos cm execute -w <ws> -c <cm> --repos <repos>`.
 commitments
     .command('execute')
     .description('Trigger an agent execution for a commitment. The API determines which agent to use based on the commitment\'s configuration. For more control, use the top-level "execute" command instead.')
     .requiredOption('-w, --worksheet <id>', 'Worksheet ID')
     .requiredOption('-c, --commitment <id>', 'Commitment ID (from "cm list" or "ws show" output)')
+    .option('--repos <repos>', 'Comma-separated repo names to clone for this execution (overrides the agent\'s default agent_config.repos)', (v) => v.split(','))
     .action(async (opts) => {
     try {
-        const data = await api('POST', `/api/worksheets/${opts.worksheet}/commitments/${opts.commitment}/execute`);
+        const body = {};
+        if (opts.repos)
+            body.repos = opts.repos;
+        const data = await api('POST', `/api/worksheets/${opts.worksheet}/commitments/${opts.commitment}/execute`, body);
         console.log(chalk.green('Execution triggered'));
         if (data.execution?.id)
             console.log(`  Execution ID: ${data.execution.id}`);
@@ -4721,7 +4765,8 @@ program
     .argument('<source-id>', 'Action, commitment, or pipeline step ID')
     .requiredOption('--agent <name>', 'Agent name or member ID')
     .option('--source-type <type>', 'Source type (action, commitment, pipeline_step)', 'action')
-    .option('--prompt <prompt>', 'Additional prompt context')
+    .option('--prompt <prompt>', 'Operator instructions for this execution. Appended to the agent\'s built prompt as a labeled "## Operator Instructions" section. Takes precedence over runtime auto-completion instructions per the precedence rule (see architecture doc).')
+    .option('--no-auto-complete', 'Suppress the runtime auto-completion instruction. The agent will not be told to mark the source done. Use this for "draft, don\'t close" workflows where a human reviews and locks. Default: off. Pipeline-step source: ignored with a warning.')
     .option('--repos <repos>', 'Comma-separated repo names to clone (overrides agent config)', (v) => v.split(','))
     .option('--model <model>', 'Model override (e.g., opus, sonnet)')
     .option('--budget <usd>', 'Max budget override in USD', parseNumber)
@@ -4732,6 +4777,22 @@ program
             console.error(chalk.red(`Invalid source type. Must be one of: ${validTypes.join(', ')}`));
             process.exit(1);
         }
+        // [F24 §3.3 / EC3 / rul_no_auto_complete_pipeline_step_ignored]
+        // --no-auto-complete is ignored for pipeline_step sources to preserve
+        // back-compat for shipped pipeline executions. Layer-1 defense:
+        // CLI prints a warning and does NOT forward the flag onto
+        // context.noAutoComplete. Layer-2 defense lives in the agent's
+        // buildSelfReportingInstructions, which gates the noAutoComplete
+        // check on source_type !== 'pipeline_step' regardless.
+        //
+        // commander.js: `--no-auto-complete` produces opts.autoComplete=false
+        // (NOT opts.noAutoComplete=true) because of the `--no-` prefix
+        // convention. We surface that as the boolean `forwardSuppress` below.
+        const operatorOptedOut = opts.autoComplete === false;
+        if (operatorOptedOut && opts.sourceType === 'pipeline_step') {
+            console.error(chalk.yellow('Warning: --no-auto-complete is ignored for pipeline_step sources; pipeline steps always auto-complete.'));
+        }
+        const forwardSuppress = operatorOptedOut && opts.sourceType !== 'pipeline_step';
         // Resolve agent name to member ID if not already a UUID
         let agentMemberId = opts.agent;
         if (!/^[0-9a-f]{8}-[0-9a-f]{4}-/.test(agentMemberId)) {
@@ -4768,8 +4829,24 @@ program
                 url: `https://github.com/formigio/${r}.git`,
                 branch: 'main',
             })),
-            cwd: '/workspace',
         };
+        // [F24 §3.2 / D8 / rul_workspace_clone_contract] cwd defaulting per
+        // resolved repo count. Pre-F24 the CLI hardcoded cwd='/workspace',
+        // which is ALWAYS empty inside the runner (clones land at
+        // /home/agent/development/<repo>). BUG20's symptom ("'.git' missing")
+        // was just the cwd misalignment — the clone itself was already
+        // correct. Fix:
+        //   single repo  → /home/agent/development/<repo>  (align with clone)
+        //   multi repo   → unset, let runner pick first repo via its existing
+        //                  fallback at fazemos-agent-runner.ts:1075
+        //   zero repos   → /workspace (preserve chat_session and no-repo cases)
+        if (repoNames.length === 1) {
+            context.cwd = `/home/agent/development/${repoNames[0]}`;
+        }
+        else if (repoNames.length === 0) {
+            context.cwd = '/workspace';
+        }
+        // multi-repo: leave context.cwd unset — runner fallback chooses
         // Resolve worksheet context for actions/commitments
         if (opts.sourceType === 'action' || opts.sourceType === 'commitment') {
             try {
@@ -4786,6 +4863,17 @@ program
                         context.worksheetName = detail.worksheet?.name;
                         context.worksheetPurpose = detail.worksheet?.purpose;
                         context.actionDescription = match.description || match.name;
+                        // [F24 §3.3 / D3 / rul_action_completion_semantics]
+                        // Propagate the action's target_value into context so the
+                        // agent's buildSelfReportingInstructions can branch:
+                        //   target_value null|1 → binary auto-complete with -v 1
+                        //   target_value > 1    → emit a "non-binary target" note,
+                        //                          NO auto-completion command
+                        // Only meaningful for source_type='action'; we set it only
+                        // here (the action-resolution branch).
+                        if (opts.sourceType === 'action') {
+                            context.actionTargetValue = match.target_value ?? null;
+                        }
                         // Include linked outcomes
                         if (detail.outcomes?.length) {
                             context.outcomes = detail.outcomes.map((o) => ({
@@ -4808,6 +4896,12 @@ program
         }
         if (opts.prompt)
             context.prompt = opts.prompt;
+        // [F24 §3.3 / D2] Forward --no-auto-complete intent. The CLI guard
+        // above suppresses this for pipeline_step (forwardSuppress is false
+        // in that case). Agent's buildSelfReportingInstructions reads
+        // context.noAutoComplete; absent/false → default behavior.
+        if (forwardSuppress)
+            context.noAutoComplete = true;
         const body = {
             sourceType: opts.sourceType,
             sourceId,