ci(detect-agent): remove the detect-agent workflow#10781
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughThe pull request removes the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
View your CI Pipeline Execution ↗ for commit 049dd35
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview1 package(s) bumped directly, 24 bumped as dependents. 🟩 Patch bumps
|
size-limit report 📦
|
🎯 Changes
Remove
detect-agent.ymlfor the same fork-PR limitation.The workflow's main value is identifying AI agents in external (fork) PRs, but with
pull_requesttheGITHUB_TOKENis downgraded to read-only on forks, so it can't apply labels there — exactly where it would be most useful. The officialbombshell-dev/automationdocs recommendpull_request_target(and note the workflow only reads PR metadata without checking out fork code), but we're keeping a hard line againstpull_request_targetto avoid carving out exceptions to that policy.Removed file:
.github/workflows/detect-agent.yml✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit