Skip to content

chore: use modern import of 'neotraverse' - #4274

Closed
gameroman wants to merge 1 commit into
swagger-api:mainfrom
gameroman:chore/modern-neotraverse
Closed

chore: use modern import of 'neotraverse'#4274
gameroman wants to merge 1 commit into
swagger-api:mainfrom
gameroman:chore/modern-neotraverse

Conversation

@gameroman

@gameroman gameroman commented Aug 24, 2026

Copy link
Copy Markdown

Description

Replaces neotraverse/legacy import with the modern one

Motivation and Context

How Has This Been Tested?

CI passes

Screenshots (if appropriate):

Types of changes

  • No code changes (changes to documentation, CI, metadata, etc)
  • Dependency changes (any modification to dependencies in package.json)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@robert-hebel-sb

Copy link
Copy Markdown
Contributor

hey @gameroman 👋
thanks for the PR 🖤

Have you checked locally whether tests were passing?

The modern neotraverse API is ESM-only (ES2022 target), but our Jest config doesn't transform node_modules. This causes SyntaxError: Cannot use import statement outside a module.

@gameroman gameroman closed this Aug 27, 2026
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