chore(vendor): bump tinymemory pin to 38a34d2 (#63–#66 hosted-adapter fixes) - #5608
Conversation
Advances vendor/tinymemory f8bd9af -> 38a34d2, the tail of the tinymemory#18 arc (PRs tinyhumansai#63-tinyhumansai#66): hosted-adapter endpoint fixes found by live testing (dead Cognee default endpoint, Mem0 null-threshold 400 on cloud search, API keys echoed into error text), the named transport-failure classes, and the incremental response-body cap. Crate versions are unchanged so Cargo.lock is untouched; verified with cargo check --locked at the root (all four tinymemory crates link unconditionally). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe ChangesTinyMemory submodule update
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This is a localized vendor pin update with no actionable merge-blocking risk remaining; it is merge-ready after normal checks and review. Possibly related PRs
Suggested labels: Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Bump
vendor/tinymemoryf8bd9af → 38a34d2 — the last 15 commits of the tinymemory#18 arc (PRs #63–#66), which are the hosted-adapter production fixes:api.cognee.aidefault (answers no TLS handshake; Cognee Cloud is per-tenant), stops sending"threshold": nullon Mem0 cloud search (the platform 400s on it, so store/list succeeded while recall failed), adds Mem0's cloud (v3/v1) API flavour, stops echoing API keys back out in error text, bounds the hosted listing walk.Crates this repo links (
tinymemory,tinymemory-api,tinymemory-core,tinymemory-tinycortex) see a small delta:api/src/lib.rs(4 lines),adapters/tinycortex(+30),core/.../namespace_store(+208,documents.rs/init.rs). Crate versions are unchanged, soCargo.lockis untouched. The bulk of the range isadapters/remoteand the conformance suite, which this repo does not link.Why now
OpenCompany is about to enable its
tinymemoryfeature in shipped tenant builds (surfacing the memory engines end to end). Its vendored tinymemory rides this submodule, and the current pin ships the exact hosted-adapter bugs #66 fixed. This is the same lockstep as #5587: tinymemory moves here first, OpenCompany bumpsvendor/openhumansecond.Verification
cargo check --lockedat the workspace root (root crate links all four tinymemory crates unconditionally)🤖 Generated with Claude Code
Summary by CodeRabbit