Add switch-main command and improve pr and cr command workflows. (#3211)

Author: domchen

.codebuddy/commands/cr.md

@@ -30,7 +30,7 @@ gh auth status
 ls -d /tmp/pr-review-* 2>/dev/null
 ```
 
-仅当上述命令输出了目录列表时，才执行清理：
+仅当上述命令输出了目录列表时，才执行清理（无输出则跳过，不打印任何信息）：
 
 ```bash
 for dir in /tmp/pr-review-*; do
@@ -116,17 +116,17 @@ gh pr view {pr_number} --comments
 - 问题列表：按序号列出真正存在的问题，每个问题包含位置、描述、修复建议
 - 若无问题则输出"无问题"
 
-**输出禁令**（严格遵守）：
-- **禁止输出任何已排除的问题**，包括禁止以"排除"、"不是问题"、"二次验证后确认正确"等形式提及
-- **禁止输出分析推理过程**，只输出最终结论
+**!! IMPORTANT - 输出禁令**：
+- 禁止输出任何已排除的问题，包括禁止以"排除"、"不是问题"、"二次验证后确认正确"等形式提及
+- 禁止输出分析推理过程，只输出最终结论
 
 ---
 
 ## 第三步：处理问题
 
 **若无问题**：
 - 本地模式 / 在线模式：流程结束
-- Worktree 模式：执行清理后流程结束
+- Worktree 模式：跳至第四步清理
 
 **若有问题**：按以下流程处理。
 
@@ -135,16 +135,20 @@ gh pr view {pr_number} --comments
 | 模式 | 自己的代码 | 别人的代码 |
 |------|----------|----------|
 | 本地模式 | 询问修复 | - |
-| Worktree 模式 | 优化标题 + 询问修复 + 推送 + 清理 | 优化标题 + 询问评论 + 清理 |
+| Worktree 模式 | 优化标题 + 询问修复 + 推送 | 优化标题 + 询问评论 + 提交评论 |
 | 在线模式 | 优化标题 + 询问评论 | 优化标题 + 询问评论 |
 
+> **!! CRITICAL - 别人的代码处理规则**：
+> - **NEVER** 自动修复并推送，只能通过 PR 评论提建议
+> - 用户回复问题序号时默认提交评论，除非明确要求"修复并推送"
+
 ### 优化标题
 
 如有必要，用 `gh pr edit {pr_number} --title "新标题"` 更新，并输出优化后的标题。
 
 格式：根据变更内容总结 120 字符内的英语概括，以英文句号结尾，中间无其他标点，侧重描述用户可感知的变化。
 
-### 询问修复
+### 询问修复（仅限自己的代码）
 
 询问用户需要修复哪些序号的问题，然后逐一修复。
 
@@ -156,9 +160,9 @@ git add . && git commit -m "{根据修复内容生成}"
 git push origin HEAD:$(gh pr view {pr_number} --json headRefName -q '.headRefName')
 ```
 
-### 询问评论
+### 询问评论（别人的代码必须走此流程）
 
-询问用户是否需要将问题作为 PR 行级评论提交。
+询问用户是否需要将问题作为 PR 行级评论提交，以及需要提交哪些序号的问题。用户确认后执行评论提交。
 
 - 使用简洁的中文描述问题和建议
 - **必须**使用 `gh api` + heredoc，**禁止**使用 `gh pr comment`、`gh pr review` 等或任何非行级评论命令
@@ -169,11 +173,13 @@ gh api repos/{owner}/{repo}/pulls/{pr_number}/reviews --input - <<'EOF'
 EOF
 ```
 
-### 清理（仅 Worktree 模式）
+---
+
+## 第四步：清理（仅 Worktree 模式）
 
-若当前分支就是 PR 分支（未创建 worktree），则跳过清理步骤。
+若当前分支就是 PR 分支（未创建 worktree），则跳过此步骤。
 
-若创建了 worktree，自动清理临时环境并输出提示：
+若创建了 worktree，在第三步所有操作完成后，**直接执行清理，无需确认**：
 
 ```bash
 cd - # 返回原目录

.codebuddy/commands/pr.md

@@ -1,77 +1,119 @@
 ---
-description: 发起 PR - 格式化代码并提交 PR
+description: 提交 PR - 自动识别新建或追加提交
 ---
 
-# 发起 PR 流程
+# 提交 PR
 
-严格按以下步骤顺序执行：
+严格按以下步骤顺序执行。
 
 ---
 
-## 步骤 1：确定提交范围
+## 前置检查
+
+### 检查是否已有开启的 PR
 
 ```bash
-git diff --cached --stat
-git diff --stat
+git branch --show-current
+gh pr list --head "$(git branch --show-current)" --state open --json number,url
 ```
 
-根据暂存区和工作区状态判断：
+根据结果选择模式：
+
+| 当前分支 | 已有开启的 PR | 模式 |
+|----------|---------------|------|
+| main | - | **新建模式** |
+| 非 main | 有 | **追加模式** |
+| 非 main | 无 | **新建模式** |
+
+---
 
-| 暂存区 | 工作区 | 处理方式 | 是否需要确认 |
-|--------|--------|----------|--------------|
-| 有内容 | 有内容 | 询问用户：只提交暂存区内容，还是提交全部变更 | **需要确认** |
-| 有内容 | 无内容 | 直接使用暂存区内容 | 无需确认 |
-| 无内容 | 有内容 | 使用 `git add .` 添加全部变更 | 无需确认 |
-| 无内容 | 无内容 | 提示无变更可提交，终止流程 | - |
+## 第一步：确定提交范围
 
-记录本步骤确定的提交范围（全部变更 / 仅暂存区）和暂存区涉及的文件列表。
+运行 `git status --porcelain`：
+- 若无输出：提示无变更，终止流程
+- 若暂存区有内容，且工作区也有内容（含未跟踪文件）：询问用户选择局部提交或全部提交
+- 否则：全部提交（执行 `git add .`）
+
+**判断方法**：第一列非空格为暂存区有内容；第二列非空格或以 `??` 开头为工作区有内容。
+
+若为**局部提交**，记录**暂存区文件列表**供后续步骤使用。
 
 ---
 
-## 步骤 2：格式化代码
+## 第二步：格式化代码
 
 ```bash
 ./codeformat.sh
 ```
 
 忽略输出的报错信息，只要运行就会完成格式化。
 
-格式化后处理：
-- 如果步骤 1 确定提交**全部变更**：执行 `git add .`
-- 如果步骤 1 确定只提交**暂存区内容**：检查格式化产生的变更，将步骤 1 记录的暂存区文件相关的格式化修改加入暂存区（`git add {文件}`）
+格式化后根据第一步确定的提交范围处理：
+
+| 提交范围 | 处理方式 |
+|----------|----------|
+| 全部提交 | 执行 `git add .` 将所有变更（包括格式化修改）加入暂存区 |
+| 局部提交 | 检查第一步记录的暂存区文件是否被格式化修改，如有则重新加入暂存区（`git add {文件}`），忽略其他文件的格式化修改 |
 
 ---
 
-## 步骤 3：生成 PR 信息并创建 PR
+## 第三步：生成提交信息
+
+查看将要提交的内容：
+
+```bash
+git diff --cached
+```
+
+根据变更内容生成**Commit 信息**（英语，120 字符内，以句号结尾，侧重描述用户可感知的变化）。
+
+---
 
-用 `git status`、`git diff --cached` 查看将要提交的内容，生成以下信息：
+## 第四步：提交并推送
+
+**直接执行，无需确认。**
+
+### 追加模式
+
+```bash
+git commit -m "{Commit 信息}"
+git push
+```
+
+输出：
+
+```
+**Commit**：{Commit 信息}
+
+**PR 链接**：{PR 链接}（已追加提交）
+```
+
+### 新建模式
+
+生成以下信息：
 
 - **分支名称**：`feature/{username}_模块名` 或 `bugfix/{username}_模块名`（`{username}` 为 GitHub 用户 ID 全小写，模块名用下划线连接，最多两个单词）
-- **PR 标题**：英语，120 字符内，以句号结尾，侧重描述用户可感知的变化
 - **PR 描述**：中文，简要说明变更内容和目的
 
-**直接执行以下命令创建 PR，无需等待用户确认：**
-
 ```bash
 git checkout -b {分支名称}
-git commit -m "{PR标题}"
+git commit -m "{Commit 信息}"
 git push -u origin {分支名称}
-gh pr create --title "{PR标题}" --body "{PR描述}"
+gh pr create --title "{Commit 信息}" --body "{PR 描述}"
 ```
 
-完成后输出以下信息：
+输出：
 
 ```
-**PR 标题**：{PR标题}
+**PR 标题**：{Commit 信息}
 
-**PR 描述**：{PR描述}
+**PR 描述**：{PR 描述}
 
-**PR 链接**：{PR链接}
+**PR 链接**：{PR 链接}（新建）
 ```
 
 ---
 
-## 重要限制
+## **!! IMPORTANT - 重要限制**
 
-- **禁止**在 main 分支直接提交推送代码
-- **禁止** force push
+- **NEVER** 执行 force push

.codebuddy/commands/switch-main.md

@@ -0,0 +1,27 @@
+---
+description: 切换到主分支 - 将本地未提交的变更转移到更新后的 main 分支，并清理已合并的旧分支
+---
+
+# 切换到主分支
+
+将本地未提交的变更转移到更新后的 main 分支，并清理已合并的旧分支。
+
+## 执行流程
+
+1. 记录当前分支名称，检查是否有本地变更
+2. 如有变更，执行 `git stash push --include-untracked -m "switch-main"`
+3. 切换到 main 并更新：`git checkout main && git pull --rebase origin main`
+4. 如有 stash，执行 `git stash pop --index` 还原变更
+5. 如原分支有已合并的 PR（`gh pr list --head "{分支}" --state merged`），删除本地分支
+
+## 冲突处理
+
+如果 `git stash pop` 遇到冲突：
+
+1. 列出冲突文件
+2. 阅读每个冲突文件的代码，分析冲突原因
+3. 提供自动修复方案（说明如何解决每个冲突）
+4. 询问用户是否执行自动修复
+5. 用户确认后执行修复，然后 `git stash drop`
+
+**注意**：解决后不修改暂存区状态，保持还原后的原始状态。

.codebuddy/rules/Git.md

@@ -3,12 +3,11 @@ description: Git 操作规范
 alwaysApply: true
 ---
 
-## 操作限制
+## **!! IMPORTANT - 操作限制**
 
-- 禁止自动执行 `git stash`、`git reset`、`git checkout` 等改变暂存区或工作区状态的命令，除非用户明确要求提交代码
+- **NEVER** 自动执行 `git stash`、`git reset`、`git checkout` 等改变暂存区或工作区状态的命令，除非用户明确要求提交代码
 - 必须 stash 时需用户确认，完成后用 `git stash pop --index` 完全还原原始的暂存区和工作区状态
-- 禁止在 main 分支直接提交推送代码，必须通过 PR 流程
-- 禁止 force push，补充 PR 的提交使用正常追加
+- **NEVER** 在 main 分支直接提交推送代码，必须通过 PR 流程
 
 ## 分支命名
 

.codebuddy/rules/Build.md .codebuddy/rules/Test.md.codebuddy/rules/Build.md renamed to .codebuddy/rules/Test.md

@@ -25,4 +25,7 @@ cmake --build cmake-build-debug --target PAGFullTest
 - 使用 `Baseline::Compare(pixels, key)` 比较截图，key 格式为 `{folder}/{name}`，例如 `PAGSurfaceTest/Mask`
 - 截图输出到 `test/out/{folder}/{name}.webp`，基准图为同目录下 `{name}_base.webp`
 - 比较机制：对比 `test/baseline/version.json`（仓库）与 `test/baseline/.cache/version.json`（本地）中同一 key 的版本号，一致时才进行基准图对比，不一致则跳过返回成功，以此接受截图变更
-- **禁止任何形式的自动接受截图基准变更**，包括禁止自动运行 `UpdateBaseline` target、禁止修改或覆盖 `version.json` 文件，必须经过用户确认后运行 `accept_baseline.sh` 脚本来接受变更
+
+**!! IMPORTANT - 截图基准变更限制**：
+- **NEVER** 自动接受截图基准变更，包括禁止自动运行 `UpdateBaseline` target、禁止修改或覆盖 `version.json` 文件
+- 必须经过用户确认后运行 `accept_baseline.sh` 脚本来接受变更

src/rendering/filters/BrightnessContrastFilter.cpp

@@ -72,7 +72,8 @@ static const char FRAGMENT_SHADER[] = R"(
 
 std::shared_ptr<tgfx::Image> BrightnessContrastFilter::Apply(std::shared_ptr<tgfx::Image> input,
                                                              RenderCache* cache, Effect* effect,
-                                                             Frame layerFrame, tgfx::Point* offset) {
+                                                             Frame layerFrame,
+                                                             tgfx::Point* offset) {
   auto* brightnessContrastEffect = static_cast<const BrightnessContrastEffect*>(effect);
   auto brightness = brightnessContrastEffect->brightness->getValueAt(layerFrame);
   auto contrast = brightnessContrastEffect->contrast->getValueAt(layerFrame);

src/rendering/filters/DisplacementMapFilter.cpp

@@ -253,9 +253,9 @@ static void Matrix3x3ToMat4(const std::array<float, 9>& m3, float* m4) {
   m4[15] = 1.0f;
 }
 
-void DisplacementMapFilter::onUpdateUniforms(
-    tgfx::RenderPass* renderPass, tgfx::GPU* gpu, const std::vector<std::shared_ptr<tgfx::Texture>>&,
-    const tgfx::Point&) const {
+void DisplacementMapFilter::onUpdateUniforms(tgfx::RenderPass* renderPass, tgfx::GPU* gpu,
+                                             const std::vector<std::shared_ptr<tgfx::Texture>>&,
+                                             const tgfx::Point&) const {
 
   DisplacementMapUniforms uniforms = {};
 
@@ -354,7 +354,6 @@ void DisplacementMapFilter::onUpdateUniforms(
       renderPass->setUniformBuffer(0, uniformBuffer, 0, sizeof(DisplacementMapUniforms));
     }
   }
-
 }
 
 tgfx::Rect DisplacementMapFilter::filterBounds(const tgfx::Rect& srcRect) const {

src/rendering/filters/HueSaturationFilter.cpp

@@ -95,9 +95,9 @@ std::shared_ptr<tgfx::Image> HueSaturationFilter::Apply(std::shared_ptr<tgfx::Im
   auto colorizeHue = hueSaturationEffect->colorizeHue->getValueAt(layerFrame);
   auto colorizeSaturation = hueSaturationEffect->colorizeSaturation->getValueAt(layerFrame);
   auto colorizeLightness = hueSaturationEffect->colorizeLightness->getValueAt(layerFrame);
-  auto filter = std::make_shared<HueSaturationFilter>(cache, hue, saturation, lightness, colorize,
-                                                      colorizeHue, colorizeSaturation,
-                                                      colorizeLightness);
+  auto filter =
+      std::make_shared<HueSaturationFilter>(cache, hue, saturation, lightness, colorize,
+                                            colorizeHue, colorizeSaturation, colorizeLightness);
   return input->makeWithFilter(tgfx::ImageFilter::Runtime(filter), offset);
 }
 

src/rendering/filters/MosaicFilter.cpp

@@ -48,7 +48,8 @@ std::shared_ptr<tgfx::Image> MosaicFilter::Apply(std::shared_ptr<tgfx::Image> in
   auto horizontalBlocks = 1.0f / mosaicEffect->horizontalBlocks->getValueAt(layerFrame);
   auto verticalBlocks = 1.0f / mosaicEffect->verticalBlocks->getValueAt(layerFrame);
 
-  auto filter = std::make_shared<MosaicFilter>(cache, horizontalBlocks, verticalBlocks, sharpColors);
+  auto filter =
+      std::make_shared<MosaicFilter>(cache, horizontalBlocks, verticalBlocks, sharpColors);
 
   return input->makeWithFilter(tgfx::ImageFilter::Runtime(filter), offset);
 }

src/rendering/filters/glow/GlowFilter.cpp

@@ -26,8 +26,8 @@
 namespace pag {
 
 std::shared_ptr<tgfx::Image> GlowFilter::Apply(std::shared_ptr<tgfx::Image> input,
-                                               RenderCache* cache, Effect* effect,
-                                               Frame layerFrame, tgfx::Point* offset) {
+                                               RenderCache* cache, Effect* effect, Frame layerFrame,
+                                               tgfx::Point* offset) {
   auto glowEffect = static_cast<GlowEffect*>(effect);
   auto glowRadius = glowEffect->glowRadius->getValueAt(layerFrame);
   auto resizeRatio = 1.0f - glowRadius / 1500.f;
@@ -37,13 +37,13 @@ std::shared_ptr<tgfx::Image> GlowFilter::Apply(std::shared_ptr<tgfx::Image> inpu
   auto blurFilterH = std::make_shared<GlowBlurRuntimeFilter>(cache, BlurDirection::Horizontal,
                                                              1.0f / blurWidth, resizeRatio);
   auto imageFilterH = tgfx::ImageFilter::Runtime(blurFilterH);
-  auto blurFilterV =
-      std::make_shared<GlowBlurRuntimeFilter>(cache, BlurDirection::Vertical, 1.0f / blurHeight, 1.0f);
+  auto blurFilterV = std::make_shared<GlowBlurRuntimeFilter>(cache, BlurDirection::Vertical,
+                                                             1.0f / blurHeight, 1.0f);
   auto imageFilterV = tgfx::ImageFilter::Runtime(blurFilterV);
   auto blurFilter = tgfx::ImageFilter::Compose(imageFilterH, imageFilterV);
   auto blurImage = input->makeWithFilter(std::move(blurFilter));
-  auto mergeFilter =
-      tgfx::ImageFilter::Runtime(std::make_shared<GlowMergeRuntimeFilter>(cache, progress, blurImage));
+  auto mergeFilter = tgfx::ImageFilter::Runtime(
+      std::make_shared<GlowMergeRuntimeFilter>(cache, progress, blurImage));
   return input->makeWithFilter(std::move(mergeFilter), offset);
 }
 }  // namespace pag