Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v2 Plan #6

Closed
24 tasks done
condorheroblog opened this issue Oct 30, 2023 · 9 comments
Closed
24 tasks done

v2 Plan #6

condorheroblog opened this issue Oct 30, 2023 · 9 comments
Assignees
Labels
enhancement New feature or request

Comments

@condorheroblog
Copy link
Owner

condorheroblog commented Oct 30, 2023

A checklist for tracking todos of the next breaking changes

  • Support CJS format, Now support ["ts", "js", "mjs", "cjs", "cts", "mts"]
  • Support infixName options(xxx.fake.{ts,js,mjs,cjs,cts,cjs}) - All fake files with filenames containing fake
  • rename mock to fake- In order to be consistent with fake in the package name(vite-plugin-fake-server)
    • The default value of include is changed to fake
    • build The default value of outDir is changed to fakeServer
  • Bug fixes and optimizations in watch mode
    • Now it only monitors file updates, but does not monitor file deletions and additions - plugin.ts#L78
    • Use HMR strategy when changing files
    • When loading the fake file for the first time, use the concurrency strategy(5.402s => 1.309s in react-sample)
  • revert logger option 保留之前的logger配置项需求 #7
  • log display request method
  • Logs are not forced to be turned on in the production environment

用于跟踪下一个重大变更的待办事项的清单

  • 支持 CJS 格式, 现在默认支持的文件是 ["ts", "js", "mjs", "cjs", "cts", "mts"]
  • 支持配置中缀名 infixName(xxx.fake.{ts,js,mjs,cjs,cts,cjs}) - 所有的 fake 文件,名字要有 fake
  • mock 重命名为 fake - 为了与包名中的 fake 保持一致(vite-plugin-fake-server)
    • include 默认值改为 fake
    • build outDir 默认值改为 fakeServer
  • watch 模式下的错误修复和优化
    • 现在只监控文件更新,不监控文件删除和添加 - plugin.ts#L78
    • 更改文件时使用 HMR 策略
    • 第一次加载 fake 文件时,使用并发策略(5.402s => 1.309s 在 react-sample 中等待时间大约减少了 5 倍)
  • 恢复 logger 参数 保留之前的logger配置项需求 #7
  • 日志中显示请求方法
  • 不在生产环境中强制显示日志信息
@condorheroblog condorheroblog added the enhancement New feature or request label Oct 30, 2023
@condorheroblog condorheroblog self-assigned this Oct 30, 2023
@condorheroblog condorheroblog pinned this issue Oct 30, 2023
@condorheroblog condorheroblog unpinned this issue Oct 30, 2023
@condorheroblog condorheroblog pinned this issue Oct 30, 2023
@xiaoxian521
Copy link

嘿,下面是一些看法

xxx.fake.{ts,js,mjs} - 所有的 fake 文件,名字要有 fake

每个文件多了个fake 其实给用户带来了负担 或者也影响了代码读取的性能 配置暴露尽量随意也符合用户习惯

@xiaoxian521
Copy link

将 mock 重命名为 fake 其实跟 xxx.fake.{ts,js,mjs} - 所有的 fake 文件,名字要有 fake 是冲突的

所有文件都在 fake 文件夹下 也就意味着这里的文件都属于 fake, 再强制要求用户添加 .fake. 或许有点多余 不要介意哈🤣

@condorheroblog
Copy link
Owner Author

我是从下面这个角度考虑的:

fake/mock 文件夹下很有可能有这种工具类的函数文件,https://github.com/vbenjs/vue-vben-admin/blob/main/mock/_util.ts 本身并不是 fake/mock 文件,这时候就需要再添加中配置忽略此文件。

如果采用 xxx.fake.{ts,js,mjs} 这种格式,文件本身很有辨识度,工具类等函数可以随意添加,这样无需任何配置,即可开箱即用。


另外我新增了 https://github.com/condorheroblog/vite-plugin-fake-server/tree/v2#infixname 来配置中缀名。

@xiaoxian521
Copy link

好的 请问 infixname 配置 “” 空 是不是就不需要任何中缀了呢 比较大多数情况下 不需要啥工具函数在里面

@condorheroblog
Copy link
Owner Author

image

对,目前是这样的,听你这么讲,infixName 加上 boolean 类型会比较好😁

@xiaoxian521
Copy link

哈哈 还真是
其实infixname也相当于 添加中配置忽略此文件

@condorheroblog
Copy link
Owner Author

我先发布了一个 v2.0.0-beta.0 ,所有的 API 应该不会变了,应该也没有 bug,目前测试是这样😂。

里面有部分逻辑,我打算抽出来放到 https://github.com/condorheroblog/bundle-import 然后发布正式版本,如果我不忙的话,明天之前就能完成。

@xiaoxian521
Copy link

那正好我先用虚拟机搞个windows 来测试测试

@condorheroblog
Copy link
Owner Author

@xiaoxian521 v2.0.0 is released.

感谢你的帮助,终于完成了,和一个陌生人完成一件还不错的事情,想想就很振奋人心,这是一段愉快的旅程,再次感谢🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants