Repository navigation
fix: rewrite tree paths that repeat the package directory in maintainer sync - #317
Conversation
🦋 Changeset detectedLatest commit: e728066 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 16 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 a87291d
☁️ Nx Cloud last updated this comment at |
commit: |
a87291d to
e728066
Compare
🎯 Changes
Stacked on #313.
A
skill_tree.yamlentry locates its skill withpackageand apathrelative to that package. A hand-written tree can repeat the package directory inpath:intent maintainerjoined the two values, looked forpackages/react-table/packages/react-table/skills/table-state/SKILL.md, and reportedMissing skillalthough the file existed. TanStack Table's tree uses this form, somaintainer statuscounted 0 of its 39 skills.maintainer syncnow recognizes this form when the file exists atpathand not at the joined location:maintainer statusandmaintainer checkfind the skill and listskill_tree.yamlas a file to synchronize.maintainer syncrewrites the entry aspath: skills/table-state/SKILL.md.A path that is wrong for any other reason still reports
Missing skill.On TanStack Table,
maintainer statusgoes from0 skill(s)with 39Missing skilllines to39 skill(s), 1 file(s) to sync, andmaintainer syncrewrites the 39 paths.