Skip to content

fix(init): make URLs clickable with OSC 8 terminal hyperlinks#423

Merged
MathurAditya724 merged 2 commits intomainfrom
fix/init-links
Mar 13, 2026
Merged

fix(init): make URLs clickable with OSC 8 terminal hyperlinks#423
MathurAditya724 merged 2 commits intomainfrom
fix/init-links

Conversation

@MathurAditya724
Copy link
Member

Summary

  • Wrap project URL, docs URL, and manual-setup URL in terminalLink() (OSC 8 escape sequences) so they become clickable in supported terminals (iTerm2, VS Code, Windows Terminal, most modern emulators)
  • Uses the existing terminalLink() utility from src/lib/formatters/colors.ts — no new dependencies
  • On terminals that don't support OSC 8, the escape sequences are silently ignored and URLs render as plain text (no degradation)

Changes

  • src/lib/init/formatters.ts — Wrap project URL and docs URL in success summary and error output
  • src/lib/init/clack-utils.ts — Wrap docs URL in the cancellation message
  • src/lib/init/wizard-runner.ts — Wrap docs URL in the early "manual setup" info message

@github-actions
Copy link
Contributor

github-actions bot commented Mar 13, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

Init

  • Add --team flag to relay team selection to project creation by MathurAditya724 in #403
  • Enforce canonical feature display order by betegon in #388
  • Accept multiple delimiter formats for --features flag by betegon in #386
  • Add git safety checks before wizard modifies files by betegon in #379
  • Add experimental warning before wizard runs by betegon in #378
  • Add init command for guided Sentry project setup by betegon in #283

Issue List

  • Auto-compact when table exceeds terminal height by BYK in #395
  • Redesign table to match Sentry web UI by BYK in #372

Other

  • (auth) Allow re-authentication without manual logout by BYK in #417
  • (trial) Auto-prompt for Seer trial + sentry trial list/start commands by BYK in #399
  • Support SENTRY_HOST as alias for SENTRY_URL by betegon in #409
  • Add --dry-run flag to mutating commands by BYK in #387
  • Return-based output with OutputConfig on buildCommand by BYK in #380
  • Add --fields flag for context-window-friendly JSON output by BYK in #373
  • Magic @ selectors (@latest, @most_frequent) for issue commands by BYK in #371
  • Input hardening against agent hallucinations by BYK in #370
  • Add response caching for read-only API calls by BYK in #330

Bug Fixes 🐛

Dsn

Init

  • Make URLs clickable with OSC 8 terminal hyperlinks by MathurAditya724 in #423
  • Remove implementation detail from help text by betegon in #385
  • Truncate uncommitted file list to first 5 entries by MathurAditya724 in #381

Other

  • (api) Convert --data to query params for GET requests by BYK in #383
  • (docs) Remove double borders and fix column alignment on landing page tables by betegon in #369
  • Show human-friendly names in trial list and surface plan trials by BYK in #412
  • Add trace ID validation to trace view + UUID dash-stripping by BYK in #375

Documentation 📚

  • Update credential storage docs and remove stale config.json references by betegon in #408

Internal Changes 🔧

Init

  • Remove --force flag by betegon in #377
  • Remove dead determine-pm step label by betegon in #374

Other

  • (log/list) Convert non-follow paths to return CommandOutput by BYK in #410
  • (tests) Remove redundant and low-value tests by BYK in #418
  • Convert list command handlers to return data instead of writing stdout by BYK in #404
  • Split api-client.ts into focused domain modules by BYK in #405
  • Migrate non-streaming commands to CommandOutput with markdown rendering by BYK in #398
  • Convert Tier 2-3 commands to return-based output and consola by BYK in #394
  • Convert remaining Tier 1 commands to return-based output by BYK in #382
  • Converge Tier 1 commands to writeOutput helper by BYK in #376

Other

  • Minify JSON on read and pretty-print on write in init local ops by MathurAditya724 in #396

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 13, 2026

Codecov Results 📊

111 passed | Total: 111 | Pass Rate: 100% | Execution Time: 0ms

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 1064 uncovered lines.
✅ Project coverage is 95.04%. Comparing base (base) to head (head).

Files with missing lines (1)
File Patch % Lines
colors.ts 98.33% ⚠️ 1 Missing
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    95.03%    95.04%    +0.01%
==========================================
  Files          159       159         —
  Lines        21428     21431        +3
  Branches         0         0         —
==========================================
+ Hits         20364     20367        +3
- Misses        1064      1064         —
- Partials         0         0         —

Generated by Codecov Action

Wrap project and docs URLs in terminalLink() so they become clickable
in terminals that support OSC 8 (iTerm2, VS Code, Windows Terminal).
On unsupported terminals, the escape sequences are silently ignored.

Also make the url parameter optional in terminalLink() — when only
the display text is given and it is already the full URL, it doubles
as the link target.
@MathurAditya724 MathurAditya724 marked this pull request as ready for review March 13, 2026 18:41
@MathurAditya724 MathurAditya724 enabled auto-merge (squash) March 13, 2026 18:41
@MathurAditya724 MathurAditya724 merged commit 89b1fcc into main Mar 13, 2026
21 checks passed
@MathurAditya724 MathurAditya724 deleted the fix/init-links branch March 13, 2026 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant