Skip to content

fix: reliable SDK resolution + CI plugin sync#85

Merged
George-iam merged 2 commits intomainfrom
feat/plugin-repo-sync-20260408
Apr 8, 2026
Merged

fix: reliable SDK resolution + CI plugin sync#85
George-iam merged 2 commits intomainfrom
feat/plugin-repo-sync-20260408

Conversation

@George-iam
Copy link
Copy Markdown
Contributor

Summary

  • Install SDK into CLAUDE_PLUGIN_ROOT via SessionStart hook (ESM resolves naturally)
  • Remove NODE_PATH hacks from bin wrapper and MCP server
  • Add CI step to sync plugin bundle to axme-code-plugin repo
  • Inform user before setup ("Initializing, 1-2 min")

Verified

  • LLM scan works: 22-25 decisions from LLM
  • bin wrapper works
  • node cli.mjs works
  • SessionStart installs SDK correctly

After merge

Need PLUGIN_REPO_TOKEN secret (already added).

🤖 Generated with Claude Code

George-iam and others added 2 commits April 8, 2026 17:43
- bin/axme-code: search ~/.claude/plugins/data/axme-code* instead of
  hardcoded fallback (plugin data dir name varies)
- axme_context: instruct agent to tell user "Initializing, 1-2 min"
  before running setup

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
ESM import resolves packages relative to the importing file, not via
NODE_PATH. Previous approach (SDK in CLAUDE_PLUGIN_DATA with NODE_PATH)
failed because ESM ignores NODE_PATH.

New approach: SessionStart hook runs npm install in CLAUDE_PLUGIN_ROOT
itself, placing node_modules/ next to cli.mjs. ESM standard resolution
finds SDK naturally. Simpler bin wrapper, no NODE_PATH hacks.

Docs confirm CLAUDE_PLUGIN_ROOT is writable (but doesn't survive
plugin updates — SDK re-installs on update, which is fine).

Verified: LLM scan works (21 decisions, 8 from LLM, $0.39).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@George-iam George-iam merged commit 80c2f78 into main Apr 8, 2026
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