Skip to content

fix: improve local dev onboarding experience#49

Open
jwcrystal wants to merge 3 commits intoqiuzhi2046:mainfrom
jwcrystal:dev-onboarding-experience
Open

fix: improve local dev onboarding experience#49
jwcrystal wants to merge 3 commits intoqiuzhi2046:mainfrom
jwcrystal:dev-onboarding-experience

Conversation

@jwcrystal
Copy link
Copy Markdown

Summary

  • add a dev-only bypass action for Electron/Vite sessions to skip blocking env detection
  • fix invalid list markup in the welcome risk notice
  • suppress upstream model fallback console logs outside dev mode

Test Plan

  • npm test -- src/pages/tests/env-check-state.test.ts src/pages/tests/welcome.test.tsx src/shared/tests/upstream-model-state.test.ts
  • npm run typecheck

@jwcrystal jwcrystal marked this pull request as ready for review April 1, 2026 03:06
Copy link
Copy Markdown
Owner

@qiuzhi2046 qiuzhi2046 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

感谢这次提交,改动目标比较聚焦,但这里引入了一个需要处理的回归风险。

[P2] src/pages/EnvCheck.tsx 新增的“开发调试:跳过环境检测”按钮目前只是直接调用 onReady(buildDevBypassReadyPayload()) 切走流程,但没有看到与之配套的运行中断或取消保护;而 runChecks() 会在页面挂载后自动启动并继续执行后续异步检查/安装链路。我担心这会导致用户点击“跳过”后,后台环境检测甚至安装流程仍可能继续运行。这里的问题在于“跳过”只改变了前端路由状态,没有充分保证正在进行的环境处理被停止,可能带来的影响是开发态下出现意外的检查/安装副作用,导致行为与用户预期不一致,也会让现有环境检测流程出现退化。建议至少补上可靠的取消/中断保护,并增加一个覆盖“runChecks 已启动后触发 bypass”场景的回归测试。

建议修改后再提交,感谢铁铁为Qclaw做出的贡献,欢迎继续提交PR🤗

@jwcrystal
Copy link
Copy Markdown
Author

感谢这次提交,改动目标比较聚焦,但这里引入了一个需要处理的回归风险。

[P2] src/pages/EnvCheck.tsx 新增的“开发调试:跳过环境检测”按钮目前只是直接调用 onReady(buildDevBypassReadyPayload()) 切走流程,但没有看到与之配套的运行中断或取消保护;而 runChecks() 会在页面挂载后自动启动并继续执行后续异步检查/安装链路。我担心这会导致用户点击“跳过”后,后台环境检测甚至安装流程仍可能继续运行。这里的问题在于“跳过”只改变了前端路由状态,没有充分保证正在进行的环境处理被停止,可能带来的影响是开发态下出现意外的检查/安装副作用,导致行为与用户预期不一致,也会让现有环境检测流程出现退化。建议至少补上可靠的取消/中断保护,并增加一个覆盖“runChecks 已启动后触发 bypass”场景的回归测试。

建议修改后再提交,感谢铁铁为Qclaw做出的贡献,欢迎继续提交PR🤗

這個只限於 dev mode, release 不會有這個跳過檢查機制

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants