## Summary

- Add `ModuleKind::CommonJS` variant to support CommonJS module type in the parser
- Enable top-level `return` statements (CommonJS files are wrapped in a function)
- Enable top-level `new.target` (allowed in CommonJS function wrapper)
- Add proper errors for `import`/`export` statements in CommonJS
- Add proper errors for `import.meta` in CommonJS

The CommonJS source type can be set via:
- `SourceType::with_commonjs(true)` method
- Babel's `sourceType: "commonjs"` option

Closes #16200

## Test plan

- [x] Added test cases in `tasks/coverage/misc/pass/` for valid CommonJS features
- [x] Added test cases in `tasks/coverage/misc/fail/` for invalid CommonJS usage
- [x] Enabled Babel conformance tests for `sourceType: "commonjs"`
- [x] All existing tests pass

🤖 Generated with [Claude Code](https://claude.com/claude-code)

…nsions

Previously, `.cjs` and `.cts` files were using `ModuleKind::Script` which
was inconsistent with the recent addition of `ModuleKind::CommonJS` in #18089.

This change:
- Updates `SourceType::from_path()` to return `ModuleKind::CommonJS` for `.cjs`/`.cts`
- Fixes `eslint/no-eval` to check `!is_module()` instead of `is_script()` to handle
  both scripts and CommonJS (matching ESLint's `sourceType: "commonjs"` behavior)
- Fixes `eslint/no-redeclare` to run for both scripts and CommonJS files
- Updates tests to use proper CommonJS assertions

Co-Authored-By: Claude Opus 4.5 <[email protected]>

…nsions

Previously, `.cjs` and `.cts` files were using `ModuleKind::Script` which
was inconsistent with the recent addition of `ModuleKind::CommonJS` in #18089.

This change:
- Updates `SourceType::from_path()` to return `ModuleKind::CommonJS` for `.cjs`/`.cts`
- Fixes `eslint/no-eval` to check `!is_module()` instead of `is_script()` to handle
  both scripts and CommonJS (matching ESLint's `sourceType: "commonjs"` behavior)
- Fixes `eslint/no-redeclare` to run for both scripts and CommonJS files
- Updates tests to use proper CommonJS assertions

Co-Authored-By: Claude Opus 4.5 <[email protected]>

…tensions (#18117)

## Summary

- Updates `SourceType::from_path()` to return `ModuleKind::CommonJS` for `.cjs`/`.cts` files (previously returned `ModuleKind::Script`)
- Fixes `eslint/no-eval` to check `!is_module()` instead of `is_script()` to handle both scripts and CommonJS (matching ESLint's `sourceType: "commonjs"` behavior)
- Fixes `eslint/no-redeclare` to run for both scripts and CommonJS files
- Updates tests to use proper CommonJS assertions

This follows up on #18089 which added `ModuleKind::CommonJS` support but didn't update the file extension mapping.

## Test plan

- [x] `cargo test -p oxc_span`
- [x] `cargo test -p oxc_linter`

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Remove 2 outdated comments. #18089 introduced `commonjs` source type.

…18197)

#18089 added support in parser for `commonjs` source type.

Extend this support to the NAPI packages - accept `sourceType: 'commonjs'` in options for `oxc-parser` and `oxc-transform`.

…18198)

`commonjs` source type was previously supported in `RuleTester` only with `eslintCompat` option enabled (and it was then treated as `script`).

#18089 added support in Oxc parser for `commonjs`. Accept it in `RuleTester` (either with or without ESLint compatibility mode) and parse properly as CommonJS.

Presumably it's safe to mangle top-level symbols in CommonJS files. Alter the mangler example to do that.

Previously, we had no way to distinguish between script and CommonJS file types. After #18089 introduced a `commonjs` source type, we now do.

…18250)

Another follow-up after #18089.

Skip checking export declarations in CommonJS files. CommonJS files can't contain `export` declarations, so there's no point searching for them.

Follow-up to #18089.

Maybe we should add `SourceType::script()` so that it's easier to create a SourceType with `ModuleType::Script`.

Follow-up to #18089.

Maybe we should add `SourceType::script()` so that it's easier to create a SourceType with `ModuleType::Script`.