eggjs / eggjs/egg

dependencies 中错误包含了 @types 开头的类型依赖

Open
#5,423 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
TypeScript
Stars
19k
Forks
1.8k
PR merge metrics
No merged PRs in 30d

Description

### 在此输入你需要反馈的 Bug 具体信息(Bug in Detail):

1. 我做了什么
安装 egg(v3)之后,完成项目构建后,需要删除 devDependencies,打包得到用于部署的产物。

2. 我的预期
正确删除 devDependencies,如各种 `@types/*`。

在一些特殊情况(pnpm 安装 + `@types/koa` 和 `@types/koa-compose` 的循环依赖)下,会导致应用启动失败。

3. 实际行为
并未正确删除,保留了如 `@types/koa` 等大量依赖。

4. 原因分析
在 [latest egg v3](https://www.npmjs.com/package/egg/v/3.30.1) 和 [latest egg-core](https://www.npmjs.com/package/egg-core/v/5.5.1?activeTab=dependencies) 中,错误依赖了 `@types/koa` 等。

在 [latest egg v4](https://www.npmjs.com/package/egg/v/4.0.10?activeTab=dependencies) 中,虽然没有直接依赖 `@types/*`,但实际上还是有遗漏:https://www.npmjs.com/package/@eggjs/koa?activeTab=dependencies

5. 补充信息
已有人提出过类似问题:#5154,但并未得到解答。

请问 v3 是否还接受 bugfix 发布?如果需要的话,我可以提 PR。

### 可复现问题的仓库地址(Reproduction Repo)

### Node 版本号:

v18.19.0

### Eggjs 版本号:

3.30.1

### 相关插件名称与版本号(PlugIn and Name):

### 操作平台与版本号(Platform and Version):

MacOS Sonoma

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.