@aragon-agent/cli 0.6.18 → 0.6.20

package/CHANGELOG.md

@@ -7,6 +7,20 @@ were written.
 
 ## Unreleased
 
+### Fixed
+
+- **`Ctrl+I` was dead on Windows.** The win32-input-mode translator only
+  recognised the chord when the console reported it as `VK_TAB` (vk 9) with
+  ctrl - a record shape no real console was ever probed for. Physical
+  Ctrl+I keeps the LETTER vk (0x49, the same Ctrl+letter rule that reports
+  Ctrl+A as vk 65 and Ctrl+J as vk 74 in the Shift+Enter probe captures)
+  while `char` carries the Tab byte, so the record fell through to the
+  generic Ctrl-combo branch, came out as plain Tab, and the keybinding
+  silently did completion instead of opening the index confirm. The
+  translator now tells Ctrl+I apart from Tab by `char === 0x09 && ctrl`
+  whatever vk it rides, with Shift still mapping to Shift+Tab; both record
+  shapes are pinned by regression rows in `win32-input-mode.test.ts`.
+
 ### Team dispatch supervisor (team-overseer)
 
 - **A time ceiling on a subagent is now an inspection trigger, not a death

package/dist/compaction/meter.d.ts

@@ -103,6 +103,8 @@ export declare class ContextMeter {
     private last;
     /** Whether the history has changed since the last measurement. */
     private dirty;
+    /** Whether the one-per-run empty-measurement warning has been spent. */
+    private hasWarnedUnmeasuredInput;
     private prefixLast;
     private sampledPrompt;
     private sampledVersion;
@@ -206,6 +208,16 @@ export declare class ContextMeter {
      */
     attach(subscribe: (listener: (event: AgentEvent) => void) => () => void): void;
     dispose(): void;
+    /**
+     * ONE warn per run (context-usage-zero-input-tokens B). A single day of
+     * real usage logged 1,973 zero-input turns; a per-turn warning is a flood
+     * that teaches whoever reads the file to ignore it.
+     *
+     * The logger is resolved LAZILY rather than captured at construction:
+     * hosts build this class before `installLogging()`, and a child captured
+     * then would stay bound to the silent pre-install logger forever.
+     */
+    private warnUnmeasuredInputOnce;
     private onAgentEvent;
     /**
      * The structural belt for a missed invalidation site (I-9).

package/dist/compaction/meter.js

@@ -27,7 +27,8 @@
  * `CompactionWiring.settlePending`'s own synchronous code (I-9), and why
  * `current()` carries a structural belt for a missed invalidation site.
  */
-import { computeEstimateOffset, computePressure } from './pressure.js';
+import { getLogger } from '../logging/logger.js';
+import { computeEstimateOffset, computePressure, isInputSideUnmeasured } from './pressure.js';
 /**
  * How long a scheduled publish waits.
  *
@@ -103,6 +104,8 @@ export class ContextMeter {
     last = null;
     /** Whether the history has changed since the last measurement. */
     dirty = true;
+    /** Whether the one-per-run empty-measurement warning has been spent. */
+    hasWarnedUnmeasuredInput = false;
     prefixLast;
     sampledPrompt;
     sampledVersion;
@@ -174,6 +177,21 @@ export class ContextMeter {
      * better number to wait for.
      */
     onTurnEnd(usage) {
+        if (isInputSideUnmeasured(usage)) {
+            // A usage that discloses NOTHING on the input side is not a
+            // measurement of this history (context-usage-zero-input-tokens B):
+            // arming the measured branch with it collapses occupancy to the
+            // turn's own outputTokens and strips the `~` off the gauge. KEEP
+            // WHATEVER IS STILL TRUSTED - an earlier good measurement of a
+            // prefix that still exists (the history only grows between splices,
+            // so "old good base + appended estimate" stays self-consistent), and
+            // the estimate offset it calibrated - and re-measure NOW so the
+            // gauge degrades in this frame instead of after the next turn.
+            this.warnUnmeasuredInputOnce(usage);
+            this.dirty = true;
+            this.current();
+            return;
+        }
         const messages = this.deps.getMessages();
         const systemPrompt = this.readSystemPrompt();
         this.lastUsage = usage;
@@ -307,6 +325,21 @@ export class ContextMeter {
     // =========================================================================
     // Internals
     // =========================================================================
+    /**
+     * ONE warn per run (context-usage-zero-input-tokens B). A single day of
+     * real usage logged 1,973 zero-input turns; a per-turn warning is a flood
+     * that teaches whoever reads the file to ignore it.
+     *
+     * The logger is resolved LAZILY rather than captured at construction:
+     * hosts build this class before `installLogging()`, and a child captured
+     * then would stay bound to the silent pre-install logger forever.
+     */
+    warnUnmeasuredInputOnce(usage) {
+        if (this.hasWarnedUnmeasuredInput)
+            return;
+        this.hasWarnedUnmeasuredInput = true;
+        getLogger().warn('compaction', 'context_input_tokens_unreported', { in: usage.inputTokens, out: usage.outputTokens });
+    }
     onAgentEvent(event) {
         switch (event.type) {
             case 'turn_end':

package/dist/compaction/pressure.d.ts

@@ -40,6 +40,35 @@ import type { Pressure } from './types.js';
  * the feature is broken, and turns it off.
  */
 export declare function occupiedTokens(usage: TokenUsage): number;
+/**
+ * Whether a usage measured ANYTHING on the input side
+ * (context-usage-zero-input-tokens B).
+ *
+ * THE EMPTY MEASUREMENT THIS EXISTS FOR. A turn that billed output tokens
+ * while reporting inputTokens 0 AND no cache terms did not measure the
+ * request - no real request is free of input. Anthropic-COMPATIBLE relays
+ * emit exactly this shape (bigmodel on every turn, Kimi intermittently),
+ * and the meter used to arm its measured branch with it: occupancy collapsed
+ * to the turn's own outputTokens, the `~` honesty marker vanished with
+ * `source: 'usage'`, and the trigger - which reads the same number - never
+ * fired. The defense is to not trust such a usage as a measurement.
+ *
+ * THE CACHE TERMS ARE PART OF THE INPUT SIDE, DELIBERATELY. Anthropic's real
+ * `input_tokens` EXCLUDES cached tokens, so a deeply cached turn reports a
+ * tiny `input_tokens` next to a huge `cache_read_input_tokens`. That IS a
+ * valid measurement (C-14), and judging on `inputTokens === 0` alone would
+ * reject it and strand a correct gauge on the estimator.
+ *
+ * `outputTokens > 0` separates "the stream ran, billed output, and disclosed
+ * no input" from a degenerate all-zero usage, which the reviewed predicate
+ * deliberately does not speak for.
+ *
+ * SINGLE AUTHORITY. The meter consumes this, and any future observability
+ * hint (fix C) must too: two inline copies of "did the gateway disclose
+ * input" would drift apart exactly the way the gauge and the trigger once
+ * could (R-11).
+ */
+export declare function isInputSideUnmeasured(usage: TokenUsage): boolean;
 /**
  * The calibration offset for this turn (D-23 / P1-11).
  *

package/dist/compaction/pressure.js

@@ -44,6 +44,38 @@ export function occupiedTokens(usage) {
         (usage.cacheWriteTokens ?? 0) +
         usage.outputTokens);
 }
+/**
+ * Whether a usage measured ANYTHING on the input side
+ * (context-usage-zero-input-tokens B).
+ *
+ * THE EMPTY MEASUREMENT THIS EXISTS FOR. A turn that billed output tokens
+ * while reporting inputTokens 0 AND no cache terms did not measure the
+ * request - no real request is free of input. Anthropic-COMPATIBLE relays
+ * emit exactly this shape (bigmodel on every turn, Kimi intermittently),
+ * and the meter used to arm its measured branch with it: occupancy collapsed
+ * to the turn's own outputTokens, the `~` honesty marker vanished with
+ * `source: 'usage'`, and the trigger - which reads the same number - never
+ * fired. The defense is to not trust such a usage as a measurement.
+ *
+ * THE CACHE TERMS ARE PART OF THE INPUT SIDE, DELIBERATELY. Anthropic's real
+ * `input_tokens` EXCLUDES cached tokens, so a deeply cached turn reports a
+ * tiny `input_tokens` next to a huge `cache_read_input_tokens`. That IS a
+ * valid measurement (C-14), and judging on `inputTokens === 0` alone would
+ * reject it and strand a correct gauge on the estimator.
+ *
+ * `outputTokens > 0` separates "the stream ran, billed output, and disclosed
+ * no input" from a degenerate all-zero usage, which the reviewed predicate
+ * deliberately does not speak for.
+ *
+ * SINGLE AUTHORITY. The meter consumes this, and any future observability
+ * hint (fix C) must too: two inline copies of "did the gateway disclose
+ * input" would drift apart exactly the way the gauge and the trigger once
+ * could (R-11).
+ */
+export function isInputSideUnmeasured(usage) {
+    const inputSide = usage.inputTokens + (usage.cacheReadTokens ?? 0) + (usage.cacheWriteTokens ?? 0);
+    return inputSide === 0 && usage.outputTokens > 0;
+}
 /**
  * The calibration offset for this turn (D-23 / P1-11).
  *

package/dist/input/win32-input-mode.js

@@ -216,6 +216,18 @@ export function translateWin32Key(record) {
             return char;
         if (!ctrl && alt)
             return `\x1b${char}`;
+        // Ctrl+I, letter-vk shape: a physical Ctrl+I keeps the LETTER vk
+        // (0x49, the same Ctrl+letter probe rule that reports Ctrl+A as 65
+        // and Ctrl+J as 74) while `char` is already the Tab byte, so the
+        // VK_TAB switch above never sees the record. 0x09 is both the Tab
+        // byte and VK_TAB, so the tell-apart from plain Tab is ctrl alone,
+        // and Shift keeps ESC [ Z exactly like that switch: the mode toggle
+        // stays intact and Ctrl+Shift+I stays on the Shift branch.
+        if (ctrl && record.char === VK_TAB) {
+            if (!shift && !alt)
+                return INDEX_KEY_FRAME;
+            return shift ? '\x1b[Z' : '\t';
+        }
         return ctrl ? `${alt ? '\x1b' : ''}${char}` : char;
     }
     if (isPrintable(record.char)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aragon-agent/cli",
-  "version": "0.6.18",
+  "version": "0.6.20",
   "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",
@@ -56,7 +56,7 @@
     "prepublishOnly": "npm run build"
   },
   "dependencies": {
-    "@aragon-agent/core": "^0.2.30",
+    "@aragon-agent/core": "^0.2.32",
     "chalk": "^5.6.2",
     "cli-highlight": "^2.0.0",
     "commander": "^12.1.0",