Skip to content

Ship diffs and trees agent skills in the published packages - #1083

Open
mdo wants to merge 1 commit into
mainfrom
mdo/publish-package-skills
Open

Ship diffs and trees agent skills in the published packages#1083
mdo wants to merge 1 commit into
mainfrom
mdo/publish-package-skills

Conversation

@mdo

@mdo mdo commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Include the diffs and trees agent skills in the published npm packages so installing @pierre/diffs or @pierre/trees version-locks the skill with the API.
  • Move those skill directories into the packages, add skills to each files list, and keep repo-root skills/diffs and skills/trees as symlinks for npx skills add pierrecomputer/pierre.

Test plan

  • Confirm packages/diffs/package.json and packages/trees/package.json list "skills" in files
  • Confirm skills/diffs/SKILL.md and skills/trees/SKILL.md still resolve via the repo-root symlinks
  • bun test packages/diffs/test/published-skill.test.ts packages/trees/test/published-skill.test.ts
  • Dry-run pack each package and check the tarball listing includes skills/<name>/SKILL.md and the reference files

@mdo
feat: ship diffs and trees agent skills in the npm packages
d793485 
Installing @pierre/diffs or @pierre/trees now includes the matching
agent skill under skills/, so the instructions stay version-locked
with the published API.

Move those skill directories into the packages, add them to the files
list, and keep repo-root skills/diffs and skills/trees as symlinks
for the GitHub Skills CLI.
@mdo
mdo requested a review from necolas August 19, 2026 20:59
@vercel

vercel Bot commented Aug 19, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pierre-diffshub Ready Ready Preview Aug 19, 2026 9:01pm
pierre-docs-diffs Ready Ready Preview Aug 19, 2026 9:01pm
pierre-docs-trees Ready Ready Preview Aug 19, 2026 9:01pm
pierrejs-diff-demo Ready Ready Preview Aug 19, 2026 9:01pm

Request Review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 19, 2026

Copy link
Copy Markdown

💡 Codex Review

pierre/skills/diffs/SKILL.md

Line 1 in d793485

../../packages/diffs/skills/diffs/SKILL.md

P1 Badge Keep CLI-installed skill files self-contained

When npx skills add pierrecomputer/pierre --skill diffs copies this skill from its temporary repository checkout, recursive copying preserves this symlink rather than materializing its contents; the link consequently points back into that temporary checkout and becomes dangling after cleanup. Every reference file is now a similar link, and trees has the same layout, so both install commands retained in the READMEs can produce unusable skills. Keep real files at the CLI discovery paths or ensure they are dereferenced before installation.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@necolas necolas Awaiting requested review from necolas

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mdo