[Gudge (MGudgin)]

This PR removes obsolete legacy network-format selection after #1382 retires
pre-v0.9 contracts. Shared normalization now applies the directional network
sections directly, without changing runtime compatibility or policy identity.

Details

• Remove unreachable legacy parser, backend checks, and loopback-only tests.
• Keep directional defaults, proxy validation, and state-aware network presence.
• Assert that all supported one-shot contracts reject legacy network fields.

Test

• cargo fmt --manifest-path .\src\Cargo.toml --all -- --check — passed.
• cargo test --manifest-path .\src\Cargo.toml -p wxc_common -p bwrap_common -p nanvix_runner --quiet — passed (922 shared-crate tests plus backend and integration groups).
• cargo check --manifest-path .\src\Cargo.toml --workspace --all-targets --quiet — passed.
• cargo clippy --manifest-path .\src\Cargo.toml -p wxc_common --all-targets -- -D warnings — passed.
• cargo clippy --manifest-path .\src\Cargo.toml --workspace --all-targets -- -D warnings — failed on pre-existing clippy::unnecessary_sort_by in src/backends/process_container/common/src/appcontainer_runner.rs:117; unchanged by this PR.

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]

Copilot review overview

🟢 Approval recommended

The simplified path matches all registered contracts and preserves directional defaults, proxy validation, and state-aware presence semantics.

Review effort: Balanced
Findings: None

What changed in this PR

Simplifies network normalization now that pre-v0.9 contracts are retired.

Changes:

• Removes legacy network format selection and validation.
• Normalizes directional networking directly.
• Adds legacy-field rejection coverage and updates documentation.

File	Description
network_parser.rs	Removes legacy parsing and directly handles directional policy.
network_parser_loopback_spec_tests.rs	Removes obsolete legacy loopback tests.
network_parser_ingress_default_tests.rs	Updates parser invocation.
config_parser.rs	Simplifies normalization and adds contract rejection tests.
docs/​versioning.md	Documents directional-only normalization.

---

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

[Jeff Whiteside (jsidewhite)]

[Jeff Whiteside (jsidewhite)]

validator.rs still contains "pub const LEGACY: Self = Self(0);"

[Gudge (MGudgin)]

Jeff Whiteside (@jsidewhite) Good catch. NetworkPolicySupport::LEGACY is a leftover zero-bit name, and the runtime policy still has other legacy fields read by backend fallbacks. #1383 is scoped to removing the now-unreachable legacy input-normalization path; I don't want to expand this approved, green PR into a cross-backend runtime-model rewrite. We'll remove LEGACY, the obsolete runtime fields, and dependent backend paths/tests in a focused follow-up PR after #1383, preserving directional defaults, proxy handling, and state-aware presence semantics. Thanks for calling out the remaining cleanup.