@gavin-hu/my-pi-agent 0.13.1 → 0.14.0

package/lib/config.ts

@@ -246,27 +246,51 @@ function setPath(source: Record<string, unknown>, path: readonly string[], value
 }
 
 /**
- * Delete one key from one top-level section, preserving its siblings and the
- * rest of the document. An emptied section is removed rather than left as `{}`.
- * A no-op when the file, section, or key is absent; refuses a malformed
- * document.
+ * Delete the leaf at `keyPath` from one top-level section, preserving every
+ * sibling key, every other section, and document metadata. An object left empty
+ * by the deletion goes with it, the section included, so a file does not fill
+ * with `{}` shells. A no-op when the file, section, or leaf is absent; refuses a
+ * malformed document and an empty path.
  */
-export function unsetConfigValue(path: string, section: string, key: string): void {
+export function unsetConfigValueAt(path: string, section: string, keyPath: readonly string[]): void {
+	if (keyPath.length === 0) throw new Error(`Cannot update ${path}: empty key path`);
 	const result = readJsonResult(path);
 	if (result.state === "absent") return;
 	if (result.state === "error") throw new Error(`Cannot update ${path}: ${result.error}`);
 
 	const document = result.value;
 	const current = configSection(document, section);
-	if (!current || !(key in current)) return;
+	if (!current) return;
 
-	const next = { ...current };
-	delete next[key];
-	if (Object.keys(next).length === 0) delete document[section];
+	const next = withoutPath(current, keyPath);
+	if (next === current) return;
+	if (next === undefined) delete document[section];
 	else document[section] = next;
 	writeConfigFile(path, document);
 }
 
+/**
+ * Copy `source` without the leaf at `path`, or `undefined` once nothing is left.
+ * Returns `source` itself when the leaf is absent or sits behind a non-object,
+ * so the caller can tell "nothing to delete" from "a deletion happened".
+ */
+function withoutPath(source: Record<string, unknown>, path: readonly string[]): Record<string, unknown> | undefined {
+	const [head, ...rest] = path;
+	if (head === undefined || !(head in source)) return source;
+	const next = { ...source };
+	if (rest.length === 0) {
+		delete next[head];
+	} else {
+		const child = asRecord(source[head]);
+		if (!child) return source;
+		const pruned = withoutPath(child, rest);
+		if (pruned === child) return source;
+		if (pruned === undefined) delete next[head];
+		else next[head] = pruned;
+	}
+	return Object.keys(next).length === 0 ? undefined : next;
+}
+
 /** Write a whole config document atomically (sibling temp file, then rename). */
 export function writeConfigFile(path: string, document: Record<string, unknown>): void {
 	mkdirSync(dirname(path), { recursive: true });

package/lib/format.ts

@@ -49,6 +49,12 @@ export function formatDuration(ms: number): string {
 	return `${hours}h${String(minutes % 60).padStart(2, "0")}m`;
 }
 
+/** `1 tool`, `3 tools`: a tool count with the right word. */
+export function formatToolCount(count: number): string {
+	const n = Number.isFinite(count) ? Math.max(0, Math.trunc(count)) : 0;
+	return `${formatCount(n)} ${n === 1 ? "tool" : "tools"}`;
+}
+
 /**
  * Replace terminal control characters (including ESC and the C1 block) with
  * spaces, keeping `\n` and `\t` for callers that render multi-line text.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gavin-hu/my-pi-agent",
-  "version": "0.13.1",
+  "version": "0.14.0",
   "description": "Personal Pi customizations, packaged as a Pi package: worktree isolation and rewind snapshots, question, todo, goal, plan, subagent, job, watch, web-access, file-browse, file-open, document, memory, wechat, and bytefly.",
   "keywords": [
     "pi-package",
@@ -105,6 +105,7 @@
     "config:migrate": "bun run scripts/migrate-config.ts",
     "compat:versions": "bun run scripts/peer-versions.ts",
     "bench:render": "bun run scripts/render-bench.ts",
+    "screenshot:file-browse": "bun run scripts/file-browse-shot.ts",
     "screenshot:plan": "bun run scripts/plan-review-shot.ts",
     "screenshot:tools": "bun run scripts/tool-renderings.ts gallery assets/tool-renderings.png",
     "screenshot:tool-pngs": "bun run scripts/tool-renderings.ts tools assets/tools",

package/themes/bytefly-dark.json

@@ -17,7 +17,6 @@
     "purple": "#bc8cff",
     "pink": "#f778ba",
     "selectionBg": "#17273a",
-    "customBg": "#26233a",
     "successBg": "#12271a",
     "errorBg": "#3d1a21",
     "searchBg": "#3d331a",
@@ -43,9 +42,9 @@
     "searchMatchText": "text",
     "userMessageBg": "",
     "userMessageText": "text",
-    "customMessageBg": "customBg",
+    "customMessageBg": "",
     "customMessageText": "muted",
-    "customMessageLabel": "purple",
+    "customMessageLabel": "blue",
     "toolPendingBg": "canvasSubtle",
     "toolSuccessBg": "successBg",
     "toolErrorBg": "errorBg",

package/themes/bytefly-light.json

@@ -16,7 +16,6 @@
     "yellow": "#9a6700",
     "purple": "#8250df",
     "pink": "#bf3989",
-    "customBg": "#f5f1fd",
     "successBg": "#e9f6ed",
     "errorBg": "#fcf0f1",
     "selectionBg": "#e8f2fc",
@@ -43,9 +42,9 @@
     "searchMatchText": "text",
     "userMessageBg": "",
     "userMessageText": "text",
-    "customMessageBg": "customBg",
+    "customMessageBg": "",
     "customMessageText": "muted",
-    "customMessageLabel": "purple",
+    "customMessageLabel": "blue",
     "toolPendingBg": "canvasSubtle",
     "toolSuccessBg": "successBg",
     "toolErrorBg": "errorBg",