Skip to content

ci: remove pull_request_target trigger from release-drafter#178

Merged
ryantm merged 1 commit into
mainfrom
ryantm/remove-pr-target-release-drafter
May 12, 2026
Merged

ci: remove pull_request_target trigger from release-drafter#178
ryantm merged 1 commit into
mainfrom
ryantm/remove-pr-target-release-drafter

Conversation

@ryantm
Copy link
Copy Markdown
Contributor

@ryantm ryantm commented May 12, 2026

Why

The recent TanStack NPM supply-chain compromise exploited a pull_request_target workflow. Per Replit security policy, we are removing pull_request_target triggers from all Replit-owned public repos as a precaution, even where the current use looks safe.

Slack thread: https://replit.slack.com/archives/C03FS477T17/p1778588219046429

What changed

Removed the pull_request_target trigger block from .github/workflows/release-drafter.yml. No other changes.

Note: the autolabeler will no longer run on PRs from forks. Release notes are still drafted on push to main, so the core release-drafter behavior is preserved.

Test plan

Static change to a workflow trigger; no runtime test plan. The remaining push (to main) and pull_request triggers continue to run release-drafter as before.

Revertibility

Safe to revert — single-file workflow change, no data or schema impact.

~ written by Zerg 👾 (ravaging-mothership-61f2)

@ryantm ryantm added the zergling-authored Authored by a zergling agent label May 12, 2026
@ryantm ryantm marked this pull request as ready for review May 12, 2026 13:00
@ryantm ryantm requested a review from a team as a code owner May 12, 2026 13:00
@ryantm ryantm requested review from daweifeng-replit and removed request for a team May 12, 2026 13:00
@ryantm ryantm enabled auto-merge (squash) May 12, 2026 15:19
@ryantm ryantm merged commit 008e539 into main May 12, 2026
3 checks passed
@ryantm ryantm deleted the ryantm/remove-pr-target-release-drafter branch May 12, 2026 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

zergling-authored Authored by a zergling agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants