allow string names in exports

Author: jackjennings

.tool-versions

@@ -0,0 +1 @@
+nodejs 20.9.0

hook.js

@@ -191,6 +191,22 @@ async function processModule ({
       continue
     }
 
+    if (NODE_MAJOR >= 15) {
+      // const modName = `v${Buffer.from(n, 'utf-8').toString('hex')}`
+      const modName = normalizeModName(n)
+
+      setters.set(`$${modName}` + ns, `
+      let $${modName} = ${ns}["${n}"]
+      export { $${modName} as "${n}" }
+      set["${n}"] = (v) => {
+        $${modName} = v
+        return true
+      }
+      `)
+
+      continue
+    }
+
     setters.set(`$${n}` + ns, `
     let $${n} = ${ns}.${n}
     export { $${n} as ${n} }

test/fixtures/string-export.mjs

@@ -0,0 +1,3 @@
+export const foo = 41
+
+export { foo as "non-valid-identifier" }

test/hook/v15-string-export.mjs

@@ -0,0 +1,15 @@
+// Unless explicitly stated otherwise all files in this repository are licensed under the Apache 2.0 License.
+//
+// This product includes software developed at Datadog (https://www.datadoghq.com/). Copyright 2021 Datadog, Inc.
+
+import Hook from '../../index.js'
+import * as stringExportMjs from '../fixtures/string-export.mjs'
+import { strictEqual } from 'assert'
+
+Hook((exports, name) => {
+  if (name.match(/fixtures\/string-export\.mjs/)) {
+    exports['non-valid-identifier'] += 1
+  }
+})
+
+strictEqual(stringExportMjs['non-valid-identifier'], 42)