chore(deps): bump vendored tinytools for the DSML tool_call tag fix - #196
Conversation
Tiny Sweeper reviewTiny 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 Review snapshot
Completeness: Complete What changedThe review could not produce a supported behavioral summary; inspect the cited changed surface and lane details below. 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
e2e
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. |
|
Warning Review limit reached
This review includes 1 billable file and costs up to $0.25. Or wait 12 minutes for your next included review. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Comment |
Picks up tinyhumansai/tinytools#21: the tagged-JSON grammar now accepts DeepSeek's DSML marker and fullwidth pipes on the `tool_call` tag family, the way `invoke_xml` already did for `<invoke>`. Without it a `<|DSML|tool_call>` block parsed as narrative and the call was dropped silently — seen live from `deepseek-v4-flash` on the code dialect. Co-authored-by: Medulla <medulla@tinyhumans.ai>
Auto-committed-on: dragonfly Co-authored-by: Medulla <medulla@tinyhumans.ai>
The checkpoint hook swept a local `wiki` checkout into this branch. The pin is unrelated to the tinytools bump, so put it back where main has it. Co-authored-by: Medulla <medulla@tinyhumans.ai>
Updated the wiki submodule to point to a newer commit, incorporating the latest changes from the upstream repository. Auto-committed-on: dragonfly Co-authored-by: Medulla <medulla@tinyhumans.ai>
Updated the wiki submodule to point to a newer commit, incorporating the latest changes from the upstream repository. Auto-committed-on: dragonfly Co-authored-by: Medulla <medulla@tinyhumans.ai>
914f820 to
28760a4
Compare
Same base main already pins (3c9ba00, the session-todo-list work this tree depends on), plus the tinytools bump from tinyhumansai/tinyagents#196 → tinyhumansai/tinytools#21: a `<|DSML|tool_call>` block parsed as narrative and the call was dropped silently. `deepseek-v4-flash` emits that form on the code dialect, which is the path this branch's integration work now leans on. Repoint to the merge commit once #196 lands. Co-authored-by: Medulla <medulla@tinyhumans.ai>
Gitlink bump for tinyhumansai/tinytools#21.
invoke_xmlalready accepted DeepSeek's DSML prefix on<invoke>; the tagged-JSON grammar did not accept it on thetool_calltag family, soparsed as narrative and the call was dropped with no error. Seen live from
deepseek-v4-flashdriving the code dialect in OpenHuman: the turn ended with the model's lead-in and the tool never ran.Merge order: tinytools#21 first, then this. The gitlink points at that PR's branch head; I will repoint it to the merge commit on
mainbefore this merges.Note for the consumer side:
openhumanmain currently pins a tinyagents commit (3c9ba00) that is not onmainhere, so the bump openhuman actually consumes lives ondsml-tool-call-tag-v2(that commit + this bump). This PR is themain-based version of the same one-line change; whichever lands first, the other rebases onto it.cargo test -p tinytools-agenton the pinned commit: 310 passed, 0 failed.Co-authored-by: Medulla medulla@tinyhumans.ai