Skip to content

feat: ship as npm package via @netresearch/agent-skill-coordinator#28

Merged
CybotTM merged 2 commits into
mainfrom
feat/npm-install
May 2, 2026
Merged

feat: ship as npm package via @netresearch/agent-skill-coordinator#28
CybotTM merged 2 commits into
mainfrom
feat/npm-install

Conversation

@CybotTM

@CybotTM CybotTM commented May 2, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds package.json so the skill is npm-installable straight from this GitHub repo (no npm-registry publication required to start).
  • README gains a parallel npm (Node Projects) section under Installation, mirroring the existing Composer (PHP Projects) section.

How discovery works (consumer side)

npm install --save-dev \
  @netresearch/agent-skill-coordinator \
  github:netresearch/cli-tools-skill

After install, the coordinator's postinstall walks node_modules, finds this package via aiAgentSkill: skills/cli-tools/SKILL.md, validates the frontmatter, and writes a <skills_system> block into the project's AGENTS.md. Same shape as the Composer plugin's output.

Why version stays at 0.0.0-source

Versioning still lives in git tags and the existing Composer release workflow. A real npm version would be set at publish time; until then, 0.0.0-source is the standard "this manifest exists for tooling, not as a release marker" placeholder.

Sibling PRs (merged)

Coordinator: https://github.com/netresearch/node-agent-skill-coordinator (@netresearch/agent-skill-coordinator@0.1.2)

Adds a package.json so the skill is npm-installable straight from this
GitHub repo (no npm-registry publication required to start). README
gains a parallel 'npm (Node Projects)' section under Installation,
mirroring the existing 'Composer (PHP Projects)' section.

Same pattern applied across the Netresearch skill collection in 2026-05.
Coordinator: https://github.com/netresearch/node-agent-skill-coordinator
Sibling PRs already merged on git-workflow-skill, github-project-skill,
security-audit-skill. Version stays at 0.0.0-source until npm publication
is decided separately.

Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adds npm support for the CLI tools skill by introducing a package.json file and updating the README.md with installation instructions. A review comment suggests including the catalog/ and references/ directories in the package.json files list to ensure all required tool definitions are available in the published package.

Comment thread package.json
@github-actions

github-actions Bot commented May 2, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR makes the skill consumable from Node projects by adding npm package metadata and documenting the Node installation flow alongside the existing Composer-based installation. It fits the repository’s distribution story by exposing the existing skill bundle to the @netresearch/agent-skill-coordinator discovery mechanism.

Changes:

  • Add a new package.json so the repository can be installed as an npm package directly from GitHub.
  • Declare the skill entrypoint for Node-side discovery via the custom aiAgentSkill manifest field.
  • Extend the README with npm installation instructions, including the pnpm postinstall allowlist note.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
package.json Adds npm package metadata, packaged file list, and the coordinator peer dependency.
README.md Documents npm-based installation and pnpm configuration for Node consumers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
Installer scripts (skills/cli-tools/scripts/installers/*.sh and lib/catalog.sh)
read tool metadata from $ROOT/catalog/*.json. Without this dir in the npm
tarball, npm consumers got a partial skill that registered but failed at
install time. Caught by reviewer.

Signed-off-by: Sebastian Mendel <sebastian.mendel@netresearch.de>
@CybotTM CybotTM merged commit 9e4a6a6 into main May 2, 2026
12 checks passed
@CybotTM CybotTM deleted the feat/npm-install branch May 2, 2026 17:19
@CybotTM CybotTM mentioned this pull request May 15, 2026
CybotTM added a commit that referenced this pull request May 15, 2026
## Release v1.7.0

Minor release. Bumps `.claude-plugin/plugin.json` and
`skills/cli-tools/SKILL.md` frontmatter `metadata.version` from `1.6.1`
to `1.7.0`.

### Highlights since v1.6.1

- Ship the skill as an npm package via
`@netresearch/agent-skill-coordinator`, with a follow-up fix to include
the `catalog/` directory in the tarball (#28).
- Release workflow grants SLSA provenance permissions and drops the
deprecated `workflow_dispatch.bump` input (#26, #27).
- CI plumbing: forward the `bump` input from `workflow_dispatch` to the
reusable release workflow (#25).

### Release plan

After merge: tag main with a signed annotated tag, push, the
`skill-repo-skill` reusable workflow publishes archives + SHA256SUMS
with cosign + SLSA attestation, then narrative notes get applied via `gh
release edit ... --notes-file`.
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.

2 participants