flux-pr-5156
Zod (TypeScript) · W2 · GPT-5.3 Codex
Tests passed. 3/3 commands passed. Strength: strong.
pnpm buildfind . -name vitest.config.ts -exec sed -i 's/test: {/test: { testTimeout: 30000,/' {} +pnpm test -- --maxWorkers 1 --maxConcurrency 1 --retry 2Partial score: 3/3
Trajectory
unknown · partial order onlyCanonical trajectory missing; showing coarse derived order only.
Quality
Equivalence Reasoning
stylistic
The agent patch applies the core behavioral fix: OpenAPI 3.0 `null` is emitted via nullable-compatible schemas, union handling no longer strips/rewrites `null` branches, and `nullable` wrappers use `$ref` + `nullable` semantics. Differences from gold (extra literal-`null` handling, no tuple spread tweak, no devDependency addition) do not change the intended runtime/schema-generation behavior for the described bug.
Code Review
The agent patch captures most of the OpenAPI 3.0 null-handling logic for tuples/unions, but it likely does not fully satisfy the intended change because a required dependency update is missing; there is also a minor implementation divergence in nullable processing.
The intended change set includes adding `@seriousme/openapi-schema-validator` in devDependencies, but this patch does not modify `package.json`. Given the task context includes test patching around OpenAPI output validity, this omission can cause test/runtime failures or leave validation coverage absent.
In the OpenAPI 3.0 nullable branch, `this.process(def.innerType, params)` is invoked but discarded. This differs from the intended minimal ref+nullable strategy and could introduce side effects in schema registration without affecting output directly.