@selesai/code 0.9.10 → 0.9.12

package/dist/extensions/pi-subagents/test/unit/workflow-receipt.test.ts

@@ -0,0 +1,113 @@
+import assert from "node:assert/strict";
+import fs from "node:fs";
+import os from "node:os";
+import path from "node:path";
+import { afterEach, describe, it } from "node:test";
+import { buildWorkflowReceipt, readWorkflowReceipt, resolveWorkflowReceiptResume, workflowReceiptPath, writeWorkflowReceipt } from "../../src/workflows/workflow-receipt.ts";
+import type { WorkflowScriptChildResult } from "../../src/workflows/scripted-workflow.ts";
+
+const roots: string[] = [];
+
+afterEach(() => {
+	for (const root of roots.splice(0)) fs.rmSync(root, { recursive: true, force: true });
+});
+
+function tempRoot(): string {
+	const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-workflow-receipt-"));
+	roots.push(root);
+	return root;
+}
+
+function child(key: string, overrides: Partial<WorkflowScriptChildResult> = {}): WorkflowScriptChildResult {
+	return {
+		key,
+		ok: true,
+		agent: "reviewer",
+		runId: `run-${key}`,
+		output: "x".repeat(100_000),
+		structuredOutput: { report: "x".repeat(100_000) },
+		artifactPaths: [`/tmp/${key}.md`],
+		requestedContext: "fresh",
+		resolvedContext: "fresh",
+		outputReference: `/tmp/${key}.md`,
+		resumability: { state: "resumable" },
+		continuation: { runIds: [`run-${key}`] },
+		...overrides,
+	};
+}
+
+describe("workflow receipts", () => {
+	it("builds one metadata-only entry per workflow child", () => {
+		const children = Array.from({ length: 1_000 }, (_, index) => child(`child-${index}`));
+		const receipt = buildWorkflowReceipt({ workflowRunId: "workflow-1", state: "complete", children, createdAt: 10 });
+
+		assert.equal(Object.keys(receipt.entries).length, children.length);
+		assert.deepEqual(receipt.entries["child-0"], {
+			key: "child-0",
+			agent: "reviewer",
+			requestedContext: "fresh",
+			resolvedContext: "fresh",
+			latestRunId: "run-child-0",
+			resumability: { state: "resumable" },
+			outputReference: "/tmp/child-0.md",
+			continuation: { runIds: ["run-child-0"] },
+		});
+		const serialized = JSON.stringify(receipt);
+		assert.doesNotMatch(serialized, /structuredOutput|artifactPaths|"output"/);
+		assert.ok(serialized.length < 400_000, `receipt metadata unexpectedly large: ${serialized.length}`);
+	});
+
+	it("writes and resolves one exact terminal receipt", () => {
+		const asyncRoot = tempRoot();
+		const asyncDir = path.join(asyncRoot, "workflow-1");
+		fs.mkdirSync(asyncDir, { recursive: true });
+		const receipt = buildWorkflowReceipt({ workflowRunId: "workflow-1", state: "complete", children: [child("advisor")] });
+		assert.equal(writeWorkflowReceipt(asyncDir, receipt), workflowReceiptPath(asyncRoot, "workflow-1"));
+		assert.deepEqual(readWorkflowReceipt(asyncRoot, "workflow-1"), receipt);
+		let validations = 0;
+		const runId = resolveWorkflowReceiptResume({
+			reference: { workflowRunId: "workflow-1", key: "advisor", latest: true },
+			asyncDirRoot: asyncRoot,
+			assertResumable(value) {
+				validations += 1;
+				assert.equal(value, "run-advisor");
+			},
+		});
+		assert.equal(runId, "run-advisor");
+		assert.equal(validations, 1);
+	});
+
+	it("fails closed for missing, non-resumable, and stale receipt references", () => {
+		const asyncRoot = tempRoot();
+		assert.throws(
+			() => resolveWorkflowReceiptResume({ reference: { workflowRunId: "missing", key: "advisor", latest: true }, asyncDirRoot: asyncRoot }),
+			/not found/,
+		);
+
+		const asyncDir = path.join(asyncRoot, "workflow-1");
+		fs.mkdirSync(asyncDir, { recursive: true });
+		writeWorkflowReceipt(asyncDir, buildWorkflowReceipt({
+			workflowRunId: "workflow-1",
+			state: "failed",
+			children: [child("advisor", { resumability: { state: "not-resumable", reason: "session removed" } })],
+		}));
+		assert.throws(
+			() => resolveWorkflowReceiptResume({ reference: { workflowRunId: "workflow-1", key: "missing", latest: true }, asyncDirRoot: asyncRoot }),
+			/no child key 'missing'/,
+		);
+		assert.throws(
+			() => resolveWorkflowReceiptResume({ reference: { workflowRunId: "workflow-1", key: "advisor", latest: true }, asyncDirRoot: asyncRoot }),
+			/not resumable: session removed/,
+		);
+
+		const receiptPath = workflowReceiptPath(asyncRoot, "workflow-1");
+		const stale = JSON.parse(fs.readFileSync(receiptPath, "utf-8")) as Record<string, unknown>;
+		((stale.entries as Record<string, Record<string, unknown>>).advisor!).latestRunId = "other-run";
+		fs.writeFileSync(receiptPath, JSON.stringify(stale));
+		assert.throws(() => readWorkflowReceipt(asyncRoot, "workflow-1"), /stale: latestRunId/);
+		assert.throws(
+			() => resolveWorkflowReceiptResume({ reference: { workflowRunId: "workflow-1", key: "advisor", latest: false } as never, asyncDirRoot: asyncRoot }),
+			/requires latest: true/,
+		);
+	});
+});

package/dist/extensions/pi-subagents/test/unit/worktree.test.ts

@@ -125,9 +125,9 @@ describe("worktree", () => {
 		}
 	});
 
-	it("rejects worktree base directories inside Selesai extensions", () => {
-		const repoDir = createRepo("selesai-worktree-extension-dir-");
-		const tempHome = fs.mkdtempSync(path.join(os.tmpdir(), "selesai-worktree-home-"));
+	it("rejects worktree base directories inside Pi extensions", () => {
+		const repoDir = createRepo("pi-worktree-extension-dir-");
+		const tempHome = fs.mkdtempSync(path.join(os.tmpdir(), "pi-worktree-home-"));
 		const previousAgentDir = process.env.SELESAI_CODING_AGENT_DIR;
 		const previousHome = process.env.HOME;
 		const previousUserProfile = process.env.USERPROFILE;
@@ -156,9 +156,9 @@ describe("worktree", () => {
 		}
 	});
 
-	it("rejects symlinked worktree base directories inside Selesai extensions", () => {
-		const repoDir = createRepo("selesai-worktree-extension-symlink-");
-		const tempHome = fs.mkdtempSync(path.join(os.tmpdir(), "selesai-worktree-home-"));
+	it("rejects symlinked worktree base directories inside Pi extensions", () => {
+		const repoDir = createRepo("pi-worktree-extension-symlink-");
+		const tempHome = fs.mkdtempSync(path.join(os.tmpdir(), "pi-worktree-home-"));
 		const previousAgentDir = process.env.SELESAI_CODING_AGENT_DIR;
 		const previousHome = process.env.HOME;
 		const previousUserProfile = process.env.USERPROFILE;

package/dist/extensions/pi-web-agent/package.json

@@ -1,31 +1,31 @@
-{
-	"name": "@selesai/bundled-pi-web-agent",
-	"version": "1.5.1",
-	"private": true,
-	"description": "Bundled pi-web-agent extension for reliable web access with explicit search, fetch, and headless boundaries. Vendored source-first from @demigodmode/pi-web-agent.",
-	"type": "module",
-	"keywords": [
-		"pi-package",
-		"pi",
-		"extension",
-		"web-search",
-		"web-fetch"
-	],
-	"license": "AGPL-3.0-only",
-	"pi": {
-		"extensions": [
-			"./src/extension.ts"
-		]
-	},
-	"dependencies": {
-		"@mozilla/readability": "^0.6.0",
-		"cheerio": "^1.1.0",
-		"jsdom": "^26.0.0",
-		"playwright": "^1.60.0",
-		"typebox": "^1.1.37"
-	},
-	"peerDependencies": {
-		"@selesai/code": "*",
-		"@earendil-works/pi-tui": "*"
-	}
-}
+{
+	"name": "@selesai/bundled-pi-web-agent",
+	"version": "1.5.1",
+	"private": true,
+	"description": "Bundled pi-web-agent extension for reliable web access with explicit search, fetch, and headless boundaries. Vendored source-first from @demigodmode/pi-web-agent.",
+	"type": "module",
+	"keywords": [
+		"pi-package",
+		"pi",
+		"extension",
+		"web-search",
+		"web-fetch"
+	],
+	"license": "AGPL-3.0-only",
+	"pi": {
+		"extensions": [
+			"./src/extension.ts"
+		]
+	},
+	"dependencies": {
+		"@mozilla/readability": "^0.6.0",
+		"cheerio": "^1.1.0",
+		"jsdom": "^26.0.0",
+		"playwright": "^1.60.0",
+		"typebox": "^1.1.37"
+	},
+	"peerDependencies": {
+		"@selesai/code": "*",
+		"@earendil-works/pi-tui": "*"
+	}
+}

package/dist/extensions/pi-web-agent/src/backends/config.ts

@@ -1,205 +1,205 @@
-export type SearxngOptions = {
-  categories?: string[];
-  language?: string;
-  safesearch?: 0 | 1 | 2;
-};
-
-export type FirecrawlOptions = {
-  formats?: string[];
-  onlyMainContent?: boolean;
-};
-
-export type SearchBackendConfig = {
-  provider: 'duckduckgo' | 'searxng' | 'brave';
-  baseUrl?: string;
-  fallback?: 'duckduckgo';
-  options?: SearxngOptions;
-};
-
-export type FetchBackendConfig = {
-  provider: 'http' | 'firecrawl';
-  baseUrl?: string;
-  apiKey?: string;
-  fallback?: 'http';
-  options?: FirecrawlOptions;
-};
-export type HeadlessBackendConfig = { provider: 'local-browser' };
-
-export type BackendConfig = {
-  search: SearchBackendConfig;
-  fetch: FetchBackendConfig;
-  headless: HeadlessBackendConfig;
-};
-
-export type BackendConfigOverride = {
-  search?: Partial<SearchBackendConfig>;
-  fetch?: Partial<FetchBackendConfig>;
-  headless?: Partial<HeadlessBackendConfig>;
-};
-
-export type BackendConfigFile = {
-  backends?: {
-    search?: { provider?: unknown; baseUrl?: unknown; fallback?: unknown; options?: unknown };
-    fetch?: { provider?: unknown; baseUrl?: unknown; apiKey?: unknown; fallback?: unknown; options?: unknown };
-    headless?: { provider?: unknown };
-  };
-};
-
-export const DEFAULT_BACKEND_CONFIG: BackendConfig = {
-  search: { provider: 'duckduckgo' },
-  fetch: { provider: 'http' },
-  headless: { provider: 'local-browser' }
-};
-
-function extractStringArray(value: unknown): string[] | undefined {
-  if (!Array.isArray(value)) return undefined;
-  const strings = value.filter((item): item is string => typeof item === 'string');
-  return strings.length === value.length ? strings : undefined;
-}
-
-function extractSearxngOptions(value: unknown): SearxngOptions | undefined {
-  if (!value || typeof value !== 'object') return undefined;
-  const raw = value as { categories?: unknown; language?: unknown; safesearch?: unknown };
-  const options: SearxngOptions = {};
-  const categories = extractStringArray(raw.categories);
-  if (categories) options.categories = categories;
-  if (typeof raw.language === 'string') options.language = raw.language;
-  if (raw.safesearch === 0 || raw.safesearch === 1 || raw.safesearch === 2) options.safesearch = raw.safesearch;
-  return Object.keys(options).length > 0 ? options : undefined;
-}
-
-function extractFirecrawlOptions(value: unknown): FirecrawlOptions | undefined {
-  if (!value || typeof value !== 'object') return undefined;
-  const raw = value as { formats?: unknown; onlyMainContent?: unknown };
-  const options: FirecrawlOptions = {};
-  const formats = extractStringArray(raw.formats);
-  if (formats) options.formats = formats;
-  if (typeof raw.onlyMainContent === 'boolean') options.onlyMainContent = raw.onlyMainContent;
-  return Object.keys(options).length > 0 ? options : undefined;
-}
-
-export function extractBackendConfigOverride(
-  file: BackendConfigFile | null | undefined
-): BackendConfigOverride {
-  const backends = file?.backends;
-  const override: BackendConfigOverride = {};
-
-  if (
-    backends?.search?.provider === 'duckduckgo' ||
-    backends?.search?.provider === 'searxng' ||
-    backends?.search?.provider === 'brave'
-  ) {
-    override.search = { provider: backends.search.provider };
-    if (backends.search.provider === 'searxng' && typeof backends.search.baseUrl === 'string') {
-      override.search.baseUrl = backends.search.baseUrl;
-    }
-    if (backends.search.fallback === 'duckduckgo') {
-      override.search.fallback = 'duckduckgo';
-    }
-    if (backends.search.provider === 'searxng') {
-      const options = extractSearxngOptions(backends.search.options);
-      if (options) {
-        override.search.options = options;
-      }
-    }
-  }
-
-  if (backends?.fetch?.provider === 'http' || backends?.fetch?.provider === 'firecrawl') {
-    override.fetch = { provider: backends.fetch.provider };
-    if (typeof backends.fetch.baseUrl === 'string') {
-      override.fetch.baseUrl = backends.fetch.baseUrl;
-    }
-    if (typeof backends.fetch.apiKey === 'string') {
-      override.fetch.apiKey = backends.fetch.apiKey;
-    }
-    if (backends.fetch.fallback === 'http') {
-      override.fetch.fallback = 'http';
-    }
-    const options = extractFirecrawlOptions(backends.fetch.options);
-    if (options) {
-      override.fetch.options = options;
-    }
-  }
-
-  if (backends?.headless?.provider === 'local-browser') {
-    override.headless = { provider: 'local-browser' };
-  }
-
-  return override;
-}
-
-export function validateBackendConfig(config: BackendConfig): string[] {
-  const issues: string[] = [];
-
-  if (config.search.provider === 'searxng' && !config.search.baseUrl) {
-    issues.push('search provider searxng requires backends.search.baseUrl');
-  }
-
-  if (config.fetch.provider === 'firecrawl' && !config.fetch.baseUrl) {
-    issues.push('fetch provider firecrawl requires backends.fetch.baseUrl');
-  }
-
-  if (config.search.fallback === 'duckduckgo' && config.search.provider !== 'searxng' && config.search.provider !== 'brave') {
-    issues.push('search fallback duckduckgo is only supported when search provider is searxng or brave');
-  }
-
-  if (config.fetch.fallback === 'http' && config.fetch.provider !== 'firecrawl') {
-    issues.push('fetch fallback http is only supported when fetch provider is firecrawl');
-  }
-
-  if (config.search.options?.categories && config.search.options.categories.length === 0) {
-    issues.push('search options.categories must contain at least one category when provided');
-  }
-
-  if (config.search.options?.language !== undefined && !config.search.options.language.trim()) {
-    issues.push('search options.language must not be empty when provided');
-  }
-
-  if (
-    config.search.options?.safesearch !== undefined &&
-    ![0, 1, 2].includes(config.search.options.safesearch)
-  ) {
-    issues.push('search options.safesearch must be 0, 1, or 2 when provided');
-  }
-
-  if (config.fetch.options?.formats && config.fetch.options.formats.length === 0) {
-    issues.push('fetch options.formats must contain at least one format when provided');
-  }
-
-  return issues;
-}
-
-function mergeSearchConfig(
-  current: SearchBackendConfig,
-  override: Partial<SearchBackendConfig> | undefined
-): SearchBackendConfig {
-  if (!override) return current;
-  if (override.provider && override.provider !== current.provider) {
-    return { ...override, provider: override.provider };
-  }
-  return { ...current, ...override };
-}
-
-function mergeFetchConfig(
-  current: FetchBackendConfig,
-  override: Partial<FetchBackendConfig> | undefined
-): FetchBackendConfig {
-  if (!override) return current;
-  if (override.provider && override.provider !== current.provider) {
-    return { ...override, provider: override.provider };
-  }
-  return { ...current, ...override };
-}
-
-export function mergeBackendConfigLayers(
-  ...layers: Array<BackendConfig | BackendConfigOverride | undefined>
-): BackendConfig {
-  return layers.reduce<BackendConfig>(
-    (merged, layer) => ({
-      search: mergeSearchConfig(merged.search, layer?.search),
-      fetch: mergeFetchConfig(merged.fetch, layer?.fetch),
-      headless: { ...merged.headless, ...layer?.headless }
-    }),
-    DEFAULT_BACKEND_CONFIG
-  );
-}
+export type SearxngOptions = {
+  categories?: string[];
+  language?: string;
+  safesearch?: 0 | 1 | 2;
+};
+
+export type FirecrawlOptions = {
+  formats?: string[];
+  onlyMainContent?: boolean;
+};
+
+export type SearchBackendConfig = {
+  provider: 'duckduckgo' | 'searxng' | 'brave';
+  baseUrl?: string;
+  fallback?: 'duckduckgo';
+  options?: SearxngOptions;
+};
+
+export type FetchBackendConfig = {
+  provider: 'http' | 'firecrawl';
+  baseUrl?: string;
+  apiKey?: string;
+  fallback?: 'http';
+  options?: FirecrawlOptions;
+};
+export type HeadlessBackendConfig = { provider: 'local-browser' };
+
+export type BackendConfig = {
+  search: SearchBackendConfig;
+  fetch: FetchBackendConfig;
+  headless: HeadlessBackendConfig;
+};
+
+export type BackendConfigOverride = {
+  search?: Partial<SearchBackendConfig>;
+  fetch?: Partial<FetchBackendConfig>;
+  headless?: Partial<HeadlessBackendConfig>;
+};
+
+export type BackendConfigFile = {
+  backends?: {
+    search?: { provider?: unknown; baseUrl?: unknown; fallback?: unknown; options?: unknown };
+    fetch?: { provider?: unknown; baseUrl?: unknown; apiKey?: unknown; fallback?: unknown; options?: unknown };
+    headless?: { provider?: unknown };
+  };
+};
+
+export const DEFAULT_BACKEND_CONFIG: BackendConfig = {
+  search: { provider: 'duckduckgo' },
+  fetch: { provider: 'http' },
+  headless: { provider: 'local-browser' }
+};
+
+function extractStringArray(value: unknown): string[] | undefined {
+  if (!Array.isArray(value)) return undefined;
+  const strings = value.filter((item): item is string => typeof item === 'string');
+  return strings.length === value.length ? strings : undefined;
+}
+
+function extractSearxngOptions(value: unknown): SearxngOptions | undefined {
+  if (!value || typeof value !== 'object') return undefined;
+  const raw = value as { categories?: unknown; language?: unknown; safesearch?: unknown };
+  const options: SearxngOptions = {};
+  const categories = extractStringArray(raw.categories);
+  if (categories) options.categories = categories;
+  if (typeof raw.language === 'string') options.language = raw.language;
+  if (raw.safesearch === 0 || raw.safesearch === 1 || raw.safesearch === 2) options.safesearch = raw.safesearch;
+  return Object.keys(options).length > 0 ? options : undefined;
+}
+
+function extractFirecrawlOptions(value: unknown): FirecrawlOptions | undefined {
+  if (!value || typeof value !== 'object') return undefined;
+  const raw = value as { formats?: unknown; onlyMainContent?: unknown };
+  const options: FirecrawlOptions = {};
+  const formats = extractStringArray(raw.formats);
+  if (formats) options.formats = formats;
+  if (typeof raw.onlyMainContent === 'boolean') options.onlyMainContent = raw.onlyMainContent;
+  return Object.keys(options).length > 0 ? options : undefined;
+}
+
+export function extractBackendConfigOverride(
+  file: BackendConfigFile | null | undefined
+): BackendConfigOverride {
+  const backends = file?.backends;
+  const override: BackendConfigOverride = {};
+
+  if (
+    backends?.search?.provider === 'duckduckgo' ||
+    backends?.search?.provider === 'searxng' ||
+    backends?.search?.provider === 'brave'
+  ) {
+    override.search = { provider: backends.search.provider };
+    if (backends.search.provider === 'searxng' && typeof backends.search.baseUrl === 'string') {
+      override.search.baseUrl = backends.search.baseUrl;
+    }
+    if (backends.search.fallback === 'duckduckgo') {
+      override.search.fallback = 'duckduckgo';
+    }
+    if (backends.search.provider === 'searxng') {
+      const options = extractSearxngOptions(backends.search.options);
+      if (options) {
+        override.search.options = options;
+      }
+    }
+  }
+
+  if (backends?.fetch?.provider === 'http' || backends?.fetch?.provider === 'firecrawl') {
+    override.fetch = { provider: backends.fetch.provider };
+    if (typeof backends.fetch.baseUrl === 'string') {
+      override.fetch.baseUrl = backends.fetch.baseUrl;
+    }
+    if (typeof backends.fetch.apiKey === 'string') {
+      override.fetch.apiKey = backends.fetch.apiKey;
+    }
+    if (backends.fetch.fallback === 'http') {
+      override.fetch.fallback = 'http';
+    }
+    const options = extractFirecrawlOptions(backends.fetch.options);
+    if (options) {
+      override.fetch.options = options;
+    }
+  }
+
+  if (backends?.headless?.provider === 'local-browser') {
+    override.headless = { provider: 'local-browser' };
+  }
+
+  return override;
+}
+
+export function validateBackendConfig(config: BackendConfig): string[] {
+  const issues: string[] = [];
+
+  if (config.search.provider === 'searxng' && !config.search.baseUrl) {
+    issues.push('search provider searxng requires backends.search.baseUrl');
+  }
+
+  if (config.fetch.provider === 'firecrawl' && !config.fetch.baseUrl) {
+    issues.push('fetch provider firecrawl requires backends.fetch.baseUrl');
+  }
+
+  if (config.search.fallback === 'duckduckgo' && config.search.provider !== 'searxng' && config.search.provider !== 'brave') {
+    issues.push('search fallback duckduckgo is only supported when search provider is searxng or brave');
+  }
+
+  if (config.fetch.fallback === 'http' && config.fetch.provider !== 'firecrawl') {
+    issues.push('fetch fallback http is only supported when fetch provider is firecrawl');
+  }
+
+  if (config.search.options?.categories && config.search.options.categories.length === 0) {
+    issues.push('search options.categories must contain at least one category when provided');
+  }
+
+  if (config.search.options?.language !== undefined && !config.search.options.language.trim()) {
+    issues.push('search options.language must not be empty when provided');
+  }
+
+  if (
+    config.search.options?.safesearch !== undefined &&
+    ![0, 1, 2].includes(config.search.options.safesearch)
+  ) {
+    issues.push('search options.safesearch must be 0, 1, or 2 when provided');
+  }
+
+  if (config.fetch.options?.formats && config.fetch.options.formats.length === 0) {
+    issues.push('fetch options.formats must contain at least one format when provided');
+  }
+
+  return issues;
+}
+
+function mergeSearchConfig(
+  current: SearchBackendConfig,
+  override: Partial<SearchBackendConfig> | undefined
+): SearchBackendConfig {
+  if (!override) return current;
+  if (override.provider && override.provider !== current.provider) {
+    return { ...override, provider: override.provider };
+  }
+  return { ...current, ...override };
+}
+
+function mergeFetchConfig(
+  current: FetchBackendConfig,
+  override: Partial<FetchBackendConfig> | undefined
+): FetchBackendConfig {
+  if (!override) return current;
+  if (override.provider && override.provider !== current.provider) {
+    return { ...override, provider: override.provider };
+  }
+  return { ...current, ...override };
+}
+
+export function mergeBackendConfigLayers(
+  ...layers: Array<BackendConfig | BackendConfigOverride | undefined>
+): BackendConfig {
+  return layers.reduce<BackendConfig>(
+    (merged, layer) => ({
+      search: mergeSearchConfig(merged.search, layer?.search),
+      fetch: mergeFetchConfig(merged.fetch, layer?.fetch),
+      headless: { ...merged.headless, ...layer?.headless }
+    }),
+    DEFAULT_BACKEND_CONFIG
+  );
+}