Port #60262: Include non-enumerable keys in __importStar helper

.github/workflows/copilot-setup-steps.yml

@@ -0,0 +1,32 @@
+name: 'Copilot Setup Steps'
+
+on: workflow_dispatch
+
+jobs:
+  # The job MUST be called `copilot-setup-steps` or it will not be picked up by Copilot.
+  copilot-setup-steps:
+    runs-on: ubuntu-latest
+
+    # Set the permissions to the lowest permissions possible needed for your steps.
+    # Copilot will be given its own token for its operations.
+    permissions:
+      # If you want to clone the repository as part of your setup steps, for example to install dependencies, you'll need the `contents: read` permission. If you don't clone the repository in your setup steps, Copilot will do this for you automatically after the steps complete.
+      contents: read
+
+    # You can define any steps you want, and they will run before the agent starts.
+    # If you do not check out your code, Copilot will do this for you.
+    steps:
+      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+        with:
+          submodules: true
+      - run: |
+          cd _submodules/TypeScript
+          git config remote.origin.fetch "+refs/heads/*:refs/remotes/origin/*"
+          git fetch --unshallow
+      - uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
+      - uses: dtolnay/rust-toolchain@fcf085fcb4b4b8f63f96906cd713eb52181b5ea4 # stable
+      - uses: ./.github/actions/setup-go
+        with:
+          cache-name: copilot-setup-steps
+      - run: npm ci
+      - run: npx hereby build

internal/printer/helpers.go

@@ -163,13 +163,23 @@ var importStarHelper = &EmitHelper{
 	Scoped:       false,
 	Dependencies: []*EmitHelper{createBindingHelper, setModuleDefaultHelper},
 	Priority:     &Priority{2},
-	Text: `var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};`,
+	Text: `var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();`,
 }
 
 var importDefaultHelper = &EmitHelper{

internal/transformers/commonjsmodule_test.go

@@ -91,13 +91,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
 const a = __importStar(require("other"));`,
 			options: core.CompilerOptions{ESModuleInterop: core.TSTrue},

testdata/baselines/reference/submodule/compiler/esModuleInterop.js

@@ -36,13 +36,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };

testdata/baselines/reference/submodule/compiler/esModuleInterop.js.diff

@@ -1,34 +1,6 @@
 --- old.esModuleInterop.js
 +++ new.esModuleInterop.js
-@@= skipped -35, +35 lines =@@
- }) : function(o, v) {
-     o["default"] = v;
- });
--var __importStar = (this && this.__importStar) || (function () {
--    var ownKeys = function(o) {
--        ownKeys = Object.getOwnPropertyNames || function (o) {
--            var ar = [];
--            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
--            return ar;
--        };
--        return ownKeys(o);
--    };
--    return function (mod) {
--        if (mod && mod.__esModule) return mod;
--        var result = {};
--        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
--        __setModuleDefault(result, mod);
--        return result;
--    };
--})();
-+var __importStar = (this && this.__importStar) || function (mod) {
-+    if (mod && mod.__esModule) return mod;
-+    var result = {};
-+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-+    __setModuleDefault(result, mod);
-+    return result;
-+};
- var __importDefault = (this && this.__importDefault) || function (mod) {
+@@= skipped -56, +56 lines =@@
      return (mod && mod.__esModule) ? mod : { "default": mod };
  };
  Object.defineProperty(exports, "__esModule", { value: true });

testdata/baselines/reference/submodule/compiler/esModuleInteropDefaultImports.js

@@ -62,13 +62,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };

testdata/baselines/reference/submodule/compiler/esModuleInteropDefaultImports.js.diff

@@ -9,36 +9,7 @@
  module.exports = mod;
  //// [b.js]
  "use strict";
-@@= skipped -20, +20 lines =@@
- }) : function(o, v) {
-     o["default"] = v;
- });
--var __importStar = (this && this.__importStar) || (function () {
--    var ownKeys = function(o) {
--        ownKeys = Object.getOwnPropertyNames || function (o) {
--            var ar = [];
--            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
--            return ar;
--        };
--        return ownKeys(o);
--    };
--    return function (mod) {
--        if (mod && mod.__esModule) return mod;
--        var result = {};
--        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
--        __setModuleDefault(result, mod);
--        return result;
--    };
--})();
-+var __importStar = (this && this.__importStar) || function (mod) {
-+    if (mod && mod.__esModule) return mod;
-+    var result = {};
-+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-+    __setModuleDefault(result, mod);
-+    return result;
-+};
- var __importDefault = (this && this.__importDefault) || function (mod) {
-     return (mod && mod.__esModule) ? mod : { "default": mod };
+@@= skipped -42, +42 lines =@@
  };
  Object.defineProperty(exports, "__esModule", { value: true });
  exports.def = exports.default = void 0;

testdata/baselines/reference/submodule/compiler/esModuleInteropImportCall.js

@@ -27,13 +27,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Promise.resolve().then(() => __importStar(require("./foo"))).then(f => {
     f.default;
 });

testdata/baselines/reference/submodule/compiler/esModuleInteropImportCall.js.diff

@@ -1,34 +1,10 @@
 --- old.esModuleInteropImportCall.js
 +++ new.esModuleInteropImportCall.js
-@@= skipped -26, +26 lines =@@
- }) : function(o, v) {
-     o["default"] = v;
- });
--var __importStar = (this && this.__importStar) || (function () {
--    var ownKeys = function(o) {
--        ownKeys = Object.getOwnPropertyNames || function (o) {
--            var ar = [];
--            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
--            return ar;
--        };
--        return ownKeys(o);
--    };
--    return function (mod) {
--        if (mod && mod.__esModule) return mod;
--        var result = {};
--        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
--        __setModuleDefault(result, mod);
--        return result;
--    };
--})();
+@@= skipped -43, +43 lines =@@
+         return result;
+     };
+ })();
 -Promise.resolve().then(function () { return __importStar(require("./foo")); }).then(function (f) {
-+var __importStar = (this && this.__importStar) || function (mod) {
-+    if (mod && mod.__esModule) return mod;
-+    var result = {};
-+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-+    __setModuleDefault(result, mod);
-+    return result;
-+};
 +Promise.resolve().then(() => __importStar(require("./foo"))).then(f => {
      f.default;
  });

testdata/baselines/reference/submodule/compiler/esModuleInteropImportNamespace.js

@@ -28,13 +28,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
 const foo = __importStar(require("./foo"));
 foo.default;

testdata/baselines/reference/submodule/compiler/esModuleInteropImportNamespace.js.diff

@@ -1,33 +1,8 @@
 --- old.esModuleInteropImportNamespace.js
 +++ new.esModuleInteropImportNamespace.js
-@@= skipped -27, +27 lines =@@
- }) : function(o, v) {
-     o["default"] = v;
- });
--var __importStar = (this && this.__importStar) || (function () {
--    var ownKeys = function(o) {
--        ownKeys = Object.getOwnPropertyNames || function (o) {
--            var ar = [];
--            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
--            return ar;
--        };
--        return ownKeys(o);
--    };
--    return function (mod) {
--        if (mod && mod.__esModule) return mod;
--        var result = {};
--        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
--        __setModuleDefault(result, mod);
--        return result;
--    };
--})();
-+var __importStar = (this && this.__importStar) || function (mod) {
-+    if (mod && mod.__esModule) return mod;
-+    var result = {};
-+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-+    __setModuleDefault(result, mod);
-+    return result;
-+};
+@@= skipped -45, +45 lines =@@
+     };
+ })();
  Object.defineProperty(exports, "__esModule", { value: true });
 -var foo = __importStar(require("./foo"));
 +const foo = __importStar(require("./foo"));