Skip to content

chore(desktop): prettier 格式化 9 文件,修复 CI 启用后暴露的既有格式债#31

Merged
XZZKANY merged 2 commits into
masterfrom
chore/prettier-format-ci-green
Jun 29, 2026
Merged

chore(desktop): prettier 格式化 9 文件,修复 CI 启用后暴露的既有格式债#31
XZZKANY merged 2 commits into
masterfrom
chore/prettier-format-ci-green

Conversation

@XZZKANY

@XZZKANY XZZKANY commented Jun 29, 2026

Copy link
Copy Markdown
Owner

背景

PR #30 启用 push/PR CI 后,首个 master CI run 在第一道 pnpm run lint 失败:prettier --check 报 9 个文件未格式化。这是 #27 Wave1/2 重构 + hidden-agent-jobs 合并引入的既有格式债,因旧 CI 仅 workflow_dispatch 未被发现(历史 master run 亦全 red)。新启用的 CI 正确抓出。

改动

对 9 个受影响文件跑 prettier --write(纯格式化,无逻辑变更)。

验证

  • pnpm run lint → exit 0(prettier 全绿)
  • npm --prefix apps/desktop/frontend run typecheck → exit 0
  • pnpm openapi → 无契约漂移

本 PR 的 CI run 将实测全量门禁(含 API/workflow pytest)。

🤖 Generated with Claude Code

XZZKANY and others added 2 commits June 29, 2026 22:44
PR #30 启用 push/PR CI 后首个 master run 在 prettier --check 失败:#27/
hidden-agent-jobs 合并引入 9 个未格式化文件,因旧 CI 仅 workflow_dispatch 未被
发现。对这些文件跑 prettier --write,纯格式化无逻辑变更。

验证:pnpm run lint exit 0;desktop typecheck exit 0;pnpm openapi 无漂移。

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
按需求去除 CI:删除 ci.yml(Core verification)与 e2e.yml(每周安全网),
不再以 GitHub Actions 作为 PR 门禁。本地验证仍走 pnpm verify / pnpm test。

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@XZZKANY
XZZKANY merged commit afece2c into master Jun 29, 2026
@XZZKANY
XZZKANY deleted the chore/prettier-format-ci-green branch June 29, 2026 16:21
XZZKANY added a commit that referenced this pull request Jun 29, 2026
CI 于 2026-06-30 整体移除(PR #31 squash 含删 ci.yml/e2e.yml),lint 与 OpenAPI
漂移两道快门禁失去自动拦截。新增本地 pre-push hook 补回:

- .githooks/pre-push 跑 pnpm run verify:fast
- scripts/check-openapi-drift.mjs:记录 digest→pnpm openapi→比对,漂移即 exit 1
- package.json:verify:fast / check:drift / hooks:install(set core.hooksPath)

一次性 pnpm hooks:install 启用,绕过用 git push --no-verify。仅覆盖 lint+漂移两道快门禁
(不含已炸的 desktop typecheck 与慢 pytest),仍须 pnpm verify 手动全量。
next-step-plan.md 同步顶部更新横幅 + CI/阶段0/Q1 相关条目。

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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.

1 participant