refactor(api): rename POST /room/document → POST /room/summary (#735)#739
Merged
Conversation
BREAKING: The room summary document endpoint is renamed from POST /room/document to POST /room/summary to eliminate confusion with the room↔document junction routes (/room/document/add, /room/document/list, /room/document/remove). Changes: - Server: POST /room/summary is the new primary endpoint - Server: POST /room/document retained as deprecated alias with warn! log - Core: room_document() → room_summary_document() across all crates - CLI: 'uteke room document' command unchanged (help text updated) - MCP: uteke_room_document tool updated to use new function - Docs: cli-reference, docker, mcp, rooms updated Closes #735
🔍 Cora AI Code Review❌ Blocked — critical issues found. 🔴 Error (1)
Review powered by cora-cli · BYOK · MIT |
…om/summary Clippy caught unreachable pattern: POST /room/summary already used by room_summary() (tag clustering). Renamed to POST /room/summary-document to eliminate ambiguity with both junction routes AND the clustering endpoint. Refs #735
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| "/room/recall", | ||
| "/room/summary", | ||
| "/room/document", | ||
| "/room/summary-document", |
This was referenced Jul 17, 2026
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
2 participants
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.
Summary
Renames
POST /room/document(room summary document generation) toPOST /room/summaryto eliminate route naming confusion with the room↔document junction endpoints.Changes
Server (uteke-server)
POST /room/summary— new primary endpoint (generates structured document from room)POST /room/document— deprecated alias, still works but logswarn!with deprecation noticeroom_document()→room_summary_document()called by new handlerCore (uteke-core)
MemoryStore::room_document()→MemoryStore::room_summary_document()Uteke::room_document()→Uteke::room_summary_document()(public API wrapper)room_documentsunchanged (not a route rename)CLI (uteke-cli)
uteke room document <id>command unchanged (still works)room_summary_document()internallyMCP (uteke-mcp)
uteke_room_documenttool updated to callroom_summary_document()Docs
cli-reference.md— route table updateddocker.md— endpoint reference updatedmcp.md— MCP tool reference updatedrooms.md— usage example updated with noteNOT Changed
/room/document/add,/room/document/list,/room/document/remove— junction routes unchanged/doc/room/list,/doc/mem-refs,/memory/doc-refs— cross-reference routes unchangedroom_documents— unchangedroom document— unchangedMigration
POST /room/documentcontinue to work (deprecated alias)POST /room/summaryDEPRECATED: POST /room/document is renamed to POST /room/summary (see #735)Closes #735