memorix 0.9.13 → 0.9.15

package/CHANGELOG.md

@@ -2,10 +2,24 @@
 
 All notable changes to this project will be documented in this file.
 
-## [0.9.13] — 2026-02-26
+## [0.9.15] — 2026-02-26
 
 ### Fixed
-- **🔴 Critical: Hooks never auto-store during development** — `extractContent()` had a fatal `parts.length === 0` guard that skipped rich `toolInput` data (file content, edit diffs, commands) whenever `toolResult` was present. Since Claude Code/Copilot/Cursor always send a short `toolResult` like `"File written successfully"` (28 chars), the extracted content was always < 100 chars and got rejected by the `MIN_STORE_LENGTH` threshold. Now always extracts `toolInput` fields (file_path, content, old_string/new_string, command, query) alongside `toolResult`, ensuring tool events produce enough content for pattern detection and auto-storage.
+- **Feedback visibility** — Hook auto-stores were silent. Now returns `systemMessage` to the agent after each save, e.g. `🟢 Memorix saved: Updated auth.ts [what-changed]`. Gives Codex-like visibility into what memorix is recording.
+- **File-modifying tools always store** — Write/Edit/MultiEdit tool events were rejected when content lacked pattern keywords (e.g., writing utility functions with no "error"/"fix" keywords). Now file-modifying tools always store if content > 100 chars, classified as `what-changed` by default.
+- **PreCompact low-quality spam** — PreCompact events stored empty/minimal observations with no meaningful content. Now requires `MIN_STORE_LENGTH` (100 chars) to store.
+- **Normalizer prompt extraction** — `normalizeClaude` only extracted `prompt` for `user_prompt` events. Now extracts for all events (PreCompact, etc.), preserving context that would otherwise be lost.
+
+## [0.9.14] — 2026-02-26
+
+### Fixed
+- **🔴 Critical: Hooks never auto-store during development** — Two root causes:
+  1. `extractContent()` had a fatal `parts.length === 0` guard that skipped rich `toolInput` data (file content, edit diffs, commands) whenever `toolResult` was present. Since all agents send short `toolResult` like `"File written successfully"` (28 chars), the content was always < 100 chars and got rejected by `MIN_STORE_LENGTH`.
+  2. Bash/shell tool events (npm install, npm test, git commands) also got rejected because their content (~90 chars) fell below the generic `post_tool` threshold of 200 chars, even though commands are inherently meaningful.
+- **Fix**: Always extract `toolInput` fields alongside `toolResult`. Bash tools now use a dedicated low-threshold path (50 chars) with noise command filtering, matching the `post_command` logic.
+
+### Added
+- **12 Claude Code E2E tests** — Validates the full hook pipeline (stdin JSON → normalize → handleHookEvent → observation) for Write, Edit, Bash, UserPromptSubmit, SessionStart, Stop, PreCompact, and edge cases (noise filtering, memorix recursion skip, short prompts).
 
 ## [0.9.12] — 2026-02-25
 

package/README.md

@@ -8,7 +8,7 @@
     <a href="https://www.npmjs.com/package/memorix"><img src="https://img.shields.io/npm/dm/memorix.svg?style=flat-square&color=blue" alt="npm downloads"></a>
     <a href="LICENSE"><img src="https://img.shields.io/badge/license-Apache%202.0-green.svg?style=flat-square" alt="License"></a>
     <a href="https://github.com/AVIDS2/memorix"><img src="https://img.shields.io/github/stars/AVIDS2/memorix?style=flat-square&color=yellow" alt="GitHub stars"></a>
-    <img src="https://img.shields.io/badge/tests-491%20passed-brightgreen?style=flat-square" alt="Tests">
+    <img src="https://img.shields.io/badge/tests-505%20passed-brightgreen?style=flat-square" alt="Tests">
   </p>
   <p align="center">
     <img src="https://img.shields.io/badge/Works%20with-Cursor-orange?style=flat-square" alt="Cursor">

package/README.zh-CN.md

@@ -8,7 +8,7 @@
     <a href="https://www.npmjs.com/package/memorix"><img src="https://img.shields.io/npm/dm/memorix.svg?style=flat-square&color=blue" alt="npm downloads"></a>
     <a href="LICENSE"><img src="https://img.shields.io/badge/license-Apache%202.0-green.svg?style=flat-square" alt="License"></a>
     <a href="https://github.com/AVIDS2/memorix"><img src="https://img.shields.io/github/stars/AVIDS2/memorix?style=flat-square&color=yellow" alt="GitHub stars"></a>
-    <img src="https://img.shields.io/badge/tests-491%20passed-brightgreen?style=flat-square" alt="Tests">
+    <img src="https://img.shields.io/badge/tests-505%20passed-brightgreen?style=flat-square" alt="Tests">
   </p>
   <p align="center">
     <img src="https://img.shields.io/badge/Works%20with-Cursor-orange?style=flat-square" alt="Cursor">

package/dist/cli/index.js

@@ -43369,8 +43369,8 @@ function normalizeClaude(payload, event) {
       result.filePath = toolInput?.file_path ?? toolInput?.filePath;
     }
   }
-  if (event === "user_prompt") {
-    result.userPrompt = payload.prompt ?? "";
+  if (payload.prompt) {
+    result.userPrompt = payload.prompt;
   }
   return result;
 }
@@ -43747,6 +43747,7 @@ var init_pattern_detector = __esm({
 var handler_exports = {};
 __export(handler_exports, {
   handleHookEvent: () => handleHookEvent,
+  resetCooldowns: () => resetCooldowns,
   runHook: () => runHook
 });
 function isInCooldown(eventKey) {
@@ -43757,6 +43758,9 @@ function isInCooldown(eventKey) {
 function markTriggered(eventKey) {
   cooldowns.set(eventKey, Date.now());
 }
+function resetCooldowns() {
+  cooldowns.clear();
+}
 function extractContent(input) {
   const parts = [];
   if (input.userPrompt) parts.push(input.userPrompt);
@@ -43822,7 +43826,8 @@ function generateTitle(input, patternType) {
 }
 function buildObservation(input, content) {
   const pattern = detectBestPattern(content);
-  const obsType = pattern ? patternToObservationType(pattern.type) : "discovery";
+  const fallbackType = input.filePath ? "what-changed" : "discovery";
+  const obsType = pattern ? patternToObservationType(pattern.type) : fallbackType;
   return {
     entityName: deriveEntityName(input),
     type: obsType,
@@ -43922,11 +43927,16 @@ ${lines.join("\n")}`;
         }
       };
     }
-    case "pre_compact":
+    case "pre_compact": {
+      const compactContent = extractContent(input);
+      if (compactContent.length < MIN_STORE_LENGTH) {
+        return { observation: null, output: defaultOutput };
+      }
       return {
-        observation: buildObservation(input, extractContent(input)),
+        observation: buildObservation(input, compactContent),
         output: defaultOutput
       };
+    }
     case "session_end":
       return {
         observation: buildObservation(input, extractContent(input)),
@@ -43976,9 +43986,32 @@ ${lines.join("\n")}`;
         return { observation: null, output: defaultOutput };
       }
       const toolContent = extractContent(input);
+      if (input.command) {
+        if (NOISE_COMMANDS.some((r4) => r4.test(input.command))) {
+          return { observation: null, output: defaultOutput };
+        }
+        if (toolContent.length < 50) {
+          return { observation: null, output: defaultOutput };
+        }
+        markTriggered(toolKey);
+        return {
+          observation: buildObservation(input, toolContent),
+          output: defaultOutput
+        };
+      }
       if (toolContent.length < MIN_STORE_LENGTH) {
         return { observation: null, output: defaultOutput };
       }
+      const isFileModifyingTool = /^(write|edit|multi_?edit|multiedittool|create|patch|insert)/i.test(
+        input.toolName ?? ""
+      );
+      if (isFileModifyingTool) {
+        markTriggered(toolKey);
+        return {
+          observation: buildObservation(input, toolContent),
+          output: defaultOutput
+        };
+      }
       const toolPattern = detectBestPattern(toolContent);
       if (!toolPattern && toolContent.length < 200) {
         return { observation: null, output: defaultOutput };
@@ -44039,6 +44072,20 @@ async function runHook() {
       const dataDir = await getProjectDataDir2(project.id);
       await initObservations2(dataDir);
       await storeObservation2({ ...observation, projectId: project.id });
+      const TYPE_EMOJI = {
+        "gotcha": "\u{1F534}",
+        "decision": "\u{1F7E4}",
+        "problem-solution": "\u{1F7E1}",
+        "trade-off": "\u2696\uFE0F",
+        "discovery": "\u{1F7E3}",
+        "how-it-works": "\u{1F535}",
+        "what-changed": "\u{1F7E2}",
+        "why-it-exists": "\u{1F7E0}",
+        "session-request": "\u{1F3AF}"
+      };
+      const emoji2 = TYPE_EMOJI[observation.type] ?? "\u{1F4DD}";
+      output.systemMessage = (output.systemMessage ?? "") + `
+${emoji2} Memorix saved: ${observation.title} [${observation.type}]`;
     } catch {
     }
   }