pi-posthorse 0.8.1 → 0.8.2

package/README.md

@@ -17,12 +17,12 @@ Required qualification targets: latest stable official Pi and latest maintained
 
 ## Install
 
-Posthorse `0.8.1` requires Pi `>=1.0.0`. Update the host before installing this release; do not activate it on an older live fork.
+Posthorse `0.8.2` requires Pi `>=1.0.0`. Update the host before installing this release; do not activate it on an older live fork.
 
 ```bash
-pi install npm:pi-posthorse@0.8.1                        # from npm
-pi install git:github.com/fitchmultz/pi-posthorse@v0.8.1 # from Git
-pi -e npm:pi-posthorse@0.8.1                             # try it for one run without installing
+pi install npm:pi-posthorse@0.8.2                        # from npm
+pi install git:github.com/fitchmultz/pi-posthorse@v0.8.2 # from Git
+pi -e npm:pi-posthorse@0.8.2                             # try it for one run without installing
 ```
 
 Omit the version or tag to follow the latest release.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-posthorse",
-	"version": "0.8.1",
+	"version": "0.8.2",
 	"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/history-entries.ts

@@ -213,7 +213,8 @@ export function toWindowedEntry(
 		images: imagesOf(entryContent(entry)),
 	};
 }
-export function windowProjection(): (entry: EntryLike) => WindowedEntry | undefined {
+/** Window ancestry is metadata-only; extracting text/images is a separate operation. */
+export function windowProjection(): (entry: EntryLike) => string {
 	const windows = new Map<string, string>();
 	return (entry) => {
 		const inherited = windows.get(entry.parentId ?? "") ?? "initial";
@@ -221,13 +222,15 @@ export function windowProjection(): (entry: EntryLike) => WindowedEntry | undefi
 		if (entry.id !== undefined && entry.id !== "") {
 			windows.set(entry.id, windowId);
 		}
-		return toWindowedEntry(entry, windowId);
+		return windowId;
 	};
 }
-export function* windowEntries(entries: readonly EntryLike[]): Generator<WindowedEntry> {
-	const project = windowProjection();
-	for (const entry of entries) {
-		const item = project(entry);
+export function* reverseWindowEntries(entries: readonly EntryLike[]): Generator<WindowedEntry> {
+	// Preserve parent-relative window labels without normalizing the whole branch.
+	// ponytail: ancestry still scans metadata; a native window-aware iterator could remove that cost.
+	const windows = entries.map(windowProjection());
+	for (let i = entries.length - 1; i >= 0; i--) {
+		const item = toWindowedEntry(entries[i], windows[i]);
 		if (item !== undefined) {
 			yield item;
 		}

package/src/history-search.ts

@@ -7,7 +7,7 @@ import {
 	type HistoryHit,
 	type ProjectedEntry,
 } from "./contracts.ts";
-import { historyHit, seenText, windowEntries } from "./history-entries.ts";
+import { historyHit, seenText, reverseWindowEntries } from "./history-entries.ts";
 import { shortKey } from "./message-text.ts";
 import {
 	filesContaining,
@@ -267,7 +267,7 @@ function branchMatches(
 	accumulator: SearchProgress,
 ): number {
 	let skipped = 0;
-	for (const item of [...windowEntries(source.branch())].reverse()) {
+	for (const item of reverseWindowEntries(source.branch())) {
 		const hit = historyHit(item, query);
 		if (hit === undefined) {
 			continue;

package/src/history.ts

@@ -3,7 +3,7 @@ import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-a
 import { Type } from "typebox";
 import { toolCards } from "../ui.ts";
 import { requireValue, type PolicyContext, type WindowedEntry } from "./contracts.ts";
-import { windowEntries } from "./history-entries.ts";
+import { toWindowedEntry, windowProjection } from "./history-entries.ts";
 import { historyReadPage, searchPage } from "./history-pages.ts";
 import { historyMatches, type HistorySource } from "./history-search.ts";
 import type { PageAccess } from "./paging.ts";
@@ -17,18 +17,26 @@ function entryReference(id: string): { readonly entryId: string; readonly fileKe
 		fileKey: separator < 0 ? undefined : id.slice(separator + 1),
 	};
 }
+function branchEntry(source: HistorySource, id: string): WindowedEntry | undefined {
+	const project = windowProjection();
+	for (const entry of source.branch()) {
+		const windowId = project(entry);
+		const item = entry.id === id ? toWindowedEntry(entry, windowId) : undefined;
+		if (item !== undefined) {
+			return item;
+		}
+	}
+	return undefined;
+}
 async function findEntry(
 	source: HistorySource,
 	id: string,
 	signal?: AbortSignal,
 ): Promise<LocatedEntry> {
 	const { entryId, fileKey } = entryReference(id);
-	if (fileKey === undefined) {
-		for (const item of windowEntries(source.branch())) {
-			if (item.entry.id === entryId) {
-				return { item, source: "" };
-			}
-		}
+	const current = fileKey === undefined ? branchEntry(source, entryId) : undefined;
+	if (current !== undefined) {
+		return { item: current, source: "" };
 	}
 	const candidates =
 		fileKey === undefined && /^[\w-]+$/.test(entryId)
@@ -42,10 +50,8 @@ async function findEntry(
 		fileKey,
 		candidates: candidates === undefined ? undefined : (candidate) => candidates.has(candidate),
 	})) {
-		for await (const item of sessionWindowEntries(file, signal)) {
-			if (item.entry.id === entryId) {
-				return { item, source: relative(source.dir, file) };
-			}
+		for await (const item of sessionWindowEntries(file, signal, entryId)) {
+			return { item, source: relative(source.dir, file) };
 		}
 	}
 	throw new Error(

package/src/session-files.ts

@@ -11,7 +11,7 @@ import {
 	type WindowedEntry,
 } from "./contracts.ts";
 import { lfLines } from "./files.ts";
-import { windowProjection } from "./history-entries.ts";
+import { toWindowedEntry, windowProjection } from "./history-entries.ts";
 import { shortKey } from "./message-text.ts";
 
 function execFileText(
@@ -141,6 +141,7 @@ function parsedEntry(line: string): EntryLike | undefined {
 export async function* sessionWindowEntries(
 	file: string,
 	signal?: AbortSignal,
+	entryId?: string,
 ): AsyncGenerator<WindowedEntry> {
 	if (!existsSync(file)) {
 		return;
@@ -151,7 +152,11 @@ export async function* sessionWindowEntries(
 		if (entry === undefined) {
 			continue;
 		}
-		const item = project(entry);
+		const windowId = project(entry);
+		if (entryId !== undefined && entry.id !== entryId) {
+			continue;
+		}
+		const item = toWindowedEntry(entry, windowId);
 		if (item !== undefined) {
 			yield item;
 		}