Skip to content

Update TinyBus dependency to 0.1.6 - #33

Merged
senamakel merged 3 commits into
tinyhumansai:mainfrom
senamakel:epic-8088-final-rollup
Oct 23, 2026
Merged

senamakel merged 3 commits into
tinyhumansai:mainfrom
senamakel:epic-8088-final-rollup

Conversation

@senamakel

@senamakel senamakel commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Update TinySearch to the current TinyBus release (0.1.6) and refresh the lockfile.

Validation:

  • cargo check --manifest-path Cargo.toml --workspace
  • cargo clippy --manifest-path Cargo.toml --workspace --all-targets -- -D warnings
  • cargo test --manifest-path Cargo.toml --workspace (all tests pass; reran the module E2E suite after one transient mock expectation failure)
  • cargo fmt --manifest-path Cargo.toml --all -- --check

Part of tinyhumansai/openhuman#8088.

Summary by CodeRabbit

  • Chores
    • Updated a bundled component to a newer revision.

senamakel and others added 3 commits October 23, 2026 06:38
Update the vendored tinybus submodule to the latest upstream commit.

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

tinysweeper Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Tiny Sweeper review

Reviewed head: f9eeeb8ee84b95896153a45fe9f96733f503715c

Verdict: Ready for maintainer review · Priority: none
Coverage: File coverage is unavailable.
Pull request purpose (title): Update TinyBus depen…

Diagnostics and supporting details

Full summary

Pull request purpose (title): Update TinyBus dependency to 0.1.6. 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

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.

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: Success
  • Scope reviewed: all assigned evidence
  • Lane summary: No sensitive content in committed files. _The code index is behind this pull request (indexed at `0ca63b68db83`), so retrieved context may be out of date._

description

  • Conclusion: Neutral
  • Scope reviewed: all assigned evidence
  • Lane summary: No added or modified lines to review.
Evidence and run details
  • Models: ladder/vectors-oai3
  • Spend: $0.000001
  • Tokens: 0 input · 0 output · 0 cached · 11 embedding
  • Updated: 2026-10-23T08:41:21Z
Head State Pass summary
f9eeeb8ee84b ready for maintainer review 0 active finding(s), 0 resolved finding(s) (at 2026-10-23T08:41:21Z)

tinysweeper 0.1.2

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 23, 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-10-23T08:44:22.727286Z f9eeeb8 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.

@coderabbitai

coderabbitai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f8f71f00-5154-4cac-8894-8557368d5a2b
📥 Commits

Reviewing files that changed from the base of the PR and between ec863c4 and f9eeeb8.

⛔ Files ignored due to path filters: 1
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing: 1
  • vendor/tinybus

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

📝 Walkthrough

The vendor/tinybus submodule pointer now references commit 5d5af90ddc297b0bcc284d8b5ea35c04842d8269 instead of eb5cf727071a6eb12f9efa3db8ec59e54039f336.

Changes: Tinybus submodule update

Layer / File(s) Summary
Update submodule reference
vendor/tinybus
The submodule pointer changes to commit 5d5af90ddc297b0bcc284d8b5ea35c04842d8269.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to f9eee

No actionable merge-blocking issue is established. The pinned TinyBus revision could not be independently inspected, so merge readiness still depends on the reported workspace checks.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title "Update TinyBus dependency to 0.1.6" matches the main change. The raw summary shows the vendor/tinybus submodule reference moved from eb5cf72 to 5d5af90 (one line changed). The PR objectives…
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.
Full details: Title check

Explanation

The title "Update TinyBus dependency to 0.1.6" matches the main change. The raw summary shows the vendor/tinybus submodule reference moved from eb5cf72 to 5d5af90 (one line changed). The PR objectives say the PR updates TinySearch to TinyBus 0.1.6 and refreshes the lockfile. The title is specific, concise, and describes the primary change from the developer's view. The excluded Cargo.lock change is consistent with the title.

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 vendor trail
One commit shifts beneath the moon
Tinybus marks a newer path
Soft paws follow the pointer’s tune
Then hops along, content and soon

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

@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.

Diagnostics

Total spend: $0.0000

@senamakel
senamakel merged commit 20537d9 into tinyhumansai:main Oct 23, 2026
19 of 20 checks passed
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