From 19cc34b172dd6ab207e09951b91f6eb71bdd2664 Mon Sep 17 00:00:00 2001 From: Saravana Achu Mac Date: Tue, 5 May 2026 11:06:29 -0700 Subject: [PATCH] docs(roadmap): record p5 request id propagation --- docs/PRODUCTION_READINESS_HANDOFF_ROADMAP.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/PRODUCTION_READINESS_HANDOFF_ROADMAP.md b/docs/PRODUCTION_READINESS_HANDOFF_ROADMAP.md index ab975d8..b727230 100644 --- a/docs/PRODUCTION_READINESS_HANDOFF_ROADMAP.md +++ b/docs/PRODUCTION_READINESS_HANDOFF_ROADMAP.md @@ -171,7 +171,7 @@ Acceptance criteria: Goal: backend behavior is production observable, testable, and consistent with common platform service conventions. - [x] **P5.1** Replace raw `process.stdout.write` / `process.stderr.write` in backend startup paths with app logger or shared logger-compatible handling. Commit: `7016fc1`; Verified: `GITEA_NPM_TOKEN=dummy pnpm --filter @notelett/backend run typecheck`; `GITEA_NPM_TOKEN=dummy pnpm --filter @notelett/backend exec vitest run src/server.test.ts src/modules/note-prompts/scheduler.test.ts`; `rg -n "process\\.(stdout|stderr)\\.write" backend/src --glob '!**/*.test.*'` returned no matches; `git diff --check`. Routed Cosmos startup diagnostics through `@bytelyst/logger`, passed the Fastify app logger into the scheduler loop, and updated server bootstrap coverage to assert logger propagation. -- [ ] **P5.2** Ensure request IDs are propagated to extraction-service, platform-service, blob, LLM, webhook, and MCP calls where supported. Add tests for local propagation helpers. Commit: +- [x] **P5.2** Ensure request IDs are propagated to extraction-service, platform-service, blob, LLM, webhook, and MCP calls where supported. Add tests for local propagation helpers. Commit: `c31f51d`; Verified: `GITEA_NPM_TOKEN=dummy pnpm --filter @notelett/backend run typecheck`; `GITEA_NPM_TOKEN=dummy pnpm --filter @notelett/backend exec vitest run src/lib/request-context.test.ts src/lib/extraction-client.test.ts src/lib/field-encrypt.test.ts src/server.test.ts src/modules/notes/routes.test.ts`; request propagation audit confirmed note summarization passes inbound `x-request-id` to extraction-service, startup platform-service flag polling sends a generated outbound `x-request-id`, MCP write audit trails already default correlation IDs from `req.id`, and no backend blob/LLM/webhook local call path exposes supported request-id headers beyond shared package ownership; `git diff --check`. - [ ] **P5.3** Add health/dependency readiness coverage for datastore, encryption, platform-service, extraction-service, and MCP. Prefer `@bytelyst/monitoring` patterns if suitable. Commit: - [ ] **P5.4** Add structured error mapping tests for validation, auth, forbidden, not found, conflict, extraction failure, LLM timeout, and blob failure paths. Commit: - [ ] **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: