fix(desktop): open links in system browser

Author: yiliang114

packages/web-shell/client/main.tsx

@@ -23,7 +23,6 @@ import {
   notifyOpenWorkTurnComplete,
   OpenWorkDesktopLayer,
   OpenWorkWelcomeFooter,
-  openInOpenWorkBrowser,
   recordOpenWorkSession,
 } from './openwork/OpenWorkDesktopLayer';
 import 'katex/dist/katex.min.css';
@@ -227,9 +226,6 @@ export function StandaloneApp({ daemonToken }: { daemonToken?: string }) {
             },
             compactThinking: true,
             markdownTableMode: 'advanced',
-            markdown: {
-              onOpenLink: openInOpenWorkBrowser,
-            },
             renderWelcomeFooter: () => (
               <OpenWorkWelcomeFooter composerRef={composerRef} />
             ),

packages/web-shell/client/openwork/OpenWorkDesktopLayer.test.ts

@@ -1,35 +1,9 @@
 /** @vitest-environment jsdom */
 
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import {
-  drainOpenWorkDeepLinks,
-  openInOpenWorkBrowser,
-} from './OpenWorkDesktopLayer';
-
-describe('OpenWork browser links', () => {
-  afterEach(() => {
-    delete (window as Window & { __TAURI__?: unknown }).__TAURI__;
-  });
-
-  it('only intercepts HTTP links in the Tauri desktop shell', () => {
-    expect(openInOpenWorkBrowser('https://qwen.ai/docs')).toBe(false);
-
-    (window as Window & { __TAURI__?: unknown }).__TAURI__ = {
-      core: { invoke: vi.fn() },
-    };
-    const opened: string[] = [];
-    window.addEventListener(
-      'openwork:open-browser',
-      (event) => opened.push((event as CustomEvent<string>).detail),
-      { once: true },
-    );
-
-    expect(openInOpenWorkBrowser('mailto:help@qwen.ai')).toBe(false);
-    expect(openInOpenWorkBrowser('https://user:secret@qwen.ai')).toBe(false);
-    expect(openInOpenWorkBrowser('https://qwen.ai/docs')).toBe(true);
-    expect(opened).toEqual(['https://qwen.ai/docs']);
-  });
+import { describe, expect, it, vi } from 'vitest';
+import { drainOpenWorkDeepLinks } from './OpenWorkDesktopLayer';
 
+describe('OpenWork desktop layer', () => {
   it('processes concurrent deep-link drains exactly once', async () => {
     const pending = ['openwork://new', 'openwork://session/session-1'];
     const take = vi.fn(async () => pending.splice(0));

packages/web-shell/client/openwork/OpenWorkDesktopLayer.tsx

@@ -223,25 +223,6 @@ export async function drainOpenWorkDeepLinks(
   (await take())?.forEach(open);
 }
 
-export function openInOpenWorkBrowser(url: string): boolean {
-  let safe = false;
-  try {
-    const parsed = new URL(url);
-    safe =
-      (parsed.protocol === 'http:' || parsed.protocol === 'https:') &&
-      Boolean(parsed.hostname) &&
-      !parsed.username &&
-      !parsed.password;
-  } catch {
-    // Fall through to the host's normal link handling.
-  }
-  if (!safe || !tauri()?.core?.invoke) return false;
-  window.dispatchEvent(
-    new CustomEvent('openwork:open-browser', { detail: url }),
-  );
-  return true;
-}
-
 export function notifyOpenWorkTurnComplete(): void {
   void invokeOpenWork('notify_turn_complete', {
     title: 'OpenWork',