teapot-coding-agent 0.37.48 → 0.37.50

package/dist/agent/agent.js

@@ -74,13 +74,6 @@ export const PROGRESS_REQUEST =
     // and every requested report appeared TWICE on the timeline
     "Record it with the report_progress TOOL (doing, goalStatus, recent, problems, next) — " +
     "call that tool; do NOT answer in prose, which would just duplicate this request.";
-/**
- * SYSTEM_TEMPLATE must stay byte-identical across every request of a session:
- * provider prefix caches key on it, so changing it (or injecting per-turn
- * state) re-prices the whole context. That is why session state lives behind
- * meta tools instead. The cache rationale itself stays HERE in a comment —
- * the model does not need our cost-engineering notes every turn.
- */
 /**
  * Recent context kept VERBATIM through a compaction, in tokens.
  *
@@ -235,6 +228,13 @@ const SUMMARY_CLOSE = "\n</conversation_checkpoint>";
  * claim, and must not be able to change what it is judging.
  */
 const AUDIT_READ_ONLY_TOOLS = new Set(["read_file", "list_dir", "grep", "bash"]);
+/**
+ * SYSTEM_TEMPLATE must stay byte-identical across every request of a session:
+ * provider prefix caches key on it, so changing it (or injecting per-turn
+ * state) re-prices the whole context. That is why session state lives behind
+ * meta tools instead. The cache rationale itself stays HERE in a comment —
+ * the model does not need our cost-engineering notes every turn.
+ */
 const SYSTEM_TEMPLATE = `You are a coding agent working autonomously inside a workspace.
 
 Most session state is not injected into prompts — fetch it with tools instead:
@@ -665,31 +665,14 @@ export class Agent {
      * are exempt — those are not "keep trying", they are "the operator/provider asked".
      */
     consecutiveCompactFailures = 0;
-    /**
-     * Rescan skill roots — but only when something actually changed.
-     * This used to run a readdir per root plus a read of every SKILL.md at the
-     * top of EVERY turn; the cache is only ever read by list_skills, so that
-     * was pure I/O on the hot path. A cheap stat fingerprint of the roots and
-     * their SKILL.md files gates the real scan, so a turn only pays when a
-     * skill was added, edited or removed (save_skill writes one, so a
-     * self-created skill still shows up on the very next turn).
-     */
-    /**
-     * Render the skill catalogue for the system prompt.
-     *
-     * Name + description only, one line each — the full SKILL.md is what
-     * load_skill() is for, and this block goes into EVERY request of the session,
-     * so it has to stay small. The description is what the model matches its task
-     * against, so a skill with a poor description is effectively undiscoverable;
-     * that is worth knowing when authoring one.
-     */
     /**
      * A bounded DIGEST of memory.md: every heading, plus its first line.
      *
-     * they are genuinely long — 2.6KB is the median, and the largest hold full project briefs. A
-     * model that never sees them cannot decide to consult them: that is the whole finding of #48
-     * (182 writes, 13 reads after a write). A model that sees the WHOLE file pays for it on every
-     * request and breaks the prefix cache.
+     * Memory notes are worth this much because they are what a later session reads instead of
+     * re-learning the workspace: they are genuinely long — 2.6KB is the median, and the largest hold
+     * full project briefs. A model that never sees them cannot decide to consult them: that is the
+     * whole finding of #48 (182 writes, 13 reads after a write). A model that sees the WHOLE file
+     * pays for it on every request and breaks the prefix cache.
      *
      * Codex solves this with a `MEMORY_SUMMARY` in the prompt plus a searchable registry on disk.
      * This is the same shape: the digest says what memory HOLDS, `read_memory(query)` fetches the
@@ -977,6 +960,11 @@ export class Agent {
             this.parkedByTool ||
             this.compactPhase !== "");
     }
+    /** recently read workspace files (most recent first) — post-compact restore */
+    recentReads = [];
+    trackFileRead(p) {
+        this.recentReads = [p, ...this.recentReads.filter((x) => x !== p)].slice(0, 5);
+    }
     /**
      * A background shell (bash background=true) finished. Two channels:
      * 1. a system_note row in the timeline — the operator sees the outcome
@@ -985,11 +973,6 @@ export class Agent {
      *    agent learns exit code + output tail WITHOUT having to poll
      *    bash_output blindly. Failed jobs say so loudly so the model reacts.
      */
-    /** recently read workspace files (most recent first) — post-compact restore */
-    recentReads = [];
-    trackFileRead(p) {
-        this.recentReads = [p, ...this.recentReads.filter((x) => x !== p)].slice(0, 5);
-    }
     onBackgroundJobExit(info) {
         const ok = info.code === 0;
         const one = info.cmd.replace(/\s+/g, " ").slice(0, 80);
@@ -4592,7 +4575,6 @@ export class Agent {
             bus.emit("update", { kind: "compaction-progress", agentId: this.opts.id, phase: "done" });
         }
     }
-    /** Ask the model for dense continuation notes over the compacted range. */
     /**
      * Summarize `old` into continuation notes.
      *

package/dist/master.js

@@ -19,7 +19,6 @@ import { bus } from "./bus.js";
 import { mcp } from "./agent/mcp-registry.js";
 import { loadPlugins } from "./agent/plugins.js";
 import { fetchModelList, peekModelList, contextLengthFor, isReasoningEffort, } from "./model-meta.js";
-/** how deep sub-agent spawning may nest (parent=0, its subs=1, …) */
 /**
  * Did the effective data dir come from an ENV OVERRIDE that disagrees with the
  * config file's own `dataDir`?
@@ -282,7 +281,6 @@ export function loadConfig(configPath) {
         providers: Object.fromEntries(Object.entries(user.providers ?? {}).map(([name, p]) => [name, migrateProviderKeys(p)])),
     };
 }
-/** Raw parsed user config (for lossless persistence of web edits). */
 /**
  * Can this path be replaced by an atomic rename?
  *
@@ -587,7 +585,6 @@ export class Master {
         this.configDirty = false;
         this.saveConfig();
     }
-    /** write-and-rename, so config.json is never observed empty or half-written */
     /**
      * The shared pool for a provider, created on first use.
      *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "teapot-coding-agent",
-  "version": "0.37.48",
+  "version": "0.37.50",
   "description": "A lightweight, always-on multi-agent harness for AI coding agents",
   "type": "module",
   "license": "AGPL-3.0-or-later",
@@ -37,6 +37,8 @@
     "typecheck-web": "tsc -p tsconfig.frontend.json",
     "typecheck:scripts": "tsc -p tsconfig.scripts.json",
     "lint": "oxlint src frontend test scripts",
+    "lint:comments": "node scripts/find-stacked-comments.ts",
+    "lint:comments:strict": "node scripts/find-stacked-comments.ts --strict",
     "start": "node dist/index.js",
     "dev": "node src/index.ts",
     "dev-web": "vite",