Skip to content

ci: skip Cross-Browser Tests on Dependabot PRs - #5914

Merged
vivekchand merged 1 commit into
mainfrom
fix/browserstack-skip-dependabot
Sep 12, 2026
Merged

ci: skip Cross-Browser Tests on Dependabot PRs#5914
vivekchand merged 1 commit into
mainfrom
fix/browserstack-skip-dependabot

Conversation

@vivekchand

Copy link
Copy Markdown
Owner

Problem

Every Dependabot PR fails the E2E Gate because BrowserStack secrets (BROWSERSTACK_USERNAME, BROWSERSTACK_ACCESS_KEY) are not passed to Dependabot workflow runs. GitHub's security model intentionally withholds repository secrets from Dependabot. The browserstack/github-actions/setup-env action then fails immediately:

Action Failed: Input required and not supplied: username
Action Failed: BROWSERSTACK_ACCESS_KEY not found.

Observed failing on PRs #5909 and #5910 (and presumably #5911, #5912).

Fix

Add a job-level if: github.actor != 'dependabot[bot]' condition to the cross-browser job. The job is skipped entirely for Dependabot PRs. BrowserStack coverage is not relevant to lock-file bumps and CI scanner requirement pins.

What is not changed

  • Behaviour on human PRs with the browserstack label: unchanged
  • Behaviour on merges to main: unchanged
  • All BrowserStack steps, pinned action SHAs, and local tunnel config: unchanged

No-PRD: CI infrastructure fix, no product behaviour change.

🤖 Generated with Claude Code

https://claude.ai/code/session_01LAkDSjSE565ZtADVdfRnie


Generated by Claude Code

Dependabot PRs run in a restricted GitHub Actions context where
repository secrets (BROWSERSTACK_USERNAME, BROWSERSTACK_ACCESS_KEY)
are not passed. The setup-env action fails with "Input required and
not supplied: username", failing the E2E Gate on every Dependabot PR.

Add a job-level condition to skip the entire cross-browser job when
the actor is dependabot[bot]. The test coverage provided by
BrowserStack is not relevant to dependency bumps in lock files and
CI scanner requirements.

No-PRD: CI infrastructure fix, no product behaviour change.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LAkDSjSE565ZtADVdfRnie
@8090-software-factory

Copy link
Copy Markdown

✅ Drift Bot (ClawMetry): no drift detected

Drift Bot analyzed the changed files against this project's blueprints and requirements and found no drift.

@vivekchand
vivekchand merged commit c045a57 into main Sep 12, 2026
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant