Skip to content

Conversation

@Ethan-Arrowood
Copy link
Contributor

@Ethan-Arrowood Ethan-Arrowood commented Oct 13, 2025

Work in progress to enable Node.js type stripping by correctly configuring code.

High level list of to dos:

  • Remove type: 'commonjs' from package.json for less-strict module resolution
  • Fix relative file paths to utilize PACKAGE_ROOT value
    • Consider some alternative for static files resolution such as moving values to JS strings
  • Fix imports and requires to include file endings
  • Fix file endings to match intended source module type
    • Pretty much all of our TS code is written using ESM syntax, but we've been transforming it to CJS anyways. Some early tests demonstrate that if we switch to ESM most of our code will continue to work fine. Maybe the easiest solution is to rename TS to .mts for now until we can get all the CJS converted over time 🤷‍♂️
  • Ensure TypeScript build configuration is outputting appropriately

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