Skip to content

Release 0.4.1 CI repair - #18

Merged
senamakel merged 1 commit into
mainfrom
ci-fix
Sep 22, 2026
Merged

senamakel merged 1 commit into
mainfrom
ci-fix

Conversation

@senamakel

Copy link
Copy Markdown
Member

Summary

  • align tinytools-jev's internal tinytools dependency with the 0.4 release
  • bump workspace packages to 0.4.1 for the CI repair patch release

Public API / behavior changes

None.

Validation

  • cargo fmt --all -- --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo build --all-targets --all-features
  • cargo test --all-features

Related issue

Fixes the CI failure on main after PR #16 and PR #17 were merged.

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 3 billable files and costs up to $0.75.

Or wait 6 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: bc7b9d37-5e6b-4c2f-bc01-bcd8433b7001

📥 Commits

Reviewing files that changed from the base of the PR and between 970bfae and 7a4e994.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (3)
  • Cargo.toml
  • crates/tinytools-agent/Cargo.toml
  • crates/tinytools-jev/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:30:37.346738Z 7a4e994 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 merged commit 3981abb into main Sep 22, 2026
9 checks passed

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7a4e994709

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[dependencies]
async-trait = { workspace = true }
tinytools = { path = "../tinytools", version = "0.3.0" }
tinytools = { path = "../tinytools", version = "0.4.1" }

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Drop the internal dependency version pins

When the workspace next advances to 0.5.0, Cargo interprets this as ^0.4.1 and rejects the local tinytools v0.5.0, recreating the same CI resolution failure this commit repairs. deny.toml:35-44 explicitly documents that unpublished workspace crates should use path-only internal dependencies for this reason; remove version here and from the matching tinytools-agent entry instead of synchronizing both pins.

Useful? React with 👍 / 👎.

@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: 7a4e994709a6
Updated: 1790044761 (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 3 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 3 files; 0 findings. _The code index is behind this pull request (indexed at `369cd3dde215`), so retrieved context may be out of date._ _Memory was unavailable (model: cortex: v1/recall: error sending request for url (http://cortexdb:3141/v1/recall\)\), so this review ran without it._

security

  • Conclusion: Success
  • Scope reviewed: all assigned evidence
  • Lane summary: Reviewed 3 files; 0 findings. _The code index is behind this pull request (indexed at `369cd3dde215`), so retrieved context may be out of date._ _Memory was unavailable (model: cortex: v1/recall: error sending request for url (http://cortexdb:3141/v1/recall\)\), so this review ran without it._

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 patch bumps workspace version to 0.4.1 and aligns two downstream crates' tinytools dependency specifiers with that version, fixing a CI workspace version mismatch. No behavioral changes; safe to merge. _The code index is behind this pull request (indexed at `369cd3dde215`), so retrieved context may be out of date._ _Memory was unavailable (model: cortex: v1/recall: error sending request for url (http://cortexdb:3141/v1/recall\)\), so this review ran without it._

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.007058
  • Tokens: 131937 input · 3487 output · 14947 cached · 67 embedding
Head State Pass summary
7a4e994709a6 ready for maintainer review 0 active finding(s), 0 resolved finding(s) (at 1790044761)

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.0071 · 131,937 in / 3,487 out · 14,947 cached (11%) · ladder/vectors, gpt-5.6-luna, deepseek/deepseek-v4-flash · 67 embedded
critique:    $0.0041 · 77,957 in  / 1,798 out · 6,081 cached (8%)   · gpt-5.6-luna
security:    $0.0023 · 48,463 in  / 757 out   · 7,135 cached (15%)  · gpt-5.6-luna
description: $0.0003 · 3,751 in   / 77 out    · 0 cached (0%)       · deepseek/deepseek-v4-flash

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant