@selesai/code 0.8.4 → 0.8.6

package/CHANGELOG.md

@@ -4,6 +4,28 @@ All notable changes to `@selesai/code` will be documented in this file.
 
 ## [Unreleased]
 
+## [0.8.6] - 2026-08-15
+
+### Added
+- **Auto-relaunching workflows.** The four `/workflow-*` modes (task, prototype, quicktype, loop) now run fully unattended: when a scripted workflow exhausts its per-run fan-out budget before the goal is clean, the async executor mints a fresh budget and re-runs the same script with the same mission and progress file, up to `config.maxWorkflowAutoRelaunches` (default 12; `0` = unlimited). Only `budget` results relaunch — real child failures still surface immediately. `AsyncStatus` gained `workflowRelaunchCount`.
+- **Packaged prompts now load.** The pi-subagents extension registers `resources_discover`, so the bundled prompts (`review-loop.md`, `parallel-*.md`, `gather-context-and-clarify.md`) are actually discoverable. The `workflow-drive.md` prompt was removed as superseded by auto-relaunch.
+- **Goal visibility on workflow launches.** Each `/workflow-*` launch emits its goal at start (visible in `subagent status` workflow emits) and names the auto-created mission after the goal (`mission.list` / `mission.show`).
+
+### Changed
+- **Bounded build→review→fix rounds.** The workflow auto-loop now instructs the builder to implement one small, self-contained slice per round ("do not attempt the whole plan") instead of the entire plan at once, with per-run timeouts (build/fix 45m, review 15m).
+- **Progress-file scoped reviews.** Builder appends a `## Round N` ledger entry (files, summary, validation) to `.pi-subagents/progress/<mode>.md`; the commentator scopes its review to the latest round entry plus the immediately preceding fix entry, falling back to the full uncommitted diff when the file is missing or empty. Blocking reviews must list a `Remaining work:` section; the next build round picks up from it.
+- **Previous review feedback now reaches the next builder round** (previously it was dropped).
+
+## [0.8.5] - 2026-08-14
+
+### Added
+- **`/settings-factory-reset` slash command.** Restores `~/.selesai/agent/settings.json` to the bundled factory defaults after a warning and confirmation. A backup is saved as `settings.json.bak` before the reset, credentials (`auth.json`), sessions, extensions, skills, and themes are untouched, and settings reload live without a restart.
+- **Install from the public repository.** `npm install -g github:SelesaiInTech/selesai-code` now works: the package gained a `prepare` script that builds `dist` from source on install.
+
+### Changed
+- **Simplified install guide.** The documented install command is now plain `npm install -g @selesai/code`. The previous `--ignore-scripts` advice was actively harmful: `@ast-grep/cli` (a runtime dependency) requires its `postinstall` to fetch its platform binary. Quickstart, README, and doc-web install pages updated, and quickstart no longer references the upstream `pi` binary or `~/.pi` paths.
+- **Bumped default `retry.maxRetries` from 5 to 7** in bundled defaults.
+
 ## [0.8.4] - 2026-08-13
 
 ### Fixed

package/README.md

@@ -7,7 +7,7 @@ The goal is simple: make a capable coding agent useful out of the box, maximize
 ## Install
 
 ```bash
-npm install -g --ignore-scripts @selesai/code
+npm install -g @selesai/code
 selesai
 ```
 
@@ -17,6 +17,12 @@ For a one-off run using the latest published package:
 npx @selesai/code
 ```
 
+To install directly from the public repository (builds the latest `main` from source):
+
+```bash
+npm install -g github:SelesaiInTech/selesai-code
+```
+
 The published npm package is **`@selesai/code`** and its executable is **`selesai`**. Selesai uses `~/.selesai/agent` for user state and `.selesai/` for project-local settings and resources.
 
 Start with a prompt:

package/dist/core/slash-commands.js

@@ -1,6 +1,7 @@
 import { APP_NAME } from "../config.js";
 export const BUILTIN_SLASH_COMMANDS = [
     { name: "settings", description: "Open settings menu" },
+    { name: "settings-factory-reset", description: "Reset settings to factory defaults (with confirmation)" },
     { name: "model", description: "Select model (opens selector UI)", argumentHint: "<provider/model>" },
     { name: "scoped-models", description: "Enable/disable models for Ctrl+P cycling" },
     { name: "export", description: "Export session (HTML default, or specify path: .html/.jsonl)" },

package/dist/defaults/models.json

@@ -15,13 +15,8 @@
           "name": "DeepSeek V4 Flash",
           "reasoning": true,
           "input": ["text"],
-          "contextWindow": 1000000,
-          "cost": {
-            "input": 0.15,
-            "output": 0.3,
-            "cacheRead": 0.003,
-            "cacheWrite": 0
-          },
+          "contextWindow": 512000,
+          "maxTokens": 64000,
           "thinkingLevelMap": {
             "minimal": null,
             "low": null,
@@ -37,50 +32,40 @@
           }
         },
         {
-          "id": "kimi-k3",
-          "name": "Kimi K3",
+          "id": "deepseek-v4-pro",
+          "name": "DeepSeek V4 pro",
           "reasoning": true,
-          "input": ["text", "image"],
-          "contextWindow": 262128,
-          "cost": {
-            "input": 3,
-            "output": 15,
-            "cacheRead": 1.5,
-            "cacheWrite": 0
+          "input": ["text"],
+          "contextWindow": 512000,
+          "maxTokens": 64000,
+          "thinkingLevelMap": {
+            "minimal": null,
+            "low": null,
+            "medium": null,
+            "high": null,
+            "xhigh": null,
+            "max": "max"
           },
           "compat": {
             "supportsDeveloperRole": false,
-            "supportsReasoningEffort": false
+            "requiresReasoningContentOnAssistantMessages": true,
+            "thinkingFormat": "deepseek"
           }
         },
         {
-          "id": "kimi-k2.7-code",
-          "name": "Kimi K2.7 Code",
+          "id": "kimi-k3",
+          "name": "Kimi K3",
           "reasoning": true,
           "input": ["text", "image"],
-          "contextWindow": 262128,
-          "cost": {
-            "input": 0.95,
-            "output": 4,
-            "cacheRead": 0.2375,
-            "cacheWrite": 0
-          },
-          "compat": {
-            "supportsDeveloperRole": false,
-            "supportsReasoningEffort": false
-          }
-        },
-        {
-          "id": "qwen3.5-397b",
-          "name": "Qwen3.5 397B",
-          "reasoning": true,
-          "input": ["text"],
-          "contextWindow": 262128,
-          "cost": {
-            "input": 0.69,
-            "output": 4.14,
-            "cacheRead": 0.1725,
-            "cacheWrite": 0
+          "contextWindow": 512000,
+          "maxTokens": 64000,
+          "thinkingLevelMap": {
+            "minimal": null,
+            "low": "low",
+            "medium": null,
+            "high": "high",
+            "xhigh": null,
+            "max": "max"
           },
           "compat": {
             "supportsDeveloperRole": false,
@@ -88,16 +73,19 @@
           }
         },
         {
-          "id": "qwen3.6-35b",
-          "name": "Qwen3.6 35B",
+          "id": "kimi-k2.7-code",
+          "name": "Kimi K2.7 Code",
           "reasoning": true,
           "input": ["text", "image"],
-          "contextWindow": 131056,
-          "cost": {
-            "input": 0.29,
-            "output": 1.15,
-            "cacheRead": 0.0725,
-            "cacheWrite": 0
+          "contextWindow": 262128,
+          "maxTokens": 64000,
+          "thinkingLevelMap": {
+            "minimal": null,
+            "low": "low",
+            "medium": null,
+            "high": "high",
+            "xhigh": null,
+            "max": "max"
           },
           "compat": {
             "supportsDeveloperRole": false,
@@ -110,11 +98,14 @@
           "reasoning": true,
           "input": ["text"],
           "contextWindow": 1000000,
-          "cost": {
-            "input": 1.45,
-            "output": 4.5,
-            "cacheRead": 0.3625,
-            "cacheWrite": 0
+          "maxTokens": 64000,
+          "thinkingLevelMap": {
+            "minimal": null,
+            "low": "low",
+            "medium": null,
+            "high": "high",
+            "xhigh": null,
+            "max": null
           },
           "compat": {
             "supportsDeveloperRole": false,

package/dist/defaults/settings.json

@@ -1,7 +1,7 @@
 {
   "autoHandoff": {
-    "enabled": false,
-    "thresholdTokens": 128000
+    "enabled": true,
+    "thresholdTokens": 256000
   },
   "collapseChangelog": true,
   "compaction": {
@@ -9,19 +9,18 @@
     "keepRecentTokens": 32000,
     "reserveTokens": 64000
   },
-  "defaultModel": "glm-5.2",
+  "defaultModel": "deepseek-v4-flash",
   "defaultProvider": "tokenin",
-  "defaultThinkingLevel": "high",
+  "defaultThinkingLevel": "max",
   "doubleEscapeAction": "tree",
   "editorPaddingX": 0,
   "outputPad": 0,
-  "showCacheMissNotices": true,
+  "showCacheMissNotices": false,
   "followUpMode": "one-at-a-time",
   "hideThinkingBlock": false,
   "images": {
     "autoResize": true
   },
-  "lastChangelogVersion": "0.80.2",
   "packages": [
     "git:github.com/MasuRii/pi-tool-display",
     "git:github.com/omaclaren/pi-markdown-preview"
@@ -29,7 +28,7 @@
   "quietStartup": true,
   "retry": {
     "enabled": true,
-    "maxRetries": 3
+    "maxRetries": 7
   },
   "steeringMode": "all",
   "terminal": {
@@ -39,7 +38,7 @@
   "subagents": {
     "agentOverrides": {
       "architect": {
-        "model": "tokenin/kimi-k3:high"
+        "model": "tokenin/deepseek-v4-pro:max"
       },
       "builder": {
         "model": "tokenin/deepseek-v4-flash:max"

package/dist/extensions/agent-browser.test.ts

@@ -1,7 +1,7 @@
 import { afterAll, beforeAll, describe, expect, it, vi } from "vitest";
-import { existsSync, mkdtempSync, rmSync } from "node:fs";
+import { existsSync, mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
-import { join } from "node:path";
+import { dirname, join } from "node:path";
 import agentBrowserSetupExtension, {
 	DECLINED_MARKER_NAME,
 	ensureAgentBrowser,
@@ -192,4 +192,180 @@ describe("agent-browser setup extension", () => {
 	it("declined marker path lives in the agent dir", () => {
 		expect(getDeclinedMarkerPath()).toBe(join(tmpDir, DECLINED_MARKER_NAME));
 	});
+
+	it("skips probing when the decline marker exists", async () => {
+		const { pi, handlers, exec } = makePi();
+		await agentBrowserSetupExtension(pi);
+		const markerPath = getDeclinedMarkerPath();
+		mkdirSync(dirname(markerPath), { recursive: true });
+		writeFileSync(markerPath, "1", "utf-8");
+		try {
+			await getSessionStartHandler(handlers)({ type: "session_start", reason: "startup" }, makeCtx());
+			expect(exec).not.toHaveBeenCalled();
+		} finally {
+			rmSync(markerPath, { force: true });
+		}
+	});
+
+	it("isAgentBrowserInstalled returns false when exec throws", async () => {
+		const { pi } = makePi(async () => {
+			throw new Error("spawn failed");
+		});
+		expect(await isAgentBrowserInstalled(pi)).toBe(false);
+	});
+
+	it("isAgentBrowserInstalled returns false when the probe is killed", async () => {
+		const { pi } = makePi(async () => ({ code: 0, stdout: "", stderr: "", killed: true }));
+		expect(await isAgentBrowserInstalled(pi)).toBe(false);
+	});
+
+	it("reports a timed-out npm install", async () => {
+		const execImpl = vi
+			.fn()
+			.mockResolvedValueOnce(fail) // probe: missing
+			.mockResolvedValueOnce({ code: 0, stdout: "", stderr: "", killed: true }); // npm killed
+		const { pi, handlers, exec } = makePi(execImpl as any);
+		await agentBrowserSetupExtension(pi);
+		const ctx = makeCtx();
+		await getSessionStartHandler(handlers)({ type: "session_start", reason: "startup" }, ctx);
+
+		expect(ctx.ui.notify).toHaveBeenCalledWith(expect.stringContaining("timed out"), "error");
+		expect(exec).toHaveBeenCalledTimes(2);
+	});
+
+	it("reports npm success but missing binary on PATH", async () => {
+		const execImpl = vi
+			.fn()
+			.mockResolvedValueOnce(fail) // probe: missing
+			.mockResolvedValueOnce(ok) // npm i -g agent-browser
+			.mockResolvedValueOnce(fail) // probe: still missing
+			.mockResolvedValueOnce(ok); // (unused)
+		const { pi, handlers, exec } = makePi(execImpl as any);
+		await agentBrowserSetupExtension(pi);
+		const ctx = makeCtx();
+		await getSessionStartHandler(handlers)({ type: "session_start", reason: "startup" }, ctx);
+
+		expect(ctx.ui.notify).toHaveBeenCalledWith(expect.stringContaining("still not on PATH"), "warning");
+		expect(exec).toHaveBeenCalledTimes(3);
+	});
+
+	it("warns when browser provisioning fails after install", async () => {
+		const execImpl = vi
+			.fn()
+			.mockResolvedValueOnce(fail) // probe: missing
+			.mockResolvedValueOnce(ok) // npm i -g agent-browser
+			.mockResolvedValueOnce(ok) // probe: now present
+			.mockResolvedValueOnce({ code: 1, stdout: "", stderr: "browser failed", killed: false }); // agent-browser install fails
+		const { pi, handlers, exec } = makePi(execImpl as any);
+		await agentBrowserSetupExtension(pi);
+		const ctx = makeCtx();
+		await getSessionStartHandler(handlers)({ type: "session_start", reason: "startup" }, ctx);
+
+		expect(ctx.ui.notify).toHaveBeenCalledWith(expect.stringContaining("Run `agent-browser install`"), "info");
+		expect(exec).toHaveBeenCalledTimes(4);
+	});
+
+	it("provisionBrowser returns false when exec throws", async () => {
+		const execImpl = vi
+			.fn()
+			.mockResolvedValueOnce(fail) // probe: missing
+			.mockResolvedValueOnce(ok) // npm i -g agent-browser
+			.mockResolvedValueOnce(ok) // probe: now present
+			.mockRejectedValueOnce(new Error("spawn failed")); // agent-browser install throws
+		const { pi, handlers, exec } = makePi(execImpl as any);
+		await agentBrowserSetupExtension(pi);
+		const ctx = makeCtx();
+		await getSessionStartHandler(handlers)({ type: "session_start", reason: "startup" }, ctx);
+
+		expect(ctx.ui.notify).toHaveBeenCalledWith(expect.stringContaining("Run `agent-browser install`"), "info");
+		expect(exec).toHaveBeenCalledTimes(4);
+	});
+
+	it("ensureAgentBrowser returns false when the user declines", async () => {
+		const execImpl = vi.fn().mockResolvedValue(fail);
+		const { pi } = makePi(execImpl as any);
+		const ctx = makeCtx({ confirmResult: false });
+		expect(await ensureAgentBrowser(pi, ctx)).toBe(false);
+	});
+
+	it("ensureAgentBrowser returns true when already installed", async () => {
+		const { pi } = makePi(async () => ok);
+		expect(await ensureAgentBrowser(pi, makeCtx())).toBe(true);
+	});
+
+	it("ensureAgentBrowser returns false without UI", async () => {
+		const warnSpy = vi.spyOn(console, "warn").mockImplementation(() => {});
+		const execImpl = vi.fn().mockResolvedValue(fail);
+		const { pi } = makePi(execImpl as any);
+		expect(await ensureAgentBrowser(pi, makeCtx({ hasUI: false }))).toBe(false);
+		expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining(INSTALL_COMMAND));
+	});
+
+	it("ensureAgentBrowser returns false when npm install fails", async () => {
+		const execImpl = vi
+			.fn()
+			.mockResolvedValueOnce(fail) // probe
+			.mockResolvedValueOnce({ code: 1, stdout: "", stderr: "EACCES", killed: false }); // npm fails
+		const { pi } = makePi(execImpl as any);
+		expect(await ensureAgentBrowser(pi, makeCtx())).toBe(false);
+	});
+
+	it("npm install failure surfaces stdout when stderr is empty", async () => {
+		const execImpl = vi
+			.fn()
+			.mockResolvedValueOnce(fail) // probe
+			.mockResolvedValueOnce({ code: 1, stdout: "npm error log", stderr: "", killed: false }); // npm fails with stdout only
+		const { pi } = makePi(execImpl as any);
+		const ctx = makeCtx();
+		expect(await ensureAgentBrowser(pi, ctx)).toBe(false);
+		expect(ctx.ui.notify).toHaveBeenCalledWith(expect.stringContaining("npm error log"), "error");
+	});
+
+	it("npm install failure falls back to exit status when output is empty", async () => {
+		const execImpl = vi
+			.fn()
+			.mockResolvedValueOnce(fail) // probe
+			.mockResolvedValueOnce({ code: 1, stdout: "", stderr: "", killed: false }); // npm fails with no output
+		const { pi } = makePi(execImpl as any);
+		const ctx = makeCtx();
+		expect(await ensureAgentBrowser(pi, ctx)).toBe(false);
+		expect(ctx.ui.notify).toHaveBeenCalledWith(expect.stringContaining("exit status 1"), "error");
+	});
+
+	it("notify is a no-op without UI even when the CLI is installed", async () => {
+		const { pi } = makePi(async () => ok);
+		const ctx = makeCtx({ hasUI: false });
+		expect(await ensureAgentBrowser(pi, ctx)).toBe(true);
+	});
+
+	it("ensureAgentBrowser returns true after full install flow", async () => {
+		const execImpl = vi
+			.fn()
+			.mockResolvedValueOnce(fail) // probe: missing
+			.mockResolvedValueOnce(ok) // npm i -g agent-browser
+			.mockResolvedValueOnce(ok) // probe: now present
+			.mockResolvedValueOnce(ok); // agent-browser install
+		const { pi } = makePi(execImpl as any);
+		expect(await ensureAgentBrowser(pi, makeCtx())).toBe(true);
+	});
+
+	it("cliCommand wraps args in cmd.exe on win32", () => {
+		const original = process.platform;
+		Object.defineProperty(process, "platform", { value: "win32" });
+		try {
+			const { ensureAgentBrowser: _e, ...rest } = {} as any;
+			// Re-import to pick up the platform at module scope is not needed;
+			// cliCommand is module-private, so exercise it through the public flow.
+			const execImpl = vi.fn().mockResolvedValue(ok);
+			const { pi } = makePi(execImpl as any);
+			return ensureAgentBrowser(pi, makeCtx()).then((result) => {
+				expect(result).toBe(true);
+				const call = execImpl.mock.calls[0];
+				expect(call[0]).toBe("cmd");
+				expect(call[1]).toEqual(["/c", "agent-browser", "--version"]);
+			});
+		} finally {
+			Object.defineProperty(process, "platform", { value: original });
+		}
+	});
 });

package/dist/extensions/agent-browser.ts

@@ -75,6 +75,8 @@ async function runNpmInstall(pi: ExtensionAPI): Promise<string | null> {
 	const result = await pi.exec(command, args, { timeout: INSTALL_TIMEOUT_MS });
 	if (result.killed) return "npm install timed out";
 	if (result.code !== 0) {
+		// The exit-status fallback is always non-empty, so the final || is unreachable.
+		/* v8 ignore next 1 */
 		return (result.stderr || result.stdout || `exit status ${result.code}`).trim() || "npm install failed";
 	}
 	return null;