feat(polkavm): run App Manifest v2 runtimes in Dotli - #185
Conversation
⚡ Performance Report
|
Bundle Size ReportChunks over 500 KB:
All files
Commit: 85eae89 |
|
@justraman the PVM change is green across formatting, build, unit, functional, and the canonical seven-app Dotli matrix. The broad host-playground E2E remains red on the same account, allowance, and navigation failures as current main (the latest main Tests runs also fail); it does not exercise this PVM path. Could you review the runtime adapter so this can follow host-rust-core#540? |
|
Revended reviewed host-rust-core commit 56c67a306fc276129829f94db85a622061e17882. Re-ran the canonical matrix on this exact vendor: Doom, Quake, Duke3D, NES, egui kitchen sink, GPU Cube, and Scene Lab all passed through Dotli's compiler backend. |
|
CI diagnosis: the red E2E job is not specific to this branch.
No Dotli source change is justified for this shared signer/product-account path; the check needs the external E2E dependency restored or the repository's existing external-outage policy applied. |
|
Repair requires provisioning the |
|
Cold first-load Bitswap failure reproduced and fixed in eac7311. Reproduction against a fresh Chromium profile on The host now gives Verification:
|
Reverts bd767c5 and 52b4b9f. Every App published by polkavm-app-kit declares runtime abiVersion 1 — the only version polkavm-app-abi-v1.md defines, and the value prepare-app.mjs enforces — so the strict-2 gate refused all 15 of them on chain. The runtime pin returns to @parity/pvm-browser-runtime 0.1.15 (host-runtime ffd72864), which carries the safe-area and virtual-keyboard inset ABI that 0.6.28 (cc9940b6) predates.
|
Final code review is complete and all findings are fixed in 8595bde. CI, westendli.dev deployment, and the six-product post-deploy smoke are green. The admin squash merge is currently blocked by the repository rule requiring approval from someone other than the last pusher; one requested reviewer approval will unblock it. |
|
Consolidated the Zed follow-ups here instead of maintaining parallel PRs #216 and #217. Latest branch commit: 249395a. Included:
Verification on the consolidated branch: host/sandbox/protocol build passed with the CI network configuration; all 41 focused sandbox PolkaVM runtime tests passed; browser runtime asset identity/checksums verified against @useragent-kit/polkavm-runtime 0.1.1 / runtime source 08cb7401087f8b715dc4f1be0007753caa4bd7c2. Prior direct browser smoke verified framebuffer resizing and cached reload. Full functional CI remains required; local Playwright browser launches previously terminated before the test assertions. Removed deploy: westendli.dev before pushing to prevent an implicit deployment. This is source consolidation only; the live build was not replaced. No native host work or new ABI was introduced. |
|
The user explicitly selected draft computer PR #215 as the branch to deploy to westendli.dev, then approved restoring it after this PR's deployment superseded it. I removed this PR's Latest #185 (36a2129) is merged into #215 as c256ccc; restoration run: https://github.com/paritytech/dotli-community/actions/runs/34302321041. Please keep westendli.dev deployment ownership on #215 for this computer task. Do not re-add the deployment label here without an explicit user decision to switch the deployed branch; the workflow concurrency key is per PR, so two labeled PRs can overwrite the same environment. |
# Conflicts: # packages/resolver/src/resolve.ts
|
❌ E2E Product suite failed on Failed tests:
Logs: https://github.com/paritytech/dotli-community/actions/runs/34384639876 |
Summary
host-frame-request/host-frame-responsetransportpvmpaths, scripts, environment variables, metrics, tests, and legacy CID cache dataScope boundary
This PR implements the runtime-aware App work tracked by paritytech/host-rust-core#507, paritytech/host-rust-core#540, and paritytech/host-rust-core#550. The experimental
polkadot-host/0.1/*computer interfaces, filesystem, TTY, process spawning, workspace orchestration, and TCP relay remain in a separate draft stacked PR pending their host-interface RFC.Runtime provenance
@useragent-kit/polkavm-runtime@0.1.06eb24c5fd4451ca339c82c73a308ac8af87b8e4b1scripts/polkavm-runtime.lock.jsonDependencies
Verification
bun run check:polkavm-runtime: verified package 0.1.0 and upstream revision6eb24c5f…bunx --bun turbo run typecheck: 13/13 workspace tasks passedbunx --bun turbo run lint: 13/13 workspace tasks passedbunx --bun turbo run test: 12/12 workspace test tasks passed, including production buildsThe deploy workflow runs the six-product post-deploy smoke for
westendli.dev,paseoli.dev, andpaseo.li.