@hicaru/pi-rlm 0.3.3 → 0.3.4

package/package.json

@@ -1,57 +1,58 @@
 {
-  "name": "@hicaru/pi-rlm",
-  "version": "0.3.3",
-  "author": "hicaru",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/openzebra/rlm.pi.git"
-  },
-  "devDependencies": {
-    "typescript": "^5.0.0"
-  },
-  "peerDependencies": {
-    "@earendil-works/pi-ai": "*",
-    "@earendil-works/pi-coding-agent": "*",
-    "@earendil-works/pi-tui": "*",
-    "typebox": "*"
-  },
-  "bugs": {
-    "url": "https://github.com/openzebra/rlm.pi/issues"
-  },
-  "description": "Save 99% tokens, Recursive Language Model (RLM) for the Pi",
-  "files": [
-    "src/",
-    "README.md",
-    "LICENSE"
-  ],
-  "homepage": "https://github.com/openzebra/rlm.pi",
-  "keywords": [
-    "pi-package",
-    "pi-extension",
-    "rlm",
-    "recursive",
-    "ai-agent"
-  ],
-  "license": "MIT",
-  "pi": {
-    "extensions": [
-      "./src/index.ts"
-    ],
-    "image": "https://raw.githubusercontent.com/openzebra/rlm.pi/main/assets/plugin-cover.png"
-  },
-  "publishConfig": {
-    "access": "public"
-  },
-  "scripts": {
-    "check": "tsc --noEmit",
-    "test": "bun run test/smoke.ts",
-    "prepublishOnly": "npm run check"
-  },
-  "type": "module",
-  "engines": {
-    "node": ">=20"
-  },
-  "dependencies": {
-    "@firecrawl/anydoc": "^0.1.7"
-  }
+	"name": "@hicaru/pi-rlm",
+	"version": "0.3.4",
+	"author": "hicaru",
+	"repository": {
+		"type": "git",
+		"url": "git+https://github.com/openzebra/rlm.pi.git"
+	},
+	"devDependencies": {
+		"typescript": "^5.0.0"
+	},
+	"peerDependencies": {
+		"@earendil-works/pi-ai": "^0.84.1",
+		"@earendil-works/pi-coding-agent": "^0.84.1",
+		"@earendil-works/pi-tui": "^0.84.1",
+		"typebox": "*"
+	},
+	"bugs": {
+		"url": "https://github.com/openzebra/rlm.pi/issues"
+	},
+	"description": "Save 99% tokens, Recursive Language Model (RLM) for the Pi",
+	"files": [
+		"src/",
+		"README.md",
+		"LICENSE"
+	],
+	"homepage": "https://github.com/openzebra/rlm.pi",
+	"keywords": [
+		"pi-package",
+		"pi-extension",
+		"rlm",
+		"recursive",
+		"ai-agent"
+	],
+	"license": "MIT",
+	"pi": {
+		"extensions": [
+			"./src/index.ts"
+		],
+		"image": "https://raw.githubusercontent.com/openzebra/rlm.pi/main/assets/plugin-cover.png"
+	},
+	"publishConfig": {
+		"access": "public"
+	},
+	"scripts": {
+		"check": "tsc --noEmit",
+		"test": "bun run test/smoke.ts",
+		"prepublishOnly": "npm run check"
+	},
+	"type": "module",
+	"engines": {
+		"node": ">=20"
+	},
+	"dependencies": {
+		"@earendil-works/pi-agent-core": "^0.84.1",
+		"@firecrawl/anydoc": "^0.1.7"
+	}
 }

package/src/bridge/model.ts

@@ -6,7 +6,7 @@
  * Used both for `llm_query` (one user prompt) and for the headless RLM root (full history).
  */
 
-import { type Api, completeSimple, type Message, type Model, type ThinkingLevel, type Usage } from "@earendil-works/pi-ai";
+import { type Api, completeSimple, type Message, type Model, type ThinkingLevel, type Usage } from "@earendil-works/pi-ai/compat";
 import type { ModelRegistry } from "@earendil-works/pi-coding-agent";
 
 export type Role = "system" | "user" | "assistant";

package/src/config/settings.ts

@@ -34,11 +34,11 @@ function validateString(v: unknown): string | undefined {
 
 /**
  * Every value pi-ai accepts for `reasoning`. Keyed by the union so a new level added upstream
- * is a compile error here rather than a silently-rejected setting. Note `off` and `max` are
- * NOT ThinkingLevels — a hand-edited rlm.json carrying one is dropped, not forwarded.
+ * is a compile error here rather than a silently-rejected setting. Note `off` is NOT a
+ * ThinkingLevel — a hand-edited rlm.json carrying one is dropped, not forwarded.
  */
 const THINKING_LEVELS: Readonly<Record<ThinkingLevel, true>> = Object.freeze({
-  minimal: true, low: true, medium: true, high: true, xhigh: true,
+  minimal: true, low: true, medium: true, high: true, xhigh: true, max: true,
 });
 
 function validateThinkingLevel(v: unknown): ThinkingLevel | undefined {

package/src/sandbox/sandbox-manager.ts

@@ -106,10 +106,23 @@ export class SandboxManager {
       awaitTimeoutS: this.config.awaitTimeoutS,
       handlers,
     }).then(async (s) => {
+      // ②A: a dispose racing an in-flight spawn must not leak the worker — kill it the
+      // instant it resolves. dispose() awaits initPromise so this always runs before it returns.
+      if (this.disposed) {
+        await s.dispose().catch(() => {});
+        throw new Error("SandboxManager disposed during spawn");
+      }
       // Load context on first creation if available (empty list is a valid starting value).
-      if (this.contextPayload !== undefined) {
-        await s.loadContext(this.contextPayload);
-        this.contextLoaded = true;
+      try {
+        if (this.contextPayload !== undefined) {
+          await s.loadContext(this.contextPayload);
+          this.contextLoaded = true;
+        }
+      } catch (err) {
+        // ②B: never leak a spawned worker whose context load failed.
+        await s.dispose().catch(() => {});
+        this.initPromise = null;
+        throw err;
       }
       this.sandbox = s;
       this.initPromise = null;
@@ -196,6 +209,11 @@ export class SandboxManager {
   async dispose(): Promise<void> {
     if (this.disposed) return;
     this.disposed = true;
+    // ②A: a spawn in flight finishes after this.disposed was set — await it so the guard
+    // inside getOrCreate disposes its worker before dispose() returns (no leaked process).
+    if (this.initPromise) {
+      try { await this.initPromise; } catch { /* spawn failed or was disposed */ }
+    }
     await this.sandbox?.dispose();
     if (this.sandbox !== null) {
       this.sandbox = null;

package/src/ui/model-picker.ts

@@ -12,7 +12,7 @@ export interface ModelSelection {
   readonly thinkingLevel?: ThinkingLevel;
 }
 
-const LEVELS = ["off", "minimal", "low", "medium", "high", "xhigh"] as const;
+const LEVELS = ["off", "minimal", "low", "medium", "high", "xhigh", "max"] as const;
 type SelectableThinkingLevel = (typeof LEVELS)[number];
 
 /** Sentinel SelectList value for "always use cheapest available". */