Skip to content

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

Merged
CybotTM merged 3 commits intomainfrom
feat/npm-install
May 2, 2026
Merged

feat: ship as npm package via @netresearch/agent-skill-coordinator#37
CybotTM merged 3 commits intomainfrom
feat/npm-install

Conversation

@CybotTM
Copy link
Copy Markdown
Member

@CybotTM CybotTM commented May 2, 2026

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/matrix-skill

After install, the coordinator's postinstall walks node_modules, finds this package via aiAgentSkill: skills/matrix-communication/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>
Copilot AI review requested due to automatic review settings May 2, 2026 09:29
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

Dependency Review

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

Scanned Files

None

Copy link
Copy Markdown

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

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 introduces npm support for the Matrix skill by adding installation instructions to the README and creating a package.json for distribution. Feedback indicates that the .claude-plugin/ directory should be included in the files array to ensure the manifest is packaged, and the aiAgentSkill field should be converted to an array to correctly register both the communication and administration skills.

Comment thread package.json
Comment thread package.json Outdated
Copy link
Copy Markdown

Copilot AI left a comment

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 adds npm-installation metadata for the Matrix skill repository so Node-based projects can consume it through @netresearch/agent-skill-coordinator, alongside the existing Composer-based distribution path. It extends the repo’s packaging/discovery story rather than changing the Matrix skill implementations themselves.

Changes:

  • Add a root package.json that makes the repo installable as @netresearch/matrix-skill from GitHub.
  • Declare npm-side skill discovery metadata and packaged file list for the coordinator workflow.
  • Document the Node/npm installation path in README.md, including pnpm postinstall allowlisting.

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 selection, and coordinator peer dependency for GitHub-based installs.
README.md Documents npm installation and pnpm setup for coordinator-based skill registration.

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

Comment thread package.json Outdated
The repo ships both skills/matrix-communication/SKILL.md and
skills/matrix-administration/SKILL.md, but only the former was
listed in composer.json's extra.ai-agent-skill (and inherited by
the new package.json's aiAgentSkill). Switch both fields to JSON
arrays so the coordinator discovers both skills.

Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
@CybotTM
Copy link
Copy Markdown
Member Author

CybotTM commented May 2, 2026

Updated composer.json + package.json so aiAgentSkill is an array of both bundled skills (matrix-communication + matrix-administration). Single-string declaration was missing matrix-administration.

Signed-off-by: Sebastian Mendel <sebastian.mendel@netresearch.de>
@CybotTM CybotTM merged commit 01971d3 into main May 2, 2026
12 checks passed
@CybotTM CybotTM deleted the feat/npm-install branch May 2, 2026 10:16
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