Skip to content

feat(web): open every link in the integrated browser by default (expbkt3) - #211

Merged
tusharbhardwaj-bk merged 1 commit into
expbkmainfrom
t3code/paris
Sep 25, 2026
Merged

tusharbhardwaj-bk merged 1 commit into
expbkmainfrom
t3code/paris

Conversation

@tusharbhardwaj-bk

@tusharbhardwaj-bk tusharbhardwaj-bk commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Staging PR for 1c1f56d. Every link opens in the integrated browser beside the open thread by default; Cmd/Ctrl-click still opens the system browser. Experimental flag: Settings → Experiments → Open links in the integrated browser (on by default). The same commit also rides on #208 for bkmain.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Links went to the system browser unless Settings → Integrations said
otherwise, and several surfaces ignored that setting altogether, such
as the sidebar's Linear, pull request and Mattermost tags and plain
target=_blank anchors.

A new experimental setting, "Open links in the integrated browser",
is on by default. While it is on:
- the "Open links in" preference resolves to the in-app browser, so
  chat markdown, useOpenLink callers and the terminal follow it
- a document-level click handler opens any remaining external anchor
  beside the open thread, after React's own handlers have had their turn
- the lifecycle sidebar's link chips use the same opener

Cmd/Ctrl-click still opens the system browser. With no open thread, a
link behaves as before.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Sep 25, 2026
@github-actions

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire — 13.8 KiB — 15.1 KiB ✅
Codex Thread snapshot wire — 7.3 KiB — 7.8 KiB ✅
Codex Live turn WebSocket wire — 6.5 KiB — 7.8 KiB ✅
Codex Live turn WebSocket decoded — 57.1 KiB — 66.4 KiB ✅
Codex Live turn messages — 9 — 21 ✅
Claude Total thread wire — 13.8 KiB — 15.1 KiB ✅
Claude Thread snapshot wire — 7.3 KiB — 7.8 KiB ✅
Claude Live turn WebSocket wire — 6.5 KiB — 7.8 KiB ✅
Claude Live turn WebSocket decoded — 57.9 KiB — 66.4 KiB ✅
Claude Live turn messages — 8 — 21 ✅

Baseline: unavailable · PR result: 1c1f56d · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 114.9 KiB
  • Claude decoded thread snapshot: 115.6 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@tusharbhardwaj-bk
tusharbhardwaj-bk merged commit c0779bd into expbkmain Sep 25, 2026
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants