refactor(polkavm): isolate optional host composition - #540
Conversation
7f73849 to
f6457f1
Compare
|
@pgherveou @eugypalu all required checks are green, including Rust/Clippy, Android, iOS, browser artifacts, and license policy. Could one of you provide the required host-sdk-team approval so the merge queue can admit this dependency? |
pgherveou
left a comment
There was a problem hiding this comment.
can we remove the generated files in the new js packages. Can't these be built from the src files ?
for the new bindings we can also added then to .gitattributes with linguist-generated=true, so that at least they show up as generated artifacts in the code review
|
Addressed in b0e178e: removed the committed browser |
|
Cleanup CI is fully green; ready for rereview and merge queue admission. |
|
DQ: why do all these things need to live in host-rust-core, they don't seem to have dependency on the rust-core stuff, and could be shipped independently ? |
b0e178e to
e0f05c4
Compare
|
They share the Rust core directly rather than only co-locating artifacts:
They could be split, but that would duplicate the Rust source/pins and create independent browser, XCFramework, and AAR versioning with an ABI-drift boundary. Keeping the runtime here makes the host-neutral Rust implementation the single source for all three targets; Epoca/Dotli/Brevity/Desktop remain independent consumers. |
|
Does that shared Rust/UniFFI dependency path resolve the repository-placement concern, or do you want the runtime split into a separate repository before merge? |
|
Additional review completed. Fixed three concrete boundary issues:
Added behavioral browser tests for malformed launch inputs and cross-profile framebuffer submission. Full workspace result: 1,205 tests passed, Clippy Vendored commit |
|
PolkaVM dependency update:
|
23f50c6 to
d0e16fc
Compare
|
Rebased the full PVM series onto current Post-rebase verification:
The PR is now up to date with |
|
Added in 2c08d25:
Consumer cutovers:
Verification remains green: |
|
The repository-placement concern is now resolved by the extraction:
Generated browser output remains source-built and verified in the standalone repository; committed Swift/FFI bindings are marked generated. The full bridge matrix is green. |
8daab08 to
5e04221
Compare
ed97e55 to
0a2172d
Compare
|
Updated this existing native integration PR in |
# Conflicts: # CHANGELOG.md # README.md # rust/crates/truapi/src/v01/signing.rs
|
CI Status: 14 required jobs green, 13 passed and 1 skipped by path filter. All job results
Commit |
Summary
truapi-pvm-hostwith the optionaltruapi-polkavm-host0.3.0 composition cratepolkavm-host-runtime0.3.0 atc7fea7467dff302b85f34d6327d87d74e77b4189, including the mediated input runtime APItruapi-servercompletely independent of PolkaVMpolkavm-host-runtimeowns those distribution artifactsarmeabi-v7aBoundary
truapi-polkavm-hostis an rlib-only composition dependency. It explicitly exposes the namespacedpolkavm_host_runtimeandtruapi_serverAPIs plus pinned version/revision constants. Downstream distribution crates decide which native ABI surfaces to export.Verification
TRUAPI_SKIP_PACKAGE_BUILD=1 ./scripts/codegen.shcargo test -p truapi-polkavm-host --locked: 1 passed