[PM-32685] enhancements for testing

[mpbw2]

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-32685

📔 Objective

• Added support for testing specific commit hash from workflow dispatch
• Test messages are posted to the automations test channel and exclude pings to any slack users
• replaced android with ${REPO_NAME} for cleaner copypasta

📸 Screenshots

[github-actions]

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

Reviewed a CI-only change to .github/workflows/send_slack_notification.yml that adds a workflow_dispatch input to build test notifications from a specific commit SHA, routes dispatch notifications to a test Slack channel while stripping user pings, and generalizes hardcoded repo references to ${REPO_NAME}. Untrusted workflow_dispatch input is passed via an environment variable (INPUT_COMMIT) rather than inlined into the run script, and shell variables are properly quoted, avoiding script injection. The added contents: read permission is appropriately scoped for the new gh api commit lookup. The prior reviewer suggestion to fetch commit message and author in a single API call has already been addressed by the latest commit.

Code Review Details

No findings. The change is well-scoped and follows safe GitHub Actions patterns.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.94%. Comparing base (5c13e26) to head (f2788b6).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7168      +/-   ##
==========================================
+ Coverage   85.69%   85.94%   +0.24%     
==========================================
  Files         941     1124     +183     
  Lines       67045    68200    +1155     
  Branches     9660     9799     +139     
==========================================
+ Hits        57454    58612    +1158     
+ Misses       6137     6065      -72     
- Partials     3454     3523      +69     

Flag	Coverage Δ
app-data	18.90% <ø> (+1.61%)	⬆️
app-ui-auth-tools	19.02% <ø> (+0.21%)	⬆️
app-ui-platform	17.10% <ø> (+0.53%)	⬆️
app-ui-vault	27.87% <ø> (-0.24%)	⬇️
authenticator	6.13% <ø> (+0.06%)	⬆️
lib-core-network-bridge	4.03% <ø> (-0.97%)	⬇️
lib-data-ui	1.15% <ø> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[vvolkgang]

Small ⛏️, not a blocker, everything else looks great!

[vvolkgang]

⛏️ we could fetch both message and author in a single call.

[Copilot]

Pull request overview

Updates the Slack notification workflow to better support manual “test” runs by allowing a specific commit SHA to be targeted and routing those notifications to a dedicated test channel without user pings.

Changes:

• Added workflow_dispatch input for a commit SHA and logic to fetch commit metadata via gh api for manual runs.
• Adjusted Slack message formatting to use ${REPO_NAME} and avoid Slack mentions during manual dispatch.
• Added retrieval of an additional Slack webhook secret and conditionally selects which webhook to use.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[vvolkgang]