fix: handle markdown links failure correctly

[mohamedmastouri-hue]

This PR replaces raw calls to Linking.openURL with tryOpenExternalUrl to properly handle links with unsupported schemes, as described in #5839.

Fixes#5839.

---

Note

Low Risk
Narrow change to link-open behavior in thread markdown UI with no auth, data, or persistence impact.

Overview
Markdown link taps in the thread feed no longer call Linking.openURL directly. External links in MarkdownExternalLink, generic link renderers in useMarkdownStyles, and onMarkdownLinkPress now use tryOpenExternalUrl(..., "markdown-link"), matching the shared helper used elsewhere on mobile.

That helper still opens URLs via Linking, but catches failures (e.g. unsupported schemes), logs structured ExternalUrlOpenError metadata, and returns false instead of surfacing an unhandled rejection—addressing inconsistent behavior when markdown contains bad or non-http(s) links.

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

Note

Fix markdown link failures by routing through tryOpenExternalUrl

Replaces direct Linking.openURL calls with tryOpenExternalUrl (tagged "markdown-link") in MarkdownExternalLink, useMarkdownStyles, and ThreadFeed so that URL open failures are handled correctly rather than silently ignored.

^{Macroscope summarized 5d9030a.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6179f082-0a3c-48cd-8838-faad20b13222

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[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 2a41326e45b0e3e56b39b11e7f117b10e91ce2cf. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved 5d9030a

Simple bug fix that wraps 3 Linking.openURL() calls with an existing tryOpenExternalUrl() helper to handle failures gracefully. The helper already exists in the codebase and adds try-catch error handling - no new behavior introduced.

^{You can customize Macroscope's approvability policy. Learn more.}

[juliusmarminge]

Closing as superseded by #5872, which has merged and covers this change on the current architecture. Thanks for the contribution.

[mohamedmastouri-hue]

This PR replaces raw calls to Linking.openURL with tryOpenExternalUrl to properly handle links with unsupported schemes, as described in #5839.

Fixes#5839.

---

Note

Low Risk
Narrow change to link-open behavior in thread markdown UI with no auth, data, or persistence impact.

Overview
Markdown link taps in the thread feed no longer call Linking.openURL directly. External links in MarkdownExternalLink, generic link renderers in useMarkdownStyles, and onMarkdownLinkPress now use tryOpenExternalUrl(..., "markdown-link"), matching the shared helper used elsewhere on mobile.

That helper still opens URLs via Linking, but catches failures (e.g. unsupported schemes), logs structured ExternalUrlOpenError metadata, and returns false instead of surfacing an unhandled rejection—addressing inconsistent behavior when markdown contains bad or non-http(s) links.

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

Note

Fix markdown link failures by routing through tryOpenExternalUrl

Replaces direct Linking.openURL calls with tryOpenExternalUrl (tagged "markdown-link") in MarkdownExternalLink, useMarkdownStyles, and ThreadFeed so that URL open failures are handled correctly rather than silently ignored.

^{Macroscope summarized 5d9030a.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6179f082-0a3c-48cd-8838-faad20b13222

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[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 2a41326e45b0e3e56b39b11e7f117b10e91ce2cf. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved 5d9030a

Simple bug fix that wraps 3 Linking.openURL() calls with an existing tryOpenExternalUrl() helper to handle failures gracefully. The helper already exists in the codebase and adds try-catch error handling - no new behavior introduced.

^{You can customize Macroscope's approvability policy. Learn more.}

[juliusmarminge]

Closing as superseded by #5872, which has merged and covers this change on the current architecture. Thanks for the contribution.

[mohamedmastouri-hue]

This PR replaces raw calls to Linking.openURL with tryOpenExternalUrl to properly handle links with unsupported schemes, as described in #5839.

Fixes#5839.

---

Note

Low Risk
Narrow change to link-open behavior in thread markdown UI with no auth, data, or persistence impact.

Overview
Markdown link taps in the thread feed no longer call Linking.openURL directly. External links in MarkdownExternalLink, generic link renderers in useMarkdownStyles, and onMarkdownLinkPress now use tryOpenExternalUrl(..., "markdown-link"), matching the shared helper used elsewhere on mobile.

That helper still opens URLs via Linking, but catches failures (e.g. unsupported schemes), logs structured ExternalUrlOpenError metadata, and returns false instead of surfacing an unhandled rejection—addressing inconsistent behavior when markdown contains bad or non-http(s) links.

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

Note

Fix markdown link failures by routing through tryOpenExternalUrl

Replaces direct Linking.openURL calls with tryOpenExternalUrl (tagged "markdown-link") in MarkdownExternalLink, useMarkdownStyles, and ThreadFeed so that URL open failures are handled correctly rather than silently ignored.

^{Macroscope summarized 5d9030a.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6179f082-0a3c-48cd-8838-faad20b13222

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[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 2a41326e45b0e3e56b39b11e7f117b10e91ce2cf. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved 5d9030a

Simple bug fix that wraps 3 Linking.openURL() calls with an existing tryOpenExternalUrl() helper to handle failures gracefully. The helper already exists in the codebase and adds try-catch error handling - no new behavior introduced.

^{You can customize Macroscope's approvability policy. Learn more.}

[juliusmarminge]

Closing as superseded by #5872, which has merged and covers this change on the current architecture. Thanks for the contribution.

[mohamedmastouri-hue]

This PR replaces raw calls to Linking.openURL with tryOpenExternalUrl to properly handle links with unsupported schemes, as described in #5839.

Fixes#5839.

---

Note

Low Risk
Narrow change to link-open behavior in thread markdown UI with no auth, data, or persistence impact.

Overview
Markdown link taps in the thread feed no longer call Linking.openURL directly. External links in MarkdownExternalLink, generic link renderers in useMarkdownStyles, and onMarkdownLinkPress now use tryOpenExternalUrl(..., "markdown-link"), matching the shared helper used elsewhere on mobile.

That helper still opens URLs via Linking, but catches failures (e.g. unsupported schemes), logs structured ExternalUrlOpenError metadata, and returns false instead of surfacing an unhandled rejection—addressing inconsistent behavior when markdown contains bad or non-http(s) links.

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

Note

Fix markdown link failures by routing through tryOpenExternalUrl

Replaces direct Linking.openURL calls with tryOpenExternalUrl (tagged "markdown-link") in MarkdownExternalLink, useMarkdownStyles, and ThreadFeed so that URL open failures are handled correctly rather than silently ignored.

^{Macroscope summarized 5d9030a.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6179f082-0a3c-48cd-8838-faad20b13222

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[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 2a41326e45b0e3e56b39b11e7f117b10e91ce2cf. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved 5d9030a

Simple bug fix that wraps 3 Linking.openURL() calls with an existing tryOpenExternalUrl() helper to handle failures gracefully. The helper already exists in the codebase and adds try-catch error handling - no new behavior introduced.

^{You can customize Macroscope's approvability policy. Learn more.}

[juliusmarminge]

Closing as superseded by #5872, which has merged and covers this change on the current architecture. Thanks for the contribution.

[mohamedmastouri-hue]

This PR replaces raw calls to Linking.openURL with tryOpenExternalUrl to properly handle links with unsupported schemes, as described in #5839.

Fixes#5839.

---

Note

Low Risk
Narrow change to link-open behavior in thread markdown UI with no auth, data, or persistence impact.

Overview
Markdown link taps in the thread feed no longer call Linking.openURL directly. External links in MarkdownExternalLink, generic link renderers in useMarkdownStyles, and onMarkdownLinkPress now use tryOpenExternalUrl(..., "markdown-link"), matching the shared helper used elsewhere on mobile.

That helper still opens URLs via Linking, but catches failures (e.g. unsupported schemes), logs structured ExternalUrlOpenError metadata, and returns false instead of surfacing an unhandled rejection—addressing inconsistent behavior when markdown contains bad or non-http(s) links.

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

Note

Fix markdown link failures by routing through tryOpenExternalUrl

Replaces direct Linking.openURL calls with tryOpenExternalUrl (tagged "markdown-link") in MarkdownExternalLink, useMarkdownStyles, and ThreadFeed so that URL open failures are handled correctly rather than silently ignored.

^{Macroscope summarized 5d9030a.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6179f082-0a3c-48cd-8838-faad20b13222

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[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 2a41326e45b0e3e56b39b11e7f117b10e91ce2cf. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved 5d9030a

Simple bug fix that wraps 3 Linking.openURL() calls with an existing tryOpenExternalUrl() helper to handle failures gracefully. The helper already exists in the codebase and adds try-catch error handling - no new behavior introduced.

^{You can customize Macroscope's approvability policy. Learn more.}

[juliusmarminge]

Closing as superseded by #5872, which has merged and covers this change on the current architecture. Thanks for the contribution.

[mohamedmastouri-hue]

This PR replaces raw calls to Linking.openURL with tryOpenExternalUrl to properly handle links with unsupported schemes, as described in #5839.

Fixes#5839.

---

Note

Low Risk
Narrow change to link-open behavior in thread markdown UI with no auth, data, or persistence impact.

Overview
Markdown link taps in the thread feed no longer call Linking.openURL directly. External links in MarkdownExternalLink, generic link renderers in useMarkdownStyles, and onMarkdownLinkPress now use tryOpenExternalUrl(..., "markdown-link"), matching the shared helper used elsewhere on mobile.

That helper still opens URLs via Linking, but catches failures (e.g. unsupported schemes), logs structured ExternalUrlOpenError metadata, and returns false instead of surfacing an unhandled rejection—addressing inconsistent behavior when markdown contains bad or non-http(s) links.

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

Note

Fix markdown link failures by routing through tryOpenExternalUrl

Replaces direct Linking.openURL calls with tryOpenExternalUrl (tagged "markdown-link") in MarkdownExternalLink, useMarkdownStyles, and ThreadFeed so that URL open failures are handled correctly rather than silently ignored.

^{Macroscope summarized 5d9030a.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6179f082-0a3c-48cd-8838-faad20b13222

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[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 2a41326e45b0e3e56b39b11e7f117b10e91ce2cf. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved 5d9030a

Simple bug fix that wraps 3 Linking.openURL() calls with an existing tryOpenExternalUrl() helper to handle failures gracefully. The helper already exists in the codebase and adds try-catch error handling - no new behavior introduced.

^{You can customize Macroscope's approvability policy. Learn more.}

[juliusmarminge]

Closing as superseded by #5872, which has merged and covers this change on the current architecture. Thanks for the contribution.

[mohamedmastouri-hue]

This PR replaces raw calls to Linking.openURL with tryOpenExternalUrl to properly handle links with unsupported schemes, as described in #5839.

Fixes#5839.

---

Note

Low Risk
Narrow change to link-open behavior in thread markdown UI with no auth, data, or persistence impact.

Overview
Markdown link taps in the thread feed no longer call Linking.openURL directly. External links in MarkdownExternalLink, generic link renderers in useMarkdownStyles, and onMarkdownLinkPress now use tryOpenExternalUrl(..., "markdown-link"), matching the shared helper used elsewhere on mobile.

That helper still opens URLs via Linking, but catches failures (e.g. unsupported schemes), logs structured ExternalUrlOpenError metadata, and returns false instead of surfacing an unhandled rejection—addressing inconsistent behavior when markdown contains bad or non-http(s) links.

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

Note

Fix markdown link failures by routing through tryOpenExternalUrl

Replaces direct Linking.openURL calls with tryOpenExternalUrl (tagged "markdown-link") in MarkdownExternalLink, useMarkdownStyles, and ThreadFeed so that URL open failures are handled correctly rather than silently ignored.

^{Macroscope summarized 5d9030a.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6179f082-0a3c-48cd-8838-faad20b13222

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[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 2a41326e45b0e3e56b39b11e7f117b10e91ce2cf. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved 5d9030a

Simple bug fix that wraps 3 Linking.openURL() calls with an existing tryOpenExternalUrl() helper to handle failures gracefully. The helper already exists in the codebase and adds try-catch error handling - no new behavior introduced.

^{You can customize Macroscope's approvability policy. Learn more.}

[juliusmarminge]

Closing as superseded by #5872, which has merged and covers this change on the current architecture. Thanks for the contribution.

[mohamedmastouri-hue]

This PR replaces raw calls to Linking.openURL with tryOpenExternalUrl to properly handle links with unsupported schemes, as described in #5839.

Fixes#5839.

---

Note

Low Risk
Narrow change to link-open behavior in thread markdown UI with no auth, data, or persistence impact.

Overview
Markdown link taps in the thread feed no longer call Linking.openURL directly. External links in MarkdownExternalLink, generic link renderers in useMarkdownStyles, and onMarkdownLinkPress now use tryOpenExternalUrl(..., "markdown-link"), matching the shared helper used elsewhere on mobile.

That helper still opens URLs via Linking, but catches failures (e.g. unsupported schemes), logs structured ExternalUrlOpenError metadata, and returns false instead of surfacing an unhandled rejection—addressing inconsistent behavior when markdown contains bad or non-http(s) links.

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

Note

Fix markdown link failures by routing through tryOpenExternalUrl

Replaces direct Linking.openURL calls with tryOpenExternalUrl (tagged "markdown-link") in MarkdownExternalLink, useMarkdownStyles, and ThreadFeed so that URL open failures are handled correctly rather than silently ignored.

^{Macroscope summarized 5d9030a.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6179f082-0a3c-48cd-8838-faad20b13222

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[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 2a41326e45b0e3e56b39b11e7f117b10e91ce2cf. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved 5d9030a

Simple bug fix that wraps 3 Linking.openURL() calls with an existing tryOpenExternalUrl() helper to handle failures gracefully. The helper already exists in the codebase and adds try-catch error handling - no new behavior introduced.

^{You can customize Macroscope's approvability policy. Learn more.}

[juliusmarminge]

Closing as superseded by #5872, which has merged and covers this change on the current architecture. Thanks for the contribution.