genshin-mods 4.1.2 → 4.3.1

package/.claude-plugin/plugin.json

@@ -4,7 +4,7 @@
   "displayName": "Genshin Mods",
   "description": "Six Claude Code mods: five under one band above the prompt, waypoints (the next steps, one press each), resin (cache, context, limits and cost, with warm, compact and a one-click handoff), veil (recording mode), commission (a goal's task list and clock) and ward (a question before editing a file another session just changed), and a delegation guard that nudges a run of lookups toward a haiku agent.",
   "author": { "name": "Esposter", "url": "https://github.com/Esposter" },
-  "homepage": "https://esposter.com/docs/infra/claude-interface/genshin-mods",
+  "homepage": "https://github.com/Esposter/Esposter/tree/main/openspec/specs/infra/claude-interface/genshin-mods",
   "repository": "https://github.com/Esposter/Esposter",
   "license": "Apache-2.0",
   "keywords": ["cache", "collision", "genshin", "handoff", "mods", "recording", "tasks"],

package/README.md

@@ -45,7 +45,7 @@ claude --plugin-dir packages/genshin-mods
 
 ## <a name="documentation">📖 Documentation</a>
 
-We highly recommend you take a look at the [documentation](https://esposter.com/docs/infra/claude-interface/genshin-mods) to level up.
+We highly recommend you take a look at the [documentation](https://github.com/Esposter/Esposter/tree/main/openspec/specs/infra/claude-interface/genshin-mods) to level up.
 
 ### Command reference
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "genshin-mods",
-  "version": "4.1.2",
+  "version": "4.3.1",
   "description": "A Claude Code plugin of mods: five under one band above the prompt (the next steps, the cache and usage with a one-click handoff, recording mode, a goal's task list and a question before editing a file another session just changed) a delegation guard that nudges a run of lookups toward a haiku agent, a disk-scan guard that refuses a Bash scan from a root or a home folder, and a whole-index guard that refuses a git command over the whole index while another session shares the checkout.",
   "keywords": [
     "claude-code",
@@ -8,7 +8,7 @@
     "genshin",
     "mods"
   ],
-  "homepage": "https://esposter.com/docs/infra/claude-interface/genshin-mods",
+  "homepage": "https://github.com/Esposter/Esposter/tree/main/openspec/specs/infra/claude-interface/genshin-mods",
   "bugs": {
     "url": "https://github.com/Esposter/Esposter/issues"
   },
@@ -31,12 +31,10 @@
     "access": "public"
   },
   "scripts": {
-    "format": "oxfmt",
-    "format:check": "oxfmt --check",
     "lint": "eslint .",
     "lint:fix": "eslint --fix .",
     "test": "vitest",
     "validate": "claude plugin validate ."
   },
-  "gitHead": "144e3f98570b015e3181bfbc6cb70d346ce826b5"
+  "gitHead": "777a39636b789c13af33ed13370246ccf6dbaf31"
 }

package/src/services/constants.ts

@@ -60,6 +60,11 @@ export const LEADING_SEGMENT_REGEX: RegExp = /^\s*(?:export\s[^&;]*&&|cd\s[^&;]*
 // oxlint-disable-next-line typescript/no-inferrable-types -- `isolatedDeclarations` demands the annotation this pattern would otherwise infer
 export const LOOKUP_COMMAND_REGEX: RegExp =
   /^\s*(?:cat|sed|grep|rg|find|ls|head|tail|awk|wc|get-content|get-childitem|select-string|git\s+(?:show|log|diff)|gh\s+run\s+view)(?:\s|$)/iu;
+// The reads that write as well: sed's in-place edit, find's actions that write, and a redirection into a file, the
+// Stderr and stdout merges and the null device aside
+// oxlint-disable-next-line typescript/no-inferrable-types -- `isolatedDeclarations` demands the annotation this pattern would otherwise infer
+export const WRITING_LOOKUP_REGEX: RegExp =
+  /\bsed\b.*\s(?:-[a-zA-Z]*i|--in-place)|\bfind\b.*\s-(?:delete|exec|execdir|ok|okdir|fprint0?|fprintf|fls)\b|>(?!&|\s*\/dev\/null\b)/iu;
 // Every third lookup in a row, the chain a haiku agent should answer as one bounded question
 export const DELEGATION_NUDGE_EVERY = 3;
 export const DELEGATION_NUDGE =

package/src/services/delegation/checkIsLookupCommand.ts

@@ -1,4 +1,4 @@
-import { LEADING_SEGMENT_REGEX, LOOKUP_COMMAND_REGEX } from "../constants";
+import { LEADING_SEGMENT_REGEX, LOOKUP_COMMAND_REGEX, WRITING_LOOKUP_REGEX } from "../constants";
 
 // A session chains `export` and `cd` ahead of its real command, so each leading segment is stripped before the
 // Command's first word is read
@@ -7,5 +7,8 @@ const stripLeadingSegments = (command: string): string => {
   return rest === command ? command : stripLeadingSegments(rest);
 };
 
-export const checkIsLookupCommand = (command: string): boolean =>
-  LOOKUP_COMMAND_REGEX.test(stripLeadingSegments(command));
+export const checkIsLookupCommand = (command: string): boolean => {
+  const rest = stripLeadingSegments(command);
+
+  return LOOKUP_COMMAND_REGEX.test(rest) && !WRITING_LOOKUP_REGEX.test(rest);
+};

package/src/services/diskScan/getDiskScanRefusal.ts

@@ -19,11 +19,15 @@ const normalizePath = (token: string) => {
 
 const isRoot = (token: string) => ROOT_REGEX.test(normalizePath(token));
 
-// A flag group holding r or R: `-R`, `-rn`, `-lR`, `--recursive`
-const isRecursiveFlag = (token: string) => /^-[a-zA-Z]*[rR]/u.test(token) || token === "--recursive";
+// The long form `--recursive`, which getopt also takes as any prefix of it from `--rec` on, since `--re` names two options
+const isRecursiveLongFlag = (token: string) => token.startsWith("--rec") && "recursive".startsWith(token.slice(2));
 
-// The ls flag recurses on the capital R alone, since its lowercase r sorts in reverse
-const isUpperRecursiveFlag = (token: string) => /^-[a-zA-Z]*R/u.test(token);
+// A flag group holding the scan's recursive letter, or the long form: grep takes r or R (`-R`, `-rn`, `-lR`), and ls takes
+// The capital R alone, since its lowercase r sorts in reverse
+const GREP_RECURSIVE_FLAG_REGEX = /^-[a-zA-Z]*[rR]/u;
+const LS_RECURSIVE_FLAG_REGEX = /^-[a-zA-Z]*R/u;
+const isRecursiveFlag = (token: string, shortFlagRegex: RegExp) =>
+  shortFlagRegex.test(token) || isRecursiveLongFlag(token);
 
 // A flag that gives grep its pattern as the next argument, or as the rest of the same one
 const SEPARATE_PATTERN_FLAG_REGEX = /^-[a-zA-Z]*[ef]$|^--(?:regexp|file)$/u;
@@ -55,9 +59,13 @@ const readGrepArguments = (args: string[]): { flags: string[]; paths: string[] }
 const getScanRefusal = (name: string, args: string[]): string | undefined => {
   if (name === "find" || name === "du") return args.some((arg) => isRoot(arg)) ? name : undefined;
   if (name === "ls")
-    return args.some((arg) => isUpperRecursiveFlag(arg)) && args.some((arg) => isRoot(arg)) ? name : undefined;
+    return args.some((arg) => isRecursiveFlag(arg, LS_RECURSIVE_FLAG_REGEX)) && args.some((arg) => isRoot(arg))
+      ? name
+      : undefined;
   const { flags, paths } = readGrepArguments(args);
-  return flags.some((flag) => isRecursiveFlag(flag)) && paths.some((path) => isRoot(path)) ? name : undefined;
+  return flags.some((flag) => isRecursiveFlag(flag, GREP_RECURSIVE_FLAG_REGEX)) && paths.some((path) => isRoot(path))
+    ? name
+    : undefined;
 };
 
 const getScanCommandRefusal: CommandRefusal = (name, args) =>

package/src/services/ward/readRecords.ts

@@ -0,0 +1,9 @@
+import type { EngineInterface } from "claude-code";
+
+import type { WardRecord } from "../../../types";
+
+import { parseRecords } from "./parseRecords";
+
+// The records as the file holds them, none when it does not yet exist
+export const readRecords = async ($: EngineInterface, path: string): Promise<Record<string, WardRecord>> =>
+  (await $.fs.exists(path)) ? parseRecords(await $.fs.read(path)) : {};

package/src/services/ward/registerWard.ts

@@ -2,26 +2,16 @@ import type { EngineInterface, On, ToolCallResult } from "claude-code";
 
 import { atom, read } from "claude-code";
 
-import type { WardRecord } from "../../../types";
-
 import { WardAnswer, WardAnswers } from "../../models/WardAnswer";
 import { InitialState } from "../InitialState";
 import { getCollidingRecord } from "./getCollidingRecord";
-import { getRecordsPath } from "./getRecordsPath";
 import { getRecordsWithEdit } from "./getRecordsWithEdit";
-import { parseRecords } from "./parseRecords";
+import { readRecords } from "./readRecords";
+import { resolveRecordsPath } from "./resolveRecordsPath";
 
 const enabledModsAtom = atom({ key: "enabledMods", plugin: "genshin-mods" } as const, InitialState.enabledMods);
 const AGE_FORMATTER = new Intl.RelativeTimeFormat("en", { numeric: "auto" });
 
-// The records file under the person's home folder, the home read from this file's own environment lookup
-const resolveRecordsPath = async ($: EngineInterface): Promise<string> =>
-  getRecordsPath((await $.env.get("HOME")) ?? (await $.env.get("USERPROFILE")) ?? "");
-
-// The records as the file holds them, none when it does not yet exist
-const readRecords = async ($: EngineInterface, path: string): Promise<Record<string, WardRecord>> =>
-  (await $.fs.exists(path)) ? parseRecords(await $.fs.read(path)) : {};
-
 // Before the edit: the question, when another session's record calls for one. After it: the edit recorded
 const ward = async (
   $: EngineInterface,

package/src/services/ward/registerWholeIndex.ts

@@ -2,26 +2,16 @@ import type { EngineInterface, On, ToolCallResult } from "claude-code";
 
 import { atom, read } from "claude-code";
 
-import type { WardRecord } from "../../../types";
-
 import { InitialState } from "../InitialState";
 import { joinDirectory } from "../shell/joinDirectory";
 import { getCheckoutPeer } from "./getCheckoutPeer";
 import { getParentFolder } from "./getParentFolder";
-import { getRecordsPath } from "./getRecordsPath";
 import { getWholeIndexDenyReason, getWholeIndexRefusal } from "./getWholeIndexRefusal";
-import { parseRecords } from "./parseRecords";
+import { readRecords } from "./readRecords";
+import { resolveRecordsPath } from "./resolveRecordsPath";
 
 const enabledModsAtom = atom({ key: "enabledMods", plugin: "genshin-mods" } as const, InitialState.enabledMods);
 
-// The records file under the person's home folder, the home read from this file's own environment lookup
-const resolveRecordsPath = async ($: EngineInterface): Promise<string> =>
-  getRecordsPath((await $.env.get("HOME")) ?? (await $.env.get("USERPROFILE")) ?? "");
-
-// The records as the file holds them, none when it does not yet exist
-const readRecords = async ($: EngineInterface, path: string): Promise<Record<string, WardRecord>> =>
-  (await $.fs.exists(path)) ? parseRecords(await $.fs.read(path)) : {};
-
 // The nearest folder at or above `folder` holding a `.git`, a worktree's file included, undefined when none does
 const findCheckoutRoot = async ($: EngineInterface, folder: string): Promise<string | undefined> => {
   if (await $.fs.exists(`${folder}/.git`)) return folder;

package/src/services/ward/resolveRecordsPath.ts

@@ -0,0 +1,7 @@
+import type { EngineInterface } from "claude-code";
+
+import { getRecordsPath } from "./getRecordsPath";
+
+// The records file under the person's home folder, the home read from the engine's environment lookup
+export const resolveRecordsPath = async ($: EngineInterface): Promise<string> =>
+  getRecordsPath((await $.env.get("HOME")) ?? (await $.env.get("USERPROFILE")) ?? "");