@joshuafolkken/kit 1.1094.0 → 1.1136.0

package/dist/hooks/{chunk-IL27YTPH.js → chunk-KOKOAV6D.js}

@@ -1,10 +1,10 @@
 import {
   lane_guard_policy,
   time_batch_guard
-} from "./chunk-O4LL4HTI.js";
+} from "./chunk-HANSKHES.js";
 import {
   hook_decision
-} from "./chunk-E54KGFEQ.js";
+} from "./chunk-NFZGFGOR.js";
 
 // scripts/hooks/batch-guard.ts
 import { text } from "node:stream/consumers";

package/dist/hooks/{chunk-7ZPFAVR7.js → chunk-LJBLGHEH.js}

@@ -1,16 +1,16 @@
 import {
   time_batch_guard
-} from "./chunk-O4LL4HTI.js";
+} from "./chunk-HANSKHES.js";
 import {
   hook_decision
-} from "./chunk-E54KGFEQ.js";
+} from "./chunk-NFZGFGOR.js";
 import {
   cost_blocks,
   cost_transcript,
   time_bundle_call,
   time_shell,
   time_spans
-} from "./chunk-NKOG3XAV.js";
+} from "./chunk-GDU26K3J.js";
 
 // scripts/delegation/investigation-reads.ts
 import path from "node:path";
@@ -19,6 +19,7 @@ import path from "node:path";
 var DELEGATE_VERDICT = "delegate";
 var KEEP_VERDICT = "keep";
 var INVESTIGATION_FILE_THRESHOLD = 3;
+var INVESTIGATION_SEARCH_TURN_THRESHOLD = 3;
 var DELEGATABLE_STEPS = [
   {
     name: "gate-fix",
@@ -68,7 +69,7 @@ var DELEGATABLE_STEPS = [
     // verifiers differ, so widening `survey` would put both in one field and lose the boundary below.
     // **No ordinal.** `${N}rd` would print `1rd` for any other threshold, past a test that only looks
     // for the numeral, so the count is phrased as a plain "N or more".
-    does: `read the Issue's subject to find out how it currently works \u2014 once ${String(INVESTIGATION_FILE_THRESHOLD)} or more files the run will not edit have to be read \u2014 and return the conclusion plus the \`file:line\` citations that support it, never the file text, which would put the cost back where it was. A throwaway probe script is written, run and deleted inside the unit, which returns its output alone`,
+    does: `read the Issue's subject to find out how it currently works \u2014 once ${String(INVESTIGATION_FILE_THRESHOLD)} or more files the run will not edit have to be read, or ${String(INVESTIGATION_SEARCH_TURN_THRESHOLD)} or more search turns run without an edit \u2014 and return the conclusion plus the \`file:line\` citations that support it, never the file text, which would put the cost back where it was. A throwaway probe script is written, run and deleted inside the unit, which returns its output alone`,
     verifier: "the parent opens the cited lines; a conclusion those lines do not support fails there, and reading a handful of cited regions costs far less than redoing the reading. The unit reports what the code does and never what to change \u2014 deciding that is `diagnosis`, which stays kept, so a unit returning a root cause would be delegating the rejected row under this name"
   },
   {
@@ -131,6 +132,7 @@ function reason_for(name) {
 var delegation_policy = {
   DELEGATE_VERDICT,
   INVESTIGATION_FILE_THRESHOLD,
+  INVESTIGATION_SEARCH_TURN_THRESHOLD,
   KEEP_VERDICT,
   DELEGATABLE_STEPS,
   REJECTED_STEPS,
@@ -155,7 +157,12 @@ var CONTENT_READ_COMMANDS = [
 var CONTENT_READ_LABELS = new Set(
   CONTENT_READ_COMMANDS.map((command) => time_shell.bash_label(command))
 );
-var REASON = `\u26D4 investigation: ${String(delegation_policy.INVESTIGATION_FILE_THRESHOLD)} files read and not edited since the last delegated unit, so the reading from here goes to a unit of its own. Ask \`pnpm josh delegate investigation\`, then brief a unit with what the main line has already concluded and what is left to find out; it returns the conclusion plus its \`file:line\` citations, never the file text. Keep a read in the main line only where this run will edit that file \u2014 an \`Edit\` cannot be issued against text you do not hold. This run's own instructions and the harness's own session files \u2014 a backgrounded call's output, a persisted tool result, this session's scratchpad \u2014 are never counted, so they are not what took the count here. The rule is \`.claude/skills/workflow-commands/SKILL.md\` \u2192 \xA72b, "The pre-implementation reading".`;
+var SEARCH_COMMANDS = ["fd", "find", "grep", "rg"];
+var SEARCH_LABELS = new Set(
+  SEARCH_COMMANDS.map((command) => time_shell.bash_label(command))
+);
+var INVESTIGATOR_AGENT = "investigator";
+var REASON = `\u26D4 investigation: ${String(delegation_policy.INVESTIGATION_FILE_THRESHOLD)} files read and not edited, or ${String(delegation_policy.INVESTIGATION_SEARCH_TURN_THRESHOLD)} search turns, since the last delegated unit, so the reading from here goes to a unit of its own (a successful write also starts the search count over). Ask \`pnpm josh delegate investigation\`, then dispatch the \`${INVESTIGATOR_AGENT}\` agent (\`kit:${INVESTIGATOR_AGENT}\` in a consumer) with a brief of what the main line has already concluded and what is left to find out; it returns the conclusion plus its \`file:line\` citations, never the file text. Keep a read in the main line only where this run will edit that file \u2014 an \`Edit\` cannot be issued against text you do not hold. This run's own instructions and the harness's own session files \u2014 a backgrounded call's output, a persisted tool result, this session's scratchpad \u2014 never count toward the files, so they are not what took the file count here; every read-only search turn counts, whatever it names. The rule is \`.claude/skills/workflow-commands/SKILL.md\` \u2192 \xA72b, "The pre-implementation reading".`;
 var INSTRUCTION_PATHS = ["prompts", ".claude/skills"];
 var INSTRUCTION_FILES = /* @__PURE__ */ new Set(["CLAUDE.md", "AGENTS.md", "GEMINI.md"]);
 var REPOSITORY_ROOT = process.cwd();
@@ -219,14 +226,32 @@ function edited_files(spans) {
 function add_all(pending, targets, at_ms) {
   for (const target of targets) if (!pending.has(target)) pending.set(target, at_ms);
 }
-function apply_span(pending, span, edited) {
+function is_read_only_search(span) {
+  return SEARCH_LABELS.has(span.label) && span.is_bundleable && !span.may_write;
+}
+function search_turn_key(searches, span) {
+  if (span.message_id !== time_spans.NO_MESSAGE_ID) return span.message_id;
+  return `${time_spans.NO_MESSAGE_ID}#${String(searches.size)}`;
+}
+function apply_search(searches, span) {
+  if (span.is_writing) {
+    searches.clear();
+    return;
+  }
+  const turn = search_turn_key(searches, span);
+  if (is_read_only_search(span) && !searches.has(turn)) searches.set(turn, span.ended_ms);
+}
+function apply_span(accumulation, span, edited) {
   if (DELEGATION_TOOLS.has(span.label)) {
-    pending.clear();
+    accumulation.pending.clear();
+    accumulation.searches.clear();
     return;
   }
-  if (!is_content_read(span.label) || is_failed_outcome(span)) return;
+  if (is_failed_outcome(span)) return;
+  apply_search(accumulation.searches, span);
+  if (!is_content_read(span.label)) return;
   const fresh = subject_targets(span.targets).filter((target) => !edited.has(target));
-  add_all(pending, fresh, span.ended_ms);
+  add_all(accumulation.pending, fresh, span.ended_ms);
 }
 function last_delegation_ms(spans) {
   return delegation_instant(spans.findLast((span) => DELEGATION_TOOLS.has(span.label)));
@@ -237,13 +262,15 @@ function pending_after(pending, since_ms) {
 function tally_of(text, since_ms = hook_decision.NEVER_MS) {
   const { spans, started_ms } = time_spans.parse_timeline(text);
   const edited = edited_files(spans);
-  const pending = /* @__PURE__ */ new Map();
-  for (const span of spans) apply_span(pending, span, edited);
+  const accumulation = { pending: /* @__PURE__ */ new Map(), searches: /* @__PURE__ */ new Map() };
+  for (const span of spans) apply_span(accumulation, span, edited);
   return {
-    pending: [...pending.keys()],
-    pending_since: pending_after(pending, since_ms),
+    pending: [...accumulation.pending.keys()],
+    pending_since: pending_after(accumulation.pending, since_ms),
     reset_ms: last_delegation_ms(spans),
     window_start_ms: started_ms,
+    searches: accumulation.searches.size,
+    searches_since: pending_after(accumulation.searches, since_ms),
     edited
   };
 }
@@ -253,6 +280,9 @@ function projected_count(pending, call, edited = /* @__PURE__ */ new Set()) {
 function is_at_threshold(pending_count) {
   return pending_count + 1 >= delegation_policy.INVESTIGATION_FILE_THRESHOLD;
 }
+function is_at_search_threshold(search_count) {
+  return search_count + 1 >= delegation_policy.INVESTIGATION_SEARCH_TURN_THRESHOLD;
+}
 function has_cleared_the_disarm(tally, refused_at_ms) {
   return tally.reset_ms > refused_at_ms || refused_at_ms < tally.window_start_ms;
 }
@@ -260,7 +290,7 @@ function is_rearmed(tally, refused_at_ms) {
   if (refused_at_ms === hook_decision.NEVER_MS || has_cleared_the_disarm(tally, refused_at_ms)) {
     return true;
   }
-  return is_at_threshold(tally.pending_since);
+  return is_at_threshold(tally.pending_since) || is_at_search_threshold(tally.searches_since);
 }
 function is_unit_transcript(transcript) {
   return transcript.split(path.sep).includes(cost_transcript.UNIT_DIRECTORY);
@@ -268,22 +298,32 @@ function is_unit_transcript(transcript) {
 function is_unit_run(run) {
   return run !== void 0 && is_unit_transcript(run.transcript);
 }
-function is_refusable_command(call) {
+function is_read_only_command(call, labels) {
+  if (call.name !== cost_blocks.BASH_TOOL) return false;
   const label = time_shell.bash_label(time_shell.bash_command(call.input));
-  return CONTENT_READ_LABELS.has(label) && time_batch_guard.is_read_only_call(call);
+  return labels.has(label) && time_batch_guard.is_read_only_call(call);
+}
+function is_search_call(call) {
+  return is_read_only_command(call, SEARCH_LABELS);
 }
 function is_refusable_call(call) {
+  if (is_search_call(call)) return true;
   if (call_targets(call).length === 0) return false;
   if (READ_TOOLS.has(call.name)) return true;
-  return call.name === cost_blocks.BASH_TOOL && is_refusable_command(call);
+  return is_read_only_command(call, CONTENT_READ_LABELS);
+}
+function is_growing_past_threshold(tally, call) {
+  if (is_search_call(call)) return is_at_search_threshold(tally.searches);
+  if (!is_at_threshold(tally.pending.length)) return false;
+  return projected_count(tally.pending, call, tally.edited) > tally.pending.length;
 }
 function should_block(text, call, refused_at_ms, run) {
   if (!is_refusable_call(call) || is_unit_run(run)) return false;
   const tally = tally_of(text, refused_at_ms);
-  if (!is_at_threshold(tally.pending.length)) return false;
-  return projected_count(tally.pending, call, tally.edited) > tally.pending.length && is_rearmed(tally, refused_at_ms);
+  return is_growing_past_threshold(tally, call) && is_rearmed(tally, refused_at_ms);
 }
 var investigation_reads = {
+  INVESTIGATOR_AGENT,
   READ_TOOLS,
   REASON,
   is_at_threshold,

package/dist/hooks/{chunk-E54KGFEQ.js → chunk-NFZGFGOR.js}

@@ -3,7 +3,7 @@ import {
   stamp_file,
   time_density_hook,
   time_hook_transcript
-} from "./chunk-NKOG3XAV.js";
+} from "./chunk-GDU26K3J.js";
 import {
   josh_environment_file
 } from "./chunk-WPALEIRX.js";

package/dist/hooks/{chunk-KHWHKJOM.js → chunk-RFWWUI5F.js}

@@ -1,12 +1,12 @@
 import {
   investigation_reads
-} from "./chunk-7ZPFAVR7.js";
+} from "./chunk-LJBLGHEH.js";
 import {
   lane_guard_policy
-} from "./chunk-O4LL4HTI.js";
+} from "./chunk-HANSKHES.js";
 import {
   hook_decision
-} from "./chunk-E54KGFEQ.js";
+} from "./chunk-NFZGFGOR.js";
 
 // scripts/delegation/investigation-guard.ts
 import { text } from "node:stream/consumers";
@@ -25,7 +25,7 @@ function should_notify(tail, call, notified_at_ms, run) {
   if (investigation_mode() !== "notice") return false;
   return investigation_reads.should_block(tail, call, notified_at_ms, run);
 }
-var NOTICE = `\u{1F4A1} investigation: files read and not edited since the last delegated unit reached the threshold, so the reading from here should go to a unit of its own. This is a notice, not a refusal \u2014 the read proceeds, because a dispatched lane child ends its turn on a denial. Ask \`pnpm josh delegate investigation\`, then brief a unit with what the main line has already concluded and what is left to find out; it returns the conclusion plus its \`file:line\` citations, never the file text. Keep a read in the main line only where this run will edit that file. This recurs on the next accumulation.`;
+var NOTICE = `\u{1F4A1} investigation: files read and not edited, or search turns, since the last delegated unit reached the threshold, so the reading from here should go to a unit of its own. This is a notice, not a refusal \u2014 the read proceeds, because a dispatched lane child ends its turn on a denial. Ask \`pnpm josh delegate investigation\`, then dispatch the \`${investigation_reads.INVESTIGATOR_AGENT}\` agent (\`kit:${investigation_reads.INVESTIGATOR_AGENT}\` in a consumer) with a brief of what the main line has already concluded and what is left to find out; it returns the conclusion plus its \`file:line\` citations, never the file text. Keep a read in the main line only where this run will edit that file. This recurs on the next accumulation.`;
 function notice_text(_call, tail) {
   const pending = investigation_reads.tally_of(tail).pending.map((absolute) => investigation_reads.repository_relative(absolute));
   return pending.length === 0 ? NOTICE : `${NOTICE} Read and not edited so far: ${pending.join(" ")}.`;

package/dist/hooks/codex-hook-adapter.js

@@ -1,68 +1,33 @@
 #!/usr/bin/env tsx
 import {
   pretool_guard
-} from "./chunk-M2XDCUKD.js";
-import "./chunk-UD24EAIL.js";
+} from "./chunk-KBXZLMRH.js";
+import {
+  bounded_pool
+} from "./chunk-JKKTVDAG.js";
 import {
   build_envelope,
   compose_context,
   format_edited_payload
-} from "./chunk-OIZUHPJI.js";
+} from "./chunk-FO2I7UT7.js";
 import "./chunk-3V37IID6.js";
-import "./chunk-IL27YTPH.js";
-import "./chunk-KHWHKJOM.js";
-import "./chunk-3C4YCCEF.js";
-import "./chunk-7ZPFAVR7.js";
-import "./chunk-O4LL4HTI.js";
+import "./chunk-KOKOAV6D.js";
+import "./chunk-RFWWUI5F.js";
+import "./chunk-HBKZLITF.js";
+import "./chunk-LJBLGHEH.js";
+import "./chunk-HANSKHES.js";
 import {
   hook_decision
-} from "./chunk-E54KGFEQ.js";
+} from "./chunk-NFZGFGOR.js";
 import {
   time_density_hook
-} from "./chunk-NKOG3XAV.js";
+} from "./chunk-GDU26K3J.js";
 import "./chunk-WPALEIRX.js";
 import "./chunk-RKUAD3I6.js";
 
 // scripts/hooks/codex-hook-adapter.ts
 import { text } from "node:stream/consumers";
 import { fileURLToPath } from "node:url";
-
-// scripts/lib/bounded-pool.ts
-function pool_width(limit, total) {
-  if (!Number.isFinite(limit)) return 1;
-  return Math.max(1, Math.min(Math.floor(limit), total));
-}
-function throw_first_failure(failures) {
-  if (failures.length === 0) return;
-  const [first] = failures;
-  throw first;
-}
-async function drain(pool, worker) {
-  for (let entry = pool.queue.shift(); entry !== void 0 && pool.failures.length === 0; entry = pool.queue.shift()) {
-    const [index, item] = entry;
-    try {
-      pool.results.push({ index, value: await worker(item, index) });
-    } catch (error) {
-      pool.failures.push(error);
-    }
-  }
-}
-async function bounded_map(items, limit, worker) {
-  const pool = {
-    failures: [],
-    queue: items.map((item, index) => [index, item]),
-    results: []
-  };
-  const consumers = Array.from({ length: pool_width(limit, items.length) }, async () => {
-    await drain(pool, worker);
-  });
-  await Promise.all(consumers);
-  throw_first_failure(pool.failures);
-  return pool.results.toSorted((left, right) => left.index - right.index).map((result) => result.value);
-}
-var bounded_pool = { bounded_map, pool_width };
-
-// scripts/hooks/codex-hook-adapter.ts
 import { z } from "zod";
 var APPLY_PATCH_TOOL = "apply_patch";
 var EDIT_TOOL = "Edit";

package/dist/hooks/duplicate-read-guard.js

@@ -9,11 +9,11 @@ import {
   is_enabled,
   notice_refusal_path,
   refusal_path
-} from "./chunk-3C4YCCEF.js";
-import "./chunk-7ZPFAVR7.js";
-import "./chunk-O4LL4HTI.js";
-import "./chunk-E54KGFEQ.js";
-import "./chunk-NKOG3XAV.js";
+} from "./chunk-HBKZLITF.js";
+import "./chunk-LJBLGHEH.js";
+import "./chunk-HANSKHES.js";
+import "./chunk-NFZGFGOR.js";
+import "./chunk-GDU26K3J.js";
 import "./chunk-WPALEIRX.js";
 import "./chunk-RKUAD3I6.js";
 export {

package/dist/hooks/format-edited.js

@@ -2,9 +2,9 @@
 import {
   report_no_payload,
   run_hook
-} from "./chunk-OIZUHPJI.js";
+} from "./chunk-FO2I7UT7.js";
 import "./chunk-3V37IID6.js";
-import "./chunk-NKOG3XAV.js";
+import "./chunk-GDU26K3J.js";
 import "./chunk-RKUAD3I6.js";
 
 // scripts/hooks/format-edited-cli.ts