fix(env-replacer): don't modify string literals (#1943)

* fix(env-replacer): don't modify string literals

fix #1941

* test(env-replacer): verify string literals are ignored

* chore: fix typo envRelacer.ts -> envReplacer.ts

* chore: fix typo envRelacer.ts -> envReplacer.ts

* chore: move strip-literal to correct package

packages/vitest/package.json

@@ -98,6 +98,7 @@
     "chai": "^4.3.6",
     "debug": "^4.3.4",
     "local-pkg": "^0.4.2",
+    "strip-literal": "^0.4.0",
     "tinypool": "^0.2.4",
     "tinyspy": "^1.0.2",
     "vite": "^2.9.12 || ^3.0.0-0"

packages/vitest/src/node/plugins/envRelacer.ts → packages/vitest/src/node/plugins/envReplacer.ts

@@ -1,5 +1,6 @@
 import MagicString from 'magic-string'
 import type { Plugin } from 'vite'
+import { stripLiteral } from 'strip-literal'
 
 // so people can reassign envs at runtime
 // import.meta.env.VITE_NAME = 'app' -> process.env.VITE_NAME = 'app'
@@ -8,9 +9,11 @@ export const EnvReplacerPlugin = (): Plugin => {
     name: 'vitest:env-replacer',
     enforce: 'pre',
     transform(code) {
-      let s: MagicString | null = null
+      if (!/\bimport\.meta\.env\b/g.test(code))
+        return null
 
-      const envs = code.matchAll(/\bimport\.meta\.env\b/g)
+      let s: MagicString | null = null
+      const envs = stripLiteral(code).matchAll(/\bimport\.meta\.env\b/g)
 
       for (const env of envs) {
         s ||= new MagicString(code)

packages/vitest/src/node/plugins/index.ts

@@ -4,7 +4,7 @@ import type { ResolvedConfig, UserConfig } from '../../types'
 import { deepMerge, ensurePackageInstalled, notNullish } from '../../utils'
 import { resolveApiConfig } from '../config'
 import { Vitest } from '../core'
-import { EnvReplacerPlugin } from './envRelacer'
+import { EnvReplacerPlugin } from './envReplacer'
 import { GlobalSetupPlugin } from './globalSetup'
 import { MocksPlugin } from './mock'
 import { CSSEnablerPlugin } from './cssEnabler'

test/core/test/env.test.ts

@@ -43,3 +43,7 @@ test('custom env', () => {
   expect(process.env.CUSTOM_ENV).toBe('foo')
   expect(import.meta.env.CUSTOM_ENV).toBe('foo')
 })
+
+test('ignores import.meta.env in string literals', () => {
+  expect('import.meta.env').toBe('import' + '.meta.env')
+})