Skip to content

Generate TypeScript declaration mappings for legacy resolution - #47

Merged
vrtmrz merged 1 commit into
mainfrom
fix/node10-types-versions
Aug 13, 2026
Merged

Generate TypeScript declaration mappings for legacy resolution#47
vrtmrz merged 1 commit into
mainfrom
fix/node10-types-versions

Conversation

@vrtmrz

@vrtmrz vrtmrz commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Generate TypeScript declaration fallbacks from the existing package export inventory so Node10-style resolution can type every supported subpath.

Summary

  • derive typesVersions from the same generated subpath inventory as exports;
  • regenerate both fields automatically after the octagonal-wheels workspace build; and
  • verify that every public subpath resolves to its declared .d.ts file with TypeScript's Node10 module resolution.

Rationale

Modern TypeScript resolution reads the declaration conditions in exports, but older or fallback resolution paths can ignore them and leave package subpaths unresolved. Those unresolved declarations may then be treated as error-like any types by downstream analysis.

The existing exports inventory remains authoritative. typesVersions mirrors that inventory for compatibility and does not change runtime resolution or the supported import surface.

Verification

  • confirmed that the focused regression test failed before the generated mappings were added;
  • ran npm run build --workspace octagonal-wheels;
  • ran npm run check --workspace octagonal-wheels;
  • ran npm run test:node --workspace octagonal-wheels;
  • resolved all 233 generated subpaths with TypeScript's Node10 resolver; and
  • packed the package and type-checked representative Self-hosted LiveSync imports together with the companion Commonlib change.

Release order

Publish an octagonal-wheels version containing this change before publishing the companion Commonlib release. Commonlib's declaration graph continues into octagonal-wheels subpaths, so both packages require the fallback mappings for complete downstream coverage.

@vrtmrz
vrtmrz marked this pull request as ready for review August 13, 2026 08:31
@vrtmrz
vrtmrz merged commit 2bd61e2 into main Aug 13, 2026
2 checks passed
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