@lebronj/pi-suite 0.1.12 → 0.1.13

package/README.md

@@ -12,7 +12,7 @@ pi install npm:@lebronj/pi-suite
 Or use the bootstrap script to install Pi, configure the team OpenAI-compatible endpoint, install this suite, and set up Bun + qmd for memory search:
 
 ```bash
-curl -fsSL https://registry.npmjs.org/@lebronj/pi-suite/-/pi-suite-0.1.12.tgz | tar -xzO package/scripts/bootstrap.sh | bash
+curl -fsSL https://registry.npmjs.org/@lebronj/pi-suite/-/pi-suite-0.1.13.tgz | tar -xzO package/scripts/bootstrap.sh | bash
 ```
 
 ## What Is Included
@@ -98,6 +98,8 @@ qmd embed
 
 Memory versioning is enabled by default. It snapshots `~/.pi/agent/memory` and `~/.pi/agent/skill-drafts` into `~/.pi/agent/evolution`, commits local changes automatically, and leaves push manual by default. `memory_curate` also scans yesterday's daily log into `REVIEW.md` when learning is enabled and the daily file changed since the last scan.
 
+The external memory curator service uses a systemd user timer when available, with cron fallback. When the service points at a vendored TypeScript CLI under `node_modules`, the launcher uses Bun or tsx instead of plain Node so Node 22 can run it reliably.
+
 Useful commands:
 
 ```bash

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lebronj/pi-suite",
-  "version": "0.1.12",
+  "version": "0.1.13",
   "description": "JHP's Pi extension suite for team coding workflows",
   "type": "module",
   "keywords": [

package/skills/pi-skill/SKILL.md

@@ -95,6 +95,7 @@ External curator service:
 
 - This is the main self-evolution maintenance loop: it can run daily outside the pi process, even when pi is closed.
 - It uses a systemd user timer when available, with cron fallback.
+- For vendored TypeScript CLI paths under `node_modules`, the service launcher uses Bun or tsx instead of plain Node so Node 22 does not fail on TypeScript type stripping.
 - On `session_start` and after `/reload`, pi-memory checks service status. If the service is disabled and UI is available, it shows a startup hint with enable/status/disable commands.
 - Enable with `/memory-curator-enable 03:00` or ask the agent to call `memory_curator_enable`.
 - Inspect with `/memory-curator-status` or `memory_curator_status`.

package/vendor/pi-memory/src/service-controller.ts

@@ -1,5 +1,5 @@
 import { execFileSync, spawnSync } from "node:child_process";
-import { existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from "node:fs";
+import { accessSync, constants, existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, join } from "node:path";
 
@@ -79,8 +79,30 @@ function writeState(state: CuratorServiceState): void {
 	writeFileSync(statePath(state.memoryDir), `${JSON.stringify(state, null, 2)}\n`, "utf-8");
 }
 
+function commandPath(command: string): string | undefined {
+	if (command.includes("/")) {
+		try {
+			accessSync(command, constants.X_OK);
+			return command;
+		} catch {
+			return undefined;
+		}
+	}
+	for (const dir of (process.env.PATH || "").split(":")) {
+		if (!dir) continue;
+		const candidate = join(dir, command);
+		try {
+			accessSync(candidate, constants.X_OK);
+			return candidate;
+		} catch {
+			// keep searching PATH
+		}
+	}
+	return undefined;
+}
+
 function hasCommand(command: string): boolean {
-	return spawnSync(command, ["--version"], { stdio: "ignore" }).status === 0;
+	return commandPath(command) !== undefined || spawnSync(command, ["--version"], { stdio: "ignore" }).status === 0;
 }
 
 function canUseSystemdUser(): boolean {
@@ -93,6 +115,29 @@ function shellQuote(value: string): string {
 	return `'${value.replace(/'/g, `'"'"'`)}'`;
 }
 
+function systemdQuote(value: string): string {
+	if (/^[A-Za-z0-9_@%+=:,./-]+$/.test(value)) return value;
+	return `"${value.replace(/\\/g, "\\\\").replace(/"/g, '\\"').replace(/\$/g, "\\$")}"`;
+}
+
+type CliInvocation = {
+	command: string;
+	args: string[];
+};
+
+function resolveCliInvocation(cliPath: string): CliInvocation {
+	const normalized = cliPath.replace(/\\/g, "/");
+	const isNodeModulesTypeScript = normalized.endsWith(".ts") && normalized.includes("/node_modules/");
+	if (!isNodeModulesTypeScript) return { command: process.execPath, args: [cliPath] };
+
+	const bunPath = commandPath("bun");
+	if (bunPath) return { command: bunPath, args: [cliPath] };
+	const tsxPath = commandPath("tsx");
+	if (tsxPath) return { command: tsxPath, args: [cliPath] };
+
+	throw new Error(`Cannot install memory curator service: Node cannot run TypeScript files under node_modules (${cliPath}). Install Bun or tsx, or publish a compiled JavaScript curator CLI.`);
+}
+
 function parseSchedule(schedule: string): { hour: string; minute: string } {
 	const match = /^(\d{1,2}):(\d{2})$/.exec(schedule);
 	if (!match) throw new Error(`Invalid schedule '${schedule}'. Expected HH:MM.`);
@@ -109,7 +154,8 @@ function systemdCalendar(schedule: string): string {
 
 function writeSystemdUnits(memoryDir: string, cliPath: string, schedule: string): void {
 	mkdirSync(systemdUserDir(), { recursive: true });
-	const execStart = `${process.execPath} ${cliPath} run-once --memory-dir ${memoryDir} --reason systemd-timer`;
+	const invocation = resolveCliInvocation(cliPath);
+	const execStart = [invocation.command, ...invocation.args, "run-once", "--memory-dir", memoryDir, "--reason", "systemd-timer"].map(systemdQuote).join(" ");
 	writeFileSync(
 		servicePath(),
 		[
@@ -187,9 +233,10 @@ function enableCron(memoryDir: string, cliPath: string, schedule: string): void
 	if (!hasCommand("crontab")) throw new Error("Neither systemd user timers nor crontab are available.");
 	const { hour, minute } = parseSchedule(schedule);
 	removeCronLine();
-	const command = `${shellQuote(process.execPath)} ${shellQuote(cliPath)} run-once --memory-dir ${shellQuote(memoryDir)} --reason cron ${CRON_MARKER}`;
+	const invocation = resolveCliInvocation(cliPath);
+	const command = [invocation.command, ...invocation.args, "run-once", "--memory-dir", memoryDir, "--reason", "cron"].map(shellQuote).join(" ");
 	const existing = currentCrontab().trim();
-	const next = `${existing ? `${existing}\n` : ""}${minute} ${hour} * * * ${command}\n`;
+	const next = `${existing ? `${existing}\n` : ""}${minute} ${hour} * * * ${command} ${CRON_MARKER}\n`;
 	installCrontab(next);
 }
 
@@ -202,7 +249,7 @@ export function enableCuratorService(options: { memoryDir?: string; cliPath: str
 			enableSystemd(memoryDir, options.cliPath, schedule);
 			const state: CuratorServiceState = { ...baseState, enabled: true, backend: "systemd-user", installedAt: new Date().toISOString() };
 			writeState(state);
-			return { ok: true, backend: "systemd-user", message: "Enabled systemd user timer for daily 03:00 memory curation.", state };
+			return { ok: true, backend: "systemd-user", message: `Enabled systemd user timer for daily ${schedule} memory curation.`, state };
 		}
 		enableCron(memoryDir, options.cliPath, schedule);
 		const state: CuratorServiceState = { ...baseState, enabled: true, backend: "cron", installedAt: new Date().toISOString() };
@@ -241,8 +288,10 @@ export function getCuratorServiceStatus(options: { memoryDir?: string; cliPath:
 	];
 	if (state.lastError) parts.push(`Last error: ${state.lastError}`);
 	if (state.backend === "systemd-user" && hasCommand("systemctl")) {
-		const active = spawnSync("systemctl", ["--user", "is-active", `${SERVICE_NAME}.timer`], { encoding: "utf-8" });
-		parts.push(`systemd timer active: ${active.stdout.trim() || "unknown"}`);
+		const timerActive = spawnSync("systemctl", ["--user", "is-active", `${SERVICE_NAME}.timer`], { encoding: "utf-8" });
+		const serviceActive = spawnSync("systemctl", ["--user", "is-active", `${SERVICE_NAME}.service`], { encoding: "utf-8" });
+		parts.push(`systemd timer active: ${timerActive.stdout.trim() || "unknown"}`);
+		parts.push(`systemd service active: ${serviceActive.stdout.trim() || "unknown"}`);
 	}
 	return { ok: true, backend: state.backend, message: parts.join("\n"), state };
 }