Skip to content

refactor(models): strengthen model nullability and enable strict analyzer settings - #446

Merged
robbecker-wf merged 1 commit into
10xfrom
10x-schema-models-and-strict-analysis
Sep 17, 2026
Merged

robbecker-wf merged 1 commit into
10xfrom
10x-schema-models-and-strict-analysis

Conversation

@robbecker-wf

@robbecker-wf robbecker-wf commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

Stacked PR: Part 4 of 7

Stack Overview:

  1. #443 10x-openapi-spec-baseline -> 10x
  2. #444 10x-security-and-transport -> 10x-openapi-spec-baseline
  3. #445 10x-service-contracts-conformance -> 10x-security-and-transport
  4. #446 10x-schema-models-and-strict-analysis (this PR) -> 10x-service-contracts-conformance
  5. #447 10x-test-pyramid-and-smoke -> 10x-schema-models-and-strict-analysis
  6. #448 10x-ci-and-spec-sync -> 10x-test-pyramid-and-smoke
  7. #449 10x-conformance-plan -> 10x-ci-and-spec-sync

Summary

Enables strict Dart analyzer settings and reinforces model nullability, types, and forward-compatible enum deserialization.

  • Analyzer Strictness: Enabled strict-casts: true, strict-inference: true, and strict-raw-types: true in analysis_options.yaml.
  • Diagnostics Resolution: Fixed all 90+ strict-mode compiler warnings and type issues across models, utilities, examples, and unit tests.
  • Forward Compatibility: Ensured enums (CheckRunAnnotationLevel, CheckRunConclusion, ReactionType) gracefully handle unexpected or future GitHub values instead of throwing unhandled exceptions.
  • Model Cleanups: Corrected parameter types and nullability in CheckRun, CheckSuite, CheckRunAnnotation, LanguageBreakdown, and RateLimit.

…yzer settings

Co-authored-by: Cursor <cursoragent@cursor.com>
@robbecker-wf
robbecker-wf force-pushed the 10x-schema-models-and-strict-analysis branch from 97b75f3 to 9cdd343 Compare September 17, 2026 00:30
@robbecker-wf
robbecker-wf merged commit afdf358 into 10x Sep 17, 2026
@robbecker-wf
robbecker-wf deleted the 10x-schema-models-and-strict-analysis branch September 17, 2026 00:30
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