Merge pull request #4223 from anyproto/ios-5420-restructure-and-update-ai-flow-2

Fix verbose code review output for clean approvals

Author: ignatovv

.claude/commands/cpp.md

@@ -27,18 +27,19 @@ Commits the current changes, performs a code review, applies fixes if needed, th
 - Apply CODE_REVIEW_GUIDE.md standards
 - Check for bugs, best practices violations, performance issues, security concerns
 
-### 3. Review Findings
+### 3. Review Findings and Auto-Proceed
 - Present review results to developer
 - If issues found, **STOP and discuss** with developer:
   - Should we fix the issues now?
   - Are the findings valid or false positives?
   - Should we proceed anyway?
 - **Developer decides next steps** - never auto-amend commits
+- **If review is approved (✅)**: Automatically proceed to push and PR without asking
 
-### 4. Push and PR (when approved)
-- Only proceed when developer approves
+### 4. Push and PR (auto-proceed when approved)
 - Push to remote with tracking
 - Create pull request with summary
+- No confirmation needed when code review is approved
 
 ## Branch Handling
 When a branch name is provided:

.claude/settings.json

@@ -43,7 +43,9 @@
       "mcp__linear__list_issue_labels",
       "mcp__linear__get_team",
       "mcp__linear__search_documentation",
-      
+
+      "SlashCommand(/codeReview)",
+
       "Bash(gh pr list:*)",
       "Bash(gh pr diff:*)",
       "Bash(git fetch:*)",

.claude/skills/code-review-developer/SKILL.md

@@ -43,6 +43,8 @@ Context-aware routing to code review guidelines. Helps you conduct thorough, act
 ✅ **Approved** - No issues found
 ```
 
+**CRITICAL**: When approving, output ONLY the line above. NO additional explanation, NO listing what the PR does, NO praise. Just the approval line.
+
 **If issues found**:
 ```
 ## Bugs/Issues
@@ -218,7 +220,21 @@ For GitHub Actions integration:
 ✅ **Approved** - No issues found
 ```
 
-**That's it! Don't add explanations, don't list changes.**
+**That's it! Absolutely nothing else. Not even in comments posted to GitHub.**
+
+**❌ WRONG** (too verbose):
+```
+✅ **Approved** - No issues found
+
+The PR correctly implements per-chat notification overrides:
+- Added force list properties with proper subscription keys
+- effectiveNotificationMode(for:) method correctly prioritizes...
+```
+
+**✅ CORRECT**:
+```
+✅ **Approved** - No issues found
+```
 
 ### Example 2: Minor Issues
 ```

.github/workflows/pr-review-automation.md

@@ -5,12 +5,22 @@
 - **PR NUMBER**: Pull request number (injected by workflow)
 - **COMMIT SHA**: Commit SHA being reviewed (injected by workflow)
 
-## Review Standards
+## 🚨 CRITICAL OUTPUT RULES
 
-**Apply the shared review guidelines from:**
-`.github/workflows/code-review-guidelines.md`
+**If NO issues found:**
+```
+✅ **Approved** - No issues found
+```
+
+**DO NOT add ANY additional text:**
+- ❌ NO "The PR correctly implements..."
+- ❌ NO explanations of what changed
+- ❌ NO lists of what was added/cleaned
+- ❌ NO implementation details
+- ❌ ONLY the one-line approval message above
 
-Follow all core rules, review sections, common mistakes, and analysis checklist defined in that file.
+**If issues ARE found:**
+Follow the format from `.claude/skills/code-review-developer/SKILL.md`
 
 ## CI-Specific Reference Documentation
 

CLAUDE.md

@@ -38,6 +38,7 @@ make setup-middle    # Initial setup
 
 ### AI Assistance
 - **Always present a detailed action plan before implementing multi-step changes and await approval before proceeding**
+- **When code review is approved**: Proceed directly with push and PR creation without asking for confirmation
 
 ### 📚 Skills System & Documentation (Progressive Disclosure)