Route chatbot-support mailto through Help WebView log-attach flow

[joashrajin]

Summary

The Ask AI widget on support.pocketcasts.com is moving from mailto:support@pocketcasts.com to mailto:chatbot-support@pocketcasts.com so AI-originated tickets land in a separate inbox.

Today, when a user taps the widget's "contact us here" link inside the in-app Help WebView, HelpWebViewClient intercepts the mailto and routes it through Support.shareLogs(), which builds an Intent.ACTION_SEND with debug.txt attached. The to-address is hardcoded to support@pocketcasts.com, and the allowlist of mailto URLs only contains support@shiftyjelly.com and support@pocketcasts.com. So once the widget switches to chatbot-support@, the WebView would stop recognizing the link, fall through to onTapUri(), and the user would email the chatbot inbox without debug logs attached.

This PR makes the recipient address configurable end-to-end while preserving existing behaviour for every other caller of shareLogs().

Changes

• HelpPage.kt — replaced the prefix-match contactSupportAction list with an exact-recipient Set<String> (supportRecipients) that now includes chatbot-support@pocketcasts.com. Parses the recipient from the matched mailto URL and passes it through. Falls back to onTapUri() for any mailto whose recipient isn't in the set. The WebViewError "Contact support" button (shown when the help page fails to load) explicitly uses support@pocketcasts.com.
• HelpViewModel.getSupportIntent() — accepts an optional recipient: String (default support@pocketcasts.com) and forwards it.
• Support.shareLogs() — adds an optional recipient: String parameter (default support@pocketcasts.com) and uses it for Intent.EXTRA_EMAIL instead of the hardcoded address. Other callers (LogsViewModel, StatusViewModel, the getFeedbackIntent path, and emailWearLogsToSupportIntent) are unaffected via the default.

Security note

The previous contactSupportAction.any { url.startsWith(it) } check matched by prefix, so mailto:support@pocketcasts.com.evil.com would also match. This was harmless before because the to-address was hardcoded — but parsing the recipient from the URL would have made it exploitable. This PR tightens to exact-equality matching against the supportRecipients set.

Test plan

• In the Pocket Casts Android app, open Settings → Help & Feedback → Ask AI.
• Send a message, then tap the "contact us here" link in the assistant's reply.
• Confirm the email composer opens with chatbot-support@pocketcasts.com in the To field and debug.txt attached.
• Tap the regular "Contact support" button on the Help page (not via the widget) and confirm the email composer opens with support@pocketcasts.com and debug.txt attached (regression check).
• In the standalone Logs screen (Settings → Help → ⋮ → View Logs → Share), confirm the email goes to support@pocketcasts.com (regression check).
• Force the Help WebView to fail to load (airplane mode), tap the "Contact support" error button, confirm the email goes to support@pocketcasts.com (regression check).
• Tap a mailto link in the WebView whose recipient is not in supportRecipients and confirm it falls through to the system email composer (no log attachment, no crash).

Related

Coordinated with the support team's move to a dedicated chatbot-support@pocketcasts.com inbox for AI-originated tickets. The widget HTML change (swapping the rendered mailto: href) is being made separately on the support site.

Draft until the widget swap is timed.

[Copilot]

Pull request overview

Updates the in-app Help WebView mailto interception so that “Ask AI” contact links can route to a new chatbot-support@pocketcasts.com inbox while still attaching debug logs, without changing behavior for existing support/log-sharing entry points.

Changes:

• Make the support email recipient configurable through HelpPage -> HelpViewModel -> Support.shareLogs.
• Tighten Help WebView mailto handling to exact-recipient allowlisting (and add chatbot-support@pocketcasts.com).
• Keep the Help-page load error “Contact support” button targeting support@pocketcasts.com.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
modules/services/repositories/src/main/java/au/com/shiftyjelly/pocketcasts/repositories/support/Support.kt	Adds optional recipient parameter to shareLogs() and uses it for Intent.EXTRA_EMAIL.
modules/features/settings/src/main/java/au/com/shiftyjelly/pocketcasts/settings/viewmodel/HelpViewModel.kt	Threads optional recipient through getSupportIntent() to Support.shareLogs().
modules/features/settings/src/main/java/au/com/shiftyjelly/pocketcasts/settings/HelpPage.kt	Parses mailto recipient in HelpWebViewClient, allowlists recipients (including chatbot-support), and passes recipient into support flow.

[joashrajin]

Some context on p1777549631343759/1774527906.406489-slack-C06DGE41ENN