chore(e2e): fix tsio reporting link and Do not post master report to release report channel

[yasserfaraazkhan]

fix tsio reporting link and Do not post master report to release report channel

NONE

Change Impact: 🟡 Medium

Regression Risk: Changes update shared Mattermost webhook routing/skip behavior across E2E/TSIO/CMT/master reporting, including a fail-closed path when dedicated webhook secrets are missing. This is limited to notifications/report publishing (not core product logic), but misrouting could affect multiple workflows and external channel updates. Unit tests were updated to match the new routing rules.
QA Recommendation: Skip full manual QA; spot-check at least one e2e run per report group (desktop-pr, desktop-master, cmt-desktop) to verify the correct webhook/channel receives (or omits) notifications when relevant webhook secrets are set vs unset.
Generated by CodeRabbitAI

[coderabbitai]

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

• @coderabbitai resume to resume automatic reviews.
• @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

• ▶️ Resume reviews
• 🔍 Trigger review

📝 Walkthrough

Walkthrough

Changes

The change separates Mattermost routing for CMT/RC, desktop master, and desktop PR reports, makes named routes fail closed when dedicated webhooks are missing, and enables staging TSIO reporting in E2E workflows.

Mattermost and TSIO workflow updates

Layer / File(s)	Summary
Fail-closed webhook routing and tests e2e/utils/cmt-channel-notify.js, e2e/utils/cmt-channel-notify.test.js	Named report groups use dedicated webhooks and return an empty URL when those variables are missing; unknown names retain shared-webhook fallback.
Workflow webhook and staging wiring .github/workflows/*.yml	E2E and compatibility-matrix workflows use separated webhook variables and staging-enabled TSIO reporting and upload actions.
TSIO routing documentation e2e/utils/tsio-report-status.js	Inline documentation lists separate webhook mappings for CMT/RC, desktop master, and desktop PR reports.

Estimated code review effort: 2 (Simple) | ~15 minutes

Possibly related PRs

• mattermost/desktop#3902: Updates the same channel-notification webhook routing behavior and tests.
• mattermost/desktop#3916: Modifies TSIO report-upload action usage in the E2E workflows.

Suggested labels: kind/chore, github_actions

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title matches the main changes: fixing TSIO reporting and preventing master reports from going to the release report channel.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch update_report_channel_webhook_variable

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@e2e/utils/cmt-channel-notify.js`:
- Around line 101-105: Update the webhook selection logic for cmt-desktop,
desktop-master, and desktop-pr so each named report returns only its dedicated
environment variable, or an empty value when that secret is missing; remove the
MATTERMOST_WEBHOOK_URL fallback for these branches. Add tests covering each
dedicated secret being absent while the shared secret is populated, asserting no
shared webhook is returned.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 43818a95-949b-4b78-b0d1-ef30772cf971

📥 Commits

Reviewing files that changed from the base of the PR and between 687f93f and d803d14.

📒 Files selected for processing (5)

• .github/workflows/compatibility-matrix-testing.yml
• .github/workflows/e2e-functional.yml
• e2e/utils/cmt-channel-notify.js
• e2e/utils/cmt-channel-notify.test.js
• e2e/utils/tsio-report-status.js

💤 Files with no reviewable changes (1)

• .github/workflows/compatibility-matrix-testing.yml