chore(vendor): bump tinycortex to 26d8833 (Notion sync + digest fixes) — Closes #5500, #5509, #5510 - #5585
Draft
YellowSnnowmann wants to merge 1 commit into
Conversation
Advances the vendored tinycortex submodule from 5fdeac98 (tinyhumansai#148) to main HEAD 26d8833, landing three merged fixes on openhuman main: - tinyhumansai#146 render Notion database-row properties into the synced document (select/status/multi_select/date/people/relation/scalar) — memory-sync read path for openhuman#5500. - tinyhumansai#145 retain observations when a digest is truncated; raise DIGEST_MAX_OUTPUT_TOKENS 4096->16384 and return a non-committable Err on a truncated digest so the window retries instead of silently dropping — openhuman#5509 (Bug 2) / tinyhumansai#5510. - tinyhumansai#144 send fetch_type in NOTION_FETCH_DATA sync args — openhuman#5506. Intervening commits pulled in: tinyhumansai#149 (re-export the TinyMemory contract, additive) and tinyhumansai#153 (move the host-side Composio normalisers to tinymemory-sync). openhuman src references none of the dropped providers::normalize symbols, so tinyhumansai#153 is source-compatible here.
Contributor
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
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.
Summary
tinycortexsubmodule from5fdeac98(Update issue templates #148) to main HEAD26d8833.main: Notion database-row property rendering (fix(skills): persist OAuth credentials and fix skill auto-start lifecycle #146), truncated-digest retention ([Feature] Subconscious loop: local-model state review (replaces heartbeat) #145), andfetch_typein Notion sync args ([Bug] Agent loop: hallucination and missing tool calls (noisy prompts / loop integration) #144).Problem
The tinycortex fixes for three openhuman bugs are merged into tinycortex
mainbut were not yet reachable from openhuman because the submodule pointer still sat five commits behind. Until this bump, the bugs still reproduce on openhumanmain.Solution
Advance the gitlink to tinycortex
mainHEAD26d8833. The pointer range5fdeac98..26d8833contains exactly:select/status/multi_select/date/people/relation/scalar Notion DB-row properties into the synced document (memory-sync read path for Bug: agent misreads Notion page content — invents values, reports wrong dropdown selections #5500).DIGEST_MAX_OUTPUT_TOKENS4096→16384 and return a non-committableErron truncation so the window retries instead of silently committing the cursor (Bug: bulk session ingest timeout — rpc.rs 30s/session budget ignores multi-window sessions, DIGEST_MAX_OUTPUT_TOKENS=4096 silently drops observations #5509 Bug 2 / Bug: DIGEST_MAX_OUTPUT_TOKENS=4096 truncates digest responses mid-JSON — soft-fail silently drops window observations and commits cursor #5510).fetch_typeinNOTION_FETCH_DATAsync args (Bug: Notion memory sync fails — tinycortex provider missing fetch_type in NOTION_FETCH_DATA args (ce98837 unfixed) #5506).tinymemory-sync). openhumansrc/references none of the droppedproviders::normalizesymbols, so feat(update): core binary self-update via GitHub Releases #153 is source-compatible here.Submission Checklist
N/A: submodule-pointer bump; behavioural tests ship in tinycortex [Bug] Agent loop: hallucination and missing tool calls (noisy prompts / loop integration) #144/[Feature] Subconscious loop: local-model state review (replaces heartbeat) #145/fix(skills): persist OAuth credentials and fix skill auto-start lifecycle #146.N/A: no changed source lines (gitlink only).N/A: behaviour-only change carried by the submodule.## Related.N/A.N/A: no release-cut surface changed in-repo.Closes #NNN— see## Related.Impact
Restores correct Notion memory sync (properties +
fetch_type) and stops silent observation loss on truncated digests. No API or migration impact on openhuman; gitlink advance only.Related
main.AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch