@askjo/pi-reflect 1.1.0 → 1.2.0

package/.github/FUNDING.yml

@@ -0,0 +1 @@
+github: skyfallsin

package/.github/workflows/publish.yml

@@ -0,0 +1,46 @@
+name: Publish to npm
+
+on:
+  push:
+    tags:
+      - "v*"
+
+concurrency:
+  group: npm-publish
+  cancel-in-progress: false
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 24
+          cache: npm
+      - run: npm ci
+      - run: npm test
+
+  publish:
+    needs: test
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      id-token: write
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 24
+          registry-url: "https://registry.npmjs.org"
+      - name: Verify tag matches package.json version
+        run: |
+          TAG_VERSION="${GITHUB_REF#refs/tags/v}"
+          PKG_VERSION=$(node -p "require('./package.json').version")
+          if [ "$TAG_VERSION" != "$PKG_VERSION" ]; then
+            echo "::error::Tag version ($TAG_VERSION) != package.json version ($PKG_VERSION)"
+            exit 1
+          fi
+      - run: npm ci --ignore-scripts
+      - name: Publish with provenance
+        run: npm publish --provenance --access public

package/README.md

@@ -1,3 +1,7 @@
+<p align="center">
+  <a href="LICENSE"><img src="https://img.shields.io/badge/License-MIT-yellow.svg" alt="License: MIT" /></a>
+  <a href="https://github.com/jo-inc/pi-reflect/stargazers"><img src="https://img.shields.io/github/stars/jo-inc/pi-reflect" alt="GitHub stars" /></a>
+</p>
 <p align="center">
   <picture>
     <source media="(prefers-color-scheme: dark)" srcset="logo-dark.png" width="300">
@@ -165,3 +169,13 @@ pi -e ./extensions/index.ts   # test locally without installing
 ## License
 
 MIT
+
+## Pi Ecosystem
+
+| Package | Description |
+|---------|-------------|
+| [pi-mem](https://github.com/jo-inc/pi-mem) | Persistent markdown memory for coding agents |
+| [pi-boss](https://github.com/skyfallsin/pi-boss) | Multi-agent orchestration via tmux |
+| [pi-room](https://github.com/skyfallsin/pi-room) | Multi-agent awareness and coordination |
+| [pi-vertex-anthropic](https://github.com/skyfallsin/pi-vertex-anthropic) | Claude via Google Cloud Vertex AI |
+| [pi-skill-posthog](https://github.com/skyfallsin/pi-skill-posthog) | PostHog analytics skill for pi agents |

package/extensions/index.ts

@@ -14,6 +14,13 @@
 
 import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
 import * as path from "node:path";
+import * as fs from "node:fs";
+
+/** Display a target path as "parentDir/filename" for compact but unambiguous listing */
+function targetLabel(filePath: string): string {
+	const dir = path.basename(path.dirname(filePath));
+	return `${dir}/${path.basename(filePath)}`;
+}
 
 import {
 	type ReflectTarget,
@@ -28,6 +35,7 @@ import {
 	runReflection,
 	collectTranscriptsForDate,
 	getAvailableSessionDates,
+	computeFileMetrics,
 	CONFIG_FILE,
 } from "./reflect.js";
 
@@ -44,14 +52,14 @@ export default function (pi: ExtensionAPI) {
 			modelRegistryRef = ctx.modelRegistry;
 			const targetPath = args?.trim();
 
-			let target: ReflectTarget;
+			let targets: ReflectTarget[];
 
 			if (targetPath) {
 				const config = loadConfig();
 				const existing = config.targets.find(
 					(t) => resolvePath(t.path) === resolvePath(targetPath),
 				);
-				target = existing ?? { ...DEFAULT_TARGET, path: targetPath };
+				targets = [existing ?? { ...DEFAULT_TARGET, path: targetPath }];
 			} else {
 				const config = loadConfig();
 				if (config.targets.length === 0) {
@@ -60,14 +68,14 @@ export default function (pi: ExtensionAPI) {
 							"No targets configured. Enter path to a markdown file to reflect on:",
 						);
 						if (!filePath) return;
-						target = { ...DEFAULT_TARGET, path: filePath };
+						targets = [{ ...DEFAULT_TARGET, path: filePath }];
 
 						const save = await ctx.ui.confirm(
 							"Save target?",
 							`Save ${filePath} as a reflection target for next time?`,
 						);
 						if (save) {
-							config.targets.push(target);
+							config.targets.push(targets[0]);
 							saveConfig(config);
 							ctx.ui.notify("Saved to reflect.json", "info");
 						}
@@ -76,18 +84,23 @@ export default function (pi: ExtensionAPI) {
 						return;
 					}
 				} else if (config.targets.length === 1) {
-					target = config.targets[0];
+					targets = [config.targets[0]];
 				} else if (ctx.hasUI) {
+					const allTargetsLabel = "All targets";
 					const choice = await ctx.ui.select(
 						"Which target?",
-						config.targets.map((t) => path.basename(t.path)),
+						[...config.targets.map((t) => targetLabel(t.path)), allTargetsLabel],
 					);
 					if (choice === undefined || choice === null) return;
-					const chosenTarget = config.targets.find((t) => path.basename(t.path) === choice);
-					if (!chosenTarget) return;
-					target = chosenTarget;
+					if (choice === allTargetsLabel) {
+						targets = config.targets;
+					} else {
+						const chosenTarget = config.targets.find((t) => targetLabel(t.path) === choice);
+						if (!chosenTarget) return;
+						targets = [chosenTarget];
+					}
 				} else {
-					target = config.targets[0];
+					targets = [config.targets[0]];
 				}
 			}
 
@@ -95,26 +108,28 @@ export default function (pi: ExtensionAPI) {
 				? (msg, level) => ctx.ui.notify(msg, level)
 				: (msg, level) => console.log(`[reflect] [${level}] ${msg}`);
 
-			// Use the current session model if available
-			let currentModel: any;
-			let currentModelApiKey: string | undefined;
-			if (ctx.model) {
-				const key = await ctx.modelRegistry.getApiKey(ctx.model);
-				if (key) {
-					currentModel = ctx.model;
-					currentModelApiKey = key;
+			// Run configured targets sequentially to avoid provider concurrency spikes,
+			// and persist each successful result immediately.
+			let completed = 0;
+			let failed = 0;
+			for (const target of targets) {
+				try {
+					const run = await runReflection(target, modelRegistryRef, notify, undefined, {});
+					if (run) {
+						const history = loadHistory();
+						history.push(run);
+						saveHistory(history);
+						completed++;
+					} else {
+						failed++;
+					}
+				} catch (error) {
+					failed++;
+					notify(`${targetLabel(target.path)} failed: ${error instanceof Error ? error.message : String(error)}`, "error");
 				}
 			}
-
-			const run = await runReflection(target, modelRegistryRef, notify, undefined, {
-				currentModel,
-				currentModelApiKey,
-			});
-
-			if (run) {
-				const history = loadHistory();
-				history.push(run);
-				saveHistory(history);
+			if (targets.length > 1) {
+				notify(`All targets finished: ${completed} completed, ${failed} failed or skipped`, failed ? "warning" : "info");
 			}
 		},
 	});
@@ -135,7 +150,7 @@ export default function (pi: ExtensionAPI) {
 			}
 
 			const lines = config.targets.map((t, i) => {
-				return `${i + 1}. **${path.basename(t.path)}** — ${t.schedule}, ${t.model}, ${t.lookbackDays}d lookback\n   ${t.path}`;
+				return `${i + 1}. **${targetLabel(t.path)}** — ${t.schedule}, ${t.model}, ${t.lookbackDays}d lookback\n   ${t.path}`;
 			});
 
 			ctx.ui.notify(
@@ -160,7 +175,7 @@ export default function (pi: ExtensionAPI) {
 			const recent = history.slice(-10).reverse();
 			const lines = recent.map((r) => {
 				const date = r.timestamp.slice(0, 16).replace("T", " ");
-				const file = path.basename(r.targetPath);
+				const file = targetLabel(r.targetPath);
 				return `- **${date}** ${file}: ${r.editsApplied} edits, ${r.correctionsFound} corrections (${r.sessionsAnalyzed} sessions)\n  ${r.summary}`;
 			});
 
@@ -197,16 +212,68 @@ export default function (pi: ExtensionAPI) {
 				byTarget.set(key, list);
 			}
 
+			// If multiple targets tracked, let user pick one (or show all)
+			if (byTarget.size > 1 && ctx.hasUI) {
+				const options = ["All targets", ...Array.from(byTarget.keys()).map((p) => targetLabel(p))];
+				const choice = await ctx.ui.select("Show stats for which target?", options);
+				if (choice === undefined || choice === null) return;
+				if (choice !== "All targets") {
+					// Filter to just the chosen target
+					const chosenPath = Array.from(byTarget.keys()).find((p) => targetLabel(p) === choice);
+					if (chosenPath) {
+						const runs = byTarget.get(chosenPath)!;
+						byTarget.clear();
+						byTarget.set(chosenPath, runs);
+					}
+				}
+			}
+
 			const output: string[] = [];
 			let targetIdx = 0;
 
 			for (const [targetPath, runs] of byTarget) {
-				const fileName = path.basename(targetPath);
+				const fileName = targetLabel(targetPath);
 				if (targetIdx > 0) output.push("", "---", "");
 				output.push(`# ${fileName}`);
 				output.push(`_${targetPath}_`);
 				output.push("");
 
+				// --- Current File Size ---
+				const resolvedPath = resolvePath(targetPath);
+				if (fs.existsSync(resolvedPath)) {
+					const current = computeFileMetrics(fs.readFileSync(resolvedPath, "utf-8"));
+					output.push(`### Current Size`);
+					output.push(`${current.chars.toLocaleString()} chars · ${current.words.toLocaleString()} words · ${current.lines.toLocaleString()} lines · ~${current.estTokens.toLocaleString()} tokens`);
+					output.push("");
+				}
+
+				// --- File Size Trend ---
+				const runsWithSize = runs.filter((r) => r.fileSize).sort((a, b) => getSourceDate(a).localeCompare(getSourceDate(b)));
+				if (runsWithSize.length >= 2) {
+					output.push("### File Size Trend");
+					output.push("");
+					for (const r of runsWithSize) {
+						const sz = r.fileSize!;
+						const date = getSourceDate(r);
+						const bar = "\u2588".repeat(Math.round(sz.estTokens / 1000));
+						output.push(`${date}  ${sz.chars.toLocaleString().padStart(7)} chars  ${sz.words.toLocaleString().padStart(6)} words  ~${sz.estTokens.toLocaleString().padStart(6)} tok  ${bar}`);
+					}
+
+					const first = runsWithSize[0].fileSize!;
+					const last = runsWithSize[runsWithSize.length - 1].fileSize!;
+					const charDelta = last.chars - first.chars;
+					const pct = first.chars > 0 ? ((charDelta / first.chars) * 100).toFixed(0) : "N/A";
+					output.push("");
+					if (charDelta > 0) {
+						output.push(`Trend: \u2191 grew ${charDelta.toLocaleString()} chars (+${pct}%) over ${runsWithSize.length} runs`);
+					} else if (charDelta < 0) {
+						output.push(`Trend: \u2193 shrank ${Math.abs(charDelta).toLocaleString()} chars (${pct}%) over ${runsWithSize.length} runs`);
+					} else {
+						output.push(`Trend: \u2194 unchanged`);
+					}
+					output.push("");
+				}
+
 				// --- Correction Rate Trend ---
 				output.push("### Correction Rate (corrections per session)");
 				output.push("");
@@ -391,7 +458,7 @@ export default function (pi: ExtensionAPI) {
 			planLines.push("**Backfill plan (dry run — no file edits):**");
 			planLines.push("");
 			for (const p of plan) {
-				const fileName = path.basename(p.target.path);
+				const fileName = targetLabel(p.target.path);
 				planLines.push(`- **${fileName}**: ${p.dates.length} date(s) [${p.dates[0]} \u2192 ${p.dates[p.dates.length - 1]}]`);
 			}
 			planLines.push("");
@@ -415,7 +482,7 @@ export default function (pi: ExtensionAPI) {
 			const updatedHistory = loadHistory();
 
 			for (const p of plan) {
-				const fileName = path.basename(p.target.path);
+				const fileName = targetLabel(p.target.path);
 
 				for (const date of p.dates) {
 					ctx.ui.notify(`[${completed + failed + 1}/${totalCalls}] ${fileName} — ${date}...`, "info");