fix: TypeScript files with dynamic require() paths now work correctly

- Fixed require() to use original .ts file path as base, not temp .mjs file
- This ensures dynamic require() with relative paths work from original location
- Fixed __dirname and __filename to use original file paths
- Handles user scenarios where require() paths are constructed at runtime
- Added test case for dynamic require() with environment-based paths
- All tests pass (config + container TypeScript support)

Fixes issue where: require(`./config/environment.${env}`) would fail
because require was relative to transpiled temp file location

Author: kobenguyent

lib/utils/typescript.js

@@ -37,18 +37,23 @@ export async function transpileTypeScript(mainFilePath, typescript) {
       let esmGlobals = ''
 
       if (usesRequire || usesModuleExports) {
+        // IMPORTANT: Use the original .ts file path as the base for require()
+        // This ensures dynamic require() calls work with relative paths from the original file location
+        const originalFileUrl = `file://${filePath.replace(/\\/g, '/')}`
         esmGlobals += `import { createRequire } from 'module';
-const require = createRequire(import.meta.url);
+const require = createRequire('${originalFileUrl}');
 const module = { exports: {} };
 const exports = module.exports;
 
 `
       }
 
       if (usesCommonJSGlobals) {
+        // For __dirname and __filename, also use the original file path
+        const originalFileUrl = `file://${filePath.replace(/\\/g, '/')}`
         esmGlobals += `import { fileURLToPath as __fileURLToPath } from 'url';
 import { dirname as __dirname_fn } from 'path';
-const __filename = __fileURLToPath(import.meta.url);
+const __filename = '${filePath.replace(/\\/g, '/')}';
 const __dirname = __dirname_fn(__filename);
 
 `
@@ -96,8 +101,16 @@ const __dirname = __dirname_fn(__filename);
 
     // Try adding .ts extension if file doesn't exist and no extension provided
     if (!path.extname(importedPath)) {
-      if (fs.existsSync(importedPath + '.ts')) {
-        importedPath = importedPath + '.ts'
+      const tsPath = importedPath + '.ts'
+      if (fs.existsSync(tsPath)) {
+        importedPath = tsPath
+      } else {
+        // Try .js extension as well
+        const jsPath = importedPath + '.js'
+        if (fs.existsSync(jsPath)) {
+          // Skip .js files, they don't need transpilation
+          continue
+        }
       }
     }
 
@@ -134,7 +147,11 @@ const __dirname = __dirname_fn(__filename);
         const tempFile = transpiledFiles.get(tsPath)
         // Get relative path from main temp file to this temp file
         const relPath = path.relative(baseDir, tempFile).replace(/\\/g, '/')
-        return `from './${relPath}'`
+        // Ensure the path starts with ./
+        if (!relPath.startsWith('.')) {
+          return `from './${relPath}'`
+        }
+        return `from '${relPath}'`
       }
 
       // Otherwise, keep the import as-is

test/data/typescript-dynamic-require/common/utils.ts

@@ -0,0 +1,9 @@
+// Simulate the user's scenario
+let env: string = process.env.E2E_ENV || "TEST";
+
+// Use path relative to this file's location  
+// Add .cjs extension since we're in an ESM project (package.json has "type": "module")
+const environmentPath: string = `../config/environment.${env}.cjs`;
+const environment = require(environmentPath);
+
+export const Environment = environment;

test/data/typescript-dynamic-require/config/environment.TEST.cjs

@@ -0,0 +1,5 @@
+module.exports = {
+  url: 'https://test.example.com',
+  apiUrl: 'https://api.test.example.com',
+  timeout: 5000
+};

test/data/typescript-dynamic-require/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "typescript-dynamic-require-test",
+  "version": "1.0.0",
+  "type": "module"
+}

test/data/typescript-dynamic-require/tests/codecept.conf.ts

@@ -0,0 +1,14 @@
+import { Environment } from '../common/utils'
+
+export const config = {
+  tests: './*_test.js',
+  output: './output',
+  helpers: {
+    REST: {
+      endpoint: Environment.apiUrl,
+    }
+  },
+  bootstrap: null,
+  mocha: {},
+  name: 'typescript-dynamic-require-test'
+}

test/unit/config_test.js

@@ -83,4 +83,16 @@ describe('Config', () => {
     expect(cfg.plugins.htmlReporter.enabled).to.equal(true)
     expect(cfg.name).to.equal('typescript-config-with-require')
   })
+
+  it('should load TypeScript config with dynamic require() paths', async () => {
+    const configPath = './test/data/typescript-dynamic-require/tests/codecept.conf.ts'
+    process.env.E2E_ENV = 'TEST'
+    const cfg = await config.load(configPath)
+    
+    expect(cfg).to.be.ok
+    expect(cfg.helpers).to.have.property('REST')
+    expect(cfg.helpers.REST.endpoint).to.equal('https://api.test.example.com')
+    expect(cfg.name).to.equal('typescript-dynamic-require-test')
+    delete process.env.E2E_ENV
+  })
 })