toomuchdesign / openapi-ts-json-schema / 25611390992 / 1

100%

master: 100%

push

github

feat: add importExtension option and deprecate moduleSystem (#585)

* feat!: replace moduleSystem option with importExtension

Rename `moduleSystem: 'esm' | 'cjs'` to `importExtension: 'js' | 'ts' | 'none'`.
The new option name directly describes what is emitted (a file extension on import
specifiers) rather than conflating module system with TypeScript moduleResolution.
Adds `'ts'` value for Bun/Deno native TypeScript runtimes.

BREAKING CHANGE: `moduleSystem: 'esm'` → `importExtension: 'js'`; `moduleSystem: 'cjs'` → `importExtension: 'none'`

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* refactor: make importExtension required in makeRelativeImportPath

The default was redundant — all callers already hold options.importExtension.
Making it required forces explicit intent at every call site.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* refactor: keep moduleSystem as deprecated alias for importExtension

Rework the prior breaking change so it can ship as a non-breaking minor in
v2 instead of a major in v3. `importExtension` is the new option;
`moduleSystem` is kept and deprecated, mapped to `importExtension` only
when the new option is absent. When both are set, `importExtension` wins
and a deprecation warning is logged.

Internal modules consume only `importExtension` — the legacy option is
read in a single resolution block at the top of `openapiToTsJsonSchema()`.
Tests for the deprecated option are isolated in
`moduleSystem.deprecated.test.ts` so v3 removal is a clean delete.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

141 of 141 branches covered (100.0%)

Branch coverage included in aggregate %.

241 of 241 relevant lines covered (100.0%)

237.52 hits per line