Skip to content

chore: remove dependabot-dedupe workflow#934

Merged
layershifter merged 1 commit intomicrosoft:mainfrom
layershifter:chore/remove-dependabot-dedupe
May 6, 2026
Merged

chore: remove dependabot-dedupe workflow#934
layershifter merged 1 commit intomicrosoft:mainfrom
layershifter:chore/remove-dependabot-dedupe

Conversation

@layershifter
Copy link
Copy Markdown
Member

@layershifter layershifter commented May 6, 2026

Summary

Removes .github/workflows/dependabot-dedupe.yml.

Why

The workflow uses a pull_request_target + actions/checkout(ref: github.head_ref) + push-back pattern. This is the canonical "untrusted code runs in privileged context" anti-pattern that GitHub Actions security guidance discourages.

🤖 Generated with Claude Code

Removes .github/workflows/dependabot-dedupe.yml. The workflow uses a
pull_request_target + checkout(head_ref) + push pattern that is the
canonical "untrusted code in privileged context" anti-pattern.

The auto-dedupe behavior the workflow provided is a convenience —
ci.yml already runs `yarn dedupe --check` on every PR, so any
un-deduped lockfile already blocks merge. The only difference after
this change is that a maintainer manually runs `yarn dedupe` on the
rare dependabot PR that needs it (most weekly bumps don't).

This matches the same removal in microsoft/keyborg (microsoft#167) and
microsoft/monosize.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@layershifter layershifter requested a review from a team as a code owner May 6, 2026 19:30
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

📊 Bundle size report

✅ No changes found

@layershifter layershifter enabled auto-merge (squash) May 6, 2026 19:42
@layershifter layershifter merged commit 03e5c10 into microsoft:main May 6, 2026
5 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.

2 participants