Repository navigation
Bump tinyagents: served_by, response_id and first_token_ms on model observations - #8420
Conversation
Update the vendored tinyagents submodule to the latest upstream commit. Auto-committed-on: macbook Co-authored-by: Medulla <medulla@tinyhumans.ai>
Update the vendored tinyagents submodule to the latest upstream commit. Auto-committed-on: macbook Co-authored-by: Medulla <medulla@tinyhumans.ai>
Co-authored-by: Medulla <medulla@tinyhumans.ai>
Updated the pinned commits for all twenty vendored tiny submodules to their current upstream revisions. Auto-committed-on: macbook Co-authored-by: Medulla <medulla@tinyhumans.ai>
The merge picked up the worktree's stale submodule checkouts. Put every gitlink back on what origin/main records. Co-authored-by: Medulla <medulla@tinyhumans.ai>
Points at tinyhumansai/tinyagents#582 (not yet merged): model observations gain the serving provider, the provider's response id and the time to first token. Repin to the merged/released commit before this is marked ready. Co-authored-by: Medulla <medulla@tinyhumans.ai>
Co-authored-by: Medulla <medulla@tinyhumans.ai>
The lockfile now pins the tinyagents family of crates at 3.0.0 instead of 3.1.2, keeping the app's resolved dependency versions in sync with the workspace. Auto-committed-on: macbook Co-authored-by: Medulla <medulla@tinyhumans.ai>
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Update tinyagents pin vendor/tinyagents |
The submodule pointer changes to commit 89df634d38fba07fcfc50ad2a9c556b7896e84da. |
Priority:
Estimated code review effort: 1 (Trivial) | ~4 minutes
Change: Other
Possibly related issues
- tinyhumansai/chottu-core#230 — The updated tinyagents commit adds model-call fields and reasoning-token usage that address part of the telemetry described in this issue.
- Pick up tinyagents model-call facts: served_by, response_id, first_token_ms #8419 — This pin update matches the issue’s request for a tinyagents commit containing the model-observation and Langfuse changes.
Possibly related PRs
- tinyhumansai/openhuman#8412 — This PR updates the same submodule pin from the commit selected in PR
#8412to the newer tinyagents commit.
Suggested reviewers: m3ga-mind
Merge Risk: 
cec03
-
The requested dependency pin change introduces no identified merge-blocking risk.
Pre-merge checks: 
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled. Title check The title names the main change: bumping the vendored tinyagents pin so model observations expose served_by, response_id and first_token_ms. The objectives confirm the pin update to the merged commit … Linked Issues check Direct issue #8419requires the tinyagents pin to include tinyagents#582. The gitlink changes from c1d350544 to merged commit 89df634d, which contains that change. The reported behavior adds `served_b…Out of Scope Changes check The reviewed change is limited to the tinyagents gitlink and lockfile synchronization described for issue #8419. The lockfile updates support the pin and restore compatible dependency resolutions. No …Docstring Coverage No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… Full details: Title check
-
Explanation
The title names the main change: bumping the vendored tinyagents pin so model observations expose served_by, response_id and first_token_ms. The objectives confirm the pin update to the merged commit with tinyagents#582 is the core change, and the title's specific fields match the linked issue
#8419. The title is accurate and specific, though it does not mention the lockfile changes or the reasoning-token export.
Full details: Linked Issues check
-
Explanation
Direct issue
#8419requires the tinyagents pin to include tinyagents#582. The gitlink changes from c1d350544 to merged commit 89df634d, which contains that change. The reported behavior addsserved_by,response_id, andfirst_token_msto model observations and exports reasoning tokens asoutput_reasoning_tokens. OpenHuman host code uses variant matches with.., so no host-code change is required. The reported OpenHuman checks pass, and the listed test failures reproduce with the base pin.
Full details: Out of Scope Changes check
-
Explanation
The reviewed change is limited to the tinyagents gitlink and lockfile synchronization described for issue
#8419. The lockfile updates support the pin and restore compatible dependency resolutions. No unrelated product code or unrelated behavior change is identified.
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
Finishing Touches: 💡 1
-
🛠️ Fix failing CI checks: 💡
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
A rabbit checks the pinned commit with care,
Tinyagents moves to a newer share.
Three fields travel in the code’s new stream,
Response facts join the token-time gleam.
The carrot waits; the pointer’s set,
One small hop completes the set.
Comment @coderabbitai help to get the list of available commands.
Co-authored-by: Medulla <medulla@tinyhumans.ai>
Update the vendored tinyeventqueues submodule to the latest upstream commit. Auto-committed-on: macbook Co-authored-by: Medulla <medulla@tinyhumans.ai>
Downgrade the tinyeventqueues and tinyeventqueues-bus lockfile entries from 0.2.3 to 0.2.2 and advance the vendored tinyagents submodule to a newer commit. Auto-committed-on: macbook Co-authored-by: Medulla <medulla@tinyhumans.ai>
[pin-rewind]: vendor/tinyagents moves from c1d350544 (tag v3.1.2) to 89df634d3, the merge of tinyagents#582 and the tip of tinyagents main. The tag commit is a release-only commit off main (it only bumps crate versions 3.0.0 -> 3.1.2), so the move is sideways and loses nothing but those version numbers; every code change in v3.1.2 is on main and in 89df634. No tinyagents release contains #582 yet. The lockfiles follow the crates back to 3.0.0 until a release is pinned. Also restores vendor/tinyeventqueues to main's v0.2.3 pin, which an earlier checkpoint on this branch had rewound by mistake. Co-authored-by: Medulla <medulla@tinyhumans.ai>
The lockfile now pins windows-core 0.57.0 instead of 0.58.0 for the affected dependency, keeping the resolved version consistent with the rest of the tree. Auto-committed-on: macbook Co-authored-by: Medulla <medulla@tinyhumans.ai>
The regeneration had re-picked windows-core/windows-sys for iana-time-zone and winapi-util; main's choices still satisfy `cargo metadata --locked`, so the lockfile diff is only tinyagents. Co-authored-by: Medulla <medulla@tinyhumans.ai>
Tiny Sweeper reviewReviewed head: Verdict: Ready for maintainer review · Priority: none Diagnostics and supporting detailsFull summaryPull request purpose (title): Bump tinyagents: served_by, response_id and first_token_ms on model observations. Tiny Sweeper reviewed this change across 5 lane(s) and found 0 active actionable finding(s). Detailed lane evidence and any incomplete work are listed below. Review snapshot
Completeness: Complete FeaturesNone identified with supported citations. TestsNo supported feature-to-test mapping was produced. Test execution is not inferred. FindingsNo active actionable findings. Before mergeNone. Agent review detailscritique
security
tests
commits
description
Evidence and run details
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![tinysweeper[bot]](https://avatars.githubusercontent.com/in/4526585?s=60&v=4){kind=small}
tinysweeper
Bot
added
the
priority: p3
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Summary
vendor/tinyagentsto89df634d38fba07fcfc50ad2a9c556b7896e84da. That is the merge of Report serving provider, response id and time to first token on model observations tinyagents#582 and the tip of tinyagentsmain, so the pin points only at a merged commit.openhuman_embed::observegainserved_by,response_idandfirst_token_ms, and OTLP export reports reasoning tokens as Langfuse'soutput_reasoning_tokensusage bucket.Cargo.lock,crates/openhuman-app/Cargo.lock): the only change is the tinyagents crates' version, from3.1.2to3.0.0(11 packages each). See "Pin and lockfile" below.Problem
completionStartTimeis null).Solution
TurnObservation::Model/ModelDispatchwith..everywhere, so no host code changes are needed. No[patch]tables changed.Pin and lockfile:
[pin-rewind], a sideways movemainpins tinyagents atc1d350544, tagv3.1.2. That commit is a release-only commit off tinyagentsmain: it changes nothing but the crate versions, 3.0.0 to 3.1.2. [Feature] Gate service lifecycle on user login/logout — stop orphan processes #582 merged after it, and no tinyagents release contains [Feature] Gate service lifecycle on user login/logout — stop orphan processes #582 yet. So the move fromc1d350544to89df634is sideways.scripts/ci/check-submodule-monotonic.mjsreports it as losing 1 base-side commit: that release commit.v3.1.2is on tinyagentsmainand so in89df634.[pin-rewind]in the pin commit's message, as chore(vendor): advance tinystoragedrivers to v2.1.1 [pin-rewind] #8390 andecfb044ce2did for the same tag-versus-main shape.3.0.0, both lockfiles go from3.1.2to3.0.0for the tinyagents crates. That is unavoidable while pinningmain. OpenHuman declares no version requirement on tinyagents (path dependencies only), so nothing else moves.windows-core/windows-sysforiana-time-zoneandwinapi-util. Those entries are put back tomain's, whichcargo metadata --lockedaccepts.Submission Checklist
Closes #NNNin## Related.Impact
TurnObservation::Model/ModelDispatchorPhysicalDispatchOutcome; nothing in this repository does.outputintooutput_reasoning_tokens.totalis unchanged.Validation (run locally on this branch, merged with
main64599d76c7)node scripts/ci/check-submodule-monotonic.mjs origin/main HEAD: OK. The sideways move is declared with[pin-rewind].scripts/ci/fetch-submodule-tags.shthennode scripts/ci/check-module-pins.mjs: OK, 13 records, all pins accounted for.rust:layoutscripts, all pass:check-openhuman-rust-layoutcheck-crate-chaincheck-core-hosted-freecheck-storage-bypasscheck-storage-ownercheck-storage-schemacheck-test-ownershipcargo check --locked --workspace --all-targets: pass.cargo check --locked --manifest-path crates/openhuman-app/Cargo.toml: pass.rust:clippy: pass:cargo clippy --locked -p openhuman -p openhuman-cli -p openhuman-tinyhumans -- -D warningscargo clippy --locked --manifest-path crates/openhuman-app/Cargo.toml -- -D warningscargo metadata --lockedon both manifests: pass.cargo test -p openhuman-embed, the observer host tests:turn_observers: pass.--lib complete: 33 passed.observed_turns::model_and_tool_observations_capture_payloads_only_with_consentanddispatch_admission_turn::repair_refusal_is_typed_and_does_not_reach_fallback_or_emit_a_phantom_dispatchfail. They fail identically withmain's tinyagents pin (c1d350544), so this bump does not cause them.Related
89df634)openhuman-embedtest failures above need their own look.AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
langfuse-model-call-factsSummary by CodeRabbit