[alongubkin]

Summary

• generate only Azure schemas transitively reachable from models used by Alien
• preserve the complete published model API behind the default full-models feature
• add real-spec reachability tests and a filtered-output size regression guard

Impact

• schemas: 1,850 -> 1,260 (-31.9%)
• generated Rust: 196,983 -> 147,886 lines (-24.9%)
• generated structs: 1,690 -> 1,126 (-33.4%)
• generated impls: 6,754 -> 5,069 (-25.0%)
• observed Azure rustc unit: 46.8s -> 26.9s
• observed generator build script: 6.2s -> 2.9s

The filtered timing run reused dependency artifacts from the baseline target, so source-size changes are the controlled evidence and timings are directional.

Validation

• cargo test -p alien-azure-clients --test schema_filter --no-default-features
• cargo test -p alien-azure-clients --lib --no-default-features (57 passed)
• cargo test -p alien-azure-clients --no-default-features --no-run
• cargo check -p alien-infra --features all-platforms,test-utils
• cargo check -p alien-azure-clients (default complete-model compatibility mode)
• git diff --check

Linear: ALIEN-862

[greptile-apps]

Confidence Score: 5/5

The PR appears safe to merge, with no new actionable issue introduced since the previous review and no outstanding prior finding.

Summary

This PR reduces Azure client generation work by retaining only schemas transitively reachable from Alien’s required models while preserving the complete published model API through the default full-models feature.

• Adds deterministic schema-reference traversal for OpenAPI components and Swagger definitions.
• Defines per-spec model roots and validates them against the checked-in Azure specifications.
• Disables default Azure client features for internal workspace dependencies so Alien builds receive the filtered model set.
• Adds filtered-output size and schema-count regression guards.
• The only change since the previous review reformats one JSON array and introduces no behavioral change.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  Build[Build alien-azure-clients] --> Feature{full-models enabled?}
  Feature -->|Yes, default public API| Full[Generate every Azure schema]
  Feature -->|No, internal Alien dependency| Roots[Load model_roots.json]
  Roots --> Traverse[Traverse transitive schema references]
  Traverse --> Filter[Remove unreachable schemas]
  Filter --> Guard[Check generated-line regression limit]
  Full --> Output[Generate Rust Azure models]
  Guard --> Output

Loading

_{Reviews (2) · Last reviewed commit: "chore: format Azure model roots"}

[alongubkin]

@greptileai Please re-review this PR. The only prior finding was explicitly withdrawn after confirming that full-models remains the default to preserve the published crate API, while Alien dependency builds intentionally disable default features and use the filtered model set. Both modes are validated.