Skip to content

feat(web): preview pull request links - #9631

Merged
maria-rcks merged 3 commits into
pingdotgg:mainfrom
maria-rcks:t3code/preview-pull-request-links
Sep 4, 2026
Merged

maria-rcks merged 3 commits into
pingdotgg:mainfrom
maria-rcks:t3code/preview-pull-request-links

chore: merge main into pull request previews

d4e4ab1
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Correctness Check succeeded Sep 4, 2026 in 1m 46s

No issues identified (11 code objects reviewed).

• Merge Base: c75299e
• Head: d4e4ab1

Details

✅ File Path U3 Bytes Comments Posted Reason
✅ apps/web/src/components/ui/preview-card.tsx 1546 0
✅ apps/web/src/components/pullRequest/PullRequestLinkPreview.tsx 4814 0
➖ apps/web/src/lib/openPullRequestLink.test.ts 1166 Excluded by default ignore patterns
✅ apps/web/src/lib/openPullRequestLink.ts 2836 0
✅ apps/web/src/components/ChatMarkdown.tsx 5082 0

Billed Total: 12.29KB of diff | $0.61
Saved $0.10 vs. the upper-bound estimate of $0.71 — cached code objects from prior reviews on this PR were not re-billed.

Filtered Issues Details

apps/web/src/components/ChatMarkdown.tsx
  • line 2123: openDeferredMarkdownLink is passed as the fallback for confirmed reference autolinks, but useOpenLink throws when readLocalApi() is unavailable. PullRequestLinkPreview has already prevented the anchor's default navigation before it calls that fallback and catches the rejection without restoring navigation. Thus, in a normal browser tab, a GitHub #123 reference for an issue (rather than a PR) in a matched project becomes a no-op instead of opening its original URL. [ Previously rejected ]