Add 0.6 configuration contract adapter

[Gudge (MGudgin)]

This PR adds the typed adapter from exact 0.6.0-alpha contract requests to the current wxc_common wire model.

Details

• Adds the contract-crate dependency and a version-scoped adapter with exhaustive source and destination mappings.
• Preserves optional-field presence, canonical compatibility aliases, and explicit absence for later-version fields.
• Adds expected-wire and current-wire equivalence coverage for representative requests, proxies, enums, and aliases.

Tests

• cargo fmt --all -- --check
• cargo check -p wxc_common
• cargo clippy -p wxc_common --all-targets -- -D warnings
• cargo test -p wxc_common

Microsoft Reviewers: Open in CodeFlow

[azure-pipelines]

Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

[Copilot]

Pull request overview

Adds the exact 0.6.0-alpha contract adapter into the canonical wxc_common wire model.

Changes:

• Adds exhaustive contract-to-wire mappings.
• Preserves aliases, optional sections, and later-version field absence.
• Adds comprehensive mapping and equivalence tests.

Show a summary per file

File	Description
src/Cargo.toml	Registers the contract crate dependency.
src/Cargo.lock	Records the new dependency edge.
src/core/wxc_common/Cargo.toml	Adds the contract crate to wxc_common.
src/core/wxc_common/src/lib.rs	Exposes the internal adapter module.
src/core/wxc_common/src/config_contract_adapters/mod.rs	Declares the versioned adapter.
src/core/wxc_common/src/config_contract_adapters/v0_6.rs	Implements and tests the 0.6 adapter.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

• Files reviewed: 5/6 changed files
• Comments generated: 0
• Review effort level: Balanced

[Branden Bonaby (bbonaby)]

suggestion: I saw these conversion helpers and immediately thought they might be a good fit for the From/Into traits. Since these are infallible conversions between the contract and wire types, we could implement From<contract::X> for wire::X, which would simplify the nested conversions quite a bit when converting from request to MxcConfig:

Self {
    // ...
    containment: containment.into_option().map(Into::into),
    process: Some(process.into()),
    lifecycle: lifecycle.into_option().map(Into::into),
    process_container: process_container.into_option().map(Into::into),
    lxc: lxc.into_option().map(Into::into),
    filesystem: filesystem.into_option().map(Into::into),
    fallback: fallback.into_option().map(Into::into),
    network: network.into_option().map(Into::into),
    ui: ui.into_option().map(Into::into),
    // ...
}

So if we implement From<contract::Request> for wire::MxcConfig, I think we could remove into_wire entirely aswell and have callers just use request.into(). What do you think?

[Gudge (MGudgin)]

Sorry, only saw this comment after I'd merged. Looking into this now.