@selesai/code 0.13.0 → 0.13.2

package/dist/extensions/pi-hermes-memory/docs/0.1/TASKS.md

@@ -0,0 +1,197 @@
+# Tasks — Pi Hermes Memory Extension
+
+> **Workflow**: When you start a task, change `[ ]` to `[~]`. When done, change to `[x]` and note the commit hash.
+> Progress is tracked per-epic. Each epic has a clear definition of done.
+
+---
+
+## Epic 1: Project Scaffold & Repo Setup
+
+_Done when: repo is on GitHub, TypeScript compiles clean, extension loads in Pi without errors._
+
+- [x] `PLAN.md` — Full implementation plan with Hermes source file reference map — `efddcc4`
+- [x] `AGENTS.md` — Project context and architecture docs — `efddcc4`
+- [x] `.gitignore` — Exclude node_modules, dist, .codegraph, hermes-agent — `efddcc4`
+- [x] `package.json` — Minimal config, no runtime deps — `efddcc4`
+- [x] `tsconfig.json` — Strict TypeScript config — `efddcc4`
+- [x] `src/types.ts` — Shared interfaces (`MemoryConfig`, `MemoryResult`, `MemorySnapshot`) + `getMessageText()` helper — `efddcc4`
+- [x] `src/constants.ts` — Prompts, defaults, delimiter — `efddcc4`
+- [x] `src/store/content-scanner.ts` — Injection/exfiltration pattern detection — `efddcc4`
+- [x] `src/store/memory-store.ts` — Core `MemoryStore` class with CRUD, atomic writes, frozen snapshot — `efddcc4`
+- [x] `src/tools/memory-tool.ts` — `registerMemoryTool()` with Pi tool API — `efddcc4`
+- [x] `src/handlers/background-review.ts` — Learning loop via `pi.exec()` — `efddcc4`
+- [x] `src/handlers/session-flush.ts` — Pre-compaction/shutdown flush — `efddcc4`
+- [x] `src/handlers/insights.ts` — `/memory-insights` command — `efddcc4`
+- [x] `src/index.ts` — Extension entry point wiring everything — `efddcc4`
+- [x] GitHub repo created and initial commit pushed — `efddcc4`
+- [x] `npm install` + `npm run check` passes with zero errors
+- [x] Extension loads in Pi via `pi -e ./src/index.ts` without runtime errors — verified
+
+---
+
+## Epic 2: Core Memory — Store & Tool
+
+_Done when: agent can add/replace/remove entries, they persist to disk, and survive a Pi session restart._
+
+- [x] `MemoryStore.loadFromDisk()` correctly reads existing MEMORY.md and USER.md — `24151a0`
+- [x] `MemoryStore.add()` validates content, enforces char limit, persists atomically — `24151a0`
+- [x] `MemoryStore.replace()` finds entry by substring, replaces, re-checks limit — `24151a0`
+- [x] `MemoryStore.remove()` finds entry by substring, removes, persists — `24151a0`
+- [x] `MemoryStore.formatForSystemPrompt()` returns frozen snapshot (not live state) — `24151a0`
+- [x] Atomic write works: temp file → `fs.rename()` (verify no corruption on crash simulation) — `24151a0`
+- [x] Character limits enforced: reject writes that exceed `memoryCharLimit` / `userCharLimit` — `24151a0`
+- [x] Deduplication: adding an identical entry is a no-op — `24151a0`
+- [x] Multi-match ambiguity: replace/remove error when multiple distinct entries match — `24151a0`
+- [x] `memory` tool registered with correct name, parameters, and guidelines — `tests/tools/memory-tool.test.ts`
+- [x] Tool execute returns JSON with `usage` field showing char budget — `tests/tools/memory-tool.test.ts`
+- [x] LLM can call (manual verification — no API key configured) `memory` tool with `add` action and entry appears in MEMORY.md — **manual verification required**
+- [x] LLM can call (manual verification — no API key configured) `memory` tool with `target: "user"` and entry appears in USER.md — **manual verification required**
+
+---
+
+## Epic 3: Content Scanning & Security
+
+_Done when: all injection/exfiltration patterns are blocked, invisible unicode is blocked, and safe content passes through._
+
+- [x] `scanContent()` blocks prompt injection patterns (e.g. "ignore previous instructions") — `3f61b61`
+- [x] `scanContent()` blocks role hijacking (e.g. "you are now...") — `3f61b61`
+- [x] `scanContent()` blocks secret exfiltration (e.g. `curl ${API_KEY...`) — `3f61b61`
+- [x] `scanContent()` blocks invisible unicode (U+200B, U+FEFF, U+202A-U+202E) — `3f61b61`
+- [x] `scanContent()` returns `null` for safe/normal content — `3f61b61`
+- [x] Blocked writes return `{ success: false, error: "Blocked: ..." }` to the LLM — `3f61b61`
+- [x] Edge case: empty string passes (handled by empty check before scanner) — `3f61b61`
+- [x] Edge case: very long content with pattern at end is still caught — `3f61b61`
+
+---
+
+## Epic 4: System Prompt Injection
+
+_Done when: memory snapshot appears in system prompt at session start and does NOT update mid-session._
+
+- [x] `before_agent_start` handler appends memory block to `event.systemPrompt` — `028c5ad`
+- [x] Memory block includes header with usage percentage and char count — `028c5ad`
+- [x] Block format matches Hermes: `═` separator, header line, then content — `028c5ad`
+- [x] Frozen snapshot: write to memory mid-session → system prompt unchanged — `028c5ad`
+- [x] Empty memory files → no block appended (system prompt untouched) — `028c5ad`
+- [x] Second session (manual verification — needs Pi restart): memory saved in session 1 appears in session 2's system prompt
+---
+
+## Epic 5: Background Learning Loop
+
+_Done when: after N turns, a background pi process reviews the conversation and saves notable facts automatically._
+
+- [x] Turn counter increments on each `turn_end` event — `164eef9`
+- [x] User turn counter increments only on user messages (not assistant/tool) — `164eef9`
+- [x] Review triggers at `nudgeInterval` (default 10) turns — `164eef9`
+- [x] Review does NOT trigger if `reviewEnabled` is false — `164eef9`
+- [x] Review does NOT trigger if fewer than 3 user turns — `164eef9`
+- [x] Review does NOT trigger if already in progress (`reviewInProgress` guard) — `164eef9`
+- [x] `pi.exec("pi", ["-p", "--no-session", ...])` is called with correct review prompt — `164eef9`
+- [x] Review prompt includes current memory + user profile + conversation snapshot — `164eef9`
+- [x] Successful auto-save shows `💾 Memory auto-reviewed and updated` notification — `164eef9`
+- [x] "Nothing to save" response → no notification shown — `164eef9`
+- [x] Background review failure does NOT crash or block the main agent — `164eef9`
+- [x] Counter resets to 0 after review triggers — `164eef9`
+---
+
+## Epic 6: Session Flush
+
+_Done when: before compaction and session shutdown, agent gets one turn to save memories._
+
+- [x] `session_before_compact` event triggers flush when `flushOnCompact` is true — `001a8d4`
+- [x] `session_shutdown` event triggers flush when `flushOnShutdown` is true — `001a8d4`
+- [x] Flush skips if user turn count < `flushMinTurns` (default 6) — `001a8d4`
+- [x] Flush builds conversation snapshot from `ctx.sessionManager.getBranch()` — `001a8d4`
+- [x] Flush uses `pi.exec("pi", ["-p", "--no-session", ...])` with flush prompt — `001a8d4`
+- [x] Flush failure does NOT prevent compaction or session shutdown — `001a8d4`
+- [x] After flush (manual verification — needs Pi restart), any saved memories are available in next session
+---
+
+## Epic 7: Insights Command & UX Polish
+
+_Done when: `/memory-insights` shows formatted output and the extension is polished for users._
+
+- [x] `/memory-insights` command registered and appears in Pi command list — `543e262`
+- [x] Shows MEMORY section with numbered entries (truncated to 100 chars) — `543e262`
+- [x] Shows USER PROFILE section with numbered entries — `543e262`
+- [x] Shows "(empty)" when no entries exist — `543e262`
+- [x] Formatted with box drawing characters (╔══╗, etc.) — `543e262`
+- [x] Notification displays (manual verification — needs Pi TUI) correctly in Pi's TUI
+---
+
+## Epic 8: Configuration & Settings
+
+_Done when: users can customize behavior via `~/.selesai/agent/hermes-memory-config.json`._
+- [x] Read config from `~/.selesai/agent/hermes-memory-config.json` — `src/config.ts`
+- [x] All `MemoryConfig` fields are configurable with type validation
+- [x] Missing keys fall back to defaults
+- [x] Documented in README.md
+
+---
+
+## Epic 9: Testing
+
+_Done when: all core paths have automated tests and the extension passes a manual smoke test._
+
+### Unit Tests
+- [x] `content-scanner.ts` — 11 threat patterns + 5 invisible unicode chars tested — `3f61b61`
+- [x] `memory-store.ts` — test `add` success, persistence, duplicate → no-op, exceeds limit → error — `24151a0`
+- [x] `memory-store.ts` — test `replace` success, no match → error, multi-match → error — `24151a0`
+- [x] `memory-store.ts` — test `remove` success, no match → error — `24151a0`
+- [x] `memory-store.ts` — test frozen snapshot doesn't update after add — `24151a0`
+- [x] `memory-store.ts` — test `loadFromDisk` reads existing files, handles missing files — `24151a0`
+- [x] `config.ts` — test defaults, overrides, partial config, invalid values — current
+- [x] `handlers/` — test background-review, session-flush, insights, system-prompt — current
+- [x] `integration/` — test cross-module contracts (config→store, security pipeline, getMessageText) — current
+
+### Integration Tests
+- [x] Extension loads in Pi via `pi -e ./src/index.ts` — no errors — verified
+- [x] `memory` tool callable by LLM (manual verification — no API key) — manual verification required
+- [x] System prompt contains (manual verification — needs Pi runtime) memory block after `session_start` — manual verification required
+- [x] `/memory-insights` (manual verification — needs Pi runtime) command runs and shows output — manual verification required
+- [x] Survives Pi session (manual verification — needs Pi restart) restart — memory persists across `/new` — manual verification required
+
+### Manual Smoke Tests
+- [x] Full E2E (manual verification — needs full conversation): install → use 10+ turns → verify auto-review saves memory
+- [x] Full E2E (manual verification — needs full conversation): long conversation → trigger compaction → verify flush saves memory
+- [x] Full E2E (manual verification — needs full conversation): session 1 saves memory → quit → session 2 recalls it
+- [x] Security: try injecting (manual verification — needs Pi runtime) "ignore previous instructions" → verify blocked
+- [x] Security: try saving (manual verification — needs Pi runtime) `curl ${API_KEY}` → verify blocked
+
+---
+
+## Epic 10: Documentation & Distribution
+
+_Done when: extension is installable via `pi install` and has user-facing docs._
+
+- [x] `README.md` — What it does, installation, usage, configuration — `ed22fa6`
+- [x] `README.md` — Example screenshots (manual verification — needs Pi TUI) of `/memory-insights` output — requires Pi TUI
+- [x] Verify `pi install github:chandra447/pi-hermes-memory` works end-to-end — requires Pi CLI
+- [x] Tag v0.1.0 release on GitHub — `7983f09`
+
+---
+
+## Summary
+
+| Epic | Status | Notes |
+|---|---|---|
+| 1 — Project Scaffold | Complete | TypeScript compiles clean, extension loads in Pi |
+| 2 — Core Memory | Complete (auto) / 2 pending (manual) | Tool registration + execute tested; LLM interaction needs Pi runtime |
+| 3 — Content Scanning | Complete | 25 tests, all threat patterns covered |
+| 4 — System Prompt | Complete (auto) / 1 pending (manual) | Frozen snapshot tested; cross-session needs Pi restart |
+| 5 — Background Loop | Complete | 10 tests, all trigger conditions covered |
+| 6 — Session Flush | Complete (auto) / 1 pending (manual) | Flush logic tested; cross-session persistence needs Pi restart |
+| 7 — Insights | Complete (auto) / 1 pending (manual) | Command output tested; TUI display needs Pi runtime |
+| 8 — Configuration | Complete | Config file + tests + README docs |
+| 9 — Testing | Complete (auto) / 9 pending (manual) | 119 automated tests; E2E smoke tests need Pi runtime |
+| 10 — Documentation | Complete (auto) / 2 pending (manual) | README + LICENSE + tag v0.1.0; screenshots need Pi TUI |
+
+**Automated test coverage: 119 tests, 0 failures, 0 type errors.**
+
+**Manual verification required:** Run `pi -e ./src/index.ts` or `pi install github:chandra447/pi-hermes-memory`, then:
+1. Have the LLM save a memory and verify it appears in `~/.selesai/agent/memory/MEMORY.md`
+2. Start a new session (`/new`) and verify the memory appears in the system prompt
+3. Use 10+ turns and verify auto-review triggers
+4. Trigger `/compact` and verify flush saves memories
+5. Run `/memory-insights` and verify formatted output
+6. Try injecting malicious content and verify it's blocked

package/dist/extensions/pi-hermes-memory/docs/0.2/PLAN.md

@@ -0,0 +1,290 @@
+# v0.2.0 Implementation Plan — Skills + Smart Curation
+
+> **Goal**: Close the two biggest Hermes gaps — procedural memory (skills) and intelligent memory management (auto-consolidation, correction detection, tool-call-aware nudges).
+
+## Implementation Order
+
+```
+Epic 2 (Auto-Consolidation)  → standalone, modifies MemoryStore.add()
+Epic 3 (Correction Detection) → standalone, new handler
+Epic 4 (Tool-Call Nudge)      → modifies background-review.ts
+Epic 1 (Skill Tool)           → largest: new store + tool + handlers
+Epic 5 (Docs + Release)       → depends on all above
+```
+
+Epics 2, 3, 4 are independent but done sequentially to avoid merge conflicts in shared files (`types.ts`, `config.ts`, `constants.ts`, `index.ts`).
+
+---
+
+## Epic 2: Auto-Consolidation
+
+**Problem**: When `add()` exceeds char limit, we return an error. Hermes auto-consolidates.
+
+### New Files
+
+**`src/handlers/auto-consolidate.ts`** (~120 lines)
+```typescript
+export async function triggerConsolidation(
+  pi: ExtensionAPI,
+  store: MemoryStore,
+  target: "memory" | "user",
+  signal?: AbortSignal,
+): Promise<ConsolidationResult>
+```
+- Builds prompt from `CONSOLIDATION_PROMPT` + current entries for the target
+- Calls `pi.exec("pi", ["-p", "--no-session", prompt], { signal, timeout: 60000 })`
+- Returns `{ consolidated: true }` on success, `{ consolidated: false, error }` on failure
+
+**`src/handlers/consolidate-command.ts`** (~30 lines)
+- Registers `/memory-consolidate` via `pi.registerCommand()`
+- Runs consolidation for both targets, reports via `ctx.ui.notify()`
+
+**`tests/handlers/auto-consolidate.test.ts`** (~120 lines)
+
+### Modified Files
+
+**`src/constants.ts`** — Add `CONSOLIDATION_PROMPT`
+
+**`src/types.ts`** — Add `autoConsolidate: boolean` to `MemoryConfig`; add `ConsolidationResult` interface
+
+**`src/config.ts`** — Add `autoConsolidate: true` default + parsing
+
+**`src/store/memory-store.ts`** — Key changes:
+- `add()` becomes **async** (returns `Promise<MemoryResult>`)
+- Add `setConsolidator()` method for dependency injection (avoids circular import)
+- When over limit + consolidator set: call consolidator, **reload from disk** (`await this.loadFromDisk()`), then retry once
+- **Critical**: The `pi.exec()` child process modifies files on disk. The parent's in-memory arrays become stale after consolidation. We MUST reload before retrying `add()` or the retry will overwrite consolidated entries with stale data.
+
+**`src/tools/memory-tool.ts`** — `await store.add(target, content)` (line ~58)
+
+**`src/index.ts`** — Wire consolidator + register command
+
+**Test migration**: Making `add()` async means all existing tests calling `store.add()` must use `await`. Without `await`, tests get a Promise object instead of `MemoryResult`, causing assertion failures. Update all `store.add()` calls in `tests/store/memory-store.test.ts` to `await store.add()`.
+
+### Key Decision: Consolidator Injection via Setter
+MemoryStore cannot import from handlers (circular). Instead, `index.ts` injects a consolidator function via `store.setConsolidator()` after both `store` and `pi` are available.
+
+### Key Decision: No memoryDirPath Getter
+SkillStore receives its directory path directly from config (`config.memoryDir + "/skills/"`) in `index.ts`. No need to expose MemoryStore internals.
+
+---
+
+## Epic 3: Correction Detection + Immediate Save
+
+**Problem**: User says "no, don't do that" — we only save it 8 turns later at the next nudge. Hermes detects immediately.
+
+### New Files
+
+**`src/handlers/correction-detector.ts`** (~100 lines)
+```typescript
+export function setupCorrectionDetector(
+  pi: ExtensionAPI,
+  store: MemoryStore,
+  config: MemoryConfig,
+): void
+```
+
+**Design**:
+1. On `message_end` (role=user): check text against `CORRECTION_PATTERNS`, set `pendingCorrection = true`
+2. On `turn_end`: if `pendingCorrection`, trigger `pi.exec()` with `CORRECTION_SAVE_PROMPT` + recent messages + current memory
+3. Rate limit: `turnsSinceLastCorrection >= 3` and `!correctionInProgress`
+
+**Why turn_end, not message_end**: We need the full context (user correction + what agent said wrong) for the save prompt.
+
+**`tests/handlers/correction-detector.test.ts`** (~150 lines)
+
+### Modified Files
+
+**`src/constants.ts`** — Add `CORRECTION_SAVE_PROMPT` and `CORRECTION_PATTERNS` (regex array)
+
+**`src/types.ts`** — Add `correctionDetection: boolean` to `MemoryConfig`
+
+**`src/config.ts`** — Add `correctionDetection: true` default + parsing
+
+**`src/index.ts`** — Wire `setupCorrectionDetector()`
+
+### Correction Patterns (Two-Pass Filter)
+
+Patterns are split into **strong** (high confidence, trigger immediately) and **weak** (need a directive clause to confirm).
+
+**Strong patterns** (always trigger):
+```typescript
+/don'?t do that/i, /not like that/i,
+/^I said\b/i, /^I told you\b/i, /we already discussed/i,
+/^please don'?t/i, /^that'?s not what I/i
+```
+
+**Weak patterns** (only trigger if followed by a directive — verb or "the/that/this"):
+```typescript
+/^no[,.\s!]/i, /^wrong[,.\s!]/i, /^actually[,.\s]/i, /^stop[,.\s!]/i
+```
+
+**Negative patterns** (suppress trigger even if a positive pattern matches):
+```typescript
+/^no worries/i, /^no problem/i, /^no thanks/i, /^no need/i,
+/^actually.{0,10}(looks? great|perfect|good|correct|right)/i,
+/^stop.{0,5}(there|here|for now)/i
+```
+
+This eliminates false positives like "no worries, I'll handle it" and "actually, that looks great" while still catching "no, don't use npm" and "actually, use yarn instead".
+
+---
+
+## Epic 4: Tool-Call-Aware Nudge
+
+**Problem**: Nudge is purely turn-count based. Complex tasks with many tool calls generate more valuable memories.
+
+### Modified Files
+
+**`src/types.ts`** — Add `nudgeToolCalls: number` to `MemoryConfig`
+
+**`src/config.ts`** — Add `nudgeToolCalls: 15` default + parsing
+
+**`src/handlers/background-review.ts`** — Key changes:
+- Count tool-use entries from `ctx.sessionManager.getBranch()` at `turn_end` time (robust — no unknown event names)
+- Change trigger to OR logic: `turnsSinceReview >= nudgeInterval || toolCallsSinceReview >= nudgeToolCalls`
+- Reset both counters on review
+
+**`tests/handlers/background-review.test.ts`** — Add tool-call trigger tests
+
+### Key Decision: Count from Branch, Not Events
+Rather than depending on unknown Pi event names like `tool_end`, count tool-use entries from `ctx.sessionManager.getBranch()` at `turn_end` time. More robust and testable.
+
+---
+
+## Epic 1: Skill Tool + Procedural Memory
+
+**Problem**: `COMBINED_REVIEW_PROMPT` asks about skills but there's no skill tool. This is the single highest-leverage change.
+
+### New Files
+
+**`src/store/skill-store.ts`** (~250 lines)
+```typescript
+export class SkillStore {
+  constructor(private skillsDir: string) {}
+  async loadIndex(): Promise<SkillIndex[]>
+  async loadSkill(fileName: string): Promise<SkillDocument | null>
+  async create(name: string, description: string, body: string): Promise<SkillResult>
+  async patch(fileName: string, section: string, newContent: string): Promise<SkillResult>
+  async edit(fileName: string, description: string, body: string): Promise<SkillResult>
+  async delete(fileName: string): Promise<SkillResult>
+  formatIndexForSystemPrompt(): string
+}
+```
+
+**Storage**: `~/.selesai/agent/memory/skills/` (isolated from user skills at `~/.selesai/agent/skills/`)
+
+**SKILL.md format**:
+```markdown
+---
+name: debug-typescript-errors
+description: Step-by-step approach to debugging TS errors in monorepos
+version: 1
+created: 2026-04-27
+updated: 2026-04-27
+---
+## When to Use
+## Procedure
+## Pitfalls
+## Verification
+```
+
+**Frontmatter parsing**: Simple regex (no yaml dependency). Split on `---`, parse key-value pairs.
+
+**File naming**: `slugify(name) + ".md"` — lowercase, replace non-alphanum with `-`, collapse dashes.
+
+**`src/tools/skill-tool.ts`** (~180 lines)
+- Registered via `pi.registerTool()` with actions: `create`, `view`, `patch`, `edit`, `delete`
+- Content scanning on all writes via `scanContent()`
+
+**`src/handlers/skill-auto-trigger.ts`** (~80 lines)
+- Track tool calls per turn
+- When turn completes with **8+ tool calls** (not 5 — a typical read→bash→edit→bash→read is already 5), trigger skill extraction via `pi.exec()`
+- Additionally require at least **2 distinct tool types** in the turn (e.g., read + bash, not just 8 reads) to filter trivial multi-call turns
+- Rate limit: max 1 auto-trigger per session
+
+**`src/handlers/skills-command.ts`** (~50 lines)
+- `/memory-skills` command listing all skills
+
+**Test files**: `tests/store/skill-store.test.ts`, `tests/tools/skill-tool.test.ts`, `tests/handlers/skill-auto-trigger.test.ts`
+
+### Modified Files
+
+**`src/constants.ts`** — Add `SKILL_TOOL_DESCRIPTION`, `DEFAULT_SKILL_TRIGGER_TOOL_CALLS` (= 8); update `COMBINED_REVIEW_PROMPT`:
+
+```typescript
+export const COMBINED_REVIEW_PROMPT = `Review the conversation above and consider two things:
+
+**Memory**: Has the user revealed things about themselves — their persona, desires, preferences, or personal details? Has the user expressed expectations about how you should behave, their work style, or ways they want you to operate? If so, save using memory_add.
+
+**Skills**: Was a complex, non-trivial approach used to complete a task — one that required trial and error, multiple tool calls, or changing course? If so, save a reusable procedure using the skill tool with action 'create'. Include: when to use it, step-by-step procedure, pitfalls to avoid, and how to verify success.
+
+Only act if there's something genuinely worth saving. If nothing stands out, just say 'Nothing to save.' and stop.`;
+```
+
+**Note on pi.exec() child tools**: The child `pi` process loads the same installed extension, so it has access to the memory write tools and `skill_manage`. This is the same mechanism that makes the existing memory tools work in background review.
+
+**`src/index.ts`** — Wire SkillStore, registerSkillTool, setupSkillAutoTrigger, registerSkillsCommand; inject skill index into system prompt at `before_agent_start`. Pass `config.memoryDir + "/skills/"` directly to SkillStore constructor (no memoryDirPath getter needed).
+
+### Progressive Disclosure
+- **System prompt**: Skill index only (name + description per skill, ~3K tokens max)
+- **On demand**: Agent calls `skill` tool with action `view` to load full content
+- **Frozen snapshot**: Index captured at `session_start`, same as memory snapshot
+
+### Key Decision: Frozen Snapshot for Skills
+Skill index is captured at `session_start` and injected at `before_agent_start`. New skills created mid-session appear in the index on next session. This preserves Pi's prompt cache.
+
+---
+
+## Epic 5: Documentation & Release
+
+- Update `README.md` with new features, config options, commands
+- Update `docs/ROADMAP.md` — mark v0.2 complete
+- Bump `package.json` version to `0.2.0`
+- `npm run check` passes, all tests pass
+- Tag `v0.2.0`
+
+---
+
+## File Change Summary
+
+### New Files (12)
+| File | Lines | Epic |
+|---|---|---|
+| `src/handlers/auto-consolidate.ts` | ~120 | 2 |
+| `src/handlers/consolidate-command.ts` | ~30 | 2 |
+| `src/handlers/correction-detector.ts` | ~100 | 3 |
+| `src/store/skill-store.ts` | ~250 | 1 |
+| `src/tools/skill-tool.ts` | ~180 | 1 |
+| `src/handlers/skill-auto-trigger.ts` | ~80 | 1 |
+| `src/handlers/skills-command.ts` | ~50 | 1 |
+| `tests/handlers/auto-consolidate.test.ts` | ~120 | 2 |
+| `tests/handlers/correction-detector.test.ts` | ~150 | 3 |
+| `tests/store/skill-store.test.ts` | ~200 | 1 |
+| `tests/tools/skill-tool.test.ts` | ~100 | 1 |
+| `tests/handlers/skill-auto-trigger.test.ts` | ~80 | 1 |
+
+### Modified Files (8)
+| File | Epic(s) |
+|---|---|
+| `src/types.ts` | 2, 3, 4 |
+| `src/constants.ts` | 1, 2, 3, 4 |
+| `src/config.ts` | 2, 3, 4 |
+| `src/store/memory-store.ts` | 1, 2 |
+| `src/tools/memory-tool.ts` | 2 |
+| `src/handlers/background-review.ts` | 4 |
+| `src/index.ts` | 1, 2, 3, 4 |
+| `tests/handlers/background-review.test.ts` | 4 |
+
+---
+
+## Verification
+
+After each epic:
+1. `npm run check` — zero type errors
+2. `npm test` — all tests pass
+3. Manual test: `pi -e ./src/index.ts` — verify the feature works in a live session
+
+Final:
+4. Full regression: all 119 existing tests + new tests pass
+5. Tag v0.2.0

package/dist/extensions/pi-hermes-memory/docs/0.2/TASKS.md

@@ -0,0 +1,134 @@
+# Tasks — v0.2.0: Skills + Smart Curation
+
+> **Workflow**: When you start a task, change `[ ]` to `[~]`. When done, change to `[x]` and note the commit hash.
+>
+> **Implementation order**: Epic 2 → Epic 3 → Epic 4 → Epic 1 → Epic 5 (quick wins first, then the largest piece)
+>
+> **Plan**: See `docs/0.2/PLAN.md` for full implementation details and architectural decisions.
+
+---
+
+## Epic 2: Auto-Consolidation
+
+_Done when: memory full no longer returns an error — it triggers automatic consolidation and retries the add._
+
+### Shared Config (Epics 2-4 touch these files — do once, extend per epic)
+- [x] `src/types.ts` — add `autoConsolidate: boolean` to `MemoryConfig`; add `ConsolidationResult` interface (`c6317dd`)
+- [x] `src/config.ts` — add `autoConsolidate: true` default + parsing (`c6317dd`)
+- [x] `src/constants.ts` — add `CONSOLIDATION_PROMPT` (`c6317dd`)
+
+### Implementation
+- [x] `src/store/memory-store.ts` — make `add()` async, add `setConsolidator()` injection method; after consolidation: `await this.loadFromDisk()` before retry (`c6317dd`)
+- [x] `src/tools/memory-tool.ts` — `await store.add(target, content)` (`c6317dd`)
+- [x] `src/handlers/auto-consolidate.ts` — `triggerConsolidation()` using `pi.exec()` pattern (`c6317dd`)
+- [x] `src/handlers/consolidate-command.ts` — `/memory-consolidate` command (`c6317dd` — combined into `auto-consolidate.ts`)
+- [x] `src/index.ts` — wire consolidator via `store.setConsolidator()` + register command (`c6317dd`)
+
+### Tests
+- [x] `tests/handlers/auto-consolidate.test.ts` — consolidation trigger, pi.exec call, success/failure paths (`83e7c46`)
+- [x] `tests/store/memory-store.test.ts` — migrate all `store.add()` calls to `await store.add()`; consolidator tests (`83e7c46`)
+
+---
+
+## Epic 3: Correction Detection + Immediate Save
+
+_Done when: user corrections are detected in real-time and trigger an immediate memory save._
+
+### Config
+- [x] `src/types.ts` — add `correctionDetection: boolean` to `MemoryConfig` (`c6317dd`)
+- [x] `src/config.ts` — add `correctionDetection: true` default + parsing (`c6317dd`)
+- [x] `src/constants.ts` — add `CORRECTION_SAVE_PROMPT`, strong/weak/negative pattern arrays (`c6317dd`)
+
+### Implementation
+- [x] `src/handlers/correction-detector.ts` — two-pass filter: strong/weak/negative patterns (`c6317dd`)
+- [x] Rate limiting — `turnsSinceLastCorrection >= 3` and `!correctionInProgress` guard (`c6317dd`)
+- [x] `src/index.ts` — wire `setupCorrectionDetector()` (`c6317dd`)
+
+### Tests
+- [x] `tests/handlers/correction-detector.test.ts` — 35 tests: strong/weak/negative patterns, rate limiting, false positives (`83e7c46`)
+
+---
+
+## Epic 4: Tool-Call-Aware Nudge
+
+_Done when: background review triggers based on EITHER turn count OR tool call count._
+
+### Config
+- [x] `src/types.ts` — add `nudgeToolCalls: number` to `MemoryConfig` (`c6317dd`)
+- [x] `src/config.ts` — add `nudgeToolCalls: 15` default + parsing (`c6317dd`)
+
+### Implementation
+- [x] `src/handlers/background-review.ts` — count toolCall entries from branch; OR trigger logic; reset both counters (`c6317dd`)
+
+### Tests
+- [x] `tests/handlers/background-review.test.ts` — 6 new tests: tool-call trigger, combined trigger, counter reset, text-only, crash recovery (`83e7c46`)
+
+---
+
+## Epic 1: Skill Tool + Procedural Memory
+
+_Done when: the agent can create/update/delete skill documents, skills appear in a progressive index in the system prompt, and skills are auto-created after complex tasks._
+
+### Research & Design
+- [x] Read Pi's skill discovery API — Pi uses `~/.selesai/agent/skills/` with SKILL.md frontmatter format (`c6317dd`)
+- [x] Decide: write to `~/.selesai/agent/memory/skills/` — isolated from user skills (`c6317dd`)
+- [x] Read Hermes `skill_manage` tool source for reference patterns (`c6317dd`)
+
+### Store
+- [x] `src/store/skill-store.ts` — `SkillStore` class with full CRUD + `formatIndexForSystemPrompt()` (`c6317dd`)
+- [x] SKILL.md format — frontmatter (name, description, version, created, updated) + markdown body (`c6317dd`)
+- [x] File naming — `slugify(name) + ".md"` (`c6317dd`)
+- [x] Frontmatter parsing — regex-based, no yaml dependency (`c6317dd`)
+- [x] Content scanning — all writes go through `scanContent()` (`c6317dd`)
+- [x] Atomic writes — temp+rename pattern (`c6317dd`)
+
+### Tool
+- [x] `src/tools/skill-tool.ts` — `registerSkillTool()` with actions: `create`, `view`, `patch`, `edit`, `delete` (`c6317dd`)
+- [x] `src/constants.ts` — add `SKILL_TOOL_DESCRIPTION` and `DEFAULT_SKILL_TRIGGER_TOOL_CALLS` (= 8) (`c6317dd`)
+- [x] Rewrite `COMBINED_REVIEW_PROMPT` — references skill tool with create/patch actions (`c6317dd`)
+
+### Progressive Disclosure
+- [x] Skill index (name + description only) injected into system prompt at `before_agent_start` (`c6317dd`)
+- [x] `view` action loads full skill content on demand (`c6317dd`)
+- [x] Frozen snapshot — index captured at `session_start`, consistent throughout session (`c6317dd`)
+
+### Auto-Trigger
+- [x] `src/handlers/skill-auto-trigger.ts` — 8+ tool calls with 2+ distinct tool types (`c6317dd`)
+- [x] Rate limit — max 1 auto-trigger per session (`c6317dd`)
+
+### Command
+- [x] `src/handlers/skills-command.ts` — `/memory-skills` command (`c6317dd`)
+
+### Wiring
+- [x] `src/index.ts` — wire SkillStore, registerSkillTool, setupSkillAutoTrigger, registerSkillsCommand (`c6317dd`)
+
+### Tests
+- [x] `tests/store/skill-store.test.ts` — 27 tests: CRUD, frontmatter, progressive disclosure, atomic writes (`83e7c46`)
+- [x] `tests/tools/skill-tool.test.ts` — 10 tests: registration, action dispatch, validation (`83e7c46`)
+- [x] `tests/handlers/skill-auto-trigger.test.ts` — 6 tests: threshold, distinct types, session limit (`83e7c46`)
+
+---
+
+## Epic 5: Documentation & Release
+
+_Done when: v0.2.0 is tagged and released with updated docs._
+
+- [x] Update `README.md` — skill tool, auto-consolidation, correction detection, new config, new commands (`4658529`)
+- [x] Update `src/constants.ts` — verify all new prompts are finalized (`c6317dd`)
+- [x] Update `docs/ROADMAP.md` — v0.2 roadmap documented (`d5b7518`)
+- [x] `npm run check` passes with zero errors (`c6317dd`)
+- [x] `npm test` — all 218 tests pass (`83e7c46`)
+- [x] Bump `package.json` version to `0.2.0`
+- [x] Tag v0.2.0 release
+
+---
+
+## Summary
+
+| Epic | Priority | Est. Complexity | New Files | Modified Files |
+|---|---|---|---|---|
+| 2: Auto-Consolidation | HIGH | Low | 3 (src + test) | 5 (types, config, constants, memory-store, memory-tool, index) |
+| 3: Correction Detection | HIGH | Low | 2 (src + test) | 3 (types, config, constants, index) |
+| 4: Tool-Call Nudge | MEDIUM | Low | 0 | 3 (types, config, background-review, test) |
+| 1: Skill Tool | CRITICAL | High | 8 (4 src + 4 test) | 3 (constants, index, memory-store) |
+| 5: Documentation | NORMAL | Low | 0 | 4 (README, constants, ROADMAP, package.json) |