Skip to content

Align tinytools dependencies with 0.4.0 vendor release - #188

Closed
senamakel wants to merge 2 commits into
mainfrom
code-tool-dialect
Closed

senamakel wants to merge 2 commits into
mainfrom
code-tool-dialect

Conversation

@senamakel

Copy link
Copy Markdown
Member

Summary

  • align every local tinytools and tinytools-agent dependency with the vendored 0.4.0 release merged in Add Python / TypeScript code-style tool dispatchers #184
  • regenerate Cargo.lock while retaining the 0.3.0 git dependencies required by tinyinference
  • restore workspace compilation after the vendor gitlink update

Verification

  • cargo fmt --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo build --workspace --all-targets
  • cargo test --workspace --quiet
  • cargo test -p tinyagents-integration-tests --test e2e_tool_dialects

Follow-up to #184, which merged before this dependency-alignment commit reached the PR head.

senamakel and others added 2 commits September 22, 2026 05:25
Update the tinytools and tinytools-agent dependencies to version 0.4.0 across the workspace, while retaining the 0.3.0 versions from the git source for compatibility. This change aligns the crates with the latest release of the tinytools ecosystem.

Auto-committed-on: dragonfly
Co-authored-by: Medulla <medulla@tinyhumans.ai>
@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown

Warning

Review limit reached

  • Run on-demand review

This review includes 6 billable files and costs up to $1.50.

Or wait 2 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: 4bf34795-2748-440a-8404-44c128b2c62c

📥 Commits

Reviewing files that changed from the base of the PR and between 492bac9 and 411542c.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (6)
  • crates/tinyagents-graph/Cargo.toml
  • crates/tinyagents-harness/Cargo.toml
  • crates/tinyagents-integration-tests/Cargo.toml
  • crates/tinyagents-orchestration/Cargo.toml
  • crates/tinyagents-registry/Cargo.toml
  • crates/tinyagents-runtime/Cargo.toml

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

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 22, 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-22T02:35:06.538804Z 411542c PR opened
ℹ️ 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 closed this Sep 22, 2026
@tinysweeper

tinysweeper Bot commented Sep 22, 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: 411542cc2b75
Updated: 1790044894 (Unix time)

Review snapshot

Change surface Files Review signal Count
Production 0 Active findings 0
Tests 1 Noted findings 0
Documentation 0 Resolved findings 0
Configuration 5 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: Success
  • Scope reviewed: all assigned evidence
  • Lane summary: Reviewed 6 files; 0 findings. _The code index is behind this pull request (indexed at `cf07569a08c9`), so retrieved context may be out of date._ _3 memory call(s) failed (model: cortex: v1/answer: timed out after 20s), so this review saw part of what the engine holds._

security

  • Conclusion: Success
  • Scope reviewed: all assigned evidence
  • Lane summary: Reviewed 6 files; 0 findings. _The code index is behind this pull request (indexed at `cf07569a08c9`), so retrieved context may be out of date._ _3 memory call(s) failed (model: cortex: v1/answer: timed out after 20s), so this review saw part of what the engine holds._

tests

  • Conclusion: Neutral
  • Scope reviewed: all assigned evidence
  • Lane summary: No behavioural change: nothing outside documentation, configuration and tests.

commits

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

description

  • Conclusion: Success
  • Scope reviewed: all assigned evidence
  • Lane summary: This is a small Cargo.toml-only change that updates the age_pool dependency from the simple git URL form to the qualified git+https form. I reviewed the diff and it only touches those dependency lines; I could not verify the surrounding build or the referenced follow-up context from here, but nothing in the change itself introduces a defect or violates repository conventions. _The code index is behind this pull request (indexed at `cf07569a08c9`), so retrieved context may be out of date._ _3 memory call(s) failed (model: cortex: v1/answer: timed out after 20s), so this review saw part of what the engine holds._

e2e

  • Conclusion: Neutral
  • Scope reviewed: all assigned evidence
  • Lane summary: No behavioural change: nothing outside documentation, configuration and tests.
Evidence and run details
  • Models: ladder/vectors, gpt-5.6-luna, deepseek/deepseek-v4-flash
  • Spend: $0.018908
  • Tokens: 365895 input · 10140 output · 50794 cached · 200 embedding
Head State Pass summary
411542cc2b75 ready for maintainer review 0 active finding(s), 0 resolved finding(s) (at 1790044894)

tinysweeper 0.1.0

@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.0189 · 365,895 in / 10,140 out · 50,794 cached (14%) · ladder/vectors, gpt-5.6-luna, deepseek/deepseek-v4-flash · 200 embedded
critique:    $0.0097 · 188,277 in / 4,838 out  · 24,770 cached (13%) · gpt-5.6-luna, deepseek/deepseek-v4-flash
security:    $0.0085 · 166,839 in / 3,900 out  · 19,647 cached (12%) · gpt-5.6-luna
description: $0.0002 · 7,674 in   / 101 out    · 3,706 cached (48%)  · deepseek/deepseek-v4-flash

@tinysweeper tinysweeper Bot added the priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect. label Sep 22, 2026
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