Add OpenCode support and rename to @wbern/agent-instructions#5
Add OpenCode support and rename to @wbern/agent-instructions#5
Conversation
* [feat] added .opencode and skills to directories in cli-generator.ts * [feat] updated getDestinationPath() to handle both agents * feat: rename package to @hevolx/agent-instructions with OpenCode support - Rename npm package from @wbern/claude-instructions to @hevolx/agent-instructions - Rename CLI binary from claude-instructions to agent-instructions - Add OpenCode as primary target (--agent=opencode default) - Generate .opencode/commands/ and .opencode/skills/ artifacts - Update all URLs, tests, and snapshots to hevolx/agent-instructions - Refactor shared frontmatter parsing into transformFrontmatter helper - Add missing test coverage for getSkillsPath, truncatePath, agent cancel/both/claude paths
* [feat] added .opencode and skills to directories in cli-generator.ts * [feat] updated getDestinationPath() to handle both agents * feat: rename package to @hevolx/agent-instructions with OpenCode support - Rename npm package from @wbern/claude-instructions to @hevolx/agent-instructions - Rename CLI binary from claude-instructions to agent-instructions - Add OpenCode as primary target (--agent=opencode default) - Generate .opencode/commands/ and .opencode/skills/ artifacts - Update all URLs, tests, and snapshots to hevolx/agent-instructions - Refactor shared frontmatter parsing into transformFrontmatter helper - Add missing test coverage for getSkillsPath, truncatePath, agent cancel/both/claude paths * ci: fix scoped npm package public publishing
* [feat] added .opencode and skills to directories in cli-generator.ts * [feat] updated getDestinationPath() to handle both agents * feat: rename package to @hevolx/agent-instructions with OpenCode support - Rename npm package from @wbern/claude-instructions to @hevolx/agent-instructions - Rename CLI binary from claude-instructions to agent-instructions - Add OpenCode as primary target (--agent=opencode default) - Generate .opencode/commands/ and .opencode/skills/ artifacts - Update all URLs, tests, and snapshots to hevolx/agent-instructions - Refactor shared frontmatter parsing into transformFrontmatter helper - Add missing test coverage for getSkillsPath, truncatePath, agent cancel/both/claude paths * ci: fix scoped npm package public publishing * ci: remove registry-url from setup-node to fix npm auth conflict
* [feat] added .opencode and skills to directories in cli-generator.ts * [feat] updated getDestinationPath() to handle both agents * feat: rename package to @hevolx/agent-instructions with OpenCode support - Rename npm package from @wbern/claude-instructions to @hevolx/agent-instructions - Rename CLI binary from claude-instructions to agent-instructions - Add OpenCode as primary target (--agent=opencode default) - Generate .opencode/commands/ and .opencode/skills/ artifacts - Update all URLs, tests, and snapshots to hevolx/agent-instructions - Refactor shared frontmatter parsing into transformFrontmatter helper - Add missing test coverage for getSkillsPath, truncatePath, agent cancel/both/claude paths * ci: fix scoped npm package public publishing * ci: remove registry-url from setup-node to fix npm auth conflict * chore: restore wbern/claude-instructions branding for upstream PR * docs: restore file priority and usage stats sections to README * fix: Update GitHub URL in AGENTS.md
hevolx
left a comment
There was a problem hiding this comment.
Checked thru the changed files, looks good. Going to test in an local clean environment now.
Test Report -
|
| Test | Description | Result |
|---|---|---|
| 1 | 957 tests with 100% coverage | |
| 2 | --agent=opencode → .opencode/commands/ |
✅ |
| 3 | --agent=claude → .claude/commands/ |
✅ |
| 4 | --agent=both → both directories |
✅ |
| 5 | allowed-tools stripped for OpenCode |
✅ |
| 6 | Template injection reads AGENTS.md first |
✅ |
| 7 | claude-instructions alias works |
✅ |
| 8 | Skills to correct agent directory | ✅ |
| 9 | Default OpenCode with warning | ✅ |
Final OpenCode Compatibility AssessmentDid a thorough pass over the full diff before merge. Here's where things stand: What works
Minor notes (non-blocking)
VerdictReady to merge. No blockers found. @hevolx — thanks for cloning the repo, running through the install on Linux, and catching the test timeout and missing build step issues. That kind of hands-on testing is exactly what catches the things automated CI misses (filesystem performance differences, fresh-clone assumptions). Both findings are now addressed in |
fed98c6 to
693df49
Compare
Summary
--agentCLI flag (opencode,claude,both) for targeting specific agents@wbern/claude-instructionsto@wbern/agent-instructionsclaude-instructionsCLI binary as a backwards-compatible aliasallowed-tools) when generating for OpenCodeAGENTS.mdfor OpenCode,CLAUDE.mdfor Claude)Breaking changes
@wbern/claude-instructions→@wbern/agent-instructionsnpm deprecate @wbern/claude-instructions "Renamed to @wbern/agent-instructions"wbern/agent-instructionsbefore merging (so the release lands on the correct repo)Testing this branch
Test plan
--agent=opencodegenerates to.opencode/commands/--agent=claudegenerates to.claude/commands/--agent=bothgenerates to both directoriesallowed-toolsfrontmatter stripped for OpenCode outputAGENTS.mdfirst for OpenCodeclaude-instructionsCLI binary still works as alias--agentomittedcc @hevolx — would appreciate if you could test the branch with OpenCode