Repository navigation
chore(skills): sync public skills from the store - #116
Conversation
|
Note The title starts with
|
|
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 45 minutes. View limit details
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. |
There was a problem hiding this comment.
ℹ️ One portability issue to fix before relying on this sync; remaining changes look consistent with the stated workflow.
Reviewed changes
Reviewed the public skill sync for diataxis-docs-site, find-docs, and gitworkflow.
- Docs-site starter — updates Node and Astro/Starlight requirements and switches completion guidance to Bun.
- Documentation lookup — expands Context7 routing for version-pinned vendor guides and adds a version verification reminder.
- Git workflow — revises changelog behavior and adds review-budget and merge-policy workflow guidance.
gpt-6-luna | 𝕏
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes — No commits have landed since the prior Pullfrog review. Rechecked the resolved discussion: the starter ships only bun.lock, defines both verification scripts, and matches the documented Bun commands. The prior finding does not apply to this head; approving as requested.
gpt-6-astra | 𝕏
|
Spec fidelity at 89f5301: no deviations. The sync job generated this PR from the store for the three mirror skills with drift (diataxis-docs-site, find-docs, gitworkflow). It changes only files under those skill folders. The one deletion is the stale package-lock.json that the store no longer ships. verify: none. |

Problem
The store changed 3 mirrored skill(s) since the last sync.
They are
diataxis-docs-site,find-docs,gitworkflow.The public copies stay stale until this PR merges.
Change
The daily sync ran
scripts/sync-public.mjs --pruneonorigin/mainataf7f859ed59e.diataxis-docs-sitefind-docsgitworkflowCandidates not in the manifest
These store skills have no manifest entry. A skill goes public only when someone adds its manifest entry by hand.
AudioEditorAwesomeReadmeBitterPillEngineeringagent-reachagents-md-pdfasc-ad-hoc-distributionasc-analytics-reportsasc-app-create-uiasc-apple-adsasc-aso-auditasc-build-lifecycleasc-cli-usageasc-crash-triageasc-id-resolverasc-localize-metadataasc-metadata-syncasc-notarizationasc-ppp-pricingasc-release-flowasc-revenuecat-catalog-syncasc-screenshot-resizeasc-shots-pipelineasc-signing-setupasc-submission-healthasc-subscription-localizationasc-testflight-orchestrationasc-wall-submitasc-whats-new-writerasc-workflowasc-xcode-buildautoresearch-promptbitwardenbluf-sbarbranch-lifecyclecanva-design-deskccloudci-worktreescode-optimizercodex-computer-usecomposecoordinatecorrectcycledefensive-code-lintevaluate-skillfusion-cadgithub-workflowsgoal-prompt-metagrill-skillherdrhermes-dispatchhindsight-fleethomelabhomelab-adguardhomelab-architecturehomelab-dockerhomelab-nashomelab-observehomelab-proxmoxhtml-communicationimage-maxinstall-mdissue-analysisjev-askjev-routejevgrepland-prlintlogo-forgemediamodel-pickeropen-prossie-contextpr-evidence-replaypr-mediaproduct-docsresolving-merge-conflictsresponsive-checkretroreverse-engineer-anythingreview-gatesbscroll-storysermon-analyzerserveskill-arcswamp-getting-startedswiftuit3-cardstesting-disciplineunifi-network-manageruntangle-codebaseuserinterface-wikivisual-pr-briefingvisual-tickets-briefingwebdesignxcloudyoutube-analyzerzoom-outProof
sync-public.mjs --prune(leak assertion, dangling references)bun run lintbun run testbun run validate:skillsbun run validate:pluginsbun scripts/validate-package-contents.mjsReview
package.json:11definescheck, andgit grep -i npmon 89f5301 finds nothing in the skill.Merge danger
Door: two-way. Revert the PR to restore the previous public copies.
Blast radius: medium. The sync deleted 1 file(s) that the store no longer has.
Changes made by sync-public via launchd