@joshuafolkken/kit 1.1128.0 → 1.1136.0

package/dist/hooks/{chunk-M7QKEKWO.js → chunk-KBXZLMRH.js}

@@ -7,7 +7,6 @@ import {
   PORCELAIN_FLAG,
   RECENT_REQUEST_WINDOW,
   agent_role_profile,
-  agent_session_environment,
   bash_triggers,
   cli_flags,
   error_text,
@@ -24,6 +23,7 @@ import {
   lane_park,
   lane_registry,
   open_issue_schema,
+  own_session,
   parse_json_object_safe,
   read_json_listing,
   repo_origin,
@@ -35,31 +35,31 @@ import {
   run_tail,
   shell_segments,
   tail_commands
-} from "./chunk-AIOIOMPX.js";
+} from "./chunk-JKKTVDAG.js";
 import {
   find_local_bin_upwards
 } from "./chunk-3V37IID6.js";
 import {
   batch_outcome
-} from "./chunk-XYUIQOW3.js";
+} from "./chunk-KOKOAV6D.js";
 import {
   investigation_refusal
-} from "./chunk-7HVB7SZG.js";
+} from "./chunk-RFWWUI5F.js";
 import {
   duplicate_read_outcome
-} from "./chunk-5SD2YM7Z.js";
+} from "./chunk-HBKZLITF.js";
 import {
   investigation_reads
-} from "./chunk-NXYMNPUQ.js";
+} from "./chunk-LJBLGHEH.js";
 import {
   lane_guard_policy,
   time_batch_guard
-} from "./chunk-QVLT52OV.js";
+} from "./chunk-HANSKHES.js";
 import {
   hook_decision,
   lane_child_marker,
   run_issue_number
-} from "./chunk-MKVGBQPA.js";
+} from "./chunk-NFZGFGOR.js";
 import {
   PLATFORM_TEMP_ROOT,
   cost_attribute,
@@ -71,7 +71,7 @@ import {
   lane_paths,
   stamp_file,
   time_shell
-} from "./chunk-4JZDP5X5.js";
+} from "./chunk-GDU26K3J.js";
 
 // scripts/rules/direct-filing.ts
 var DIRECT_FILING_REASON = '\u26D4 direct filing: an Issue is filed with `pnpm josh issue:file "<title>" --body-file <path> --depth <0|1|2> [--route <route>] [--label <name>] [--repo <owner/repo>]`, never with `gh api \u2026/issues` or `gh issue create`. The command runs every filing step in order \u2014 the duplicate scout, the body lint and its classification labels, the depth and `route:` labels, the `## Origin` check when filing into another repository, and `epic:bundle` after the create \u2014 so none of them is a step to remember. When the scout prints candidates it holds the filing: read each, then reissue with `--distinct <N,\u2026>` for the ones that are separate deliverables. Reference: `docs/josh-commands-automation.md` \u2192 "`josh issue:file`". **This rule fires on every occurrence, not once per run.**';
@@ -893,25 +893,6 @@ var cost_verdict = {
   report_over
 };
 
-// scripts/cost-runtime/own-session.ts
-var NOT_FOUND = -1;
-function session_id_of(environment) {
-  const value = environment[agent_session_environment.SESSION_ID_KEY];
-  return value === void 0 || value === "" ? void 0 : value;
-}
-function select_own_session(files, environment) {
-  const session_id = session_id_of(environment);
-  if (session_id === void 0) {
-    return { kind: "own", index: cost_transcript.latest_own_index(files) };
-  }
-  const index = files.findIndex((file) => file.session_id === session_id);
-  return index === NOT_FOUND ? { kind: "absent", session_id } : { kind: "own", index };
-}
-var own_session = {
-  session_id_of,
-  select_own_session
-};
-
 // scripts/cost-runtime/transcript-cwd.ts
 import path3 from "node:path";
 function resolve(target, cwd) {

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

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

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

@@ -1,16 +1,16 @@
 import {
   time_batch_guard
-} from "./chunk-QVLT52OV.js";
+} from "./chunk-HANSKHES.js";
 import {
   hook_decision
-} from "./chunk-MKVGBQPA.js";
+} from "./chunk-NFZGFGOR.js";
 import {
   cost_blocks,
   cost_transcript,
   time_bundle_call,
   time_shell,
   time_spans
-} from "./chunk-4JZDP5X5.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-MKVGBQPA.js → chunk-NFZGFGOR.js}

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

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

@@ -1,12 +1,12 @@
 import {
   investigation_reads
-} from "./chunk-NXYMNPUQ.js";
+} from "./chunk-LJBLGHEH.js";
 import {
   lane_guard_policy
-} from "./chunk-QVLT52OV.js";
+} from "./chunk-HANSKHES.js";
 import {
   hook_decision
-} from "./chunk-MKVGBQPA.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,27 +1,27 @@
 #!/usr/bin/env tsx
 import {
   pretool_guard
-} from "./chunk-M7QKEKWO.js";
+} from "./chunk-KBXZLMRH.js";
 import {
   bounded_pool
-} from "./chunk-AIOIOMPX.js";
+} from "./chunk-JKKTVDAG.js";
 import {
   build_envelope,
   compose_context,
   format_edited_payload
-} from "./chunk-77J7IEQM.js";
+} from "./chunk-FO2I7UT7.js";
 import "./chunk-3V37IID6.js";
-import "./chunk-XYUIQOW3.js";
-import "./chunk-7HVB7SZG.js";
-import "./chunk-5SD2YM7Z.js";
-import "./chunk-NXYMNPUQ.js";
-import "./chunk-QVLT52OV.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-MKVGBQPA.js";
+} from "./chunk-NFZGFGOR.js";
 import {
   time_density_hook
-} from "./chunk-4JZDP5X5.js";
+} from "./chunk-GDU26K3J.js";
 import "./chunk-WPALEIRX.js";
 import "./chunk-RKUAD3I6.js";
 

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

@@ -9,11 +9,11 @@ import {
   is_enabled,
   notice_refusal_path,
   refusal_path
-} from "./chunk-5SD2YM7Z.js";
-import "./chunk-NXYMNPUQ.js";
-import "./chunk-QVLT52OV.js";
-import "./chunk-MKVGBQPA.js";
-import "./chunk-4JZDP5X5.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-77J7IEQM.js";
+} from "./chunk-FO2I7UT7.js";
 import "./chunk-3V37IID6.js";
-import "./chunk-4JZDP5X5.js";
+import "./chunk-GDU26K3J.js";
 import "./chunk-RKUAD3I6.js";
 
 // scripts/hooks/format-edited-cli.ts