pi-hermes-memory 0.6.0 → 0.6.1

package/README.md

@@ -38,6 +38,7 @@ pi install npm:pi-hermes-memory
 |---|---|
 | 🔍 **Session Search** | Search across all past conversations via SQLite FTS5 |
 | 🧠 **Persistent Memory** | Facts, preferences, lessons saved to markdown files |
+| ⚠️ **Failure Memory** | Learn from failures — stores what didn't work and why |
 | 📚 **Procedural Skills** | The agent saves *how* it solved problems as reusable docs |
 | ⚡ **Background Learning** | Every 10 turns (or 15 tool calls) the agent reviews and saves |
 | 🔧 **Correction Detection** | When you correct the agent, it saves immediately |
@@ -124,9 +125,51 @@ System Prompt
 │ • tests use node:test with tsx          │
 │ ═══ END MEMORY ═══                     │
 │ </memory-context>                       │
+│                                         │
+│ <memory-context>                        │
+│ RECENT FAILURES & LESSONS (learn from): │
+│ • [correction] Use pnpm, not npm        │
+│ • [failure] Tried localStorage — XSS    │
+│ • [insight] Auth0 handles refresh tokens│
+│ ═══ END MEMORY ═══                     │
+│ </memory-context>                       │
 └─────────────────────────────────────────┘
 ```
 
+## Failure Memory
+
+The agent learns from failures, corrections, and insights — just like humans do.
+
+### Memory Categories
+
+| Category | What it stores | Example |
+|---|---|---|
+| `failure` | What didn't work and why | "Tried localStorage for tokens — XSS vulnerability" |
+| `correction` | User corrections | "Use pnpm, not npm" |
+| `insight` | Learnings from experience | "Auth0 SDK handles refresh tokens automatically" |
+| `preference` | User preferences | "Prefers dark theme" |
+| `convention` | Project conventions | "Monorepo uses turborepo" |
+| `tool-quirk` | Tool-specific knowledge | "CI needs --frozen-lockfile" |
+
+### How It Works
+
+1. **Auto-detection**: Background review extracts failures from conversations
+2. **Correction capture**: When you correct the agent, it saves what went wrong
+3. **System prompt injection**: Recent failures (last 7 days) are injected at session start
+4. **Searchable**: Use `memory_search("auth", category: "failure")` to find past failures
+
+### Example
+
+```
+User: No, use pnpm not npm
+Agent: [saves correction memory]
+
+Next session:
+Agent: "I remember you prefer pnpm over npm. Let me use that."
+```
+
+The agent learns from its mistakes so you don't have to repeat yourself.
+
 Memory blocks are wrapped in `<memory-context>` XML tags with a guard note ("NOT new user input") to prevent the LLM from treating stored facts as instructions.
 
 ## Usage

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-hermes-memory",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "🧠 Persistent memory + 🔍 session search + 🛡️ secret scanning for Pi. SQLite FTS5 search across every conversation, auto-consolidation, memory aging, procedural skills. 272 tests. Ported from Hermes agent.",
   "type": "module",
   "main": "src/index.ts",

package/src/handlers/auto-consolidate.ts

@@ -15,7 +15,7 @@ import type { ConsolidationResult } from "../types.js";
 export async function triggerConsolidation(
   pi: ExtensionAPI,
   store: MemoryStore,
-  target: "memory" | "user",
+  target: "memory" | "user" | "failure",
   signal?: AbortSignal,
 ): Promise<ConsolidationResult> {
   const entries =

package/src/store/memory-store.ts

@@ -29,7 +29,7 @@ export class MemoryStore {
   private userEntries: string[] = [];
   private failureEntries: string[] = [];
   private snapshot: MemorySnapshot = { memory: "", user: "" };
-  private consolidator: ((target: "memory" | "user", signal?: AbortSignal) => Promise<ConsolidationResult>) | null = null;
+  private consolidator: ((target: "memory" | "user" | "failure", signal?: AbortSignal) => Promise<ConsolidationResult>) | null = null;
 
   constructor(private config: MemoryConfig) {}
 
@@ -37,7 +37,7 @@ export class MemoryStore {
    * Inject a consolidation function (avoids circular imports).
    * Called from index.ts after both store and pi are available.
    */
-  setConsolidator(fn: (target: "memory" | "user", signal?: AbortSignal) => Promise<ConsolidationResult>): void {
+  setConsolidator(fn: (target: "memory" | "user" | "failure", signal?: AbortSignal) => Promise<ConsolidationResult>): void {
     this.consolidator = fn;
   }
 
@@ -100,7 +100,7 @@ export class MemoryStore {
 
   // ─── CRUD ───
 
-  async add(target: "memory" | "user", content: string, signal?: AbortSignal): Promise<MemoryResult> {
+  async add(target: "memory" | "user" | "failure", content: string, signal?: AbortSignal): Promise<MemoryResult> {
     return this._add(target, content, signal);
   }
 
@@ -202,7 +202,7 @@ export class MemoryStore {
     return this.successResponse(target, "Entry added.");
   }
 
-  async replace(target: "memory" | "user", oldText: string, newContent: string): Promise<MemoryResult> {
+  async replace(target: "memory" | "user" | "failure", oldText: string, newContent: string): Promise<MemoryResult> {
     oldText = oldText.trim();
     newContent = newContent.trim();
     if (!oldText) return { success: false, error: "old_text cannot be empty." };
@@ -248,7 +248,7 @@ export class MemoryStore {
     return this.successResponse(target, "Entry replaced.");
   }
 
-  async remove(target: "memory" | "user", oldText: string): Promise<MemoryResult> {
+  async remove(target: "memory" | "user" | "failure", oldText: string): Promise<MemoryResult> {
     oldText = oldText.trim();
     if (!oldText) return { success: false, error: "old_text cannot be empty." };