eagle-mem 4.10.13 → 4.12.0

package/tests/fixtures/agent-hooks/opencode-todo-updated.json

@@ -0,0 +1,13 @@
+{
+  "type": "todo.updated",
+  "properties": {
+    "sessionID": "opencode-session-fixture",
+    "todos": [
+      {
+        "content": "Verify OAuth token refresh after OpenCode compact",
+        "status": "in_progress",
+        "priority": "high"
+      }
+    ]
+  }
+}

package/tests/fixtures/agent-hooks/opencode-tool-execute-after.json

@@ -0,0 +1,15 @@
+{
+  "input": {
+    "tool": "read",
+    "sessionID": "opencode-session-fixture",
+    "callID": "opencode-call-fixture",
+    "args": {
+      "filePath": "/tmp/eagle-mem/project/lib/auth.ts"
+    }
+  },
+  "output": {
+    "title": "Read lib/auth.ts",
+    "output": "export function refreshToken() {}",
+    "metadata": {}
+  }
+}

package/tests/fixtures/agent-hooks/opencode-tool-execute-before.json

@@ -0,0 +1,12 @@
+{
+  "input": {
+    "tool": "bash",
+    "sessionID": "opencode-session-fixture",
+    "callID": "opencode-call-fixture"
+  },
+  "output": {
+    "args": {
+      "command": "git push origin main"
+    }
+  }
+}

package/tests/test_agent_compatibility_docs_gate.sh

@@ -0,0 +1,123 @@
+#!/usr/bin/env bash
+# Guard against stale agent lifecycle assumptions when editing hook, statusline,
+# config-installation, or instruction-file integration code.
+set -euo pipefail
+
+ROOT_DIR="$(cd "$(dirname "$0")/.." && pwd)"
+DOC_DIR="$ROOT_DIR/docs/agent-compatibility"
+FIXTURE_DIR="$ROOT_DIR/tests/fixtures/agent-hooks"
+
+fail() {
+    echo "agent compatibility docs gate failed: $*" >&2
+    exit 1
+}
+
+require_file() {
+    local file="$1"
+    [ -f "$file" ] || fail "missing required file: ${file#$ROOT_DIR/}"
+}
+
+require_contains() {
+    local file="$1"
+    local pattern="$2"
+    local label="$3"
+    if ! grep -Eq "$pattern" "$file"; then
+        fail "${file#$ROOT_DIR/} does not mention $label"
+    fi
+}
+
+require_json() {
+    local file="$1"
+    jq -e . "$file" >/dev/null || fail "invalid JSON fixture: ${file#$ROOT_DIR/}"
+}
+
+require_file "$DOC_DIR/README.md"
+require_file "$DOC_DIR/claude-code.md"
+require_file "$DOC_DIR/codex.md"
+require_file "$DOC_DIR/opencode.md"
+require_file "$FIXTURE_DIR/claude-user-prompt-submit.json"
+require_file "$FIXTURE_DIR/claude-statusline.json"
+require_file "$FIXTURE_DIR/codex-user-prompt-submit.json"
+require_file "$FIXTURE_DIR/codex-pre-tool-use.json"
+require_file "$FIXTURE_DIR/opencode-chat-message.json"
+require_file "$FIXTURE_DIR/opencode-tool-execute-before.json"
+require_file "$FIXTURE_DIR/opencode-tool-execute-after.json"
+require_file "$FIXTURE_DIR/opencode-todo-updated.json"
+require_file "$FIXTURE_DIR/opencode-session-compacting.json"
+
+require_json "$FIXTURE_DIR/claude-user-prompt-submit.json"
+require_json "$FIXTURE_DIR/claude-statusline.json"
+require_json "$FIXTURE_DIR/codex-user-prompt-submit.json"
+require_json "$FIXTURE_DIR/codex-pre-tool-use.json"
+require_json "$FIXTURE_DIR/opencode-chat-message.json"
+require_json "$FIXTURE_DIR/opencode-tool-execute-before.json"
+require_json "$FIXTURE_DIR/opencode-tool-execute-after.json"
+require_json "$FIXTURE_DIR/opencode-todo-updated.json"
+require_json "$FIXTURE_DIR/opencode-session-compacting.json"
+
+require_contains "$DOC_DIR/README.md" "Before modifying" "the required pre-edit workflow"
+require_contains "$DOC_DIR/README.md" "official documentation" "official documentation"
+require_contains "$DOC_DIR/README.md" "fixture|golden-test" "fixture or golden-test coverage"
+
+require_contains "$DOC_DIR/claude-code.md" "Last verified: [0-9]{4}-[0-9]{2}-[0-9]{2}" "a Last verified date"
+require_contains "$DOC_DIR/claude-code.md" "https://code\\.claude\\.com/docs/en/hooks" "Claude Code hooks source"
+require_contains "$DOC_DIR/claude-code.md" "https://code\\.claude\\.com/docs/en/statusline" "Claude Code statusline source"
+require_contains "$DOC_DIR/claude-code.md" "UserPromptSubmit" "Claude UserPromptSubmit"
+require_contains "$DOC_DIR/claude-code.md" "PreCompact" "Claude PreCompact"
+require_contains "$DOC_DIR/claude-code.md" "PostCompact" "Claude PostCompact"
+require_contains "$DOC_DIR/claude-code.md" "statusLine" "Claude statusLine config"
+require_contains "$DOC_DIR/claude-code.md" "tests/fixtures/agent-hooks/claude-statusline\\.json" "Claude statusline fixture"
+
+require_contains "$DOC_DIR/codex.md" "Last verified: [0-9]{4}-[0-9]{2}-[0-9]{2}" "a Last verified date"
+require_contains "$DOC_DIR/codex.md" "https://developers\\.openai\\.com/codex/codex-manual\\.md" "Codex manual source"
+require_contains "$DOC_DIR/codex.md" "https://developers\\.openai\\.com/codex/hooks\\.md" "Codex hooks source"
+require_contains "$DOC_DIR/codex.md" "AGENTS\\.md" "Codex AGENTS.md behavior"
+require_contains "$DOC_DIR/codex.md" "features\\.hooks" "canonical Codex hooks feature flag"
+require_contains "$DOC_DIR/codex.md" "deprecated alias" "deprecated Codex hook alias"
+require_contains "$DOC_DIR/codex.md" "UserPromptSubmit" "Codex UserPromptSubmit"
+require_contains "$DOC_DIR/codex.md" "tests/fixtures/agent-hooks/codex-pre-tool-use\\.json" "Codex PreToolUse fixture"
+
+require_contains "$DOC_DIR/opencode.md" "Last verified: [0-9]{4}-[0-9]{2}-[0-9]{2}" "an OpenCode Last verified date"
+require_contains "$DOC_DIR/opencode.md" "https://opencode\\.ai/docs/plugins/" "OpenCode plugins source"
+require_contains "$DOC_DIR/opencode.md" "https://opencode\\.ai/docs/config/" "OpenCode config source"
+require_contains "$DOC_DIR/opencode.md" "https://opencode\\.ai/config\\.json" "OpenCode config schema source"
+require_contains "$DOC_DIR/opencode.md" "~/.config/opencode/plugins/" "OpenCode global local plugin directory"
+require_contains "$DOC_DIR/opencode.md" "chat\\.message" "OpenCode chat.message hook"
+require_contains "$DOC_DIR/opencode.md" "tool\\.execute\\.before" "OpenCode tool.execute.before hook"
+require_contains "$DOC_DIR/opencode.md" "tool\\.execute\\.after" "OpenCode tool.execute.after hook"
+require_contains "$DOC_DIR/opencode.md" "todo\\.updated" "OpenCode todo.updated event"
+require_contains "$DOC_DIR/opencode.md" "experimental\\.session\\.compacting" "OpenCode compaction hook"
+require_contains "$DOC_DIR/opencode.md" "opencode --pure" "OpenCode pure-mode caveat"
+require_contains "$DOC_DIR/opencode.md" "tests/fixtures/agent-hooks/opencode-tool-execute-before\\.json" "OpenCode PreToolUse fixture"
+require_contains "$DOC_DIR/opencode.md" "tests/test_opencode_plugin_adapter\\.sh" "OpenCode adapter regression test"
+
+if grep -R "features\\.codex_hooks" "$ROOT_DIR/lib" "$ROOT_DIR/scripts" "$ROOT_DIR/hooks" "$ROOT_DIR/bin" >/dev/null 2>&1; then
+    fail "implementation still uses deprecated Codex features.codex_hooks instead of features.hooks"
+fi
+
+if grep -R "codex_hooks[[:space:]]*=[[:space:]]*true" "$ROOT_DIR/lib" "$ROOT_DIR/scripts" "$ROOT_DIR/hooks" "$ROOT_DIR/bin" >/dev/null 2>&1; then
+    fail "implementation still writes deprecated Codex codex_hooks config instead of hooks"
+fi
+
+changed_sensitive=$(
+    {
+        git -C "$ROOT_DIR" diff --name-only HEAD -- 2>/dev/null || true
+        git -C "$ROOT_DIR" ls-files --others --exclude-standard 2>/dev/null || true
+    } | awk '
+        /^(hooks\/.*\.sh|lib\/hooks\.sh|lib\/codex-hooks\.sh|lib\/opencode-hooks\.sh|integrations\/opencode_eagle_mem_plugin\.js|scripts\/install\.sh|scripts\/update\.sh|scripts\/uninstall\.sh|scripts\/doctor\.sh|scripts\/statusline-em\.sh|lib\/common\.sh|AGENTS\.md|CLAUDE\.md)$/ { print }
+    ' | sort -u
+)
+
+if [ -n "$changed_sensitive" ]; then
+    changed_evidence=$(
+        {
+            git -C "$ROOT_DIR" diff --name-only HEAD -- 2>/dev/null || true
+            git -C "$ROOT_DIR" ls-files --others --exclude-standard 2>/dev/null || true
+        } | awk '
+            /^(docs\/agent-compatibility\/.*\.md|tests\/fixtures\/agent-hooks\/.*\.json|tests\/test_agent_compatibility_docs_gate\.sh)$/ { print }
+        ' | sort -u
+    )
+    [ -n "$changed_evidence" ] || fail "sensitive integration files changed without updated compatibility docs or fixtures: $changed_sensitive"
+fi
+
+echo "agent compatibility docs gate passed"

package/tests/test_auto_orchestration_detection.sh

@@ -0,0 +1,109 @@
+#!/usr/bin/env bash
+# Broad prompts should create durable orchestration state automatically. This
+# proves Eagle detects the need for lanes instead of relying only on agent
+# obedience to instructions.
+set -euo pipefail
+
+ROOT_DIR="$(cd "$(dirname "$0")/.." && pwd)"
+EAGLE_BIN="$ROOT_DIR/bin/eagle-mem"
+
+tmp_dir=$(mktemp -d "$ROOT_DIR/.tmp-auto-orchestration.XXXXXX")
+trap 'rm -rf "$tmp_dir"' EXIT
+
+export HOME="$tmp_dir/home"
+export EAGLE_MEM_DIR="$tmp_dir/eagle-mem"
+mkdir -p "$HOME" "$EAGLE_MEM_DIR"
+
+. "$ROOT_DIR/lib/common.sh"
+"$ROOT_DIR/db/migrate.sh" >/dev/null
+. "$ROOT_DIR/lib/db.sh"
+
+assert_json() {
+    local json="$1" filter="$2" message="$3"
+    if ! printf '%s' "$json" | jq -e "$filter" >/dev/null; then
+        echo "$message" >&2
+        echo "$json" >&2
+        exit 1
+    fi
+}
+
+assert_contains() {
+    local haystack="$1" needle="$2" message="$3"
+    case "$haystack" in
+        *"$needle"*) ;;
+        *)
+            echo "$message" >&2
+            echo "Expected to find: $needle" >&2
+            echo "$haystack" >&2
+            exit 1
+            ;;
+    esac
+}
+
+project="project-auto-orchestration"
+repo="$HOME/$project"
+mkdir -p "$repo"
+
+hook_input=$(jq -nc \
+    --arg sid "auto-orchestration-session" \
+    --arg cwd "$repo" \
+    --arg prompt "Please plan and get started on a broad full codebase release. Split the work into lanes, coordinate workers, implement the fixes, and verify the ship path." \
+    '{session_id:$sid, cwd:$cwd, prompt:$prompt, transcript_path:"/tmp/transcript.jsonl"}')
+
+hook_output=$(EAGLE_MEM_PROJECT="$project" EAGLE_MEM_DIR="$EAGLE_MEM_DIR" EAGLE_AGENT_SOURCE="codex" bash "$ROOT_DIR/hooks/user-prompt-submit.sh" <<< "$hook_input")
+assert_contains "$hook_output" "created durable orchestration 'auto'" "broad prompt did not report auto orchestration creation"
+assert_contains "$hook_output" "Continue from the durable lanes" "hook did not tell the agent to continue from durable lanes"
+
+orchestration_json=$(cd "$repo" && EAGLE_MEM_PROJECT="$project" EAGLE_MEM_DIR="$EAGLE_MEM_DIR" "$EAGLE_BIN" orchestrate --name auto --json)
+assert_json "$orchestration_json" '
+    length == 3
+    and ([.[].lane_key] | sort == ["implementation","scope","verification"])
+    and all(.[]; .status == "pending")
+' "auto orchestration did not create the expected pending lanes"
+
+task_json=$(cd "$repo" && EAGLE_MEM_PROJECT="$project" EAGLE_MEM_DIR="$EAGLE_MEM_DIR" "$EAGLE_BIN" tasks --json)
+assert_json "$task_json" '
+    length >= 3
+    and ([.[] | select(.source_task_id | startswith("lane-auto-"))] | length == 3)
+' "auto orchestration did not mirror lanes into durable tasks"
+
+event_json=$(eagle_db_json "SELECT session_id, project, agent, prompt_snippet, orchestration_name, lanes, status
+                            FROM orchestration_auto_events
+                            WHERE project = '$project'
+                            ORDER BY id DESC;")
+assert_json "$event_json" '
+    length == 1
+    and .[0].session_id == "auto-orchestration-session"
+    and .[0].orchestration_name == "auto"
+    and .[0].status == "created"
+    and ((.[0].lanes | fromjson) | length == 3)
+' "auto orchestration event was not recorded"
+
+hook_output_again=$(EAGLE_MEM_PROJECT="$project" EAGLE_MEM_DIR="$EAGLE_MEM_DIR" EAGLE_AGENT_SOURCE="codex" bash "$ROOT_DIR/hooks/user-prompt-submit.sh" <<< "$hook_input")
+assert_contains "$hook_output_again" "created durable orchestration 'auto'" "repeated broad prompt did not reuse auto orchestration"
+
+lane_count=$(eagle_db "SELECT COUNT(*)
+                       FROM orchestration_lanes l
+                       JOIN orchestrations o ON o.id = l.orchestration_id
+                       WHERE o.project = '$project'
+                         AND o.name = 'auto';")
+task_count=$(eagle_db "SELECT COUNT(*)
+                       FROM agent_tasks
+                       WHERE project = '$project'
+                         AND source_task_id LIKE 'lane-auto-%';")
+event_count=$(eagle_db "SELECT COUNT(*) FROM orchestration_auto_events WHERE project = '$project';")
+
+[ "$lane_count" = "3" ] || { echo "expected 3 auto lanes after repeat, got $lane_count" >&2; exit 1; }
+[ "$task_count" = "3" ] || { echo "expected 3 auto tasks after repeat, got $task_count" >&2; exit 1; }
+[ "$event_count" = "2" ] || { echo "expected 2 auto events after repeat, got $event_count" >&2; exit 1; }
+
+quiet_input=$(jq -nc \
+    --arg sid "quiet-session" \
+    --arg cwd "$repo" \
+    --arg prompt "Please inspect the config value." \
+    '{session_id:$sid, cwd:$cwd, prompt:$prompt}')
+EAGLE_MEM_PROJECT="$project" EAGLE_MEM_DIR="$EAGLE_MEM_DIR" EAGLE_AGENT_SOURCE="codex" bash "$ROOT_DIR/hooks/user-prompt-submit.sh" <<< "$quiet_input" >/dev/null
+quiet_event_count=$(eagle_db "SELECT COUNT(*) FROM orchestration_auto_events WHERE project = '$project';")
+[ "$quiet_event_count" = "2" ] || { echo "quiet prompt should not create auto orchestration event" >&2; exit 1; }
+
+echo "auto orchestration detection regressions passed"

package/tests/test_claude_stop_hook_registration.sh

@@ -0,0 +1,56 @@
+#!/usr/bin/env bash
+# Regression coverage for Claude Stop hook registration.
+set -euo pipefail
+
+ROOT_DIR="$(cd "$(dirname "$0")/.." && pwd)"
+tmp_dir=$(mktemp -d "$ROOT_DIR/.tmp-claude-stop.XXXXXX")
+trap 'rm -rf "$tmp_dir"' EXIT
+
+export HOME="$tmp_dir/home"
+export EAGLE_MEM_DIR="$tmp_dir/eagle-mem"
+mkdir -p "$HOME" "$EAGLE_MEM_DIR/hooks"
+
+. "$ROOT_DIR/scripts/style.sh"
+. "$ROOT_DIR/lib/common.sh"
+. "$ROOT_DIR/lib/hooks.sh"
+
+settings="$tmp_dir/settings.json"
+old_stop="$EAGLE_MEM_DIR/hooks/stop.sh"
+new_stop="bash \"$EAGLE_MEM_DIR/hooks/stop.sh\""
+
+fail() {
+    echo "claude stop hook registration test failed: $*" >&2
+    exit 1
+}
+
+cat > "$settings" <<JSON
+{
+  "hooks": {
+    "Stop": [
+      {
+        "hooks": [
+          {
+            "type": "command",
+            "command": "$old_stop"
+          }
+        ]
+      }
+    ]
+  }
+}
+JSON
+
+eagle_clean_hook_entries "$settings" "Stop" "$old_stop"
+eagle_patch_hook "$settings" "Stop" "" "$new_stop" >/dev/null
+
+jq -e --arg cmd "$old_stop" 'all(.hooks.Stop[]?.hooks[]?; .command != $cmd)' "$settings" >/dev/null \
+    || fail "path-only Stop hook registration remained"
+
+jq -e --arg cmd "$new_stop" '.hooks.Stop[]? | select(.matcher == null and (.hooks[]?.command == $cmd))' "$settings" >/dev/null \
+    || fail "bash-wrapped Stop hook registration missing"
+
+eagle_patch_hook "$settings" "Stop" "" "$new_stop" >/dev/null
+count=$(jq --arg cmd "$new_stop" '[.hooks.Stop[]?.hooks[]? | select(.command == $cmd)] | length' "$settings")
+[ "$count" -eq 1 ] || fail "bash-wrapped Stop hook registration duplicated: $count"
+
+echo "claude stop hook registration test passed"

package/tests/test_clean_session_capture.sh

@@ -0,0 +1,105 @@
+#!/usr/bin/env bash
+# ═══════════════════════════════════════════════════════════
+# Eagle Mem — Clean, branded session capture regression test
+# Covers: CLI-first capture, clobber prevention, backward-compat
+#         <eagle-summary> override, and enrichment fill-only.
+# ═══════════════════════════════════════════════════════════
+set -uo pipefail
+
+ROOT_DIR="$(cd "$(dirname "$0")/.." && pwd)"
+tmp_dir=$(mktemp -d)
+trap 'rm -rf "$tmp_dir"' EXIT
+
+export EAGLE_MEM_DIR="$tmp_dir/em"
+export EAGLE_AGENT_SOURCE="claude-code"
+export EAGLE_MEM_DISABLE_HOOKS=0
+# Pin the project so both the CLI save and the Stop hook resolve identically
+# (the temp cwd lives under /var/folders, which Eagle Mem treats as ephemeral).
+export EAGLE_MEM_PROJECT="test/clean-capture"
+mkdir -p "$EAGLE_MEM_DIR"
+
+pass=0; fail=0
+ok()  { echo "  ok: $1"; pass=$((pass+1)); }
+bad() { echo "  FAIL: $1" >&2; fail=$((fail+1)); }
+
+bash "$ROOT_DIR/db/migrate.sh" >/dev/null 2>&1
+
+. "$ROOT_DIR/lib/common.sh"
+. "$ROOT_DIR/lib/db.sh"
+
+# A real cwd for the hook payload; project is pinned via EAGLE_MEM_PROJECT.
+work="$tmp_dir/work"
+mkdir -p "$work"
+PROJECT="$EAGLE_MEM_PROJECT"
+ok "pinned project: $PROJECT"
+
+field() { eagle_db "SELECT COALESCE($2,'') FROM summaries WHERE session_id='$1';"; }
+
+# ── Scenario A: CLI capture then a heuristic Stop must NOT clobber it ──
+SID="clean-capture-aaa111"
+eagle_upsert_session "$SID" "$PROJECT" "$work" "" "startup" "claude-code"
+
+bash "$ROOT_DIR/scripts/session.sh" save --session-id "$SID" --project "$PROJECT" \
+    --completed "Rich agent completed" \
+    --decisions "Pin v1 — why safety; Use TOTP — why no SMTP" \
+    --gotchas "needs PUBLIC_URL" >/dev/null 2>&1
+
+[ "$(field "$SID" capture_source)" = "agent" ] && ok "A: capture_source=agent after CLI save" || bad "A: capture_source not agent"
+
+# Heuristic Stop: transcript has an Edit (fills files), NO <eagle-summary> block.
+tA="$tmp_dir/transcriptA.jsonl"
+{
+  printf '%s\n' '{"type":"user","message":{"content":[{"type":"text","text":"please harden the pipeline"}]}}'
+  printf '%s\n' '{"type":"assistant","message":{"content":[{"type":"tool_use","name":"Edit","input":{"file_path":"'"$work"'/foo.sh"}}]}}'
+  printf '%s\n' '{"type":"assistant","message":{"content":[{"type":"text","text":"Done. We hardened the pipeline this session."}]}}'
+} > "$tA"
+
+stopA=$(jq -nc --arg s "$SID" --arg c "$work" --arg t "$tA" \
+    '{session_id:$s, cwd:$c, transcript_path:$t}')
+echo "$stopA" | EAGLE_MEM_STOP_BACKGROUND_ENRICH=0 bash "$ROOT_DIR/hooks/stop.sh" >/dev/null 2>&1
+
+[ "$(field "$SID" completed)" = "Rich agent completed" ] && ok "A: completed NOT clobbered by heuristic" || bad "A: completed was clobbered -> '$(field "$SID" completed)'"
+[ "$(field "$SID" capture_source)" = "agent" ] && ok "A: capture_source still agent after Stop" || bad "A: capture_source downgraded"
+case "$(field "$SID" files_modified)" in *foo.sh*) ok "A: files_modified gap-filled by heuristic" ;; *) bad "A: files_modified not filled -> '$(field "$SID" files_modified)'" ;; esac
+case "$(field "$SID" decisions)" in *"Pin v1"*) ok "A: decisions preserved" ;; *) bad "A: decisions lost" ;; esac
+mrows=$(eagle_db "SELECT COUNT(*) FROM summaries WHERE session_id LIKE 'manual-%';")
+[ "$mrows" = "0" ] && ok "A: no stray manual-* row" || bad "A: $mrows manual-* rows created"
+sstatus=$(eagle_db "SELECT status FROM sessions WHERE id='$SID';")
+[ "$sstatus" = "active" ] && ok "A: live session still active (not ended)" || bad "A: session status=$sstatus"
+
+# ── Scenario B: a <eagle-summary> block still overrides (backward compat) ──
+SIDB="clean-capture-bbb222"
+eagle_upsert_session "$SIDB" "$PROJECT" "$work" "" "startup" "claude-code"
+tB="$tmp_dir/transcriptB.jsonl"
+blocktext='Summary below.
+
+<eagle-summary>
+request: legacy block path
+completed: shipped via block
+decisions: keep parser — why backward compat
+gotchas: none
+</eagle-summary>'
+{
+  printf '%s\n' '{"type":"user","message":{"content":[{"type":"text","text":"do the legacy thing"}]}}'
+  printf '%s\n' "$(jq -nc --arg t "$blocktext" '{type:"assistant",message:{content:[{type:"text",text:$t}]}}')"
+} > "$tB"
+stopB=$(jq -nc --arg s "$SIDB" --arg c "$work" --arg t "$tB" '{session_id:$s, cwd:$c, transcript_path:$t}')
+echo "$stopB" | EAGLE_MEM_STOP_BACKGROUND_ENRICH=0 bash "$ROOT_DIR/hooks/stop.sh" >/dev/null 2>&1
+
+[ "$(field "$SIDB" capture_source)" = "agent" ] && ok "B: block sets capture_source=agent" || bad "B: capture_source=$(field "$SIDB" capture_source)"
+case "$(field "$SIDB" completed)" in *"shipped via block"*) ok "B: completed parsed from block" ;; *) bad "B: block completed not parsed" ;; esac
+case "$(field "$SIDB" decisions)" in *"keep parser"*) ok "B: decisions parsed from block" ;; *) bad "B: block decisions not parsed" ;; esac
+
+# ── Scenario C: enrichment must not overwrite an agent row ──
+SIDC="clean-capture-ccc333"
+eagle_upsert_session "$SIDC" "$PROJECT" "$work" "" "startup" "claude-code"
+eagle_insert_summary "$SIDC" "$PROJECT" "req" "" "" "agent completed C" "" "[]" "[]" "" "agent decision C" "" "" "claude-code" "agent"
+# fill-only path used by enrich when source=agent: simulate enrich trying to overwrite
+eagle_insert_summary_fill_only "$SIDC" "$PROJECT" "" "" "enriched learned" "ENRICH OVERWRITE" "" "[]" "[]" "" "ENRICH DECISION" "" "" "claude-code" "enrich"
+[ "$(field "$SIDC" completed)" = "agent completed C" ] && ok "C: enrich fill-only did NOT overwrite completed" || bad "C: enrich overwrote -> '$(field "$SIDC" completed)'"
+case "$(field "$SIDC" learned)" in *"enriched learned"*) ok "C: enrich filled empty learned gap" ;; *) bad "C: enrich did not fill empty learned" ;; esac
+[ "$(field "$SIDC" capture_source)" = "agent" ] && ok "C: capture_source stays agent through enrich" || bad "C: capture_source changed"
+
+echo ""
+echo "test_clean_session_capture: $pass passed, $fail failed"
+[ "$fail" -eq 0 ]

package/tests/test_codex_hooks_config.sh

@@ -0,0 +1,73 @@
+#!/usr/bin/env bash
+# Codex hook enablement should use the current canonical features.hooks flag
+# while cleaning up the older codex_hooks alias when encountered.
+set -euo pipefail
+
+ROOT_DIR="$(cd "$(dirname "$0")/.." && pwd)"
+
+tmp_dir=$(mktemp -d "$ROOT_DIR/.tmp-codex-hooks-config.XXXXXX")
+trap 'rm -rf "$tmp_dir"' EXIT
+
+export HOME="$tmp_dir/home"
+export EAGLE_MEM_DIR="$tmp_dir/eagle-mem"
+export EAGLE_CODEX_DIR="$HOME/.codex"
+export EAGLE_CODEX_CONFIG="$EAGLE_CODEX_DIR/config.toml"
+export EAGLE_CODEX_HOOKS="$EAGLE_CODEX_DIR/hooks.json"
+
+mkdir -p "$HOME" "$EAGLE_MEM_DIR" "$EAGLE_CODEX_DIR"
+
+. "$ROOT_DIR/lib/common.sh"
+. "$ROOT_DIR/lib/codex-hooks.sh"
+
+fail() {
+    echo "codex hooks config regression failed: $*" >&2
+    exit 1
+}
+
+assert_has_hooks_true() {
+    grep -q '^hooks = true$' "$EAGLE_CODEX_CONFIG" || fail "missing canonical hooks = true"
+}
+
+assert_no_legacy_write() {
+    if grep -q '^codex_hooks[[:space:]]*=' "$EAGLE_CODEX_CONFIG"; then
+        fail "deprecated codex_hooks key was left in generated config"
+    fi
+}
+
+rm -f "$EAGLE_CODEX_CONFIG"
+eagle_enable_codex_hooks
+assert_has_hooks_true
+assert_no_legacy_write
+
+cat > "$EAGLE_CODEX_CONFIG" <<'TOML'
+[other]
+name = "preserve-me"
+
+[features]
+codex_hooks = false
+memories = true
+
+[sandbox]
+mode = "workspace-write"
+TOML
+
+eagle_enable_codex_hooks
+assert_has_hooks_true
+assert_no_legacy_write
+grep -q '^memories = true$' "$EAGLE_CODEX_CONFIG" || fail "features table values were not preserved"
+grep -q '^name = "preserve-me"$' "$EAGLE_CODEX_CONFIG" || fail "other config sections were not preserved"
+grep -q '^mode = "workspace-write"$' "$EAGLE_CODEX_CONFIG" || fail "later config sections were not preserved"
+
+cat > "$EAGLE_CODEX_CONFIG" <<'TOML'
+[features]
+hooks = false
+TOML
+
+eagle_enable_codex_hooks
+assert_has_hooks_true
+assert_no_legacy_write
+hook_lines=$(grep -c '^hooks = true$' "$EAGLE_CODEX_CONFIG")
+[ "$hook_lines" -eq 1 ] || fail "expected one hooks = true line, found $hook_lines"
+
+echo "codex hooks config regressions passed"
+