fix(vscode): stop walling API-key providers behind the sign-in screen

The init status reported not-logged-in whenever the login check was false, even
with models configured. That check is true only for a Pythinker account or a
provider OAuth token, so a provider authenticated by a plain API key in
config.toml never satisfied it and the sign-in screen returned on every reload
-- the skip is component state, so it never persisted.

A configured model is what makes the extension usable, so it now decides ready
on its own. The status derivation moves into a pure resolveInitStatus so the
three branches collapse into one rule and can be tested directly.

Author: elkaix

.changeset/vscode-api-key-no-signin-wall.md

@@ -0,0 +1,5 @@
+---
+"@pymodel/pythinker-code": patch
+---
+
+Fix the VS Code extension opening on the sign-in screen for providers authenticated with a plain API key: a configured model now opens straight into the chat.

apps/vscode/test/app-init.test.ts

@@ -18,7 +18,7 @@ vi.mock("@/components/ui/sonner", () => ({
   toast: { error: vi.fn(), warning: vi.fn() },
 }));
 
-import { resolveAppView, type AppStatus } from "../webview-ui/src/hooks/useAppInit";
+import { resolveAppView, resolveInitStatus, type AppStatus } from "../webview-ui/src/hooks/useAppInit";
 
 function resolve(
   status: AppStatus,
@@ -84,3 +84,24 @@ describe("resolveAppView", () => {
     }
   });
 });
+
+describe("resolveInitStatus", () => {
+  it("is ready with configured models even when nothing reports a login", () => {
+    // Regression: an API-key-only provider satisfies neither `isAuthenticated()`
+    // nor a provider OAuth token, so this state walled a fully configured user
+    // behind a sign-in screen on every reload.
+    expect(resolveInitStatus({ modelsCount: 2, loggedIn: false })).toBe("ready");
+  });
+
+  it("is ready with configured models when a login is reported", () => {
+    expect(resolveInitStatus({ modelsCount: 2, loggedIn: true })).toBe("ready");
+  });
+
+  it("asks for setup when a login exists but no model is configured", () => {
+    expect(resolveInitStatus({ modelsCount: 0, loggedIn: true })).toBe("no-models");
+  });
+
+  it("asks a brand-new user to add a provider", () => {
+    expect(resolveInitStatus({ modelsCount: 0, loggedIn: false })).toBe("not-logged-in");
+  });
+});

apps/vscode/webview-ui/src/hooks/useAppInit.ts

@@ -17,6 +17,22 @@ export type AppViewResolution =
     }
   | { readonly view: "main" };
 
+/**
+ * Pure status router for init. A configured model is what makes the extension
+ * usable, so it decides `ready` on its own: `loggedIn` covers a Pythinker
+ * account or a provider OAuth token, and a provider authenticated by a plain
+ * API key in `config.toml` has neither. Gating on it walled off every
+ * API-key-only user behind a sign-in screen they could not satisfy and had to
+ * dismiss on each reload, because the skip is component state.
+ */
+export function resolveInitStatus(input: {
+  readonly modelsCount: number;
+  readonly loggedIn: boolean;
+}): AppStatus {
+  if (input.modelsCount > 0) return "ready";
+  return input.loggedIn ? "no-models" : "not-logged-in";
+}
+
 /**
  * Pure view router for App. The `no-models` status (a managed OAuth token
  * exists but config.toml has no models — e.g. a first login whose model
@@ -111,22 +127,11 @@ export function useAppInit(): AppInitState {
 
         const modelsCount = modelsConfig.models?.length ?? 0;
 
-        if (modelsCount === 0 && !loginStatus.loggedIn) {
-          setState({ status: "not-logged-in", errorMessage: null, modelsCount });
-          return;
-        }
-
-        if (modelsCount === 0) {
-          setState({ status: "no-models", errorMessage: null, modelsCount: 0 });
-          return;
-        }
-
-        if (!loginStatus.loggedIn) {
-          setState({ status: "not-logged-in", errorMessage: null, modelsCount });
-          return;
-        }
-
-        setState({ status: "ready", errorMessage: null, modelsCount });
+        setState({
+          status: resolveInitStatus({ modelsCount, loggedIn: loginStatus.loggedIn }),
+          errorMessage: null,
+          modelsCount,
+        });
       } catch (error) {
         if (!cancelled) {
           setState({