@youngjurry/pi-agents 0.8.0 → 0.8.1

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.8.1 - 2026-09-10
+
+- Remove automatic legacy-file archival and its full main-session scan; the extension no longer archives or deletes user session data.
+- Retain only the lightweight `codex-agents` → `pi-agents` compatibility migration required for users upgrading directly from 0.7.x; it is a no-op for fresh installations.
+
 ## 0.8.0 - 2026-09-09
 
 - Add `/agent-usage`, a read-only overlay that reports separate main/sub-agent token and cache totals plus combined tokens and cost without modifying root context.

package/README.md

@@ -189,8 +189,8 @@ The settings file is optional, but spawning requires a model from either the tas
 - Each root storage group records its owning main-session file in `owner.json`
 - The former `~/.pi/agent/codex-agents/` directory and `agents-setting.json` filename migrate automatically without overwriting newer files
 - Resuming an existing main session removes groups whose owning main-session file has been deleted; new sessions and `/reload` do not trigger grouped cleanup
-- Legacy flat files are archived only after every ordinary Pi main session has been scanned and no persisted agent-state reference exists
 - Referenced legacy flat child files are migrated when their main session is resumed
+- The extension never automatically archives or deletes legacy flat files
 - Parents receive a compact completion notice instead of the full answer; use `list_agents(view="results")` or read the result file on demand
 - Notices to a busy agent are queued safely: `wait_agent` returns them in its own result, and any leftovers are delivered right after a successful recipient turn
 - `wait_agent` sends only newly queued mailbox notices to the model; its child status tree excludes the active caller, is folded in the TUI by default, and can be toggled with `Ctrl+O`

package/index.ts

@@ -5,7 +5,7 @@ import { Text, truncateToWidth } from "@earendil-works/pi-tui";
 import { AgentControl } from "./control.ts";
 import { getAgentSettingsPath, loadAgentSettings, resolveAgentLimits } from "./settings.ts";
 import { createCollaborationTools } from "./tools.ts";
-import { archiveUnownedLegacyFiles, migrateLegacyAgentStorage } from "./storage.ts";
+import { migrateLegacyAgentStorage } from "./storage.ts";
 import { EXTENSION_ID, ROOT_PATH, type AgentLifecycleStatus, type AgentView } from "./types.ts";
 import { AgentPickerComponent, AgentTranscriptViewer, AgentUsageViewer, formatAgentUsage } from "./viewer.ts";
 
@@ -101,7 +101,6 @@ export default function codexAgentsExtension(pi: ExtensionAPI): void {
 	let activeContext: ExtensionContext | undefined;
 	let widgetTui: { requestRender(): void } | undefined;
 	let storageMigrationReported = false;
-	let legacyArchiveStarted = false;
 
 	const updateUi = () => {
 		const ctx = activeContext;
@@ -135,18 +134,6 @@ export default function codexAgentsExtension(pi: ExtensionAPI): void {
 			}
 			for (const warning of storageMigration.warnings) ctx.ui.notify(`Agent storage migration: ${warning}`, "warning");
 		}
-		if (!legacyArchiveStarted) {
-			legacyArchiveStarted = true;
-			void archiveUnownedLegacyFiles().then((report) => {
-				const current = activeContext;
-				if (!current) return;
-				if (report.error) {
-					current.ui.notify(`Legacy agent archive skipped: ${report.error}`, "warning");
-				} else if (report.archivedFiles > 0) {
-					current.ui.notify(`Archived ${report.archivedFiles} unowned legacy agent files to ${report.archiveDirectory}.`, "info");
-				}
-			});
-		}
 		const resumedExistingSession = event.reason === "resume"
 			|| (event.reason === "startup" && ctx.sessionManager.getEntries().some((entry) => entry.type === "message"));
 		if (resumedExistingSession) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@youngjurry/pi-agents",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "Persistent in-process Codex-style multi-agent collaboration for Pi",
   "author": "youngjurry",
   "type": "module",

package/storage.ts

@@ -1,8 +1,6 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
-import { createInterface } from "node:readline";
 import { getAgentDir } from "@earendil-works/pi-coding-agent";
-import { STATE_ENTRY_TYPE } from "./types.ts";
 
 const STORAGE_DIRECTORY_NAME = "pi-agents";
 const LEGACY_STORAGE_DIRECTORY_NAME = "codex-agents";
@@ -14,14 +12,6 @@ export interface StorageMigrationReport {
 	warnings: string[];
 }
 
-export interface LegacyArchiveReport {
-	archivedFiles: number;
-	retainedFiles: number;
-	scannedMainSessions: number;
-	archiveDirectory?: string;
-	error?: string;
-}
-
 export function getAgentStorageDirectory(): string {
 	return path.join(getAgentDir(), STORAGE_DIRECTORY_NAME);
 }
@@ -65,7 +55,10 @@ function mergeWithoutOverwrite(source: string, destination: string, report: Stor
 	}
 }
 
-/** Move the old package-owned directory without overwriting newer data. */
+/**
+ * Preserve the 0.7.x upgrade path. For a fresh installation this is only an
+ * existence check and performs no writes.
+ */
 export function migrateLegacyAgentStorage(): StorageMigrationReport {
 	const report: StorageMigrationReport = { movedEntries: 0, warnings: [] };
 	const source = getLegacyAgentStorageDirectory();
@@ -87,161 +80,8 @@ export function resolveMigratedStoragePath(file: string): string {
 	const legacyRoot = path.resolve(getLegacyAgentStorageDirectory());
 	if (source !== legacyRoot && !source.startsWith(`${legacyRoot}${path.sep}`)) return source;
 	// A collision-safe merge leaves the legacy source in place. Prefer the exact
-	// persisted path whenever it still exists rather than shadowing it with a
-	// different destination file that happens to share its relative name.
+	// persisted path instead of shadowing it with a different destination file.
 	if (fs.existsSync(source)) return source;
 	const translated = path.join(getAgentStorageDirectory(), path.relative(legacyRoot, source));
 	return fs.existsSync(translated) ? translated : source;
 }
-
-function listMainSessionFiles(): string[] {
-	const sessionsRoot = path.join(getAgentDir(), "sessions");
-	if (!fs.existsSync(sessionsRoot)) return [];
-	const files: string[] = [];
-	for (const project of fs.readdirSync(sessionsRoot, { withFileTypes: true })) {
-		if (!project.isDirectory() && !project.isSymbolicLink()) continue;
-		const projectDirectory = path.join(sessionsRoot, project.name);
-		try {
-			for (const entry of fs.readdirSync(projectDirectory, { withFileTypes: true })) {
-				if (entry.isFile() && entry.name.endsWith(".jsonl")) files.push(path.join(projectDirectory, entry.name));
-			}
-		} catch {
-			// A failed directory scan makes archival unsafe.
-			throw new Error(`could not scan main session directory: ${projectDirectory}`);
-		}
-	}
-	return files;
-}
-
-function sessionFileIds(file: string): string[] {
-	const ids = new Set<string>();
-	const basename = path.basename(file);
-	const filenameMatch = basename.match(/_([0-9a-f-]{16,})\.jsonl$/i);
-	if (filenameMatch?.[1]) ids.add(filenameMatch[1]);
-	let descriptor: number | undefined;
-	try {
-		descriptor = fs.openSync(file, "r");
-		const buffer = Buffer.allocUnsafe(4096);
-		const bytesRead = fs.readSync(descriptor, buffer, 0, buffer.length, 0);
-		const firstLine = buffer.subarray(0, bytesRead).toString("utf8").split("\n", 1)[0];
-		if (firstLine) {
-			const header = JSON.parse(firstLine) as { type?: unknown; id?: unknown };
-			if (header.type === "session" && typeof header.id === "string") ids.add(header.id);
-		}
-	} catch {
-		// The filename ID remains usable for conservative reference detection.
-	} finally {
-		if (descriptor !== undefined) fs.closeSync(descriptor);
-	}
-	return [...ids];
-}
-
-function legacyFlatFiles(): Array<{ file: string; ids: string[] }> {
-	const roots = [getAgentStorageDirectory(), getLegacyAgentStorageDirectory()];
-	const files: Array<{ file: string; ids: string[] }> = [];
-	for (const root of roots) {
-		for (const kind of ["sessions", "results"] as const) {
-			const directory = path.join(root, kind);
-			if (!fs.existsSync(directory)) continue;
-			for (const entry of fs.readdirSync(directory, { withFileTypes: true })) {
-				if (!entry.isFile()) continue;
-				if (kind === "sessions" && !entry.name.endsWith(".jsonl")) continue;
-				if (kind === "results" && !entry.name.endsWith(".md")) continue;
-				const file = path.join(directory, entry.name);
-				const ids = kind === "sessions"
-					? sessionFileIds(file)
-					: [path.basename(file, path.extname(file))];
-				files.push({ file, ids });
-			}
-		}
-	}
-	return files;
-}
-
-function uniqueArchiveTarget(directory: string, basename: string): string {
-	let target = path.join(directory, basename);
-	let suffix = 2;
-	while (fs.existsSync(target)) {
-		const extension = path.extname(basename);
-		const stem = path.basename(basename, extension);
-		target = path.join(directory, `${stem}-${suffix}${extension}`);
-		suffix++;
-	}
-	return target;
-}
-
-/**
- * Archive legacy flat files only after scanning every ordinary Pi session for a
- * persisted agent-state reference. Any scan failure leaves all candidates in place.
- */
-export async function archiveUnownedLegacyFiles(): Promise<LegacyArchiveReport> {
-	let candidates: Array<{ file: string; ids: string[] }>;
-	let mainSessions: string[];
-	try {
-		candidates = legacyFlatFiles();
-		if (candidates.length === 0) return { archivedFiles: 0, retainedFiles: 0, scannedMainSessions: 0 };
-		mainSessions = listMainSessionFiles();
-	} catch (error) {
-		return {
-			archivedFiles: 0,
-			retainedFiles: 0,
-			scannedMainSessions: 0,
-			error: error instanceof Error ? error.message : String(error),
-		};
-	}
-
-	const referencedIds = new Set<string>();
-	const candidateIds = new Set(candidates.flatMap((candidate) => candidate.ids));
-	let scannedMainSessions = 0;
-	try {
-		for (const sessionFile of mainSessions) {
-			const lines = createInterface({ input: fs.createReadStream(sessionFile, { encoding: "utf8" }), crlfDelay: Infinity });
-			for await (const line of lines) {
-				if (!line.includes(STATE_ENTRY_TYPE)) continue;
-				for (const id of candidateIds) {
-					if (line.includes(id)) referencedIds.add(id);
-				}
-			}
-			scannedMainSessions++;
-		}
-	} catch (error) {
-		return {
-			archivedFiles: 0,
-			retainedFiles: candidates.length,
-			scannedMainSessions,
-			error: error instanceof Error ? error.message : String(error),
-		};
-	}
-
-	// An unidentifiable file cannot be proven unowned and must remain untouched.
-	const unowned = candidates.filter((candidate) => candidate.ids.length > 0 && !candidate.ids.some((id) => referencedIds.has(id)));
-	if (unowned.length === 0) {
-		return { archivedFiles: 0, retainedFiles: candidates.length, scannedMainSessions };
-	}
-	const stamp = new Date().toISOString().replace(/[:.]/g, "-");
-	const archiveDirectory = path.join(getAgentStorageDirectory(), "archive", "legacy-unowned", stamp);
-	let archivedFiles = 0;
-	try {
-		for (const candidate of unowned) {
-			const kind = candidate.file.endsWith(".jsonl") ? "sessions" : "results";
-			const destinationDirectory = path.join(archiveDirectory, kind);
-			fs.mkdirSync(destinationDirectory, { recursive: true });
-			fs.renameSync(candidate.file, uniqueArchiveTarget(destinationDirectory, path.basename(candidate.file)));
-			archivedFiles++;
-		}
-	} catch (error) {
-		return {
-			archivedFiles,
-			retainedFiles: candidates.length - archivedFiles,
-			scannedMainSessions,
-			archiveDirectory,
-			error: error instanceof Error ? error.message : String(error),
-		};
-	}
-	return {
-		archivedFiles,
-		retainedFiles: candidates.length - archivedFiles,
-		scannedMainSessions,
-		archiveDirectory,
-	};
-}