search

umijs / umi

A framework in react community ✨
https://umijs.org
MIT License
15.19k stars 2.64k forks source link

[Bug] pnpm 安装的依赖过多导致 useModel 无法智能提示 @umijs/max #12545

Closed crazyair closed 2 weeks ago

crazyair commented 3 weeks ago

What happens?

使用 vscode 编辑器，代码中编写 useModel('usexxx')，鼠标 hover 到 useModel 上，快速修复没有从 @umijs/max 引入提示 如果 pnpm 是用 v6 是没问题的，但是用 v8.15.3 则失效

Mini Showcase Repository(REQUIRED)

Please provide a minimal reproduction then upload to your GitHub. 请提供 最小重现，并上传到你的 GitHub 仓库

https://github.com/crazyair/umi （本仓库没用，重要的是你要用 pnpm v8 安装依赖）

如果是用了 corepack，这可以在 pacakge.json 中增加 "packageManager": "pnpm@8.15.3"，然后 install

How To Reproduce

Steps to reproduce the behavior: 1. 2.

Expected behavior 1. 2.

Context

github-actions[bot] commented 3 weeks ago

由于缺乏足够的信息，我们暂时关闭了该 Issue。请修改（不要回复） Issue 提供最小重现以重新开启。谢谢。

crazyair commented 2 weeks ago

@fz6m 麻烦看下这个

fz6m commented 2 weeks ago

确保你的 vscode 是最新版本的，并在这个项目根目录打开 vscode 。

我新建了一个项目，没遇到你说的丢失提示这个问题，输入 useModel 可以正常提示，如还有问题，可以给一个最小复现视频看看。

crazyair commented 2 weeks ago

复现出来了 https://github.com/crazyair/umi-max-use-model-bug

crazyair commented 2 weeks ago

是 dependencies 安装的依赖过多，比如现在 17 个，就会无法推导类型

PeachScript commented 2 weeks ago

这周已发版，该 issue 对应 PR 下周发

crazyair commented 2 weeks ago

这周已发版，该 issue 对应 PR 下周发

一天发2个版本啊

PeachScript commented 2 weeks ago

@Jinbao1001 给 @zombieJ 补了个版本，他既要 Mako 也要改 node_modules 做调试 👀

crazyair commented 2 weeks ago

@fz6m 等会，我验证了下，@umijs/max 的路径变成了 ../../node_modules/xxx 然后自己又突然变成了 /Users/xxx

crazyair commented 2 weeks ago

@fz6m 等会，我验证了下，@umijs/max 的路径变成了 ../../node_modules/xxx 然后自己又突然变成了 /Users/xxx

知道了，更新 4.3.5 后，路径正确了，但是改 src 代码触发了 start 的刷新，导致又恢复回去了