@joshuafolkken/kit 1.1453.0 → 1.1471.0

package/dist/hooks/pretool-guard.js

@@ -1,22 +1,22 @@
 #!/usr/bin/env tsx
 import {
   pretool_guard
-} from "./chunk-5C7L6FRC.js";
-import "./chunk-SHYUQOIS.js";
-import "./chunk-Y3GKIKGW.js";
+} from "./chunk-IIGFYXKC.js";
+import "./chunk-YYPI7SJZ.js";
+import "./chunk-43ZHT22P.js";
 import "./chunk-IX6TTTI6.js";
-import "./chunk-Z4XI7CZS.js";
+import "./chunk-EUMACUK2.js";
 import "./chunk-CDY4NFQG.js";
-import "./chunk-EF5GQ6WS.js";
-import "./chunk-LBFYEOXJ.js";
-import "./chunk-NO4AALP5.js";
-import "./chunk-UT7YC3N2.js";
+import "./chunk-YHBZAJSV.js";
+import "./chunk-K4LMBQC2.js";
+import "./chunk-H5HVWYXK.js";
+import "./chunk-SHGPANFB.js";
 import {
   hook_decision
-} from "./chunk-W5ZKGTSY.js";
-import "./chunk-MBJYQPEK.js";
-import "./chunk-OLMO72ON.js";
-import "./chunk-3C4XRYG6.js";
+} from "./chunk-J47HU5H5.js";
+import "./chunk-AS74OU3D.js";
+import "./chunk-K4EOONYM.js";
+import "./chunk-E7OIKX5Z.js";
 import "./chunk-DGT54IH5.js";
 
 // scripts/hooks/pretool-guard-cli.ts

package/dist/hooks/session-lang.js

@@ -8,19 +8,19 @@ import {
   run_event_stream_emit,
   run_headless,
   session_language
-} from "./chunk-Z4XI7CZS.js";
+} from "./chunk-EUMACUK2.js";
 import {
   error_text
 } from "./chunk-CDY4NFQG.js";
 import {
   josh_environment_file
-} from "./chunk-OLMO72ON.js";
+} from "./chunk-K4EOONYM.js";
 import {
   agent_session_environment,
   json_value,
   process_identity,
   run_carry
-} from "./chunk-3C4XRYG6.js";
+} from "./chunk-E7OIKX5Z.js";
 import {
   stamp_file
 } from "./chunk-DGT54IH5.js";
@@ -227,8 +227,9 @@ function ensure_ledger_directory(directory) {
   }
 }
 function ledger_arguments(common_directory, profile) {
-  if (common_directory === void 0 || profile.role !== agent_role_profile.WORKER) return [];
-  if (path2.basename(common_directory) !== GIT_DIRECTORY) return [];
+  if (common_directory === void 0 || profile.role !== agent_role_profile.WORKER || path2.basename(common_directory) !== GIT_DIRECTORY) {
+    return [];
+  }
   const directory = path2.join(path2.dirname(common_directory), LEDGER_DIRECTORY);
   ensure_ledger_directory(directory);
   return [ADD_DIRECTORY_FLAG, directory];

package/dist/hooks/stop-guard.js

@@ -13,7 +13,7 @@ import {
   repo_party,
   run_cut,
   run_hold
-} from "./chunk-Y3GKIKGW.js";
+} from "./chunk-43ZHT22P.js";
 import {
   agent_headless,
   agent_role_profile,
@@ -23,22 +23,22 @@ import {
   run_event_stream_emit,
   run_headless,
   session_language
-} from "./chunk-Z4XI7CZS.js";
+} from "./chunk-EUMACUK2.js";
 import {
   timed_fetch
 } from "./chunk-723HKZO5.js";
 import "./chunk-CDY4NFQG.js";
 import {
   hook_decision
-} from "./chunk-W5ZKGTSY.js";
+} from "./chunk-J47HU5H5.js";
 import {
   time_density_hook,
   time_hook_transcript,
   time_reported_failure
-} from "./chunk-MBJYQPEK.js";
+} from "./chunk-AS74OU3D.js";
 import {
   lane_child_marker
-} from "./chunk-OLMO72ON.js";
+} from "./chunk-K4EOONYM.js";
 import {
   backlog_budget,
   issue_number_shape,
@@ -47,7 +47,7 @@ import {
   process_owner_schema,
   run_carry,
   run_invocation
-} from "./chunk-3C4XRYG6.js";
+} from "./chunk-E7OIKX5Z.js";
 import {
   stamp_file
 } from "./chunk-DGT54IH5.js";
@@ -243,13 +243,16 @@ var NONE2 = 0;
 async function live_lane_count() {
   return lane_registry.live_count(await lane_registry.list_lanes());
 }
+async function free_lanes_beside(lanes) {
+  const limit = await lane_capacity.lane_limit();
+  if (limit.kind !== "limit") return void 0;
+  return lane_capacity.free_lanes(limit.limit, lane_registry.live_count(lanes));
+}
 async function free_under(limit) {
   return lane_capacity.free_lanes(limit, await live_lane_count());
 }
 async function free_lane_count() {
-  const limit = await lane_capacity.lane_limit();
-  if (limit.kind !== "limit") return NO_FREE;
-  return await free_under(limit.limit);
+  return await free_lanes_beside(await lane_registry.list_lanes()) ?? NO_FREE;
 }
 async function drive_free_lane_count() {
   const limit = await lane_capacity.lane_limit();
@@ -306,6 +309,7 @@ var backlog_ready = {
   read_backlog_next,
   print_offer_hint,
   free_lane_count,
+  free_lanes_beside,
   live_lane_count,
   drive_free_lane_count,
   drains_pool,
@@ -317,6 +321,52 @@ var backlog_ready = {
 
 // scripts/notify/telegram-notify.ts
 import { z } from "zod";
+
+// scripts/notify/confirmation-options.ts
+var JAPANESE_LANG_PREFIX = "ja";
+var JAPANESE_HEADINGS = {
+  recommended: "\u63A8\u5968:",
+  alternatives: "\u4ED6\u306E\u9078\u629E\u80A2:",
+  none: "\u306A\u3057"
+};
+var ENGLISH_HEADINGS = {
+  recommended: "Recommended:",
+  alternatives: "Alternatives:",
+  none: "none"
+};
+function headings() {
+  const { lang } = session_language.resolve_session_lang();
+  return lang.toLowerCase().startsWith(JAPANESE_LANG_PREFIX) ? JAPANESE_HEADINGS : ENGLISH_HEADINGS;
+}
+function alternatives_block(options, labels) {
+  if (options.alternatives.length === 0) return `${labels.alternatives} ${labels.none}`;
+  const lines = options.alternatives.map((alternative) => `- ${alternative}`);
+  return [labels.alternatives, ...lines].join("\n");
+}
+function block(options) {
+  const labels = headings();
+  return `${labels.recommended} ${options.recommended}
+${alternatives_block(options, labels)}`;
+}
+var confirmation_options = { block };
+
+// scripts/notify/notify-opt-in.ts
+var OPT_IN_KEY = "JOSH_NOTIFY_OPT_IN";
+var OPTIONAL_TASK_TYPES = /* @__PURE__ */ new Set(["planning", "completion"]);
+var LIST_SEPARATOR = ",";
+function opted_in_types() {
+  return (process.env[OPT_IN_KEY] ?? "").split(LIST_SEPARATOR).map((entry) => entry.trim().toLowerCase()).filter((entry) => entry.length > 0);
+}
+function is_skipped(task_type) {
+  if (!OPTIONAL_TASK_TYPES.has(task_type)) return false;
+  return !opted_in_types().includes(task_type);
+}
+function skip_line(task_type) {
+  return `\u{1F515} ${task_type} notifications are not opted in (${OPT_IN_KEY}).`;
+}
+var notify_opt_in = { is_skipped, skip_line };
+
+// scripts/notify/telegram-notify.ts
 var TELEGRAM_API_BASE = "https://api.telegram.org";
 var TELEGRAM_TEXT_LIMIT = 4096;
 var BODY_TRUNCATION_MARK = "\n...";
@@ -377,6 +427,12 @@ function load_config() {
 function is_notify_disabled() {
   return parse_environment_string(process.env[NOTIFY_SWITCH_KEY]).toLowerCase() === NOTIFY_OFF_VALUE;
 }
+function skip_line2(task_type) {
+  if (is_notify_disabled()) {
+    return `\u{1F515} Telegram notifications are disabled (${NOTIFY_SWITCH_KEY}=${NOTIFY_OFF_VALUE}).`;
+  }
+  return notify_opt_in.is_skipped(task_type) ? notify_opt_in.skip_line(task_type) : void 0;
+}
 function build_header(task_type, repo_name) {
   const { icon, label } = TASK_DEFINITIONS[task_type];
   if (repo_name === void 0 || repo_name.length === 0) return `${icon} ${label}`;
@@ -402,6 +458,7 @@ function build_url_parts(input) {
 function build_blocks(input) {
   const blocks = [build_title_block(input)];
   push_if_present(blocks, input.body);
+  push_if_present(blocks, input.choices);
   blocks.push(...build_url_parts(input));
   return blocks;
 }
@@ -435,10 +492,9 @@ function build_send_failure(error, config) {
   return new Error(`${SEND_FAILED_PREFIX}: ${reason}`, { cause: new Error(reason) });
 }
 async function send(input) {
-  if (is_notify_disabled()) {
-    console.info(
-      `\u{1F515} Telegram notifications are disabled (${NOTIFY_SWITCH_KEY}=${NOTIFY_OFF_VALUE}).`
-    );
+  const skipped = skip_line2(input.task_type);
+  if (skipped !== void 0) {
+    console.info(skipped);
     return;
   }
   const config = load_config();
@@ -487,6 +543,7 @@ async function confirm(input) {
       repo_name: gh_spawn.get_repo_name_with_owner_within(REPO_LOOKUP_TIMEOUT_MS),
       issue_title: input.issue_title,
       body: input.body,
+      choices: confirmation_options.block(input),
       issue_url: void 0,
       pr_url: void 0
     },
@@ -762,23 +819,23 @@ var stop_payload_schema = z3.object({
   last_assistant_message: z3.string().nullish()
 });
 var NO_OUTCOME = { reason: void 0 };
-var STOP_NOTIFY_REASON = '\u26D4 mid-workflow stop notification: this working tree is still held by a run and no `confirmation` Telegram was sent this turn, so a person is being left to wait off-screen without knowing the run paused. `CLAUDE.md` \u2192 "Mid-workflow stop notification" requires `pnpm josh notify --task-type confirmation --issue-url "<url>" --body=$\'<reason>\'` before any mid-workflow pause. Send it, then end with a one-line confirmation that it was sent \u2014 do not repeat your previous reply. This fires only while the hold is held and no notify is on the transcript, and `stop_hook_active` lets a second stop through so a run is never wedged.';
-var HOLD_RELEASE_REASON = "\u26D4 working-tree hold not released: this working tree is clean but its `run:hold` record is still in place, so the next run here runs `pnpm josh ms` believing the tree is free while you hold it. The workflow-commands skill \u2192 `working-tree-hold.md`: a stop that leaves the tree clean releases the hold with `pnpm josh run:release <N>` (bare for a `new` entry). A `halfrun` pre-commit stop and a `needs-human-review` stop keep the hold because their tree is dirty, and a `prrun` stop keeps it by its stop mark \u2014 this row is silent there. Release it, then end with a one-line confirmation that it was released \u2014 do not repeat your previous reply.";
+var STOP_NOTIFY_REASON = '\u26D4 mid-workflow stop notification: this working tree is still held by a run and no `confirmation` Telegram was sent this turn. Send `pnpm josh notify --task-type confirmation --issue-url "<url>" --body=$\'<reason>\' --recommended "<choice>" --alternative "<choice>"` (`CLAUDE.md` \u2192 "Mid-workflow stop notification"), then confirm in one line that it was sent \u2014 do not repeat your previous reply.';
+var HOLD_RELEASE_REASON = "\u26D4 working-tree hold not released: this working tree is clean but its `run:hold` record is still in place. Release it with `pnpm josh run:release <N>` (bare for a `new` entry; `working-tree-hold.md`), then confirm in one line that it was released \u2014 do not repeat your previous reply.";
 var ISSUE_CITATION_REASON = "Session-facing output cites an Issue as a number-link so the reader can click through and see which repository it is (`CLAUDE.md`, `prompts/collaboration-workflow/issue-citation.md`).";
 function build_citation_reason(references) {
   const numbers = references.join(", ");
   const command = `pnpm josh issue:cite ${issue_citation.cite_arguments(references).join(" ")}`;
   return `\u26D4 issue citation: your reply names an Issue as a bare \`#N\` (${numbers}). Run \`${command}\` to print the paste-ready citation lines, then print the corrected citation lines alone \u2014 do not repeat your previous reply. ${ISSUE_CITATION_REASON}`;
 }
-var FILING_OFFER_REASON = '\u26D4 filing offer: your reply offers to file an Issue instead of filing it, and this run is unattended \u2014 nobody is there to answer. Filing into a first-party repository there is Tier A \u2014 the trigger is the judgement that it is worth filing, not the run\'s progress (the workflow-commands skill \u2192 `observation-filing.md`). Run `pnpm josh issue:file "<title>" --body-file <path> --depth <0|1|2>` \u2014 it scouts, lints, labels and runs `epic:bundle` itself \u2014 then end with the one-line citation of what was filed; do not repeat your previous reply. If it is not worth filing after all, say so in one line instead.';
+var FILING_OFFER_REASON = '\u26D4 filing offer: your reply offers to file an Issue instead of filing it, and this run is unattended \u2014 nobody is there to answer. Run `pnpm josh issue:file "<title>" --body-file <path> --depth <0|1|2>` (Tier A, `observation-filing.md`), then end with the one-line citation of what was filed; do not repeat your previous reply. If it is not worth filing after all, say so in one line instead.';
 var HEADLESS_WAIT_MARKER = "\u26D4 headless parent:";
 var HEADLESS_REFUSAL_CAP = 3;
 var HEADLESS_WAIT_CALLS = ['"command":"pnpm josh lane:await'];
-var HEADLESS_WAIT_BODY = ' this is a `claude -p` `backlogrun` parent (`JOSH_RUN_HEADLESS`) with lanes still in flight, and ending this turn ends the process \u2014 its background waits are killed with it and the lanes are left without a parent. Do not watch them yourself: the `run:wake` supervisor\'s driver watches the lanes and wakes a session only for a branch that needs judging. Hand the loop back with `pnpm josh run:carry --cut --owner "$PPID"` (or `--end` when the run is over), then end the turn (`backlogrun-progress.md` \u2192 "Picking the lanes up in the fresh session").';
+var HEADLESS_WAIT_BODY = ' this is a `claude -p` `backlogrun` parent with lanes in flight, and ending this turn kills the process and its background waits. Hand the loop back with `pnpm josh run:carry --cut --owner "$PPID"` (or `--end` when the run is over), then end the turn (`backlogrun-progress.md`).';
 var HEADLESS_WAIT_REASON = `${HEADLESS_WAIT_MARKER}${HEADLESS_WAIT_BODY}`;
 var INDEX_COMMAND_PATTERN = /git (?:add|stage|rm --cached|restore --staged|restore -S)\b/u;
 var PERMISSION_PATTERN = /許可|承認|permission|approve|approval|authorization/iu;
-var LANE_INDEX_REASON = '\u26D4 lane index permission: this is a dispatched lane child (`JOSH_LANE_CHILD`) and your reply asks a person for permission to change the git index. A lane is a per-run work tree with no person\'s staging to protect, and the sanctioned commit flow is already authorized: remove any conflict markers, then run `pnpm josh git -y`, which stages and commits them \u2014 a conflicted merge included. This is Tier A (`.claude/skills/workflow-commands/chain-rule.md` \u2192 "origin/main is merged in before the gate"). Do it and continue the run \u2014 do not repeat your previous reply.';
+var LANE_INDEX_REASON = "\u26D4 lane index permission: this is a dispatched lane child (`JOSH_LANE_CHILD`) and your reply asks a person for permission to change the git index. That is Tier A here (`chain-rule.md`): remove any conflict markers, run `pnpm josh git -y` (it stages and commits, a conflicted merge included) and continue the run \u2014 do not repeat your previous reply.";
 function asks_index_permission(message2) {
   return INDEX_COMMAND_PATTERN.test(message2) && PERMISSION_PATTERN.test(message2);
 }

package/dist/josh.js

@@ -743,8 +743,9 @@ function is_worker_flag(argument) {
   return WORKER_FLAGS.some((flag) => argument === flag || argument.startsWith(`${flag}=`));
 }
 function worker_arguments(vitest_arguments, share) {
-  if (share === void 0) return [];
-  if (vitest_arguments.some((argument) => is_worker_flag(argument))) return [];
+  if (share === void 0 || vitest_arguments.some((argument) => is_worker_flag(argument))) {
+    return [];
+  }
   return [`${WORKER_FLAG}=${String(share)}`];
 }
 function mark_run_started(target) {
@@ -2747,7 +2748,7 @@ var WORKFLOW_COMMANDS = {
     description: "Send Telegram notification",
     category: "Workflow",
     reference: [
-      "[--task-type <type>] [--body <text> | --body-file <path>] [--issue-url <url>] [--pr-url <url>] [--issue-title <text>] [--repo-name <name>]",
+      "[--task-type <type>] [--body <text> | --body-file <path>] [--issue-url <url>] [--pr-url <url>] [--issue-title <text>] [--repo-name <name>] [--recommended <text>] [--alternative <text>]...",
       "automation",
       ["notifications"]
     ],
@@ -3024,8 +3025,7 @@ function parse_scp_like(url) {
   const user_end = url.indexOf(USER_SEPARATOR);
   const authority = user_end === NOT_FOUND ? url : url.slice(user_end + 1);
   const host_end = authority.indexOf(HOST_SEPARATOR);
-  if (host_end === NOT_FOUND) return void 0;
-  if (!is_github_ssh_host(authority.slice(0, host_end))) return void 0;
+  if (host_end === NOT_FOUND || !is_github_ssh_host(authority.slice(0, host_end))) return void 0;
   return to_identity(authority.slice(host_end + 1));
 }
 function parse_url_like(url) {
@@ -3053,8 +3053,7 @@ function origin_section_lines(lines) {
 }
 function url_value(line) {
   const separator = line.indexOf("=");
-  if (separator === NOT_FOUND) return void 0;
-  if (line.slice(0, separator).trim() !== URL_KEY) return void 0;
+  if (separator === NOT_FOUND || line.slice(0, separator).trim() !== URL_KEY) return void 0;
   return line.slice(separator + 1).trim();
 }
 function parse_origin_from_config(content) {

package/dist/managed-marker/index.js

@@ -18,8 +18,7 @@ function is_marked(text) {
   return text.startsWith(MARKER_PREFIX);
 }
 function apply_marker_for_destination(destination, text, package_name) {
-  if (!is_workflow_destination(destination)) return text;
-  if (is_marked(text)) return text;
+  if (!is_workflow_destination(destination) || is_marked(text)) return text;
   return `${marker_block(package_name)}${text}`;
 }
 var managed_marker_logic = {

package/dist/version/index.js

@@ -141,8 +141,7 @@ function read_matching_version(manifest_path, package_name) {
   const parsed = package_named_version_schema.safeParse(
     json_value.parse_or_undefined(readFileSync(manifest_path, "utf8"))
   );
-  if (!parsed.success) return void 0;
-  if (parsed.data.name !== package_name) return void 0;
+  if (!parsed.success || parsed.data.name !== package_name) return void 0;
   return parsed.data.version;
 }
 function walk_up_for_version(start_directory, package_name) {
@@ -152,9 +151,9 @@ function walk_up_for_version(start_directory, package_name) {
   }
   return void 0;
 }
-function resolve_from_base(base_url, package_name, resolve_marker) {
+function resolve_from_base(base_url, package_name, resolve_marker = package_name) {
   try {
-    return createRequire(base_url).resolve(resolve_marker ?? package_name);
+    return createRequire(base_url).resolve(resolve_marker);
   } catch {
     return void 0;
   }
@@ -281,8 +280,9 @@ function is_latest_withheld(times, latest, minimum_age_minutes, now_ms) {
   return published_ms > now_ms - minimum_age_minutes * MS_PER_MINUTE2;
 }
 function resolve_installable(times, latest, minimum_age_minutes, now_ms) {
-  if (times === void 0) return void 0;
-  if (!is_latest_withheld(times, latest, minimum_age_minutes, now_ms)) return void 0;
+  if (times === void 0 || !is_latest_withheld(times, latest, minimum_age_minutes, now_ms)) {
+    return void 0;
+  }
   const installable = release_age.select_aged_version(times, void 0, minimum_age_minutes, now_ms);
   return installable === latest ? void 0 : installable;
 }
@@ -307,8 +307,9 @@ function format_release_hold_note(version, latest, installable, minimum_age_minu
   return `${head}; an unpinned resolve lands on ${installable}`;
 }
 function build_release_hold_notes(version, latest, installable, minimum_age_minutes) {
-  if (version === void 0 || installable === void 0) return [];
-  if (!is_release_age_hold(version, latest, installable)) return [];
+  if (version === void 0 || installable === void 0 || !is_release_age_hold(version, latest, installable)) {
+    return [];
+  }
   return [format_release_hold_note(version, latest, installable, minimum_age_minutes)];
 }
 var release_hold = {
@@ -1399,8 +1400,9 @@ function build_extras(config) {
 }
 function build_upstream_effective(upstream, context) {
   const { resolve_effective_version, resolve_global_upgrade_command } = upstream;
-  if (resolve_effective_version === void 0) return void 0;
-  if (resolve_global_upgrade_command === void 0) return void 0;
+  if (resolve_effective_version === void 0 || resolve_global_upgrade_command === void 0) {
+    return void 0;
+  }
   return {
     version: resolve_effective_version(context),
     upgrade_command: resolve_global_upgrade_command(context)
@@ -1485,8 +1487,9 @@ function build_idle_message(reports) {
 }
 function describe_effective_outcome(upstream, report, snapshot) {
   const { resolve_effective_version } = upstream;
-  if (resolve_effective_version === void 0 || report === void 0) return [];
-  if (version_check_logic.build_effective_upgrade_commands(report).length === 0) return [];
+  if (resolve_effective_version === void 0 || report === void 0 || version_check_logic.build_effective_upgrade_commands(report).length === 0) {
+    return [];
+  }
   const after = resolve_effective_version({
     latest: snapshot.latest,
     upstream_latest: report.latest

package/eslint/base.js

@@ -29,7 +29,7 @@ import {
 	SPEC_FILENAME_PATTERNS,
 } from './rules/test-filename.js'
 import { typescript_rules } from './rules/typescript.js'
-import { unicorn_rules } from './rules/unicorn.js'
+import { unicorn_rules, unicorn_test_rules } from './rules/unicorn.js'
 
 const FILE_PATTERNS = {
 	d_ts: ['**/*.d.ts'],
@@ -234,6 +234,7 @@ export function create_base_config({ gitignore_path, tsconfig_root_dir }) {
 				// `expect(x).toBe(0.05)` compares deterministic config constants, not computed
 				// floats; working around the rule only weakens the assertion.
 				'sonarjs/no-floating-point-equality': 'off',
+				...unicorn_test_rules,
 			},
 		},
 		{

package/eslint/rules/unicorn.js

@@ -205,3 +205,19 @@ export const unicorn_rules = {
 	// たびにエディタと衝突し、可読性の利得なく差分ノイズだけを生むため無効化する（joshuafolkken/kit#3273）。
 	'unicorn/no-asterisk-prefix-in-documentation-comments': 'off',
 }
+
+// unicorn 78 のベースライン方針（joshuafolkken/kit#3882）: 78 は新ルール 2 つ
+// （no-unnecessary-default-arguments / no-repeated-call-wrappers）に加え、型解決とコメントを含む
+// コードの検査を広げたため、既存ルールが新たに報告する。prefer-combined-guards /
+// prefer-minimal-ternary / no-negated-condition / prefer-default-parameters は正当な
+// クリーンアップなので採用し、コードを直す。consistent-boolean-name だけは下記のとおり
+// テストファイルで外す。
+export const unicorn_test_rules = {
+	// 78 は vi.fn() / vi.spyOn() / vi.mocked() の戻り型を解決できるようになり、真偽値を返す関数の
+	// モックやスパイ（`const write = vi.spyOn(process.stdout, 'write')`）を真偽値の束縛として報告する。
+	// 関数値の const を checkVariables で検査するのはプラグインの文書どおりの仕様（77 でも
+	// `const f = () => true` は報告される）で、誤検知ではない。ただしテストの束縛はほぼすべて
+	// 差し替える関数を名前で示すテストダブルで、`is_write` のような名前は述語の結果のように読めて
+	// 誤解を招く。kit が関数名を検査しない（checkFunctions: 'never'）のと同じ理由でテストでは外す。
+	'unicorn/consistent-boolean-name': 'off',
+}

package/package.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.1453.0",
+  "version": "1.1471.0",
   "name": "@joshuafolkken/kit",
   "description": "Development toolkit for AI-assisted projects: formatting, AI assistant rules and a Git/GitHub Issue workflow for any repository, plus lint, type check, tests and hooks for Node projects",
   "keywords": [
@@ -178,7 +178,7 @@
     "eslint-plugin-import-x": "^4.17.1",
     "eslint-plugin-promise": "^7.3.0",
     "eslint-plugin-sonarjs": "^4.2.1",
-    "eslint-plugin-unicorn": "^77.0.0",
+    "eslint-plugin-unicorn": "^78.0.0",
     "globals": "^17.12.0",
     "prettier": "^3.9.9",
     "prettier-plugin-svelte": "^4.1.1",
@@ -260,7 +260,7 @@
     "eslint-plugin-import-x": "^4.17.1",
     "eslint-plugin-promise": "^7.3.0",
     "eslint-plugin-sonarjs": "^4.2.2",
-    "eslint-plugin-unicorn": "^77.0.0",
+    "eslint-plugin-unicorn": "^78.0.0",
     "eslint_d": "^15.0.3",
     "globals": "^17.13.0",
     "lefthook": "^2.2.1",

package/prompts/coding-standards.md

@@ -1,27 +1,21 @@
 # コーディング指針 — lint では表せない書き方
 
-命名・エクスポート・ファイル名・内容の規約の単一ソースは下記「Conventions」、関数の構文・品質上限・型安全・早期 return・マジックナンバーの単一ソースは `CLAUDE.md` → "Critical Conventions (non-standard — always apply)" である。その大半は ESLint が強制する（クラスのプロパティ・メソッドの `snake_case` などここに書いていない細目は `eslint/rules/naming-convention.js` が持つ）。ただし **`function` 構文（アロー関数を使わない）と早期 return の 1 行形式は lint が検査しない**ので、書き手とレビューが確かめる。このファイルは本文を写さず、lint では表せない書き方の指針だけを持つ。
-
-lint・型検査の実行手順は `CLAUDE.md` → "Code Change Rules" の検証ゲート（`pnpm josh gate`、実装中は `pnpm josh lint:related` / `pnpm josh test:related`）が単一ソースである。
+関数の構文・品質上限・型安全・早期 return・マジックナンバーは `CLAUDE.md` → "Critical Conventions (non-standard — always apply)"、lint・型検査の実行は `CLAUDE.md` → "Code Change Rules"（`pnpm josh gate`、実装中は `pnpm josh lint:related` / `pnpm josh test:related`）に従う。ここにない命名の細目（クラスのプロパティ・メソッドの `snake_case` など）は `eslint/rules/naming-convention.js` が持つ。**`function` 構文（アロー関数を使わない）と早期 return の 1 行形式は lint が検査しない**ので、書き手とレビューが確かめる。
 
 ## Conventions
 
-`CLAUDE.md` から移した、lint が強制する規約の単一ソース。
-
-- **Naming**: `snake_case` variables / functions / params · `PascalCase` types / classes / interfaces / enums · `UPPER_CASE` enum members · booleans prefixed `is_` / `has_` / `should_` / `can_` / `will_` / `did_` · constants `UPPER_CASE` or `snake_case`
-- **Functions & exports**: multiple functions → a namespace object `export { my_module }` (constants exempt) · no `export default`
-- **Files**: Svelte `PascalCase.svelte(.ts)` · TypeScript `kebab-case.ts` · tests `*.test.ts` / `*.svelte.test.ts`, colocated, never `*.spec.ts` (`prompts/testing-guide.md`) · in `scripts/`, no `../` imports — use `#scripts/*`
-- **Content rules**: user-visible strings use i18n message keys in every locale · comments / test titles English only (`eslint/rules/` may explain rationale in Japanese) · no duplication; `/* @refactor-ignore */` at file top excludes a file from refactoring
+- Naming: `snake_case` variables / functions / params · `PascalCase` types / classes / interfaces / enums · `UPPER_CASE` enum members · booleans prefixed `is_` / `has_` / `should_` / `can_` / `will_` / `did_` · constants `UPPER_CASE` or `snake_case`
+- Functions & exports: multiple functions → a namespace object `export { my_module }` (constants exempt) · no `export default`
+- Files: Svelte `PascalCase.svelte(.ts)` · TypeScript `kebab-case.ts` · tests `*.test.ts` / `*.svelte.test.ts`, colocated, never `*.spec.ts` (`prompts/testing-guide.md`) · in `scripts/`, no `../` imports — use `#scripts/*`
+- Content rules: user-visible strings use i18n message keys in every locale · comments / test titles English only (`eslint/rules/` may explain rationale in Japanese) · no duplication; `/* @refactor-ignore */` at file top excludes a file from refactoring
 
 ## 📝 基本方針
 
-一般的な書き方の助言（シンプルさ・命名・過度な抽象化の回避など）は繰り返さない。lint が強制せず、一般論でもない指針だけを置く。
-
-- **意図が不明瞭な処理は関数に抽出する**: 条件式や処理の意図がわかりにくければ、たとえ 1 行でも抽出し、関数名で意図を表す（`if (is_eligible_for_service(user))`）
-- **コメントは「なぜ」だけ、経緯は書かない**: 残すのは一行の理由（そのルールの意図）だけ。「以前は…」という経緯、Issue 番号への参照、計測値は git 履歴と Issue が保持している
-- **文字列も定数化する**: 数値リテラルの抽出は lint が強制するが、繰り返し使用する文字列や識別子も定数化し、意味の分かる名前を付ける
+- **意図が不明瞭な処理は関数に抽出する**: 1 行でも抽出し、関数名で意図を表す（`if (is_eligible_for_service(user))`）
+- **コメントは「なぜ」だけ**: 一行の理由だけを残す。「以前は…」の経緯・Issue 番号・計測値は書かない（git 履歴と Issue が持つ）
+- **文字列も定数化する**: 繰り返す文字列や識別子にも意味の分かる名前の定数を付ける
 - **既存のコード体系に寄り添う**: そのうえで、より良く書けるなら目指す
-- **仕組みを足す前に原因を問う**: 設計時の判断は `prompts/collaboration-workflow/principles.md` → "elegant-design" の担当。コメントの英語限定と重複の撲滅は上の「Conventions」と `principles.md` → "no-clones" が単一ソースである
+- **仕組みを足す前に原因を問う**: `prompts/collaboration-workflow/principles.md` → "elegant-design"。重複の撲滅は "no-clones"
 
 ## 📁 新規作成時の手順
 
@@ -34,16 +28,12 @@ find src -name "*.ts" -o -name "*.svelte" | grep -i [関連キーワード]
 
 ### 2. ファイル分割の指針
 
-- **上限に近づいたら分割を検討**: 関数やファイルが `CLAUDE.md` → "Quality limits" の上限に近づいたら分割を検討する。編集前に `pnpm josh lines <path>` で余裕を確認する
-- **責任が複数ある場合**: 1つのファイルが複数の責任を持つ場合は、責任ごとにファイルを分割
-- **再利用性**: 他のファイルからも使用される可能性があるロジックは、別ファイルに分離
-- **メインファイルの肥大化を避ける**: メインファイルの肥大化を避けることを優先する。再利用可能な機能や独立した責任を持つ機能は、積極的に別ファイルに分割する。メインファイルは主要な処理フローに集中し、詳細な実装は別ファイルに分離する
-- **過度な分割は避ける**: 小さすぎるファイルの分割は避け、関連する機能は同じファイルにまとめる
+- `CLAUDE.md` → "Quality limits" の上限に近づいたら分割を検討する。編集前に `pnpm josh lines <path>` で余裕を確認する
+- 責任ごと、また他ファイルからも使われうるロジックごとに別ファイルに分け、メインファイルは主要な処理フローに集中させる
+- 小さすぎる分割は避け、関連する機能は同じファイルにまとめる
 
 ### 3. 名前空間オブジェクトの名付け方
 
-複数の関数を名前空間オブジェクトにまとめてエクスポートする規則そのものは上記「Conventions」にある（定数は個別エクスポートでよい）。ここでは名付け方だけを述べる。
-
 - ファイル名と名前空間名で意図を明確にする（例: `git-command.ts` → `git_command`）
 - メソッド名は短く、名前空間で補完する（例: `exec_git_branch()` → `git_command.branch()`）