docs(mcp-server): post-audit — document /api prefix bug, lysnrai path fixes, +17 gap tools

This commit is contained in:
saravanakumardb1 2026-03-05 13:16:07 -08:00
parent 481760fba1
commit ecfbaa62b6

View File

@ -132,7 +132,32 @@ New files per product: `src/lib/<product>-client.ts` + `src/modules/<product>/<p
- [x] 6 new env vars in `config.ts` + `.env.example`: `MINDLYST/LYSNRAI/JARVISJR/CHRONOMIND/NOMGAP/PEAKPULSE_BACKEND_URL`
## 7) Phase 3+ quick sanity checks
## 7) Phase 3 Post-Audit — Bug Fixes & Gap Fill ✅ COMPLETE
### Commit c902266 — Critical bug fixes
- [x] **`/api` prefix missing on ALL 6 product clients** — every route was 404 in production
- Root cause: all backends register routes with `{ prefix: '/api' }` but clients called paths without it
- Fix: single-line change per client — `${BASE_URL}/api${path}` in all 6 fetch helpers
- [x] **LysnrAI client path `/organizations` → `/orgs`** — backend route is `GET /orgs` not `/organizations`
- [x] **LysnrAI `lysnraiOrgsList` / `lysnraiApiTokensList` had fake `total` field** — backend doesn't return it
- [x] **`lysnraiApiTokenRotate` calls non-existent endpoint** — `/api-tokens/:id/rotate` doesn't exist in backend; function and `lysnrai.apiTokens.rotate` tool both removed
- [x] **`server.ts` description** — updated to include chronomind/nomgap/peakpulse namespaces
### Commit 481760f — DOMAIN_PRODUCTS.md gap fill (+17 tools, total now 52)
| Product | New tools |
| -------------- | -------------------------------------------------------------------------------------------------------------------------------------------------- |
| **MindLyst** | `mindlyst.memory.get` — single item + full TriageResult |
| **LysnrAI** | `lysnrai.transcripts.get`, `lysnrai.orgs.get` |
| **JarvisJr** | `jarvis.agents.create`, `jarvis.marketplace.listPending`, `jarvis.marketplace.certify`, `jarvis.marketplace.suspend`, `jarvis.marketplace.feature` |
| **ChronoMind** | `chronomind.timers.create`, `chronomind.households.list` |
| **NomGap** | `nomgap.protocols.list`, `nomgap.bodyStages.list` |
| **PeakPulse** | `peakpulse.sessions.get` |
All new client functions verified against actual backend route files before addition.
## 8) Phase 3+ quick sanity checks
- If you make telemetry policy-aware clients:
- ensure `GET /api/telemetry/config` consumption is cached (ETag) and privacy-safe.