Skip to content

fix: include skills/ directory in npm package - #278

Merged
Pavel Feldman (pavelfeldman) merged 2 commits into
mainfrom
copilot/fix-npmignore-for-skills
Feb 25, 2026
Merged

Pavel Feldman (pavelfeldman) merged 2 commits into
mainfrom
copilot/fix-npmignore-for-skills

Conversation

Copilot AI commented Feb 25, 2026

Copy link
Copy Markdown
Contributor

.npmignore used a deny-all (**/*) + allowlist pattern but omitted skills/, causing the directory to be stripped from the published package. Consumers referencing ./node_modules/@playwright/cli/skills/playwright-cli would find it missing after npm install.

Change

  • .npmignore — add !skills/** to the allowlist
 **/*
 !README.md
 !LICENSE
 !playwright-cli.js
+!skills/**
Original prompt

This section details on the original issue you should resolve

<issue_title>.npmignore shouldn't ignore skills directory</issue_title>
<issue_description>Hi!
I'm using Pi Coding Agent as my solution to keep organized skills in a good way, using a package json to version them.
Unfortunately, when i "pi install", all dependencies are downloaded, but the playwright directory doesn't have the skill directory i need.

Image Image

I believe maybe you've been too aggressive on .npmignore file?

Image</issue_description>

<agent_instructions>Let's make sure the skills folder is deployed with npm</agent_instructions>

Comments on the Issue (you are Copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: pavelfeldman <883973+pavelfeldman@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix .npmignore to include skills directory fix: include skills/ directory in npm package Feb 25, 2026
@pavelfeldman
Pavel Feldman (pavelfeldman) marked this pull request as ready for review February 25, 2026 03:25
@pavelfeldman
Pavel Feldman (pavelfeldman) merged commit bf14c9b into main Feb 25, 2026
4 checks passed
@pavelfeldman
Pavel Feldman (pavelfeldman) deleted the copilot/fix-npmignore-for-skills branch February 25, 2026 03:26
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.

.npmignore shouldn't ignore skills directory

2 participants