feat: ship the AGH Marketplace - #318
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedToo many files! This PR contains 619 files, which is 469 over the limit of 150. To get a review, narrow the scope: Upgrade to Pro+ to raise the limit. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (463)
📒 Files selected for processing (630)
You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests (beta)
|
|
React Doctor found no new issues. 🎉 Reviewed by React Doctor for commit |
Review-scoped core of the marketplace implementation (299 files) sized to fit CodeRabbit's 300-file review window. Shared @agh/ui primitives, peripheral web systems, storybook, and lint tooling are left unstaged for a follow-up commit. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Summary
Changes
/api/marketplace, matching UDS and CLI clients, andagh__marketplace_search; lifecycle remains available through its owning CLI/HTTP/UDS surfaces with structured output.Release Notes
🎉 Features
🐛 Bug Fixes
♻️ Refactoring
📚 Documentation
📦 Build System
QA
Final Status: PASS after fixes. The strict Northstar auditor reports zero blockers and zero warnings; all owned QA and visual processes have clean teardown with no survivors.
Coverage: Six personas completed six Marketplace/provider charters covering 50 reconciled scenarios, plus one real-provider Northstar kickoff that completed all 12 root tasks and three disruption recoveries.
Issues: 15 tracked QA defects and 42 official review findings were corrected; no completion-blocking, data-loss, trust-damage, friction, or cosmetic issue remains open in the release scope.
Evidence:
Blocked / needs human: None.
AGH Impact Audit
agh__marketplace_searchis the canonical native discovery ID; descriptors, schemas, capability gates, exact inventory, and caller-workspace tests co-ship. Native config mutation exposes only safe Marketplace timing keys and projects real apply records.skills/agh/documents the shipped Marketplace, MCP, extension, bundle, config, scheduler, and native-tool behavior without claiming unsupported mutations.Migrations and compatibility
00006and00007for definition-bound MCP OAuth credentials and legacy-ref garbage collection; existing migration bytes remain immutable.Test plan
make verify— codegen, installer, Bun lint/typecheck/test, Web build, Go fmt/lint/test/build, and package boundaries; 15,769 race-enabled Go tests passed.audit-qa-evidence.py --strictagainst the completed Northstar lab — PASS, 0 blockers, 0 warnings.make test-e2e-runtimeandmake test-e2e-web— runtime lane passed and 78/78 Playwright journeys passed in the Marketplace QA cycle.-raceowner suites for Marketplace, API/spec/transports, CLI, daemon, settings, extension, MCP auth/executor, global store, config, and native tools.