mirror of
https://github.com/MoonshotAI/kimi-code.git
synced 2026-08-22 07:05:41 +00:00
Some checks are pending
CI / build (push) Waiting to run
CI / test (1) (push) Waiting to run
CI / test (2) (push) Waiting to run
CI / test (3) (push) Waiting to run
CI / test (4) (push) Waiting to run
CI / test (5) (push) Waiting to run
CI / test-pi-tui (push) Waiting to run
CI / test-vscode-legacy (push) Waiting to run
CI / test-windows (push) Waiting to run
CI / lint (push) Waiting to run
CI / typecheck (push) Waiting to run
Nix Build / Check flake.nix workspace sync (push) Waiting to run
Nix Build / nix build .#kimi-code (push) Blocked by required conditions
Release / Publish native release assets (push) Blocked by required conditions
Release / Release (push) Waiting to run
Release / Deploy docs (push) Blocked by required conditions
Release / Native release artifact (push) Blocked by required conditions
The extension now runs on the agent-core-v2 engine by default. The interface, sessions, and workflows do not change. Two rollback paths exist, and one function makes the decision (config/vscode-settings.ts): - the kimi.useAgentCoreV1 setting (temporary; a window reload applies the change); - the KIMI_CODE_LEGACY_FLAG environment variable, which wins over the setting and has the same semantics as in the CLI. An engine startup failure shows an explicit error that names the rollback setting. There is no silent fallback. CI runs the extension test suite on both engines: the sharded run covers the default v2 engine, and a new test-vscode-legacy job reruns the suite with KIMI_CODE_LEGACY_FLAG=1. To keep the v2 path identical to v1 for every method the extension uses, this change also completes the v2-backed SDK client and the v2 engine: - Implement session deletion in the v2 SDK client. - Implement fork truncation at a turn index in the v2 engine, with the same rules as v1, and reject a fork while the source session has an active turn. - Stop the session-level /init run when the turn is cancelled, as v1 does. - Read session metadata without the archived field as not-archived, so sessions written by the v1 engine open correctly. The SDK parity suite now covers session deletion, cancel, and fork truncation. The known-difference list for the methods the extension uses is empty.
739 lines
26 KiB
TypeScript
739 lines
26 KiB
TypeScript
/**
|
|
* Scenario: the VS Code host owns one Kimi harness and routes Webviews to shared SDK sessions.
|
|
* Responsibilities: create/resume/switch, per-session settings, multi-view ownership, detach, and disposal.
|
|
* Wiring: KimiRuntime and SessionRuntime are real; in-memory Session/KimiHarness fakes form the public SDK boundary.
|
|
* Run: pnpm exec vitest run --config apps/vscode/vitest.config.ts test/kimi-runtime.test.ts
|
|
*/
|
|
|
|
import type {
|
|
ApprovalHandler,
|
|
CreateSessionOptions,
|
|
Event,
|
|
JsonObject,
|
|
KimiHarness,
|
|
PermissionMode,
|
|
PromptInput,
|
|
QuestionHandler,
|
|
ResumeSessionInput,
|
|
Session,
|
|
SessionStatus,
|
|
SessionSummary,
|
|
ThinkingEffort,
|
|
} from "@moonshot-ai/kimi-code-sdk";
|
|
import { describe, expect, it, vi } from "vitest";
|
|
|
|
import { Events } from "../shared/bridge";
|
|
import { KimiRuntime, type OpenSessionOptions } from "../src/runtime/kimi-runtime";
|
|
|
|
const sdkFactories = vi.hoisted(() => {
|
|
const v1Harness = { homeDir: "/tmp/kimi-runtime-v1-home", close: vi.fn(async () => undefined) };
|
|
const v2Harness = { homeDir: "/tmp/kimi-runtime-v2-home", close: vi.fn(async () => undefined) };
|
|
return {
|
|
v1Harness,
|
|
v2Harness,
|
|
createKimiHarness: vi.fn(() => v1Harness),
|
|
createKimiHarnessV2: vi.fn(() => v2Harness),
|
|
};
|
|
});
|
|
|
|
vi.mock("@moonshot-ai/kimi-code-sdk", async (importOriginal) => {
|
|
const original = await importOriginal<typeof import("@moonshot-ai/kimi-code-sdk")>();
|
|
return {
|
|
...original,
|
|
createKimiHarness: sdkFactories.createKimiHarness,
|
|
createKimiHarnessV2: sdkFactories.createKimiHarnessV2,
|
|
};
|
|
});
|
|
|
|
interface FakeSessionBoundary {
|
|
readonly session: Session;
|
|
readonly setModels: string[];
|
|
readonly setThinkingEfforts: ThinkingEffort[];
|
|
readonly setPermissions: PermissionMode[];
|
|
readonly metadataUpdates: JsonObject[];
|
|
readonly handlerInstallations: { approval: number; question: number };
|
|
readonly subscriptionCount: () => number;
|
|
readonly closeCount: () => number;
|
|
readonly emit: (event: Event) => void;
|
|
readonly setPromptImpl: (impl: (input: string | PromptInput) => Promise<void>) => void;
|
|
}
|
|
|
|
function createFakeSession(
|
|
id: string,
|
|
workDir: string,
|
|
initial: Partial<SessionStatus> = {},
|
|
metadata?: JsonObject,
|
|
): FakeSessionBoundary {
|
|
const listeners = new Set<(event: Event) => void>();
|
|
const setModels: string[] = [];
|
|
const setThinkingEfforts: ThinkingEffort[] = [];
|
|
const setPermissions: PermissionMode[] = [];
|
|
const metadataUpdates: JsonObject[] = [];
|
|
const handlerInstallations = { approval: 0, question: 0 };
|
|
let subscriptions = 0;
|
|
let closes = 0;
|
|
let promptImpl: (input: string | PromptInput) => Promise<void> = async () => {};
|
|
let status: SessionStatus = {
|
|
model: initial.model ?? "kimi-test",
|
|
thinkingEffort: initial.thinkingEffort ?? "off",
|
|
permission: initial.permission ?? "manual",
|
|
planMode: initial.planMode ?? false,
|
|
contextTokens: 0,
|
|
maxContextTokens: 128_000,
|
|
contextUsage: 0,
|
|
};
|
|
let summary: SessionSummary = {
|
|
id,
|
|
workDir,
|
|
sessionDir: `/home/sessions/${id}`,
|
|
createdAt: 1,
|
|
updatedAt: 2,
|
|
metadata,
|
|
};
|
|
|
|
const session = {
|
|
id,
|
|
workDir,
|
|
get summary() {
|
|
return summary;
|
|
},
|
|
set summary(value: SessionSummary | undefined) {
|
|
if (value !== undefined) summary = value;
|
|
},
|
|
setApprovalHandler(handler: ApprovalHandler | undefined) {
|
|
if (handler !== undefined) handlerInstallations.approval += 1;
|
|
},
|
|
setQuestionHandler(handler: QuestionHandler | undefined) {
|
|
if (handler !== undefined) handlerInstallations.question += 1;
|
|
},
|
|
onEvent(listener: (event: Event) => void) {
|
|
subscriptions += 1;
|
|
listeners.add(listener);
|
|
return () => listeners.delete(listener);
|
|
},
|
|
async prompt(input: string | PromptInput) {
|
|
await promptImpl(input);
|
|
},
|
|
async steer(_input: string | PromptInput) {},
|
|
async cancel() {},
|
|
async getStatus() {
|
|
return status;
|
|
},
|
|
async setModel(model: string) {
|
|
setModels.push(model);
|
|
status = { ...status, model };
|
|
},
|
|
async setThinking(effort: ThinkingEffort) {
|
|
setThinkingEfforts.push(effort);
|
|
status = { ...status, thinkingEffort: effort };
|
|
},
|
|
async setPermission(permission: PermissionMode) {
|
|
setPermissions.push(permission);
|
|
status = { ...status, permission };
|
|
},
|
|
async updateMetadata(patch: JsonObject) {
|
|
metadataUpdates.push(patch);
|
|
summary = { ...summary, metadata: { ...summary.metadata, ...patch } };
|
|
},
|
|
async close() {
|
|
closes += 1;
|
|
},
|
|
} as unknown as Session;
|
|
|
|
return {
|
|
session,
|
|
setModels,
|
|
setThinkingEfforts,
|
|
setPermissions,
|
|
metadataUpdates,
|
|
handlerInstallations,
|
|
subscriptionCount: () => subscriptions,
|
|
closeCount: () => closes,
|
|
emit: (event: Event) => {
|
|
for (const listener of [...listeners]) listener(event);
|
|
},
|
|
setPromptImpl: (impl) => {
|
|
promptImpl = impl;
|
|
},
|
|
};
|
|
}
|
|
|
|
interface FakeHarnessBoundary {
|
|
readonly harness: KimiHarness;
|
|
readonly createInputs: CreateSessionOptions[];
|
|
readonly resumeInputs: ResumeSessionInput[];
|
|
readonly closeSessionIds: string[];
|
|
readonly deleteSessionIds: string[];
|
|
readonly closeCount: () => number;
|
|
readonly sessions: Map<string, FakeSessionBoundary>;
|
|
addSession(
|
|
id: string,
|
|
workDir: string,
|
|
initial?: Partial<SessionStatus>,
|
|
metadata?: JsonObject,
|
|
): FakeSessionBoundary;
|
|
}
|
|
|
|
function createFakeHarness(
|
|
normalizeCreatedWorkDir: (workDir: string) => string = (workDir) => workDir,
|
|
): FakeHarnessBoundary {
|
|
const sessions = new Map<string, FakeSessionBoundary>();
|
|
const createInputs: CreateSessionOptions[] = [];
|
|
const resumeInputs: ResumeSessionInput[] = [];
|
|
const closeSessionIds: string[] = [];
|
|
const deleteSessionIds: string[] = [];
|
|
let creates = 0;
|
|
let closes = 0;
|
|
|
|
const addSession = (
|
|
id: string,
|
|
workDir: string,
|
|
initial?: Partial<SessionStatus>,
|
|
metadata?: JsonObject,
|
|
) => {
|
|
const boundary = createFakeSession(id, workDir, initial, metadata);
|
|
sessions.set(id, boundary);
|
|
return boundary;
|
|
};
|
|
|
|
const harness = {
|
|
async createSession(options: CreateSessionOptions) {
|
|
createInputs.push(options);
|
|
creates += 1;
|
|
return addSession(
|
|
`created-${creates}`,
|
|
normalizeCreatedWorkDir(options.workDir),
|
|
{
|
|
model: options.model,
|
|
thinkingEffort: options.thinking,
|
|
permission: options.permission,
|
|
},
|
|
options.metadata,
|
|
).session;
|
|
},
|
|
async resumeSession(input: ResumeSessionInput) {
|
|
resumeInputs.push(input);
|
|
const boundary = sessions.get(input.id);
|
|
if (boundary === undefined) throw new Error(`Unknown session: ${input.id}`);
|
|
return boundary.session;
|
|
},
|
|
async closeSession(id: string) {
|
|
closeSessionIds.push(id);
|
|
},
|
|
async deleteSession(id: string) {
|
|
deleteSessionIds.push(id);
|
|
},
|
|
async close() {
|
|
closes += 1;
|
|
},
|
|
} as unknown as KimiHarness;
|
|
|
|
return {
|
|
harness,
|
|
createInputs,
|
|
resumeInputs,
|
|
closeSessionIds,
|
|
deleteSessionIds,
|
|
closeCount: () => closes,
|
|
sessions,
|
|
addSession,
|
|
};
|
|
}
|
|
|
|
function openOptions(overrides: Partial<OpenSessionOptions> = {}): OpenSessionOptions {
|
|
return {
|
|
webviewId: "view-1",
|
|
workDir: "/workspace",
|
|
model: "kimi-test",
|
|
effort: "off",
|
|
yoloMode: false,
|
|
...overrides,
|
|
};
|
|
}
|
|
|
|
function createRuntime(
|
|
normalizeCreatedWorkDir?: (workDir: string) => string,
|
|
) {
|
|
const sdk = createFakeHarness(normalizeCreatedWorkDir);
|
|
const runtime = new KimiRuntime({
|
|
version: "0.6.0",
|
|
harness: sdk.harness,
|
|
broadcast: () => undefined,
|
|
captureBaseline: () => undefined,
|
|
log: () => undefined,
|
|
});
|
|
return { runtime, sdk };
|
|
}
|
|
|
|
describe("Kimi runtime (owns shared SDK sessions for Webviews)", () => {
|
|
it("creates the v2 harness by default and the v1 harness for rollback", async () => {
|
|
const defaults = new KimiRuntime({
|
|
version: "0.6.0",
|
|
broadcast: () => undefined,
|
|
captureBaseline: () => undefined,
|
|
log: () => undefined,
|
|
});
|
|
expect(sdkFactories.createKimiHarnessV2).toHaveBeenCalledOnce();
|
|
expect(sdkFactories.createKimiHarnessV2).toHaveBeenCalledWith({
|
|
homeDir: undefined,
|
|
identity: {
|
|
productName: "kimi-code-vscode",
|
|
version: "0.6.0",
|
|
platform: "kimi_code_vscode",
|
|
},
|
|
uiMode: "vscode",
|
|
});
|
|
expect(sdkFactories.createKimiHarness).not.toHaveBeenCalled();
|
|
expect(defaults.harness).toBe(sdkFactories.v2Harness as unknown as KimiHarness);
|
|
await defaults.dispose();
|
|
|
|
const rollback = new KimiRuntime({
|
|
version: "0.6.0",
|
|
useAgentCoreV1: true,
|
|
broadcast: () => undefined,
|
|
captureBaseline: () => undefined,
|
|
log: () => undefined,
|
|
});
|
|
expect(sdkFactories.createKimiHarness).toHaveBeenCalledOnce();
|
|
expect(rollback.harness).toBe(sdkFactories.v1Harness as unknown as KimiHarness);
|
|
await rollback.dispose();
|
|
});
|
|
|
|
it("forwards the requested settings when creating an SDK session", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
|
|
const opened = await runtime.openSession(
|
|
openOptions({ model: "kimi-k2", effort: "high", yoloMode: true }),
|
|
);
|
|
|
|
expect(sdk.createInputs).toEqual([
|
|
{
|
|
workDir: "/workspace",
|
|
model: "kimi-k2",
|
|
thinking: "high",
|
|
permission: "yolo",
|
|
metadata: { vscode_legacy_approval: { yolo: true, afk: false } },
|
|
},
|
|
]);
|
|
expect(opened.subscribers).toEqual(["view-1"]);
|
|
});
|
|
|
|
it("accepts the normalized SDK workDir when a Windows session is created", async () => {
|
|
const { runtime } = createRuntime((workDir) => workDir.replaceAll("\\", "/"));
|
|
|
|
const opened = await runtime.openSession(openOptions({
|
|
workDir: "C:\\Users\\Example User\\项目",
|
|
}));
|
|
|
|
expect(opened.session.workDir).toBe("C:/Users/Example User/项目");
|
|
});
|
|
|
|
it("resumes a Windows session when only separators and casing differ", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
sdk.addSession("saved-win", "C:/Users/Example User/项目");
|
|
|
|
const opened = await runtime.openSession(openOptions({
|
|
sessionId: "saved-win",
|
|
workDir: "c:\\users\\example user\\项目",
|
|
}));
|
|
|
|
expect(opened.id).toBe("saved-win");
|
|
});
|
|
|
|
it("uses off thinking when a new session receives an empty effort", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
|
|
await runtime.openSession(openOptions({ effort: " " }));
|
|
|
|
expect(sdk.createInputs[0]?.thinking).toBe("off");
|
|
});
|
|
|
|
it("resumes the requested SDK session instead of creating a replacement", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
sdk.addSession("saved-1", "/workspace");
|
|
|
|
const opened = await runtime.openSession(openOptions({ sessionId: "saved-1" }));
|
|
|
|
expect(opened.id).toBe("saved-1");
|
|
expect(sdk.resumeInputs).toEqual([{ id: "saved-1", includeSubagents: true }]);
|
|
expect(sdk.createInputs).toEqual([]);
|
|
});
|
|
|
|
it("switches a Webview to the requested session", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
await runtime.openSession(openOptions());
|
|
sdk.addSession("saved-2", "/workspace");
|
|
|
|
const selected = await runtime.openSession(openOptions({ sessionId: "saved-2" }));
|
|
|
|
expect(runtime.getSessionForView("view-1")?.id).toBe("saved-2");
|
|
expect(selected.id).toBe("saved-2");
|
|
});
|
|
|
|
it("closes an unshared old session when its Webview switches away", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const old = await runtime.openSession(openOptions());
|
|
const oldBoundary = sdk.sessions.get(old.id)!;
|
|
sdk.addSession("saved-2", "/workspace");
|
|
|
|
await runtime.openSession(openOptions({ sessionId: "saved-2" }));
|
|
|
|
expect(oldBoundary.closeCount()).toBe(1);
|
|
});
|
|
|
|
it("shares one SDK subscription when two Webviews attach to the same session", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const first = await runtime.openSession(openOptions({ webviewId: "view-1" }));
|
|
const boundary = sdk.sessions.get(first.id)!;
|
|
|
|
const second = await runtime.openSession(
|
|
openOptions({ webviewId: "view-2", sessionId: first.id }),
|
|
);
|
|
|
|
expect(second).toBe(first);
|
|
expect(first.subscribers).toEqual(["view-1", "view-2"]);
|
|
expect(boundary.subscriptionCount()).toBe(1);
|
|
expect(boundary.handlerInstallations).toEqual({ approval: 1, question: 1 });
|
|
});
|
|
|
|
it("preserves the resumed session's model instead of reapplying the configured default", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const session = sdk.addSession("saved-1", "/workspace", { model: "old-model" });
|
|
|
|
const opened = await runtime.openSession(openOptions({ sessionId: "saved-1", model: "new-model" }));
|
|
|
|
expect(session.setModels).toEqual([]);
|
|
await expect(opened.session.getStatus()).resolves.toMatchObject({ model: "old-model" });
|
|
});
|
|
|
|
it("preserves the resumed session's thinking effort instead of reapplying the configured default", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const session = sdk.addSession("saved-1", "/workspace", { thinkingEffort: "max" });
|
|
|
|
const opened = await runtime.openSession(openOptions({ sessionId: "saved-1", effort: "medium" }));
|
|
|
|
expect(session.setThinkingEfforts).toEqual([]);
|
|
await expect(opened.session.getStatus()).resolves.toMatchObject({ thinkingEffort: "max" });
|
|
});
|
|
|
|
it("announces the session's actual status to the attaching view so the display matches it", async () => {
|
|
const sdk = createFakeHarness();
|
|
const broadcasts: { event: string; data: unknown; webviewId?: string }[] = [];
|
|
const runtime = new KimiRuntime({
|
|
version: "0.6.0",
|
|
harness: sdk.harness,
|
|
broadcast: (event, data, webviewId) => {
|
|
broadcasts.push({ event, data, webviewId });
|
|
},
|
|
captureBaseline: () => undefined,
|
|
log: () => undefined,
|
|
});
|
|
sdk.addSession("saved-1", "/workspace", {
|
|
model: "kimi-test",
|
|
thinkingEffort: "max",
|
|
planMode: true,
|
|
});
|
|
|
|
await runtime.openSession(openOptions({ sessionId: "saved-1", effort: "medium" }));
|
|
|
|
expect(broadcasts).toContainEqual({
|
|
event: Events.StreamEvent,
|
|
data: {
|
|
type: "StatusUpdate",
|
|
payload: { model: "kimi-test", thinking_effort: "max", plan_mode: true },
|
|
_sessionId: "saved-1",
|
|
},
|
|
webviewId: "view-1",
|
|
});
|
|
});
|
|
|
|
it("uses the yolo setting as the initial value for an unmarked resumed session", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const session = sdk.addSession("saved-1", "/workspace", { permission: "manual" });
|
|
|
|
await runtime.openSession(openOptions({ sessionId: "saved-1", yoloMode: true }));
|
|
|
|
expect(session.metadataUpdates).toEqual([
|
|
{ vscode_legacy_approval: { yolo: true, afk: false } },
|
|
]);
|
|
});
|
|
|
|
it("lets the global yolo setting override a persisted off flag on resume", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const session = sdk.addSession(
|
|
"saved-1",
|
|
"/workspace",
|
|
{ permission: "manual" },
|
|
{ vscode_legacy_approval: { yolo: false, afk: false } },
|
|
);
|
|
|
|
const opened = await runtime.openSession(openOptions({ sessionId: "saved-1", yoloMode: true }));
|
|
|
|
expect(session.setPermissions).toEqual(["yolo"]);
|
|
expect(session.metadataUpdates).toEqual([
|
|
{ vscode_legacy_approval: { yolo: true, afk: false } },
|
|
]);
|
|
expect(opened.legacyApprovalFlags).toEqual({ yolo: true, afk: false });
|
|
});
|
|
|
|
it("lets the global yolo setting disable a persisted session yolo flag on resume", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const session = sdk.addSession(
|
|
"saved-1",
|
|
"/workspace",
|
|
{ permission: "yolo" },
|
|
{ vscode_legacy_approval: { yolo: true, afk: false } },
|
|
);
|
|
|
|
const opened = await runtime.openSession(openOptions({ sessionId: "saved-1", yoloMode: false }));
|
|
|
|
expect(session.setPermissions).toEqual(["manual"]);
|
|
expect(session.metadataUpdates).toEqual([
|
|
{ vscode_legacy_approval: { yolo: false, afk: false } },
|
|
]);
|
|
expect(opened.legacyApprovalFlags).toEqual({ yolo: false, afk: false });
|
|
});
|
|
|
|
it("keeps the persisted afk flag while applying the global yolo setting on resume", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const session = sdk.addSession(
|
|
"saved-1",
|
|
"/workspace",
|
|
{ permission: "manual" },
|
|
{ vscode_legacy_approval: { yolo: false, afk: true } },
|
|
);
|
|
|
|
const opened = await runtime.openSession(openOptions({ sessionId: "saved-1", yoloMode: true }));
|
|
|
|
expect(session.setPermissions).toEqual(["auto"]);
|
|
expect(opened.legacyApprovalFlags).toEqual({ yolo: true, afk: true });
|
|
});
|
|
|
|
it("restores persisted afk with core auto permission", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const session = sdk.addSession(
|
|
"saved-1",
|
|
"/workspace",
|
|
{ permission: "manual" },
|
|
{ vscode_legacy_approval: { yolo: false, afk: true } },
|
|
);
|
|
|
|
await runtime.openSession(openOptions({ sessionId: "saved-1", yoloMode: false }));
|
|
|
|
expect(session.setPermissions).toEqual(["auto"]);
|
|
});
|
|
|
|
it("changes the setting-backed yolo flag without clearing session afk", async () => {
|
|
const { runtime } = createRuntime();
|
|
const opened = await runtime.openSession(openOptions());
|
|
await opened.toggleLegacyApproval("afk");
|
|
|
|
await runtime.setYoloModeForActiveSessions(true);
|
|
|
|
expect(opened.legacyApprovalFlags).toEqual({ yolo: true, afk: true });
|
|
await expect(opened.session.getStatus()).resolves.toMatchObject({ permission: "auto" });
|
|
});
|
|
|
|
it("keeps a shared session open when one of its Webviews detaches", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const opened = await runtime.openSession(openOptions({ webviewId: "view-1" }));
|
|
await runtime.openSession(openOptions({ webviewId: "view-2", sessionId: opened.id }));
|
|
const boundary = sdk.sessions.get(opened.id)!;
|
|
|
|
await runtime.detachView("view-1");
|
|
|
|
expect(boundary.closeCount()).toBe(0);
|
|
expect(runtime.getSessionForView("view-2")?.id).toBe(opened.id);
|
|
});
|
|
|
|
it("closes an SDK session when its last Webview detaches", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const opened = await runtime.openSession(openOptions());
|
|
const boundary = sdk.sessions.get(opened.id)!;
|
|
|
|
await runtime.detachView("view-1");
|
|
|
|
expect(boundary.closeCount()).toBe(1);
|
|
expect(runtime.getSession(opened.id)).toBeUndefined();
|
|
});
|
|
|
|
it("reattaches the same resumed session without replacing its handlers", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const boundary = sdk.addSession("saved-1", "/workspace");
|
|
const first = await runtime.attachResumedSession("view-1", boundary.session);
|
|
|
|
const second = await runtime.attachResumedSession("view-1", boundary.session);
|
|
|
|
expect(second).toBe(first);
|
|
expect(boundary.subscriptionCount()).toBe(1);
|
|
expect(boundary.handlerInstallations).toEqual({ approval: 1, question: 1 });
|
|
expect(boundary.closeCount()).toBe(0);
|
|
});
|
|
|
|
it("removes every Webview mapping when a shared session is closed", async () => {
|
|
const { runtime } = createRuntime();
|
|
const opened = await runtime.openSession(openOptions({ webviewId: "view-1" }));
|
|
await runtime.openSession(openOptions({ webviewId: "view-2", sessionId: opened.id }));
|
|
|
|
await runtime.closeSession(opened.id);
|
|
|
|
expect(runtime.getSessionForView("view-1")).toBeUndefined();
|
|
expect(runtime.getSessionForView("view-2")).toBeUndefined();
|
|
});
|
|
|
|
it("delegates deletion after the active session has been closed", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const opened = await runtime.openSession(openOptions());
|
|
const boundary = sdk.sessions.get(opened.id)!;
|
|
|
|
await runtime.deleteSession(opened.id);
|
|
|
|
expect(boundary.closeCount()).toBe(1);
|
|
expect(sdk.deleteSessionIds).toEqual([opened.id]);
|
|
});
|
|
|
|
it("closes every active SDK session when the host runtime is disposed", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const first = await runtime.openSession(openOptions({ webviewId: "view-1" }));
|
|
const secondBoundary = sdk.addSession("saved-2", "/workspace");
|
|
await runtime.openSession(openOptions({ webviewId: "view-2", sessionId: "saved-2" }));
|
|
|
|
await runtime.dispose();
|
|
|
|
expect(sdk.sessions.get(first.id)?.closeCount()).toBe(1);
|
|
expect(secondBoundary.closeCount()).toBe(1);
|
|
expect(sdk.closeCount()).toBe(1);
|
|
});
|
|
|
|
it("does not retain a resumed session when it belongs to a different working directory", async () => {
|
|
const { runtime, sdk } = createRuntime();
|
|
const foreign = sdk.addSession("foreign-1", "/other-workspace");
|
|
|
|
await expect(
|
|
runtime.openSession(openOptions({ sessionId: "foreign-1" })),
|
|
).rejects.toThrow("The selected session belongs to a different working directory.");
|
|
|
|
expect(runtime.getSession("foreign-1")).toBeUndefined();
|
|
expect(foreign.closeCount()).toBe(1);
|
|
});
|
|
|
|
function createRecordingRuntime() {
|
|
const sdk = createFakeHarness();
|
|
const broadcasts: Array<{ event: string; data: unknown }> = [];
|
|
const runtime = new KimiRuntime({
|
|
version: "test",
|
|
harness: sdk.harness,
|
|
broadcast: (event, data) => {
|
|
broadcasts.push({ event, data });
|
|
},
|
|
captureBaseline: () => undefined,
|
|
log: () => undefined,
|
|
});
|
|
return { runtime, sdk, broadcasts };
|
|
}
|
|
|
|
it("fails a reentrant prompt without disturbing the running turn", async () => {
|
|
const { runtime, sdk, broadcasts } = createRecordingRuntime();
|
|
const opened = await runtime.openSession(openOptions());
|
|
const boundary = sdk.sessions.get(opened.id)!;
|
|
|
|
let releaseTurn!: () => void;
|
|
boundary.setPromptImpl(() => new Promise<void>((resolve) => {
|
|
releaseTurn = resolve;
|
|
}));
|
|
const first = opened.prompt("first message");
|
|
boundary.emit({ type: "turn.started", agentId: "main", sessionId: opened.id, turnId: "t1" } as unknown as Event);
|
|
expect(opened.isBusy).toBe(true);
|
|
|
|
await expect(opened.prompt("concurrent message")).resolves.toEqual({ status: "failed" });
|
|
|
|
// The rejection surfaces as a mid-turn warning; the active turn is untouched.
|
|
expect(opened.isBusy).toBe(true);
|
|
const busyWarning = broadcasts.find(({ data }) => (data as { type?: string }).type === "error");
|
|
expect(busyWarning?.data).toMatchObject({
|
|
type: "error",
|
|
phase: "runtime",
|
|
detail: "A response is already being generated for this session.",
|
|
terminal: false,
|
|
});
|
|
|
|
boundary.emit({ type: "turn.ended", agentId: "main", sessionId: opened.id, turnId: "t1", reason: "completed" } as unknown as Event);
|
|
releaseTurn();
|
|
await expect(first).resolves.toEqual({ status: "finished" });
|
|
expect(opened.isBusy).toBe(false);
|
|
});
|
|
|
|
it("rejects a prompt during an exclusive operation with a terminal error", async () => {
|
|
const { runtime, broadcasts } = createRecordingRuntime();
|
|
const opened = await runtime.openSession(openOptions());
|
|
|
|
let releaseExclusive!: () => void;
|
|
const exclusive = opened.runExclusiveAfterCancelling(
|
|
() => new Promise<void>((resolve) => {
|
|
releaseExclusive = resolve;
|
|
}),
|
|
);
|
|
// No active work means no later stream_complete: the rejection must be
|
|
// terminal so the caller's composer can unlock.
|
|
expect(opened.isBusy).toBe(true);
|
|
|
|
await expect(opened.prompt("during fork")).resolves.toEqual({ status: "failed" });
|
|
const rejection = broadcasts.find(({ data }) => (data as { type?: string }).type === "error");
|
|
expect(rejection?.data).toMatchObject({ type: "error", phase: "runtime" });
|
|
expect((rejection?.data as Record<string, unknown>)["terminal"]).toBeUndefined();
|
|
|
|
releaseExclusive();
|
|
await exclusive;
|
|
expect(opened.isBusy).toBe(false);
|
|
});
|
|
|
|
it("marks a mid-turn core error as non-terminal until the turn ends", async () => {
|
|
const { runtime, sdk, broadcasts } = createRecordingRuntime();
|
|
const opened = await runtime.openSession(openOptions());
|
|
const boundary = sdk.sessions.get(opened.id)!;
|
|
|
|
let releaseTurn!: () => void;
|
|
boundary.setPromptImpl(() => new Promise<void>((resolve) => {
|
|
releaseTurn = resolve;
|
|
}));
|
|
const first = opened.prompt("first message");
|
|
boundary.emit({ type: "turn.started", agentId: "main", sessionId: opened.id, turnId: "t1" } as unknown as Event);
|
|
|
|
boundary.emit({
|
|
type: "error",
|
|
agentId: "main",
|
|
sessionId: opened.id,
|
|
code: "records.write_failed",
|
|
message: "Failed to write agent records: EACCES",
|
|
} as unknown as Event);
|
|
|
|
// The turn is still running: no settlement, no terminal error on the wire.
|
|
expect(opened.isBusy).toBe(true);
|
|
const warning = broadcasts.find(({ data }) => (data as { type?: string }).type === "error");
|
|
expect(warning?.data).toMatchObject({
|
|
type: "error",
|
|
code: "records.write_failed",
|
|
phase: "runtime",
|
|
terminal: false,
|
|
});
|
|
|
|
boundary.emit({ type: "turn.ended", agentId: "main", sessionId: opened.id, turnId: "t1", reason: "completed" } as unknown as Event);
|
|
releaseTurn();
|
|
await expect(first).resolves.toEqual({ status: "finished" });
|
|
expect(opened.isBusy).toBe(false);
|
|
});
|
|
|
|
it("keeps preflight failures terminal", async () => {
|
|
const { runtime, sdk, broadcasts } = createRecordingRuntime();
|
|
const opened = await runtime.openSession(openOptions());
|
|
const boundary = sdk.sessions.get(opened.id)!;
|
|
|
|
boundary.setPromptImpl(() => Promise.reject(new Error("provider down")));
|
|
|
|
await expect(opened.prompt("hi")).resolves.toEqual({ status: "failed" });
|
|
const failure = broadcasts.find(({ data }) => (data as { type?: string }).type === "error");
|
|
expect(failure?.data).toMatchObject({ type: "error", phase: "preflight" });
|
|
expect((failure?.data as Record<string, unknown>)["terminal"]).toBeUndefined();
|
|
expect(opened.isBusy).toBe(false);
|
|
});
|
|
});
|