docs(roadmap): record p5 lifecycle verification

This commit is contained in:
Saravana Achu Mac 2026-05-05 11:37:43 -07:00
parent 3aa385774e
commit b1c358def3

View File

@ -175,7 +175,7 @@ Goal: backend behavior is production observable, testable, and consistent with c
- [x] **P5.3** Add health/dependency readiness coverage for datastore, encryption, platform-service, extraction-service, and MCP. Prefer `@bytelyst/monitoring` patterns if suitable. Commit: `874dda2`; Verified: `zsh -lc 'source ~/.zshrc; export GITEA_NPM_TOKEN; pnpm install --offline'` to link local `@bytelyst/monitoring`; `zsh -lc 'source ~/.zshrc; export GITEA_NPM_TOKEN; pnpm --filter @notelett/backend run typecheck'`; `zsh -lc 'source ~/.zshrc; export GITEA_NPM_TOKEN; pnpm --filter @notelett/backend exec vitest run src/lib/dependency-readiness.test.ts src/diagnostics.test.ts'`; `git diff --check`. Added `/api/diagnostics/readiness` with datastore, field-encryption, platform-service, extraction-service, and MCP dependency checks using common-platform monitoring status vocabulary and local MCP registration coverage.
- [x] **P5.4** Add structured error mapping tests for validation, auth, forbidden, not found, conflict, extraction failure, LLM timeout, and blob failure paths. Commit: `ff33a05`; Verified: `zsh -lc 'source ~/.zshrc; export GITEA_NPM_TOKEN; pnpm --filter @notelett/backend run typecheck'`; `zsh -lc 'source ~/.zshrc; export GITEA_NPM_TOKEN; pnpm --filter @notelett/backend exec vitest run src/lib/error-mapping.test.ts src/lib/extraction-client.test.ts src/modules/note-prompts/runner.test.ts'`; `git diff --check`. Added stable dependency error mapping helpers for extraction-service, LLM, and blob failures, mapped extraction HTTP/fetch failures and LLM timeout failures to structured `ServiceError` responses, and added regression tests for validation, auth, forbidden, not-found, conflict, extraction failure, LLM timeout, and blob failure response shapes.
- [x] **P5.5** Review Cosmos partition keys and query patterns for current containers; document any cross-partition or count-heavy operations and add repository tests for scope isolation. Commit: `205c44b`; Verified: `zsh -lc 'source ~/.zshrc; export GITEA_NPM_TOKEN; pnpm --filter @notelett/backend run typecheck'`; `zsh -lc 'source ~/.zshrc; export GITEA_NPM_TOKEN; pnpm --filter @notelett/backend exec vitest run src/modules/repository-scope.test.ts src/modules/note-shares/repository.test.ts src/modules/note-agent-actions/routes.integration.test.ts'`; `git diff --check`. Added `docs/COSMOS_QUERY_REVIEW.md` documenting container partition keys, intentional cross-partition/count-heavy operations, guardrails, and follow-ups; added repository regression tests for notes, agent actions, shares, and collaborators across user/product/workspace/note scope boundaries.
- [ ] **P5.6** Verify webhook/scheduler loops shut down cleanly on Fastify close and do not start unexpectedly in tests. Commit:
- [x] **P5.6** Verify webhook/scheduler loops shut down cleanly on Fastify close and do not start unexpectedly in tests. Commit: `3aa3857`; Verified: `zsh -lc 'source ~/.zshrc; export GITEA_NPM_TOKEN; pnpm --filter @notelett/backend run typecheck'`; `zsh -lc 'source ~/.zshrc; export GITEA_NPM_TOKEN; pnpm --filter @notelett/backend exec vitest run src/modules/note-prompts/scheduler.test.ts src/lib/webhook-subscriber.test.ts src/server.test.ts'`; `git diff --check`. Added scheduler running-state checks, made webhook subscriber initialization idempotent with test reset helpers, verified route registration does not start the scheduler loop, and asserted server `onClose` stops both scheduler and webhook subscriber.
Acceptance criteria:
- Backend production logs are structured.