Skip to content

Commit 9782886

Browse files
committed
fix: fix duplicate createRequire
1 parent 9b85fdf commit 9782886

File tree

2 files changed

+0
-42
lines changed

2 files changed

+0
-42
lines changed

packages/vite/src/node/optimizer/index.ts

-4
Original file line numberDiff line numberDiff line change
@@ -812,10 +812,6 @@ async function prepareRolldownOptimizerRun(
812812
sourcemap: true,
813813
dir: processingCacheDir,
814814
entryFileNames: '[name].js',
815-
banner:
816-
platform === 'node'
817-
? `import { createRequire } from 'module';const require = createRequire(import.meta.url);`
818-
: undefined,
819815
})
820816
await bundle.close()
821817
return result

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

-38
Original file line numberDiff line numberDiff line change
@@ -82,44 +82,6 @@ async function ssrTransformScript(
8282
originalCode: string,
8383
): Promise<TransformResult | null> {
8484
if (!process.env['DISABLE_OXC_MODULE_TRANSFORM']) {
85-
// something weird in playground/ssr-deps
86-
/*
87-
$ pnpm -C playground/ssr-deps dev
88-
89-
> @vitejs/[email protected] dev /home/hiroshi/code/others/rolldown-vite/playground/ssr-deps
90-
> node server
91-
92-
http://localhost:5173
93-
[
94-
{
95-
"severity": "Error",
96-
"message": "Identifier `createRequire` has already been declared",
97-
"labels": [
98-
{
99-
"message": "`createRequire` has already been declared here",
100-
"start": 9,
101-
"end": 22
102-
},
103-
{
104-
"message": "It can not be redeclared here",
105-
"start": 96,
106-
"end": 109
107-
}
108-
]
109-
}
110-
]
111-
import { createRequire } from 'module';const require = createRequire(import.meta.url);
112-
import { createRequire } from "module";
113-
*/
114-
if (
115-
code.includes(`import { createRequire } from 'module'`) &&
116-
code.includes(`import { createRequire } from "module"`)
117-
) {
118-
code = code.replace(
119-
`import { createRequire } from "module"`,
120-
' '.repeat('import { createRequire } from "module"'.length),
121-
)
122-
}
12385
const result = moduleRunnerTransform('test.js', code, { sourcemap: true })
12486
if (result.errors.length) {
12587
// eslint-disable-next-line no-console

0 commit comments

Comments
 (0)