This looks behavior-preserving rather than a parser fix. The original condition already only keeps `minItems` when the value is `0` or `1`; `min_items == 1` is false for `None`, so the added tests would pass on the current code too.

[abobakr200023-ship-it]

This looks behavior-preserving rather than a parser fix. The original condition already only keeps minItems when the value is 0 or 1; min_items == 1 is false for None, so the added tests would pass on the current code too.

If the goal is readability, it may be clearer to frame this as a small cleanup. If there is an actual failing schema case, the regression test should cover a value that fails before this change.

Originally posted by @kiwigitops in #1600 (comment)

[kiwigitops]

For context, my original note was a review comment on #1600 rather than a separate bug report. I do not have a distinct failing SDK case here; the point was that the PR looked behavior-preserving unless it added a regression that failed on current main.

If there is no separate schema case being reported, I think this issue can be closed.