Skip to content

chore(vendor): bump tinytools for parse_replayed_results - #210

Merged
senamakel merged 3 commits into
mainfrom
tool-call-presentation
Sep 24, 2026
Merged

senamakel merged 3 commits into
mainfrom
tool-call-presentation

Conversation

@senamakel

@senamakel senamakel commented Sep 24, 2026

Copy link
Copy Markdown
Member

Summary

Bumps the vendor/tinytools gitlink to pick up parse_replayed_results (tinyhumansai/tinytools#23), the inverse of the text-dialect [Tool results] replay frame.

No tinyagents code changes: the runtime and session writer behave to contract. OpenHuman needs the parser to pair each replayed tool result with its call when projecting a persisted transcript (tinyhumansai/openhuman PR to follow).

Merge order

  1. feat(render): parse the [Tool results] replay frame back into entries tinytools#23
  2. this PR (re-point the gitlink at the merged tinytools main commit if the merge rewrote it)
  3. the openhuman PR

Test plan

  • cargo test -p tinytools-agent (in the bumped tinytools): 313 passed.
  • Consumed end to end by openhuman's transcript projection tests.

Summary by CodeRabbit

  • Chores
    • Updated an internal supporting component. No user-facing changes are included.

Co-authored-by: Medulla <medulla@tinyhumans.ai>
@tinysweeper

tinysweeper Bot commented Sep 24, 2026

Copy link
Copy Markdown

Tiny Sweeper review

Tiny Sweeper reviewed this change across 6 lane(s) and found 0 active actionable finding(s). Detailed lane evidence and any incomplete work are listed below.

State: Ready for maintainer review
Priority: none
Reviewed head: 79e4b914410e
Updated: 1790228909 (Unix time)

Review snapshot

Change surface Files Review signal Count
Production 0 Active findings 0
Tests 0 Noted findings 0
Documentation 0 Resolved findings 0
Configuration 0 Pending checks/questions 0

Completeness: Complete
Test assessment: No supported feature-to-test mapping was available; this does not mean tests are absent or passed.

What changed

The review could not produce a supported behavioral summary; inspect the cited changed surface and lane details below.

Features

None identified with supported citations.

Tests

No supported feature-to-test mapping was produced. Test execution is not inferred.

Findings

No active actionable findings.

Before merge

None.

Agent review details

critique

  • Conclusion: Neutral
  • Scope reviewed: all assigned evidence
  • Lane summary: No added or modified lines to review.

security

  • Conclusion: Neutral
  • Scope reviewed: all assigned evidence
  • Lane summary: No added or modified lines to review.

tests

  • Conclusion: Neutral
  • Scope reviewed: all assigned evidence
  • Lane summary: No added or modified lines to review.

commits

  • Conclusion: Neutral
  • Scope reviewed: all assigned evidence
  • Lane summary: Nothing sensitive found in what this pull request commits.

description

  • Conclusion: Neutral
  • Scope reviewed: all assigned evidence
  • Lane summary: No added or modified lines to review.

e2e

  • Conclusion: Neutral
  • Scope reviewed: all assigned evidence
  • Lane summary: No added or modified lines to review.
Evidence and run details
  • Models: ladder/vectors
  • Spend: $0.000000
  • Tokens: 0 input · 0 output · 0 cached · 25 embedding
Head State Pass summary
635511fd1058 ready for maintainer review 0 active finding(s), 0 resolved finding(s) (at 1790225356)
79e4b914410e ready for maintainer review 0 active finding(s), 0 resolved finding(s) (at 1790228233)
79e4b914410e ready for maintainer review 0 active finding(s), 0 resolved finding(s) (at 1790228909)

tinysweeper 0.1.0

@coderabbitai

coderabbitai Bot commented Sep 24, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Warning

Review limit reached

  • Run on-demand review

This review includes 1 billable file and costs up to $0.25.

Or wait 29 minutes for your next included review.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: dc29750f-a831-4fe5-9266-5e3b1fb36282

📥 Commits

Reviewing files that changed from the base of the PR and between 635511f and 79e4b91.

📒 Files selected for processing (1)
  • vendor/tinytools

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 6752f686-1e2f-420f-9809-a78be4b8daee

📥 Commits

Reviewing files that changed from the base of the PR and between f4735ff and 635511f.

📒 Files selected for processing (1)
  • vendor/tinytools

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The pinned vendor/tinytools subproject commit changed from b47ccd1a58ec1bb0de51f8a732261bda701bbab6 to db4dc5831421a5bdeac7b60981d97cd36e1e3d06.

Changes

tinytools Commit Update

Layer / File(s) Summary
Update pinned commit
vendor/tinytools
The subproject commit reference changed to db4dc5831421a5bdeac7b60981d97cd36e1e3d06.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Suggested reviewers: enamakel

Merge Risk: ⚪ Minimal · up to 63551

No specific failure is established in this change. The parser behavior could not be verified from the available source, so confirm the upstream implementation and tests before relying on the OpenHuman integration.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the vendor bump and names the specific tinytools feature included by the update. It matches the main change.
Docstring Coverage ✅ Passed 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…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
⚔️ Resolve merge conflicts 💡
  • Resolve merge conflict in branch tool-call-presentation

A rabbit checks the pinned commit,
Then hops along with care.
One tiny pointer takes a step,
To mark a newer place.
Soft paws leave no extra trace.

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

@senamakel senamakel self-assigned this Sep 24, 2026

@tinysweeper tinysweeper Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tinysweeper found nothing blocking. Approving.

$0.0000 · 0 in / 0 out · 25 embedded · ladder/vectors

@tinysweeper tinysweeper Bot added the priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect. label Sep 24, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 24, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-24T05:21:31.757112Z 79e4b91 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@senamakel
senamakel merged commit 2f9e2eb into main Sep 24, 2026
9 checks passed
@senamakel

Copy link
Copy Markdown
Member Author

Consumed by tinyhumansai/openhuman#6598.

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

Labels

priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant