Convert project to ESM and update authors#52
Merged
Conversation
- Add "type": "module" to package.json - Convert all src files from require/module.exports to import/export - Convert all test files to ESM imports; add __dirname shim where needed - Convert server.js and profile.js to ESM - Update test script to use --experimental-vm-modules - Replace author/contributors with authors array: Henric Persson and Joe Lencioni Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Use NODE_OPTIONS env var instead of running jest via node directly, which would try to parse the bash wrapper script as JavaScript. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
"type": "module"topackage.jsonto enable native ESMsrc/*.jsfiles fromrequire/module.exportstoimport/export__dirnameshim viafileURLToPathwhere needed; importjestexplicitly in snapshots testserver.jsandprofile.jsto ESMnode --experimental-vm-modulesauthor/contributorsfields withauthorsarray: Henric Persson and Joe Lencioni (with updated emails)Test plan
npm test🤖 Generated with Claude Code