Skip to content

fix: include skills/ directory in published npm package#294

Merged
kome12 merged 1 commit intonansen-ai:mainfrom
yodablocks:fix/152-skills-npm-package
Mar 13, 2026
Merged

fix: include skills/ directory in published npm package#294
kome12 merged 1 commit intonansen-ai:mainfrom
yodablocks:fix/152-skills-npm-package

Conversation

@yodablocks
Copy link
Contributor

Closes #152

Problem

The skills/ directory (27 skill files) was missing from the published npm package because package.json's files allowlist never included skills/**. Every version since launch has shipped without skills files.

Fix

Added "skills/**" to the files array in package.json.

Verification

Ran npm pack --dry-run — skills/ files now appear in the tarball output.

Note

This also unblocks PR #260, which adds new skill files to skills/ that would otherwise be invisible to npm users until this packaging bug is fixed.

Tests

All tests pass, lint clean.

@nansen-pr-reviewer
Copy link

pr-reviewer Summary

No issues found

The code review completed successfully with no findings.

Review effort: 1/5 (Trivial)

Summary

This is a minimal, well-targeted bug fix. The PR adds "skills/**" to the files array in package.json, ensuring the skills/ directory (containing ~28 skill subdirectories with SKILL.md definitions used by AI agents) is included when the package is published to npm. Without this entry, the skills directory was silently omitted from the published package. A correctly scoped changeset entry (patch) accompanies the change. No issues found.


Token usage: 2,072 input, 1,781 output, 84,479 cache read, 15,725 cache write

Copy link
Contributor

@kome12 kome12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@kome12 kome12 merged commit 9ac31a4 into nansen-ai:main Mar 13, 2026
7 checks passed
This was referenced Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🔴 [High] skills/ directory missing from published npm package

2 participants