Repository navigation
Storage: transcript/tool-result stores on driver ports, residual failures and coverage (follow-up to #7783) #7809
Description
Part of #7522
Follow-up to #7783 (closed by #7807 for everything that was in reach).
Remaining
- tinyagents: move the path-bound JSONL transcripts and
ToolResultArtifactStoreonto driver ports (tracked upstream in Storage: move JSONL transcripts and the tool-result artifact store onto driver ports tinyagents#453; artifact offload and media already moved in tinyagents#446). - Fix
channels::...::channel_processed_event_records_resolved_agent_route, which fails only in the full lib suite, on upstream/main as well as on the storage(#7783): cross-process claim for legacy importers, triage and coverage #7807 branch. - Raise diff coverage for storage(#7638): tinyagents v2.2.0 driver stores, default sqlite backend, classic mode retired #7686 above the 80% gate (75.1% measured, 1115/1485;
ghcaps at 100 commits so the figure undercounts added lines). - Fix
pnpm rust:layouton upstream/main:orchestrator/prompt_tests.rsis 751 lines against a 750 limit.
Activity
tinysweeper commentedon Oct 14, 2026
Follow-up tracking issue for residual storage/transcript work after #7783: moving stores onto driver ports, a flaky test in the full suite, raising diff coverage above the 80% gate, and fixing rust:layout line-limit violations. A collection of assorted remaining tasks rather than a single defect, so triaged as a task with p2 priority (broken gates have tracked follow-ups and no outage or data loss). No candidate duplicates exist, so prior art is null.
Labelled
priority: p2.senamakel commentedon Oct 14, 2026
openhuman#7821: fixes the resolved-route channel test (bus ring overwrite), raises #7686 changed-line coverage 74.2% -> 83.4%; layout already green; tinyagents#453 re-pin still pending (#7821)
senamakel commentedon Oct 14, 2026
openhuman#7821: fixes order-dependent channel test and raises #7686 changed-line coverage (#7821)
Merge pull request #7821 from senamakel/storage-7809-host-1
senamakel commentedon Oct 14, 2026
Progress: PRs in this stage: #7821 (merged) fixes the channel test (bus ring overwrite), raises #7686 changed-line coverage 74.2% -> 83.4%; rust:layout already green on main (prompt_tests.rs now 633 lines).
REMAINING: tinyagents JSONL transcripts + ToolResultArtifactStore onto driver ports (tinyhumansai/tinyagents#453, issue still open; only #456 artifact store merged). After it lands, re-pin vendor/tinyagents here. Also noted: composio test direct_connected_integrations_fetches_schemas_without_backend_composio_route fails in the full lib run before and after #7821 (not in scope).
senamakel commentedon Oct 15, 2026
Constraint for the host PR (coordination with the #7801 context work): the session_store / session_host adaptation must not add new calls to
CoreContext::current()/CoreContext::scope()/scoped/current_slot/DispatchCtx::ambient, and must not readcurrent_tenant/profile_keyambiently. Take the context (or the scoped storage handle) as an explicit parameter instead; those APIs are being deleted by the #7801 leaves. Merge upstream/main right before merging and keep the newer vendor/tinyagents main sha.senamakel commentedon Oct 15, 2026
tinyagents#461: WorkspaceStores + one-time read-only JSONL transcript import onto driver ports (first slice of tinyagents#453 transcript half; removal of path API deferred) (tinyhumansai/tinyagents#461)
senamakel commentedon Oct 15, 2026
tinyagents#461 merged (0b03f02): workspace-dir transcript stores over tsd with one-time legacy JSONL import (tinyhumansai/tinyagents#461)
senamakel commentedon Oct 15, 2026
Progress (stage close-out): PRs in this stage: tinyhumansai/tinyagents#461 (merged, 0b03f02, released as v2.7.0): WorkspaceStores plus one-time read-only JSONL transcript import onto driver ports. Earlier: #7821 (merged), tinyagents#446/#456.
No checkbox ticked: the remaining item (transcripts + ToolResultArtifactStore on driver ports, tinyagents#453) is only partly done.
REMAINING:
senamakel commentedon Oct 15, 2026
Note for the storage-7809-host-1 PR: merge upstream/main before merging. It now includes #7857 (test_support handlers take DispatchCtx; fixes the 'Storage e2e on MongoDB' build) and #7856 (ConversationStore::new and the threads::store free functions take &DispatchCtx first). Our threads/ops edit/transcript changes must pass the ctx through rather than resolve it ambiently. Also build with the e2e-test-support feature when checking.
Merge pull request #7859 from senamakel/storage-7809-host-1
senamakel commentedon Oct 15, 2026
Host half merged: #7859 (merge 072bb5f). It pins vendor/tinyagents at the main merge commit 3dbf2ef8cc of tinyagents#463 (release v2.8.0, cut by hand because the Release workflow computes v2.2.0 from the stale 2.1.4 workspace version and fails).
Still pending, so this issue and tinyagents#453 stay open:
senamakel commentedon Oct 15, 2026
Gate for the final #7809 host PR (agreed with the #7801 work): before merging, run
cargo check -p openhuman --features e2e-test-support --all-targetsand build the MongoDB storage e2e targets with--no-run(the openhuman-cli storage e2e tests used by the 'Storage e2e on MongoDB' job). The #7801 leaf fu-7801e-6 touches threads ops (crud/purge), running_subagents and session_host/runtime_session, so whoever lands second merges upstream/main and adapts.senamakel commentedon Oct 15, 2026
tinyagents#466: removes the path-based transcript API (FileTranscriptLocator/History and workspace-path entry points), fixes the two #463 follow-ups with regression tests, and fixes the Release workflow's version source (tinyhumansai/tinyagents#466)
senamakel commentedon Oct 15, 2026
tinyagents#466 merged (8f1bc5f24d): path-based transcript API removed, #463 findings fixed, release version script. Re-pin PR in openhuman still to follow. tinyhumansai/tinyagents#466
senamakel commentedon Oct 15, 2026
openhuman#7871: re-pins tinyagents past the path-transcript-API removal, records a thread's transcript key so a chat_agent_id change keeps earlier turns readable/editable, and isolates the agent::session_store parallel races (#7871)
Merge pull request #7871 from senamakel/storage-7809-host-1
senamakel commentedon Oct 15, 2026
Progress (stage close-out). PRs in this stage: tinyagents#466 (merged, v2.9.0, 8f1bc5f24d): removes the path-based transcript API, fixes the #463 follow-ups, fixes the Release version source. openhuman#7871 (merged): re-pin, host fixture ports, per-thread transcript key (a chat_agent_id flip keeps earlier turns), agent::session_store race fixes. Earlier: #7821, tinyagents#461, #463, #446, #456. tinyagents#453 is closed.
REMAINING (moved out of this stage): ToolResultArtifactStore onto driver ports, tracked in #7872. Not run locally: 80% changed-line coverage gate (CI measures it). The Release workflow still hits GH013 on the push step; v2.9.0 was created by hand.