feat(evasive-transform): expose createEvasiveTransformPass for use with @endo/parser-pipeline

[boneskull]

This exposes a new function, createEvasiveTransformPass(), which can be used with @endo/parser-pipeline to unify AST parsing.

Refactored out some common code to maximize logic reuse. Both the new function and the existing transformAst() use the new makeTransformCommentsVisitor() to build the visitors for @babel/traverse.

[boneskull]

Warning

This PR is part of a stack and targets branch boneskull/module-source-pipeline, not master.
DO NOT MERGE until feat(module-source): expose createModuleSourcePasses() #3159 is merged into master.

📚 Pull Request Stack

• branch: master
  • feat(parser-pipeline): create @endo/parser-pipeline #3158 - branch: boneskull/ast-service
    • feat(module-source): expose createModuleSourcePasses() #3159 - branch: boneskull/module-source-pipeline
      • feat(evasive-transform): expose createEvasiveTransformPass for use with @endo/parser-pipeline #3161 - branch: boneskull/evasive-pipeline (this PR)

---

Managed by gh-stack

[changeset-bot]

⚠️ No Changeset found

Latest commit: dff723b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[Copilot]

Pull request overview

This PR exposes a new createEvasiveTransformPass() API from @endo/evasive-transform so it can be used as a TransformPass within @endo/parser-pipeline, while refactoring shared traversal logic into makeTransformCommentsVisitor().

Changes:

• Added createEvasiveTransformPass() (+ types) and exported it from the package entry point.
• Refactored AST traversal visitor construction into makeTransformCommentsVisitor() for reuse by both transformAst() and the new pass.
• Added AVA coverage for the new transform pass behavior.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
packages/evasive-transform/test/transform-pass.test.js	Adds tests validating the new parser-pipeline-compatible transform pass.
packages/evasive-transform/src/transform-pass.js	Introduces createEvasiveTransformPass() that returns a Babel-traverse visitor wrapper.
packages/evasive-transform/src/transform-pass-types.ts	Defines TransformPass and options types for the new API.
packages/evasive-transform/src/transform-code.js	JSDoc type-import cleanup for Babel AST/visitor types.
packages/evasive-transform/src/transform-ast.js	Extracts shared visitor logic into makeTransformCommentsVisitor() and reuses it in transformAst().
packages/evasive-transform/src/index.js	Exports createEvasiveTransformPass() from the public entry point.
packages/evasive-transform/src/generate.js	Adds a @ts-expect-error suppression around inputSourceMap.
packages/evasive-transform/package.json	Adds Babel-related @types/* packages for typing support.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

// @ts-expect-error here is brittle: once @babel/generator option types include inputSourceMap (or once inputSourceMap is correctly typed), tsc will fail due to an unused expect-error. Prefer fixing the underlying type mismatch by normalizing/casting inputSourceMap to the type expected by Babel generator (and updating the sourceMap option typedef accordingly), and remove the @ts-expect-error suppression.

[boneskull]

This is valid. I didn't find any such option. It may have been leftover from the Agoric fork of @babel/generator. @kriskowal does that ring a bell?

[boneskull]

n/m; I found the option in the source. It's just undocumented in the types, apparently.

[Copilot]

@types/babel__traverse is a types-only package and is typically kept in devDependencies across this repo (e.g. packages/module-source/package.json). If the intent is to ship it for downstream type resolution, consider also whether @types/babel__generator should be treated the same way for consistency (since this package’s public JSDoc types reference @babel/generator). Otherwise, move @types/babel__traverse back to devDependencies to avoid expanding the runtime dependency set unnecessarily.

[boneskull]

This is intentional; @types/babel__generator is only used in internal types. I've updated tsconfig.json to strip internal types.