Skip to content

fix(plugins): install marketplace plugins under ~/.pizzapi, not ~/.claude - #1017

Merged
Pizzaface merged 1 commit into
mainfrom
fix/plugin-install-pizzapi-dir
Oct 28, 2026
Merged

Pizzaface merged 1 commit into
mainfrom
fix/plugin-install-pizzapi-dir

Conversation

@Pizzaface

Copy link
Copy Markdown
Owner

Problem

/plugin and pizza plugins (marketplace add, install, uninstall, enable/disable) wrote into Claude Code's ~/.claude/ (plugins/known_marketplaces.json, plugins/marketplaces/, plugins/cache/, plugins/installed_plugins.json, settings.json enabledPlugins).

Fix

  • marketplace.ts: all state now lives under ~/.pizzapi/marketplace/, using the same file formats.
  • discover.ts: reads PizzaPi's installed_plugins.json first, then Claude Code's (read-only). ~/.pizzapi/marketplace/settings.json counts as user-level enable state, so project files still can't re-enable a plugin you disabled there.
  • Help text and docs (claude-plugins.mdx, architecture.mdx) updated.

Notes

  • No migration. Plugins installed earlier through PizzaPi stay in ~/.claude/plugins and are still discovered.
  • pizza plugins installed/uninstall now cover only PizzaPi-managed installs.

Tests

  • marketplace.test.ts asserts install paths under ~/.pizzapi/marketplace and that ~/.claude is never created. A new test checks that an installed plugin is discovered, then hidden once disabled.
  • bun test src/plugins src/plugins.test.ts src/extensions/plugin-command.test.ts src/extensions/mcp-overlay.test.ts: all tests pass. tsc --noEmit: clean.

…aude

/plugin and pizza plugins wrote marketplaces, plugin cache, installed_plugins.json
and enabledPlugins into Claude Code's ~/.claude. Move all writes to
~/.pizzapi/marketplace/ (same file formats). Discovery reads both PizzaPi's and
Claude Code's manifests; PizzaPi settings count as user-level enable state.
@Pizzaface
Pizzaface marked this pull request as draft October 28, 2026 17:56
@Pizzaface
Pizzaface marked this pull request as ready for review October 28, 2026 17:57
@Pizzaface
Pizzaface merged commit 88c67c5 into main Oct 28, 2026
14 checks passed
@Pizzaface
Pizzaface deleted the fix/plugin-install-pizzapi-dir branch October 28, 2026 18:06
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