@xynogen/pix-todo 0.1.2 → 0.1.4

package/README.md

@@ -14,6 +14,8 @@ pi install npm:@xynogen/pix-todo
 
 ## Full distro
 
+Source: [github.com/xynogen/pix-mono](https://github.com/xynogen/pix-mono)
+
 To install the complete pix suite (all packages + Pi itself):
 
 ```bash

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-todo",
-	"version": "0.1.2",
+	"version": "0.1.4",
 	"description": "Pi tool — durable execution checklist (todo)",
 	"type": "module",
 	"main": "src/index.ts",

package/src/once.ts

@@ -1,15 +1,25 @@
 /**
- * Idempotency guard for extension activation.
+ * Per-instance idempotency guard for extension activation.
  *
- * pix-core (the meta-package) invokes this package's factory in addition to a
- * possible direct install. Pi's loader uses jiti with `moduleCache: false`, so
- * each load pass re-evaluates modules — a plain module-level flag would not be
- * shared. The dedupe key therefore lives on `globalThis`, which persists for
- * the lifetime of the process across all load passes.
+ * pix-core (the meta-package) invokes this package's factory, and a standalone
+ * install makes Pi invoke it again — sometimes against the SAME `pi`. We must
+ * dedupe that. But Pi rebuilds the extension runtime on /new, /resume, /fork,
+ * and /reload, handing the factory a BRAND-NEW `pi`; that must re-register.
+ *
+ * Keying the registry on the `pi` instance satisfies both: same instance =>
+ * skip, new instance => run. The registry lives on globalThis because jiti
+ * (`moduleCache: false`) re-evaluates this module on every load pass, so a
+ * module-scoped WeakMap would not be shared between the aggregator pass and the
+ * standalone pass within a single session.
  */
-export function once(key: string, fn: () => void): void {
-	const g = globalThis as { __pixLoaded?: Set<string> };
-	const loaded = (g.__pixLoaded ??= new Set<string>());
+export function once(pi: object, key: string, fn: () => void): void {
+	const g = globalThis as { __pixOnce?: WeakMap<object, Set<string>> };
+	const registry = (g.__pixOnce ??= new WeakMap<object, Set<string>>());
+	let loaded = registry.get(pi);
+	if (!loaded) {
+		loaded = new Set<string>();
+		registry.set(pi, loaded);
+	}
 	if (loaded.has(key)) return;
 	loaded.add(key);
 	fn();

package/src/todo.test.ts

@@ -1,12 +1,12 @@
 import { beforeEach, describe, expect, test } from "bun:test";
 import registerTodo, { renderTodoLines, type TodoItem } from "./todo.ts";
 
-// registerTodo wraps its body in once("pix-todo") — a process-wide globalThis
-// guard that dedupes activation across pix-core + a standalone install. Tests
-// re-register a fresh host per case, so clear the registry first to let each
-// registerTodo run (mirrors pix-core/once.test.ts clearing __pixLoaded).
+// registerTodo wraps its body in once(pi, "pix-todo") — a per-instance
+// WeakMap guard that dedupes activation across pix-core + a standalone install.
+// Tests re-register a fresh host per case. Clear the registry between tests so
+// that the same pi object can be re-used without cross-test interference.
 beforeEach(() => {
-	delete (globalThis as { __pixLoaded?: Set<string> }).__pixLoaded;
+	delete (globalThis as { __pixOnce?: WeakMap<object, Set<string>> }).__pixOnce;
 });
 
 // Stub theme tags each fragment with its color/bold so assertions can verify

package/src/todo.ts

@@ -69,7 +69,7 @@ const parseItems = (raw: string): string[] =>
 		.filter(Boolean);
 
 export default function registerTodo(pi: ExtensionAPI): void {
-	once("pix-todo", () => {
+	once(pi, "pix-todo", () => {
 		let todos: TodoItem[] = [];
 		let nextTodoId = 1;