Repository navigation
feat(intent): add official TanStack skills - #435
Merged
Merged
Conversation
Pin the newest release that is older than the one-day minimumReleaseAge. Allow @tanstack/* skills from the installed packages. Co-authored-by: otaki0413.it <otaki0413.it@gmail.com>
Add the generated Intent skill-loading block and link the Start docs and Intent. Template rules win when a skill disagrees. Keep the generated block out of the formatter so a later intent install stays idempotent. Co-authored-by: otaki0413.it <otaki0413.it@gmail.com>
otaki0413
marked this pull request as ready for review
October 17, 2026 04:10
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. |
Oxfmt 0.73.0 inserts blank lines that the next intent install removes, which makes fmt --check fail. Keep the reason outside the markers so install does not delete it. Co-authored-by: otaki0413.it <otaki0413.it@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Agents can read the TanStack skills that already ship inside the installed packages. Intent is the official loader. The pin is the newest release that is older than the one-day release-age gate.
What changed
@tanstack/intent0.5.5 and setintent.skillsto@tanstack/*.pnpm exec intent installdeletes them, andvp fmt --checkthen fails. The comment sits outside the markers sointent installleaves it in place.Scope
Covers the consumer Intent setup. Leaves out copied SKILL.md files, an MCP server, and any change to the release-age exclude list.
Tradeoffs
The formatter would insert blank lines inside the generated block, and the next
intent installwould delete them. A prettier-ignore range keeps the official block byte-for-byte. Text inside the markers is regenerated, so the explanation lives aboveprettier-ignore-start.Blast Radius
This adds one dev dependency and changes agent docs. App runtime, deploy, and the release-age exclude list stay as they are.
Verification
vp install --frozen-lockfilepassed.vp checkpassed.vp testpassed, 1 file and 1 test.vp buildpassed.pnpm exec intent listshowed 6 packages and 23 skills.intent --versionprinted 0.5.5.f25707a,pnpm exec intent installleft the new HTML comment and the precedence sentence in place, andvp checkpassed.