refactor(agent): compress verbose comments in the tool classification module

Author: Gustav-Simonsson

src/agent/tool-classification.test.ts

@@ -5,8 +5,8 @@ import {
   SEARCH_QUERY_TOOLS,
 } from "./tool-classification.js";
 
-// Pins membership so a future edit to these sets or their base READ_TOOLS
-// fails CI instead of silently drifting call sites out of sync.
+// Pins membership so edits to these sets or READ_TOOLS fail CI instead of
+// drifting call sites out of sync.
 describe("AUTO_ALLOW_READ_TOOLS", () => {
   test("gates auto-allow with exactly this membership", () => {
     expect([...AUTO_ALLOW_READ_TOOLS].sort()).toEqual(

src/agent/tool-classification.ts

@@ -1,31 +1,27 @@
 /**
  * Shared tool-name classification constants. One name (`READ_TOOLS`) used to
  * mean different memberships in four places, one of them security-relevant,
- * so the concepts are now separate sets derived from one base:
- *   - the director read surface (tool-sets.ts READ_TOOLS): everything a
- *     read-only leaf may call, including run_shell and the web tools;
- *   - the auto-allow gate: never needs an approval prompt — a strict subset
- *     (shell/web keep their own narrower rules) plus manage_tasks;
- *   - compaction's re-read dedup and thrash's read tracking: both key off
- *     read_file's path-keyed result, unified here.
- * Sets stay separate where the concepts differ, so a future difference reads
- * as intentional instead of drift.
+ * so the concepts are now separate sets derived from one base: the director
+ * read surface (tool-sets.ts READ_TOOLS), the auto-allow gate, and the
+ * path-keyed / query-keyed sets behind compaction's dedup and thrash's
+ * read tracking. Sets stay separate where the concepts differ, so a future
+ * difference reads as intentional instead of drift.
  */
 
 import { READ_TOOLS as DIRECTOR_READ_TOOLS } from "./directors/tool-sets.js";
 
 /**
- * The one read tool whose result is keyed by path: a newer read supersedes
- * an older one for the same path. Shared by compaction's re-read dedup and
- * thrash's read-count bookkeeping.
+ * The read tool whose result is keyed by path: a newer read supersedes an
+ * older one for the same path. Shared by compaction's re-read dedup and
+ * thrash's read tracking.
  */
 export const PATH_KEYED_READ_TOOLS: ReadonlySet<string> = new Set([
   "read_file",
 ]);
 
 /**
  * grep / search_files: pattern-keyed query tools — a repeated identical call
- * reflects current workspace state, not stale history. Base for compaction
+ * reflects the current workspace, not stale history. Base for compaction
  * and thrash; each adds/omits list_dir for its own reason.
  */
 export const SEARCH_QUERY_TOOLS: ReadonlySet<string> = new Set([