Skip to content

fix: expose runtime deps to copied user adapters#798

Merged
jackwener merged 3 commits intomainfrom
fix-issue-792
Apr 5, 2026
Merged

fix: expose runtime deps to copied user adapters#798
jackwener merged 3 commits intomainfrom
fix-issue-792

Conversation

@jackwener
Copy link
Copy Markdown
Owner

@jackwener jackwener commented Apr 5, 2026

Summary

  • expose OpenCLI runtime dependencies to ~/.opencli/node_modules/ alongside the @jackwener/opencli package symlink
  • resolve dependency package roots via Node module resolution so both hoisted and local installs work
  • cover copied user adapters importing bare deps like chalk in engine tests

Testing

  • npm test -- --run src/engine.test.ts
  • npm run build
  • HOME=$(mktemp -d) node dist/src/main.js --version

@jackwener jackwener merged commit 8c41411 into main Apr 5, 2026
11 checks passed
@jackwener jackwener deleted the fix-issue-792 branch April 5, 2026 09:31
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.

1 participant