remove conditional logic for page object type generation

Author: DenysKuchma

lib/command/definitions.js

@@ -230,12 +230,7 @@ function getImportString(testsPath, targetFolderPath, pathsToType, pathsToValue)
 
   for (const name in pathsToType) {
     const relativePath = getPath(pathsToType[name], targetFolderPath, testsPath)
-    // Page objects and other support modules are exported as plain objects, not as default exports
-    if (relativePath.endsWith('.ts')) {
-      importStrings.push(`type ${name} = typeof import('${relativePath}')['default'];`)
-    } else {
-      importStrings.push(`type ${name} = typeof import('${relativePath}');`)
-    }
+    importStrings.push(`type ${name} = typeof import('${relativePath}');`)
   }
 
   for (const name in pathsToValue) {

test/runner/definitions_test.js

@@ -107,8 +107,8 @@ describe('Definitions', function () {
       const definitionFile = types.getSourceFileOrThrow(`${codecept_dir}/steps.d.ts`)
       const extend = definitionFile.getFullText()
 
-      // Page objects are exported as plain objects, not classes
-      // So they should not have ['default'] to allow autocompletion
+      // Page objects are exported as plain objects in .js files
+      // Use typeof import() without ['default'] to allow TypeScript to infer the object type
       extend.should.include("type CurrentPage = typeof import('./po/custom_steps.js');")
       assert(!err)
       done()