Skip to content

chore(vendor): bump openhuman to the tinymemory-38a34d2 pin - #1271

Closed
YellowSnnowmann wants to merge 2 commits into
tinyhumansai:mainfrom
YellowSnnowmann:chore/bump-openhuman-tinymemory
Closed

YellowSnnowmann wants to merge 2 commits into
tinyhumansai:mainfrom
YellowSnnowmann:chore/bump-openhuman-tinymemory

Conversation

@YellowSnnowmann

Copy link
Copy Markdown
Collaborator

Summary

Bump vendor/openhuman ded703dd → 3911b7b8 (openhuman#5608's head: upstream main 92bab8df + the tinymemory pin bump).

What rides in, for the memory surfacing work this unblocks:

  • tinymemory dc3a725 → 38a34d2 — the vendored tinymemory jumps from a mid-arc Aug-17 commit (97 behind) to the completed tinymemory#18 arc. Concretely: the tinymemory-conformance crate exists on disk (the next PR wires it into a CI lane), and the hosted adapters gain the feat(chat): surface scrubbed tool-call/thinking steps in the operator chat #63Add a company→operator attention signal (bidirectional chat via GET /events SSE) #66 production fixes — dead Cognee default endpoint removed, Mem0 cloud search no longer 400s on a null threshold, API keys no longer echoed into error text, named transport-failure classes, incremental response-body cap. Today's vendored adapters have all of those bugs, which is what made this bump a prerequisite for enabling tinymemory in shipped builds.
  • tinycortex 8401346b → bfd2ea21 — openhuman#5605's pin ("cc-corroboration").
  • Everything else openhuman merged between ded703dd and current main.

Two OpenCompany-side changes ride along, both mechanical:

  • New [patch."https://github.com/tinyhumansai/tinymemory"] table in the root Cargo.toml. The new tinycortex pin's tinycortex-api takes tinymemory-api by git (tinycortex#149), and openhuman's own patch table unifying that onto its vendored path copy is ignored when openhuman is a path dependency — the WS4 replication rule already documented above [patch.crates-io]. Without the entry cargo resolves two tinymemory-api crates and tinymemory-core fails with E0308s where the engine trait meets the contract (hit exactly as the handoff's "two-copies hazard" predicts; found by the compiler, fixed with the one-line patch mirroring openhuman's own).
  • Cargo.lock regenerated for the vendored crate-version moves (openhuman 0.63.11 → 0.63.13, new tinymemory-sources/tinymemory-sync path crates, toml/winnow transitive bumps).

No OpenCompany source changes.

Stacking

Depends on openhuman#5608 (draft). This PR pins its head commit so the chain can be validated now; re-pin to the merge commit once #5608 lands, before this merges.

Follow-ups that stack on this: the memory CI lane (union build + vendored driver-conformance), then enabling tinymemory in the staging tenant feature set.

Verification

  • scripts/ci/init-vendored-submodules.sh clean at the new pin
  • cargo metadata --locked resolves after the lock regen
  • cargo check --features openhuman,tinycortex (the gated tenant shape)
  • cargo check --features acp,runner,tinymemory-embedded (the memory-seam shape)

🤖 Generated with Claude Code

Advances vendor/openhuman ded703dd -> 3911b7b8 (openhuman#5608's head:
upstream main plus the tinymemory pin bump). The vendored tinymemory
jumps dc3a725 -> 38a34d2 (the completed tinymemory#18 arc: the
conformance crate the next PR wires into CI, and the tinyhumansai#63-tinyhumansai#66
hosted-adapter production fixes); tinycortex rides to bfd2ea21.

Adds the [patch] table unifying tinycortex-api's by-git tinymemory-api
onto the vendored path copy — the same WS4 replication rule as the
existing [patch.crates-io] block; without it cargo resolves two
tinymemory-api crates and tinymemory-core fails with E0308s.

Cargo.lock regenerated. Verified: cargo metadata --locked, then
cargo check --locked under openhuman,tinycortex and under
acp,runner,tinymemory-embedded.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e4462003-21d4-4003-af56-21d9a7ab50a2

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@YellowSnnowmann

Copy link
Copy Markdown
Collaborator Author

Superseded by #1248, which carries the same vendor/openhuman bump to upstream main (92bab8df) and the identical [patch."…/tinymemory"] table — plus the #1201 fix itself. The one piece this PR had beyond that (advancing the vendored tinymemory f8bd9af → 38a34d2 via openhuman#5608, the #63#66 hosted-adapter fixes) now rides #1273 as a single commit stacked on #1248. Merge order: openhuman#5608 → #1248#1273.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant