@aragon-agent/cli 0.5.0 → 0.5.1

package/dist/ui/overlays/QuestionOverlay.d.ts

@@ -0,0 +1,30 @@
+/**
+ * QuestionOverlay — the keyboard-driven multiple-choice wizard `ask_user`
+ * renders (plan-mode §6.5).
+ *
+ * `OverlayFrame` MODE B (self-managed): the content is a small state machine,
+ * not a list of rows, and slicing it by element would be meaningless.
+ *
+ * THE SINGLE HIGHEST-VALUE DECISION HERE is that the recommended option is the
+ * initial highlight for every question, so `Enter Enter Enter` is a correct,
+ * deliberate "use your judgement" path rather than an accident.
+ *
+ * `Esc` IS NOT HANDLED HERE. `App` owns it, and the bridge's `settle()` is the
+ * only place a human request is ever answered. `ConfirmDialog` does the
+ * opposite — both it and `App` call `resolve` — which is harmless there only by
+ * accident of promise semantics, and is not a pattern to copy (R-P12).
+ */
+import React from 'react';
+import type { Theme } from '../theme.js';
+import type { TermCapabilities } from '../capabilities.js';
+import type { Answer, NormalizedQuestion } from '../../tools/human-input.js';
+interface QuestionOverlayProps {
+    questions: NormalizedQuestion[];
+    maxRows: number;
+    cols: number;
+    theme: Theme;
+    caps: TermCapabilities;
+    onSubmit: (answers: Answer[]) => void;
+}
+export declare function QuestionOverlay({ questions, maxRows, cols, theme, caps, onSubmit, }: QuestionOverlayProps): React.ReactElement;
+export {};

package/dist/ui/overlays/QuestionOverlay.js

@@ -0,0 +1,193 @@
+import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
+/**
+ * QuestionOverlay — the keyboard-driven multiple-choice wizard `ask_user`
+ * renders (plan-mode §6.5).
+ *
+ * `OverlayFrame` MODE B (self-managed): the content is a small state machine,
+ * not a list of rows, and slicing it by element would be meaningless.
+ *
+ * THE SINGLE HIGHEST-VALUE DECISION HERE is that the recommended option is the
+ * initial highlight for every question, so `Enter Enter Enter` is a correct,
+ * deliberate "use your judgement" path rather than an accident.
+ *
+ * `Esc` IS NOT HANDLED HERE. `App` owns it, and the bridge's `settle()` is the
+ * only place a human request is ever answered. `ConfirmDialog` does the
+ * opposite — both it and `App` call `resolve` — which is harmless there only by
+ * accident of promise semantics, and is not a pattern to copy (R-P12).
+ */
+import { useState } from 'react';
+import { Box, Text, useInput } from 'ink';
+import { pickGlyphs } from '../glyphs.js';
+import { OverlayFrame } from '../layout/OverlayFrame.js';
+/** Which option index each question starts on: the recommended one. */
+function initialCursors(questions) {
+    return questions.map((q) => Math.max(0, q.options.findIndex((o) => o.recommended)));
+}
+function initialSelections(questions) {
+    return questions.map((q) => {
+        const recommended = q.options.findIndex((o) => o.recommended);
+        return new Set(recommended >= 0 ? [recommended] : [0]);
+    });
+}
+export function QuestionOverlay({ questions, maxRows, cols, theme, caps, onSubmit, }) {
+    const glyphs = pickGlyphs(caps);
+    const [step, setStep] = useState(0);
+    const [phase, setPhase] = useState('choosing');
+    const [cursors, setCursors] = useState(() => initialCursors(questions));
+    const [selections, setSelections] = useState(() => initialSelections(questions));
+    const [customs, setCustoms] = useState(() => questions.map(() => ''));
+    const [draft, setDraft] = useState('');
+    const question = questions[Math.min(step, questions.length - 1)];
+    const cursor = cursors[step] ?? 0;
+    const setCursorAt = (index, value) => setCursors((prev) => prev.map((c, i) => (i === index ? value : c)));
+    const setCustomAt = (index, value) => setCustoms((prev) => prev.map((c, i) => (i === index ? value : c)));
+    const toggleAt = (index, option) => setSelections((prev) => prev.map((set, i) => {
+        if (i !== index)
+            return set;
+        const next = new Set(set);
+        if (next.has(option))
+            next.delete(option);
+        else
+            next.add(option);
+        return next;
+    }));
+    const chooseAt = (index, option) => setSelections((prev) => prev.map((set, i) => (i === index ? new Set([option]) : set)));
+    const buildAnswers = () => questions.map((q, i) => {
+        const chosen = [...(selections[i] ?? new Set())].sort((a, b) => a - b);
+        const labels = chosen.map((index) => q.options[index]?.label ?? '').filter((l) => l.length > 0);
+        const pickedOther = chosen.some((index) => q.options[index]?.isOther);
+        return {
+            id: q.id,
+            question: q.question,
+            selected: labels,
+            custom: pickedOther && (customs[i] ?? '').length > 0 ? customs[i] : null,
+        };
+    });
+    const advance = () => {
+        if (step + 1 >= questions.length)
+            setPhase('review');
+        else {
+            setStep(step + 1);
+            setPhase('choosing');
+        }
+    };
+    const goBack = () => {
+        if (step === 0)
+            return; // No-op on the first question, by design.
+        setStep(step - 1);
+        setPhase('choosing');
+    };
+    useInput((input, key) => {
+        // App owns Esc for both new overlays (R-P12). Swallow it here so the
+        // free-text editor below cannot eat the raw escape byte as a character.
+        if (key.escape)
+            return;
+        if (phase === 'review') {
+            if (key.return) {
+                onSubmit(buildAnswers());
+                return;
+            }
+            if (key.leftArrow) {
+                setStep(questions.length - 1);
+                setPhase('choosing');
+            }
+            return;
+        }
+        if (phase === 'typingOther') {
+            if (key.return) {
+                // An empty commit falls back to the recommended option rather than
+                // recording a blank answer: the user pressed Enter, which everywhere
+                // else in this wizard means "accept the sensible default".
+                if (draft.trim().length === 0) {
+                    const recommended = Math.max(0, question.options.findIndex((o) => o.recommended));
+                    chooseAt(step, recommended);
+                    setCursorAt(step, recommended);
+                    setCustomAt(step, '');
+                }
+                else {
+                    setCustomAt(step, draft.trim());
+                }
+                setDraft('');
+                setPhase('choosing');
+                advance();
+                return;
+            }
+            if (key.leftArrow) {
+                setDraft('');
+                setPhase('choosing');
+                return;
+            }
+            if (key.backspace || key.delete) {
+                setDraft(draft.slice(0, -1));
+                return;
+            }
+            if (input && !key.ctrl && !key.meta && !key.tab)
+                setDraft(draft + input);
+            return;
+        }
+        // phase === 'choosing'
+        if (key.upArrow) {
+            setCursorAt(step, (cursor - 1 + question.options.length) % question.options.length);
+            return;
+        }
+        if (key.downArrow) {
+            setCursorAt(step, (cursor + 1) % question.options.length);
+            return;
+        }
+        if (key.leftArrow) {
+            goBack();
+            return;
+        }
+        if (input === ' ' && question.allowMultiple) {
+            toggleAt(step, cursor);
+            return;
+        }
+        if (key.return) {
+            if (question.options[cursor]?.isOther) {
+                chooseAt(step, cursor);
+                setDraft(customs[step] ?? '');
+                setPhase('typingOther');
+                return;
+            }
+            if (!question.allowMultiple)
+                chooseAt(step, cursor);
+            else if (!(selections[step] ?? new Set()).has(cursor))
+                toggleAt(step, cursor);
+            advance();
+        }
+    });
+    const title = phase === 'review' ? 'Review your answers' : 'Clarify the request';
+    const position = phase === 'review' ? '' : `  ${step + 1}/${questions.length}`;
+    const hint = phase === 'typingOther'
+        ? `type your answer ${glyphs.midDot} ${glyphs.enterKey} commit ${glyphs.midDot} ${glyphs.arrowLeft} back ${glyphs.midDot} esc cancel`
+        : phase === 'review'
+            ? `${glyphs.enterKey} submit ${glyphs.midDot} ${glyphs.arrowLeft} back ${glyphs.midDot} esc cancel`
+            : `${glyphs.arrowUp}${glyphs.arrowDown} choose ${glyphs.midDot} ${glyphs.enterKey} next${question.allowMultiple ? ` ${glyphs.midDot} space toggle` : ''} ${glyphs.midDot} ${glyphs.arrowLeft} back ${glyphs.midDot} esc cancel`;
+    return (_jsx(OverlayFrame, { title: `${title}${position}`, hint: hint, maxRows: maxRows, cols: cols, theme: theme, caps: caps, children: phase === 'review' ? (_jsx(ReviewBody, { questions: questions, answers: buildAnswers(), theme: theme })) : (_jsx(QuestionBody, { question: question, cursor: cursor, selected: selections[step] ?? new Set(), typing: phase === 'typingOther', draft: draft, theme: theme, caps: caps })) }));
+}
+/**
+ * ONE QUESTION PER SCREEN, not all of them at once. Three questions x four
+ * options x a description line is 20+ rows against a ~15-row viewport on a
+ * 24-row terminal.
+ */
+function QuestionBody({ question, cursor, selected, typing, draft, theme, caps, }) {
+    const glyphs = pickGlyphs(caps);
+    const markOn = question.allowMultiple ? glyphs.boxChecked : glyphs.keyOn;
+    const markOff = question.allowMultiple ? glyphs.boxEmpty : glyphs.keyOff;
+    return (_jsxs(Box, { flexDirection: "column", children: [_jsx(Text, { wrap: "truncate", color: theme.accent, bold: true, children: question.header }), _jsx(Text, { wrap: "truncate", color: theme.assistant, children: question.question }), _jsx(Text, { children: " " }), question.options.map((option, i) => {
+                const active = i === cursor;
+                const on = selected.has(i);
+                return (_jsxs(Text, { wrap: "truncate", children: [_jsxs(Text, { color: active ? theme.primary : theme.muted, children: [active ? glyphs.caret : ' ', ' '] }), _jsxs(Text, { color: on ? theme.primary : theme.muted, children: [on ? markOn : markOff, " "] }), _jsx(Text, { color: active ? theme.assistant : undefined, bold: active, children: option.isOther ? `${option.label}${glyphs.ellipsis}` : option.label }), option.description.length > 0 && (_jsxs(Text, { color: theme.muted, children: ['  ', option.description] })), option.recommended && (_jsxs(Text, { backgroundColor: theme.chip.bg, color: theme.chip.fg, children: [' ', "RECOMMENDED", ' '] }))] }, `${question.id}-${i}`));
+            }), typing && (_jsxs(_Fragment, { children: [_jsx(Text, { children: " " }), _jsxs(Text, { wrap: "truncate", children: [_jsxs(Text, { color: theme.primary, bold: true, children: [glyphs.caret, ' '] }), draft.length > 0 ? (_jsx(Text, { children: draft })) : (_jsx(Text, { color: theme.muted, children: "type your own answer" })), _jsx(Text, { inverse: true, children: " " })] })] }))] }));
+}
+/**
+ * The review step exists because a wizard that fires on the last `Enter` with no
+ * confirmation makes a mis-keyed answer unrecoverable.
+ */
+function ReviewBody({ questions, answers, theme, }) {
+    return (_jsx(Box, { flexDirection: "column", children: questions.map((q, i) => {
+            const answer = answers[i];
+            const value = answer?.custom ?? answer?.selected.join(', ') ?? '';
+            return (_jsxs(Text, { wrap: "truncate", children: [_jsx(Text, { color: theme.accent, children: q.header.padEnd(14) }), _jsx(Text, { color: theme.assistant, children: value })] }, q.id));
+        }) }));
+}

package/dist/ui/overlays/SettingsScreen.d.ts

@@ -9,6 +9,7 @@
  */
 import React from 'react';
 import type { ThinkingLevel } from '@aragon-agent/core';
+import { type LogLevelName } from '../../logging/levels.js';
 import type { Theme } from '../theme.js';
 import type { TermCapabilities } from '../capabilities.js';
 export interface SettingsValues {
@@ -18,6 +19,12 @@ export interface SettingsValues {
     thinkingLevel: ThinkingLevel;
     maxTokens: string;
     apiKey: string;
+    /**
+     * Only the LEVEL is editable here. The rest of `LogConfig` — and above all
+     * `redactSecrets` — stays command-line-only, so the one switch that can put
+     * credentials on disk cannot be flipped by a stray arrow key.
+     */
+    logLevel: LogLevelName;
 }
 interface SettingsScreenProps {
     initial: SettingsValues;

package/dist/ui/overlays/SettingsScreen.js

@@ -11,6 +11,7 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { useState } from 'react';
 import { Text, useInput } from 'ink';
 import { ADAPTER_PROVIDERS, THINKING_LEVELS, maskSecret } from '../../config/schema.js';
+import { LOG_LEVEL_NAMES } from '../../logging/levels.js';
 import { pickGlyphs } from '../glyphs.js';
 import { OverlayFrame } from '../layout/OverlayFrame.js';
 const FIELDS = [
@@ -20,6 +21,7 @@ const FIELDS = [
     { key: 'thinkingLevel', label: 'Thinking', kind: 'enum', options: THINKING_LEVELS },
     { key: 'maxTokens', label: 'Max tokens', kind: 'text' },
     { key: 'apiKey', label: 'API key', kind: 'secret' },
+    { key: 'logLevel', label: 'Log level', kind: 'enum', options: LOG_LEVEL_NAMES },
 ];
 export function SettingsScreen({ initial, apiKeys, maxRows, cols, scrollOffset, onScrollClamp, theme, caps, onSave, }) {
     const [values, setValues] = useState(initial);
@@ -51,7 +53,12 @@ export function SettingsScreen({ initial, apiKeys, maxRows, cols, scrollOffset,
             setIndex((i) => (i - 1 + FIELDS.length) % FIELDS.length);
             return;
         }
-        if (key.downArrow || key.tab) {
+        // `&& !key.shift` is a real key collision, not a style choice: Shift+Tab is
+        // the global mode toggle, and without the guard it would also advance the
+        // field cursor here — the user changes mode and their selection jumps
+        // (AC-P4). The mode toggle itself is a no-op while an overlay is open, so
+        // this branch simply has to stay out of the way.
+        if (key.downArrow || (key.tab && !key.shift)) {
             setIndex((i) => (i + 1) % FIELDS.length);
             return;
         }

package/dist/ui/use-startup-notices.d.ts

@@ -0,0 +1,15 @@
+/**
+ * The two things the user needs to be told about that happened BEFORE the frame
+ * existed, and the one that can happen at any moment afterwards.
+ *
+ * Both arrive as toasts, and that is not a style choice: with the TUI mounted, a
+ * toast is the only legal user-visible channel. Any direct write to stdout or
+ * stderr shifts the fixed frame by a row and breaks its line accounting for good
+ * (invariant I-4).
+ *
+ * Lives outside `App.tsx` because that file is already at its size ceiling, and
+ * because these two effects share one job — surfacing a condition the user could
+ * not otherwise see — rather than belonging to the view.
+ */
+import type { ViewAction } from '../agent/reducer.js';
+export declare function useStartupNotices(dispatch: (action: ViewAction) => void): void;

package/dist/ui/use-startup-notices.js

@@ -0,0 +1,38 @@
+/**
+ * The two things the user needs to be told about that happened BEFORE the frame
+ * existed, and the one that can happen at any moment afterwards.
+ *
+ * Both arrive as toasts, and that is not a style choice: with the TUI mounted, a
+ * toast is the only legal user-visible channel. Any direct write to stdout or
+ * stderr shifts the fixed frame by a row and breaks its line accounting for good
+ * (invariant I-4).
+ *
+ * Lives outside `App.tsx` because that file is already at its size ceiling, and
+ * because these two effects share one job — surfacing a condition the user could
+ * not otherwise see — rather than belonging to the view.
+ */
+import { useEffect } from 'react';
+import { readConfigFile } from '../config/store.js';
+import { getLogger } from '../logging/logger.js';
+export function useStartupNotices(dispatch) {
+    // The sink gives up after repeated write failures (a read-only log directory,
+    // a full disk). Silently logging nothing is exactly the state a user must not
+    // discover later, from an empty file, while trying to explain a bug.
+    useEffect(() => {
+        getLogger().onFailure((reason) => dispatch({ type: 'pushToast', level: 'warn', text: `Logging stopped: ${reason}` }));
+    }, [dispatch]);
+    // `main()` already wrote this to stderr, but entering the alternate screen
+    // wipes it — so full-screen users would be left with the very failure the
+    // message exists to prevent: every setting back at its default, and nothing
+    // anywhere connecting that to the comma they typed into config.json.
+    useEffect(() => {
+        const { parseError } = readConfigFile();
+        if (!parseError)
+            return;
+        dispatch({
+            type: 'pushToast',
+            level: 'warn',
+            text: `Using default settings: could not parse ${parseError}`,
+        });
+    }, [dispatch]);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aragon-agent/cli",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "AragonAgent CLI — a Claude-Code / Codex-style interactive terminal UI (TUI) on top of the @aragon-agent/core engine. Installable and runnable via npm / npx.",
   "license": "MIT",
   "author": "AragonAgent contributors",
@@ -32,6 +32,7 @@
   },
   "files": [
     "dist",
+    "runtime",
     "skills",
     "README.md",
     "CHANGELOG.md"
@@ -50,7 +51,7 @@
     "prepublishOnly": "npm run build"
   },
   "dependencies": {
-    "@aragon-agent/core": "^0.2.0",
+    "@aragon-agent/core": "^0.2.1",
     "chalk": "^5.6.2",
     "cli-highlight": "^2.0.0",
     "commander": "^12.1.0",

package/runtime/insecure-tls-warning.cjs

@@ -0,0 +1,62 @@
+'use strict';
+
+const INSECURE_TLS_WARNING =
+  "Setting the NODE_TLS_REJECT_UNAUTHORIZED environment variable to '0' " +
+  'makes TLS connections and HTTPS requests insecure by disabling ' +
+  'certificate verification.';
+
+const FILTER_STATE_KEY = Symbol.for('@aragon-agent/cli.insecure-tls-warning-filter');
+
+function installInsecureTlsWarningFilter(target = process) {
+  const existingState = target[FILTER_STATE_KEY];
+  if (existingState) return existingState.restore;
+
+  const originalEmitWarning = target.emitWarning;
+  let installedState;
+
+  function wrappedEmitWarning(warning) {
+    const message =
+      typeof warning === 'string'
+        ? warning
+        : warning instanceof Error
+          ? warning.message
+          : undefined;
+
+    if (
+      target.env.NODE_TLS_REJECT_UNAUTHORIZED === '0' &&
+      message === INSECURE_TLS_WARNING
+    ) {
+      return;
+    }
+
+    return Reflect.apply(originalEmitWarning, target, arguments);
+  }
+
+  function restore() {
+    if (
+      target[FILTER_STATE_KEY] !== installedState ||
+      target.emitWarning !== wrappedEmitWarning
+    ) {
+      return;
+    }
+
+    target.emitWarning = originalEmitWarning;
+    delete target[FILTER_STATE_KEY];
+  }
+
+  installedState = {
+    originalEmitWarning,
+    wrappedEmitWarning,
+    restore,
+  };
+  target[FILTER_STATE_KEY] = installedState;
+  target.emitWarning = wrappedEmitWarning;
+  return restore;
+}
+
+module.exports = {
+  INSECURE_TLS_WARNING,
+  installInsecureTlsWarningFilter,
+};
+
+installInsecureTlsWarningFilter();

package/runtime/insecure-tls-warning.d.cts

@@ -0,0 +1,5 @@
+export const INSECURE_TLS_WARNING: string;
+
+export function installInsecureTlsWarningFilter(
+  target?: NodeJS.Process,
+): () => void;