pi-posthorse 0.9.2 → 0.9.3

package/README.md

@@ -73,7 +73,7 @@ Removal leaves notes and Pi session history in place. See the [installation refe
 
 ## Data and privacy
 
-Posthorse makes no network requests. Notes are plaintext files under `.pi/notes/`; linked Git worktrees share them. Add `.pi/notes/` to `.gitignore` if the notes must stay private.
+Posthorse makes no network requests. Notes are plaintext files under `.pi/notes/`; linked Git worktrees share them. Notes for the Git packages Pi installs in `~/.pi/agent/git/` live under `~/.pi/agent/notes/` instead, because `pi update` deletes untracked files in those checkouts. Add `.pi/notes/` to `.gitignore` if the notes must stay private.
 
 History can include user messages, assistant thinking, tool arguments, errors, and images. Content you retrieve enters the context of your selected model and provider.
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-posthorse",
-	"version": "0.9.2",
+	"version": "0.9.3",
 	"type": "module",
 	"description": "Posthorse: fresh context, same journey. No-summary context rollover for Pi, with sparse budget reminders, handoffs, notes, and history.",
 	"keywords": [

package/src/files.ts

@@ -1,5 +1,15 @@
 import { randomUUID } from "node:crypto";
-import { createReadStream, existsSync, readFileSync, realpathSync, statSync } from "node:fs";
+import {
+	createReadStream,
+	existsSync,
+	lstatSync,
+	mkdirSync,
+	readFileSync,
+	realpathSync,
+	renameSync,
+	statSync,
+	symlinkSync,
+} from "node:fs";
 import {
 	access,
 	constants,
@@ -12,7 +22,8 @@ import {
 	stat,
 	unlink,
 } from "node:fs/promises";
-import { basename, dirname, isAbsolute, join, resolve, sep } from "node:path";
+import { basename, dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
+import { getAgentDir } from "@earendil-works/pi-coding-agent";
 import { isRecord } from "./contracts.ts";
 
 async function lstatOrMissing(path: string) {
@@ -171,7 +182,7 @@ function gitRoot(dir: string, marker: string): string {
 	return basename(common) === ".git" ? dirname(common) : common;
 }
 /** Conventional repos share the primary checkout; separate Git directories own their shared root. */
-export function notesRoot(cwd: string): string {
+function notesRoot(cwd: string): string {
 	for (let dir = cwd; ; dir = dirname(dir)) {
 		const marker = join(dir, ".git");
 		if (existsSync(marker)) {
@@ -186,3 +197,46 @@ export function notesRoot(cwd: string): string {
 		}
 	}
 }
+/**
+ * `pi update` resets the Git packages Pi installs in the agent directory and runs `git clean -fdx`
+ * in them, deleting notes kept inside. Their notes live in the agent directory under the same host
+ * and path instead.
+ * ponytail: packages installed for one project (its `.pi/git/`) still keep notes in the checkout;
+ * give them a per-project location if sessions start working inside them.
+ * ponytail: notes a package commits under `.pi/notes` return with each update and move into
+ * another `from-checkout-*` folder; skip tracked files if a package ever commits its notes.
+ */
+function packageNotes(root: string): string | undefined {
+	const agentDir = getAgentDir();
+	const installs = join(agentDir, "git");
+	const path = relative(existsSync(installs) ? realpathSync(installs) : installs, root);
+	const inside =
+		path !== "" && path !== ".." && !path.startsWith(`..${sep}`) && !isAbsolute(path);
+	// Pi cleans only the checkouts; folders that hold several of them keep their own notes.
+	return inside && existsSync(join(root, ".git")) ? join(agentDir, "notes", path) : undefined;
+}
+/** The notes directory for `cwd`, moving any notes found inside a Pi package checkout. */
+export function notesDir(cwd: string): string {
+	const root = notesRoot(cwd);
+	const local = join(root, ".pi", "notes");
+	const moved = packageNotes(root);
+	if (moved === undefined) {
+		return local;
+	}
+	if (lstatSync(local, { throwIfNoEntry: false })?.isDirectory() === true) {
+		mkdirSync(dirname(moved), { recursive: true });
+		// Notes an older Posthorse wrote here after an earlier move keep a folder of their own.
+		const target = existsSync(moved) ? join(moved, `from-checkout-${Date.now()}`) : moved;
+		try {
+			renameSync(local, target);
+			// Sessions still running an older Posthorse keep reaching the notes until they reload.
+			symlinkSync(moved, local, "junction");
+		} catch (error) {
+			if (lstatSync(local, { throwIfNoEntry: false })?.isDirectory() === true) {
+				throw error;
+			}
+			// Another session moved them first, or only the compatibility link failed.
+		}
+	}
+	return moved;
+}

package/src/notes.ts

@@ -23,7 +23,7 @@ import {
 	type PolicyContext,
 	type TextResult,
 } from "./contracts.ts";
-import { isBinaryFile, notesRoot, publishFile, readText } from "./files.ts";
+import { isBinaryFile, notesDir, publishFile, readText } from "./files.ts";
 import { excerpt, excerptAround } from "./message-text.ts";
 import type { PageAccess } from "./paging.ts";
 
@@ -45,7 +45,7 @@ const SEARCH_OFFSET_LIMIT = 1_000_000;
 function safeJoin(dir: string, path: string): string {
 	const relative = normalize(path.replace(/^[/\\]+/, ""));
 	if (relative === ".." || relative.startsWith(`..${sep}`) || isAbsolute(relative)) {
-		throw new Error(`Invalid path "${path}": must stay inside .pi/notes/.`);
+		throw new Error(`Invalid path "${path}": must stay inside the notes directory.`);
 	}
 	return join(dir, relative);
 }
@@ -278,7 +278,7 @@ export function registerNotes(
 				{ description: "Operation to perform" },
 			),
 			path: Type.Optional(
-				Type.String({ description: "Note or folder path relative to .pi/notes/" }),
+				Type.String({ description: "Note or folder path relative to the notes directory" }),
 			),
 			content: Type.Optional(
 				Type.String({ description: "Full file content (write) or text to add (append)" }),
@@ -296,7 +296,7 @@ export function registerNotes(
 		// Pi's fixed tool contract supplies id, params, signal, updates, and context.
 		// oxlint-disable-next-line max-params
 		async execute(_id, params, signal, _onUpdate, ctx) {
-			const dir = join(notesRoot(ctx.cwd), ".pi", "notes");
+			const dir = notesDir(ctx.cwd);
 			switch (params.op) {
 				case "list":
 					return listNotes(dir, params, policy(ctx), pages);