Fixed text formatting
@@ -94,23 +94,17 @@ Review changes in `%SOURCE_BRANCH%` using `.review/diff.patch` against the follo
|
||||
### Step 3: Build Review Structure
|
||||
|
||||
#### A. Summary Comment (`body` parameter)
|
||||
```
|
||||
[⛔|⚠️|✅] Performance assessment: [Critical Issues Found|Warnings Found|No Issues Found]
|
||||
|
||||
ℹ️ Conclusion: [Your performance analysis summary here]
|
||||
|
||||
[✅|❌] Merge: [Yes/No with brief reason]
|
||||
```
|
||||
|
||||
#### B. Inline Comments (`comments` parameter)
|
||||
```
|
||||
[⛔|⚠️|ℹ️] [Issue|Suggestion|Note] ([Blocking|Non Blocking]): [Description]
|
||||
|
||||
🤖 Suggested Code (include when you have a clear optimization):
|
||||
```language
|
||||
[optimized code]
|
||||
```
|
||||
```
|
||||
|
||||
**Performance comment rules:**
|
||||
- Critical bottlenecks (e.g., N+1 queries, memory leaks) are `blocking`
|
||||
|
||||
@@ -93,23 +93,17 @@ Review changes in `%SOURCE_BRANCH%` using `.review/diff.patch` against the follo
|
||||
### Step 3: Build Review Structure
|
||||
|
||||
#### A. Summary Comment (`body` parameter)
|
||||
```
|
||||
[⛔|⚠️|✅] Security assessment: [Critical Issues Found|Warnings Found|No Issues Found]
|
||||
|
||||
ℹ️ Conclusion: [Your security analysis summary here]
|
||||
|
||||
[✅|❌] Merge: [Yes/No with brief reason]
|
||||
```
|
||||
|
||||
#### B. Inline Comments (`comments` parameter)
|
||||
```
|
||||
[⛔|⚠️|ℹ️] [Issue|Suggestion|Note] ([Blocking|Non Blocking|Security]): [Description]
|
||||
|
||||
🤖 Suggested Code (include when you have a clear fix):
|
||||
```language
|
||||
[secure code fix]
|
||||
```
|
||||
```
|
||||
|
||||
**Security comment rules:**
|
||||
- All security vulnerabilities MUST use the `security` decoration
|
||||
|
||||
@@ -75,28 +75,23 @@ Review changes in `%SOURCE_BRANCH%` using `.review/diff.patch` against:
|
||||
### Step 3: Build Review Structure
|
||||
|
||||
#### A. Summary Comment (`body` parameter)
|
||||
```
|
||||
[⛔|⚠️|✅] Documentation quality assessment: [Poor|Acceptable|Excellent]
|
||||
|
||||
ℹ️ Conclusion: [Your analysis summary here]
|
||||
|
||||
[✅|❌] Merge: [Yes/No with brief reason]
|
||||
```
|
||||
|
||||
#### B. Inline Comments (`comments` parameter)
|
||||
```
|
||||
[⛔|⚠️|ℹ️] [Suggestion|Issue|TODO|Typo|Note|Polish] ([Non Blocking|Blocking]): [Description]
|
||||
|
||||
🤖 Suggested Fix (include when you have a clear correction):
|
||||
[corrected text]
|
||||
```
|
||||
|
||||
### Step 4: Submit Review
|
||||
Call `create_pull_request_review` with owner, repo, index, body, comments, and state.
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Allowed Tools (`cc_allowed_tool` / `ALLOWED_TOOLS.md`)
|
||||
|
||||
```
|
||||
|
||||
Reference in New Issue
Block a user