eagle-mem 1.6.0 → 1.6.2

package/bin/eagle-mem

@@ -29,7 +29,7 @@ case "$command" in
     help|--help|-h)
                 bash "$SCRIPTS_DIR/help.sh" ;;
     version|--version|-v|-V)
-                version=$(node -e "console.log(require('$PACKAGE_DIR/package.json').version)" 2>/dev/null || echo "unknown")
+                version=$(jq -r .version "$PACKAGE_DIR/package.json" 2>/dev/null || echo "unknown")
                 echo -e "  ${BOLD}Eagle Mem${RESET} v${version}"
                 ;;
     *)

package/db/009_drop_dead_tasks.sql

@@ -0,0 +1,29 @@
+-- ═══════════════════════════════════════════════════════════
+-- Migration 009: Drop dead tasks table
+-- The original tasks table (from schema.sql / migration 001)
+-- was replaced by claude_tasks (migration 007). No code
+-- references the old table. This cleans up the vestigial
+-- schema: 3 FTS triggers, the FTS virtual table, 3 indexes,
+-- and the table itself.
+--
+-- NOTE: schema.sql (migration 001) still contains the CREATE
+-- TABLE tasks definition. We do NOT edit applied migrations.
+-- Fresh installs will CREATE (001) then DROP (009). Existing
+-- installs just run 009. Verified 0 rows in production.
+-- ═══════════════════════════════════════════════════════════
+
+-- Drop FTS sync triggers first (they reference the tasks table)
+DROP TRIGGER IF EXISTS tasks_ai;
+DROP TRIGGER IF EXISTS tasks_ad;
+DROP TRIGGER IF EXISTS tasks_au;
+
+-- Drop the FTS virtual table
+DROP TABLE IF EXISTS tasks_fts;
+
+-- Drop indexes (implicit in DROP TABLE, but explicit for clarity)
+DROP INDEX IF EXISTS idx_tasks_project;
+DROP INDEX IF EXISTS idx_tasks_status;
+DROP INDEX IF EXISTS idx_tasks_parent;
+
+-- Drop the dead tasks table
+DROP TABLE IF EXISTS tasks;

package/db/migrate.sh

@@ -25,7 +25,8 @@ run_migration() {
         body=$(grep -v -E '^[[:space:]]*PRAGMA ' "$file")
 
         # Set connection PRAGMAs, then run migration body + tracking insert atomically
-        { echo "PRAGMA trusted_schema=ON;"; echo "PRAGMA foreign_keys=ON;"; echo "PRAGMA busy_timeout=5000;"; echo "BEGIN;"; echo "$body"; echo "INSERT INTO _migrations (name) VALUES ('$name');"; echo "COMMIT;"; } | sqlite3 "$DB"
+        # .bail on ensures sqlite3 stops on the first error instead of continuing
+        { echo ".bail on"; echo "PRAGMA trusted_schema=ON;"; echo "PRAGMA foreign_keys=ON;"; echo "PRAGMA busy_timeout=5000;"; echo "BEGIN;"; echo "$body"; echo "INSERT INTO _migrations (name) VALUES ('$name');"; echo "COMMIT;"; } | sqlite3 "$DB"
         echo "  applied: $name"
     fi
 }
@@ -40,28 +41,14 @@ sqlite3 "$DB" "CREATE TABLE IF NOT EXISTS _migrations (
 # Set PRAGMAs (these must be set on every connection)
 sqlite3 "$DB" "PRAGMA journal_mode = WAL; PRAGMA synchronous = NORMAL; PRAGMA busy_timeout = 5000; PRAGMA foreign_keys = ON;"
 
-# ─── Migration 001: Initial schema ─────────────────────────
+# ─── Migration 001: Initial schema (special name) ──────────
 run_migration "001_initial_schema" "$SCRIPT_DIR/schema.sql"
 
-# ─── Migration 002: Project overviews ──────────────────────
-run_migration "002_overviews" "$SCRIPT_DIR/002_overviews.sql"
-
-# ─── Migration 003: Code chunks ───────────────────────────
-run_migration "003_code_chunks" "$SCRIPT_DIR/003_code_chunks.sql"
-
-# ─── Migration 004: Observation indexes ──────────────────
-run_migration "004_observation_indexes" "$SCRIPT_DIR/004_observation_indexes.sql"
-
-# ─── Migration 005: Claude Code memory mirror ────────────
-run_migration "005_claude_memories" "$SCRIPT_DIR/005_claude_memories.sql"
-
-# ─── Migration 006: Claude Code plan mirror ──────────────
-run_migration "006_claude_plans" "$SCRIPT_DIR/006_claude_plans.sql"
-
-# ─── Migration 007: Claude Code task mirror ──────────────
-run_migration "007_claude_tasks" "$SCRIPT_DIR/007_claude_tasks.sql"
-
-# ─── Migration 008: Summary UPSERT (unique session_id) ───
-run_migration "008_summary_upsert" "$SCRIPT_DIR/008_summary_upsert.sql"
+# ─── Run numbered migrations (002+) via sorted glob ───────
+for migration_file in "$SCRIPT_DIR"/[0-9][0-9][0-9]_*.sql; do
+    [ ! -f "$migration_file" ] && continue
+    migration_name=$(basename "$migration_file" .sql)
+    run_migration "$migration_name" "$migration_file"
+done
 
 echo "  Eagle Mem database ready: $DB"

package/db/schema.sql

@@ -75,28 +75,6 @@ CREATE TABLE IF NOT EXISTS summaries (
 CREATE INDEX IF NOT EXISTS idx_summaries_session ON summaries(session_id);
 CREATE INDEX IF NOT EXISTS idx_summaries_project ON summaries(project);
 
--- ─── Tasks (TaskAware Compact Loop) ───────────────────────
--- Subtasks for multi-step work with compaction between each
-
-CREATE TABLE IF NOT EXISTS tasks (
-    id              INTEGER PRIMARY KEY AUTOINCREMENT,
-    project         TEXT NOT NULL,
-    session_id      TEXT REFERENCES sessions(id),
-    parent_id       INTEGER REFERENCES tasks(id),
-    title           TEXT NOT NULL,
-    instructions    TEXT,
-    context_snapshot TEXT,
-    status          TEXT NOT NULL DEFAULT 'pending' CHECK (status IN ('pending', 'active', 'done', 'blocked', 'cancelled')),
-    ordinal         INTEGER NOT NULL DEFAULT 0,
-    created_at      TEXT NOT NULL DEFAULT (strftime('%Y-%m-%dT%H:%M:%fZ', 'now')),
-    started_at      TEXT,
-    completed_at    TEXT
-);
-
-CREATE INDEX IF NOT EXISTS idx_tasks_project ON tasks(project);
-CREATE INDEX IF NOT EXISTS idx_tasks_status ON tasks(status);
-CREATE INDEX IF NOT EXISTS idx_tasks_parent ON tasks(parent_id);
-
 -- ─── FTS5: Full-text search on summaries ───────────────────
 
 CREATE VIRTUAL TABLE IF NOT EXISTS summaries_fts USING fts5(
@@ -127,30 +105,3 @@ CREATE TRIGGER IF NOT EXISTS summaries_au AFTER UPDATE ON summaries BEGIN
     INSERT INTO summaries_fts(rowid, request, investigated, learned, completed, next_steps, notes)
     VALUES (new.id, new.request, new.investigated, new.learned, new.completed, new.next_steps, new.notes);
 END;
-
--- ─── FTS5: Full-text search on tasks ──────────────────────
-
-CREATE VIRTUAL TABLE IF NOT EXISTS tasks_fts USING fts5(
-    title,
-    instructions,
-    context_snapshot,
-    content='tasks',
-    content_rowid='id'
-);
-
-CREATE TRIGGER IF NOT EXISTS tasks_ai AFTER INSERT ON tasks BEGIN
-    INSERT INTO tasks_fts(rowid, title, instructions, context_snapshot)
-    VALUES (new.id, new.title, new.instructions, new.context_snapshot);
-END;
-
-CREATE TRIGGER IF NOT EXISTS tasks_ad AFTER DELETE ON tasks BEGIN
-    INSERT INTO tasks_fts(tasks_fts, rowid, title, instructions, context_snapshot)
-    VALUES ('delete', old.id, old.title, old.instructions, old.context_snapshot);
-END;
-
-CREATE TRIGGER IF NOT EXISTS tasks_au AFTER UPDATE ON tasks BEGIN
-    INSERT INTO tasks_fts(tasks_fts, rowid, title, instructions, context_snapshot)
-    VALUES ('delete', old.id, old.title, old.instructions, old.context_snapshot);
-    INSERT INTO tasks_fts(rowid, title, instructions, context_snapshot)
-    VALUES (new.id, new.title, new.instructions, new.context_snapshot);
-END;

package/hooks/post-tool-use.sh

@@ -83,7 +83,11 @@ esac
 case "$tool_name" in
     Write|Edit)
         if [ -n "$fp" ]; then
+            # Reject path traversal: bash case `*` matches `/`, so
+            # patterns like projects/*/memory/*.md would match paths
+            # containing /../ segments. Block any path with `..` first.
             case "$fp" in
+                *..*) ;; # path traversal — skip
                 "$HOME/.claude/projects"/*/memory/*.md)
                     mem_base=$(basename "$fp")
                     if [ "$mem_base" != "MEMORY.md" ] && [ -f "$fp" ]; then

package/hooks/session-start.sh

@@ -31,8 +31,10 @@ eagle_log "INFO" "SessionStart: session=$session_id project=$project source=$sou
 eagle_upsert_session "$session_id" "$project" "$cwd" "$model" "$source_type"
 
 # ─── Sweep stuck sessions (older than 4 hours) ─────────────
+# Exclude the current session — it may be a resumed session older than 4h
 eagle_db "UPDATE sessions SET status = 'abandoned'
     WHERE status = 'active'
+    AND id != '$(eagle_sql_escape "$session_id")'
     AND started_at < strftime('%Y-%m-%dT%H:%M:%fZ', 'now', '-4 hours');"
 
 # ─── Build context injection ────────────────────────────────
@@ -115,7 +117,7 @@ fi
 synced_tasks=$(eagle_db "SELECT subject, status, blocked_by FROM claude_tasks
     WHERE project = '$(eagle_sql_escape "$project")'
     AND status IN ('in_progress', 'pending')
-    AND updated_at > datetime('now', '-7 days')
+    AND updated_at > strftime('%Y-%m-%dT%H:%M:%fZ', 'now', '-7 days')
     ORDER BY
         CASE status WHEN 'in_progress' THEN 0 ELSE 1 END,
         updated_at DESC
@@ -144,8 +146,7 @@ You have persistent memory powered by Eagle Mem. When you recall context from a
 IMPORTANT: At the start of your VERY NEXT response (this fires on session start, /clear, AND context compaction — always show this block, even if you think you showed it before, because prior context may have been compressed away). Show the user what Eagle Mem loaded using this exact format:
 
 \`\`\`
-█▀▀ ▄▀█ █▀▀ █   █▀▀   █▀▄▀█ █▀▀ █▀▄▀█
-██▄ █▀█ █▄█ █▄▄ ██▄   █ ▀ █ ██▄ █ ▀ █
+$eagle_logo
 
 Project: <project name>
 Sessions: N recent | Memories: N | Tasks: N pending

package/lib/common.sh

@@ -46,6 +46,8 @@ eagle_fts_sanitize() {
 # Claude Code session IDs are UUIDs or hex strings — reject anything else.
 eagle_validate_session_id() {
     local sid="$1"
+    # Length cap: Claude Code IDs are UUIDs/hex (36-64 chars). Reject oversized input.
+    [ ${#sid} -gt 128 ] && return 1
     [[ "$sid" =~ ^[A-Za-z0-9_-]+$ ]]
 }
 

package/lib/db.sh

@@ -18,7 +18,7 @@ eagle_db() {
 }
 
 eagle_db_pipe() {
-    { echo "$EAGLE_DB_SETUP"; cat; } | sqlite3 "$EAGLE_MEM_DB" 2>>"$EAGLE_MEM_LOG"
+    { echo "$EAGLE_DB_SETUP"; echo ".bail on"; cat; } | sqlite3 "$EAGLE_MEM_DB" 2>>"$EAGLE_MEM_LOG"
 }
 
 eagle_db_json() {
@@ -45,7 +45,8 @@ eagle_upsert_session() {
               ON CONFLICT(id) DO UPDATE SET
                   cwd = COALESCE(excluded.cwd, sessions.cwd),
                   model = COALESCE(excluded.model, sessions.model),
-                  source = COALESCE(excluded.source, sessions.source);"
+                  source = COALESCE(excluded.source, sessions.source),
+                  status = 'active';"
 }
 
 eagle_end_session() {
@@ -78,7 +79,7 @@ eagle_insert_summary() {
     local notes; notes=$(eagle_sql_escape "${10:-}")
 
     eagle_db_pipe <<SQL
-INSERT OR REPLACE INTO summaries (session_id, project, request, investigated, learned, completed, next_steps, files_read, files_modified, notes)
+INSERT INTO summaries (session_id, project, request, investigated, learned, completed, next_steps, files_read, files_modified, notes)
 VALUES (
     '$session_id',
     '$project',
@@ -90,7 +91,17 @@ VALUES (
     '$files_read',
     '$files_modified',
     '$notes'
-);
+)
+ON CONFLICT(session_id) DO UPDATE SET
+    project        = excluded.project,
+    request        = COALESCE(NULLIF(excluded.request, ''), summaries.request),
+    investigated   = COALESCE(NULLIF(excluded.investigated, ''), summaries.investigated),
+    learned        = COALESCE(NULLIF(excluded.learned, ''), summaries.learned),
+    completed      = COALESCE(NULLIF(excluded.completed, ''), summaries.completed),
+    next_steps     = COALESCE(NULLIF(excluded.next_steps, ''), summaries.next_steps),
+    files_read     = COALESCE(NULLIF(excluded.files_read, '[]'), summaries.files_read),
+    files_modified = COALESCE(NULLIF(excluded.files_modified, '[]'), summaries.files_modified),
+    notes          = COALESCE(NULLIF(excluded.notes, ''), summaries.notes);
 SQL
 }
 
@@ -107,7 +118,8 @@ eagle_get_recent_summaries() {
 }
 
 eagle_search_summaries() {
-    local query; query=$(eagle_sql_escape "$1")
+    local query; query=$(eagle_fts_sanitize "$1")
+    query=$(eagle_sql_escape "$query")
     local project="${2:-}"
     local limit; limit=$(eagle_sql_int "${3:-10}")
 
@@ -156,7 +168,8 @@ eagle_get_overview() {
 }
 
 eagle_search_code_chunks() {
-    local query; query=$(eagle_sql_escape "$1")
+    local query; query=$(eagle_fts_sanitize "$1")
+    query=$(eagle_sql_escape "$query")
     local project; project=$(eagle_sql_escape "$2")
     local limit; limit=$(eagle_sql_int "${3:-5}")
 

package/lib/hooks.sh

@@ -0,0 +1,30 @@
+#!/usr/bin/env bash
+# ═══════════════════════════════════════════════════════════
+# Eagle Mem — Hook registration helpers
+# Shared by install.sh and update.sh
+# ═══════════════════════════════════════════════════════════
+
+eagle_patch_hook() {
+    local settings="$1"
+    local event="$2"
+    local matcher="$3"
+    local command="$4"
+    local description="${5:-}"
+
+    if jq -e ".hooks.${event}[]? | select(.hooks[]?.command == \"$command\")" "$settings" &>/dev/null; then
+        [ -n "$description" ] && eagle_ok "$description ${DIM}(already registered)${RESET}"
+        return
+    fi
+
+    local entry
+    if [ -n "$matcher" ]; then
+        entry=$(jq -nc --arg m "$matcher" --arg c "$command" '{matcher: $m, hooks: [{type: "command", command: $c}]}')
+    else
+        entry=$(jq -nc --arg c "$command" '{hooks: [{type: "command", command: $c}]}')
+    fi
+
+    local tmp
+    tmp=$(mktemp)
+    jq --argjson entry "$entry" ".hooks.${event} = ((.hooks.${event} // []) + [\$entry])" "$settings" > "$tmp" && mv "$tmp" "$settings"
+    [ -n "$description" ] && eagle_ok "$description"
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eagle-mem",
-  "version": "1.6.0",
+  "version": "1.6.2",
   "description": "Lightweight persistent memory for Claude Code — SQLite + FTS5, no daemon, no bloat",
   "bin": {
     "eagle-mem": "bin/eagle-mem"

package/scripts/help.sh

@@ -8,7 +8,7 @@ PACKAGE_DIR="$(cd "$SCRIPTS_DIR/.." && pwd)"
 
 . "$SCRIPTS_DIR/style.sh"
 
-version=$(node -e "console.log(require('$PACKAGE_DIR/package.json').version)" 2>/dev/null || echo "unknown")
+version=$(jq -r .version "$PACKAGE_DIR/package.json" 2>/dev/null || echo "unknown")
 
 eagle_banner
 

package/scripts/index.sh

@@ -104,7 +104,6 @@ fi
 project_sql=$(eagle_sql_escape "$PROJECT")
 NEEDS_INDEX="$TMPDIR_IDX/needs_index"
 
-indexed_count=0
 skipped_count=0
 
 while IFS= read -r file; do

package/scripts/install.sh

@@ -11,6 +11,7 @@ LIB_DIR="$SCRIPTS_DIR/../lib"
 
 . "$SCRIPTS_DIR/style.sh"
 . "$LIB_DIR/common.sh"
+. "$LIB_DIR/hooks.sh"
 
 SETTINGS="$EAGLE_SETTINGS"
 
@@ -160,47 +161,23 @@ if [ ! -f "$SETTINGS" ]; then
     echo '{}' > "$SETTINGS"
 fi
 
-patch_hook() {
-    local event="$1"
-    local matcher="$2"
-    local command="$3"
-    local description="$4"
-
-    if jq -e ".hooks.${event}[]? | select(.hooks[]?.command == \"$command\")" "$SETTINGS" &>/dev/null; then
-        eagle_ok "$description ${DIM}(already registered)${RESET}"
-        return
-    fi
-
-    local entry
-    if [ -n "$matcher" ]; then
-        entry="{\"matcher\": \"$matcher\", \"hooks\": [{\"type\": \"command\", \"command\": \"$command\"}]}"
-    else
-        entry="{\"hooks\": [{\"type\": \"command\", \"command\": \"$command\"}]}"
-    fi
-
-    local tmp
-    tmp=$(mktemp)
-    jq --argjson entry "$entry" ".hooks.${event} = ((.hooks.${event} // []) + [\$entry])" "$SETTINGS" > "$tmp" && mv "$tmp" "$SETTINGS"
-    eagle_ok "$description"
-}
-
-patch_hook "SessionStart" "" \
+eagle_patch_hook "$SETTINGS" "SessionStart" "" \
     "$EAGLE_MEM_DIR/hooks/session-start.sh" \
     "SessionStart hook"
 
-patch_hook "Stop" "" \
+eagle_patch_hook "$SETTINGS" "Stop" "" \
     "$EAGLE_MEM_DIR/hooks/stop.sh" \
     "Stop hook"
 
-patch_hook "PostToolUse" "Read|Write|Edit|Bash|TaskCreate|TaskUpdate" \
+eagle_patch_hook "$SETTINGS" "PostToolUse" "Read|Write|Edit|Bash|TaskCreate|TaskUpdate" \
     "$EAGLE_MEM_DIR/hooks/post-tool-use.sh" \
     "PostToolUse hook"
 
-patch_hook "SessionEnd" "" \
+eagle_patch_hook "$SETTINGS" "SessionEnd" "" \
     "$EAGLE_MEM_DIR/hooks/session-end.sh" \
     "SessionEnd hook"
 
-patch_hook "UserPromptSubmit" "" \
+eagle_patch_hook "$SETTINGS" "UserPromptSubmit" "" \
     "$EAGLE_MEM_DIR/hooks/user-prompt-submit.sh" \
     "UserPromptSubmit hook"
 

package/scripts/prune.sh

@@ -15,6 +15,33 @@ LIB_DIR="$SCRIPTS_DIR/../lib"
 
 eagle_ensure_db
 
+# ─── Help ────────────────────────────────────────────────
+
+show_help() {
+    echo -e "  ${BOLD}eagle-mem prune${RESET} — Clean up old data"
+    echo ""
+    echo -e "  ${BOLD}Usage:${RESET}"
+    echo -e "    eagle-mem prune                     ${DIM}# prune observations > 90 days${RESET}"
+    echo -e "    eagle-mem prune ${CYAN}--days 30${RESET}           ${DIM}# prune observations > 30 days${RESET}"
+    echo -e "    eagle-mem prune ${CYAN}--dry-run${RESET}           ${DIM}# show what would be pruned${RESET}"
+    echo ""
+    echo -e "  ${BOLD}What gets pruned:${RESET}"
+    echo -e "    ${DOT} Observations older than --days (default: 90)"
+    echo -e "    ${DOT} Code chunks for files that no longer exist"
+    echo ""
+    echo -e "  ${BOLD}What is preserved:${RESET}"
+    echo -e "    ${DOT} All sessions and summaries (your session history)"
+    echo -e "    ${DOT} All tasks"
+    echo -e "    ${DOT} Project overviews"
+    echo ""
+    echo -e "  ${BOLD}Options:${RESET}"
+    echo -e "    ${CYAN}-d, --days${RESET} <N>        Age threshold (default: 90)"
+    echo -e "    ${CYAN}-p, --project${RESET} <name>  Only prune for this project"
+    echo -e "    ${CYAN}-n, --dry-run${RESET}         Show counts without deleting"
+    echo ""
+    exit 0
+}
+
 # ─── Parse arguments ──────────────────────────────────────
 
 days=90
@@ -26,30 +53,7 @@ while [ $# -gt 0 ]; do
         --days|-d)      days="$2"; shift 2 ;;
         --project|-p)   project="$2"; shift 2 ;;
         --dry-run|-n)   dry_run=true; shift ;;
-        --help|-h)
-            echo -e "  ${BOLD}eagle-mem prune${RESET} — Clean up old data"
-            echo ""
-            echo -e "  ${BOLD}Usage:${RESET}"
-            echo -e "    eagle-mem prune                     ${DIM}# prune observations > 90 days${RESET}"
-            echo -e "    eagle-mem prune ${CYAN}--days 30${RESET}           ${DIM}# prune observations > 30 days${RESET}"
-            echo -e "    eagle-mem prune ${CYAN}--dry-run${RESET}           ${DIM}# show what would be pruned${RESET}"
-            echo ""
-            echo -e "  ${BOLD}What gets pruned:${RESET}"
-            echo -e "    ${DOT} Observations older than --days (default: 90)"
-            echo -e "    ${DOT} Code chunks for files that no longer exist"
-            echo ""
-            echo -e "  ${BOLD}What is preserved:${RESET}"
-            echo -e "    ${DOT} All sessions and summaries (your session history)"
-            echo -e "    ${DOT} All tasks"
-            echo -e "    ${DOT} Project overviews"
-            echo ""
-            echo -e "  ${BOLD}Options:${RESET}"
-            echo -e "    ${CYAN}-d, --days${RESET} <N>        Age threshold (default: 90)"
-            echo -e "    ${CYAN}-p, --project${RESET} <name>  Only prune for this project"
-            echo -e "    ${CYAN}-n, --dry-run${RESET}         Show counts without deleting"
-            echo ""
-            exit 0
-            ;;
+        --help|-h)  show_help ;;
         *)
             eagle_err "Unknown option: $1"
             exit 1
@@ -109,7 +113,6 @@ if [ -n "$projects" ]; then
 
         if [ -n "$proj_cwd" ] && [ -d "$proj_cwd" ]; then
             if [ "$dry_run" = true ]; then
-                orphan_count=$(eagle_db "SELECT COUNT(DISTINCT file_path) FROM code_chunks WHERE project = '$(eagle_sql_escape "$proj")';")
                 # Count files that no longer exist
                 orphans=0
                 paths=$(eagle_db "SELECT DISTINCT file_path FROM code_chunks WHERE project = '$(eagle_sql_escape "$proj")';")

package/scripts/search.sh

@@ -14,6 +14,27 @@ LIB_DIR="$SCRIPTS_DIR/../lib"
 
 eagle_ensure_db
 
+# ─── Help ────────────────────────────────────────────────
+
+show_help() {
+    echo -e "  ${BOLD}eagle-mem search${RESET} — Search persistent memory"
+    echo ""
+    echo -e "  ${BOLD}Usage:${RESET}"
+    echo -e "    eagle-mem search ${CYAN}<query>${RESET}                   ${DIM}# keyword search${RESET}"
+    echo -e "    eagle-mem search ${CYAN}--timeline${RESET}               ${DIM}# recent sessions${RESET}"
+    echo -e "    eagle-mem search ${CYAN}--session <id>${RESET}            ${DIM}# session details${RESET}"
+    echo -e "    eagle-mem search ${CYAN}--files${RESET}                  ${DIM}# frequently modified files${RESET}"
+    echo -e "    eagle-mem search ${CYAN}--stats${RESET}                  ${DIM}# project statistics${RESET}"
+    echo ""
+    echo -e "  ${BOLD}Options:${RESET}"
+    echo -e "    ${CYAN}-p, --project${RESET} <name>    Project name (default: current dir)"
+    echo -e "    ${CYAN}-n, --limit${RESET} <N>         Max results (default: 10)"
+    echo -e "    ${CYAN}-a, --all${RESET}               Search across all projects"
+    echo -e "    ${CYAN}-j, --json${RESET}              Output as JSON"
+    echo ""
+    exit 0
+}
+
 # ─── Parse arguments ──────────────────────────────────────
 
 mode="keyword"
@@ -34,24 +55,7 @@ while [ $# -gt 0 ]; do
         --limit|-n)          limit="$2"; shift 2 ;;
         --all|-a)            cross_project=true; shift ;;
         --json|-j)           json_output=true; shift ;;
-        --help|-h)
-            echo -e "  ${BOLD}eagle-mem search${RESET} — Search persistent memory"
-            echo ""
-            echo -e "  ${BOLD}Usage:${RESET}"
-            echo -e "    eagle-mem search ${CYAN}<query>${RESET}                   ${DIM}# keyword search${RESET}"
-            echo -e "    eagle-mem search ${CYAN}--timeline${RESET}               ${DIM}# recent sessions${RESET}"
-            echo -e "    eagle-mem search ${CYAN}--session <id>${RESET}            ${DIM}# session details${RESET}"
-            echo -e "    eagle-mem search ${CYAN}--files${RESET}                  ${DIM}# frequently modified files${RESET}"
-            echo -e "    eagle-mem search ${CYAN}--stats${RESET}                  ${DIM}# project statistics${RESET}"
-            echo ""
-            echo -e "  ${BOLD}Options:${RESET}"
-            echo -e "    ${CYAN}-p, --project${RESET} <name>    Project name (default: current dir)"
-            echo -e "    ${CYAN}-n, --limit${RESET} <N>         Max results (default: 10)"
-            echo -e "    ${CYAN}-a, --all${RESET}               Search across all projects"
-            echo -e "    ${CYAN}-j, --json${RESET}              Output as JSON"
-            echo ""
-            exit 0
-            ;;
+        --help|-h)  show_help ;;
         -*)
             eagle_err "Unknown option: $1"
             exit 1

package/scripts/tasks.sh

@@ -29,6 +29,7 @@ show_help() {
     echo -e "  ${BOLD}Usage:${RESET}"
     echo -e "    eagle-mem tasks                  ${DIM}# list pending/in-progress tasks${RESET}"
     echo -e "    eagle-mem tasks ${CYAN}list${RESET}             ${DIM}# list all tasks${RESET}"
+    echo -e "    eagle-mem tasks ${CYAN}completed${RESET}        ${DIM}# list completed tasks${RESET}"
     echo -e "    eagle-mem tasks ${CYAN}search${RESET} <query>   ${DIM}# search tasks by keyword${RESET}"
     echo ""
     echo -e "  ${BOLD}Options:${RESET}"

package/scripts/update.sh

@@ -12,6 +12,7 @@ LIB_DIR="$SCRIPTS_DIR/../lib"
 . "$SCRIPTS_DIR/style.sh"
 . "$LIB_DIR/common.sh"
 . "$LIB_DIR/db.sh"
+. "$LIB_DIR/hooks.sh"
 
 SETTINGS="$EAGLE_SETTINGS"
 
@@ -57,37 +58,17 @@ fi
 # ─── Re-register hooks (idempotent) ───────────────────────
 
 if [ -f "$SETTINGS" ] && command -v jq &>/dev/null; then
-    patch_hook() {
-        local event="$1"
-        local matcher="$2"
-        local command="$3"
-
-        if jq -e ".hooks.${event}[]? | select(.hooks[]?.command == \"$command\")" "$SETTINGS" &>/dev/null; then
-            return
-        fi
-
-        local entry
-        if [ -n "$matcher" ]; then
-            entry="{\"matcher\": \"$matcher\", \"hooks\": [{\"type\": \"command\", \"command\": \"$command\"}]}"
-        else
-            entry="{\"hooks\": [{\"type\": \"command\", \"command\": \"$command\"}]}"
-        fi
-
-        local tmp
-        tmp=$(mktemp)
-        jq --argjson entry "$entry" ".hooks.${event} = ((.hooks.${event} // []) + [\$entry])" "$SETTINGS" > "$tmp" && mv "$tmp" "$SETTINGS"
-    }
-
-    patch_hook "SessionStart" "" "$EAGLE_MEM_DIR/hooks/session-start.sh"
-    patch_hook "Stop" "" "$EAGLE_MEM_DIR/hooks/stop.sh"
     # Update PostToolUse matcher if it has the old value (pre-v1.3.0)
     if jq -e '.hooks.PostToolUse[]? | select(.matcher == "Read|Write|Edit|Bash")' "$SETTINGS" &>/dev/null; then
         _tmp=$(mktemp)
         jq '(.hooks.PostToolUse[] | select(.matcher == "Read|Write|Edit|Bash")).matcher = "Read|Write|Edit|Bash|TaskCreate|TaskUpdate"' "$SETTINGS" > "$_tmp" && mv "$_tmp" "$SETTINGS"
     fi
-    patch_hook "PostToolUse" "Read|Write|Edit|Bash|TaskCreate|TaskUpdate" "$EAGLE_MEM_DIR/hooks/post-tool-use.sh"
-    patch_hook "SessionEnd" "" "$EAGLE_MEM_DIR/hooks/session-end.sh"
-    patch_hook "UserPromptSubmit" "" "$EAGLE_MEM_DIR/hooks/user-prompt-submit.sh"
+
+    eagle_patch_hook "$SETTINGS" "SessionStart" "" "$EAGLE_MEM_DIR/hooks/session-start.sh"
+    eagle_patch_hook "$SETTINGS" "Stop" "" "$EAGLE_MEM_DIR/hooks/stop.sh"
+    eagle_patch_hook "$SETTINGS" "PostToolUse" "Read|Write|Edit|Bash|TaskCreate|TaskUpdate" "$EAGLE_MEM_DIR/hooks/post-tool-use.sh"
+    eagle_patch_hook "$SETTINGS" "SessionEnd" "" "$EAGLE_MEM_DIR/hooks/session-end.sh"
+    eagle_patch_hook "$SETTINGS" "UserPromptSubmit" "" "$EAGLE_MEM_DIR/hooks/user-prompt-submit.sh"
 
     eagle_ok "Hooks registered"
 fi
@@ -117,5 +98,5 @@ fi
 
 # ─── Summary ───────────────────────────────────────────────
 
-version=$(node -e "console.log(require('$PACKAGE_DIR/package.json').version)" 2>/dev/null || echo "unknown")
+version=$(jq -r .version "$PACKAGE_DIR/package.json" 2>/dev/null || echo "unknown")
 eagle_footer "Eagle Mem updated to v${version}."