@defold-typescript/transpiler 0.23.0 → 0.24.0

package/dist/index.js

@@ -307,11 +307,18 @@ function isFactoryOnlyImport(node) {
   if (clause === undefined || clause.name !== undefined) {
     return false;
   }
+  if (clause.isTypeOnly) {
+    return false;
+  }
   const bindings = clause.namedBindings;
   if (bindings === undefined || !ts2.isNamedImports(bindings)) {
     return false;
   }
-  return bindings.elements.every((element) => FACTORY_NAMES.has((element.propertyName ?? element.name).text));
+  const runtime = bindings.elements.filter((element) => !element.isTypeOnly);
+  if (runtime.length === 0) {
+    return false;
+  }
+  return runtime.every((element) => FACTORY_NAMES.has((element.propertyName ?? element.name).text));
 }
 var lifecycleErasurePlugin = {
   visitors: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defold-typescript/transpiler",
-  "version": "0.23.0",
+  "version": "0.24.0",
   "description": "TypeScript-to-Lua build pipeline tuned for Defold's runtime.",
   "license": "MIT",
   "repository": {
@@ -35,7 +35,7 @@
     "test": "bun test"
   },
   "dependencies": {
-    "@defold-typescript/types": "0.23.0",
+    "@defold-typescript/types": "0.24.0",
     "@typescript-to-lua/language-extensions": "1.19.0",
     "typescript-to-lua": "^1.36.0"
   }

package/src/lifecycle-erasure.ts

@@ -312,13 +312,22 @@ export function isFactoryOnlyImport(node: ts.ImportDeclaration): boolean {
   if (clause === undefined || clause.name !== undefined) {
     return false;
   }
+  // A whole-clause `import type { ... }` binds nothing at runtime; leave it for
+  // the normal transform, which elides it.
+  if (clause.isTypeOnly) {
+    return false;
+  }
   const bindings = clause.namedBindings;
   if (bindings === undefined || !ts.isNamedImports(bindings)) {
     return false;
   }
-  return bindings.elements.every((element) =>
-    FACTORY_NAMES.has((element.propertyName ?? element.name).text),
-  );
+  // Type-only specifiers ride along and drop with the erased statement; erase
+  // only when every runtime specifier is a lifecycle factory.
+  const runtime = bindings.elements.filter((element) => !element.isTypeOnly);
+  if (runtime.length === 0) {
+    return false;
+  }
+  return runtime.every((element) => FACTORY_NAMES.has((element.propertyName ?? element.name).text));
 }
 
 export const lifecycleErasurePlugin: Plugin = {