fix: restore app vitest coverage and last-used model wiring

Put desktop, pythinker-web, and vscode projects back in the root vitest
matrix. Port last-used model events onto the live llm-adapter
ModelService so provider detail routes stop throwing during initialize.
Keep exact modelCatalog assertions, fix web load test deps, and align
vscode bridge/telemetry/migration fixtures with the reconciled APIs.

Author: elkaix

apps/pythinker-web/test/composer-toolbar.test.ts

@@ -404,7 +404,7 @@ describe('Composer toolbar overflow valves', () => {
 
     expect(wrapper.get('.composer-card').classes()).not.toContain('labels-collapsed');
     expect(wrapper.get('.model-pill').classes()).not.toContain('icon-only');
-    expect(wrapper.get('.perm-pill-label').text()).toBe('Manual');
+    expect(wrapper.get('.perm-pill-label').text()).toBe('Always Ask');
   });
 
   it('opens thinking effort separately and closes it with Escape', async () => {

apps/pythinker-web/test/workspace-state.test.ts

@@ -138,10 +138,20 @@ function createDeps(): UseWorkspaceStateDeps {
   return {
     taskPoller: {},
     sideChat: { clearSideChatForSession: vi.fn() },
-    modelProvider: { resolveThinkingForPrompt: async () => undefined },
+    modelProvider: {
+      resolveThinkingForPrompt: async () => undefined,
+      loadModels: vi.fn().mockResolvedValue(undefined),
+    },
     pushOperationFailure: vi.fn(),
     activity: computed(() => 'running'),
     sessionsKnownEmpty: new Set(),
+    bootStage: ref('idle'),
+    bootRetries: ref(0),
+    selectedDiffPath: ref(null),
+    fileDiffLines: ref([]),
+    fileDiffLoading: ref(false),
+    initialized: ref(true),
+    connectIssue: ref(null),
     setSessions: vi.fn(),
     updateSession: vi.fn(),
     upsertSessionFront: vi.fn(),
@@ -174,10 +184,6 @@ function createDeps(): UseWorkspaceStateDeps {
     resetFastMoon: vi.fn(),
     getExpertTalkArmId: () => undefined,
     onExpertTalkPromptAccepted: vi.fn(),
-    initialized: ref(true),
-    selectedDiffPath: ref(null),
-    fileDiffLines: ref([]),
-    fileDiffLoading: ref(false),
   } as unknown as UseWorkspaceStateDeps;
 }
 
@@ -1368,9 +1374,9 @@ describe('useWorkspaceState — first-load auth gate', () => {
   ): UseWorkspaceStateDeps {
     return {
       ...createDeps(),
-      modelProvider: { loadModels: vi.fn().mockResolvedValue(undefined) },
       initialized,
       connectIssue,
+      bootStage: ref('auth'),
     } as unknown as UseWorkspaceStateDeps;
   }
 

apps/vscode/src/migration/index.ts

@@ -0,0 +1 @@
+export { LegacyMigrationManager } from './legacy-migration.manager';

apps/vscode/test/bridge-handler.test.ts

@@ -319,7 +319,7 @@ describe("Webview RPC boundary (validates requests before host dispatch)", () =>
     ] as never);
 
     const result = await bridge.handle(
-      { id: "rpc-1", method: Methods.GetPythinkerSessions },
+      { id: "rpc-1", method: Methods.GetSessions },
       "view-1",
     );
 
@@ -349,7 +349,7 @@ describe("Webview RPC boundary (validates requests before host dispatch)", () =>
     const result = await bridge.handle(
       {
         id: "rpc-1",
-        method: Methods.ForkPythinkerSession,
+        method: Methods.ForkSession,
         params: { sessionId: "session-1", turnIndex: 0 },
       },
       "view-1",
@@ -382,7 +382,7 @@ describe("Webview RPC boundary (validates requests before host dispatch)", () =>
     const result = await bridge.handle(
       {
         id: "rpc-1",
-        method: Methods.ForkPythinkerSession,
+        method: Methods.ForkSession,
         params: { sessionId: "session-1", turnIndex: 0 },
       },
       "view-1",
@@ -407,7 +407,7 @@ describe("Webview RPC boundary (validates requests before host dispatch)", () =>
     const result = await bridge.handle(
       {
         id: "rpc-1",
-        method: Methods.ForkPythinkerSession,
+        method: Methods.ForkSession,
         params: { sessionId: "session-1", turnIndex: 0 },
       },
       "view-1",
@@ -441,8 +441,8 @@ describe("Webview RPC boundary (validates requests before host dispatch)", () =>
     const result = await bridge.handle(
       {
         id: "rpc-1",
-        method: Methods.LoadPythinkerSessionHistory,
-        params: { pythinkerSessionId: "session-1" },
+        method: Methods.LoadSessionHistory,
+        params: { sessionId: "session-1" },
       },
       "view-1",
     );
@@ -467,8 +467,8 @@ describe("Webview RPC boundary (validates requests before host dispatch)", () =>
     const failed = await bridge.handle(
       {
         id: "rpc-1",
-        method: Methods.LoadPythinkerSessionHistory,
-        params: { pythinkerSessionId: "session-1" },
+        method: Methods.LoadSessionHistory,
+        params: { sessionId: "session-1" },
       },
       "view-1",
     );

apps/vscode/test/pythinker-harness.integration.test.ts

@@ -364,15 +364,16 @@ async function runSlash(
 }
 
 describe("VS Code Pythinker harness integration (shares one in-process SDK home)", () => {
-  it("only intercepts released slash commands and user-invoked skills", () => {
-    expect(parseHostSlashCommand("/plan on")).toEqual({ name: "plan", args: "on", raw: "/plan on" });
-    expect(parseHostSlashCommand(" /skill:review carefully ")).toEqual({
+  it("only intercepts released slash commands and user-invoked skills", async () => {
+    expect(await parseHostSlashCommand("/plan on")).toEqual({ name: "plan", args: "on", raw: "/plan on" });
+    expect(await parseHostSlashCommand(" /skill:review carefully ")).toEqual({
       name: "skill:review",
       args: "carefully",
       raw: "/skill:review carefully",
+      skillName: "review",
     });
-    expect(parseHostSlashCommand("/not-a-host-command")).toBeUndefined();
-    expect(parseHostSlashCommand([{ type: "text", text: "/clear" }])).toBeUndefined();
+    expect(await parseHostSlashCommand("/not-a-host-command")).toBeUndefined();
+    expect(await parseHostSlashCommand([{ type: "text", text: "/clear" }])).toBeUndefined();
   });
 
   it("combines the released slash commands with user-activatable workspace skills", async () => {

apps/vscode/test/settings-store.test.ts

@@ -1,3 +1,6 @@
+/**
+ * @vitest-environment jsdom
+ */
 /**
  * Scenario: Webview state crosses the VS Code bridge during settings changes, MCP edits, and chat failures.
  * Responsibilities: model metadata and selections remain provider-aware; MCP edits stay lossless; chat errors recover visibly.
@@ -202,7 +205,10 @@ describe("Webview MCP update bridge", () => {
     const posted: unknown[] = [];
     let receiveMessage: ((event: { data: unknown }) => void) | undefined;
     vi.stubGlobal("document", {
-      body: { getAttribute: () => "mcp-test-view" },
+      body: {
+        dataset: { webviewid: "mcp-test-view" },
+        getAttribute: () => "mcp-test-view",
+      },
     });
     vi.stubGlobal("window", {
       addEventListener: (_type: string, listener: (event: { data: unknown }) => void) => {

apps/vscode/test/telemetry.test.ts

@@ -123,8 +123,6 @@ describe("extension telemetry composition root", () => {
       model: "kimi-k2",
     });
     expect(options["endpoint"]).toBeTypeOf("function");
-    expect(options["getAccessToken"]).toBeTypeOf("function");
-    await expect((options["getAccessToken"] as () => Promise<string | null>)()).resolves.toBe("token-1");
     expect(host.shouldEnableTelemetry).toHaveBeenCalledWith({ enabled: true });
   });
 
@@ -144,9 +142,8 @@ describe("extension telemetry composition root", () => {
     expect(endpoint()).toBe("https://telemetry-logs.example.net/v1/event");
     expect(host.resolvePythinkerRegion).toHaveBeenCalledWith(
       expect.objectContaining({
-        configuredOAuthHost: "https://auth.kimi.ai",
-        configuredOAuthKey: "oauth/pythinker-code-global",
         homeDir: "/pythinker-home",
+        readMarker: true,
       }),
     );
   });

packages/agent-core-v2/src/app/kosongConfig/kosongConfigService.ts

@@ -102,15 +102,10 @@ export class KosongConfigService extends Disposable implements IKosongConfigServ
         );
         break;
       case MODELS_SECTION:
-        const models = this.models as {
-          loadAll: (section: ModelsSection, defaultModel: string | undefined, lastUsed?: string | undefined) => void;
-          getDefaultModel: () => string | undefined;
-          getLastUsedModel?: () => string | undefined;
-        };
-        models.loadAll(
+        this.models.loadAll(
           (e.value as ModelsSection | undefined) ?? {},
-          models.getDefaultModel(),
-          models.getLastUsedModel?.(),
+          this.models.getDefaultModel(),
+          this.models.getLastUsedModel(),
         );
         break;
       case DEFAULT_PROVIDER_SECTION:

packages/agent-core-v2/src/llm-adapter/model/model-service.ts

@@ -20,6 +20,7 @@ export class ModelService extends Disposable implements IModelService {
 
   private models: Readonly<Record<string, ModelRecord>> = {};
   private defaultModel: string | undefined;
+  private lastUsedModel: string | undefined;
   private hydrated = false;
   private resolveReady!: () => void;
   readonly ready: Promise<void> = new Promise<void>((resolve) => {
@@ -36,6 +37,11 @@ export class ModelService extends Disposable implements IModelService {
   );
   readonly onDidChangeDefaultModel: Event<DefaultModelChangedEvent & IWaitUntil> =
     this._onDidChangeDefaultModel.event;
+  private readonly _onDidChangeLastUsedModel = this._register(
+    new AsyncEmitter<DefaultModelChangedEvent & IWaitUntil>(),
+  );
+  readonly onDidChangeLastUsedModel: Event<DefaultModelChangedEvent & IWaitUntil> =
+    this._onDidChangeLastUsedModel.event;
 
   get(id: string): ModelRecord | undefined {
     return this.models[id];
@@ -49,9 +55,18 @@ export class ModelService extends Disposable implements IModelService {
     return this.defaultModel;
   }
 
-  loadAll(models: ModelsSection, defaultModel: string | undefined): void {
+  getLastUsedModel(): string | undefined {
+    return this.lastUsedModel;
+  }
+
+  loadAll(
+    models: ModelsSection,
+    defaultModel: string | undefined,
+    lastUsedModel?: string | undefined,
+  ): void {
     void this.applyRecords(models);
     void this.applyDefaultModel(defaultModel);
+    void this.applyLastUsedModel(lastUsedModel);
     if (!this.hydrated) {
       this.hydrated = true;
       this.resolveReady();
@@ -81,6 +96,11 @@ export class ModelService extends Disposable implements IModelService {
     await this.applyDefaultModel(id);
   }
 
+  async setLastUsedModel(id: string | undefined): Promise<void> {
+    await this.ready;
+    await this.applyLastUsedModel(id);
+  }
+
   private async applyRecords(next: Readonly<Record<string, ModelRecord>>): Promise<void> {
     const diff = diffRecords(this.models, next);
     if (isEmptyDiff(diff)) return;
@@ -93,6 +113,12 @@ export class ModelService extends Disposable implements IModelService {
     this.defaultModel = id;
     await this._onDidChangeDefaultModel.fireAsync({ id }, NO_ABORT);
   }
+
+  private async applyLastUsedModel(id: string | undefined): Promise<void> {
+    if (this.lastUsedModel === id) return;
+    this.lastUsedModel = id;
+    await this._onDidChangeLastUsedModel.fireAsync({ id }, NO_ABORT);
+  }
 }
 
 registerScopedService(LifecycleScope.App, IModelService, ModelService, ScopeActivation.OnScopeCreated, 'model');

packages/agent-core-v2/src/llm-adapter/model/model.ts

@@ -66,14 +66,21 @@ export interface IModelService {
   readonly ready: Promise<void>;
   readonly onDidChangeModels: Event<ModelsChangedEvent & IWaitUntil>;
   readonly onDidChangeDefaultModel: Event<DefaultModelChangedEvent & IWaitUntil>;
+  readonly onDidChangeLastUsedModel: Event<DefaultModelChangedEvent & IWaitUntil>;
   get(id: string): ModelRecord | undefined;
   list(): Readonly<Record<string, ModelRecord>>;
   getDefaultModel(): string | undefined;
+  getLastUsedModel(): string | undefined;
   set(id: string, model: ModelRecord): Promise<void>;
   delete(id: string): Promise<void>;
-  loadAll(models: ModelsSection, defaultModel: string | undefined): void;
+  loadAll(
+    models: ModelsSection,
+    defaultModel: string | undefined,
+    lastUsedModel?: string | undefined,
+  ): void;
   replaceAll(models: ModelsSection): Promise<void>;
   setDefaultModel(id: string | undefined): Promise<void>;
+  setLastUsedModel(id: string | undefined): Promise<void>;
 }
 
 export const IModelService: ServiceIdentifier<IModelService> =