feat: createRequire feature parity with periscopic.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@knighted/specifier",
-  "version": "3.1.0",
+  "version": "3.2.0",
   "description": "Node.js tool for updating your ES module and CommonJS specifiers.",
   "type": "module",
   "main": "dist",

src/format.ts

@@ -391,6 +391,30 @@ const format = async (src: string, ast: ParseResult, cb: Callback) => {
     'BlockStatement:exit'() {
       popScope()
     },
+    ForStatement() {
+      pushScope()
+    },
+    'ForStatement:exit'() {
+      popScope()
+    },
+    ForInStatement() {
+      pushScope()
+    },
+    'ForInStatement:exit'() {
+      popScope()
+    },
+    ForOfStatement() {
+      pushScope()
+    },
+    'ForOfStatement:exit'() {
+      popScope()
+    },
+    SwitchStatement() {
+      pushScope()
+    },
+    'SwitchStatement:exit'() {
+      popScope()
+    },
     CatchClause(node) {
       pushScope()
 

test/updateSrc.ts

@@ -329,4 +329,102 @@ describe('updateSrc', () => {
 
     assert.ok(update.indexOf("loader('./outside.mjs')") > -1)
   })
+
+  it('does not rewrite require shadowed by for-of scope', async () => {
+    const update = await specifier.updateSrc(
+      `
+        for (const require of [value => value]) {
+          require('./skip.js')
+        }
+
+        require('./rewrite.js')
+      `,
+      'js',
+      spec => {
+        return spec.value.replace('.js', '.mjs')
+      },
+    )
+
+    assert.ok(update.indexOf("require('./skip.js')") > -1)
+    assert.ok(update.indexOf("require('./rewrite.mjs')") > -1)
+  })
+
+  it('does not rewrite require shadowed by for-in scope', async () => {
+    const update = await specifier.updateSrc(
+      `
+        for (const require in { value: 1 }) {
+          require('./skip.js')
+        }
+
+        require('./rewrite.js')
+      `,
+      'js',
+      spec => {
+        return spec.value.replace('.js', '.mjs')
+      },
+    )
+
+    assert.ok(update.indexOf("require('./skip.js')") > -1)
+    assert.ok(update.indexOf("require('./rewrite.mjs')") > -1)
+  })
+
+  it('does not rewrite require shadowed by for scope', async () => {
+    const update = await specifier.updateSrc(
+      `
+        for (let require = value => value; false; ) {
+          require('./skip.js')
+        }
+
+        require('./rewrite.js')
+      `,
+      'js',
+      spec => {
+        return spec.value.replace('.js', '.mjs')
+      },
+    )
+
+    assert.ok(update.indexOf("require('./skip.js')") > -1)
+    assert.ok(update.indexOf("require('./rewrite.mjs')") > -1)
+  })
+
+  it('does not rewrite require shadowed by switch scope', async () => {
+    const update = await specifier.updateSrc(
+      `
+        switch (0) {
+          default:
+            const require = value => value
+            require('./skip.js')
+        }
+
+        require('./rewrite.js')
+      `,
+      'js',
+      spec => {
+        return spec.value.replace('.js', '.mjs')
+      },
+    )
+
+    assert.ok(update.indexOf("require('./skip.js')") > -1)
+    assert.ok(update.indexOf("require('./rewrite.mjs')") > -1)
+  })
+
+  it('does not desync createRequire tracking after a re-export declaration', async () => {
+    const update = await specifier.updateSrc(
+      `
+        export { foo } from './reexport.js'
+
+        const { createRequire } = require('node:module')
+        const customRequire = createRequire(import.meta.url)
+
+        customRequire('./rewrite.js')
+      `,
+      'js',
+      spec => {
+        return spec.value.replace('.js', '.mjs')
+      },
+    )
+
+    assert.ok(update.indexOf("from './reexport.mjs'") > -1)
+    assert.ok(update.indexOf("customRequire('./rewrite.mjs')") > -1)
+  })
 })