Setting resolve.conditions causes Vite dev server run into infinite loop

[guyutongxue]

When setting vite.config.js as following:

import { defineConfig } from 'vite';
import solid from 'vite-plugin-solid';

// https://vite.dev/config/
export default defineConfig({
  resolve: {
    conditions: [], // <- Here, or any array that missed `"development|production"` condition
  },
  plugins: [solid()],
});

And following files including circular import:

// main.js
import { Bar } from './App.jsx';
export const foo = 0;


// App.jsx
import { foo } from './main.js';
export function Bar() {
  return foo;
}

Then the hot reload of dev server will make Vite run into infinite loops:

01:14:18 [vite] (client) hmr update /src/App.jsx
01:14:18 [vite] (client) hmr invalidate /src/App.jsx
01:14:18 [vite] (client) hmr update /src/App.jsx
01:14:18 [vite] (client) hmr invalidate /src/App.jsx
01:14:18 [vite] (client) hmr update /src/App.jsx
01:14:18 [vite] (client) hmr invalidate /src/App.jsx
01:14:18 [vite] (client) hmr update /src/App.jsx
01:14:18 [vite] (client) hmr invalidate /src/App.jsx
01:14:18 [vite] (client) hmr update /src/App.jsx
01:14:18 [vite] (client) hmr invalidate /src/App.jsx
# [ continuing ... ]

Reproducible playground

StackBlitz

Step to reproduce:

1. Open above link and wait for the startup command to complete.
2. Refresh the preview browser on the right.
3. Edit src/main.js content and save. That's it, bug occurs.

[birkskyum]

Which Vite version is this? 5 or 6

[guyutongxue]

Which Vite version is this? 5 or 6

Vite 6.0.7

[birkskyum]

@guyutongxue can you check if the fix here works, with i.e. pnpm link?

• Fix vite6 environment detection #173

steps: clone, pnpm i, pnpm build, and then from your repo pnpm link <path-to-vite-plugin-solid>.

[guyutongxue]

@guyutongxue can you check if the fix here works, with i.e. pnpm link?

• Fix vite6 environment detection #173

steps: clone, pnpm i, pnpm build, and then from your repo pnpm link <path-to-vite-plugin-solid>.

Yes, confirmed that #173 will fix my issue.

[birkskyum]

I can confirm it too in this stackblitz

https://stackblitz.com/edit/vitejs-vite-1k7n1gjp?file=src%2FApp.jsx