Skip to content

feat: expose linked TinyBus module entry point - #14

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

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

Conversation

@senamakel

@senamakel senamakel commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Add opt-in tinyvoice-module/static-link using TinyBus module_export_optional_static!, which selects dynamic exports by default and a Rust-addressable module in static builds.
  • Expose tinyvoice_module::linked_module() for direct host registration.
  • Pin TinyBus commit 433d9ed and use =0.1.2 registry dependency identities, with local Cargo patches for standalone builds.

Related issue

Depends on tinyhumansai/tinybus#29, which builds on merged #28. Ready condition: #29 is merged, this gitlink is reachable from upstream TinyBus, and CI passes. This PR remains a draft until then.

API or behavior changes

Additive public linked_module() under static-link. The default loadable module path and wire contract are unchanged.

Validation

  • cargo fmt --manifest-path crates/tinyvoice-module/Cargo.toml --all -- --check
  • cargo check --manifest-path crates/tinyvoice-module/Cargo.toml -p tinyvoice-module --offline
  • cargo check --manifest-path crates/tinyvoice-module/Cargo.toml -p tinyvoice-module --features static-link --offline
  • Full clippy, build, test, and coverage suites have not been rerun on this final commit.

Tests

The two feature builds verify both TinyBus macro expansions and the public root reexport. No runtime behavior changes were added in this package.

Documentation

The feature and root entry point are documented in this PR and by TinyBus #29.

Checklist

  • The change is focused on static module registration.
  • No lint levels were relaxed in this update.
  • No secrets or environment contents are included.

@coderabbitai

coderabbitai Bot commented Sep 25, 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 55 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: b82bcf8a-d670-43c7-b053-0a5ca29f88c3

📥 Commits

Reviewing files that changed from the base of the PR and between 4778ca5 and bf8e22f.

📒 Files selected for processing (3)
  • crates/tinyvoice-module/Cargo.toml
  • crates/tinyvoice-module/src/lib.rs
  • crates/tinyvoice-module/src/service/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:41
@senamakel
senamakel merged commit d8ba432 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