Merge pull request #1624 from jaylfc/dev

release: promote dev to master (v1.0.0-beta.24)

Author: jaylfc

CHANGELOG.md

@@ -7,6 +7,13 @@ Versions follow semver beta: `1.0.0-beta.N`, bumped on each dev->master promotio
 
 ## [Unreleased]
 
+## [1.0.0-beta.24] - 2026-07-04
+
+### Fixed
+- taOS Agent: the agent chat no longer wrongly reports "runtime unavailable" when opencode is installed system-wide. taOS now finds opencode on the PATH and at its default install location, and when the runtime genuinely can't start it shows the real error instead of a misleading generic message. The taOS Agent dialog also no longer opens as a blank window when launched from the Launchpad or Dock. Reported by @mandresve (#1615, #1616).
+- Settings: theme, wallpaper, dock position and dock icon size now persist across logout/login. They were applying in-session but a restore that ran before login completed left them reverting to defaults on the next session. Reported by @mandresve (#1601, #1603).
+- Models & Providers: when a downloaded model can't be used because the backend that serves it isn't running, taOS now says exactly that and how to fix it (install/start the backend) instead of a generic "not found anywhere", and provider connection tests report the real failure reason instead of "unknown error". Reported by @mandresve (#1599, #1600, #1614).
+
 ## [1.0.0-beta.23] - 2026-07-04
 
 ### Security

desktop/package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "tinyagentos-desktop",
   "private": true,
-  "version": "1.0.0-beta.23",
+  "version": "1.0.0-beta.24",
   "type": "module",
   "scripts": {
     "dev": "vite",

desktop/package.json

@@ -11,6 +11,7 @@ import { useSessionPersistence } from "@/hooks/use-session-persistence";
 import { useDeviceMode } from "@/hooks/use-device-mode";
 import { useIsPwa } from "@/hooks/use-is-pwa";
 import { useThemeStore, restoreActiveTheme, installWebkitRepaintGuards } from "@/stores/theme-store";
+import { useOnAuthReady } from "@/hooks/use-on-auth-ready";
 import { useProcessStore } from "@/stores/process-store";
 import { useDockStore } from "@/stores/dock-store";
 import { getApp } from "@/registry/app-registry";
@@ -204,15 +205,27 @@ export function App() {
   // primary push channel; useServerNotifications falls back to polling.
   useEventStream();
 
-  // Re-apply the persisted active theme on app boot so a reload keeps the
-  // user's chosen theme app-wide (not only when Settings is opened).
+  // WebKit blanks backdrop-filter surfaces when the tab is hidden then shown
+  // again (switching back into taOS); re-composite them on return. Not
+  // auth-gated: it only installs a visibility listener, no per-user fetch.
   useEffect(() => {
-    void restoreActiveTheme();
-    // WebKit blanks backdrop-filter surfaces when the tab is hidden then shown
-    // again (switching back into taOS); re-composite them on return.
     installWebkitRepaintGuards();
   }, []);
 
+  // Re-apply the persisted active theme once authenticated, so a reload keeps
+  // the user's chosen theme app-wide (not only when Settings is opened).
+  //
+  // SystemShortcuts mounts as a sibling of LoginGate, before LoginGate's own
+  // /auth/status check resolves, so firing this on bare mount races the auth
+  // check: right after a logout there is no session cookie yet, the fetch
+  // inside restoreActiveTheme 401s, and — since this used to run once on
+  // mount — it was never retried once the user logged back in (#1601).
+  // useOnAuthReady defers it until auth is confirmed and re-runs it on a
+  // later re-login.
+  useOnAuthReady(() => {
+    void restoreActiveTheme();
+  });
+
   // Apply reduce-effects / performance mode (#58) as a root attribute so the CSS
   // in tokens.css can strip blur, big shadows and continuous animations on
   // low-end devices. Reactive so the Settings toggle takes effect immediately.

desktop/src/App.tsx

@@ -0,0 +1,109 @@
+import { render, screen, fireEvent, waitFor } from "@testing-library/react";
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+import { TaosAssistantWindow } from "./TaosAssistantWindow";
+import { useTaosAgentStore } from "@/stores/taos-agent-store";
+import { useProcessStore } from "@/stores/process-store";
+
+// Mock child that opens a modal — keeps the test surface narrow, mirrors
+// TaosAssistantPanel.test.tsx's approach.
+vi.mock("@/components/TaosAssistantSettings", () => ({
+  TaosAssistantSettings: ({ open }: { open: boolean }) =>
+    open ? <div data-testid="settings-modal" /> : null,
+}));
+
+function resetStores() {
+  useTaosAgentStore.setState({
+    isOpen: false,
+    messages: [],
+    model: "qwen3",
+    streaming: false,
+    settingsOpen: false,
+  });
+  useProcessStore.setState({ windows: [] });
+}
+
+describe("TaosAssistantWindow", () => {
+  beforeEach(() => {
+    resetStores();
+  });
+
+  afterEach(() => {
+    vi.unstubAllGlobals();
+    vi.clearAllMocks();
+  });
+
+  it("renders the chat shell when launched directly (no popOut prop) — regression for #1615", () => {
+    // Direct Launchpad/Dock launch calls openWindow("taos-agent", size) with
+    // no props, so windows never carries a matching entry with
+    // props.popOut. The window must still show its UI, not go blank.
+    vi.stubGlobal(
+      "fetch",
+      vi.fn().mockResolvedValue({ ok: true, json: async () => ({ model: "qwen3" }) }),
+    );
+
+    render(<TaosAssistantWindow windowId="win-1" />);
+
+    expect(screen.getByText("taOS agent")).toBeInTheDocument();
+    expect(screen.getByRole("textbox", { name: /message taOS agent/i })).toBeInTheDocument();
+    expect(screen.getByRole("button", { name: /send message/i })).toBeInTheDocument();
+  });
+
+  it("renders the chat shell when opened via the pop-out button (props.popOut)", () => {
+    useProcessStore.setState({
+      windows: [
+        {
+          id: "win-2",
+          appId: "taos-agent",
+          position: { x: 0, y: 0 },
+          size: { w: 420, h: 640 },
+          zIndex: 1,
+          minimized: false,
+          maximized: false,
+          snapped: null,
+          focused: true,
+          props: { popOut: true },
+          launchNonce: 0,
+        },
+      ],
+    });
+    vi.stubGlobal(
+      "fetch",
+      vi.fn().mockResolvedValue({ ok: true, json: async () => ({ model: "qwen3" }) }),
+    );
+
+    render(<TaosAssistantWindow windowId="win-2" />);
+
+    expect(screen.getByText("taOS agent")).toBeInTheDocument();
+    expect(screen.getByRole("textbox", { name: /message taOS agent/i })).toBeInTheDocument();
+  });
+
+  it("surfaces the runtime error inline instead of leaving the dialog blank", async () => {
+    vi.stubGlobal(
+      "fetch",
+      vi.fn().mockImplementation((url: string) => {
+        if (typeof url === "string" && url.includes("/api/taos-agent/settings")) {
+          return Promise.resolve({ ok: true, json: async () => ({ model: "qwen3" }) });
+        }
+        // /api/taos-agent/chat — simulate the runtime-unavailable 503.
+        return Promise.resolve({
+          ok: false,
+          body: null,
+          text: async () =>
+            JSON.stringify({ error: "opencode is not installed. Install it with: curl -fsSL https://opencode.ai/install | bash" }),
+        });
+      }),
+    );
+
+    render(<TaosAssistantWindow windowId="win-3" />);
+
+    const textarea = screen.getByRole("textbox", { name: /message taOS agent/i });
+    fireEvent.change(textarea, { target: { value: "hello" } });
+    fireEvent.click(screen.getByRole("button", { name: /send message/i }));
+
+    // The shell stays up (input still present) and the error is shown inline.
+    await waitFor(() => {
+      expect(screen.getByText(/opencode is not installed/i)).toBeInTheDocument();
+    });
+    expect(screen.getByRole("textbox", { name: /message taOS agent/i })).toBeInTheDocument();
+  });
+});

desktop/src/apps/TaosAssistantWindow.test.tsx

@@ -6,15 +6,17 @@ export function TaosAssistantWindow({ windowId }: { windowId: string }) {
   const closeWindow = useProcessStore((s) => s.closeWindow);
   const removeWindow = useProcessStore((s) => s.removeWindow);
   const snap = useProcessStore((s) => s.snapWindow);
-  const isPopOut = useProcessStore((s) => s.windows.find((w) => w.id === windowId)?.props?.popOut);
 
   const handleClose = useCallback(() => {
     closeWindow(windowId);
     setTimeout(() => removeWindow(windowId), 250);
   }, [closeWindow, removeWindow, windowId]);
 
-  if (!isPopOut) return null;
-
+  // This window renders the chat unconditionally: it is reached both via the
+  // docked panel's "Pop out" button (which passes props.popOut) and directly
+  // from the Launchpad/dock ("taos-agent" is a regular app-registry entry).
+  // Gating render on props.popOut left the direct-launch path with nothing
+  // to show -- an empty window with no error, no content (#1615).
   return (
     <div className="h-full w-full flex flex-col bg-shell-bg-deep">
       <div className="flex items-center gap-2 px-4 py-3 border-b border-white/5 shrink-0 select-none">

desktop/src/apps/TaosAssistantWindow.tsx

@@ -3,6 +3,7 @@ import { Lock } from "lucide-react";
 import { OnboardingScreen } from "./OnboardingScreen";
 import { OffNetworkScreen } from "./OffNetworkScreen";
 import { SESSION_EXPIRED_EVENT } from "@/lib/auth-guard";
+import { useAuthReadyStore } from "@/stores/auth-ready-store";
 
 interface Props {
   children: React.ReactNode;
@@ -61,6 +62,14 @@ export function LoginGate({ children }: Props) {
     refreshStatus();
   }, [refreshStatus]);
 
+  // Publish auth-readiness for session-scoped restore effects that live
+  // outside LoginGate's children (e.g. useSessionPersistence, active-theme
+  // restore) — they must not fetch per-user data until this flips true, and
+  // must re-fetch the next time it does (see auth-ready-store.ts).
+  useEffect(() => {
+    useAuthReadyStore.getState().setReady(status.phase === "ready");
+  }, [status.phase]);
+
   // Listen for session-expired events from the global auth guard. Any
   // /api/* call returning 401 fires this; we re-run /auth/status which
   // will flip phase to "login" and unmount the app shell back to the

desktop/src/components/LoginGate.tsx

@@ -0,0 +1,63 @@
+import { describe, it, expect, vi, beforeEach } from "vitest";
+import { renderHook, act } from "@testing-library/react";
+import { useOnAuthReady } from "../use-on-auth-ready";
+import { useAuthReadyStore } from "@/stores/auth-ready-store";
+
+beforeEach(() => {
+  useAuthReadyStore.setState({ ready: false });
+});
+
+describe("useOnAuthReady (#1601, #1603)", () => {
+  it("does not run the callback while logged out", () => {
+    const callback = vi.fn();
+    renderHook(() => useOnAuthReady(callback));
+    expect(callback).not.toHaveBeenCalled();
+  });
+
+  it("runs the callback once auth becomes ready", () => {
+    const callback = vi.fn();
+    renderHook(() => useOnAuthReady(callback));
+
+    act(() => {
+      useAuthReadyStore.setState({ ready: true });
+    });
+
+    expect(callback).toHaveBeenCalledTimes(1);
+  });
+
+  it("does not re-run while still authenticated (no duplicate restores per session)", () => {
+    const callback = vi.fn();
+    const { rerender } = renderHook(() => useOnAuthReady(callback));
+
+    act(() => {
+      useAuthReadyStore.setState({ ready: true });
+    });
+    rerender();
+    rerender();
+
+    expect(callback).toHaveBeenCalledTimes(1);
+  });
+
+  it("runs the callback again after a logout/login cycle", () => {
+    // This is the exact bug: a theme/settings restore that only ever fires
+    // once on mount is never retried after the user logs back in following
+    // a logout. useOnAuthReady must re-arm on the falling edge so the next
+    // rising edge (the next login) restores the new session's data.
+    const callback = vi.fn();
+    renderHook(() => useOnAuthReady(callback));
+
+    act(() => {
+      useAuthReadyStore.setState({ ready: true });
+    });
+    expect(callback).toHaveBeenCalledTimes(1);
+
+    act(() => {
+      useAuthReadyStore.setState({ ready: false }); // logout
+    });
+    act(() => {
+      useAuthReadyStore.setState({ ready: true }); // login again
+    });
+
+    expect(callback).toHaveBeenCalledTimes(2);
+  });
+});