test(backend): expand unit coverage; add docs & architecture diagrams - #28
Merged
Conversation
Tests - Add comprehensive MessagesService streaming/HITL/error-mapping tests (sendMessage stream + persistence, permission/clarification resume & event-merge, agent error mapping 502/503/504, malformed-line handling). messages.service.ts coverage 15% -> 98%. - Cover the @res() streaming controller handlers, _addSession, and the null-title session mapper branch. - Add message.findFirst to the shared Prisma mock. - Enforce a coverage floor via jest coverageThreshold (lines/statements 80, branches 75, functions 85). Overall line coverage 54% -> 87%, 78 -> 105 tests. Docs - Fix inaccuracies in README and DOCUMENTATION (agent resume endpoints, handleEvent word re-chunking/pacing, DATABASE_URL resolution, method counts, corrected route table). - Rename Docs/ -> docs/ and add docs/TESTING.md (coverage & methodology). Diagrams - Add diagrams/generate_diagrams.py (Kroki/PlantUML) plus 17 rendered, A4-friendly PNGs covering architecture, sequences, flows and data model. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tests
Docs
Diagrams