Repository navigation
docs: explain opt-in Nuxt template dependencies - #90
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Latest result Pull request merged. Final answersSession Pull request merged. Ready; no repair needed. All supplied current-head checks passed, and both review bots reported no findings. `node scripts/check-skill-links.mjs`, `git diff --check origin/main...HEAD`, and manifest-version assertions passed; the checkout is clean. Local lint/typecheck relied on green CI because host installation rejected the unchanged minimumReleaseAgeExcludePrune setting. Host configuration needs support for that setting; it does not block this PR. Session Waiting for review checks before the next pass. |
![pullfrog[bot]](https://avatars.githubusercontent.com/in/1768019?s=60&v=4){kind=small}
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes Checked the complete documentation and manifest changes against the pinned Nuxt implementation and the linked Kit documentation.
- Dependency guidance: Adds a reference covering opt-in
dependsOn,getContentsversussrcdefaults, empty and named declarations, predicates, watcher setup, and stale-output risks. - Regeneration semantics: Correctly distinguishes existing-file edits from add/remove events and explicit
updateTemplatesregeneration, including support throughaddTypeTemplate. - Discovery and packaging: Links the reference from the module skill, development guide, and README, and aligns plugin manifest versions at
1.3.6. - Validation: Relative skill links pass locally; CI and skill validation checks pass. Framework behavior was verified by source inspection at
fa1f7656e2c7c9743e8686d5f95dfc1779188123; no runtime development fixture or downstream Hub sync was run in this review.
gpt-6.1-sol (free via Pullfrog for OSS) ｜ 𝕏
|
ViteHub Babysitter: please refresh the live checks for 9912064. CI, skill validation, and Pullfrog are all successful, and Pullfrog reported no new issues. Maxi has authorized the bot to squash-merge after its normal gates pass. The companion onmax/nuxt-skill-hub#47 is intentionally draft until this source guide merges. |
Module authors currently get template examples without guidance on development regeneration. Add one authoritative reference for addTemplate/addTypeTemplate dependsOn, linked from the nuxt-modules entrypoint and development guide.
Explain the opt-in API, getContents versus src defaults, empty and named dependency declarations, predicates, watcher setup, and the stale-output risk. An empty declaration skips edits to existing watched files; it does not mean a template runs only once. Nuxt Skill Hub consumes this source through a generated section.
Validation: targeted Markdown/manifest lint, skill validation, relative-link check, diff check, and exact source-to-hub sync check passed. The hub publication and real nuxt prepare checks passed during the coordinated change.
Context: nuxt/nuxt#36744 (comment). Behavior was checked against Nuxt fa1f7656e2c7c9743e8686d5f95dfc1779188123.
Merge handling: authorized for ViteHub Babysitter to repair in scope and squash-merge after checks/review. The companion hub PR should become ready after this source guide is on main. No upstream Nuxt patch or production deployment is part of this change.