api.appData.outputPath 为 undefined 会导致配置了 forkTSChecker 的项目从根部被完全删除

[tommytroylin]

kmi/packages/preset-bundler/src/features/rspack/rspack.ts

Lines 100 to 108 in 6f2b2cc

	api.onBuildComplete(async ({ err }) => {
	if (err && api.userConfig.forkTSChecker) {
	// rspack's type checking output will continue to be generated, to avoid interfering with subsequent processes, manually remove the file if it exists
	const absOutputPath = pathe.join(api.cwd, api.appData.outputPath)
	if (await fsExtra.exists(absOutputPath)) {
	await fsExtra.remove(absOutputPath)
	}
	}
	})

api.appData.outputPath undefined
api.paths.absOutputPath "/Users/Tommy/project-playground/kmi-bug/dist"
info  - Umi v4.4.12

需要换成 api.paths.absOutputPath

[tommytroylin]

仔细看了下。换掉也会导致 dist 目录被删。
这块得看下逻辑为啥这样写了。。

[xierenyuan]

仔细看了下。换掉也会导致 dist 目录被删。 这块得看下逻辑为啥这样写了。。

forkTSChecker 开启有 err 的情况下 如果不删除目录会导致有些 流水线认为是成功的、理论上这只会导致 检查失败的情况下删除 dist、webpack 如果 forkTSChecker 检查失败是不生成代码的 rspack 的插件还会生成代码

[xierenyuan]

@tommytroylin 可以给具体的 case 我看看 这个逻辑是否有必要保留

[tommytroylin]

@xierenyuan 理解了。
应该是类似 webpack bail 或者 ts noEmitOnError 这种配置？

如果是为了阻塞 CI 应该可以通过 process.exit(1) 这种方式达成。
删文件有点暴力。

不过绕不过的话。应该也得给个配置控制行为。比如 fork-ts-checker-webpack-plugin 里有个 async 配置。

[xierenyuan]

@xierenyuan 理解了。 应该是类似 webpack bail 或者 ts noEmitOnError 这种配置？

如果是为了阻塞 CI 应该可以通过 process.exit(1) 这种方式达成。 删文件有点暴力。

不过绕不过的话。应该也得给个配置控制行为。比如 fork-ts-checker-webpack-plugin 里有个 async 配置。

嗯好建议 可以在这一行 提个 pr 通过 process.exit(1) 解决这个问题吗