fix(web): open PR toast actions in app

[eimexdev]

What Changed

• Route plain View PR clicks from Git action success toasts to the existing in-app pull request panel.
• Preserve Command/Control-click and unsupported-link fallback behavior that opens the system browser.

Why

The success toast bypassed the in-app viewer by calling shell.openExternal directly. This made creating a pull request or pushing to an existing one unexpectedly leave T3 Code.

Closes#9004.

UI Changes

No visual styling or layout changed. Screenshots are not applicable; only the existing button's destination changes.

Validation

• vp test run apps/web/src/lib/openPullRequestLink.test.ts — 28 passed
• vp fmt --check apps/web/src/components/GitActionsControl.tsx
• vp lint --report-unused-disable-directives apps/web/src/components/GitActionsControl.tsx
• vp run typecheck in apps/web — blocked by existing unrelated origin/main errors; none reference the changed component

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes — not applicable; no visual change
• I included a video for animation/interaction changes — not applicable; no animation or timing change

Model: GPT-5.6-sol
Harness: Codex in T3 Code

---

Note

Low Risk
Navigation-only change in Git toast CTAs; no auth, data, or API surface changes.

Overview
Git action success toasts with an open PR CTA no longer call shell.openExternal directly. They use useOpenPrLink from openPullRequestLink, aligned with other PR links in the app.

A normal click opens the in-app change request panel when possible; Command/Control-click and unsupported cases still fall back to the system browser. The toast button handler now receives the click MouseEvent so that modifier behavior works. The inline readLocalApi() guard in the toast path is dropped because the hook covers it.

^{Reviewed by Cursor Bugbot for commit 7847f88. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Open PR toast action in app via useOpenPrLink

Replaces the direct api.shell.openExternal(url) call in the 'open_pr' toast CTA with openPrLink(event, url) from the useOpenPrLink hook in GitActionsControl.tsx. The explicit readLocalApi() null guard is removed since the hook handles that.

^{Macroscope summarized 7847f88.}

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed2026-09-02T01:35:37.205721Z	7847f88	New commits
🔒 Security Review	✅ Completed2026-09-01T05:19:04.691004Z	6b3b720	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:6b3b720091

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 6b3b720. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved at bdc048d

Macroscope's review found this PR approvable — This is a focused one-file fix that routes PR toast actions through the existing in-app link handler while preserving modifier-click and external fallback behavior. Its runtime impact is limited to the toast navigation path, with no defaults, schemas, infrastructure, or sensitive packages changed.

^{You can add or adjust custom eligibility rules. Learn more.}

[eimexdev]

What Changed

• Route plain View PR clicks from Git action success toasts to the existing in-app pull request panel.
• Preserve Command/Control-click and unsupported-link fallback behavior that opens the system browser.

Why

The success toast bypassed the in-app viewer by calling shell.openExternal directly. This made creating a pull request or pushing to an existing one unexpectedly leave T3 Code.

Closes#9004.

UI Changes

No visual styling or layout changed. Screenshots are not applicable; only the existing button's destination changes.

Validation

• vp test run apps/web/src/lib/openPullRequestLink.test.ts — 28 passed
• vp fmt --check apps/web/src/components/GitActionsControl.tsx
• vp lint --report-unused-disable-directives apps/web/src/components/GitActionsControl.tsx
• vp run typecheck in apps/web — blocked by existing unrelated origin/main errors; none reference the changed component

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes — not applicable; no visual change
• I included a video for animation/interaction changes — not applicable; no animation or timing change

Model: GPT-5.6-sol
Harness: Codex in T3 Code

---

Note

Low Risk
Navigation-only change in Git toast CTAs; no auth, data, or API surface changes.

Overview
Git action success toasts with an open PR CTA no longer call shell.openExternal directly. They use useOpenPrLink from openPullRequestLink, aligned with other PR links in the app.

A normal click opens the in-app change request panel when possible; Command/Control-click and unsupported cases still fall back to the system browser. The toast button handler now receives the click MouseEvent so that modifier behavior works. The inline readLocalApi() guard in the toast path is dropped because the hook covers it.

^{Reviewed by Cursor Bugbot for commit 7847f88. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Open PR toast action in app via useOpenPrLink

Replaces the direct api.shell.openExternal(url) call in the 'open_pr' toast CTA with openPrLink(event, url) from the useOpenPrLink hook in GitActionsControl.tsx. The explicit readLocalApi() null guard is removed since the hook handles that.

^{Macroscope summarized 7847f88.}

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed2026-09-02T01:35:37.205721Z	7847f88	New commits
🔒 Security Review	✅ Completed2026-09-01T05:19:04.691004Z	6b3b720	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:6b3b720091

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 6b3b720. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved at bdc048d

Macroscope's review found this PR approvable — This is a focused one-file fix that routes PR toast actions through the existing in-app link handler while preserving modifier-click and external fallback behavior. Its runtime impact is limited to the toast navigation path, with no defaults, schemas, infrastructure, or sensitive packages changed.

^{You can add or adjust custom eligibility rules. Learn more.}

[eimexdev]

What Changed

• Route plain View PR clicks from Git action success toasts to the existing in-app pull request panel.
• Preserve Command/Control-click and unsupported-link fallback behavior that opens the system browser.

Why

The success toast bypassed the in-app viewer by calling shell.openExternal directly. This made creating a pull request or pushing to an existing one unexpectedly leave T3 Code.

Closes#9004.

UI Changes

No visual styling or layout changed. Screenshots are not applicable; only the existing button's destination changes.

Validation

• vp test run apps/web/src/lib/openPullRequestLink.test.ts — 28 passed
• vp fmt --check apps/web/src/components/GitActionsControl.tsx
• vp lint --report-unused-disable-directives apps/web/src/components/GitActionsControl.tsx
• vp run typecheck in apps/web — blocked by existing unrelated origin/main errors; none reference the changed component

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes — not applicable; no visual change
• I included a video for animation/interaction changes — not applicable; no animation or timing change

Model: GPT-5.6-sol
Harness: Codex in T3 Code

---

Note

Low Risk
Navigation-only change in Git toast CTAs; no auth, data, or API surface changes.

Overview
Git action success toasts with an open PR CTA no longer call shell.openExternal directly. They use useOpenPrLink from openPullRequestLink, aligned with other PR links in the app.

A normal click opens the in-app change request panel when possible; Command/Control-click and unsupported cases still fall back to the system browser. The toast button handler now receives the click MouseEvent so that modifier behavior works. The inline readLocalApi() guard in the toast path is dropped because the hook covers it.

^{Reviewed by Cursor Bugbot for commit 7847f88. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Open PR toast action in app via useOpenPrLink

Replaces the direct api.shell.openExternal(url) call in the 'open_pr' toast CTA with openPrLink(event, url) from the useOpenPrLink hook in GitActionsControl.tsx. The explicit readLocalApi() null guard is removed since the hook handles that.

^{Macroscope summarized 7847f88.}

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed2026-09-02T01:35:37.205721Z	7847f88	New commits
🔒 Security Review	✅ Completed2026-09-01T05:19:04.691004Z	6b3b720	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:6b3b720091

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 6b3b720. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved at bdc048d

Macroscope's review found this PR approvable — This is a focused one-file fix that routes PR toast actions through the existing in-app link handler while preserving modifier-click and external fallback behavior. Its runtime impact is limited to the toast navigation path, with no defaults, schemas, infrastructure, or sensitive packages changed.

^{You can add or adjust custom eligibility rules. Learn more.}

[eimexdev]

What Changed

• Route plain View PR clicks from Git action success toasts to the existing in-app pull request panel.
• Preserve Command/Control-click and unsupported-link fallback behavior that opens the system browser.

Why

The success toast bypassed the in-app viewer by calling shell.openExternal directly. This made creating a pull request or pushing to an existing one unexpectedly leave T3 Code.

Closes#9004.

UI Changes

No visual styling or layout changed. Screenshots are not applicable; only the existing button's destination changes.

Validation

• vp test run apps/web/src/lib/openPullRequestLink.test.ts — 28 passed
• vp fmt --check apps/web/src/components/GitActionsControl.tsx
• vp lint --report-unused-disable-directives apps/web/src/components/GitActionsControl.tsx
• vp run typecheck in apps/web — blocked by existing unrelated origin/main errors; none reference the changed component

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes — not applicable; no visual change
• I included a video for animation/interaction changes — not applicable; no animation or timing change

Model: GPT-5.6-sol
Harness: Codex in T3 Code

---

Note

Low Risk
Navigation-only change in Git toast CTAs; no auth, data, or API surface changes.

Overview
Git action success toasts with an open PR CTA no longer call shell.openExternal directly. They use useOpenPrLink from openPullRequestLink, aligned with other PR links in the app.

A normal click opens the in-app change request panel when possible; Command/Control-click and unsupported cases still fall back to the system browser. The toast button handler now receives the click MouseEvent so that modifier behavior works. The inline readLocalApi() guard in the toast path is dropped because the hook covers it.

^{Reviewed by Cursor Bugbot for commit 7847f88. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Open PR toast action in app via useOpenPrLink

Replaces the direct api.shell.openExternal(url) call in the 'open_pr' toast CTA with openPrLink(event, url) from the useOpenPrLink hook in GitActionsControl.tsx. The explicit readLocalApi() null guard is removed since the hook handles that.

^{Macroscope summarized 7847f88.}

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed2026-09-02T01:35:37.205721Z	7847f88	New commits
🔒 Security Review	✅ Completed2026-09-01T05:19:04.691004Z	6b3b720	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:6b3b720091

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 6b3b720. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved at bdc048d

Macroscope's review found this PR approvable — This is a focused one-file fix that routes PR toast actions through the existing in-app link handler while preserving modifier-click and external fallback behavior. Its runtime impact is limited to the toast navigation path, with no defaults, schemas, infrastructure, or sensitive packages changed.

^{You can add or adjust custom eligibility rules. Learn more.}

[eimexdev]

What Changed

• Route plain View PR clicks from Git action success toasts to the existing in-app pull request panel.
• Preserve Command/Control-click and unsupported-link fallback behavior that opens the system browser.

Why

The success toast bypassed the in-app viewer by calling shell.openExternal directly. This made creating a pull request or pushing to an existing one unexpectedly leave T3 Code.

Closes#9004.

UI Changes

No visual styling or layout changed. Screenshots are not applicable; only the existing button's destination changes.

Validation

• vp test run apps/web/src/lib/openPullRequestLink.test.ts — 28 passed
• vp fmt --check apps/web/src/components/GitActionsControl.tsx
• vp lint --report-unused-disable-directives apps/web/src/components/GitActionsControl.tsx
• vp run typecheck in apps/web — blocked by existing unrelated origin/main errors; none reference the changed component

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes — not applicable; no visual change
• I included a video for animation/interaction changes — not applicable; no animation or timing change

Model: GPT-5.6-sol
Harness: Codex in T3 Code

---

Note

Low Risk
Navigation-only change in Git toast CTAs; no auth, data, or API surface changes.

Overview
Git action success toasts with an open PR CTA no longer call shell.openExternal directly. They use useOpenPrLink from openPullRequestLink, aligned with other PR links in the app.

A normal click opens the in-app change request panel when possible; Command/Control-click and unsupported cases still fall back to the system browser. The toast button handler now receives the click MouseEvent so that modifier behavior works. The inline readLocalApi() guard in the toast path is dropped because the hook covers it.

^{Reviewed by Cursor Bugbot for commit 7847f88. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Open PR toast action in app via useOpenPrLink

Replaces the direct api.shell.openExternal(url) call in the 'open_pr' toast CTA with openPrLink(event, url) from the useOpenPrLink hook in GitActionsControl.tsx. The explicit readLocalApi() null guard is removed since the hook handles that.

^{Macroscope summarized 7847f88.}

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed2026-09-02T01:35:37.205721Z	7847f88	New commits
🔒 Security Review	✅ Completed2026-09-01T05:19:04.691004Z	6b3b720	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:6b3b720091

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 6b3b720. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved at bdc048d

Macroscope's review found this PR approvable — This is a focused one-file fix that routes PR toast actions through the existing in-app link handler while preserving modifier-click and external fallback behavior. Its runtime impact is limited to the toast navigation path, with no defaults, schemas, infrastructure, or sensitive packages changed.

^{You can add or adjust custom eligibility rules. Learn more.}

[eimexdev]

What Changed

• Route plain View PR clicks from Git action success toasts to the existing in-app pull request panel.
• Preserve Command/Control-click and unsupported-link fallback behavior that opens the system browser.

Why

The success toast bypassed the in-app viewer by calling shell.openExternal directly. This made creating a pull request or pushing to an existing one unexpectedly leave T3 Code.

Closes#9004.

UI Changes

No visual styling or layout changed. Screenshots are not applicable; only the existing button's destination changes.

Validation

• vp test run apps/web/src/lib/openPullRequestLink.test.ts — 28 passed
• vp fmt --check apps/web/src/components/GitActionsControl.tsx
• vp lint --report-unused-disable-directives apps/web/src/components/GitActionsControl.tsx
• vp run typecheck in apps/web — blocked by existing unrelated origin/main errors; none reference the changed component

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes — not applicable; no visual change
• I included a video for animation/interaction changes — not applicable; no animation or timing change

Model: GPT-5.6-sol
Harness: Codex in T3 Code

---

Note

Low Risk
Navigation-only change in Git toast CTAs; no auth, data, or API surface changes.

Overview
Git action success toasts with an open PR CTA no longer call shell.openExternal directly. They use useOpenPrLink from openPullRequestLink, aligned with other PR links in the app.

A normal click opens the in-app change request panel when possible; Command/Control-click and unsupported cases still fall back to the system browser. The toast button handler now receives the click MouseEvent so that modifier behavior works. The inline readLocalApi() guard in the toast path is dropped because the hook covers it.

^{Reviewed by Cursor Bugbot for commit 7847f88. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Open PR toast action in app via useOpenPrLink

Replaces the direct api.shell.openExternal(url) call in the 'open_pr' toast CTA with openPrLink(event, url) from the useOpenPrLink hook in GitActionsControl.tsx. The explicit readLocalApi() null guard is removed since the hook handles that.

^{Macroscope summarized 7847f88.}

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed2026-09-02T01:35:37.205721Z	7847f88	New commits
🔒 Security Review	✅ Completed2026-09-01T05:19:04.691004Z	6b3b720	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:6b3b720091

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 6b3b720. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved at bdc048d

Macroscope's review found this PR approvable — This is a focused one-file fix that routes PR toast actions through the existing in-app link handler while preserving modifier-click and external fallback behavior. Its runtime impact is limited to the toast navigation path, with no defaults, schemas, infrastructure, or sensitive packages changed.

^{You can add or adjust custom eligibility rules. Learn more.}

[eimexdev]

What Changed

• Route plain View PR clicks from Git action success toasts to the existing in-app pull request panel.
• Preserve Command/Control-click and unsupported-link fallback behavior that opens the system browser.

Why

The success toast bypassed the in-app viewer by calling shell.openExternal directly. This made creating a pull request or pushing to an existing one unexpectedly leave T3 Code.

Closes#9004.

UI Changes

No visual styling or layout changed. Screenshots are not applicable; only the existing button's destination changes.

Validation

• vp test run apps/web/src/lib/openPullRequestLink.test.ts — 28 passed
• vp fmt --check apps/web/src/components/GitActionsControl.tsx
• vp lint --report-unused-disable-directives apps/web/src/components/GitActionsControl.tsx
• vp run typecheck in apps/web — blocked by existing unrelated origin/main errors; none reference the changed component

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes — not applicable; no visual change
• I included a video for animation/interaction changes — not applicable; no animation or timing change

Model: GPT-5.6-sol
Harness: Codex in T3 Code

---

Note

Low Risk
Navigation-only change in Git toast CTAs; no auth, data, or API surface changes.

Overview
Git action success toasts with an open PR CTA no longer call shell.openExternal directly. They use useOpenPrLink from openPullRequestLink, aligned with other PR links in the app.

A normal click opens the in-app change request panel when possible; Command/Control-click and unsupported cases still fall back to the system browser. The toast button handler now receives the click MouseEvent so that modifier behavior works. The inline readLocalApi() guard in the toast path is dropped because the hook covers it.

^{Reviewed by Cursor Bugbot for commit 7847f88. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Open PR toast action in app via useOpenPrLink

Replaces the direct api.shell.openExternal(url) call in the 'open_pr' toast CTA with openPrLink(event, url) from the useOpenPrLink hook in GitActionsControl.tsx. The explicit readLocalApi() null guard is removed since the hook handles that.

^{Macroscope summarized 7847f88.}

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed2026-09-02T01:35:37.205721Z	7847f88	New commits
🔒 Security Review	✅ Completed2026-09-01T05:19:04.691004Z	6b3b720	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:6b3b720091

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 6b3b720. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved at bdc048d

Macroscope's review found this PR approvable — This is a focused one-file fix that routes PR toast actions through the existing in-app link handler while preserving modifier-click and external fallback behavior. Its runtime impact is limited to the toast navigation path, with no defaults, schemas, infrastructure, or sensitive packages changed.

^{You can add or adjust custom eligibility rules. Learn more.}

[eimexdev]

What Changed

• Route plain View PR clicks from Git action success toasts to the existing in-app pull request panel.
• Preserve Command/Control-click and unsupported-link fallback behavior that opens the system browser.

Why

The success toast bypassed the in-app viewer by calling shell.openExternal directly. This made creating a pull request or pushing to an existing one unexpectedly leave T3 Code.

Closes#9004.

UI Changes

No visual styling or layout changed. Screenshots are not applicable; only the existing button's destination changes.

Validation

• vp test run apps/web/src/lib/openPullRequestLink.test.ts — 28 passed
• vp fmt --check apps/web/src/components/GitActionsControl.tsx
• vp lint --report-unused-disable-directives apps/web/src/components/GitActionsControl.tsx
• vp run typecheck in apps/web — blocked by existing unrelated origin/main errors; none reference the changed component

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes — not applicable; no visual change
• I included a video for animation/interaction changes — not applicable; no animation or timing change

Model: GPT-5.6-sol
Harness: Codex in T3 Code

---

Note

Low Risk
Navigation-only change in Git toast CTAs; no auth, data, or API surface changes.

Overview
Git action success toasts with an open PR CTA no longer call shell.openExternal directly. They use useOpenPrLink from openPullRequestLink, aligned with other PR links in the app.

A normal click opens the in-app change request panel when possible; Command/Control-click and unsupported cases still fall back to the system browser. The toast button handler now receives the click MouseEvent so that modifier behavior works. The inline readLocalApi() guard in the toast path is dropped because the hook covers it.

^{Reviewed by Cursor Bugbot for commit 7847f88. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Open PR toast action in app via useOpenPrLink

Replaces the direct api.shell.openExternal(url) call in the 'open_pr' toast CTA with openPrLink(event, url) from the useOpenPrLink hook in GitActionsControl.tsx. The explicit readLocalApi() null guard is removed since the hook handles that.

^{Macroscope summarized 7847f88.}

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed2026-09-02T01:35:37.205721Z	7847f88	New commits
🔒 Security Review	✅ Completed2026-09-01T05:19:04.691004Z	6b3b720	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:6b3b720091

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 6b3b720. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved at bdc048d

Macroscope's review found this PR approvable — This is a focused one-file fix that routes PR toast actions through the existing in-app link handler while preserving modifier-click and external fallback behavior. Its runtime impact is limited to the toast navigation path, with no defaults, schemas, infrastructure, or sensitive packages changed.

^{You can add or adjust custom eligibility rules. Learn more.}