Skip to content

chore: pin merged TinyBus stream fix - #22

Merged
senamakel merged 1 commit into
tinyhumansai:mainfrom
senamakel:tinybus-refresh
Sep 24, 2026
Merged

senamakel merged 1 commit into
tinyhumansai:mainfrom
senamakel:tinybus-refresh

Conversation

@senamakel

Copy link
Copy Markdown
Member

Summary

Advance the TinyBus gitlink to the merged stream idle deadline fix (tinyhumansai/tinybus#23). The earlier TinyBus refresh has already landed; this follow-up ensures the next native module release uses the corrected revision.

Related issue

None.

API or behavior changes

Streamed replies may continue past the reply-handle timeout while a stalled body fails on the stream idle deadline.

Validation

Tests

No test code changed in this pin-only PR.

Documentation

No documentation change needed.

Checklist

  • The change is focused on one logical change
  • No new #[allow(...)], #[ignore], or relaxed lints
  • No secrets, tokens, or .env contents in the diff or the description

@coderabbitai

coderabbitai Bot commented Sep 24, 2026

Copy link
Copy Markdown

Warning

Review limit reached

  • Run on-demand review

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

Or wait 40 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: 1c27979e-c578-4e10-9c03-6e9ccb9771e2

📥 Commits

Reviewing files that changed from the base of the PR and between c389e62 and d7c0826.

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

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

@senamakel
senamakel merged commit 7d0d9bb into tinyhumansai:main Sep 24, 2026
5 checks passed
@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: d7c0826ddaf7
Updated: 1790269692 (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 · 14 embedding
Head State Pass summary
d7c0826ddaf7 ready for maintainer review 0 active finding(s), 0 resolved finding(s) (at 1790269692)

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.0000 · 0 in / 0 out · 14 embedded · ladder/vectors

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