Skip to content

🐛 apply schema compatibility fixes to generated models - #69

Merged
igotinfected merged 1 commit into
fix/update-upstream-schema-urlsfrom
fix/update-generated-model-behaviour
Oct 3, 2026
Merged

igotinfected merged 1 commit into
fix/update-upstream-schema-urlsfrom
fix/update-generated-model-behaviour

Conversation

@igotinfected

@igotinfected igotinfected commented Oct 3, 2026 •

Copy link
Copy Markdown
Member

Update the vendored generator and analyzer together so consumers get the choice and serialization fixes across existing schema versions.

Changes

Enabled scalar default-presence tracking and added SIRI regressions for repeated and optional choices, fixed values, and default assignment. Added a NeTEx regression for edits inside default objects. Object-valued defaults retain their existing serialization behaviour.

XCGA002 still requires mandatory choice selection in an initializer. Fixed-value properties remain getter-only, with hidden XML properties and inclusion methods for optional values.

Validation

Regenerated every model project, then built and tested the combined worktree. Checked the packed assemblies and bundled analyzer. The vendored packages are 99.0.14-local. The Release build has zero warnings or errors, and all 191 Transmodel tests pass. A separate packaged consumer accepts valid choices and reports XCGA001, XCGA002, and CS0200 for the corresponding invalid cases.

Transmodel stack, bottom to top: #68 → #69 → #70 → #71 → #72. Generator prerequisite: XmlSchemaClassGenerator #1.

This PR was generated with gpt-6-astra in Codex via T3 Code.

@coderabbitai

coderabbitai Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 55 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 85e18480-f9e1-4c63-9f93-1e5507f8a3ef
📥 Commits

Reviewing files that changed from the base of the PR and between 9f0a07d and 64229b1.

📒 Files selected for processing (13)
  • packages/XmlSchemaClassGenerator-beta.99.0.11-local.nupkg
  • packages/XmlSchemaClassGenerator-beta.99.0.14-local.nupkg
  • packages/XmlSchemaClassGenerator.Analyzer.99.0.11-local.nupkg
  • packages/XmlSchemaClassGenerator.Analyzer.99.0.14-local.nupkg
  • src/Directory.Packages.props
  • src/generator/Spillgebees.Transmodel.Generator/Services/CodeGenerator.cs
  • src/netex/Spillgebees.NeTEx.Models.Tests/Serialization/Netex/v1/SerializationTests.cs
  • src/netex/Spillgebees.NeTEx.Models.Tests/XsdTypeMappingTests.cs
  • src/siri/Spillgebees.SIRI.Models.Tests/ChoiceSchemaRegressionTests.cs
  • src/siri/Spillgebees.SIRI.Models.Tests/Serialization/DefaultPresenceTests.cs
  • src/siri/Spillgebees.SIRI.Models.Tests/Serialization/FixedValueSerializationTests.cs
  • src/siri/Spillgebees.SIRI.Models.Tests/Serialization/SiriSerializationTests.cs
  • src/siri/Spillgebees.SIRI.Models.Tests/XsdTypeMappingTests.cs
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@igotinfected
igotinfected added this pull request to stack #73 October 3, 2026 22:40
@igotinfected
igotinfected marked this pull request as ready for review October 3, 2026 22:42
@igotinfected
igotinfected merged commit 65264c2 into main Oct 3, 2026
5 checks passed
@igotinfected
igotinfected deleted the fix/update-generated-model-behaviour branch October 3, 2026 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant