[Bug]: modern build failed on windows10 but work successfully on windows11 and macOS

[AuglyXu]

Version

System： windows10
version: 2.39.0

Details

我的配置如下：
import { moduleTools, defineConfig } from '@modern-js/module-tools';

export default defineConfig({
buildConfig: [
{
format: 'cjs',
target: 'es6',
buildType: 'bundleless',
outDir: './build/lib',
sideEffects: [/.css$/],
dts: false,
/** 只需拷贝一次即可 */
copy: {
patterns: [
{
from: 'package.json',
context: __dirname,
to: './../package.json',
},
],
},
},
{
format: 'esm',
target: 'es6',
buildType: 'bundleless',
outDir: './build/es',
sideEffects: [/.css$/],
dts: false,
},
],
plugins: [moduleTools()],
});

Reproduce link

base config enough

Reproduce Steps

[chenjiahan]

@10Derozan Please check this

[10Derozan]

重装依赖可以解决吗

[AuglyXu]

重装依赖可以解决吗

解决不了诶

[10Derozan]

我理解这是本地环境的问题吧，可以提供一些解答么？感谢@HerringtonDarkholme

[fengjiling123]

@10Derozan 把buildType: 'bundleless'改成buildType: 'bundle'就好了，麻烦看一下为什么bundleless会报这个错

[fengjiling123]

@10Derozan 把demo的配置改成npm-component也会报这个错

[HerringtonDarkholme]

Hey @AuglyXu can you list the content inside node_modules of @ast-grep. Probably we need all directories inside there.

[HerringtonDarkholme]

I can see napi's node binding in the npm package. Is it caused by pnpm + windows?

[AuglyXu]

I can see napi's node binding in the npm package. Is it caused by pnpm + windows?

Is this what you want to see?
It is caused by pnpm + windows.

[HerringtonDarkholme]

@AuglyXu can you try these commands in a new project? https://ast-grep.github.io/guide/api-usage.html

pnpm add @ast-grep/napi

import { js } from '@ast-grep/napi';

const source = `console.log("hello world")`
const ast = js.parse(source)
const node = ast.root().find('console.log')

If the code example works, I would speculate the problem is pnpm.

[HerringtonDarkholme]

@AuglyXu I think there is some issues in your deps.

pnpm is finding ast-grep/napi 0.12.0

while modern only installs 0.1.13

[AuglyXu]

@AuglyXu I think there is some issues in your deps.

pnpm is finding ast-grep/napi 0.12.0

while modern only installs 0.1.13

Thank you @HerringtonDarkholme .Actually it not append on my computer, it's my partner's computer happened.My partner firmly believe that I pasted the wrong picture and she proposed the new issue https://github.com/web-infra-dev/modern.js/issues/4900, so she unwilling to try and i'm also distressed.She say may be you can position from the new picture.Does she need to run the above code or you can position the question combined the two pictures？

[10Derozan]

@10Derozan 把buildType: 'bundleless'改成buildType: 'bundle'就好了，麻烦看一下为什么bundleless会报这个错

bundleless 的场景我才会用 ast-grep 做一些例如别名替换的操作

[10Derozan]

I recommend you can follow the steps below:

1. Check the package version. Now modern.js use [email protected], please use the latest modern.js and reinstall.
2. Then check the node_modules, you had better add a screenshot about @ast-grep in node_modules.
3. If the version is right, and win32-x64-msvc.node exist, use ast-grep directly, and follow the tips, check the same error or not.

It doesn't look like it's an ast grep problem. If you can not reappear, i'll close this issue two days later

@AuglyXu I think there is some issues in your deps.
pnpm is finding ast-grep/napi 0.12.0

while modern only installs 0.1.13

Thank you @HerringtonDarkholme .Actually it not append on my computer, it's my partner's computer happened.My partner firmly believe that I pasted the wrong picture and she proposed the new issue https://github.com/web-infra-dev/modern.js/issues/4900, so she unwilling to try and i'm also distressed.She say may be you can position from the new picture.Does she need to run the above code or you can position the question combined the two pictures？

[10Derozan]

Ok ,fine, i found another issue the same with this, switch there.