Skip to content

Fix native model imports and test generator configuration parity - #96

Merged
urbankrepel-povio merged 2 commits into
mainfrom
codex/native-model-imports
Sep 10, 2026
Merged

Fix native model imports and test generator configuration parity#96
urbankrepel-povio merged 2 commits into
mainfrom
codex/native-model-imports

Conversation

@urbankrepel-povio

@urbankrepel-povio urbankrepel-povio commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Native module-local output could import a schema directly while referencing CommonModels, causing ReferenceError at module load. The original parity CI missed that layout. Fix model ownership/imports and expand CI to exercise 236 configurations against two checked-in schemas.

The matrix covers all 64 layout combinations (48 valid, 16 explicitly rejected contradictory placements), representative values for every other renderer option, and transport/import interactions. It compares exact file lists and SHA-256 hashes across JavaScript/native on Linux/macOS. Native tests assert the binding is actually invoked, distinguish full and hybrid routes, and include empty/rejected cases in the manifest contract. Generated models are loaded across all valid layouts, and runner integration covers YAML/JSON input, cleanup, preservation of user files, and unchanged writes.

The expanded cases also fix unsplit naming/tag behavior, shared and module-local ownership, custom suffix imports, standalone imports, request-config forwarding, empty client import paths, and description handling. Model namespace routing preserves URL strings and validation regexes. Both demo fixtures include shared-schema regressions; the configuration fixture activates ACL, pagination, builders, defaults, descriptions, deprecated endpoints, and domain errors.

Validation: full JS and native test suites, Rust regression test, typecheck, lint, TypeScript/native release builds, and local exact-hash comparison of 4,320 generated files across 472 fixture/configuration cases. Cross-platform checks run in the separate Renderer parity workflow. Arbitrary option strings/lists use representative values; generated runtime execution covers models. incremental remains a compatibility option with identical unchanged-write behavior for both values.

@urbankrepel-povio urbankrepel-povio changed the title Fix native model namespace imports and expand demo parity coverage Fix native model imports and test generator configuration parity Sep 10, 2026
@urbankrepel-povio
urbankrepel-povio merged commit b0518b3 into main Sep 10, 2026
6 checks passed
@urbankrepel-povio
urbankrepel-povio deleted the codex/native-model-imports branch September 10, 2026 12:56
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.

2 participants