Repository navigation
Conversation
|
Thanks for porting the Marketplace work from #40108 to V2. You clearly put a lot of care into it. At ~19k changed lines across core, protocol, server, clients, app, TUI, CLI and docs, though, it's too large to review well as one pull request, so I'm closing this one. Please split it into smaller, focused pull requests, each of which can be reviewed and tested on its own. For example:
It would also help to agree on the overall design in #28696 before the core pull request, since this adds a new persistent subsystem. You're welcome to open these pull requests. |
|
The focused follow-up is #60821 (local agent catalogs only), now in draft pending the design discussion in #28696. It addresses the package-boundary and agent-name review comments with Core-owned runtime state and qualified agent IDs; 66 tests and full checks pass. I also audited the earlier generated OpenAPI diff: all 138 existing paths and 489 schemas, including their key order, are unchanged. |
Issue for this PR
Closes #28696.
Ports #40108 to the current V2 runtime, following the maintainer's request when that PR was closed.
Type of change
What does this PR do?
Adds an experimental Marketplace for plugins, skills, agents, commands, MCP servers, instructions, and bundles.
/marketplacein the TUI, CLI workflows, and/api/experimental/marketplacewith regenerated clients and OpenAPI.The five commits are based on current upstream
dev(1d479d7642).How did you verify your code works?
bun run check: all 35 typecheck tasks passed; lint reported no errors.typecheck:tsc.Visual
Screenshots / recordings
Checklist