Repository navigation
fix: write the hook script parser as fixed text - #326
Conversation
🦋 Changeset detectedLatest commit: cfb57e7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 26 minutes. View limit detailsLimit details: You’ve used all 2 included reviews currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (4)
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 |
|
View your CI Pipeline Execution ↗ for commit cfb57e7
☁️ Nx Cloud last updated this comment at |
commit: |
🎯 Changes
intent hooks installwrote the command parser into the hook script withparseIntentInvocation.toString(), so the installed file held whatever text the build produced for that function. A bundler upgrade or a minified build would rewrite the script that users install.policy.tsexports it asINTENT_INVOCATION_PATTERN, and the template prints that same constant. The pattern does not change.intent hooks installrewrites an installed script once and reportsUpdated Intent hooks. A script that is not reinstalled keeps working.✅ Checklist
pnpm run test:pr(runpnpm build:allfirst).🚀 Release Impact