oh-my-customcode 0.87.3 → 0.88.1

package/dist/cli/index.js

@@ -2148,18 +2148,41 @@ __export(exports_registry, {
   registerProject: () => registerProject,
   readRegistry: () => readRegistry,
   migrateFromLockfiles: () => migrateFromLockfiles,
+  isTempPath: () => isTempPath,
   cleanRegistry: () => cleanRegistry,
   _setRegistryDirForTesting: () => _setRegistryDirForTesting
 });
 import { mkdir, readFile, writeFile } from "node:fs/promises";
-import { homedir } from "node:os";
-import { basename, join, resolve } from "node:path";
+import { homedir, tmpdir } from "node:os";
+import { basename, join, resolve, sep } from "node:path";
 function _setRegistryDirForTesting(dir) {
   _registryDirOverride = dir;
 }
+function isTempPath(projectPath) {
+  const normalized = resolve(projectPath);
+  const candidates = new Set;
+  candidates.add(resolve(tmpdir()));
+  for (const envKey of ["TMPDIR", "TMP", "TEMP"]) {
+    const value = process.env[envKey];
+    if (value)
+      candidates.add(resolve(value));
+  }
+  candidates.add("/tmp");
+  candidates.add("/var/tmp");
+  candidates.add("/var/folders");
+  for (const candidate of candidates) {
+    if (normalized === candidate || normalized.startsWith(candidate + sep)) {
+      return true;
+    }
+  }
+  return false;
+}
 function registryDir() {
   if (_registryDirOverride !== undefined)
     return _registryDirOverride;
+  const envOverride = process.env.OMCUSTOM_REGISTRY_DIR;
+  if (envOverride)
+    return envOverride;
   const home = process.env.HOME ?? homedir();
   return join(home, ".oh-my-customcode");
 }
@@ -2188,6 +2211,10 @@ async function readRegistry() {
 }
 async function registerProject(projectPath, version) {
   const normalizedPath = resolve(projectPath);
+  if (!process.env.OMCUSTOM_REGISTRY_DIR && _registryDirOverride === undefined) {
+    if (isTempPath(normalizedPath))
+      return;
+  }
   const registry = await readRegistryRaw();
   const existing = registry.projects[normalizedPath];
   const now = new Date().toISOString();
@@ -2211,7 +2238,13 @@ async function cleanRegistry() {
   const registry = await readRegistryRaw();
   const paths = Object.keys(registry.projects);
   let removed = 0;
+  const purgeTempPaths = !process.env.OMCUSTOM_REGISTRY_DIR && _registryDirOverride === undefined;
   for (const projectPath of paths) {
+    if (purgeTempPaths && isTempPath(projectPath)) {
+      delete registry.projects[projectPath];
+      removed++;
+      continue;
+    }
     try {
       await fsAccess(projectPath);
     } catch {
@@ -2301,7 +2334,7 @@ var init_package = __esm(() => {
     workspaces: [
       "packages/*"
     ],
-    version: "0.87.3",
+    version: "0.88.1",
     description: "Batteries-included agent harness for Claude Code",
     type: "module",
     bin: {
@@ -6052,10 +6085,10 @@ var require_resolve_block_map = __commonJS((exports) => {
     let offset = bm.offset;
     let commentEnd = null;
     for (const collItem of bm.items) {
-      const { start, key, sep, value } = collItem;
+      const { start, key, sep: sep2, value } = collItem;
       const keyProps = resolveProps.resolveProps(start, {
         indicator: "explicit-key-ind",
-        next: key ?? sep?.[0],
+        next: key ?? sep2?.[0],
         offset,
         onError,
         parentIndent: bm.indent,
@@ -6069,7 +6102,7 @@ var require_resolve_block_map = __commonJS((exports) => {
           else if ("indent" in key && key.indent !== bm.indent)
             onError(offset, "BAD_INDENT", startColMsg);
         }
-        if (!keyProps.anchor && !keyProps.tag && !sep) {
+        if (!keyProps.anchor && !keyProps.tag && !sep2) {
           commentEnd = keyProps.end;
           if (keyProps.comment) {
             if (map.comment)
@@ -6094,7 +6127,7 @@ var require_resolve_block_map = __commonJS((exports) => {
       ctx.atKey = false;
       if (utilMapIncludes.mapIncludes(ctx, map.items, keyNode))
         onError(keyStart, "DUPLICATE_KEY", "Map keys must be unique");
-      const valueProps = resolveProps.resolveProps(sep ?? [], {
+      const valueProps = resolveProps.resolveProps(sep2 ?? [], {
         indicator: "map-value-ind",
         next: value,
         offset: keyNode.range[2],
@@ -6110,7 +6143,7 @@ var require_resolve_block_map = __commonJS((exports) => {
           if (ctx.options.strict && keyProps.start < valueProps.found.offset - 1024)
             onError(keyNode.range, "KEY_OVER_1024_CHARS", "The : indicator must be at most 1024 chars after the start of an implicit block mapping key");
         }
-        const valueNode = value ? composeNode(ctx, value, valueProps, onError) : composeEmptyNode(ctx, offset, sep, null, valueProps, onError);
+        const valueNode = value ? composeNode(ctx, value, valueProps, onError) : composeEmptyNode(ctx, offset, sep2, null, valueProps, onError);
         if (ctx.schema.compat)
           utilFlowIndentCheck.flowIndentCheck(bm.indent, value, onError);
         offset = valueNode.range[2];
@@ -6196,7 +6229,7 @@ var require_resolve_end = __commonJS((exports) => {
     let comment = "";
     if (end) {
       let hasSpace = false;
-      let sep = "";
+      let sep2 = "";
       for (const token of end) {
         const { source, type } = token;
         switch (type) {
@@ -6210,13 +6243,13 @@ var require_resolve_end = __commonJS((exports) => {
             if (!comment)
               comment = cb;
             else
-              comment += sep + cb;
-            sep = "";
+              comment += sep2 + cb;
+            sep2 = "";
             break;
           }
           case "newline":
             if (comment)
-              sep += source;
+              sep2 += source;
             hasSpace = true;
             break;
           default:
@@ -6256,18 +6289,18 @@ var require_resolve_flow_collection = __commonJS((exports) => {
     let offset = fc.offset + fc.start.source.length;
     for (let i = 0;i < fc.items.length; ++i) {
       const collItem = fc.items[i];
-      const { start, key, sep, value } = collItem;
+      const { start, key, sep: sep2, value } = collItem;
       const props = resolveProps.resolveProps(start, {
         flow: fcName,
         indicator: "explicit-key-ind",
-        next: key ?? sep?.[0],
+        next: key ?? sep2?.[0],
         offset,
         onError,
         parentIndent: fc.indent,
         startOnNewline: false
       });
       if (!props.found) {
-        if (!props.anchor && !props.tag && !sep && !value) {
+        if (!props.anchor && !props.tag && !sep2 && !value) {
           if (i === 0 && props.comma)
             onError(props.comma, "UNEXPECTED_TOKEN", `Unexpected , in ${fcName}`);
           else if (i < fc.items.length - 1)
@@ -6319,8 +6352,8 @@ var require_resolve_flow_collection = __commonJS((exports) => {
           }
         }
       }
-      if (!isMap && !sep && !props.found) {
-        const valueNode = value ? composeNode(ctx, value, props, onError) : composeEmptyNode(ctx, props.end, sep, null, props, onError);
+      if (!isMap && !sep2 && !props.found) {
+        const valueNode = value ? composeNode(ctx, value, props, onError) : composeEmptyNode(ctx, props.end, sep2, null, props, onError);
         coll.items.push(valueNode);
         offset = valueNode.range[2];
         if (isBlock(value))
@@ -6332,7 +6365,7 @@ var require_resolve_flow_collection = __commonJS((exports) => {
         if (isBlock(key))
           onError(keyNode.range, "BLOCK_IN_FLOW", blockMsg);
         ctx.atKey = false;
-        const valueProps = resolveProps.resolveProps(sep ?? [], {
+        const valueProps = resolveProps.resolveProps(sep2 ?? [], {
           flow: fcName,
           indicator: "map-value-ind",
           next: value,
@@ -6343,8 +6376,8 @@ var require_resolve_flow_collection = __commonJS((exports) => {
         });
         if (valueProps.found) {
           if (!isMap && !props.found && ctx.options.strict) {
-            if (sep)
-              for (const st of sep) {
+            if (sep2)
+              for (const st of sep2) {
                 if (st === valueProps.found)
                   break;
                 if (st.type === "newline") {
@@ -6361,7 +6394,7 @@ var require_resolve_flow_collection = __commonJS((exports) => {
           else
             onError(valueProps.start, "MISSING_CHAR", `Missing , or : between ${fcName} items`);
         }
-        const valueNode = value ? composeNode(ctx, value, valueProps, onError) : valueProps.found ? composeEmptyNode(ctx, valueProps.end, sep, null, valueProps, onError) : null;
+        const valueNode = value ? composeNode(ctx, value, valueProps, onError) : valueProps.found ? composeEmptyNode(ctx, valueProps.end, sep2, null, valueProps, onError) : null;
         if (valueNode) {
           if (isBlock(value))
             onError(valueNode.range, "BLOCK_IN_FLOW", blockMsg);
@@ -6538,7 +6571,7 @@ var require_resolve_block_scalar = __commonJS((exports) => {
         chompStart = i + 1;
     }
     let value = "";
-    let sep = "";
+    let sep2 = "";
     let prevMoreIndented = false;
     for (let i = 0;i < contentStart; ++i)
       value += lines[i][0].slice(trimIndent) + `
@@ -6556,33 +6589,33 @@ var require_resolve_block_scalar = __commonJS((exports) => {
         indent = "";
       }
       if (type === Scalar.Scalar.BLOCK_LITERAL) {
-        value += sep + indent.slice(trimIndent) + content;
-        sep = `
+        value += sep2 + indent.slice(trimIndent) + content;
+        sep2 = `
 `;
       } else if (indent.length > trimIndent || content[0] === "\t") {
-        if (sep === " ")
-          sep = `
+        if (sep2 === " ")
+          sep2 = `
 `;
-        else if (!prevMoreIndented && sep === `
+        else if (!prevMoreIndented && sep2 === `
 `)
-          sep = `
+          sep2 = `
 
 `;
-        value += sep + indent.slice(trimIndent) + content;
-        sep = `
+        value += sep2 + indent.slice(trimIndent) + content;
+        sep2 = `
 `;
         prevMoreIndented = true;
       } else if (content === "") {
-        if (sep === `
+        if (sep2 === `
 `)
           value += `
 `;
         else
-          sep = `
+          sep2 = `
 `;
       } else {
-        value += sep + content;
-        sep = " ";
+        value += sep2 + content;
+        sep2 = " ";
         prevMoreIndented = false;
       }
     }
@@ -6763,27 +6796,27 @@ var require_resolve_flow_scalar = __commonJS((exports) => {
     if (!match)
       return source;
     let res = match[1];
-    let sep = " ";
+    let sep2 = " ";
     let pos = first.lastIndex;
     line.lastIndex = pos;
     while (match = line.exec(source)) {
       if (match[1] === "") {
-        if (sep === `
+        if (sep2 === `
 `)
-          res += sep;
+          res += sep2;
         else
-          sep = `
+          sep2 = `
 `;
       } else {
-        res += sep + match[1];
-        sep = " ";
+        res += sep2 + match[1];
+        sep2 = " ";
       }
       pos = line.lastIndex;
     }
     const last = /[ \t]*(.*)/sy;
     last.lastIndex = pos;
     match = last.exec(source);
-    return res + sep + (match?.[1] ?? "");
+    return res + sep2 + (match?.[1] ?? "");
   }
   function doubleQuotedValue(source, onError) {
     let res = "";
@@ -7556,14 +7589,14 @@ var require_cst_stringify = __commonJS((exports) => {
       }
     }
   }
-  function stringifyItem({ start, key, sep, value }) {
+  function stringifyItem({ start, key, sep: sep2, value }) {
     let res = "";
     for (const st of start)
       res += st.source;
     if (key)
       res += stringifyToken(key);
-    if (sep)
-      for (const st of sep)
+    if (sep2)
+      for (const st of sep2)
         res += st.source;
     if (value)
       res += stringifyToken(value);
@@ -8693,18 +8726,18 @@ var require_parser = __commonJS((exports) => {
       if (this.type === "map-value-ind") {
         const prev = getPrevProps(this.peek(2));
         const start = getFirstKeyStartProps(prev);
-        let sep;
+        let sep2;
         if (scalar.end) {
-          sep = scalar.end;
-          sep.push(this.sourceToken);
+          sep2 = scalar.end;
+          sep2.push(this.sourceToken);
           delete scalar.end;
         } else
-          sep = [this.sourceToken];
+          sep2 = [this.sourceToken];
         const map = {
           type: "block-map",
           offset: scalar.offset,
           indent: scalar.indent,
-          items: [{ start, key: scalar, sep }]
+          items: [{ start, key: scalar, sep: sep2 }]
         };
         this.onKeyLine = true;
         this.stack[this.stack.length - 1] = map;
@@ -8858,15 +8891,15 @@ var require_parser = __commonJS((exports) => {
               } else if (isFlowToken(it.key) && !includesToken(it.sep, "newline")) {
                 const start2 = getFirstKeyStartProps(it.start);
                 const key = it.key;
-                const sep = it.sep;
-                sep.push(this.sourceToken);
+                const sep2 = it.sep;
+                sep2.push(this.sourceToken);
                 delete it.key;
                 delete it.sep;
                 this.stack.push({
                   type: "block-map",
                   offset: this.offset,
                   indent: this.indent,
-                  items: [{ start: start2, key, sep }]
+                  items: [{ start: start2, key, sep: sep2 }]
                 });
               } else if (start.length > 0) {
                 it.sep = it.sep.concat(start, this.sourceToken);
@@ -9060,13 +9093,13 @@ var require_parser = __commonJS((exports) => {
           const prev = getPrevProps(parent);
           const start = getFirstKeyStartProps(prev);
           fixFlowSeqItems(fc);
-          const sep = fc.end.splice(1, fc.end.length);
-          sep.push(this.sourceToken);
+          const sep2 = fc.end.splice(1, fc.end.length);
+          sep2.push(this.sourceToken);
           const map = {
             type: "block-map",
             offset: fc.offset,
             indent: fc.indent,
-            items: [{ start, key: fc, sep }]
+            items: [{ start, key: fc, sep: sep2 }]
           };
           this.onKeyLine = true;
           this.stack[this.stack.length - 1] = map;
@@ -9315,7 +9348,7 @@ __export(exports_fs, {
   copyDirectory: () => copyDirectory,
   calculateChecksum: () => calculateChecksum
 });
-import { dirname as dirname2, isAbsolute, join as join3, relative, resolve as resolve2, sep } from "node:path";
+import { dirname as dirname2, isAbsolute, join as join3, relative, resolve as resolve2, sep as sep2 } from "node:path";
 import { fileURLToPath } from "node:url";
 function validatePreserveFilePath(filePath, projectRoot) {
   if (!filePath || filePath.trim() === "") {
@@ -9411,7 +9444,7 @@ function shouldSkipPath(destPath, destRoot, skipPaths) {
   for (const skipPath of skipPaths) {
     if (skipPath.endsWith("/")) {
       const dirPath = skipPath.slice(0, -1);
-      if (relativePath === dirPath || relativePath.startsWith(dirPath + sep)) {
+      if (relativePath === dirPath || relativePath.startsWith(dirPath + sep2)) {
         return true;
       }
     } else {
@@ -9574,7 +9607,7 @@ __export(exports_projects, {
   default: () => projects_default
 });
 import { homedir as homedir3 } from "node:os";
-import { basename as basename4, join as join10, sep as sep2 } from "node:path";
+import { basename as basename4, join as join10, sep as sep3 } from "node:path";
 async function readLockFile(projectDir) {
   const lockFilePath = join10(projectDir, ".omcustom.lock.json");
   try {
@@ -9611,10 +9644,10 @@ async function getTemplateVersion() {
 function matchesSearchPaths(projectPath, searchPaths) {
   if (!searchPaths || searchPaths.length === 0)
     return true;
-  return searchPaths.some((searchPath) => projectPath === searchPath || projectPath.startsWith(searchPath + sep2));
+  return searchPaths.some((searchPath) => projectPath === searchPath || projectPath.startsWith(searchPath + sep3));
 }
 function isUnderHome(projectPath, home) {
-  return projectPath.startsWith(home + sep2) || projectPath === home;
+  return projectPath.startsWith(home + sep3) || projectPath === home;
 }
 function sortProjects(projects) {
   return projects.sort((a, b) => {
@@ -23791,10 +23824,10 @@ class Interpolator {
     let value;
     let clonedOptions;
     const handleHasOptions = (key, inheritedOptions) => {
-      const sep = this.nestingOptionsSeparator;
-      if (!key.includes(sep))
+      const sep2 = this.nestingOptionsSeparator;
+      if (!key.includes(sep2))
         return key;
-      const c = key.split(new RegExp(`${regexEscape(sep)}[ ]*{`));
+      const c = key.split(new RegExp(`${regexEscape(sep2)}[ ]*{`));
       let optionsString = `{${c[1]}`;
       key = c[0];
       optionsString = this.interpolate(optionsString, clonedOptions);
@@ -23812,7 +23845,7 @@ class Interpolator {
           };
       } catch (e) {
         this.logger.warn(`failed parsing options string in nesting for key ${key}`, e);
-        return `${key}${sep}${optionsString}`;
+        return `${key}${sep2}${optionsString}`;
       }
       if (clonedOptions.defaultValue && clonedOptions.defaultValue.includes(this.prefix))
         delete clonedOptions.defaultValue;
@@ -31465,8 +31498,10 @@ async function checkCliVersion(checkFn) {
 }
 function exitWithChildStatus(child) {
   if (child.signal) {
-    const signalNumber = osConstants.signals[child.signal] ?? 0;
-    process.exit(128 + signalNumber);
+    const signalNumber = osConstants.signals[child.signal];
+    if (signalNumber !== undefined) {
+      process.exit(128 + signalNumber);
+    }
   }
   process.exit(child.status ?? 1);
 }

package/dist/index.js

@@ -299,18 +299,41 @@ __export(exports_registry, {
   registerProject: () => registerProject,
   readRegistry: () => readRegistry,
   migrateFromLockfiles: () => migrateFromLockfiles,
+  isTempPath: () => isTempPath,
   cleanRegistry: () => cleanRegistry,
   _setRegistryDirForTesting: () => _setRegistryDirForTesting
 });
 import { mkdir, readFile, writeFile } from "node:fs/promises";
-import { homedir } from "node:os";
-import { basename as basename3, join as join6, resolve as resolve2 } from "node:path";
+import { homedir, tmpdir } from "node:os";
+import { basename as basename3, join as join6, resolve as resolve2, sep as sep2 } from "node:path";
 function _setRegistryDirForTesting(dir2) {
   _registryDirOverride = dir2;
 }
+function isTempPath(projectPath) {
+  const normalized = resolve2(projectPath);
+  const candidates = new Set;
+  candidates.add(resolve2(tmpdir()));
+  for (const envKey of ["TMPDIR", "TMP", "TEMP"]) {
+    const value = process.env[envKey];
+    if (value)
+      candidates.add(resolve2(value));
+  }
+  candidates.add("/tmp");
+  candidates.add("/var/tmp");
+  candidates.add("/var/folders");
+  for (const candidate of candidates) {
+    if (normalized === candidate || normalized.startsWith(candidate + sep2)) {
+      return true;
+    }
+  }
+  return false;
+}
 function registryDir() {
   if (_registryDirOverride !== undefined)
     return _registryDirOverride;
+  const envOverride = process.env.OMCUSTOM_REGISTRY_DIR;
+  if (envOverride)
+    return envOverride;
   const home = process.env.HOME ?? homedir();
   return join6(home, ".oh-my-customcode");
 }
@@ -339,6 +362,10 @@ async function readRegistry() {
 }
 async function registerProject(projectPath, version) {
   const normalizedPath = resolve2(projectPath);
+  if (!process.env.OMCUSTOM_REGISTRY_DIR && _registryDirOverride === undefined) {
+    if (isTempPath(normalizedPath))
+      return;
+  }
   const registry = await readRegistryRaw();
   const existing = registry.projects[normalizedPath];
   const now = new Date().toISOString();
@@ -362,7 +389,13 @@ async function cleanRegistry() {
   const registry = await readRegistryRaw();
   const paths = Object.keys(registry.projects);
   let removed = 0;
+  const purgeTempPaths = !process.env.OMCUSTOM_REGISTRY_DIR && _registryDirOverride === undefined;
   for (const projectPath of paths) {
+    if (purgeTempPaths && isTempPath(projectPath)) {
+      delete registry.projects[projectPath];
+      removed++;
+      continue;
+    }
     try {
       await fsAccess(projectPath);
     } catch {
@@ -1974,7 +2007,7 @@ var package_default = {
   workspaces: [
     "packages/*"
   ],
-  version: "0.87.3",
+  version: "0.88.1",
   description: "Batteries-included agent harness for Claude Code",
   type: "module",
   bin: {

package/package.json

@@ -3,7 +3,7 @@
   "workspaces": [
     "packages/*"
   ],
-  "version": "0.87.3",
+  "version": "0.88.1",
   "description": "Batteries-included agent harness for Claude Code",
   "type": "module",
   "bin": {

package/templates/.claude/rules/MUST-completion-verification.md

@@ -10,7 +10,7 @@ Before declaring any task `[Done]`, verify completion against task-type-specific
 
 | Task Type | REQUIRED Verification Before [Done] |
 |-----------|-------------------------------------|
-| Release | All issues closed, version bumped, PR merged, GitHub Release created |
+| Release | All issues closed, version bumped, PR merged, GitHub Release created; **External automation verified**: `.github/workflows/` listed AND `gh run list --limit 10` checked for auto-publish workflows |
 | Implementation | Code compiles/passes lint, tests pass (if exist), no TODO markers left |
 | Documentation | Links valid, counts accurate, cross-references updated |
 | Git Operations | Operation succeeded (check exit code), working tree clean |
@@ -28,6 +28,19 @@ Before [Done]: (1) Verify ACTUAL outcome not just attempt — "ran command" ≠
 4. Would I bet $100 this is truly complete? YES: Declare [Done] / NO: Identify uncertain and verify
 -->
 
+## Subagent Self-Report Verification
+
+Subagents often report failures as "pre-existing", "baseline", or "unchanged". These claims MUST be verified against the base branch before acceptance.
+
+| Subagent Claim | Required Verification |
+|----------------|----------------------|
+| "X test already failing on base" | `git stash && git checkout {base} && run test X && compare` |
+| "This warning is pre-existing" | `git log -S "warning-text" {base}` or run on clean checkout |
+| "File was unchanged" | `git diff {base}..HEAD -- {file}` |
+| "Dependency issue not from this PR" | `git show {base}:package.json` compare |
+
+Never accept "pre-existing" without direct base-branch evidence. A false "pre-existing" claim can mask a regression introduced by the current change.
+
 ## Common False Completion Patterns
 
 | Pattern | Reality | Fix |
@@ -37,6 +50,8 @@ Before [Done]: (1) Verify ACTUAL outcome not just attempt — "ran command" ≠
 | "PR created" | CI not checked | Wait for CI, verify green |
 | "Issue closed" | Related issues not updated | Check parent epic, cross-refs |
 | "Tests pass" | Only ran subset | Run full test suite |
+| "Waiting for manual publish" | External CI/CD auto-publishes on merge | Check `.github/workflows/` BEFORE assuming manual step |
+| "Subagent said pre-existing" | Claim not verified against base branch | Run test on base branch, compare directly |
 
 ## Completion Contract Format
 
@@ -58,6 +73,24 @@ Then at completion:
 └── ✓ Criterion N: {evidence}
 ```
 
+## Autonomous Mode Entry Checklist
+
+When entering autonomous mode (user grants extended execution without per-step confirmation), perform this inventory BEFORE first action:
+
+1. **Workflow inventory**: `ls .github/workflows/` — identify auto-publish, auto-tag, release, docs-sync, CI workflows
+2. **Recent runs**: `gh run list --limit 10` — check success/failure patterns of automated workflows
+3. **External publish targets**: Check if npm/PyPI/Docker Hub/GitHub Releases are auto-triggered on merge
+4. **Manual intervention points**: Identify which steps require human approval vs. fully automated
+5. **Cross-reference with task**: Which workflows will the planned work trigger?
+
+Record findings in session context. Failure to inventory automation is a R020 violation (unknown external state = unverifiable completion).
+
+### Cross-reference
+
+Related memory records:
+- `feedback_github_workflows_inventory.md` — original incident (v0.87.2~v0.88.0 session)
+- `feedback_subagent_pre_existing_claims.md` — subagent false-positive pattern
+
 ## Integration
 
 | Rule | Interaction |

package/templates/.claude/rules/MUST-intent-transparency.md

@@ -37,6 +37,34 @@ Display reasoning when routing to agents. Users must always know which agent was
 
 Users can specify agent directly with `@{agent-name} {command}`. Override bypasses detection.
 
+## User Directive Persistence
+
+When a user explicitly names a tool, skill, or workflow (e.g., "use /pipeline auto-dev", "always run tests with bun test"), this preference persists for the entire session — including after autonomous mode transitions.
+
+### Persistence Triggers
+
+| User Statement Pattern | Persistence Scope |
+|------------------------|-------------------|
+| "use X for development" | Entire session |
+| "always / every time" | Entire session |
+| "from now on" | Entire session + memory save candidate |
+| "for this task" | Current task only |
+| Named slash command | Subsequent similar invocations |
+
+### Cycle Start Self-Check
+
+At the start of every work cycle (issue, task, release, or autonomous sub-loop):
+1. Review recent user messages in the conversation
+2. Identify any named tool/skill/workflow directives
+3. Apply those directives unless explicitly rescinded
+4. If unsure whether a directive applies, default to the stated preference
+
+**Anti-pattern**: Treating autonomous mode as a clean slate that discards earlier user preferences. Autonomous mode means "continue without per-step confirmation" — NOT "reset user directives".
+
+### Cross-reference
+
+- Related memory: session v0.87.2~v0.88.0 (issue #869) — `/pipeline auto-dev` preference was lost after autonomous mode transition
+
 ## Agent Triggers
 
 Defined in `.claude/skills/intent-detection/patterns/agent-triggers.yaml`. Each agent has keywords, file patterns, actions, and base confidence.

package/templates/.claude/rules/SHOULD-memory-integration.md

@@ -277,6 +277,30 @@ User Model data feeds into intent-detection (R015) and routing skill confidence
 
 -->
 
+## Mid-Session Immediate Save
+
+Save memory IMMEDIATELY upon surprising discovery — do not defer to session end.
+
+| Trigger | Action | Rationale |
+|---------|--------|-----------|
+| Repeated pattern observed (2nd time) | Save `feedback_*.md` now | Pattern will recur within session |
+| Unexpected tool behavior / workaround | Save `feedback_*.md` now | Session state defense |
+| Subagent false-positive detected | Save `feedback_*.md` now | Prevent repeat in same session |
+| User correction / feedback | Save `feedback_*.md` now | Honor correction immediately |
+
+### Why Immediate?
+
+Session-end saves lose context: by the time the session ends, multiple discoveries have compounded and nuance is lost. Immediate saves preserve the exact trigger context that makes the memory actionable.
+
+**Anti-pattern**: "I'll batch all learnings at session end" — by then you'll have forgotten WHY each one mattered, and further violations may have occurred using the un-saved pattern.
+
+### Cross-reference
+
+Related records from session v0.87.2~v0.88.0 (issue #869):
+- `feedback_subagent_pre_existing_claims.md`
+- `feedback_github_workflows_inventory.md`
+- `feedback_bun_mock_module.md`
+
 ## Session-End Auto-Save
 
 ### Trigger

package/templates/manifest.json

@@ -1,5 +1,5 @@
 {
-  "version": "0.87.3",
+  "version": "0.88.1",
   "lastUpdated": "2026-04-14T00:00:00.000Z",
   "components": [
     {