fix: remove compromised setup hooks#44
Conversation
|
Warning Review limit reached
More reviews will be available in 33 minutes and 55 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
2 issues found across 8 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
864967f to
adeb8c1
Compare
|



Summary
.github/setup.jspayloadnode .github/setup.jswhere presentsecurity:checkguard where the repo has package scripts to prevent reintroducing the setup payload/hooksVerification
bun run security:checkpasses locallybun install --ignore-scriptsbefore verification to avoid lifecycle script executionSummary by cubic
Removed compromised auto-executing setup hooks and the obfuscated
.github/setup.js. Added a symlink‑safe security check that runs inprepublishOnlyto block reintroduction and fail fast.Bug Fixes
.github/setup.jsand auto-run IDE hooks from.claude/settings.json,.gemini/settings.json,.cursor/rules/setup.mdc, and.vscode/tasks.json.node .github/setup.jsfrom thetestscript.New Features
scripts/security-check.tsandsecurity:check(symlink‑safe scan blocking.github/setup.js, IDE session hooks, andrunOn: "folderOpen"), run first inprepublishOnly.lint,format, andfixscripts to callultracitedirectly; added.changeset/clean-setup-hooks.mdfor a patch release.Written for commit adeb8c1. Summary will update on new commits.
Note
Remove compromised setup hooks and add security check to block reintroduction
.github/setup.jsand removes all editor/IDE hooks (Claude, Cursor, Gemini, VS Code) that auto-executed it on session start or folder open.security:checkscript to package.json and runs it as the first step ofprepublishOnlyto block publishing if violations exist.npm run testno longer attempts to run the removed setup script;prepublishOnlywill now fail fast if forbidden patterns are detected.Macroscope summarized adeb8c1.