Skip to content

Open terminal URLs in preview or browser#24

Merged
BunsDev merged 1 commit into
mainfrom
okcode/terminal-url-preview
Mar 27, 2026
Merged

Open terminal URLs in preview or browser#24
BunsDev merged 1 commit into
mainfrom
okcode/terminal-url-preview

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Mar 27, 2026

Summary

  • Route terminal-detected URLs to the in-app preview when running in Electron, unless the user has opted to open links externally.
  • Broaden preview URL validation to accept HTTP(S) URLs and update preview copy to reflect the new behavior.
  • Simplify the code viewer and sidebar by removing the old code-context shortcut flow, collapsing the PR review page, and streamlining related UI state.
  • Adjust app defaults and settings for the new link-opening behavior and local thread environment mode.

Testing

  • Not run (PR content generation only).
  • Expected checks for the branch: bun fmt, bun lint, bun typecheck.
  • Manual validation: open a terminal URL in Electron with preview enabled, verify it lands in the preview panel; toggle external-link behavior in settings and verify URLs open in the browser instead.

- Add http/https preview URL validation
- Route terminal URL matches to the embedded preview when enabled
- Add a setting to open links externally instead
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Mar 27, 2026 10:27pm

@BunsDev BunsDev merged commit b4e74e3 into main Mar 27, 2026
7 of 8 checks passed
@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Mar 27, 2026
@BunsDev BunsDev deleted the okcode/terminal-url-preview branch March 28, 2026 05:25
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.

1 participant