Skip to content

Add linked TinyBus module entrypoint - #10

Merged
senamakel merged 6 commits into
tinyhumansai:mainfrom
senamakel:windows-linked-modules
Sep 30, 2026
Merged

senamakel merged 6 commits into
tinyhumansai:mainfrom
senamakel:windows-linked-modules

Conversation

@senamakel

@senamakel senamakel commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Summary

Adds static-link (with linked as an alias) and uses module_export_optional_static! to select the Rust-addressable module export. The normal build keeps its dynamic C ABI. The nested TinyBus checkout is pinned to commit 433d9ed from tinyhumansai/tinybus#29.

Draft readiness condition: TinyBus PR #29 must merge, then this repository must pin its merged commit and pass CI before this PR is marked ready or merged.

Related issue

None.

API or behavior changes

With static-link, Rust hosts can call tinybox::linked_module() to register this module in process. linked remains an alias for static-link.

Validation

  • cargo check -p tinybox-module --locked
  • cargo check -p tinybox-module --features static-link --locked
  • cargo fmt --all -- --check
  • git diff --check

Tests

TinyBus PR #29 covers the shared module export behavior. Full repository suites were not run locally.

Documentation

The crate-root function has rustdoc; the TinyBus dependency PR documents the host registration API.

Checklist

  • The change is focused on linked module integration
  • No new #[ignore] or relaxed lint settings
  • No secrets or credentials in the change

@coderabbitai

coderabbitai Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

  • Run on-demand review

This review includes 4 billable files and costs up to $1.00.

Or wait 50 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: 7433d224-4743-4cf7-96a8-056ffe2b7976

📥 Commits

Reviewing files that changed from the base of the PR and between e8828da and 5e9053c.

📒 Files selected for processing (4)
  • Cargo.toml
  • crates/tinybox-module/Cargo.toml
  • crates/tinybox-module/src/lib.rs
  • crates/tinybox-module/src/tinybus_module/mod.rs

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

@senamakel senamakel self-assigned this Sep 30, 2026
Co-authored-by: Medulla <medulla@tinyhumans.ai>
@senamakel
senamakel marked this pull request as ready for review September 30, 2026 08:47
@senamakel
senamakel merged commit 5a62849 into tinyhumansai:main Sep 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant