bytelyst/learning_ai_notes
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs(roadmap): record p5 request id propagation

Browse Source
This commit is contained in:
Saravana Achu Mac 2026-05-05 11:06:29 -07:00
parent c31f51ddbd
commit 19cc34b172
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
docs/PRODUCTION_READINESS_HANDOFF_ROADMAP.md
View File

@ -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: