Skip to content

style: prettier 统一格式并简化 needLogin#178

Open
Enigma-Soul wants to merge 1 commit into
yltx:masterfrom
Enigma-Soul:style/prettier-line-endings
Open

style: prettier 统一格式并简化 needLogin#178
Enigma-Soul wants to merge 1 commit into
yltx:masterfrom
Enigma-Soul:style/prettier-line-endings

Conversation

@Enigma-Soul

Copy link
Copy Markdown
Contributor

背景

整改自 #176(按维护者反馈拆分)。本 PR 为纯格式化,配合 #177(核心修复)。

改动

  • prettier 统一代码格式(换行重排)
  • 简化 needLogin:用 Promise.resolve + finally 取代两个重复的 then 回调(逻辑等价)

说明

Test plan

  • npm run typecheck
  • npm run test(52/52)
  • npm run prettier --check(幂等)

- prettier 统一代码格式(换行重排)
- 简化 needLogin:用 Promise.resolve + finally 取代两个重复的 then 回调

纯格式与等价重构,无逻辑变更。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@yltx yltx left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

严格审查后暂不建议合并,需整改以下问题:

  1. Major:needLogin 不是等价重构
    src/utils/uiUtils.ts.then(success, failure) 改为 .finally()。旧实现会消费 showErrorMessage 或前一回调产生的 rejection;新实现执行清理后会继续传播 rejection,但该 Promise 没有返回或 catch,可能形成未处理 Promise rejection。该行为变更不应混在“纯格式化”PR 中。

  2. Major:登录提示去重行为没有测试
    需要覆盖提示成功、提示被拒绝、点击登录、5 秒内去重以及 5 秒后恢复。当前 52 个测试没有覆盖 needLogin,不能证明逻辑等价。

  3. Scope:PR 描述不准确
    除 Prettier 外包含实际 Promise 错误语义变化。建议从本 PR 移除 needLogin 重构,只保留格式化;若确实要修改,请独立提交并明确行为、捕获 rejection、补充测试。

  4. Validation:仓库 CI 不完整
    GitHub 当前只有 GitGuardian 检查。Windows checkout 下 npm run lint 会因全仓库 CRLF 报数千个 Prettier 错误;typecheck 与 52 个测试通过。建议先明确 .gitattributes/Prettier 行尾策略,并增加正式 lint/typecheck/test 检查。

完成以上整改后再重新审查。建议在 #177 完成并合并后 rebase 本 PR,避免格式化影响核心修复审查。

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