docs(roadmap): record llm router decision

This commit is contained in:
Saravana Achu Mac 2026-05-05 09:37:40 -07:00
parent 744b938e8d
commit e9afb9fa88

View File

@ -127,7 +127,7 @@ Goal: prove that NoteLett is using common platform services and packages at runt
- [x] **P2.4** Verify web and mobile shared clients propagate product identity, auth token, and request IDs where supported by common platform clients. Add tests where behavior is local. Commit: `efa2097`; Verified: `GITEA_NPM_TOKEN=dummy pnpm --filter @notelett/web exec vitest run src/lib/api-helpers.test.ts src/lib/platform.test.ts`; `GITEA_NPM_TOKEN=dummy pnpm --filter @notelett/mobile exec vitest run src/api/client.test.ts src/lib/platform-api.test.ts`; `git diff --check`. Added local web/mobile tests proving product API clients send `x-product-id`, bearer auth, and shared `@bytelyst/api-client` request IDs, and proving platform-client wrappers are configured with NoteLett product identity plus auth-token accessors.
- [x] **P2.5** Decide whether mobile should adopt `@bytelyst/react-native-platform-sdk` now or explicitly defer it. If adopted, replace redundant local composition; if deferred, document why direct clients are still preferred. Commit: `bea7579`; Verified: `git diff --check`; `rg -n "react-native-platform-sdk|direct shared-client|MOBILE_PLATFORM_SDK_DECISION|Mobile Platform SDK Decision" docs/ROADMAP.md docs/MOBILE_DELEGATION_ROADMAP.md docs/roadmaps/04_MOBILE_ROADMAP.md docs/MOBILE_PLATFORM_SDK_DECISION.md`. Deferred direct SDK adoption in `docs/MOBILE_PLATFORM_SDK_DECISION.md` because current mobile already uses direct shared `@bytelyst/*` clients and the SDK does not yet cover product-backend API calls, blob uploads, diagnostics metadata, billing, feedback, offline queue, or the existing MMKV auth-client token contract without duplicate initialization.
- [x] **P2.6** Decide whether `@bytelyst/sync` should back mobile offline/sync behavior now or be deferred. Record the decision and update implementation accordingly. Commit: `f90f358`; Verified: `git diff --check`; `rg -n "@bytelyst/sync|MOBILE_SYNC_DECISION|Mobile Sync Decision|offline-queue" docs/MOBILE_SYNC_DECISION.md docs/MOBILE_DELEGATION_ROADMAP.md docs/roadmaps/04_MOBILE_ROADMAP.md docs/ROADMAP.md`. Deferred `@bytelyst/sync` in `docs/MOBILE_SYNC_DECISION.md` because current mobile uses a narrower `@bytelyst/offline-queue` retry model and full sync adoption needs explicit backend sync windows, tombstones, conflict strategy, MMKV migration, telemetry, and stale/conflict UX.
- [ ] **P2.7** Evaluate backend LLM usage against `@bytelyst/llm-router`. Either adopt the router for provider/model/fallback governance or document why `@bytelyst/llm` remains sufficient for NoteLett release 1. Commit:
- [x] **P2.7** Evaluate backend LLM usage against `@bytelyst/llm-router`. Either adopt the router for provider/model/fallback governance or document why `@bytelyst/llm` remains sufficient for NoteLett release 1. Commit: `744b938`; Verified: `git diff --check`; `rg -n 'llm-router|BACKEND_LLM_ROUTER_DECISION|Backend LLM Router Decision|@bytelyst/llm' docs/BACKEND_LLM_ROUTER_DECISION.md docs/ROADMAP.md docs/SMART_ACTIONS_ROADMAP.md docs/PRODUCTION_READINESS_HANDOFF_ROADMAP.md`. Deferred router adoption in `docs/BACKEND_LLM_ROUTER_DECISION.md` because release-1 backend needs `@bytelyst/llm` chat, embeddings, mock provider, Azure/OpenAI env compatibility, and multipart vision prompt support; current `@bytelyst/llm-router` is useful future governance but does not yet replace the full `LLMProvider` contract.
Acceptance criteria:
- Platform-service, extraction-service, blob, telemetry, diagnostics, flags, kill switch, and MCP are documented with real smoke steps.