@lebronj/pi-suite 0.1.7 → 0.1.9

package/README.md

@@ -15,7 +15,7 @@ pi install npm:pi-subagents
 Or use the bootstrap script to install Pi, configure the team OpenAI-compatible endpoint, install this suite, and set up Bun + qmd for memory search:
 
 ```bash
-curl -fsSL https://registry.npmjs.org/@lebronj/pi-suite/-/pi-suite-0.1.7.tgz | tar -xzO package/scripts/bootstrap.sh | bash
+curl -fsSL https://registry.npmjs.org/@lebronj/pi-suite/-/pi-suite-0.1.8.tgz | tar -xzO package/scripts/bootstrap.sh | bash
 ```
 
 ## What Is Included
@@ -67,7 +67,7 @@ qmd collection add ~/.pi/agent/memory --name pi-memory
 qmd embed
 ```
 
-Memory versioning is enabled by default. It snapshots `~/.pi/agent/memory` and `~/.pi/agent/skill-drafts` into `~/.pi/agent/evolution`, commits local changes automatically, and leaves push manual by default.
+Memory versioning is enabled by default. It snapshots `~/.pi/agent/memory` and `~/.pi/agent/skill-drafts` into `~/.pi/agent/evolution`, commits local changes automatically, and leaves push manual by default. `memory_curate` also scans yesterday's daily log into `REVIEW.md` when learning is enabled and the daily file changed since the last scan.
 
 Useful commands:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lebronj/pi-suite",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "JHP's Pi extension suite for team coding workflows",
   "type": "module",
   "keywords": [

package/skills/pi-skill/SKILL.md

@@ -46,7 +46,7 @@ Memory tools:
 - `memory_edit`: read/add/replace/remove/replace_all/compact structured entries in `MEMORY.md`, `USER.md`, `STATE.md`, and `REVIEW.md`.
 - `scratchpad`: add/done/undo/clear/list checklist items.
 - `memory_search`: qmd-backed keyword, semantic, or deep search across memory files.
-- `memory_curate`: manually run curator lifecycle rules.
+- `memory_curate`: manually run curator lifecycle rules and scan yesterday's daily log into `REVIEW.md` when learning is enabled.
 - `memory_learning_approve`: approve a proposed memory promotion or disabled skill draft by exact id.
 - `memory_learning_reject`: reject or archive a review candidate/proposal without deleting it.
 - `memory_skill_drafts`: list proposed skill drafts.
@@ -74,6 +74,7 @@ Curator and learning behavior:
 - Quotas reset when `month` or `reset` rolls over.
 - Mutations are audited to `audit/curator.jsonl`.
 - Session shutdown may extract conservative learning candidates into `REVIEW.md`; they are not injected as normal memory and are not auto-enabled.
+- `memory_curate` scans yesterday's daily log once per content hash into review candidates, then curator lifecycle and proposal rules process those candidates.
 - Repeated candidates can become proposed memory promotions or proposed disabled skill drafts after `memory_curate`.
 - Approval is explicit by default: memory proposals write to memory stores; skill proposals write disabled drafts under `~/.pi/agent/skill-drafts/`.
 - The curator avoids semantic auto-delete/merge; ambiguous learning stays in review first.
@@ -84,6 +85,7 @@ Memory versioning:
 - Versioning mirror and snapshots live at `~/.pi/agent/evolution` by default.
 - No remote is configured by default; memory evolution stays local per user/machine unless the user adds a personal private remote.
 - Automatic local snapshot + commit is enabled by default; automatic push is disabled unless `PI_EVOLUTION_AUTO_PUSH=1`.
+- Snapshots use a sliding window: keep the latest 100 by default, and delete the oldest snapshot directory and manifest when a new snapshot exceeds the limit.
 - Snapshots run before mutating memory tools, curator runs, learning approve/reject, session summaries/handoffs, compact handoffs, restore, and external curator `run-once`.
 - Slash commands: `/memory-version-status`, `/memory-version-snapshot [reason]`, `/memory-version-list`, `/memory-version-restore <snapshot-id> [memory|skill-drafts|all]`, `/memory-version-push`.
 - Restore always writes a pre-restore snapshot first, then restores selected files and commits the restored state.
@@ -125,6 +127,7 @@ Useful memory environment variables:
 - `PI_EVOLUTION_BRANCH`: override branch; default `main`.
 - `PI_EVOLUTION_AUTO_COMMIT=0`: disable automatic local commits.
 - `PI_EVOLUTION_AUTO_PUSH=1`: push automatically after commits.
+- `PI_EVOLUTION_MAX_SNAPSHOTS`: maximum local snapshots to keep; default `100`.
 
 ## Web And Research
 

package/vendor/pi-memory/README.md

@@ -196,7 +196,7 @@ Pi-memory mirrors the authoritative runtime directories into a local evolution r
 
 Authoritative runtime data remains `~/.pi/agent/memory` and `~/.pi/agent/skill-drafts`; `~/.pi/agent/evolution` is a versioned mirror and backup repo.
 
-Automatic hooks snapshot before and sync/commit after `memory_write`, mutating `memory_edit`, mutating `scratchpad`, `memory_curate`, learning approve/reject, session summary/handoff writes, compaction handoffs, and external `jhp-pi-memory-curator run-once`. Read-only operations do not snapshot.
+Automatic hooks snapshot before and sync/commit after `memory_write`, mutating `memory_edit`, mutating `scratchpad`, `memory_curate`, learning approve/reject, session summary/handoff writes, compaction handoffs, and external `jhp-pi-memory-curator run-once`. `memory_curate` also scans yesterday's daily log into `REVIEW.md` when learning is enabled and that daily file changed since the last scan. Read-only operations do not snapshot. Snapshots use a sliding window and keep the latest 100 snapshots by default; creating snapshot 101 deletes the oldest snapshot directory and manifest.
 
 Tools and slash commands:
 
@@ -242,7 +242,7 @@ The controller uses a systemd user timer when available and falls back to cron.
 | `PI_MEMORY_QMD_UPDATE` | `background`, `manual`, `off` | `background` | Control qmd update after writes |
 | `PI_MEMORY_NO_SEARCH` | `1` | unset | Disable per-turn search injection |
 | `PI_MEMORY_SUMMARIZE_TRANSITIONS` | `1`, `true`, `yes`, `on` | unset | Also summarize lifecycle transitions |
-| `PI_MEMORY_LEARNING` | `off`, `review`, `auto-review` | `review` | Control session learning candidate extraction |
+| `PI_MEMORY_LEARNING` | `off`, `review`, `auto-review` | `review` | Control session and curator daily learning candidate extraction |
 | `PI_MEMORY_LEARNING_MIN_CONFIDENCE` | `low`, `medium`, `high` | `medium` | Minimum extractor confidence to keep |
 | `PI_MEMORY_SKILL_DRAFTS` | `off`, `review` | `review` | Allow curator to propose disabled skill drafts |
 | `PI_MEMORY_AUTO_APPROVE_MEMORY` | `1`, `true`, `yes`, `on` | unset | YOLO mode for approving newly created memory proposals |
@@ -253,6 +253,7 @@ The controller uses a systemd user timer when available and falls back to cron.
 | `PI_EVOLUTION_BRANCH` | branch | `main` | Local branch used for init/clone |
 | `PI_EVOLUTION_AUTO_COMMIT` | `0`, `1`, `true`, `false` | `1` | Commit sync/snapshot changes automatically |
 | `PI_EVOLUTION_AUTO_PUSH` | `0`, `1`, `true`, `false` | `0` | Push after commits automatically |
+| `PI_EVOLUTION_MAX_SNAPSHOTS` | positive integer | `100` | Maximum snapshots to keep in the local sliding window |
 
 ## Development
 

package/vendor/pi-memory/index.ts

@@ -26,6 +26,7 @@
  */
 
 import { type ExecFileOptions, execFile } from "node:child_process";
+import { createHash } from "node:crypto";
 import * as fs from "node:fs";
 import * as path from "node:path";
 import { complete, type Message, StringEnum } from "@earendil-works/pi-ai";
@@ -89,6 +90,7 @@ let USER_FILE = path.join(MEMORY_DIR, "USER.md");
 let STATE_FILE = path.join(MEMORY_DIR, "STATE.md");
 let REVIEW_FILE = path.join(MEMORY_DIR, "REVIEW.md");
 let SCRATCHPAD_FILE = path.join(MEMORY_DIR, "SCRATCHPAD.md");
+let LEARNING_STATE_FILE = path.join(MEMORY_DIR, ".learning-state.json");
 let DAILY_DIR = path.join(MEMORY_DIR, "daily");
 let SKILL_DRAFTS_DIR = path.join(path.dirname(MEMORY_DIR), "skill-drafts");
 
@@ -104,6 +106,7 @@ export function _setBaseDir(baseDir: string) {
 	STATE_FILE = path.join(baseDir, "STATE.md");
 	REVIEW_FILE = path.join(baseDir, "REVIEW.md");
 	SCRATCHPAD_FILE = path.join(baseDir, "SCRATCHPAD.md");
+	LEARNING_STATE_FILE = path.join(baseDir, ".learning-state.json");
 	DAILY_DIR = path.join(baseDir, "daily");
 	SKILL_DRAFTS_DIR = path.join(path.dirname(baseDir), "skill-drafts");
 }
@@ -562,9 +565,9 @@ function shouldKeepLearningCandidate(confidence: "low" | "medium" | "high", env:
 	return confidenceRank(confidence) >= confidenceRank(getMemoryLearningMinConfidence(env));
 }
 
-function buildLearningExtractorPrompt(conversationText: string, truncated: boolean, totalChars: number): string {
+function buildLearningExtractorPrompt(conversationText: string, truncated: boolean, totalChars: number, sourceLabel = "session transcript"): string {
 	const lines = [
-		"Extract zero or more review candidates from this session transcript.",
+		`Extract zero or more review candidates from this ${sourceLabel}.`,
 		"Return JSON exactly shaped as: {\"candidates\":[{\"kind\":\"bug_fix|skill_candidate|preference|project_fact\",\"confidence\":\"low|medium|high\",\"signature\":\"short stable signature\",\"summary\":\"optional concise summary\",\"targetHints\":[\"memory\",\"skill\"],\"evidence\":\"optional compact evidence\"}]}",
 		"Only include verified bug fixes when a failure was followed by an edit/action and successful validation.",
 		"Drop one-off trivia, transient status, workflow artifacts, and loop artifacts.",
@@ -671,31 +674,90 @@ export function parseLearningExtractorResponse(raw: string): ReviewCandidateInpu
 	return candidates;
 }
 
-async function runSessionLearningExtractor(ctx: ExtensionContext): Promise<number> {
-	if (getMemoryLearningMode() === "off") return 0;
-	const branch = getSessionBranch(ctx);
-	if (!branch || !ctx.model) return 0;
+type DailyLearningState = {
+	daily?: Record<string, { hash: string; scannedAt: string; candidates: number }>;
+};
+
+function readLearningState(): DailyLearningState {
+	try {
+		const parsed = JSON.parse(fs.readFileSync(LEARNING_STATE_FILE, "utf-8"));
+		return parsed && typeof parsed === "object" ? parsed as DailyLearningState : {};
+	} catch {
+		return {};
+	}
+}
+
+function writeLearningState(state: DailyLearningState): void {
+	fs.writeFileSync(LEARNING_STATE_FILE, `${JSON.stringify(state, null, 2)}\n`, "utf-8");
+}
+
+function contentHash(content: string): string {
+	return createHash("sha256").update(content).digest("hex");
+}
+
+async function extractLearningCandidates(
+	ctx: ExtensionContext,
+	text: string,
+	sourceLabel: string,
+	source: string,
+	date?: string,
+): Promise<ReviewCandidateInput[]> {
+	if (getMemoryLearningMode() === "off" || !ctx.model) return [];
 	const apiKey = await resolveExitSummaryApiKey(ctx);
-	if (!apiKey) return 0;
-	const conversation = serializeSessionConversation(branch);
-	if (!conversation.hasMessages || !conversation.text.trim()) return 0;
-	const truncated = truncateText(conversation.text.trim(), LEARNING_EXTRACTOR_MAX_CHARS, "end");
+	if (!apiKey) return [];
+	const trimmed = text.trim();
+	if (!trimmed) return [];
+	const truncated = truncateText(trimmed, LEARNING_EXTRACTOR_MAX_CHARS, "end");
 	const messages: Message[] = [{
 		role: "user",
-		content: [{ type: "text", text: buildLearningExtractorPrompt(truncated.text, truncated.truncated, conversation.text.trim().length) }],
+		content: [{ type: "text", text: buildLearningExtractorPrompt(truncated.text, truncated.truncated, trimmed.length, sourceLabel) }],
 		timestamp: Date.now(),
 	}];
+	const response = await complete(ctx.model, { systemPrompt: LEARNING_EXTRACTOR_SYSTEM_PROMPT, messages }, { apiKey, reasoningEffort: "low" });
+	const raw = response.content.filter((part): part is { type: "text"; text: string } => part.type === "text").map((part) => part.text).join("\n");
+	return parseLearningExtractorResponse(raw).map((candidate) => ({ ...candidate, source, date }));
+}
+
+async function writeLearningCandidates(candidates: ReviewCandidateInput[]): Promise<number> {
+	let written = 0;
+	const store = new FileMemoryStore(MEMORY_DIR);
+	for (const candidate of candidates) {
+		const result = await upsertReviewCandidate(store, candidate);
+		if (result.changed) written += 1;
+	}
+	return written;
+}
+
+type DailyLearningScanResult = { scanned: boolean; changed: number; skipped?: string };
+
+async function runYesterdayDailyLearningScan(ctx: ExtensionContext): Promise<DailyLearningScanResult> {
+	if (getMemoryLearningMode() === "off") return { scanned: false, changed: 0, skipped: "learning off" };
+	const date = yesterdayStr();
+	const dailyContent = readFileSafe(dailyPath(date));
+	if (!dailyContent?.trim()) return { scanned: false, changed: 0, skipped: `daily/${date}.md empty or missing` };
+	const hash = contentHash(dailyContent);
+	const state = readLearningState();
+	const previous = state.daily?.[date];
+	if (previous?.hash === hash) return { scanned: false, changed: 0, skipped: `daily/${date}.md already scanned` };
 	try {
-		const response = await complete(ctx.model, { systemPrompt: LEARNING_EXTRACTOR_SYSTEM_PROMPT, messages }, { apiKey, reasoningEffort: "low" });
-		const raw = response.content.filter((part): part is { type: "text"; text: string } => part.type === "text").map((part) => part.text).join("\n");
-		const candidates = parseLearningExtractorResponse(raw);
-		let written = 0;
-		const store = new FileMemoryStore(MEMORY_DIR);
-		for (const candidate of candidates) {
-			const result = await upsertReviewCandidate(store, candidate);
-			if (result.changed) written += 1;
-		}
-		return written;
+		const candidates = await extractLearningCandidates(ctx, dailyContent, `daily log for ${date}`, `daily/${date}`, date);
+		const changed = await writeLearningCandidates(candidates);
+		state.daily = { ...(state.daily || {}), [date]: { hash, scannedAt: new Date().toISOString(), candidates: candidates.length } };
+		writeLearningState(state);
+		return { scanned: true, changed };
+	} catch {
+		return { scanned: false, changed: 0, skipped: `daily/${date}.md scan failed` };
+	}
+}
+
+async function runSessionLearningExtractor(ctx: ExtensionContext): Promise<number> {
+	const branch = getSessionBranch(ctx);
+	if (!branch) return 0;
+	const conversation = serializeSessionConversation(branch);
+	if (!conversation.hasMessages || !conversation.text.trim()) return 0;
+	try {
+		const candidates = await extractLearningCandidates(ctx, conversation.text, "session transcript", "session_shutdown");
+		return writeLearningCandidates(candidates);
 	} catch {
 		return 0;
 	}
@@ -1439,7 +1501,7 @@ function getSnapshotMode(): "stable" | "per-turn" {
 }
 
 
-async function runCurator(reason: string): Promise<string> {
+async function runCurator(reason: string, ctx?: ExtensionContext): Promise<string> {
 	ensureDirs();
 	await evolutionBeforeChange(`curator before ${reason}`, "memory: snapshot before curate", "tool");
 	const store = new FileMemoryStore(MEMORY_DIR);
@@ -1448,6 +1510,7 @@ async function runCurator(reason: string): Promise<string> {
 		auditLog: new JsonlAuditLog(MEMORY_DIR),
 		reason,
 	});
+	const dailyLearningResult = ctx ? await runYesterdayDailyLearningScan(ctx) : { scanned: false, changed: 0 };
 	const lifecycleResult = await applyReviewLifecycle(store);
 	const memoryResult = await proposeMemoryPromotions(store);
 	const skillResult = getMemorySkillDraftsMode() === "off" ? { created: 0, proposals: [] } : await proposeSkillDrafts(store, { draftsDir: SKILL_DRAFTS_DIR });
@@ -1465,7 +1528,7 @@ async function runCurator(reason: string): Promise<string> {
 			autoApprovedSkills += 1;
 		}
 	}
-	const learningChanges = lifecycleResult.changed + memoryResult.created + skillResult.created + autoApprovedMemory + autoApprovedSkills;
+	const learningChanges = dailyLearningResult.changed + lifecycleResult.changed + memoryResult.created + skillResult.created + autoApprovedMemory + autoApprovedSkills;
 	if (result.patches.length > 0 || learningChanges > 0) {
 		snapshotDirty = true;
 		await ensureQmdAvailableForUpdate();
@@ -1473,6 +1536,7 @@ async function runCurator(reason: string): Promise<string> {
 	}
 	await evolutionAfterChange("memory: sync after curate");
 	const notes = [
+		dailyLearningResult.scanned ? `scanned yesterday daily, wrote ${dailyLearningResult.changed} review candidate change(s)` : dailyLearningResult.skipped ? `daily learning skipped: ${dailyLearningResult.skipped}` : "",
 		memoryResult.created > 0 ? `proposed ${memoryResult.created} memory promotion(s)` : "",
 		skillResult.created > 0 ? `proposed ${skillResult.created} skill draft(s)` : "",
 		autoApprovedMemory > 0 ? `auto-approved ${autoApprovedMemory} memory promotion(s)` : "",
@@ -1509,6 +1573,7 @@ function formatEvolutionStatusText(status: ReturnType<typeof getEvolutionGitStat
 		`dirty: ${status.dirty}`,
 		`autoCommit: ${status.autoCommit}`,
 		`autoPush: ${status.autoPush}`,
+		`snapshotLimit: ${status.maxSnapshots}`,
 		`lastCommit: ${status.lastCommit || "n/a"}`,
 		status.status ? `status:\n${status.status}` : "status: clean",
 	].join("\n");
@@ -2270,11 +2335,11 @@ export default function (pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "memory_curate",
 		label: "Memory Curate",
-		description: "Run the time-aware memory curator now. It deduplicates exact entries, updates event/quota lifecycle metadata, and appends stale temporary memories to REVIEW.md.",
+		description: "Run the time-aware memory curator now. It scans yesterday's daily log into REVIEW.md, deduplicates exact entries, updates event/quota lifecycle metadata, and appends stale temporary memories to REVIEW.md.",
 		parameters: Type.Object({}),
 		async execute(_toolCallId, _params, _signal, _onUpdate, _ctx) {
 			try {
-				const summary = await runCurator("memory_curate tool");
+				const summary = await runCurator("memory_curate tool", _ctx);
 				return { content: [{ type: "text", text: summary }], details: { summary } };
 			} catch (error) {
 				const message = error instanceof Error ? error.message : String(error);

package/vendor/pi-memory/src/evolution/config.ts

@@ -4,6 +4,7 @@ export interface EvolutionConfig {
 	enabled: boolean;
 	autoCommit: boolean;
 	autoPush: boolean;
+	maxSnapshots: number;
 	repoDir: string;
 	remote: string | null;
 	branch: string;
@@ -19,6 +20,7 @@ type EvolutionEnv = Partial<
 		| "PI_EVOLUTION_ENABLED"
 		| "PI_EVOLUTION_AUTO_COMMIT"
 		| "PI_EVOLUTION_AUTO_PUSH"
+		| "PI_EVOLUTION_MAX_SNAPSHOTS"
 		| "HOME"
 		| "USERPROFILE"
 		| "HOMEDRIVE"
@@ -30,6 +32,7 @@ type EvolutionEnv = Partial<
 export const DEFAULT_EVOLUTION_REMOTE = "";
 export const LEGACY_SHARED_EVOLUTION_REMOTE = "https://github.com/LRM-Teams/pi-evolution.git";
 export const DEFAULT_EVOLUTION_BRANCH = "main";
+export const DEFAULT_EVOLUTION_MAX_SNAPSHOTS = 100;
 
 function homeDir(env: EvolutionEnv): string {
 	return env.HOME ?? env.USERPROFILE ?? (env.HOMEDRIVE && env.HOMEPATH ? `${env.HOMEDRIVE}${env.HOMEPATH}` : undefined) ?? "~";
@@ -43,6 +46,12 @@ function truthy(value: string | undefined, fallback: boolean): boolean {
 	return fallback;
 }
 
+function positiveInteger(value: string | undefined, fallback: number): number {
+	if (value === undefined) return fallback;
+	const parsed = Number.parseInt(value.trim(), 10);
+	return Number.isFinite(parsed) && parsed > 0 ? parsed : fallback;
+}
+
 function expandHome(input: string, env: EvolutionEnv): string {
 	if (input === "~") return homeDir(env);
 	if (input.startsWith("~/")) return path.join(homeDir(env), input.slice(2));
@@ -55,6 +64,7 @@ export function resolveEvolutionConfig(memoryDir: string, env: EvolutionEnv = pr
 		enabled: truthy(env.PI_EVOLUTION_ENABLED, true),
 		autoCommit: truthy(env.PI_EVOLUTION_AUTO_COMMIT, true),
 		autoPush: truthy(env.PI_EVOLUTION_AUTO_PUSH, false),
+		maxSnapshots: positiveInteger(env.PI_EVOLUTION_MAX_SNAPSHOTS, DEFAULT_EVOLUTION_MAX_SNAPSHOTS),
 		repoDir: path.resolve(expandHome(env.PI_EVOLUTION_DIR || path.join(agentDir, "evolution"), env)),
 		remote: env.PI_EVOLUTION_REMOTE?.trim() || null,
 		branch: env.PI_EVOLUTION_BRANCH || DEFAULT_EVOLUTION_BRANCH,

package/vendor/pi-memory/src/evolution/git.ts

@@ -14,6 +14,7 @@ export interface GitStatus {
 	lastCommit: string | null;
 	autoPush: boolean;
 	autoCommit: boolean;
+	maxSnapshots: number;
 	enabled: boolean;
 }
 
@@ -104,6 +105,7 @@ export function getEvolutionGitStatus(config: EvolutionConfig): GitStatus {
 			lastCommit: null,
 			autoPush: config.autoPush,
 			autoCommit: config.autoCommit,
+			maxSnapshots: config.maxSnapshots,
 			enabled: config.enabled,
 		};
 	}
@@ -118,6 +120,7 @@ export function getEvolutionGitStatus(config: EvolutionConfig): GitStatus {
 		lastCommit: runGit(config.repoDir, ["log", "-1", "--oneline"], { allowFailure: true }) || null,
 		autoPush: config.autoPush,
 		autoCommit: config.autoCommit,
+		maxSnapshots: config.maxSnapshots,
 		enabled: config.enabled,
 	};
 }

package/vendor/pi-memory/src/evolution/index.ts

@@ -1,4 +1,4 @@
-export { DEFAULT_EVOLUTION_BRANCH, DEFAULT_EVOLUTION_REMOTE, LEGACY_SHARED_EVOLUTION_REMOTE, resolveEvolutionConfig, type EvolutionConfig } from "./config.ts";
+export { DEFAULT_EVOLUTION_BRANCH, DEFAULT_EVOLUTION_MAX_SNAPSHOTS, DEFAULT_EVOLUTION_REMOTE, LEGACY_SHARED_EVOLUTION_REMOTE, resolveEvolutionConfig, type EvolutionConfig } from "./config.ts";
 export { commitEvolutionChanges, ensureEvolutionRepo, getEvolutionGitStatus, pushEvolution, type GitCommitResult, type GitStatus } from "./git.ts";
 export { buildManifest, createSnapshotId, listManifests, readManifest, writeManifest, type EvolutionManifest } from "./manifest.ts";
 export { restoreEvolutionSnapshot, type RestoreResult, type RestoreTarget } from "./restore.ts";

package/vendor/pi-memory/src/evolution/manifest.ts

@@ -1,7 +1,7 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
 import type { EvolutionConfig } from "./config.ts";
-import { countFiles } from "./file-utils.ts";
+import { countFiles, pathExists } from "./file-utils.ts";
 
 export interface EvolutionManifest {
 	id: string;
@@ -57,12 +57,27 @@ export function readManifest(config: EvolutionConfig, id: string): EvolutionMani
 }
 
 export function listManifests(config: EvolutionConfig, limit = 20): EvolutionManifest[] {
+	return readAllManifests(config).slice(0, limit);
+}
+
+export function pruneOldSnapshots(config: EvolutionConfig): EvolutionManifest[] {
+	const manifests = readAllManifests(config);
+	if (manifests.length <= config.maxSnapshots) return [];
+
+	const removed = manifests.slice(config.maxSnapshots);
+	for (const manifest of removed) {
+		fs.rmSync(path.join(config.repoDir, "snapshots", manifest.id), { recursive: true, force: true });
+		fs.rmSync(path.join(config.repoDir, "manifests", `${manifest.id}.json`), { force: true });
+	}
+	return removed;
+}
+
+function readAllManifests(config: EvolutionConfig): EvolutionManifest[] {
 	const dir = path.join(config.repoDir, "manifests");
 	if (!fs.existsSync(dir)) return [];
 	return fs.readdirSync(dir)
 		.filter((file) => file.endsWith(".json"))
-		.sort()
-		.reverse()
-		.slice(0, limit)
-		.map((file) => JSON.parse(fs.readFileSync(path.join(dir, file), "utf-8")) as EvolutionManifest);
+		.map((file) => JSON.parse(fs.readFileSync(path.join(dir, file), "utf-8")) as EvolutionManifest)
+		.filter((manifest) => pathExists(path.join(config.repoDir, "snapshots", manifest.id)))
+		.sort((a, b) => b.createdAt.localeCompare(a.createdAt) || b.id.localeCompare(a.id));
 }

package/vendor/pi-memory/src/evolution/snapshot.ts

@@ -2,7 +2,7 @@ import * as path from "node:path";
 import type { EvolutionConfig } from "./config.ts";
 import { copyDirContents, emptyDir } from "./file-utils.ts";
 import { commitEvolutionChanges, ensureEvolutionRepo, type GitCommitResult } from "./git.ts";
-import { buildManifest, createSnapshotId, writeManifest, type EvolutionManifest } from "./manifest.ts";
+import { buildManifest, createSnapshotId, pruneOldSnapshots, writeManifest, type EvolutionManifest } from "./manifest.ts";
 import { syncCurrentToEvolution } from "./sync.ts";
 
 export interface SnapshotOptions {
@@ -29,6 +29,7 @@ export function createEvolutionSnapshot(config: EvolutionConfig, options: Snapsh
 	copyDirContents(config.skillDraftsDir, path.join(snapshotDir, "skill-drafts"));
 	const manifest = buildManifest(config, id, options.reason, options.trigger || "manual", options.sessionId);
 	writeManifest(config, manifest);
+	pruneOldSnapshots(config);
 	syncCurrentToEvolution(config);
 	const commit = commitEvolutionChanges(config, options.commitMessage || `memory: snapshot ${id}`);
 	return { manifest, commit };

package/vendor/pi-memory/test/evolution.test.ts

@@ -1,6 +1,6 @@
 import assert from "node:assert/strict";
 import { execFileSync } from "node:child_process";
-import { existsSync, mkdirSync, mkdtempSync, readFileSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, mkdtempSync, readFileSync, readdirSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { test } from "node:test";
@@ -51,6 +51,24 @@ test("creates snapshot, manifest, current mirrors, and git commit", () => {
 	assert.match(getEvolutionGitStatus(config).lastCommit || "", /memory: test snapshot/);
 });
 
+test("prunes old snapshots beyond the configured window", () => {
+	const { memoryDir, repoDir, config } = testConfig();
+	config.maxSnapshots = 3;
+	mkdirSync(memoryDir, { recursive: true });
+
+	const ids: string[] = [];
+	for (let i = 0; i < 4; i += 1) {
+		writeFileSync(join(memoryDir, "MEMORY.md"), `version ${i}\n`, "utf-8");
+		const result = createEvolutionSnapshot(config, { reason: `snapshot ${i}`, trigger: "test", commitMessage: `memory: snapshot ${i}` });
+		ids.push(result.manifest?.id || "");
+	}
+
+	assert.equal(existsSync(join(repoDir, "snapshots", ids[0])), false);
+	assert.equal(existsSync(join(repoDir, "manifests", `${ids[0]}.json`)), false);
+	assert.equal(readdirSync(join(repoDir, "snapshots")).length, 3);
+	assert.deepEqual(listManifests(config, 10).map((manifest) => manifest.id), ids.slice(1).reverse());
+});
+
 test("local-only config removes the legacy shared team remote", () => {
 	const { config, repoDir } = testConfig();
 	mkdirSync(repoDir, { recursive: true });