omo-memory 0.1.15 → 0.1.16

package/README.md

@@ -56,6 +56,8 @@ npm link
 omo-memory init
 ```
 
+Install and update run a local Codex cleanup migration through `postinstall`. It removes stale `omo-memory@islee23520` hook state and legacy hook files from `~/.codex` when they exist; it does not touch unrelated OMO/LazyCodex hooks.
+
 ## MCP registration
 
 Register the same MCP server in every host that should read/write the current project's memory DB.
@@ -102,8 +104,7 @@ The response contains a new `sessionId` and project metadata only. It deliberate
 }
 ```
 
-This is local routing, not transcript scraping. OMO Memory does not automatically read full Codex or Grok transcripts. Hooks should record concise user actions, decisions, QA evidence, and handoffs; they should retrieve memory only when the user explicitly asks for OMO Memory or when the current user input can be matched to recorded intent.
-The packaged `scripts/omo-memory-user-prompt.mjs` helper is the supported UserPromptSubmit hook target for adapters that can invoke a command with the hook payload on stdin. It records only the current user prompt as a redacted `user_prompt` event, ignores assistant output, and exits successfully without blocking the host when OMO Memory is unavailable.
+This is local routing, not transcript scraping. OMO Memory does not automatically read full Codex or Grok transcripts. Adapters should record concise user actions, decisions, QA evidence, and handoffs through the CLI or MCP tools; they should retrieve memory only when the user explicitly asks for OMO Memory or when the current user input can be matched to recorded intent.
 
 Use explicit retrieval for memory reads:
 
@@ -135,7 +136,7 @@ Initial stdio MCP tools:
 
 ## Updates
 
-Installed CLI commands automatically launch a quiet background `npm install -g omo-memory@latest` at most once per day. MCP startup does not run the updater, so stdio handshakes stay clean.
+Installed CLI commands automatically launch a quiet background `npm install -g omo-memory@latest` at most once per day. MCP startup does not run the updater, so stdio handshakes stay clean. The package `postinstall` cleanup also runs during these updates and removes legacy Codex `omo-memory@islee23520` hook registrations.
 
 Manual update:
 

package/docs/adapter-integration.md

@@ -105,7 +105,7 @@ The tool returns:
 - `sessionId`: the new session row for subsequent event/handoff writes.
 - `project`: the current git/project namespace.
 
-During the session, hooks should write concise user-action summaries, task state, decisions, and evidence with the returned `sessionId`:
+During the session, adapters should write concise user-action summaries, task state, decisions, and evidence with the returned `sessionId`:
 
 ```json
 {
@@ -125,7 +125,7 @@ Retrieval is opt-in or intent-gated:
 - Use `memory_recent_events` only when the user explicitly asks for recent OMO Memory context.
 - Use `memory_recall_events` when the current user input has a concrete query that can be matched to recorded summaries, decisions, or evidence.
 - Do not automatically attach the last session to every user prompt.
-- To preserve user intent across sessions, adapters may invoke the packaged `scripts/omo-memory-user-prompt.mjs` UserPromptSubmit helper with the hook payload on stdin. The helper records only the current user prompt as a redacted `user_prompt` event and ignores assistant output.
+- OMO Memory no longer installs Codex hooks. New package installs and updates run a `postinstall` cleanup that removes stale `omo-memory@islee23520` hook state and legacy hook files from `~/.codex`.
 
 Use these tools:
 
@@ -144,7 +144,7 @@ Use these tools:
 - `memory_global_list`
 
 
-CLI updates: normal CLI commands automatically launch a quiet background `npm install -g omo-memory@latest` at most once per day. MCP startup intentionally does not auto-update because stdout/stderr must remain reserved for the protocol. Hosts that need pinned installs should set `OMO_MEMORY_AUTO_UPDATE=0` in the CLI environment.
+CLI updates: normal CLI commands automatically launch a quiet background `npm install -g omo-memory@latest` at most once per day. MCP startup intentionally does not auto-update because stdout/stderr must remain reserved for the protocol. Hosts that need pinned installs should set `OMO_MEMORY_AUTO_UPDATE=0` in the CLI environment. Package install/update also runs the Codex stale-hook cleanup migration through `postinstall`.
 
 Global migration is copy/import only. Adapters may scan for existing project-local `.omo/memory/state.sqlite` databases and import their event/session/handoff ledgers into a user-selected global SQLite file, but they must preserve source DBs and retain source provenance in the global store.
 
@@ -201,4 +201,4 @@ Example QA evidence:
 
 OMO Memory's chronological ledger is authoritative: sessions, events, and handoffs record what happened in a project. Adapters should write concise summaries and evidence references to that ledger, then use explicit `recent`, `recall`, `handoff`, `export`, `purge`, and global event import/list operations.
 
-OMO Memory does not provide automatic ontology extraction, graph visualization, retention scoring, durable-memory promotion, or OpenTUI workflows. Do not build adapter behavior that depends on those unavailable surfaces.
+OMO Memory does not provide automatic ontology extraction, graph visualization, retention scoring, durable-memory promotion, or OpenTUI workflows. Do not build adapter behavior that depends on those unavailable surfaces.

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "omo-memory",
-  "version": "0.1.15",
+  "version": "0.1.16",
   "description": "Host-neutral local SQLite memory and session ledger for OMO adapters, exposed through CLI and MCP.",
   "type": "module",
   "files": [
     "dist",
     "docs",
-    "scripts/omo-memory-user-prompt.mjs",
+    "scripts/postinstall-cleanup.mjs",
     "README.md"
   ],
   "bin": {
@@ -31,13 +31,14 @@
     "format": "biome format --write .",
     "check": "npm run lint && npm run typecheck && npm run smoke",
     "start": "node dist/cli.js",
+    "postinstall": "node scripts/postinstall-cleanup.mjs",
     "prepack": "npm run build",
     "presmoke": "npm run build",
-    "smoke": "npm run smoke:cli && npm run smoke:mcp && npm run smoke:hook && npm run smoke:legacy",
+    "smoke": "npm run smoke:cli && npm run smoke:mcp && npm run smoke:legacy && npm run smoke:codex-hook-migration",
     "smoke:cli": "node scripts/smoke-cli.mjs",
     "smoke:mcp": "node scripts/smoke-mcp.mjs",
-    "smoke:hook": "node scripts/verify-user-prompt-hook.mjs",
     "smoke:legacy": "node scripts/verify-legacy-schema-compat.mjs",
+    "smoke:codex-hook-migration": "node scripts/verify-codex-hook-migration.mjs",
     "mcp": "node dist/cli.js mcp",
     "issue:epic": "node scripts/create-epic-issue.mjs"
   },

package/scripts/postinstall-cleanup.mjs

@@ -0,0 +1,91 @@
+#!/usr/bin/env node
+import { existsSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import { homedir } from "node:os";
+import { join } from "node:path";
+
+const CODEX_ROOT = join(homedir(), ".codex");
+const LEGACY_PLUGIN_ROOT = join(CODEX_ROOT, "local-marketplaces", "islee23520", "plugins", "omo-memory");
+const LEGACY_HOOK_STATE_PREFIX = '[hooks.state."omo-memory@islee23520:';
+const LEGACY_HOOK_ARTIFACTS = [
+  join(LEGACY_PLUGIN_ROOT, "hooks", "hooks.json"),
+  join(LEGACY_PLUGIN_ROOT, "scripts", "omo-memory-user-prompt.mjs"),
+  join(LEGACY_PLUGIN_ROOT, "scripts", "omo-memory-session.mjs"),
+];
+
+function cleanupCodexHooks() {
+  const changed = [];
+  const configPath = join(CODEX_ROOT, "config.toml");
+  if (rewriteCodexConfig(configPath)) changed.push(configPath);
+  if (rewritePluginManifest(join(LEGACY_PLUGIN_ROOT, ".codex-plugin", "plugin.json"))) {
+    changed.push(join(LEGACY_PLUGIN_ROOT, ".codex-plugin", "plugin.json"));
+  }
+  for (const artifact of LEGACY_HOOK_ARTIFACTS) {
+    if (!existsSync(artifact)) continue;
+    rmSync(artifact, { force: true });
+    changed.push(artifact);
+  }
+  return changed;
+}
+
+function rewriteCodexConfig(configPath) {
+  if (!existsSync(configPath)) return false;
+  const before = readFileSync(configPath, "utf8");
+  const after = removeTomlTables(before, (header) => header.startsWith(LEGACY_HOOK_STATE_PREFIX));
+  if (after === before) return false;
+  writeFileSync(configPath, after);
+  return true;
+}
+
+function rewritePluginManifest(manifestPath) {
+  if (!existsSync(manifestPath)) return false;
+  const raw = readFileSync(manifestPath, "utf8");
+  const parsed = JSON.parse(raw);
+  if (!isObject(parsed)) return false;
+  const next = removeHookManifestFields(parsed);
+  const after = `${JSON.stringify(next, null, 2)}\n`;
+  if (after === raw) return false;
+  writeFileSync(manifestPath, after);
+  return true;
+}
+
+function removeHookManifestFields(manifest) {
+  const next = { ...manifest };
+  delete next.hooks;
+  next.keywords = removeStringFromArray(next.keywords, "hooks");
+  if (isObject(next.interface)) {
+    next.interface = {
+      ...next.interface,
+      capabilities: removeStringFromArray(next.interface.capabilities, "Hooks"),
+    };
+  }
+  return next;
+}
+
+function removeStringFromArray(value, removedValue) {
+  if (!Array.isArray(value)) return value;
+  return value.filter((item) => item !== removedValue);
+}
+
+function removeTomlTables(input, shouldRemoveHeader) {
+  const lines = input.split(/\r?\n/);
+  const output = [];
+  let removing = false;
+  for (const line of lines) {
+    if (/^\[[^\]]+/.test(line)) removing = shouldRemoveHeader(line.trim());
+    if (!removing) output.push(line);
+  }
+  return output.join("\n");
+}
+
+function isObject(value) {
+  return value !== null && typeof value === "object" && !Array.isArray(value);
+}
+
+try {
+  const changed = cleanupCodexHooks();
+  process.stdout.write(`${JSON.stringify({ ok: true, migration: "codex-omo-memory-hooks-removed", changed }, null, 2)}\n`);
+} catch (error) {
+  const message = error instanceof Error ? error.message : String(error);
+  process.stdout.write(`${JSON.stringify({ ok: false, migration: "codex-omo-memory-hooks-removed", error: message }, null, 2)}\n`);
+  process.exitCode = 0;
+}

package/scripts/omo-memory-user-prompt.mjs

@@ -1,107 +0,0 @@
-#!/usr/bin/env node
-import { spawnSync } from "node:child_process";
-import { readFileSync } from "node:fs";
-
-const MAX_SUMMARY_CHARS = 4000;
-
-const host = process.env["OMO_MEMORY_HOST"] ?? "unknown";
-const adapter = process.env["OMO_MEMORY_ADAPTER"] ?? "unknown";
-
-const input = readStdin();
-const payload = parseJson(input);
-const prompt = extractPrompt(payload);
-
-if (prompt === null) process.exit(0);
-
-const summary = truncate(prompt.replace(/\s+/g, " ").trim(), MAX_SUMMARY_CHARS);
-if (summary.length === 0) process.exit(0);
-
-const hookSessionId = readString(payload, "sessionId") ?? readString(payload, "session_id");
-const workspaceRoot = readString(payload, "workspaceRoot") ?? readString(payload, "cwd");
-const metadata = {
-  source: "hook",
-  hookEventName: readString(payload, "hookEventName") ?? process.env["GROK_HOOK_EVENT"] ?? process.env["CODEX_HOOK_EVENT"] ?? "UserPromptSubmit",
-  host,
-  adapter,
-  ...(hookSessionId === null ? {} : { hookSessionId }),
-  ...(workspaceRoot === null ? {} : { workspaceRoot }),
-};
-
-const args = ["event", "record", "--type", "user_prompt", "--summary", summary, "--payload-json", JSON.stringify(metadata)];
-
-const result = runOmoMemory(args) ?? runNpx(args);
-if (result === undefined || result.status !== 0) process.exit(0);
-
-function readStdin() {
-  try {
-    return readFileSync(0, "utf8");
-  } catch (error) {
-    if (error instanceof Error) process.exit(0);
-    throw error;
-  }
-}
-
-function parseJson(raw) {
-  if (raw.trim().length === 0) return {};
-  try {
-    const parsed = JSON.parse(raw);
-    return isRecord(parsed) ? parsed : {};
-  } catch (error) {
-    if (error instanceof SyntaxError) return {};
-    throw error;
-  }
-}
-
-function extractPrompt(value) {
-  if (!isRecord(value)) return null;
-  for (const key of ["prompt", "userPrompt", "user_prompt", "message", "text", "input"]) {
-    const direct = readString(value, key);
-    if (direct !== null) return direct;
-  }
-  const nestedPrompt =
-    readNestedString(value, ["toolInput", "prompt"]) ?? readNestedString(value, ["payload", "prompt"]) ?? readNestedString(value, ["data", "prompt"]);
-  if (nestedPrompt !== null) return nestedPrompt;
-  return null;
-}
-
-function readNestedString(value, path) {
-  let cursor = value;
-  for (const key of path) {
-    if (!isRecord(cursor)) return null;
-    cursor = cursor[key];
-  }
-  return typeof cursor === "string" ? cursor : null;
-}
-
-function readString(value, key) {
-  const item = value[key];
-  return typeof item === "string" ? item : null;
-}
-
-function truncate(value, maxLength) {
-  if (value.length <= maxLength) return value;
-  return `${value.slice(0, maxLength - 15)} [TRUNCATED]`;
-}
-
-function runOmoMemory(args) {
-  const command = process.env["OMO_MEMORY_CLI"] ?? "omo-memory";
-  return run(command, args);
-}
-
-function runNpx(args) {
-  return run("npx", ["-y", "omo-memory", ...args]);
-}
-
-function run(command, args) {
-  const result = spawnSync(command, args, {
-    encoding: "utf8",
-    stdio: ["ignore", "pipe", "pipe"],
-    timeout: 5000,
-  });
-  if (result.error?.code === "ENOENT") return undefined;
-  return result;
-}
-
-function isRecord(value) {
-  return value !== null && typeof value === "object" && !Array.isArray(value);
-}