agen-vektor 0.3.19 → 0.3.21

package/README.md

@@ -408,7 +408,7 @@ npm run build
 - **"model not found" on the free tier** — the free catalog rotates; run `/model` to pick from the live list.
 - **"Rate limited (429) / cooling down"** — VectorHead retries automatically with backoff, and the free gateway cools down briefly instead of burning the shared key. Switch to another free model via `/model` and retry.
 - **Nothing happens on `vector` in Termux** — make sure your terminal is a TTY and TERM is set (e.g. `export TERM=xterm-256color`). If glyphs look broken (█ everywhere, torn logo), ASCII safe mode is already default-on; desktop users can opt back in with `VECTOR_ASCII=0`.
-- **Runs out of iterations** — raise `maxIterations` in `~/.vector/config.json` or refine the request.
+- **Runs out of iterations** — open `/settings` in the TUI and press `+`/`-` (or PgUp/PgDn, or type a number) to raise `maxIterations` — persisted to `~/.vector/config.json` — or refine the request.
 
 ## Roadmap
 

package/dist/tools/shell.js

@@ -7,39 +7,94 @@ exports.createShellTool = createShellTool;
  * Every command is classified by the command policy and gated by permissions.
  */
 const node_child_process_1 = require("node:child_process");
+// After the shell itself exits, wait this long for the stdio pipes to drain
+// (normal commands flush and close within milliseconds) before giving up.
+// A command that ORPHANS a child holding the pipes (background job, server,
+// watcher …) would otherwise keep the tool — and the whole agent loop —
+// hanging until the grandchild dies ("sering nyangkut di shell", 2026-09-10).
+const ORPHAN_GRACE_MS = 500;
 function runShell(command, cwd, opts = {}) {
     return new Promise((resolve) => {
         const timeoutMs = opts.timeoutMs || 120_000;
+        // Own process group (detached) → the timeout / orphan guard can kill the
+        // whole tree (shell AND grandchildren), same policy as background.ts.
         const child = (0, node_child_process_1.spawn)(command, {
             cwd,
             shell: '/bin/sh',
             env: { ...process.env, ...opts.env },
             stdio: ['ignore', 'pipe', 'pipe'],
+            detached: true,
         });
         let stdout = '';
         let stderr = '';
         let timedOut = false;
+        let settled = false;
+        let graceTimer = null;
+        const killTree = () => {
+            if (child.pid != null) {
+                try {
+                    process.kill(-child.pid, 'SIGKILL'); // detached spawn ⇒ own process group
+                }
+                catch {
+                    /* group already gone */
+                }
+            }
+            try {
+                child.kill('SIGKILL');
+            }
+            catch {
+                /* already gone */
+            }
+        };
+        const finish = (code) => {
+            if (settled)
+                return;
+            settled = true;
+            clearTimeout(timer);
+            if (graceTimer)
+                clearTimeout(graceTimer);
+            resolve({ stdout, stderr, exitCode: code, timedOut });
+        };
         const timer = setTimeout(() => {
             timedOut = true;
-            child.kill('SIGKILL');
+            killTree(); // was: child.kill() only — orphaned children kept the pipe open and the promise hanging past the timeout
+            // 'exit' fires once the shell dies and resolves immediately.
         }, timeoutMs);
         child.stdout.on('data', (d) => {
             stdout += d.toString();
             if (stdout.length > 2_000_000)
-                child.kill('SIGKILL');
+                killTree();
         });
         child.stderr.on('data', (d) => {
             stderr += d.toString();
             if (stderr.length > 2_000_000)
-                child.kill('SIGKILL');
+                killTree();
+        });
+        // 'exit' = the shell itself died; 'close' = shell AND stdio pipes closed.
+        // Resolve on whichever happens first with a small grace window after
+        // 'exit': a normal command flushes and closes well inside the window, so
+        // full output still arrives — but an orphaned grandchild holding the
+        // pipes can no longer hang the tool past it.
+        child.on('exit', (code) => {
+            if (timedOut) {
+                finish(code);
+                return;
+            }
+            if (graceTimer)
+                return;
+            graceTimer = setTimeout(() => {
+                killTree(); // stop the orphan so its output stops too
+                finish(code);
+            }, ORPHAN_GRACE_MS);
         });
         child.on('close', (code) => {
-            clearTimeout(timer);
-            resolve({ stdout, stderr, exitCode: code, timedOut });
+            finish(code);
         });
         child.on('error', (e) => {
-            clearTimeout(timer);
-            resolve({ stdout, stderr: `spawn error: ${e.message}`, exitCode: -1, timedOut });
+            // set stderr BEFORE finish — the resolved object snapshots these strings
+            if (!stdout && !stderr)
+                stderr = `spawn error: ${e.message}`;
+            finish(-1);
         });
     });
 }
@@ -52,7 +107,7 @@ function createShellTool() {
     return {
         definition: {
             name: 'shell',
-            description: 'Run a shell command in the project directory. The command is classified for safety; risky commands require permission. Use for building, testing, installing, git, or any command execution. Long output is truncated. For git commits prefer the dedicated `git` tool and follow its commit-attribution guidance.',
+            description: 'Run a shell command in the project directory. The command is classified for safety; risky commands require permission. Use for building, testing, installing, git, or any command execution. Long output is truncated. Commands that keep running (dev servers, watchers, polling loops) must use run_in_background instead — a foreground shell is killed after its timeout. For git commits prefer the dedicated `git` tool and follow its commit-attribution guidance.',
             parameters: {
                 type: 'object',
                 properties: {

package/dist/tui/app.js

@@ -2373,7 +2373,6 @@ class App {
             case 'help':
             case 'commands':
             case 'status':
-            case 'settings':
                 if (ev.name === 'escape' || ev.name === 'enter' || ev.name === 'ctrl_c' || (ev.name === 'char' && ev.char === '?')) {
                     this.modal = { type: 'none' };
                 }
@@ -2392,6 +2391,74 @@ class App {
                     this.followBottom = this.scroll >= Math.max(0, all.length - this.chatHeight());
                 }
                 break;
+            case 'settings':
+                // /settings is mostly read-only, but Max iterations is LIVE-EDITABLE:
+                // hitting the iteration limit mid-task used to require manual
+                // ~/.vector/config.json surgery. +/- (and up/down) step the budget,
+                // 0 and number keys type an exact value, Enter/commit keeps the
+                // dialog open (backspace too — stray presses must not vaporize it),
+                // Esc still closes. Changes persist via saveConfig immediately.
+                {
+                    const cfg = this.agent.config;
+                    const step = (d) => {
+                        const next = Math.min(200, Math.max(1, cfg.maxIterations + d));
+                        if (next === cfg.maxIterations)
+                            return;
+                        cfg.maxIterations = next;
+                        try {
+                            (0, config_1.saveConfig)(cfg);
+                        }
+                        catch {
+                            /* read-only home — in-memory value still applies this session */
+                        }
+                        this.addSystem(`Max iterations: ${next}.`);
+                    };
+                    if (ev.name === 'escape') {
+                        this.modal = { type: 'none' };
+                    }
+                    else if (ev.name === 'enter' || ev.name === 'ctrl_c') {
+                        this.addSystem(`Max iterations: ${cfg.maxIterations} (tersimpan di ~/.vector/config.json).`);
+                    }
+                    else if (ev.name === 'up' || ev.name === 'char' && ev.char === '+') {
+                        step(+1);
+                    }
+                    else if (ev.name === 'down' || ev.name === 'char' && ev.char === '-') {
+                        step(-1);
+                    }
+                    else if (ev.name === 'pageup') {
+                        step(+10);
+                    }
+                    else if (ev.name === 'pagedown') {
+                        step(-10);
+                    }
+                    else if (ev.name === 'char' && ev.char === '0') {
+                        step(-cfg.maxIterations + 1);
+                    }
+                    else if (ev.name === 'char' && ev.char && /[0-9]/.test(ev.char)) {
+                        // Number keys type the exact value (not digit-append — no
+                        // visible field, so append would be invisible state).
+                        const next = Math.min(200, Math.max(1, Number(ev.char)));
+                        if (next !== cfg.maxIterations) {
+                            cfg.maxIterations = next;
+                            try {
+                                (0, config_1.saveConfig)(cfg);
+                            }
+                            catch {
+                                /* read-only home */
+                            }
+                            this.addSystem(`Max iterations: ${next}.`);
+                        }
+                    }
+                    else if (ev.name === 'char' && ev.char === '?') {
+                        this.modal = { type: 'none' };
+                    }
+                    else if (ev.name === 'char' && ev.char) {
+                        // Other letters: keep the old close-and-insert behavior.
+                        this.modal = { type: 'none' };
+                        this.input.insert(ev.char);
+                    }
+                }
+                break;
             case 'provider':
                 await this.handleSelector(modal, ev, this.providerPickerOptions(), async (opt) => {
                     const c = this.agent.config;
@@ -3166,7 +3233,9 @@ class App {
                         label('Model', this.agent.config.model),
                         label('Theme', this.agent.config.theme || 'vectorhead'),
                         label('Mode', this.agent.config.permissionMode),
-                        label('Max iterations', String(this.agent.config.maxIterations)),
+                        // LIVE-EDITABLE: +/- / ↑↓ / PgUp-PgDn step the budget, digits
+                        // type an exact value — persisted to ~/.vector/config.json.
+                        label('Max iterations', `${this.agent.config.maxIterations}  ${theme_1.THEME.muted}(+/- / ↑↓ / PgUp-PgDn / angka)${theme_1.THEME.reset}`),
                         label('Max retries', String(this.agent.config.maxRetries)),
                         label('API URL', this.agent.config.apiUrl || '(default)'),
                         label('Key configured', (0, credentials_1.hasApiKey)(this.agent.config.provider) ? 'yes' : 'no'),
@@ -3184,7 +3253,7 @@ class App {
                         label('Env vars', 'VECTOR_API_KEY VECTOR_API_URL'),
                         label('', 'VECTOR_MODEL VECTOR_PROVIDER'),
                     ],
-                    footer: 'Esc/Enter close · edit via the commands above',
+                    footer: 'Esc close · +/- ubah max iterations (tersimpan otomatis)',
                 }, rows, cols);
             }
             case 'status':
@@ -3216,9 +3285,16 @@ class App {
                         body: ['Fetching models from provider…', '', 'Esc to cancel'],
                     }, rows, cols);
                 }
-                if (m.error) {
+                if (m.error && m.options.length === 0) {
                     // Fetch failed (bad/missing URL or stored key) — show the real
                     // reason (e.g. "HTTP 401 …") instead of a misleading 1-item list.
+                    // BUT only when there is NOTHING selectable: if the provider def
+                    // declares models in config, those options were already merged in
+                    // (openModelModal always populates options from the fallback list
+                    // before setting error) and the picker must stay usable — endpoints
+                    // without GET /models (e.g. Cloudflare Workers AI compat: HTTP 405
+                    // "GET not supported") would otherwise brick /model entirely
+                    // (live VPS 2026-09-10).
                     return (0, components_1.renderModal)({
                         title: `Model (${(0, factory_1.displayProviderName)(this.agent.config)})`,
                         body: [m.error, '', 'Esc to cancel'],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agen-vektor",
-  "version": "0.3.19",
+  "version": "0.3.21",
   "description": "VectorHead (agen-vektor) — AI Coding Agent CLI/TUI for Linux & Termux. Multi-provider, tool calling, session, permission system.",
   "type": "commonjs",
   "bin": {