Skip to content

fix: run npm install after extension copy in setup.sh#36

Open
madtank wants to merge 2 commits intomainfrom
fix/setup-npm-install
Open

fix: run npm install after extension copy in setup.sh#36
madtank wants to merge 2 commits intomainfrom
fix/setup-npm-install

Conversation

@madtank
Copy link
Copy Markdown
Member

@madtank madtank commented Feb 24, 2026

Summary

  • setup.sh sync does rm -rf on the extension dir then copies fresh files via plugins install, but never runs npm install
  • This causes the plugin to fail with "Cannot find module" errors (e.g. @sinclair/typebox) after a fresh sync
  • Adds npm install --production step after extension copy

Test plan

  • Run ./setup.sh sync on a fresh gateway install
  • Verify plugin loads without "Cannot find module" errors
  • Verify gateway logs show "Plugin loaded successfully"

🤖 Generated with Claude Code

madtank and others added 2 commits February 23, 2026 22:20
plugins install copies extension files but doesn't install npm
dependencies. After rm -rf of the extension dir, node_modules are
lost and the plugin fails to load with "Cannot find module" errors.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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