Repository navigation
feat: reject skill links that leave the package - #316
Conversation
🦋 Changeset detectedLatest commit: 7b40b6d 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 54 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 7b40b6d
☁️ Nx Cloud last updated this comment at |
commit: |
# Conflicts: # packages/intent/src/validate/blocks.ts
🎯 Changes
intent validatenow reportsLink target is outside the package: <target>when a relative Markdown link in aSKILL.mdresolves to a path outside the package that owns the skill.../../../docs/guide.mdpassed. Only the package is installed in a consumer project, so such a link is dead for every consumer.intent loadtests it, sovalidaterejects the links thatloadleaves unrewritten. Thefileslist inpackage.jsonand.npmignoreare not consulted.intent validatereference describes the check.✅ Checklist
pnpm run test:pr(runpnpm build:allfirst).🚀 Release Impact