ssrTransform incorrectly transforms references in "for" block

[sheremet-va]

Describe the bug

Variable test is replaced with __vite_ssr_import_0__.test, even though the user referenced the local variable and not the named import.

Original code:

import { test } from './test.js'

for(const test of tests) {
  console.log(test)
}

for(let test = 0; test < 10; test++) {
  console.log(test)
}

for(const test in tests) {
  console.log(test)
}

Processed result:

const __vite_ssr_import_0__ = await __vite_ssr_import__("./test.js");

    

for(const test of tests) {
  console.log(__vite_ssr_import_0__.test)
}

for(let test = 0; __vite_ssr_import_0__.test < 10; __vite_ssr_import_0__.test++) {
  console.log(__vite_ssr_import_0__.test)
}

for(const test in tests) {
  console.log(__vite_ssr_import_0__.test)
}

Reproduction

https://stackblitz.com/edit/node-isfe8x?file=index.mjs

Steps to reproduce

Run node index.js

System Info

Stackblitz

Used Package Manager

npm

Logs

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[stackblitz]

Start a new pull request in StackBlitz Codeflow.

• node-isfe8x (Open reproduction)