@joshuafolkken/kit 1.1455.0 → 1.1514.0

package/scripts/version/kit-version-config.ts

@@ -13,8 +13,14 @@ const SELF_DIRECTORY = path.dirname(fileURLToPath(import.meta.url))
 function read_shadow_warning(): string | undefined {
 	const path_josh = doctor_io.resolve_path_josh()
 	const global_josh = doctor_io.resolve_pnpm_global_josh()
-	if (path_josh === undefined || global_josh === undefined) return undefined
-	if (!doctor_logic.is_shadowed(path_josh, global_josh)) return undefined
+
+	if (
+		path_josh === undefined ||
+		global_josh === undefined ||
+		!doctor_logic.is_shadowed(path_josh, global_josh)
+	) {
+		return undefined
+	}
 
 	return doctor_logic.format_shadow_warning(path_josh, global_josh)
 }

package/scripts/version/latest-corepack.ts

@@ -259,8 +259,7 @@ function restore_package_json(
 
 function decode_integrity(encoded: string): string | undefined {
 	const bytes = Buffer.from(encoded, 'base64')
-	if (bytes.length !== SHA512_BYTES) return undefined
-	if (bytes.toString('base64') !== encoded) return undefined
+	if (bytes.length !== SHA512_BYTES || bytes.toString('base64') !== encoded) return undefined
 
 	return `sha512.${bytes.toString('hex')}`
 }

package/scripts/version/release-hold.ts

@@ -51,8 +51,10 @@ function resolve_installable(
 	minimum_age_minutes: number,
 	now_ms: number,
 ): string | undefined {
-	if (times === undefined) return undefined
-	if (!is_latest_withheld(times, latest, minimum_age_minutes, now_ms)) return undefined
+	if (times === undefined || !is_latest_withheld(times, latest, minimum_age_minutes, now_ms)) {
+		return undefined
+	}
+
 	const installable = release_age.select_aged_version(times, undefined, minimum_age_minutes, now_ms)
 
 	return installable === latest ? undefined : installable
@@ -122,8 +124,13 @@ function build_release_hold_notes(
 	installable: string | undefined,
 	minimum_age_minutes: number,
 ): Array<string> {
-	if (version === undefined || installable === undefined) return []
-	if (!is_release_age_hold(version, latest, installable)) return []
+	if (
+		version === undefined ||
+		installable === undefined ||
+		!is_release_age_hold(version, latest, installable)
+	) {
+		return []
+	}
 
 	return [format_release_hold_note(version, latest, installable, minimum_age_minutes)]
 }

package/scripts/version/version-commands.ts

@@ -101,8 +101,10 @@ function build_upstream_effective(
 	context: UpstreamHookContext,
 ): UpstreamEffective | undefined {
 	const { resolve_effective_version, resolve_global_upgrade_command } = upstream
-	if (resolve_effective_version === undefined) return undefined
-	if (resolve_global_upgrade_command === undefined) return undefined
+
+	if (resolve_effective_version === undefined || resolve_global_upgrade_command === undefined) {
+		return undefined
+	}
 
 	return {
 		version: resolve_effective_version(context),
@@ -264,8 +266,14 @@ function describe_effective_outcome(
 	snapshot: VersionSnapshot,
 ): Array<string> {
 	const { resolve_effective_version } = upstream
-	if (resolve_effective_version === undefined || report === undefined) return []
-	if (version_check_logic.build_effective_upgrade_commands(report).length === 0) return []
+
+	if (
+		resolve_effective_version === undefined ||
+		report === undefined ||
+		version_check_logic.build_effective_upgrade_commands(report).length === 0
+	) {
+		return []
+	}
 
 	const after = resolve_effective_version({
 		latest: snapshot.latest,

package/templates/gitignore

@@ -43,13 +43,17 @@ project.inlang/cache/
 *storybook.log
 storybook-static
 
-# Verification gate caches (josh lint / josh check / josh cspell:dot), plus the edit hook's own
-# eslint cache (josh format:edited), which is kept apart from the gate's on purpose
+# Verification gate caches (josh lint / josh check / josh cspell:dot), plus the edit hook's, the
+# scoped lint's and the pre-commit hook's own, which are kept apart from the gate's on purpose
 .eslintcache
 .eslintcache.edit
 .eslintcache.related
+.eslintcache.commit
 .tsbuildinfo
 .cspellcache
+.cspellcache.commit
+.prettiercache
+.prettiercache.related
 
 # Bundle analysis
 stats-client.html

package/dist/hooks/chunk-D5QBTOCZ.js

@@ -1,68 +0,0 @@
-import {
-  investigation_reads
-} from "./chunk-WNLVGT7E.js";
-import {
-  lane_guard_policy
-} from "./chunk-UT7YC3N2.js";
-import {
-  hook_decision
-} from "./chunk-GPLGZ6IC.js";
-
-// scripts/delegation/investigation-guard.ts
-import { text } from "node:stream/consumers";
-import { fileURLToPath } from "node:url";
-var SWITCH_ENV_KEY = "JOSH_INVESTIGATION_GUARD";
-var STAMP_PREFIX = "josh-investigation-guard-";
-var NOTICE_STAMP_PREFIX = "josh-investigation-guard-notice-";
-function investigation_mode() {
-  return lane_guard_policy.mode_here("investigation");
-}
-function should_block(tail, call, refused_at_ms, run) {
-  if (investigation_mode() !== "refuse") return false;
-  return investigation_reads.should_block(tail, call, refused_at_ms, run);
-}
-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, 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(" ")}.`;
-}
-var GUARD = hook_decision.create_transcript_guard({
-  prefix: STAMP_PREFIX,
-  switch_key: SWITCH_ENV_KEY,
-  is_candidate: investigation_reads.is_refusable_call,
-  should_block,
-  reason: investigation_reads.REASON,
-  // The lane-child notice fires where the refusal is withheld, on a record of its own so it
-  // never spends the refusal's stamp.
-  notify: {
-    prefix: NOTICE_STAMP_PREFIX,
-    should_notify,
-    text: notice_text
-  }
-});
-var investigation_refusal = GUARD.refusal;
-var investigation_outcome = GUARD.outcome;
-var { is_enabled, refusal_path } = GUARD;
-var investigation_notice_path = hook_decision.create_refusal_stamp(NOTICE_STAMP_PREFIX).path;
-var { deny_envelope, load_environment_file, DISABLED_VALUES } = hook_decision;
-if (process.argv[1] === fileURLToPath(import.meta.url)) {
-  if (process.stdin.isTTY) hook_decision.report_no_payload("investigation:guard");
-  else hook_decision.write_outcome(await text(process.stdin), investigation_outcome);
-}
-
-export {
-  SWITCH_ENV_KEY,
-  NOTICE,
-  investigation_refusal,
-  investigation_outcome,
-  is_enabled,
-  refusal_path,
-  investigation_notice_path,
-  deny_envelope,
-  load_environment_file,
-  DISABLED_VALUES
-};

package/dist/hooks/chunk-UT7YC3N2.js

@@ -1,56 +0,0 @@
-import {
-  lane_child_marker
-} from "./chunk-OLMO72ON.js";
-
-// scripts/lane/lane-guard-policy.ts
-var DEFAULT_MODE = "refuse";
-var OFF_MODE = "off";
-var LANE_GUARD_POLICY = [
-  {
-    id: "investigation",
-    mode_in_lane_child: "notice",
-    because: 'a refusal ends a child turn so it cannot fire as one (kit#2138), which is why it was `off` from #2138 to #2382. The exemption reason \u2014 "the child is itself the delegated unit, so it cannot dispatch a sub-unit to read its own edit targets" \u2014 did not hold in measurement: kit#2382 re-measured the six lane children after #2347 (#2366 #2370 #2371 #2372 #2374 #2375) and every one dispatched sub-agents (10 launches, 2 of them investigation fan-outs) and read 4\u201317 files it never edited, all six over the threshold of 3. So the reading the guard exists to send out was there to send. It takes `notice`, not `off`: the child is nudged to delegate rather than killed, on the `additionalContext` path batching and duplicate-read already use, and the notice names the concrete unedited files this run read (the #2276 precedent) rather than the generic "delegate" #2164 \u2192 #2178 measured not to move the number. The density is a genuine experiment \u2014 re-measured on the next backlogrun, and a notice that still does not move it is redesigned rather than kept'
-  },
-  {
-    id: "batching",
-    mode_in_lane_child: "off",
-    because: "a refusal ends a child turn so it cannot fire as one (kit#2138); kit#2164 replaced it with a notice, kit#2178 took that notice off after it did not move the density, and kit#2276 restored it \u2014 naming the concrete recent calls and recurring every single-call turn \u2014 as a genuine experiment to be redesigned rather than kept if the next backlogrun still missed 1.40. kit#2405 is that re-measurement, and it cuts the notice off (`off`). `josh time:density` read 1.13 calls per round trip over the ten most recent lanes (1.04 in the Issue\u2019s per-message frame), short of the 1.40 target as every advisory attempt before it was (kit#1304/#1329/#1337/#1390/#2164/#2276). The cause is structural rather than wording: a `PreToolUse` hook cannot see the turn it is in \u2014 a turn\u2019s later `tool_use` lines arrive 1.4\u201315s after its first \u2014 so it judges only closed turns, and a notice at turn close still asks for the parallel `tool_use` blocks the model resists. The one lever measured to move the density is a composite command the model emits in a single call (`read:files`/`edit:files`, kit#2202/#2311), which needs no notice. So per the issue\u2019s instruction to discontinue advisory means when the target is missed, the notice is cut off rather than rewritten a seventh time; the density stays readable through `josh time:density`"
-  },
-  {
-    id: "duplicate-read",
-    mode_in_lane_child: "notice",
-    because: "a refusal ends a child turn so it cannot fire as one (kit#2138), and the unchanged re-reads the guard catches are measured in lane children above all (kit#2298 baseline: 8.0 per lane run) \u2014 so silencing it there (`off`) would neuter it exactly where the count lives. It takes `notice` instead: the child is nudged to scroll up to the earlier read rather than killed. The mode is a genuine experiment, as the batching notice is \u2014 a notice that does not reduce the duplicate reads on the next backlogrun is redesigned rather than kept"
-  },
-  {
-    id: "rule",
-    mode_in_lane_child: "refuse",
-    because: "it carries the lane-only rules a child depends on (`pre-gate-cut`, `implementation-cut`, `lane-park`) and the safety rules a child must still obey (`shell-body`, `piped-verification`); a blanket suppression would disarm exactly the guards written for the child. `implementation-cut` (joshuafolkken/kit#2310) needs `refuse` for the same reason the pre-gate cut does \u2014 kit#1864 and kit#2310 both measured a cut carried as prose firing 0 times, so a notice, being prose in front of the run, would drop enforcement without a procedural replacement. The pre-gate-cut refusal does collateral a sibling call batched in the same turn \u2014 a `PreToolUse` deny cancels the turn's other parallel calls, which is the harness behavior kit#2177 measured on #2160 (gate refused, its batched `review:brief` cancelled) \u2014 and that collateral is not avoidable at the guard level. `notice` was considered for the pre-gate-cut point and rejected: kit#1864 measured the cut taken 0 times while it was carried as prose, and a notice is prose in front of the run (kit#2178 measured a lane-child notice not moving the number), so a notice would drop enforcement without a procedural replacement. kit#2177 supplies that replacement in the procedure instead \u2014 the chain orders the cut before the gate, so the cut is issued on its own (its `cut` verdict ends the turn, batching with nothing) and the gate runs in the fresh process where the carried cut keeps this guard silent; the refusal therefore stays `refuse` as insurance and the collateral is gone from the default path rather than from the guard"
-  }
-];
-var KNOWN_GUARD_IDS = LANE_GUARD_POLICY.map((entry) => entry.id);
-function entry_for(id) {
-  return LANE_GUARD_POLICY.find((entry) => entry.id === id.trim());
-}
-function mode_in_lane_child(id) {
-  return entry_for(id)?.mode_in_lane_child ?? DEFAULT_MODE;
-}
-function mode_here(id, directory = process.cwd(), source = process.env) {
-  const lane_mode = mode_in_lane_child(id);
-  if (lane_mode === DEFAULT_MODE) return DEFAULT_MODE;
-  return lane_child_marker.is_child_of(directory, source) ? lane_mode : DEFAULT_MODE;
-}
-function is_suppressed_here(id, directory = process.cwd(), source = process.env) {
-  return mode_here(id, directory, source) === OFF_MODE;
-}
-var lane_guard_policy = {
-  LANE_GUARD_POLICY,
-  KNOWN_GUARD_IDS,
-  entry_for,
-  mode_in_lane_child,
-  mode_here,
-  is_suppressed_here
-};
-
-export {
-  lane_guard_policy
-};