Skip to content

fix(modules): align native release with TinyBus 0.3.3 - #31

Merged
senamakel merged 6 commits into
tinyhumansai:epic/8056-modules-wiringfrom
senamakel:epic-8056-tinysearch-sdk032-source
Oct 19, 2026
Merged

senamakel merged 6 commits into
tinyhumansai:epic/8056-modules-wiringfrom
senamakel:epic-8056-tinysearch-sdk032-source

Conversation

@senamakel

@senamakel senamakel commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Advance TinySearch’s module source to the released TinyBus 0.3.3 SDK while preserving compatible <0.4 dependency bounds. Update the lockfile and release checks, including a parent-tree gitlink guard that also works without a recursive checkout. TinySearch remains at the prepared 0.5.2 release version.

Part of tinyhumansai/openhuman#8056.

Validation: check, build, strict Clippy, formatting, default/all-feature tests, feature matrix, per-file coverage (all >=90%), Rust 1.88 build, cargo-deny, docs, release workflow tests and actionlint passed. The actual native archive passed all three module E2E tests; its manifest library hash matches the bundled dylib. Hosted CI must pass before merge. This PR prepares source for the reviewed native release; publication verification is separate.

senamakel and others added 6 commits October 18, 2026 23:04
The release workflow test now expects the vendored tinybus revision to be 0.3.3, so the pinned SHA and the workspace dependency version constraints were bumped to match. An assertion was added to verify the vendored Cargo.toml declares version 0.3.3.

Auto-committed-on: macbook
…he__/test-e2e-version.cpython-3

Auto-committed-on: macbook
Deleted the __pycache__ .pyc files that had been committed to the
repository. These are generated artifacts that should not be tracked.

Auto-committed-on: macbook
Auto-committed-on: macbook
Co-authored-by: Medulla <medulla@tinyhumans.ai>
@chatgpt-codex-connector

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 ⚠️ Failed 2026-10-19T10:04:40.578703Z d42e2ef 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.

@tinysweeper

tinysweeper Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Tiny Sweeper review

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.

State: Incomplete
Priority: none
Reviewed head: d42e2efc1fcc
Updated: 2026-10-19T10:04:57Z

Review snapshot

Change surface Files Review signal Count
Production 0 Active findings 0
Tests 2 Noted findings 0
Documentation 0 Resolved findings 0
Configuration 1 Pending checks/questions 8

Completeness: Incomplete
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.

  • Unreviewed: tinysweeper/tests

Findings

No active actionable findings.

Could not review: Cargo.toml, scripts/test-e2e-version.py, scripts/test-release-workflow.py, tinysweeper/description, tinysweeper/tests

Before merge

  • Complete the critique review for Cargo.toml, scripts/test-e2e-version.py, scripts/test-release-workflow.py.
  • Complete the security review for Cargo.toml, scripts/test-release-workflow.py, scripts/test-e2e-version.py.
  • Complete the tests review for tinysweeper/tests.
  • Complete the description review for tinysweeper/description.
Agent review details

critique

  • Conclusion: Neutral
  • Scope reviewed: incomplete; unanswered: Cargo.toml, scripts/test-e2e-version.py, scripts/test-release-workflow.py
  • Lane summary: Reviewed 0 files; 0 findings. 3 files could not be reviewed: Cargo.toml, scripts/test-e2e-version.py, scripts/test-release-workflow.py.

security

  • Conclusion: Neutral
  • Scope reviewed: incomplete; unanswered: Cargo.toml, scripts/test-release-workflow.py, scripts/test-e2e-version.py
  • Lane summary: Reviewed 0 files; 0 findings. 3 files could not be reviewed: Cargo.toml, scripts/test-release-workflow.py, scripts/test-e2e-version.py.

tests

  • Conclusion: Neutral
  • Scope reviewed: incomplete; unanswered: tinysweeper/tests
  • Lane summary: No reviewer produced a usable response.

commits

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

description

  • Conclusion: Neutral
  • Scope reviewed: incomplete; unanswered: tinysweeper/description
  • Lane summary: No reviewer produced a usable response.
Evidence and run details
  • Models: ladder/vectors-oai3
  • Spend: $0.000029
  • Tokens: 0 input · 0 output · 0 cached · 224 embedding
Head State Pass summary
d42e2efc1fcc incomplete 0 active finding(s), 0 resolved finding(s) (at 2026-10-19T10:04:57Z)

tinysweeper 0.1.2

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@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, but could not review everything, so this is not an approval: Cargo.toml, scripts/test-e2e-version.py, scripts/test-release-workflow.py, tinysweeper/description, tinysweeper/tests.

$0.0000 · 0 in / 0 out · 224 embedded · ladder/vectors-oai3

@senamakel
senamakel merged commit 059d7c2 into tinyhumansai:epic/8056-modules-wiring Oct 19, 2026
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