@lebronj/pi-suite 0.1.8 → 0.1.9

package/package.json

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

package/skills/pi-skill/SKILL.md

@@ -85,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.
@@ -126,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`. `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.
+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:
 
@@ -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

@@ -1573,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");

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 });