fix: strip CJS require('vitest') and require('@jest/globals') in Mocha tests

The AI backend generates vitest/jest-style imports for Mocha projects.
Our sanitize_mocha_imports() stripped ESM `import { ... } from 'vitest'`,
but process_generated_test_strings() runs BEFORE postprocessing and calls
ensure_module_system_compatibility() which converts these to CJS requires.
Result: `const { ... } = require('vitest')` survived sanitization.

Added regexes for the CJS variants of vitest and @jest/globals requires.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Saga4

codeflash/languages/javascript/edit_tests.py

@@ -233,7 +233,13 @@ def inject_test_globals(
 
 
 _VITEST_IMPORT_RE = re.compile(r"^.*import\s+\{[^}]*\}\s+from\s+['\"]vitest['\"].*\n?", re.MULTILINE)
+_VITEST_REQUIRE_RE = re.compile(
+    r"^.*(?:const|let|var)\s+\{[^}]*\}\s*=\s*require\s*\(\s*['\"]vitest['\"]\s*\).*\n?", re.MULTILINE
+)
 _JEST_GLOBALS_IMPORT_RE = re.compile(r"^.*import\s+\{[^}]*\}\s+from\s+['\"]@jest/globals['\"].*\n?", re.MULTILINE)
+_JEST_GLOBALS_REQUIRE_RE = re.compile(
+    r"^.*(?:const|let|var)\s+\{[^}]*\}\s*=\s*require\s*\(\s*['\"]@jest/globals['\"]\s*\).*\n?", re.MULTILINE
+)
 _MOCHA_REQUIRE_RE = re.compile(
     r"^.*(?:const|let|var)\s+\{[^}]*\}\s*=\s*require\s*\(\s*['\"]mocha['\"]\s*\).*\n?", re.MULTILINE
 )
@@ -256,7 +262,9 @@ def sanitize_mocha_imports(source: str) -> str:
 
     """
     source = _VITEST_IMPORT_RE.sub("", source)
+    source = _VITEST_REQUIRE_RE.sub("", source)
     source = _JEST_GLOBALS_IMPORT_RE.sub("", source)
+    source = _JEST_GLOBALS_REQUIRE_RE.sub("", source)
     source = _MOCHA_REQUIRE_RE.sub("", source)
     return _VITEST_COMMENT_RE.sub("", source)
 

tests/test_languages/test_mocha_runner.py

@@ -491,6 +491,30 @@ def test_strips_vitest_comment(self):
         assert "vitest" not in result
         assert "const x = 1;" in result
 
+    def test_strips_vitest_require_cjs(self):
+        from codeflash.languages.javascript.edit_tests import sanitize_mocha_imports
+
+        source = "const { describe, test, expect, vi, beforeEach, afterEach } = require('vitest');\nconst x = 1;\n"
+        result = sanitize_mocha_imports(source)
+        assert "vitest" not in result
+        assert "const x = 1;" in result
+
+    def test_strips_jest_globals_require_cjs(self):
+        from codeflash.languages.javascript.edit_tests import sanitize_mocha_imports
+
+        source = "const { jest, describe, it } = require('@jest/globals');\nconst x = 1;\n"
+        result = sanitize_mocha_imports(source)
+        assert "@jest/globals" not in result
+        assert "const x = 1;" in result
+
+    def test_strips_vitest_comment_and_cjs_require(self):
+        from codeflash.languages.javascript.edit_tests import sanitize_mocha_imports
+
+        source = "// vitest imports (REQUIRED for vitest - globals are NOT enabled by default)\nconst { describe, test, expect, vi, beforeEach, afterEach } = require('vitest');\nconst { setCharset } = require('../lib/utils');\n"
+        result = sanitize_mocha_imports(source)
+        assert "vitest" not in result
+        assert "require('../lib/utils')" in result
+
     def test_preserves_unrelated_imports(self):
         from codeflash.languages.javascript.edit_tests import sanitize_mocha_imports