Skip to content

fix(marketplace): store marketplaces under plugins directory#93

Merged
christso merged 2 commits intomainfrom
fix/marketplace-path
Feb 6, 2026
Merged

fix(marketplace): store marketplaces under plugins directory#93
christso merged 2 commits intomainfrom
fix/marketplace-path

Conversation

@christso
Copy link
Copy Markdown
Contributor

@christso christso commented Feb 6, 2026

Summary

  • getMarketplacesDir() returned ~/.allagents/marketplaces instead of ~/.allagents/plugins/marketplaces, causing marketplace data to be stored in two separate locations when running marketplace add followed by plugin install
  • Fixed the path to use .allagents/plugins/marketplaces consistently, matching getPluginCachePath()
  • Updated the hardcoded test path in marketplace-update.test.ts

Test plan

  • All 41 marketplace tests pass
  • Pre-commit hooks (lint, typecheck, test) pass

getMarketplacesDir() returned ~/.allagents/marketplaces instead of
~/.allagents/plugins/marketplaces, causing marketplace data to be
stored in two separate locations.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Feb 6, 2026

Deploying allagents with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7178155
Status: ✅  Deploy successful!
Preview URL: https://7d3bf17d.allagents.pages.dev
Branch Preview URL: https://fix-marketplace-path.allagents.pages.dev

View logs

The test compared raw process.env.HOME against a resolve()'d path,
causing a mismatch on Windows when path separators differ.
@christso christso merged commit a189e31 into main Feb 6, 2026
1 check passed
@christso christso deleted the fix/marketplace-path branch February 6, 2026 05:15
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