agent-harness-kit 0.10.0 → 0.10.2

package/.claude-plugin/marketplace.json

@@ -11,9 +11,9 @@
       "source": {
         "source": "github",
         "repo": "tuanle96/agent-harness-kit",
-        "ref": "v0.10.0"
+        "ref": "v0.10.2"
       },
-      "version": "0.10.0",
+      "version": "0.10.2",
       "description": "Solo-dev harness engineering kit — layered architecture, GC ritual, structural tests, review subagents.",
       "category": "development",
       "keywords": [

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-harness-kit",
-  "version": "0.10.0",
+  "version": "0.10.2",
   "description": "Solo-dev harness engineering kit — layered architecture, garbage-collection ritual, structural tests, review subagents. Optimized for Claude Code 2.1+.",
   "author": {
     "name": "Tuan Le"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-harness-kit",
-  "version": "0.10.0",
+  "version": "0.10.2",
   "description": "Solo-dev harness engineering kit for Claude Code. Layered architecture, structural tests, garbage-collection ritual, review subagents — without the enterprise overhead.",
   "type": "module",
   "bin": {

package/src/core/upgrade.mjs

@@ -20,6 +20,7 @@ import {
   pathForStack,
   buildContext,
   mergeHooksIntoSettings,
+  mergeStatusLineIntoSettings,
   USER_OWNED_FILES as USER_OWNED_FROM_RENDERER,
   EXEC_BITS,
   SUPPORTED_HUMAN_LANGS,
@@ -313,6 +314,16 @@ export async function upgrade({ cwd, kitVersion, yes }) {
     }
   }
 
+  // v0.8 — statusLine injection. Mirrors renderAll's tail (render-templates.mjs:474-480).
+  // Idempotent; never clobbers a user-customised statusLine.command pointing elsewhere.
+  if (existsSync(resolve(cwd, "scripts/statusline.mjs"))) {
+    const sl = await mergeStatusLineIntoSettings(cwd);
+    if (sl.changed) {
+      lockfile.files[".claude/settings.json"] = sha256(sl.rawContent);
+      console.log(pc.dim(`  ${pc.green("~")} .claude/settings.json (statusLine merged)`));
+    }
+  }
+
   lockfile.version = kitVersion;
   await writeFile(lockPath, JSON.stringify(lockfile, null, 2) + "\n");
 

package/src/templates/.claude/skills/inspect-module/scripts/module-summary.mjs

@@ -83,17 +83,41 @@ function outboundDeps(target) {
   return [...out].slice(0, 50);
 }
 
-function inboundDeps(target) {
+function inboundDeps(target, cfg) {
   const relTarget = relative(ROOT, resolve(ROOT, target));
   const name = relTarget.split("/").pop().replace(/\.[a-z]+$/i, "");
   if (!name) return [];
   const seen = new Set();
+  const patterns = [];
+
+  // Standard pattern: import/from/require referencing the directory name.
+  // Works for TS/JS/Python where the import path mirrors the dir name.
+  patterns.push(
+    new RegExp(`(import|from|require\\().*['"][^'"]*${name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}`),
+  );
+
+  // Rust workspace pattern: when domain has `useIdentPattern` (e.g.
+  // "unibot_{layer}"), the crate is `use`d under its layer-derived ident
+  // — NOT the dir name. For `crates/unibot-types/` with pattern
+  // "unibot_{layer}", we should also match `use unibot_types::`. Without
+  // this branch the inbound list silently misses every workspace caller.
+  const layerInfo = whichLayer(target, cfg);
+  if (layerInfo) {
+    const domain = (cfg?.domains || []).find((d) => (d.name || "default") === layerInfo.domain);
+    if (domain?.useIdentPattern) {
+      const ident = domain.useIdentPattern.replace("{layer}", layerInfo.layer);
+      const escaped = ident.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+      patterns.push(new RegExp(`\\b(?:pub\\s+)?use\\s+${escaped}\\b`));
+    }
+  }
+
   // Search the whole project root for references back to the target
   // module. Filter out self-references.
-  const re = new RegExp(`(import|from|require\\().*['"][^'"]*${name.replace(/[.*+?^${}()|[\\\]\\\\]/g, "\\\\$&")}`);
-  for (const line of scan(".", re)) {
-    const m = line.match(/^([^:]+):\d+:/);
-    if (m && m[1] !== relTarget && !m[1].endsWith(`/${relTarget}`)) seen.add(m[1]);
+  for (const re of patterns) {
+    for (const line of scan(".", re)) {
+      const m = line.match(/^([^:]+):\d+:/);
+      if (m && m[1] !== relTarget && !m[1].startsWith(`${relTarget}/`)) seen.add(m[1]);
+    }
   }
   return [...seen].slice(0, 30);
 }
@@ -103,14 +127,22 @@ function readLayers() {
   catch { return null; }
 }
 
+// Resolve the layer for a module path. Honors `layerDirPattern` on the
+// domain so workspaces that prefix layer directories (e.g. Rust workspace
+// `crates/unibot-types/` with `layerDirPattern: "unibot-{layer}"`) match
+// correctly. Without this, paths with custom prefixes would silently fail
+// to match and return layer:null — the bug that ships /inspect-module's
+// most useful columns blank.
 function whichLayer(target, cfg) {
   if (!cfg?.domains) return null;
   const rel = relative(ROOT, resolve(ROOT, target));
   for (const d of cfg.domains) {
     if (!d?.layers || !d.root) continue;
+    const pattern = d.layerDirPattern || "{layer}";
     for (const layer of d.layers) {
-      const prefix = `${d.root}/${layer}/`;
-      if (rel.startsWith(prefix) || rel === `${d.root}/${layer}`) {
+      const dirName = pattern.replace("{layer}", layer);
+      const prefix = `${d.root}/${dirName}/`;
+      if (rel.startsWith(prefix) || rel === `${d.root}/${dirName}`) {
         return { domain: d.name || "default", layer };
       }
     }
@@ -135,7 +167,7 @@ function main() {
     layer: whichLayer(target, cfg),
     exports: listExports(target),
     outbound: outboundDeps(target),
-    inbound: inboundDeps(target),
+    inbound: inboundDeps(target, cfg),
     recent: recentCommits(target),
   };
   process.stdout.write(JSON.stringify(out, null, 2) + "\n");

package/src/templates/scripts/session-end.sh.hbs

@@ -4,13 +4,17 @@
 # Claude Code docs).
 #
 # Output line shape:
-#   YYYY-MM-DD HH:MM | session_end | <reason> | <branch> | <sha>
+#   YYYY-MM-DD HH:MM | session_end | <reason> | <branch> | <sha> | <session_id>
 #
 # Example:
-#   2026-05-16 19:00 | session_end | clear | main | abc1234
+#   2026-05-16 19:00 | session_end | clear | main | abc1234 | sess_abc123
 #
 # Reasons (per Claude Code docs): clear, resume, logout, prompt_input_exit,
 # bypass_permissions_disabled, other.
+#
+# Dedup: a line is only appended when (session_id, reason) differs from the
+# most recent matching entry in PROGRESS.md. Prevents the duplicate-spam
+# bug where Claude Code fires SessionEnd more than once on a single teardown.
 set -eo pipefail
 
 INPUT=$(cat)
@@ -30,10 +34,21 @@ jp() {
   fi
 }
 
-REASON="unknown"
+REASON=""
+SESSION_ID=""
 if have_jp; then
-  REASON=$(echo "$INPUT" | jp '.end_reason // "unknown"' 2>/dev/null || echo "unknown")
+  # Fallback chain: prefer .end_reason (current Claude Code key), accept
+  # .reason as a legacy synonym. Done as two separate jp calls because the
+  # Node-fallback (json-pick.mjs) only supports a single `// default` per
+  # expression — chaining `// .reason //` would parse-fail there.
+  REASON=$(echo "$INPUT" | jp '.end_reason // ""' 2>/dev/null || echo "")
+  if [ -z "$REASON" ] || [ "$REASON" = "null" ]; then
+    REASON=$(echo "$INPUT" | jp '.reason // ""' 2>/dev/null || echo "")
+  fi
+  SESSION_ID=$(echo "$INPUT" | jp '.session_id // ""' 2>/dev/null || echo "")
+  [ "$SESSION_ID" = "null" ] && SESSION_ID=""
 fi
+[ -z "$REASON" ] || [ "$REASON" = "null" ] && REASON="unknown"
 
 BR="(no-git)"
 SHA="(no-git)"
@@ -44,7 +59,21 @@ fi
 
 mkdir -p .harness
 TS=$(date +"%Y-%m-%d %H:%M")
-echo "$TS | session_end | $REASON | $BR | $SHA" >> .harness/PROGRESS.md
+LINE="$TS | session_end | $REASON | $BR | $SHA | $SESSION_ID"
+
+# Idempotency guard: when the SessionEnd hook fires twice on the same
+# teardown (Claude Code sometimes emits both `clear` and a follow-up
+# `prompt_input_exit`, or repeats the same reason), drop the duplicate
+# rather than spamming PROGRESS.md. Match on (session_id, reason): if
+# both are present in the most recent entry for this session, skip.
+DEDUP_KEY="| $REASON | $BR | $SHA | $SESSION_ID"
+if [ -f .harness/PROGRESS.md ] && \
+   [ -n "$SESSION_ID" ] && \
+   tail -n 5 .harness/PROGRESS.md 2>/dev/null | grep -qF "$DEDUP_KEY"; then
+  : # duplicate within the last 5 entries — skip silently
+else
+  echo "$LINE" >> .harness/PROGRESS.md
+fi
 
 # Rollup side-car — writes a JSONL record to .harness/telemetry.jsonl.
 # Best-effort: never blocks the cleanup-only SessionEnd contract.