@lardexx/arenacode 2.6.0 → 2.6.1

package/README.md

@@ -115,7 +115,7 @@ How the copy is built and tested: [`docs/TUI.md`](docs/TUI.md).
 | `!command` | run a command locally |
 | `↑` / `↓` | prompt history (or move through the slash list) |
 | `pgup` / `pgdn` | scroll the transcript |
-| `ctrl+x` then | `q` quit · `n` new session · `l` task queue · `t` theme · `d` tool output · `b` sidebar · `s` stop |
+| `ctrl+x` then | `q` quit · `n` new session · `l` task queue · `t` theme · `a` Ask and Proceed / Proceed · `d` tool output · `s` stop |
 | `ctrl+c` | interrupt, then quit |
 
 **The same four flows as the desktop client.** The Windows app is the reference for how a task reaches the bridge,
@@ -134,6 +134,13 @@ so the port reproduces it byte for byte:
 
 The exact strings are checked by `test/parity.js` against the ones extracted from `ArenaCode-x64.exe`.
 
+**Ask and Proceed / Proceed.** The desktop's Settings switch, `ctrl+x a` here (also `/ask`, `/proceed`,
+`--proceed`, and `arenacode config set approval proceed`). The default, **Ask and Proceed**, asks before commands,
+file changes and plans; in Computer mode it asks once, on the plan, and after that the commands run on their own
+while administrator commands still ask. **Proceed** approves its own plan, runs commands and installations and
+writes files without asking: turning it on shows the desktop's own warning (*"Pericolo: modalità Proceed"*).
+`--yes` is the older switch and still answers yes to commands and files, but never to a plan.
+
 **Dialogs.** Local actions open opencode-shaped dialogs — panel background, title on the left, `esc` on the right,
 the selected row painted in the primary colour, hints underneath: `Run once / Don't run` for terminal commands,
 `Approve / Reject / Revise` for the agent's plan, and file operations reported back to the bridge. `--yes` skips the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lardexx/arenacode",
-  "version": "2.6.0",
+  "version": "2.6.1",
   "description": "Terminal client for ArenaCode: talks to an arena-bridge, streams the Agent Mode timeline, and runs local commands and file operations on your PC. Node port of the ArenaCode desktop client.",
   "license": "MIT",
   "type": "commonjs",

package/src/approval.js

@@ -0,0 +1,98 @@
+'use strict';
+
+// The desktop client has one setting, Settings → "Ask and Proceed" (default) or
+// "Proceed", and it changes every approval the client makes:
+//
+//   Ask and Proceed (IDE mode)   file changes wait for Accept/Reject, commands wait for Run
+//   Ask and Proceed (Computer)   one confirmation, on the plan; then commands run alone
+//                                (commands with administrator rights still ask)
+//   Proceed                      the client approves its own plan, runs commands and
+//                                installations and uses mouse and keyboard without asking
+//
+// `--yes` is the old switch of this client and keeps meaning "say yes to commands
+// and file changes", but never to a plan: a plan is the user's decision.
+const APPROVALS = ['ask', 'proceed'];
+
+const DANGER = {
+  it: {
+    title: 'Pericolo: modalità Proceed',
+    body:
+      'PERICOLO: in Proceed il Computer Agent approva da solo il piano, esegue comandi e installazioni ' +
+      'e usa mouse e tastiera senza chiedere. Può cancellare o modificare dati e programmi.',
+  },
+  en: {
+    title: 'Danger: Proceed mode',
+    body:
+      'DANGER: in Proceed the Computer Agent approves its own plan, runs commands and installations, and ' +
+      'uses mouse and keyboard without asking. It can delete or change data and programs.',
+  },
+  es: {
+    title: 'Peligro: modo Proceed',
+    body:
+      'PELIGRO: en Proceed el Computer Agent aprueba su propio plan, ejecuta comandos e instalaciones y ' +
+      'usa el ratón y el teclado sin preguntar. Puede borrar o modificar datos y programas.',
+  },
+};
+
+const HELP_TEXT = {
+  it: {
+    ide:
+      'Ask and Proceed: Arena chiede conferma per comandi, modifiche ai file e piani. ' +
+      'Proceed: esegue tutto da solo.',
+    computer:
+      'Ask and Proceed: il Computer Agent chiede conferma una sola volta, sul piano. Dopo ' +
+      "l'approvazione esegue i comandi da solo (i comandi con permessi di amministratore chiedono " +
+      'ancora conferma). Proceed: agisce da solo, senza chiedere nulla.',
+  },
+  en: {
+    ide:
+      'Ask and Proceed: Arena asks before commands, file changes and plans. ' +
+      'Proceed: does everything on its own.',
+    computer:
+      'Ask and Proceed: the Computer Agent asks for confirmation once, on the plan. After approval ' +
+      'it runs commands on its own (commands with administrator rights still ask). Proceed: it acts ' +
+      'on its own, without asking anything.',
+  },
+  es: {
+    ide:
+      'Ask and Proceed: Arena pide confirmación para comandos, cambios en archivos y planes. ' +
+      'Proceed: hace todo por sí solo.',
+    computer:
+      'Ask and Proceed: el Computer Agent pide confirmación una sola vez, sobre el plan. Tras ' +
+      'aprobarlo ejecuta los comandos por sí solo (los comandos con permisos de administrador ' +
+      'siguen pidiendo confirmación). Proceed: actúa por sí solo, sin preguntar nada.',
+  },
+};
+
+function normaliseApproval(value) {
+  return APPROVALS.includes(value) ? value : 'ask';
+}
+
+/**
+ * What the client should do with one request from the agent.
+ * kind: 'plan' | 'fs' | 'exec'
+ * Returns 'auto' (answer yes without showing anything) or 'ask' (show the card).
+ */
+function decide({ approval = 'ask', mode = 'agent', planApproved = false, kind, admin = false, yes = false }) {
+  if (normaliseApproval(approval) === 'proceed') return 'auto';
+  if (kind === 'plan') return 'ask';
+  if (yes) return 'auto';
+  if (kind === 'fs') return 'ask';
+  if (kind === 'exec') {
+    if (admin) return 'ask';
+    if (mode === 'computer-agent' && planApproved) return 'auto';
+    return 'ask';
+  }
+  return 'ask';
+}
+
+function dangerText(lang) {
+  return DANGER[lang] || DANGER.en;
+}
+
+function approvalHelp(lang, mode) {
+  const set = HELP_TEXT[lang] || HELP_TEXT.en;
+  return mode === 'computer-agent' ? set.computer : set.ide;
+}
+
+module.exports = { APPROVALS, decide, dangerText, approvalHelp, normaliseApproval, DANGER, HELP_TEXT };

package/src/cli.js

@@ -11,6 +11,8 @@ const {
 const { Bridge } = require('./bridge');
 const { MODE_LINES, MODES, normaliseMode } = require('./modes');
 const { attachRefs, decorate } = require('./attach');
+const { decide, dangerText, normaliseApproval } = require('./approval');
+const session = require('./session');
 const { buildMeta, readProjectContext } = require('./meta');
 const { followTask } = require('./turn');
 const { c, write, ask, banner, truncate, indentLines } = require('./ui');
@@ -37,7 +39,7 @@ const INSTALL_HINT = process.platform === 'win32'
 // argument parsing
 // --------------------------------------------------------------------------- //
 
-const BOOLEAN_FLAGS = new Set(['yes', 'json', 'help', 'quiet', 'version', 'confirm-writes', 'no-pc', 'piped']);
+const BOOLEAN_FLAGS = new Set(['yes', 'json', 'help', 'quiet', 'version', 'confirm-writes', 'no-pc', 'piped', 'proceed', 'ask']);
 
 function parseArgs(argv) {
   const flags = {};
@@ -100,6 +102,7 @@ function context(flags) {
     yes: Boolean(flags.yes),
     json: Boolean(flags.json),
     confirmWrites: Boolean(flags['confirm-writes']),
+    approval: normaliseApproval(flags.proceed ? 'proceed' : flags.ask ? 'ask' : cfg.approval),
     idleTimeout: Number(flags['idle-timeout'] || cfg.idle_timeout || 180),
     bridge: new Bridge(url, key, device),
   };
@@ -191,7 +194,13 @@ function cmdConfig(positional, flags) {
     const name = key;
     const val = rest.join(' ');
     if (!name || val === '') {
-      write(`usage: ${SELF} config set <url|key|lang|mode|pc_access> <value>`);
+      write(`usage: ${SELF} config set <url|key|lang|mode|approval|pc_access> <value>`);
+      return 2;
+    }
+    if (name === 'approval' && !['ask', 'proceed'].includes(String(val))) {
+      write(c.red('  ✗ approval must be ask or proceed'));
+      write(c.gray(`    ask      ${'Ask and Proceed'}: the client asks before commands, file changes and plans`));
+      write(c.gray(`    proceed  Proceed: the client does everything on its own`));
       return 2;
     }
     const patch = {};
@@ -200,6 +209,12 @@ function cmdConfig(positional, flags) {
     else patch[name] = val;
     const file = patchConfig(patch);
     write(`  ${c.green('✓')} ${name} saved ${c.gray(file)}`);
+    if (name === 'approval' && patch.approval === 'proceed') {
+      const d = dangerText(cfg.lang);
+      write('');
+      write(`  ${c.bold(c.red('⚠ ' + d.title))}`);
+      write(indentLines(c.yellow(d.body), '  '));
+    }
     return 0;
   }
   if (action === 'clear') {
@@ -283,6 +298,7 @@ async function createTask(ctx, instruction, { extraInstructions = '' } = {}) {
   if (extraInstructions) body.instruction = `${instruction}\n\n${extraInstructions}`;
 
   const res = await ctx.bridge.createTask(body);
+  if (res.status === 202 || res.status === 200) session.remember(res.json && res.json.task_id, ctx.mode);
   if (res.status !== 202 && res.status !== 200) {
     const reason = (res.json && (res.json.error || res.json.hint)) || `HTTP ${res.status}`;
     throw new Error(reason);
@@ -322,6 +338,7 @@ async function runInstruction(ctx, text, { attachments = [] } = {}) {
       folder: ctx.folder,
       lang: ctx.lang,
       yes: ctx.yes,
+      approval: ctx.approval,
       confirmWrites: ctx.confirmWrites,
       idleTimeout: ctx.idleTimeout,
     });
@@ -381,6 +398,7 @@ async function cmdUi(flags) {
     lang: ctx.lang,
     mode: ctx.mode,
     yes: ctx.yes,
+    approval: ctx.approval,
     confirmWrites: ctx.confirmWrites,
   });
   await app.start();
@@ -743,7 +761,9 @@ function help() {
   write(`    --lang it|en|es            language for instructions and cards`);
   write(`    --no-pc                    tell the agent the PC terminal is off`);
   write(`    --yes                      run local commands without asking`);
-  write(`    --confirm-writes           ask before write/replace/rm/mv on your folder`);
+  write(`    --ask                      Ask and Proceed: ask before commands, file changes and plans (default)
+    --proceed                  Proceed: approve the plan and run everything without asking
+    --confirm-writes           ask before write/replace/rm/mv on your folder`);
   write(`    --attach a,b               upload files with the task`);
   write(`    --json                     raw JSON output where available`);
   write('');

package/src/config.js

@@ -29,6 +29,7 @@ const DEFAULTS = {
   folder: '',
   lang: 'it',
   mode: 'agent',
+  approval: 'ask',
   pc_access: true,
   device: '',
   vision: false,

package/src/local.js

@@ -1,5 +1,8 @@
 'use strict';
 
+const { decide } = require('./approval');
+const session = require('./session');
+
 const fs = require('node:fs');
 const os = require('node:os');
 const path = require('node:path');
@@ -98,7 +101,18 @@ async function handleLocalExec(bridge, taskId, request, opts) {
   if (admin) write(`  ${c.yellow('!')} ${t(lang, 'Richiede i permessi di amministratore: qui il comando gira senza UAC.', 'Needs administrator rights: here it runs without UAC.', 'Necesita permisos de administrador: aquí se ejecuta sin UAC.')}`);
   if (sandboxReason) write(`  ${c.yellow('!')} ${t(lang, 'L’agente chiede di installare sul PC', 'The agent asks to install on the PC', 'El agente pide instalar en el PC')}: ${c.gray(sandboxReason)}`);
 
-  let approved = opts.yes === true;
+  const verdict = decide({
+    approval: opts.approval,
+    mode: session.modeOf(taskId),
+    planApproved: session.planApproved(taskId),
+    kind: 'exec',
+    admin: Boolean(admin),
+    yes: opts.yes === true,
+  });
+  let approved = verdict === 'auto';
+  if (verdict === 'auto') {
+    write(`  ${c.green('✓')} ${c.gray(t(lang, 'approvato dal client (Proceed)', 'approved by the client (Proceed)', 'aprobado por el cliente (Proceed)'))}`);
+  }
   if (!approved) {
     const key = await askKey(`  ${t(lang, 'Eseguo? [r] sì  [d] no', 'Run it? [r] yes  [d] no', '¿Ejecuto? [r] sí  [d] no')} `, ['r', 'd']);
     approved = key === 'r';
@@ -368,7 +382,18 @@ async function handleLocalFs(bridge, taskId, request, opts) {
   write(`  ${c.bold(c.yellow('▤ files'))} ${c.dim('·')} ${c.cyan(op)} ${c.gray(request.path || '.')}${request.to ? c.gray(` → ${request.to}`) : ''}`);
 
   const mutating = ['write', 'replace', 'rm', 'mv', 'mkdir'].includes(op);
-  if (mutating && opts.confirmWrites) {
+  const verdict = mutating
+    ? decide({
+        approval: opts.approval,
+        mode: session.modeOf(taskId),
+        planApproved: session.planApproved(taskId),
+        kind: 'fs',
+        yes: opts.yes === true,
+      })
+    : 'auto';
+  // --confirm-writes keeps asking even when the client runs in Proceed
+  const mustAsk = mutating && (verdict === 'ask' || opts.confirmWrites);
+  if (mustAsk) {
     const key = await askKey(`  ${t(lang, 'Applico? [s] sì  [n] no', 'Apply? [y] yes  [n] no', '¿Aplico? [s] sí  [n] no')} `, ['s', 'y', 'n']);
     if (key === 'n') {
       await bridge.localResult(taskId, { rid, output: JSON.stringify({ ok: false, error: 'rejected by the user' }), code: 1 });
@@ -406,6 +431,20 @@ async function handleLocalUi(bridge, taskId, request, opts) {
   const lang = opts.lang;
 
   if (op === 'plan') {
+    const planVerdict = decide({
+      approval: opts.approval,
+      mode: session.modeOf(taskId),
+      kind: 'plan',
+      yes: opts.yes === true,
+    });
+    if (planVerdict === 'auto') {
+      session.markPlanApproved(taskId);
+      write('');
+      write(`  ${c.bold(c.magenta('✦ plan'))} ${c.dim('·')} ${t(lang, 'approvato dal client (Proceed)', 'approved by the client (Proceed)', 'aprobado por el cliente (Proceed)')}`);
+      write(indentLines(request.text || '', '    '));
+      await bridge.localResult(taskId, { rid, output: JSON.stringify({ ok: true, approved: true }), code: 0 });
+      return true;
+    }
     write('');
     write(`  ${c.bold(c.magenta('✦ plan'))} ${c.dim('·')} ${t(lang, 'da approvare', 'to approve', 'para aprobar')}`);
     write(indentLines(request.text || '', '    '));
@@ -415,6 +454,7 @@ async function handleLocalUi(bridge, taskId, request, opts) {
       ['a', 'r', 'f'],
     );
     if (key === 'a') {
+      session.markPlanApproved(taskId);
       await bridge.localResult(taskId, { rid, output: JSON.stringify({ ok: true, approved: true }), code: 0 });
       write(`  ${c.green('✓')} ${t(lang, 'piano approvato', 'plan approved', 'plan aprobado')}`);
     } else if (key === 'f') {

package/src/session.js

@@ -0,0 +1,33 @@
+'use strict';
+
+// Per-task state the approval rules need: the mode the task was started in and
+// whether its plan has already been approved (Computer Agent mode runs its
+// commands alone after that one approval).
+const tasks = new Map();
+
+function remember(taskId, mode) {
+  if (!taskId) return;
+  tasks.set(taskId, { mode: mode || 'agent', planApproved: false });
+}
+
+function modeOf(taskId) {
+  const t = tasks.get(taskId);
+  return t ? t.mode : 'agent';
+}
+
+function markPlanApproved(taskId) {
+  const t = tasks.get(taskId);
+  if (t) t.planApproved = true;
+  else if (taskId) tasks.set(taskId, { mode: 'agent', planApproved: true });
+}
+
+function planApproved(taskId) {
+  const t = tasks.get(taskId);
+  return Boolean(t && t.planApproved);
+}
+
+function forget(taskId) {
+  tasks.delete(taskId);
+}
+
+module.exports = { remember, modeOf, markPlanApproved, planApproved, forget };

package/src/tui/app.js

@@ -20,6 +20,8 @@ const { readConfig, patchConfig, deviceId, resolveUrl, resolveKey, openFolder, V
 const { MODE_LINES, modeLabel, nextMode, normaliseMode } = require('../modes');
 const { attachRefs, attachImage, decorate, strings } = require('../attach');
 const { grabImage, readClipboardText, stamp } = require('../clipboard');
+const { decide, dangerText, approvalHelp, normaliseApproval } = require('../approval');
+const session = require('../session');
 
 const AGENT_PROMPT = (() => {
   try {
@@ -46,6 +48,7 @@ class App {
     this.theme = getTheme(this.themeName);
     this.autoApprove = Boolean(opts.yes);
     this.confirmWrites = Boolean(opts.confirmWrites);
+    this.approval = normaliseApproval(opts.approval);
 
     this.state = {
       mode: normaliseMode(opts.mode || cfg.mode || 'agent'),
@@ -93,6 +96,12 @@ class App {
     });
     this.term.on('resize', () => this.invalidate());
     this.running = true;
+    if (this.approval === 'proceed') {
+      this.push({
+        kind: 'error',
+        text: `approval: Proceed · ${approvalHelp(this.lang, this.state.mode)}`,
+      });
+    }
 
     const interval = setInterval(() => {
       this.state.spinner = (this.state.spinner + 1) % SPINNER.length;
@@ -285,6 +294,8 @@ class App {
         return undefined;
       case 's':
         return this.stopTurn();
+      case 'a':
+        return this.toggleApproval();
       case 'p':
         return this.openPalette();
       default:
@@ -450,6 +461,7 @@ class App {
         this.state.mode = nextMode(this.state.mode);
         this.toast(`mode: ${modeLabel(this.state.mode)}`);
       } },
+      { title: `Approval: ${this.approval === 'proceed' ? 'Proceed' : 'Ask and Proceed'}`, hint: 'ctrl+x a', run: () => this.toggleApproval() },
       { title: 'Theme', hint: 'ctrl+x t', run: () => {
         const names = themeNames();
         const next = names[(names.indexOf(this.themeName) + 1) % names.length];
@@ -470,17 +482,62 @@ class App {
 
   /** `/help`, `/new`, `/tasks` … — opencode-style slash commands */
   slashCommands() {
+    const by = (prefix) => {
+      const found = this.commands().find((cmd) => cmd.title.startsWith(prefix));
+      return () => (found ? found.run() : undefined);
+    };
     return [
-      { slash: '/new', title: 'Start a new session', hint: 'ctrl+x n', group: 'Session', run: () => this.commands()[1].run() },
+      { slash: '/new', title: 'Start a new session', hint: 'ctrl+x n', group: 'Session', run: by('New session') },
       { slash: '/tasks', title: 'Show the task queue', hint: 'ctrl+x l', group: 'Session', run: () => this.openTasks() },
-      { slash: '/mode', title: 'Switch agent / plan', hint: 'tab', group: 'Session', run: () => this.commands()[5].run() },
-      { slash: '/theme', title: 'Cycle the colour theme', hint: 'ctrl+x t', group: 'Session', run: () => this.commands()[6].run() },
-      { slash: '/status', title: 'Ping the bridge', hint: '', group: 'Session', run: () => this.commands()[7].run() },
+      { slash: '/mode', title: 'Switch agent / plan', hint: 'tab', group: 'Session', run: by('Switch mode') },
+      { slash: '/ask', title: 'Ask and Proceed: ask before commands, files and plans', hint: '', group: 'Session', run: () => this.setApproval('ask') },
+      { slash: '/proceed', title: 'Proceed: approve the plan and run everything without asking', hint: '', group: 'Session', run: () => this.setApproval('proceed') },
+      { slash: '/theme', title: 'Cycle the colour theme', hint: 'ctrl+x t', group: 'Session', run: by('Theme') },
+      { slash: '/status', title: 'Ping the bridge', hint: '', group: 'Session', run: by('Bridge status') },
       { slash: '/help', title: 'Every keybinding', hint: 'f1', group: 'Session', run: () => this.openHelp() },
       { slash: '/quit', title: 'Leave arenacode', hint: 'ctrl+x q', group: 'Session', run: () => this.quit() },
     ];
   }
 
+  /** Ask and Proceed ↔ Proceed, the desktop's Settings switch. */
+  async toggleApproval() {
+    await this.setApproval(this.approval === 'proceed' ? 'ask' : 'proceed');
+  }
+
+  async setApproval(mode) {
+    const next = normaliseApproval(mode);
+    if (next === this.approval) {
+      this.toast(`approval: ${next === 'proceed' ? 'Proceed' : 'Ask and Proceed'}`);
+      return;
+    }
+    if (next === 'proceed') {
+      const danger = dangerText(this.lang);
+      const answer = await this.confirm({
+        title: danger.title,
+        subject: danger.body,
+        options: [
+          { key: 'y', label: 'enable Proceed', tone: 'err' },
+          { key: 'n', label: 'keep Ask and Proceed', tone: 'ok' },
+        ],
+      });
+      if (answer !== 'y') {
+        this.toast('approval: Ask and Proceed');
+        return;
+      }
+    }
+    this.approval = next;
+    try {
+      patchConfig({ approval: next });
+    } catch {
+      /* a read-only HOME keeps the switch in memory only */
+    }
+    this.push({
+      kind: next === 'proceed' ? 'error' : 'system',
+      text: `approval: ${next === 'proceed' ? 'Proceed' : 'Ask and Proceed'} · ${approvalHelp(this.lang, this.state.mode)}`,
+    });
+    this.toast(`approval: ${next === 'proceed' ? 'Proceed' : 'Ask and Proceed'}`, next === 'proceed' ? 'warn' : 'ok');
+  }
+
   onOverlayKey(key) {
     const ov = this.state.overlay;
     if (key.name === 'escape') {
@@ -659,6 +716,7 @@ class App {
       return;
     }
     st.taskId = res.json.task_id;
+    session.remember(st.taskId, st.mode);
     st.since = 0;
     st.seqItems.clear();
   }
@@ -839,7 +897,15 @@ class App {
 
     if (type === 'exec') {
       item.kindDetail = payload.command;
-      const answer = this.autoApprove
+      const verdict = decide({
+        approval: this.approval,
+        mode: session.modeOf(taskId),
+        planApproved: session.planApproved(taskId),
+        kind: 'exec',
+        admin: Boolean(payload.admin),
+        yes: this.autoApprove,
+      });
+      const answer = verdict === 'auto'
         ? 'y'
         : await this.confirm({
             title: 'terminal',
@@ -873,7 +939,16 @@ class App {
     if (type === 'fs') {
       const op = String(payload.op || '');
       const mutating = ['write', 'replace', 'rm', 'mv', 'mkdir'].includes(op);
-      if (mutating && this.confirmWrites) {
+      const fsVerdict = mutating
+        ? decide({
+            approval: this.approval,
+            mode: session.modeOf(taskId),
+            planApproved: session.planApproved(taskId),
+            kind: 'fs',
+            yes: this.autoApprove,
+          })
+        : 'auto';
+      if (mutating && (fsVerdict === 'ask' || this.confirmWrites)) {
         const approved = (await this.confirm({
           title: `file · ${op}`,
           subject: payload.path || '',
@@ -910,6 +985,20 @@ class App {
     if (type === 'ui') {
       const op = String(payload.op || '');
       if (op === 'plan') {
+        const planVerdict = decide({
+          approval: this.approval,
+          mode: session.modeOf(taskId),
+          kind: 'plan',
+          yes: false,
+        });
+        if (planVerdict === 'auto') {
+          session.markPlanApproved(taskId);
+          this.push({ kind: 'system', text: `plan approved by the client (Proceed): ${String(payload.text || '').split('\n')[0]}` });
+          await this.bridge.localResult(taskId, { rid: payload.rid, output: JSON.stringify({ ok: true, approved: true }), code: 0 });
+          item.state = 'done';
+          this.invalidate();
+          return;
+        }
         const answer = await this.confirm({
           title: 'plan',
           subject: payload.text || '',
@@ -921,7 +1010,10 @@ class App {
           ],
         });
         let payloadOut;
-        if (answer === 'a') payloadOut = { ok: true, approved: true };
+        if (answer === 'a') {
+          payloadOut = { ok: true, approved: true };
+          session.markPlanApproved(taskId);
+        }
         else if (answer === 'f') {
           const feedback = await this.prompt('how should it be revised?');
           payloadOut = { ok: false, approved: false, revise: true, feedback: feedback || '' };
@@ -1375,14 +1467,16 @@ function HELP_ROWS() {
     ['Prompt', 'enter', 'send the message'],
     ['Prompt', 'shift+enter', 'newline'],
     ['Prompt', 'esc', 'interrupt the turn'],
-    ['Prompt', 'tab', 'switch agent / plan'],
+    ['Prompt', 'tab', 'switch mode: Build / Plan / Computer'],
     ['Prompt', 'ctrl+p', 'command palette'],
     ['Prompt', '/', 'slash commands'],
-    ['Prompt', '@', 'attach a file from the folder'],
+    ['Prompt', '@', 'mention a file or a folder'],
+    ['Prompt', 'ctrl+v', 'paste: a clipboard image becomes an attachment'],
     ['Prompt', '!', 'run a command locally'],
     ['Prompt', 'up / down', 'history'],
     ['Session', 'ctrl+x n', 'new session'],
     ['Session', 'ctrl+x l', 'task queue'],
+    ['Session', 'ctrl+x a', 'Ask and Proceed / Proceed'],
     ['Session', 'ctrl+x t', 'switch theme'],
     ['Session', 'ctrl+x d', 'toggle tool output'],
     ['Session', 'pgup / pgdn', 'scroll'],