fix(uteke-client): prefer /room/summary-document with legacy fallback#178
Merged
Conversation
uteke #735 (post-v0.8.0-tag develop) renames POST /room/document to
POST /room/summary-document to avoid colliding with the existing
GET /room/summary. The old route still works on newer servers but logs
a deprecation warning and will eventually be removed.
room_document() now prefers /room/summary-document and falls back to
the legacy /room/document on HTTP 404, so Corin works against both the
released v0.8.0 tag (which only has the old route) and the upcoming
release cut from develop.
Also documents /recall and /search compat in the module doc: both
return the stable [{memory, score}] shape across uteke 0.7.x-0.8.x, so
no /api/v1/ prefix is needed (and that prefix is currently broken in
uteke develop anyway due to a leading-slash stripping bug in
ApiVersion::from_path).
🔍 Cora AI Code Review✅ No issues found. Code looks good! Review powered by cora-cli · BYOK · MIT |
Merged
ajianaz
added a commit
that referenced
this pull request
Jul 20, 2026
Patch release — Windows Documents-tab fix, uteke 0.8.x compatibility, and a full dependency sweep (incl. vite 6→8). - bump version 0.3.1 → 0.3.2 (package.json, Cargo.toml, Cargo.lock, tauri.conf.json) - CHANGELOG entry Contents (since v0.3.1): - Fixed: Windows "uteke CLI not found" — HTTP-first version gate reads /health `version`; CLI discovery now PATHEXT-aware (#171, #180) - Fixed: uteke 0.8.x room-document compat — /room/summary-document with legacy /room/document fallback (#178) - Changed: version gate HTTP-first for local servers too (#180) - Deps: vite 6→8 + @sveltejs/vite-plugin-svelte 5→7 (#179), tokio 1.52→1.53 (#175), serde/thiserror/toml/tauri-plugin-dialog, @codemirror/language/dompurify/marked, actions/setup-node 6→7 (#172)
Merged
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
1 participant
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.
What
Migrates
room_document()to the canonical endpoint name introduced by uteke #735 (post-v0.8.0-tag develop), with a backward-compatible fallback so Corin keeps working against the already-releasedv0.8.0tag.Why
Uteke #735 renamed
POST /room/document→POST /room/summary-documentto avoid colliding with the existingGET /room/summary. The old route still works on newer servers but logs a deprecation warning and will eventually be removed.This is the only deprecated/renamed endpoint in uteke develop that Corin uses. Cross-checked every other endpoint Corin calls (
/recall,/search,/doc/*,/memory,/graph/*,/room/*,/tags,/timeline,/edges) — all unchanged and stable.How
room_document()now:POST /room/summary-document(canonical, develop).POST /room/document.Also adds a module-doc compat note and updates the
format_room_documentdoc comment.Compat note:
/api/v1/prefix NOT usedVerified that
/recall(plain, nosearch_type) and/searchreturn the stable[{memory, score}]shape across uteke 0.7.x–0.8.x, so no version prefix is needed. The/api/v1///api/v2/prefix (uteke #737) is currently broken in develop anyway —ApiVersion::from_pathstrips the leading slash so versioned routes 404 against the route match arms.Verification (all green)
cargo fmt --all -- --checkcargo clippy -- -D warningscargo testsvelte-checkvitest --runnpm run buildRust-only change (+43/-1 across 2 files). No frontend touched.