Open terminal localhost links in the Browser preview#2610
Conversation
|
PR description follow-up: the implementation now correctly excludes orchestrator sessions because they have no Browser inspector, but the Summary and Behavior sections still promise Browser mirroring for every current session. Please qualify those statements with worker sessions. Also update the Validation count: the listed focused command currently runs 76 tests, not 77. |
- Share link activation across OSC 8 and detected URLs - Keep loopback preview validation at its call site
done |
|
From the NotesBot call on Relevant discussion excerpts:
Source thread: https://discord.com/channels/1476302178913357958/1525926468444749957/1525926468444749957 Generated by AO Bot based on the transcript from NotesBot. |
Summary
Behavior
Clicked URLs for
localhost,*.localhost,127.0.0.1, and::1in worker sessions open both externally and in AO's Browser panel. External and non-HTTP(S) links retain the existing external-only behavior.Validation
npm test -- src/renderer/components/BrowserPanel.test.tsx src/renderer/components/XtermTerminal.test.tsx src/renderer/components/TerminalPane.test.tsx src/renderer/components/SessionView.test.tsx— 77/77 passednpm run typecheck