[Bug report] Endless loop in resolveSSRExternal in monorepo

[meteorlxy]

⚠️ IMPORTANT ⚠️ Please do not ignore this template. If you do, your issue will be closed immediately.

• Read the docs.
• 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 https://github.com/vuejs/vue-next instead.
• This is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

Describe the bug

When using vite in monorepo, resolveSSRExternal might cause endless loop and then "Maximum call stack size exceeded".

Reproduction

Related codes:

vite/packages/vite/src/node/ssr/ssrExternal.ts

Line 26 in 23f57ee

const deps = [...knownImports, ...Object.keys(pkg.dependencies || {})]

vite/packages/vite/src/node/ssr/ssrExternal.ts

Lines 57 to 64 in 23f57ee

	resolveSSRExternal(
	{
	...config,
	root: depRoot
	},
	knownImports,
	ssrExternals
	)

If the monorepo dependency appears in the knownImports, then it will be forwared to next resolveSSRExternal call and loop forever.

That can reproduce in vuepress-next repo:

yarn
yarn copy && yarn build
yarn docs:dev

System Info

• vite version: 2.1.2
• Operating System: Linux
• Node version: 12.16.1
• Package manager (npm/yarn/pnpm) and version: yarn 1.22.4

Logs (Optional if provided reproduction)

[vite] error while updating dependencies:
RangeError: Maximum call stack size exceeded
    at Function.resolveSync [as sync] (/data/meteorlxy/projects/@vuepress/vuepress-next/node_modules/resolve/lib/sync.js:90:53)
    at resolveFrom$2 (/data/meteorlxy/projects/@vuepress/vuepress-next/node_modules/vite/dist/node/chunks/dep-efe32886.js:3883:29)
    at resolveSSRExternal (/data/meteorlxy/projects/@vuepress/vuepress-next/node_modules/vite/dist/node/chunks/dep-efe32886.js:32155:51)
    at resolveSSRExternal (/data/meteorlxy/projects/@vuepress/vuepress-next/node_modules/vite/dist/node/chunks/dep-efe32886.js:32156:13)
    at resolveSSRExternal (/data/meteorlxy/projects/@vuepress/vuepress-next/node_modules/vite/dist/node/chunks/dep-efe32886.js:32156:13)
    at resolveSSRExternal (/data/meteorlxy/projects/@vuepress/vuepress-next/node_modules/vite/dist/node/chunks/dep-efe32886.js:32156:13)
    at resolveSSRExternal (/data/meteorlxy/projects/@vuepress/vuepress-next/node_modules/vite/dist/node/chunks/dep-efe32886.js:32156:13)
    at resolveSSRExternal (/data/meteorlxy/projects/@vuepress/vuepress-next/node_modules/vite/dist/node/chunks/dep-efe32886.js:32156:13)
    at resolveSSRExternal (/data/meteorlxy/projects/@vuepress/vuepress-next/node_modules/vite/dist/node/chunks/dep-efe32886.js:32156:13)
    at resolveSSRExternal (/data/meteorlxy/projects/@vuepress/vuepress-next/node_modules/vite/dist/node/chunks/dep-efe32886.js:32156:13)