fix(tests): make surviving package tests buildable; park pre-engine-free relics - #1
Merged
Merged
Conversation
…ne-free relics The snapshot inherited test/example files that no longer build, for two different reasons: 1. Missing dev-deps (fixable): network's msgpack wire-format test uses rmp-serde — declared as a dev-dependency now (version dep only; pkg build rejects path deps). 2. Structural era mismatch (not fixable by deps): five test files + one example register processors IN-PROCESS via PROCESSOR_REGISTRY.register::<T>(), which only worked when these packages compiled against the streamlib engine facade. As engine-free packages their processors implement the plugin SDK's traits, so in-process typed registration is impossible by design. Those files are parked verbatim in legacy-tests/ (with the jpeg fixtures) pending a rewrite against the module-loading flow — see legacy-tests/README.md. Also documents the local dev-registry flow in the README (packages with cross-package schema deps need STREAMLIB_REGISTRY_URL — emit from a streamlib checkout via xtask static-registry, publish this repo's packages into it), and fixes the .gitignore .cargo/config.toml pattern to match nested link artifacts (it was root-anchored). Verified: network cargo test 9+2 green, vadr-vision 22+4 green, mavlink 23 green — each via streamlib link --engine + the file:// dev registry. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019w7Tw9EYuEvniYzGgFakrB
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.
The snapshot inherited test/example files that no longer build, for
two different reasons:
Missing dev-deps (fixable): network's msgpack wire-format test
uses rmp-serde — declared as a dev-dependency now (version dep
only; pkg build rejects path deps).
Structural era mismatch (not fixable by deps): five test files +
one example register processors IN-PROCESS via
PROCESSOR_REGISTRY.register::(), which only worked when these
packages compiled against the streamlib engine facade. As
engine-free packages their processors implement the plugin SDK's
traits, so in-process typed registration is impossible by design.
Those files are parked verbatim in legacy-tests/ (with the jpeg
fixtures) pending a rewrite against the module-loading flow —
see legacy-tests/README.md.
Also documents the local dev-registry flow in the README (packages
with cross-package schema deps need STREAMLIB_REGISTRY_URL — emit
from a streamlib checkout via xtask static-registry, publish this
repo's packages into it), and fixes the .gitignore .cargo/config.toml
pattern to match nested link artifacts (it was root-anchored).
Verified: network cargo test 9+2 green, vadr-vision 22+4 green,
mavlink 23 green — each via streamlib link --engine + the file://
dev registry.
Co-Authored-By: Claude Fable 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_019w7Tw9EYuEvniYzGgFakrB