Repository navigation
feat(skills): ship a sync-markdown-code-blocks Agent Skill with TanStack Intent (#30) - #55
Merged
Merged
Conversation
|
The changes in this PR will be included in the next version bump.
|
|
|
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.


Closes #30
Summary
The package now ships an Agent Skill, versioned with each release. TanStack Intent manages it.
Decisions:
distribution.mode: none). Consumers runnpx @tanstack/intent install. Repository and plugin exports can be added later.dist/main.jsbuilt from this revision. The package README disagrees with the CLI in places:list,runanddumptake one file,--fileis an exact match, and directories are never accepted. The skill follows the CLI, and I filed docs: CLI examples pass directories, globs and --file patterns that the CLI doesn't accept #54 for the README.file=inside the base, and that containment is check-then-use.Evidence
pnpm exec intent maintainer check --base origin/main:Validated 1 skill files — all passed,0 authoring issue(s), 0 pending review item(s).skill-sync-markdown-code-blocks.test.ts:update --apply --project;sourceRoot;package/skills/sync-markdown-code-blocks/{SKILL.md,references/json-results.md,references/transformers.md},README.md,LICENSEand the 60dist/files._artifacts/is not in it, andexportsandbinare unchanged.intent install, a separate agent got the task text and a disposable project, set up withintent install→ "Enable all" (intent.skills: ["*"], broader than a consumer should use; a single-skill selection was not tested). It ranintent list, thenintent load mdcode-ts#sync-markdown-code-blocks, thenupdate --diff --project, thenupdate --apply --project. The unchanged grader passed on its result (SKILL_TASK_DIR=…).skill_spec.mdrecords it as such.pnpm checkpasses (pre-push).Merge Danger
Door: two-way
Blast Radius: package contents
The npm package gains
skills/and atanstack-intentkeyword.exports,binand runtime code are unchanged. A newCheck Skillsworkflow runs on PRs. It fails a PR that changes a mapped source (commands, parser, region code, README, examples, the task check) without recording a review:pnpm exec intent maintainer review --jsonthen--record, asAGENTS.mdexplains. Theminorbump file announces the skill.