Skip to content

fix: make wired brain CLI invocation explicit - #21

Merged
LinkinStars merged 1 commit into
mainfrom
fix/issue-20-wire-cli-invocation
Aug 20, 2026
Merged

fix: make wired brain CLI invocation explicit#21
LinkinStars merged 1 commit into
mainfrom
fix/issue-20-wire-cli-invocation

Conversation

@LinkinStars

Copy link
Copy Markdown
Collaborator

Summary

  • state the portable bundled Node CLI invocation directly in wired agent blocks
  • replace the ambiguous brain init suggestion with the direct invocation
  • add regression coverage for AGENTS.md and make the missing-CLI hook test portable on macOS

Verification

  • npm test
  • fresh-project validation with brain absent from PATH: generated AGENTS.md provides the direct invocation, and node <brain-page-skill-dir>/bin/brain.mjs list-pages and init succeed

Fixes #20

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

Makes wired agent guidance explicitly invoke the bundled Node CLI without relying on brain being on PATH.

Changes:

  • Adds explicit CLI and initialization commands to generated agent blocks.
  • Adds regression assertions for AGENTS.md.
  • Makes the missing-CLI hook test portable across macOS and Linux.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
skills/brain-page/bin/brain.mjs Emits explicit bundled CLI commands.
skills/brain-page/test/cli-init-wire.test.mjs Verifies generated invocation guidance.
skills/brain-page/test/cli-hooks.test.mjs Includes /bin in the test PATH.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@LinkinStars
LinkinStars merged commit ba53ba5 into main Aug 20, 2026
4 checks passed
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.

Wired AGENTS.md block presents brain as a PATH command; only CLAUDE.md gets the contract inlined

2 participants