@clinebot/core 0.0.35 → 0.0.37

package/src/runtime/rpc-spawn-lease.test.ts

@@ -1,81 +0,0 @@
-import assert from "node:assert/strict";
-import { mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
-import os from "node:os";
-import path from "node:path";
-import { afterEach, describe, expect, it } from "vitest";
-import { tryAcquireRpcSpawnLease } from "./rpc-spawn-lease";
-
-describe("tryAcquireRpcSpawnLease", () => {
-	const tempDirs: string[] = [];
-
-	afterEach(() => {
-		delete process.env.CLINE_DATA_DIR;
-		for (const dir of tempDirs.splice(0)) {
-			rmSync(dir, { recursive: true, force: true });
-		}
-	});
-
-	it("allows only one active lease per address", () => {
-		const dataDir = mkdtempSync(path.join(os.tmpdir(), "rpc-spawn-lease-"));
-		tempDirs.push(dataDir);
-		process.env.CLINE_DATA_DIR = dataDir;
-
-		const first = tryAcquireRpcSpawnLease("127.0.0.1:4317");
-		const second = tryAcquireRpcSpawnLease("127.0.0.1:4317");
-
-		expect(first).toBeDefined();
-		expect(second).toBeUndefined();
-
-		first?.release();
-
-		const third = tryAcquireRpcSpawnLease("127.0.0.1:4317");
-		expect(third).toBeDefined();
-		third?.release();
-	});
-
-	it("lets different addresses acquire independent leases", () => {
-		const dataDir = mkdtempSync(path.join(os.tmpdir(), "rpc-spawn-lease-"));
-		tempDirs.push(dataDir);
-		process.env.CLINE_DATA_DIR = dataDir;
-
-		const first = tryAcquireRpcSpawnLease("127.0.0.1:4317");
-		const second = tryAcquireRpcSpawnLease("127.0.0.1:4318");
-
-		expect(first).toBeDefined();
-		expect(second).toBeDefined();
-
-		first?.release();
-		second?.release();
-	});
-
-	it("reclaims a leftover lease owned by the current process", () => {
-		const dataDir = mkdtempSync(path.join(os.tmpdir(), "rpc-spawn-lease-"));
-		tempDirs.push(dataDir);
-		process.env.CLINE_DATA_DIR = dataDir;
-
-		const first = tryAcquireRpcSpawnLease("127.0.0.1:4317");
-		expect(first).toBeDefined();
-		const leasePath = first?.path;
-		expect(leasePath).toBeTruthy();
-		first?.release();
-
-		writeFileSync(
-			leasePath!,
-			JSON.stringify({
-				address: "127.0.0.1:4317",
-				pid: process.pid,
-				createdAt: Date.now(),
-			}),
-			"utf8",
-		);
-
-		const second = tryAcquireRpcSpawnLease("127.0.0.1:4317");
-		assert(second);
-		expect(
-			JSON.parse(readFileSync(second.path, "utf8")) as { pid: number },
-		).toMatchObject({
-			pid: process.pid,
-		});
-		second.release();
-	});
-});

package/src/runtime/rpc-spawn-lease.ts

@@ -1,156 +0,0 @@
-import {
-	closeSync,
-	existsSync,
-	mkdirSync,
-	openSync,
-	readFileSync,
-	rmSync,
-	writeFileSync,
-} from "node:fs";
-import { dirname, resolve } from "node:path";
-import { resolveSessionDataDir } from "@clinebot/shared/storage";
-
-const DEFAULT_LEASE_TTL_MS = 5_000;
-
-interface RpcSpawnLeaseRecord {
-	address: string;
-	pid: number;
-	createdAt: number;
-}
-
-export interface RpcSpawnLease {
-	path: string;
-	release: () => void;
-}
-
-const ACTIVE_LEASE_RELEASES = new Set<() => void>();
-const ACTIVE_LEASE_PATHS = new Set<string>();
-let EXIT_CLEANUP_REGISTERED = false;
-
-function encodeAddress(address: string): string {
-	return Buffer.from(address).toString("base64url");
-}
-
-function getLeasePath(address: string): string {
-	return resolve(
-		resolveSessionDataDir(),
-		"rpc",
-		"spawn-leases",
-		`${encodeAddress(address)}.lock`,
-	);
-}
-
-function isProcessAlive(pid: number): boolean {
-	if (!Number.isInteger(pid) || pid <= 0) {
-		return false;
-	}
-	try {
-		process.kill(pid, 0);
-		return true;
-	} catch {
-		return false;
-	}
-}
-
-function shouldClearLease(path: string, ttlMs: number): boolean {
-	try {
-		const raw = readFileSync(path, "utf8");
-		const parsed = JSON.parse(raw) as Partial<RpcSpawnLeaseRecord>;
-		const pid = Number(parsed.pid ?? 0);
-		const createdAt = Number(parsed.createdAt ?? 0);
-		if (pid === process.pid && !ACTIVE_LEASE_PATHS.has(path)) {
-			return true;
-		}
-		if (!Number.isFinite(createdAt) || createdAt <= 0) {
-			return true;
-		}
-		if (Date.now() - createdAt > ttlMs) {
-			return true;
-		}
-		return !isProcessAlive(pid);
-	} catch {
-		return true;
-	}
-}
-
-function cleanupActiveLeases(): void {
-	for (const release of [...ACTIVE_LEASE_RELEASES]) {
-		try {
-			release();
-		} catch {
-			// Best effort.
-		}
-	}
-}
-
-function registerExitCleanupOnce(): void {
-	if (EXIT_CLEANUP_REGISTERED) {
-		return;
-	}
-	EXIT_CLEANUP_REGISTERED = true;
-	process.once("exit", cleanupActiveLeases);
-	process.once("SIGINT", cleanupActiveLeases);
-	process.once("SIGTERM", cleanupActiveLeases);
-}
-
-export function tryAcquireRpcSpawnLease(
-	address: string,
-	options?: { ttlMs?: number },
-): RpcSpawnLease | undefined {
-	const ttlMs = Math.max(1_000, options?.ttlMs ?? DEFAULT_LEASE_TTL_MS);
-	const path = getLeasePath(address);
-	mkdirSync(dirname(path), { recursive: true });
-
-	if (existsSync(path) && shouldClearLease(path, ttlMs)) {
-		rmSync(path, { force: true });
-	}
-
-	let fd: number | undefined;
-	try {
-		fd = openSync(path, "wx");
-		const record: RpcSpawnLeaseRecord = {
-			address,
-			pid: process.pid,
-			createdAt: Date.now(),
-		};
-		writeFileSync(fd, JSON.stringify(record), "utf8");
-	} catch {
-		if (typeof fd === "number") {
-			try {
-				closeSync(fd);
-			} catch {
-				// Best effort.
-			}
-		}
-		return undefined;
-	}
-
-	let released = false;
-	const release = () => {
-		if (released) {
-			return;
-		}
-		released = true;
-		ACTIVE_LEASE_RELEASES.delete(release);
-		ACTIVE_LEASE_PATHS.delete(path);
-		try {
-			if (typeof fd === "number") {
-				closeSync(fd);
-			}
-		} catch {
-			// Best effort.
-		}
-		try {
-			rmSync(path, { force: true });
-		} catch {
-			// Best effort.
-		}
-	};
-	registerExitCleanupOnce();
-	ACTIVE_LEASE_RELEASES.add(release);
-	ACTIVE_LEASE_PATHS.add(path);
-	return {
-		path,
-		release,
-	};
-}

package/src/runtime/runtime-builder.team-persistence.test.ts

@@ -1,245 +0,0 @@
-import { describe, expect, it, vi } from "vitest";
-
-const createBuiltinToolsMock = vi.fn(() => []);
-const bootstrapAgentTeamsMock = vi.fn(() => ({
-	tools: [],
-	restoredFromPersistence: true,
-	restoredTeammates: ["restored-1"],
-}));
-
-let runtimeInstance: MockAgentTeamsRuntime | undefined;
-class MockAgentTeamsRuntime {
-	private readonly onTeamEvent?: (event: any) => void;
-
-	constructor(options: { onTeamEvent?: (event: any) => void }) {
-		this.onTeamEvent = options.onTeamEvent;
-		runtimeInstance = this;
-	}
-
-	emit(event: any): void {
-		this.onTeamEvent?.(event);
-	}
-
-	hydrateState = vi.fn();
-	exportState = vi.fn(() => ({
-		members: [],
-		tasks: [],
-		mailbox: [],
-		missionLog: [],
-		runs: [],
-		outcomes: [],
-		outcomeFragments: [],
-	}));
-	markStaleRunsInterrupted = vi.fn();
-	getTeammateIds = vi.fn(() => []);
-	shutdownTeammate = vi.fn();
-}
-
-vi.mock("../extensions/tools/team", () => ({
-	AgentTeamsRuntime: MockAgentTeamsRuntime,
-	bootstrapAgentTeams: bootstrapAgentTeamsMock,
-	createDelegatedAgentConfigProvider: (config: Record<string, unknown>) => {
-		let runtimeConfig = { ...config };
-		return {
-			getRuntimeConfig: () => runtimeConfig,
-			getConnectionConfig: () => ({
-				providerId: runtimeConfig.providerId,
-				modelId: runtimeConfig.modelId,
-				apiKey: runtimeConfig.apiKey,
-				baseUrl: runtimeConfig.baseUrl,
-				headers: runtimeConfig.headers,
-				providerConfig: runtimeConfig.providerConfig,
-				knownModels: runtimeConfig.knownModels,
-				thinking: runtimeConfig.thinking,
-			}),
-			updateConnectionDefaults: (overrides: Record<string, unknown>) => {
-				runtimeConfig = { ...runtimeConfig, ...overrides };
-			},
-		};
-	},
-}));
-
-vi.mock("../extensions/tools", () => ({
-	ALL_DEFAULT_TOOL_NAMES: [],
-	createBuiltinTools: createBuiltinToolsMock,
-	ToolPresets: {
-		development: {},
-		readonly: {},
-	},
-	resolveToolPresetName: () => "development",
-	resolveToolRoutingConfig: () => [],
-	DEFAULT_MODEL_TOOL_ROUTING_RULES: [],
-}));
-
-let teamStoreInstance: MockTeamStore | undefined;
-class MockTeamStore {
-	constructor() {
-		teamStoreInstance = this;
-	}
-
-	loadRuntime = vi.fn(() => ({
-		state: {
-			teamId: "team_1",
-			teamName: "test",
-			members: [],
-			tasks: [],
-			mailbox: [],
-			missionLog: [],
-			runs: [],
-			outcomes: [],
-			outcomeFragments: [],
-		},
-		teammates: [
-			{
-				agentId: "restored-1",
-				rolePrompt: "Persisted teammate",
-				modelId: "claude-sonnet-4-5-20250929",
-				maxIterations: 4,
-			},
-		],
-		interruptedRunIds: ["run_00001"],
-	}));
-	handleTeamEvent = vi.fn();
-	persistRuntime = vi.fn();
-}
-
-vi.mock("../services/storage/team-store", () => ({
-	createLocalTeamStore: () => new MockTeamStore(),
-}));
-
-describe("DefaultRuntimeBuilder team persistence boundary", () => {
-	it("persists teammate specs and runtime state from team events", async () => {
-		const { DefaultRuntimeBuilder } = await import("./runtime-builder");
-		const onTeamRestored = vi.fn();
-
-		await new DefaultRuntimeBuilder().build({
-			config: {
-				providerId: "anthropic",
-				modelId: "claude-sonnet-4-6",
-				apiKey: "key",
-				headers: {
-					Authorization: "Bearer team-token",
-				},
-				systemPrompt: "test",
-				cwd: process.cwd(),
-				enableTools: false,
-				enableSpawnAgent: false,
-				enableAgentTeams: true,
-			},
-			onTeamRestored,
-		});
-
-		expect(bootstrapAgentTeamsMock).toHaveBeenCalledWith(
-			expect.objectContaining({
-				restoredFromPersistence: true,
-				restoredTeammates: [expect.objectContaining({ agentId: "restored-1" })],
-				teammateConfigProvider: expect.objectContaining({
-					getRuntimeConfig: expect.any(Function),
-				}),
-			}),
-		);
-		const bootstrapCall = (
-			bootstrapAgentTeamsMock.mock.calls as unknown as Array<
-				[Record<string, any>]
-			>
-		)[0]?.[0];
-		expect(bootstrapCall).toBeDefined();
-		expect(bootstrapCall?.teammateConfigProvider.getRuntimeConfig()).toEqual(
-			expect.objectContaining({
-				headers: {
-					Authorization: "Bearer team-token",
-				},
-			}),
-		);
-		expect(onTeamRestored).toHaveBeenCalledTimes(1);
-		expect(runtimeInstance).toBeDefined();
-		expect(teamStoreInstance).toBeDefined();
-		if (!runtimeInstance || !teamStoreInstance) {
-			throw new Error("Expected mocked runtime and team store instances");
-		}
-
-		expect(runtimeInstance.markStaleRunsInterrupted).toHaveBeenCalledWith(
-			"runtime_recovered",
-		);
-
-		runtimeInstance.emit({
-			type: "teammate_spawned",
-			agentId: "python-poet",
-			teammate: {
-				rolePrompt: "Write concise Python-focused haiku",
-				modelId: "claude-sonnet-4-5-20250929",
-				maxIterations: 7,
-			},
-		});
-		expect(teamStoreInstance.handleTeamEvent).toHaveBeenCalledWith(
-			expect.any(String),
-			expect.objectContaining({
-				type: "teammate_spawned",
-				agentId: "python-poet",
-			}),
-		);
-		expect(teamStoreInstance.persistRuntime).toHaveBeenCalled();
-
-		runtimeInstance.emit({
-			type: "teammate_shutdown",
-			agentId: "python-poet",
-		});
-		expect(teamStoreInstance.handleTeamEvent).toHaveBeenCalledWith(
-			expect.any(String),
-			expect.objectContaining({
-				type: "teammate_shutdown",
-				agentId: "python-poet",
-			}),
-		);
-	});
-
-	it("forwards cline workspace metadata to teammate runtime bootstrap config", async () => {
-		const { DefaultRuntimeBuilder } = await import("./runtime-builder");
-		bootstrapAgentTeamsMock.mockClear();
-
-		await new DefaultRuntimeBuilder().build({
-			config: {
-				providerId: "cline",
-				modelId: "anthropic/claude-sonnet-4.6",
-				apiKey: "key",
-				systemPrompt: `Base instructions.
-
-# Workspace Configuration
-{
-  "workspaces": {
-    "/repo/demo": {
-      "hint": "demo",
-      "latestGitBranchName": "main"
-    }
-  }
-}`,
-				cwd: "/repo/demo",
-				enableTools: false,
-				enableSpawnAgent: false,
-				enableAgentTeams: true,
-			},
-		});
-
-		expect(bootstrapAgentTeamsMock).toHaveBeenCalledWith(
-			expect.objectContaining({
-				teammateConfigProvider: expect.objectContaining({
-					getRuntimeConfig: expect.any(Function),
-				}),
-			}),
-		);
-		const clineBootstrapCall = (
-			bootstrapAgentTeamsMock.mock.calls as unknown as Array<
-				[Record<string, any>]
-			>
-		)[0]?.[0];
-		expect(clineBootstrapCall).toBeDefined();
-		expect(
-			clineBootstrapCall?.teammateConfigProvider.getRuntimeConfig(),
-		).toEqual(
-			expect.objectContaining({
-				providerId: "cline",
-				cwd: "/repo/demo",
-			}),
-		);
-	});
-});