Repository navigation
perf: share package context, path checks, and frontmatter across maintainer batches - #337
Conversation
🦋 Changeset detectedLatest commit: 2156afb 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 |
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing: 8
✨ Finishing Touches
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 2156afb
☁️ Nx Cloud last updated this comment at |
commit: |
Merging this PR will not alter performance
|
🎯 Changes
intent maintainer setupdiscovered existing skills infindExistingSkillsand then passed them toplanAddSkills, which repeatedprojectPathlstatSyncwalks, uncachedresolveProjectContextancestor walks,package.jsonreads, andSKILL.mdfrontmatter reads across the batch.resolveProjectContextnow cachesfindOwningPackageRootresults perIntentFsCacheand uses a singlestatSync(path, { throwIfNoEntry: false })probe intoSearchDir.projectPath,skillPath,recordPath,readRecord, andskillEntriesaccept a per-batchSet<string>of verified non-symlink paths so shared directory prefixes are checked once per operation.findExistingSkills,planSetupRecords, andplanAddSkillsshare oneIntentFsCache, verified path set, and parsedSKILL.mdfrontmatter acrossmaintainer setup(planMaintainerSyncalso sharesfsCacheand verified paths across its skill loop), andfindExistingSkillscounts duplicate names in aMapinstead of anskills.some(...)scan.benchmarks/intent/maintainer.bench.ts, Vitest 4.1.11 on Windows Node)main(ab9877c) mean (rme)2156afb) mean (rme)maintainer setup with 20 existing skillsmaintainer setup with 200 existing skills✅ Checklist
pnpm run test:pr(runpnpm build:allfirst).🚀 Release Impact