Repository navigation
refactor(host): migrate pd-edge to ABI25 descriptors - #2
Merged
fffonion merged 9 commits intoSep 18, 2026
Merged
Conversation
…eturns - Pin pd-vm and pd-host-function to the frozen RustScript descriptor core b1d6cffede77f49410bf63525f30b9a46b02dc01 (git deps) and refresh Cargo.lock. - Edge macro: one return-type normalization in the async transform. The authored body is awaited into an explicitly typed VmResult<CallOutcome>, so unannotated Ok(CallOutcome::...) bodies infer without per-call-site annotations (335 E0282/E0283 -> 0). - Frozen core returns VmResult from set_async_bridge/clear_async_bridge and reports reset failures/close-pending through reset_for_reuse + poll_reset_for_reuse: handle the results and only pool a VM that reached a clean quiescent reset.
Join ABI declarations with generated adapters into HostFunctionDescriptor, install them transactionally, and keep linkme as discovery metadata only.
Keep abi.json as the full-surface snapshot and fail the crate test when it drifts from a --all-features generator run.
Cover the exact SHA lock, every checked-in .rss program, MQTT sample inference fixes, and the documented raw/dynamic descriptor boundaries.
Record the named MqttEvent catalog overlay as an explicit ABI-version and fingerprint change. Coarse mqtt::connection::read_event wire type stays map.
…tadata Install published ABI adapters through the frozen-core exact catalog path and authorize restricted registries in the same transaction. Overlay closed mqtt::connection::read_event as Named MqttEvent, keep remaining header and query maps on an explicit allowlist, and declare per-function raw-handle modes for every handle family including apply_exchange*.
Route production and test compile helpers through the edge catalog so exact import identity is preserved. Repair the MQTT broker sample and sweep all 26 checked-in RSS fixtures with no accepted-error escape.
Frozen pd-vm's optional edge-abi depends on crates.io pd-edge-abi 0.1.1 (ABI24). HTTP now compiles against the in-tree ABI25 catalog only. Architecture tests cover the feature graph and catalog contracts.
Keep master's ABI25/host-descriptor work and drop vm/edge-abi so HTTP stays on the in-tree catalog. Resolve Cargo.lock and dependency pin tests onto the candidate: no registry ABI24 family, frozen core pin b1d6cffede77f49410bf63525f30b9a46b02dc01 unchanged.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Merge the independently implemented and reviewed host-descriptor/resource-effects migration candidate into the repository default branch.
Verification
Merge method must remain merge commit so frozen dependency SHAs stay reachable from the default branch.