Skip to content

fix(ack): align wrapper and lazy nullability contracts - #228

Open
leoafarias wants to merge 1 commit into
release/2.0from
fix/2.0-nullability-contracts
Open

leoafarias wants to merge 1 commit into
release/2.0from
fix/2.0-nullability-contracts

Conversation

@leoafarias

@leoafarias leoafarias commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Align boundary parsing and exported null policies with effective wrapper and lazy-schema flags.
  • Preserve explicit non-null assertions when an inner union or imported schema admits null.
  • Add 37 parse/encode/export regressions covering native, union, imported, defaulted, and lazy schemas.

Stacked on #224 (release/2.0). This is the first layer of the 2.0 readiness cleanup; published alpha.1 notes remain unchanged.

Validation

  • Confirmed 23 new regressions fail before the fix.
  • Focused contract suite: 154 passed.
  • Full core suite: 2,048 passed.
  • JSON Schema conformance: 78 passed.
  • Strict analysis and formatting passed.

CI and release preflight passed. The complete cleanup stack and coordinated alpha.2 preparation are linked from #237.

@leoafarias
leoafarias marked this pull request as ready for review October 26, 2026 16:17
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-26T16:19:00.560780Z 60261c4 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@leoafarias
leoafarias added this pull request to stack #238 October 26, 2026 16:30

This branch has not been deployed

No deployments
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