[Backport v9] agent-features: Fix the package being published without skills and rules - #6206
Merged
VPS-thodax merged 4 commits intoAug 17, 2026
Merged
Conversation
…linking them The package's skills/ and rules/ folders were symlinks to the folders at the repository root. pnpm stopped following symlinked directories when packing in v11, so every release since @comet/agent-features v9.4.0 shipped nothing but the manifest and install-agent-features found no skills or rules to install. The folders have to stay at the repository root because install-agent-features sparse-checks-out skills/ and rules/ from there when this repository is used as an external source. So build them into the package instead: the new build script copies both folders in and fails when either is missing or empty. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0181iiTBFCfqvBjEwjzxEEzm (cherry picked from commit 6f56eb1)
Node 24 strips the types natively, so the script runs without a build step of its own. Set the package's module type to avoid Node re-parsing the file as an ES module and warning about it on every build; the package ships only markdown, so the field has no effect on consumers. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0181iiTBFCfqvBjEwjzxEEzm (cherry picked from commit 5933d3f)
The release pipeline builds every package before publishing, so prepack only duplicated that work. It also wouldn't have prevented the empty releases, since those were packed after a full build; the guard in the build script covers that. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0181iiTBFCfqvBjEwjzxEEzm (cherry picked from commit 489c1d7)
The v9.x.x branch still ships the package as @comet/agent-features (the rename to @dextinity/agent-features happened after this branch diverged), so update the changeset and the build script's error messages to match.
manuelblum
approved these changes
Aug 14, 2026
vps-manuel-blum
approved these changes
Aug 17, 2026
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
4 participants
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.
Backport of #6205 to
v9.x.x.Problem
@comet/agent-featuresships empty. The published tarball contains nothing butpackage.jsonandLICENSE, soinstall-agent-featuresfinds no skills and no rules in it.Cause
The package's
skills/andrules/folders were symlinks to the folders at the repository root. pnpm stopped following symlinked directories when packing in v11, so theskills/**andrules/**patterns infilesmatched nothing.Fix
The package builds its
skills/andrules/folders instead of symlinking them, like every other package in the repository builds itslib/. Thebuildscript copies both folders in from the repository root and fails when either is missing or empty, so a regression of this kind breaks the release instead of publishing silently. The copies are gitignored.Adaptation notes
The 3 commits from #6205 cherry-picked cleanly. One follow-up commit adapts the changeset and
build.tserror messages from@dextinity/agent-featuresto@comet/agent-features, since the package rename to@dextinity/*(#6069) happened afterv9.x.xdiverged frommain.Verification
pnpm --filter '@comet/agent-features' run buildcopies 25 skill files and 12 rule files.pnpm pack --dry-runlists 37 files in the tarball (previously 2), matching what@comet/agent-features@9.3.0shipped.Generated by Claude Code