@in-the-loop-labs/pair-review 3.0.5 → 3.1.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/package.json +2 -1
- package/plugin/.claude-plugin/plugin.json +1 -1
- package/plugin-code-critic/.claude-plugin/plugin.json +1 -1
- package/plugin-code-critic/skills/analyze/references/level1-balanced.md +8 -0
- package/plugin-code-critic/skills/analyze/references/level1-fast.md +7 -0
- package/plugin-code-critic/skills/analyze/references/level1-thorough.md +8 -0
- package/plugin-code-critic/skills/analyze/references/level2-balanced.md +9 -0
- package/plugin-code-critic/skills/analyze/references/level2-fast.md +8 -0
- package/plugin-code-critic/skills/analyze/references/level2-thorough.md +9 -0
- package/plugin-code-critic/skills/analyze/references/level3-balanced.md +9 -0
- package/plugin-code-critic/skills/analyze/references/level3-fast.md +8 -0
- package/plugin-code-critic/skills/analyze/references/level3-thorough.md +9 -0
- package/plugin-code-critic/skills/analyze/references/orchestration-balanced.md +9 -0
- package/plugin-code-critic/skills/analyze/references/orchestration-fast.md +5 -0
- package/plugin-code-critic/skills/analyze/references/orchestration-thorough.md +9 -0
- package/public/css/analysis-config.css +83 -0
- package/public/css/pr.css +191 -4
- package/public/index.html +20 -0
- package/public/js/components/AIPanel.js +1 -1
- package/public/js/components/AdvancedConfigTab.js +83 -8
- package/public/js/components/AnalysisConfigModal.js +155 -5
- package/public/js/components/ChatPanel.js +22 -5
- package/public/js/components/CouncilProgressModal.js +239 -22
- package/public/js/components/TimeoutSelect.js +2 -0
- package/public/js/components/VoiceCentricConfigTab.js +179 -12
- package/public/js/index.js +119 -1
- package/public/js/local.js +141 -47
- package/public/js/modules/suggestion-manager.js +2 -1
- package/public/js/pr.js +71 -12
- package/public/js/repo-settings.js +2 -2
- package/public/local.html +32 -11
- package/public/pr.html +2 -0
- package/src/ai/analyzer.js +371 -111
- package/src/ai/claude-provider.js +2 -0
- package/src/ai/codex-provider.js +1 -1
- package/src/ai/copilot-provider.js +2 -0
- package/src/ai/executable-provider.js +534 -0
- package/src/ai/gemini-provider.js +2 -0
- package/src/ai/index.js +9 -1
- package/src/ai/pi-provider.js +10 -8
- package/src/ai/prompts/baseline/consolidation/balanced.js +54 -2
- package/src/ai/prompts/baseline/consolidation/fast.js +31 -1
- package/src/ai/prompts/baseline/consolidation/thorough.js +46 -3
- package/src/ai/prompts/baseline/level1/balanced.js +12 -0
- package/src/ai/prompts/baseline/level1/fast.js +11 -0
- package/src/ai/prompts/baseline/level1/thorough.js +12 -0
- package/src/ai/prompts/baseline/level2/balanced.js +13 -0
- package/src/ai/prompts/baseline/level2/fast.js +12 -0
- package/src/ai/prompts/baseline/level2/thorough.js +13 -0
- package/src/ai/prompts/baseline/level3/balanced.js +13 -0
- package/src/ai/prompts/baseline/level3/fast.js +12 -0
- package/src/ai/prompts/baseline/level3/thorough.js +13 -0
- package/src/ai/prompts/baseline/orchestration/balanced.js +15 -0
- package/src/ai/prompts/baseline/orchestration/fast.js +11 -0
- package/src/ai/prompts/baseline/orchestration/thorough.js +15 -0
- package/src/ai/prompts/render-for-skill.js +3 -0
- package/src/ai/prompts/shared/output-schema.js +8 -0
- package/src/ai/provider.js +89 -4
- package/src/chat/prompt-builder.js +17 -1
- package/src/chat/session-manager.js +32 -28
- package/src/config.js +15 -2
- package/src/database.js +59 -15
- package/src/git/base-branch.js +133 -52
- package/src/local-review.js +15 -9
- package/src/main.js +3 -2
- package/src/routes/analyses.js +34 -8
- package/src/routes/chat.js +15 -8
- package/src/routes/config.js +3 -120
- package/src/routes/councils.js +15 -6
- package/src/routes/executable-analysis.js +494 -0
- package/src/routes/local.js +160 -26
- package/src/routes/mcp.js +9 -4
- package/src/routes/pr.js +166 -29
- package/src/routes/reviews.js +31 -5
- package/src/routes/shared.js +72 -5
- package/src/routes/worktrees.js +4 -2
- package/src/utils/comment-formatter.js +28 -11
- package/src/utils/instructions.js +22 -8
- package/src/utils/logger.js +20 -10
|
@@ -22,6 +22,7 @@
|
|
|
22
22
|
* - {{reviewIntro}} - Review introduction line
|
|
23
23
|
* - {{lineNumberGuidance}} - Line number guidance section
|
|
24
24
|
* - {{customInstructions}} - Custom instructions section (optional)
|
|
25
|
+
* - {{dedupInstructions}} - Dedup instructions section (optional)
|
|
25
26
|
* - {{reviewerSuggestions}} - Formatted reviewer suggestions input
|
|
26
27
|
* - {{suggestionCount}} - Total number of input suggestions
|
|
27
28
|
* - {{reviewerCount}} - Number of reviewers being consolidated
|
|
@@ -51,6 +52,23 @@ Multiple independent AI reviewers have analyzed the same code changes. Your job
|
|
|
51
52
|
{{customInstructions}}
|
|
52
53
|
</section>
|
|
53
54
|
|
|
55
|
+
<section name="dedup-instructions" optional="true">
|
|
56
|
+
{{dedupInstructions}}
|
|
57
|
+
</section>
|
|
58
|
+
|
|
59
|
+
<section name="reviewer-context-guidance" required="true">
|
|
60
|
+
### Reviewer Context Awareness
|
|
61
|
+
Each reviewer below may have been configured with custom instructions. These fall into two categories:
|
|
62
|
+
|
|
63
|
+
- **Domain-focused reviewers**: Instructions that specify a code review focus area (e.g., "focus on security", "review error handling", "check performance"). Their findings *within that focus area* carry higher weight than generalist reviewers.
|
|
64
|
+
- **General reviewers**: Either no custom instructions, or instructions about methodology/style/persona (e.g., "be thorough", "use a friendly tone"). Treat their suggestions at face value across all categories.
|
|
65
|
+
|
|
66
|
+
**Weighting rules:**
|
|
67
|
+
- Only boost a reviewer's findings when their instructions indicate domain expertise relevant to the finding's category
|
|
68
|
+
- Cross-specialty findings from a domain-focused reviewer should be treated as general findings
|
|
69
|
+
- In conflicts within a domain, prefer the domain-focused reviewer's analysis over a generalist's
|
|
70
|
+
</section>
|
|
71
|
+
|
|
54
72
|
<section name="input-suggestions" locked="true">
|
|
55
73
|
## Input: {{reviewerCount}} Reviewer(s), {{suggestionCount}} Total Suggestions
|
|
56
74
|
|
|
@@ -78,15 +96,39 @@ Multiple independent AI reviewers have analyzed the same code changes. Your job
|
|
|
78
96
|
### 4. Quality Filter
|
|
79
97
|
- Drop suggestions with very low confidence (< 0.3) unless multiple reviewers agree
|
|
80
98
|
- Boost confidence when multiple reviewers independently identify the same issue
|
|
99
|
+
|
|
100
|
+
### 5. Severity Assessment
|
|
101
|
+
Assess severity based on the evidence and reasoning across all reviewers. When reviewers assign different severities, apply the same conflict resolution principles above. When truly uncertain, preserve the highest severity. Omit severity for praise items.
|
|
102
|
+
|
|
103
|
+
**Severity Definitions:**
|
|
104
|
+
- **critical**: Production incidents, system failures, or security vulnerabilities — runtime crashes, data corruption or loss, race conditions, deadlocks, breaking changes, changes that will cause existing tests to fail
|
|
105
|
+
- **medium**: Degraded functionality or reliability — missing error handling, N+1 queries, missing validation, missing or poor test coverage for new functionality
|
|
106
|
+
- **minor**: Code quality concerns — documentation gaps, minor optimizations, style inconsistencies
|
|
81
107
|
</section>
|
|
82
108
|
|
|
83
109
|
<section name="consensus-handling" required="true">
|
|
84
|
-
###
|
|
110
|
+
### 6. Consensus Handling
|
|
85
111
|
- **Agreement**: When multiple reviewers flag the same issue, increase confidence by 0.1-0.2 (cap at 1.0)
|
|
86
112
|
- **Partial overlap**: Merge related but distinct observations into a richer suggestion
|
|
87
113
|
- **Contradiction**: Use your judgment; prefer the more actionable analysis
|
|
88
114
|
</section>
|
|
89
115
|
|
|
116
|
+
<section name="balanced-output" required="true">
|
|
117
|
+
### 7. Balanced Output
|
|
118
|
+
- **Deduplicate, don't concatenate**: If two reviewers flagged the same issue, merge them into one suggestion. Distinct findings should each be preserved — the goal is to eliminate redundancy, not to reduce count.
|
|
119
|
+
- **Limit praise** to 2–3 most noteworthy items across all reviewers
|
|
120
|
+
- **Avoid suggestion overload** — aim for quality over quantity
|
|
121
|
+
- **Include confidence scores** reflecting cross-reviewer agreement
|
|
122
|
+
</section>
|
|
123
|
+
|
|
124
|
+
<section name="summary-synthesis" required="true">
|
|
125
|
+
## Summary Synthesis
|
|
126
|
+
The summary field should synthesize the findings, not list them.
|
|
127
|
+
- Synthesize the key findings into a single cohesive paragraph
|
|
128
|
+
- **Draw on reviewer summaries**: Use these as evidence for your own synthesis — integrate their insights into a cohesive conclusion rather than listing them individually
|
|
129
|
+
- Write as if a single reviewer produced this analysis — do not mention consolidation, merging, or multiple reviewers
|
|
130
|
+
</section>
|
|
131
|
+
|
|
90
132
|
<section name="output-schema" locked="true">
|
|
91
133
|
## Output Format
|
|
92
134
|
|
|
@@ -100,6 +142,7 @@ Output JSON with this structure:
|
|
|
100
142
|
"line": 42,
|
|
101
143
|
"old_or_new": "NEW",
|
|
102
144
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
145
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
103
146
|
"title": "Brief title",
|
|
104
147
|
"description": "Detailed explanation",
|
|
105
148
|
"suggestion": "How to fix/improve (omit for praise)",
|
|
@@ -110,13 +153,14 @@ Output JSON with this structure:
|
|
|
110
153
|
"fileLevelSuggestions": [{
|
|
111
154
|
"file": "path/to/file",
|
|
112
155
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
156
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
113
157
|
"title": "Brief title describing file-level concern",
|
|
114
158
|
"description": "Explanation of the file-level observation",
|
|
115
159
|
"suggestion": "How to address the file-level concern (omit for praise items)",
|
|
116
160
|
"confidence": 0.0-1.0,
|
|
117
161
|
"reasoning": ["Step-by-step reasoning explaining why this issue was flagged (optional)"]
|
|
118
162
|
}],
|
|
119
|
-
"summary": "
|
|
163
|
+
"summary": "Single cohesive paragraph summarizing key findings. Write as a single reviewer."
|
|
120
164
|
}
|
|
121
165
|
|
|
122
166
|
### GitHub Suggestion Syntax
|
|
@@ -155,9 +199,13 @@ const sections = [
|
|
|
155
199
|
{ name: 'critical-output', locked: true },
|
|
156
200
|
{ name: 'role-description', required: true },
|
|
157
201
|
{ name: 'custom-instructions', optional: true },
|
|
202
|
+
{ name: 'dedup-instructions', optional: true },
|
|
203
|
+
{ name: 'reviewer-context-guidance', required: true },
|
|
158
204
|
{ name: 'input-suggestions', locked: true },
|
|
159
205
|
{ name: 'consolidation-rules', required: true },
|
|
160
206
|
{ name: 'consensus-handling', required: true },
|
|
207
|
+
{ name: 'balanced-output', required: true },
|
|
208
|
+
{ name: 'summary-synthesis', required: true },
|
|
161
209
|
{ name: 'output-schema', locked: true },
|
|
162
210
|
{ name: 'diff-instructions', required: true },
|
|
163
211
|
{ name: 'guidelines', required: true }
|
|
@@ -173,9 +221,13 @@ const defaultOrder = [
|
|
|
173
221
|
'critical-output',
|
|
174
222
|
'role-description',
|
|
175
223
|
'custom-instructions',
|
|
224
|
+
'dedup-instructions',
|
|
225
|
+
'reviewer-context-guidance',
|
|
176
226
|
'input-suggestions',
|
|
177
227
|
'consolidation-rules',
|
|
178
228
|
'consensus-handling',
|
|
229
|
+
'balanced-output',
|
|
230
|
+
'summary-synthesis',
|
|
179
231
|
'output-schema',
|
|
180
232
|
'diff-instructions',
|
|
181
233
|
'guidelines'
|
|
@@ -23,6 +23,7 @@
|
|
|
23
23
|
* - {{reviewIntro}} - Review introduction line
|
|
24
24
|
* - {{lineNumberGuidance}} - Line number guidance section
|
|
25
25
|
* - {{customInstructions}} - Custom instructions section (optional)
|
|
26
|
+
* - {{dedupInstructions}} - Dedup instructions section (optional)
|
|
26
27
|
* - {{reviewerSuggestions}} - Formatted reviewer suggestions input
|
|
27
28
|
* - {{suggestionCount}} - Total number of input suggestions
|
|
28
29
|
* - {{reviewerCount}} - Number of reviewers being consolidated
|
|
@@ -52,6 +53,15 @@ Merge suggestions from multiple AI reviewers. Deduplicate. Resolve conflicts. Ke
|
|
|
52
53
|
{{customInstructions}}
|
|
53
54
|
</section>
|
|
54
55
|
|
|
56
|
+
<section name="dedup-instructions" optional="true">
|
|
57
|
+
{{dedupInstructions}}
|
|
58
|
+
</section>
|
|
59
|
+
|
|
60
|
+
<section name="reviewer-context-guidance" required="true" tier="fast">
|
|
61
|
+
### Reviewer Context
|
|
62
|
+
Reviewers may have custom instructions below. Only boost findings when instructions indicate domain expertise relevant to the finding (e.g., "focus on security" boosts security findings). Persona/methodology instructions (e.g., "be thorough") don't confer specialist weight. In domain conflicts, prefer the domain-focused reviewer.
|
|
63
|
+
</section>
|
|
64
|
+
|
|
55
65
|
<section name="input-suggestions" locked="true">
|
|
56
66
|
## Input: {{reviewerCount}} Reviewer(s), {{suggestionCount}} Total Suggestions
|
|
57
67
|
|
|
@@ -64,6 +74,16 @@ Merge suggestions from multiple AI reviewers. Deduplicate. Resolve conflicts. Ke
|
|
|
64
74
|
- When reviewers disagree, keep the more specific analysis.
|
|
65
75
|
- Preserve unique insights from individual reviewers.
|
|
66
76
|
- Drop very low confidence (< 0.3) items unless multiple reviewers agree.
|
|
77
|
+
- Assess severity using evidence across reviewers; preserve highest when uncertain. Omit for praise.
|
|
78
|
+
- Severity: critical (crashes, security, data loss, race conditions, breaking changes, test failures), medium (degraded functionality, missing error handling/validation/test coverage), minor (code quality, docs, optimizations).
|
|
79
|
+
</section>
|
|
80
|
+
|
|
81
|
+
<section name="balanced-output" required="true" tier="fast">
|
|
82
|
+
Deduplicate, don't concatenate: merge duplicate findings, preserve distinct ones. Max 2 praise items.
|
|
83
|
+
</section>
|
|
84
|
+
|
|
85
|
+
<section name="summary-synthesis" required="true" tier="fast">
|
|
86
|
+
Draw on reviewer summaries as evidence for your own synthesis. Write one cohesive paragraph — no mention of consolidation/merging.
|
|
67
87
|
</section>
|
|
68
88
|
|
|
69
89
|
<section name="output-schema" locked="true">
|
|
@@ -74,6 +94,7 @@ Merge suggestions from multiple AI reviewers. Deduplicate. Resolve conflicts. Ke
|
|
|
74
94
|
"line": 42,
|
|
75
95
|
"old_or_new": "NEW",
|
|
76
96
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
97
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
77
98
|
"title": "Brief title",
|
|
78
99
|
"description": "Detailed explanation",
|
|
79
100
|
"suggestion": "How to fix/improve (omit for praise)",
|
|
@@ -82,12 +103,13 @@ Merge suggestions from multiple AI reviewers. Deduplicate. Resolve conflicts. Ke
|
|
|
82
103
|
"fileLevelSuggestions": [{
|
|
83
104
|
"file": "path/to/file",
|
|
84
105
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
106
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
85
107
|
"title": "Brief title describing file-level concern",
|
|
86
108
|
"description": "Explanation of the file-level observation",
|
|
87
109
|
"suggestion": "How to address the file-level concern (omit for praise items)",
|
|
88
110
|
"confidence": 0.0-1.0
|
|
89
111
|
}],
|
|
90
|
-
"summary": "
|
|
112
|
+
"summary": "Single cohesive paragraph of key findings. Write as single reviewer."
|
|
91
113
|
}
|
|
92
114
|
|
|
93
115
|
### GitHub Suggestion Syntax
|
|
@@ -121,8 +143,12 @@ const sections = [
|
|
|
121
143
|
{ name: 'critical-output', locked: true },
|
|
122
144
|
{ name: 'role-description', required: true, tier: ['fast'] },
|
|
123
145
|
{ name: 'custom-instructions', optional: true, tier: ['fast', 'balanced', 'thorough'] },
|
|
146
|
+
{ name: 'dedup-instructions', optional: true },
|
|
147
|
+
{ name: 'reviewer-context-guidance', required: true, tier: ['fast'] },
|
|
124
148
|
{ name: 'input-suggestions', locked: true },
|
|
125
149
|
{ name: 'consolidation-rules', required: true, tier: ['fast'] },
|
|
150
|
+
{ name: 'balanced-output', required: true, tier: ['fast'] },
|
|
151
|
+
{ name: 'summary-synthesis', required: true, tier: ['fast'] },
|
|
126
152
|
{ name: 'output-schema', locked: true },
|
|
127
153
|
{ name: 'diff-instructions', required: true, tier: ['fast'] },
|
|
128
154
|
{ name: 'guidelines', required: true, tier: ['fast'] }
|
|
@@ -139,8 +165,12 @@ const defaultOrder = [
|
|
|
139
165
|
'critical-output',
|
|
140
166
|
'role-description',
|
|
141
167
|
'custom-instructions',
|
|
168
|
+
'dedup-instructions',
|
|
169
|
+
'reviewer-context-guidance',
|
|
142
170
|
'input-suggestions',
|
|
143
171
|
'consolidation-rules',
|
|
172
|
+
'balanced-output',
|
|
173
|
+
'summary-synthesis',
|
|
144
174
|
'output-schema',
|
|
145
175
|
'diff-instructions',
|
|
146
176
|
'guidelines'
|
|
@@ -27,6 +27,7 @@
|
|
|
27
27
|
* - {{reviewIntro}} - Review introduction line
|
|
28
28
|
* - {{lineNumberGuidance}} - Line number guidance section
|
|
29
29
|
* - {{customInstructions}} - Custom instructions section (optional)
|
|
30
|
+
* - {{dedupInstructions}} - Dedup instructions section (optional)
|
|
30
31
|
* - {{reviewerSuggestions}} - Formatted reviewer suggestions input
|
|
31
32
|
* - {{suggestionCount}} - Total number of input suggestions
|
|
32
33
|
* - {{reviewerCount}} - Number of reviewers being consolidated
|
|
@@ -69,6 +70,23 @@ Take your time to analyze the reviewer findings thoroughly. For each cluster of
|
|
|
69
70
|
{{customInstructions}}
|
|
70
71
|
</section>
|
|
71
72
|
|
|
73
|
+
<section name="dedup-instructions" optional="true">
|
|
74
|
+
{{dedupInstructions}}
|
|
75
|
+
</section>
|
|
76
|
+
|
|
77
|
+
<section name="reviewer-context-guidance" required="true" tier="thorough">
|
|
78
|
+
### Reviewer Context Awareness
|
|
79
|
+
Each reviewer below may have been configured with custom instructions. These fall into two categories:
|
|
80
|
+
|
|
81
|
+
- **Domain-focused reviewers**: Instructions that specify a code review focus area (e.g., "focus on security", "review error handling", "check performance"). Their findings *within that focus area* carry higher weight than generalist reviewers.
|
|
82
|
+
- **General reviewers**: Either no custom instructions, or instructions about methodology/style/persona (e.g., "be thorough", "use a friendly tone"). Treat their suggestions at face value across all categories.
|
|
83
|
+
|
|
84
|
+
**Weighting rules:**
|
|
85
|
+
- Only boost a reviewer's findings when their instructions indicate domain expertise relevant to the finding's category
|
|
86
|
+
- Cross-specialty findings from a domain-focused reviewer should be treated as general findings
|
|
87
|
+
- In conflicts within a domain, prefer the domain-focused reviewer's analysis over a generalist's
|
|
88
|
+
</section>
|
|
89
|
+
|
|
72
90
|
<section name="input-suggestions" locked="true">
|
|
73
91
|
## Input: {{reviewerCount}} Reviewer(s), {{suggestionCount}} Total Suggestions
|
|
74
92
|
|
|
@@ -111,10 +129,18 @@ When reviewers disagree about an issue:
|
|
|
111
129
|
### 4. Quality Filter
|
|
112
130
|
- Drop suggestions with very low confidence (< 0.3) unless multiple reviewers agree
|
|
113
131
|
- Elevate suggestions where reviewers independently converge
|
|
132
|
+
|
|
133
|
+
### 5. Severity Assessment
|
|
134
|
+
Assess severity based on the evidence and reasoning across all reviewers. When reviewers assign different severities, apply the same conflict resolution principles above. When truly uncertain, preserve the highest severity. Omit severity for praise items.
|
|
135
|
+
|
|
136
|
+
**Severity Definitions:**
|
|
137
|
+
- **critical**: Production incidents, system failures, or security vulnerabilities — runtime crashes, data corruption or loss, race conditions, deadlocks, breaking changes, changes that will cause existing tests to fail
|
|
138
|
+
- **medium**: Degraded functionality or reliability — missing error handling, N+1 queries, missing validation, missing or poor test coverage for new functionality
|
|
139
|
+
- **minor**: Code quality concerns — documentation gaps, minor optimizations, style inconsistencies
|
|
114
140
|
</section>
|
|
115
141
|
|
|
116
142
|
<section name="consensus-handling" required="true" tier="thorough">
|
|
117
|
-
###
|
|
143
|
+
### 6. Consensus Handling and Confidence Calibration
|
|
118
144
|
|
|
119
145
|
**Cross-Reviewer Agreement:**
|
|
120
146
|
- **Strong consensus** (3+ reviewers): Boost confidence by 0.2 (cap at 1.0)
|
|
@@ -131,6 +157,14 @@ When reviewers disagree about an issue:
|
|
|
131
157
|
Note: Confidence is about certainty of value, not severity.
|
|
132
158
|
</section>
|
|
133
159
|
|
|
160
|
+
<section name="balanced-output" required="true" tier="thorough">
|
|
161
|
+
### 7. Balanced Output
|
|
162
|
+
- **Deduplicate, don't concatenate**: If two reviewers flagged the same issue, merge them into one suggestion. If three reviewers each found one style nit, consider whether one representative example suffices. Distinct findings should each be preserved — the goal is to eliminate redundancy, not to reduce count.
|
|
163
|
+
- **Limit praise** to 2–3 most noteworthy items across all reviewers. Praise should highlight genuinely commendable patterns, not routine correctness.
|
|
164
|
+
- **Avoid suggestion overload** — a review with 30 suggestions is harder to act on than one with 12 well-chosen ones. Aim for quality over quantity.
|
|
165
|
+
- **Include confidence scores** that reflect cross-reviewer agreement: consensus findings get boosted, lone findings keep their original confidence.
|
|
166
|
+
</section>
|
|
167
|
+
|
|
134
168
|
<section name="summary-synthesis" required="true" tier="thorough">
|
|
135
169
|
## Summary Synthesis Guidance
|
|
136
170
|
The summary field should synthesize the findings, not list them.
|
|
@@ -139,7 +173,8 @@ The summary field should synthesize the findings, not list them.
|
|
|
139
173
|
- **Lead with the most important insight**: What should the reviewer focus on first?
|
|
140
174
|
- **Connect the dots**: How do individual findings relate to each other?
|
|
141
175
|
- **Calibrate severity**: Is this code fundamentally sound with minor issues, or are there structural problems?
|
|
142
|
-
- **
|
|
176
|
+
- **Draw on reviewer summaries**: Use these as evidence for your own synthesis — integrate their insights into a cohesive conclusion rather than listing them individually.
|
|
177
|
+
- **Write as a single reviewer**: Do not mention consolidation, merging, or multiple reviewers -- unless specifically requested
|
|
143
178
|
</section>
|
|
144
179
|
|
|
145
180
|
<section name="output-schema" locked="true">
|
|
@@ -155,6 +190,7 @@ Output JSON with this structure:
|
|
|
155
190
|
"line": 42,
|
|
156
191
|
"old_or_new": "NEW",
|
|
157
192
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
193
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
158
194
|
"title": "Brief title",
|
|
159
195
|
"description": "Detailed explanation",
|
|
160
196
|
"suggestion": "How to fix/improve (omit for praise)",
|
|
@@ -165,13 +201,14 @@ Output JSON with this structure:
|
|
|
165
201
|
"fileLevelSuggestions": [{
|
|
166
202
|
"file": "path/to/file",
|
|
167
203
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
204
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
168
205
|
"title": "Brief title describing file-level concern",
|
|
169
206
|
"description": "Explanation of the file-level observation",
|
|
170
207
|
"suggestion": "How to address the file-level concern (omit for praise items)",
|
|
171
208
|
"confidence": 0.0-1.0,
|
|
172
209
|
"reasoning": ["Step-by-step reasoning explaining why this issue was flagged"]
|
|
173
210
|
}],
|
|
174
|
-
"summary": "
|
|
211
|
+
"summary": "Single cohesive paragraph synthesizing key findings and their significance. Write as a single reviewer — do not mention consolidation, merging, or multiple reviewers."
|
|
175
212
|
}
|
|
176
213
|
|
|
177
214
|
### GitHub Suggestion Syntax
|
|
@@ -227,9 +264,12 @@ const sections = [
|
|
|
227
264
|
{ name: 'role-description', required: true, tier: ['thorough'] },
|
|
228
265
|
{ name: 'reasoning-encouragement', required: true, tier: ['thorough'] },
|
|
229
266
|
{ name: 'custom-instructions', optional: true, tier: ['balanced', 'thorough'] },
|
|
267
|
+
{ name: 'dedup-instructions', optional: true },
|
|
268
|
+
{ name: 'reviewer-context-guidance', required: true, tier: ['thorough'] },
|
|
230
269
|
{ name: 'input-suggestions', locked: true },
|
|
231
270
|
{ name: 'consolidation-rules', required: true, tier: ['thorough'] },
|
|
232
271
|
{ name: 'consensus-handling', required: true, tier: ['thorough'] },
|
|
272
|
+
{ name: 'balanced-output', required: true, tier: ['thorough'] },
|
|
233
273
|
{ name: 'summary-synthesis', required: true, tier: ['thorough'] },
|
|
234
274
|
{ name: 'output-schema', locked: true },
|
|
235
275
|
{ name: 'diff-instructions', required: true, tier: ['thorough'] },
|
|
@@ -247,9 +287,12 @@ const defaultOrder = [
|
|
|
247
287
|
'role-description',
|
|
248
288
|
'reasoning-encouragement',
|
|
249
289
|
'custom-instructions',
|
|
290
|
+
'dedup-instructions',
|
|
291
|
+
'reviewer-context-guidance',
|
|
250
292
|
'input-suggestions',
|
|
251
293
|
'consolidation-rules',
|
|
252
294
|
'consensus-handling',
|
|
295
|
+
'balanced-output',
|
|
253
296
|
'summary-synthesis',
|
|
254
297
|
'output-schema',
|
|
255
298
|
'diff-instructions',
|
|
@@ -99,6 +99,15 @@ Identify the following in changed code:
|
|
|
99
99
|
Do NOT modify files or run write commands. Analyze and report only.
|
|
100
100
|
</section>
|
|
101
101
|
|
|
102
|
+
<section name="severity-classification" required="true">
|
|
103
|
+
### Severity Classification
|
|
104
|
+
Assign a severity to each suggestion (except praise):
|
|
105
|
+
- **critical**: Production incidents, system failures, or security vulnerabilities — runtime crashes, data corruption or loss, race conditions, deadlocks, breaking changes, changes that will cause existing tests to fail
|
|
106
|
+
- **medium**: Degraded functionality or reliability — missing error handling, N+1 queries, missing validation, missing or poor test coverage for new functionality
|
|
107
|
+
- **minor**: Code quality concerns — documentation gaps, minor optimizations, style inconsistencies
|
|
108
|
+
Omit severity for praise items.
|
|
109
|
+
</section>
|
|
110
|
+
|
|
102
111
|
<section name="output-schema" locked="true">
|
|
103
112
|
## Output Format
|
|
104
113
|
|
|
@@ -111,6 +120,7 @@ Do NOT modify files or run write commands. Analyze and report only.
|
|
|
111
120
|
"line": 42,
|
|
112
121
|
"old_or_new": "NEW",
|
|
113
122
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
123
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
114
124
|
"title": "Brief title",
|
|
115
125
|
"description": "Detailed explanation",
|
|
116
126
|
"suggestion": "How to fix/improve (omit for praise items)",
|
|
@@ -183,6 +193,7 @@ const sections = [
|
|
|
183
193
|
{ name: 'initial-setup', required: true },
|
|
184
194
|
{ name: 'focus-areas', required: true },
|
|
185
195
|
{ name: 'available-commands', required: true },
|
|
196
|
+
{ name: 'severity-classification', required: true },
|
|
186
197
|
{ name: 'output-schema', locked: true },
|
|
187
198
|
{ name: 'diff-instructions', required: true },
|
|
188
199
|
{ name: 'category-definitions', required: true },
|
|
@@ -204,6 +215,7 @@ const defaultOrder = [
|
|
|
204
215
|
'initial-setup',
|
|
205
216
|
'focus-areas',
|
|
206
217
|
'available-commands',
|
|
218
|
+
'severity-classification',
|
|
207
219
|
'output-schema',
|
|
208
220
|
'diff-instructions',
|
|
209
221
|
'category-definitions',
|
|
@@ -79,6 +79,14 @@ ONLY suggest for files in this list:
|
|
|
79
79
|
Annotated diff tool, \`cat -n\`, ls, find, grep. Do NOT modify files.
|
|
80
80
|
</section>
|
|
81
81
|
|
|
82
|
+
<section name="severity-classification" required="true" tier="fast">
|
|
83
|
+
### Severity
|
|
84
|
+
Assign severity to each suggestion (omit for praise):
|
|
85
|
+
- **critical**: Production incidents, crashes, security vulnerabilities, data corruption/loss, race conditions, breaking changes, test failures
|
|
86
|
+
- **medium**: Degraded functionality, missing error handling, missing validation, missing/poor test coverage
|
|
87
|
+
- **minor**: Code quality, docs gaps, minor optimizations
|
|
88
|
+
</section>
|
|
89
|
+
|
|
82
90
|
<section name="output-schema" locked="true">
|
|
83
91
|
## Output Format
|
|
84
92
|
|
|
@@ -91,6 +99,7 @@ Annotated diff tool, \`cat -n\`, ls, find, grep. Do NOT modify files.
|
|
|
91
99
|
"line": 42,
|
|
92
100
|
"old_or_new": "NEW",
|
|
93
101
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
102
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
94
103
|
"title": "Brief title",
|
|
95
104
|
"description": "Explanation",
|
|
96
105
|
"suggestion": "How to fix (omit for praise)",
|
|
@@ -139,6 +148,7 @@ const sections = [
|
|
|
139
148
|
{ name: 'initial-setup', required: true, tier: ['fast'] },
|
|
140
149
|
{ name: 'focus-areas', required: true, tier: ['fast'] },
|
|
141
150
|
{ name: 'available-commands', required: true, tier: ['fast'] },
|
|
151
|
+
{ name: 'severity-classification', required: true, tier: ['fast'] },
|
|
142
152
|
{ name: 'output-schema', locked: true },
|
|
143
153
|
{ name: 'diff-instructions', required: true, tier: ['fast'] },
|
|
144
154
|
{ name: 'guidelines', required: true, tier: ['fast'] }
|
|
@@ -159,6 +169,7 @@ const defaultOrder = [
|
|
|
159
169
|
'initial-setup',
|
|
160
170
|
'focus-areas',
|
|
161
171
|
'available-commands',
|
|
172
|
+
'severity-classification',
|
|
162
173
|
'output-schema',
|
|
163
174
|
'diff-instructions',
|
|
164
175
|
'guidelines'
|
|
@@ -140,6 +140,15 @@ Your role is strictly to analyze and report findings.
|
|
|
140
140
|
Note: You may optionally use parallel read-only Tasks to analyze different parts of the changes if that would be helpful for a thorough analysis.
|
|
141
141
|
</section>
|
|
142
142
|
|
|
143
|
+
<section name="severity-classification" required="true">
|
|
144
|
+
### Severity Classification
|
|
145
|
+
Assign a severity to each suggestion (except praise):
|
|
146
|
+
- **critical**: Production incidents, system failures, or security vulnerabilities — runtime crashes, data corruption or loss, race conditions, deadlocks, breaking changes, changes that will cause existing tests to fail
|
|
147
|
+
- **medium**: Degraded functionality or reliability — missing error handling, N+1 queries, missing validation, missing or poor test coverage for new functionality
|
|
148
|
+
- **minor**: Code quality concerns — documentation gaps, minor optimizations, style inconsistencies
|
|
149
|
+
Omit severity for praise items.
|
|
150
|
+
</section>
|
|
151
|
+
|
|
143
152
|
<section name="output-schema" locked="true">
|
|
144
153
|
## Output Format
|
|
145
154
|
|
|
@@ -153,6 +162,7 @@ Output JSON with this structure:
|
|
|
153
162
|
"line": 42,
|
|
154
163
|
"old_or_new": "NEW",
|
|
155
164
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
165
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
156
166
|
"title": "Brief title",
|
|
157
167
|
"description": "Detailed explanation",
|
|
158
168
|
"suggestion": "How to fix/improve (omit this field for praise items - no action needed)",
|
|
@@ -260,6 +270,7 @@ const sections = [
|
|
|
260
270
|
{ name: 'initial-setup', required: true, tier: ['thorough'] },
|
|
261
271
|
{ name: 'focus-areas', required: true, tier: ['thorough'] },
|
|
262
272
|
{ name: 'available-commands', required: true, tier: ['thorough'] },
|
|
273
|
+
{ name: 'severity-classification', required: true },
|
|
263
274
|
{ name: 'output-schema', locked: true },
|
|
264
275
|
{ name: 'diff-instructions', required: true, tier: ['thorough'] },
|
|
265
276
|
{ name: 'confidence-guidance', required: true, tier: ['thorough'] },
|
|
@@ -283,6 +294,7 @@ const defaultOrder = [
|
|
|
283
294
|
'initial-setup',
|
|
284
295
|
'focus-areas',
|
|
285
296
|
'available-commands',
|
|
297
|
+
'severity-classification',
|
|
286
298
|
'output-schema',
|
|
287
299
|
'diff-instructions',
|
|
288
300
|
'confidence-guidance',
|
|
@@ -97,6 +97,15 @@ Do NOT modify files. Your role is strictly to analyze and report findings.
|
|
|
97
97
|
Note: You may use parallel read-only Tasks to examine multiple files simultaneously.
|
|
98
98
|
</section>
|
|
99
99
|
|
|
100
|
+
<section name="severity-classification" required="true">
|
|
101
|
+
### Severity Classification
|
|
102
|
+
Assign a severity to each suggestion (except praise):
|
|
103
|
+
- **critical**: Production incidents, system failures, or security vulnerabilities — runtime crashes, data corruption or loss, race conditions, deadlocks, breaking changes, changes that will cause existing tests to fail
|
|
104
|
+
- **medium**: Degraded functionality or reliability — missing error handling, N+1 queries, missing validation, missing or poor test coverage for new functionality
|
|
105
|
+
- **minor**: Code quality concerns — documentation gaps, minor optimizations, style inconsistencies
|
|
106
|
+
Omit severity for praise items.
|
|
107
|
+
</section>
|
|
108
|
+
|
|
100
109
|
<section name="output-schema" locked="true">
|
|
101
110
|
## Output Format
|
|
102
111
|
|
|
@@ -109,6 +118,7 @@ Note: You may use parallel read-only Tasks to examine multiple files simultaneou
|
|
|
109
118
|
"line": 42,
|
|
110
119
|
"old_or_new": "NEW",
|
|
111
120
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
121
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
112
122
|
"title": "Brief title",
|
|
113
123
|
"description": "Explanation mentioning why full file context was needed",
|
|
114
124
|
"suggestion": "How to fix/improve (omit for praise items)",
|
|
@@ -118,6 +128,7 @@ Note: You may use parallel read-only Tasks to examine multiple files simultaneou
|
|
|
118
128
|
"fileLevelSuggestions": [{
|
|
119
129
|
"file": "path/to/file",
|
|
120
130
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
131
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
121
132
|
"title": "File-level concern",
|
|
122
133
|
"description": "File-level observation (architecture, organization, naming, etc.)",
|
|
123
134
|
"suggestion": "How to address (omit for praise items)",
|
|
@@ -194,6 +205,7 @@ const sections = [
|
|
|
194
205
|
{ name: 'analysis-process', required: true },
|
|
195
206
|
{ name: 'focus-areas', required: true },
|
|
196
207
|
{ name: 'available-commands', required: true },
|
|
208
|
+
{ name: 'severity-classification', required: true },
|
|
197
209
|
{ name: 'output-schema', locked: true },
|
|
198
210
|
{ name: 'diff-instructions', required: true },
|
|
199
211
|
{ name: 'file-level-guidance', optional: true },
|
|
@@ -214,6 +226,7 @@ const defaultOrder = [
|
|
|
214
226
|
'analysis-process',
|
|
215
227
|
'focus-areas',
|
|
216
228
|
'available-commands',
|
|
229
|
+
'severity-classification',
|
|
217
230
|
'output-schema',
|
|
218
231
|
'diff-instructions',
|
|
219
232
|
'file-level-guidance',
|
|
@@ -80,6 +80,14 @@ ONLY suggest for files in this list:
|
|
|
80
80
|
Annotated diff tool (preferred), \`cat -n <file>\`, ls, find, grep. Do NOT modify files.
|
|
81
81
|
</section>
|
|
82
82
|
|
|
83
|
+
<section name="severity-classification" required="true" tier="fast">
|
|
84
|
+
### Severity
|
|
85
|
+
Assign severity to each suggestion (omit for praise):
|
|
86
|
+
- **critical**: Production incidents, crashes, security vulnerabilities, data corruption/loss, race conditions, breaking changes, test failures
|
|
87
|
+
- **medium**: Degraded functionality, missing error handling, missing validation, missing/poor test coverage
|
|
88
|
+
- **minor**: Code quality, docs gaps, minor optimizations
|
|
89
|
+
</section>
|
|
90
|
+
|
|
83
91
|
<section name="output-schema" locked="true">
|
|
84
92
|
## Output Format
|
|
85
93
|
|
|
@@ -92,6 +100,7 @@ Annotated diff tool (preferred), \`cat -n <file>\`, ls, find, grep. Do NOT modif
|
|
|
92
100
|
"line": 42,
|
|
93
101
|
"old_or_new": "NEW",
|
|
94
102
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
103
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
95
104
|
"title": "Brief title",
|
|
96
105
|
"description": "Why file context was needed",
|
|
97
106
|
"suggestion": "How to fix (omit for praise)",
|
|
@@ -100,6 +109,7 @@ Annotated diff tool (preferred), \`cat -n <file>\`, ls, find, grep. Do NOT modif
|
|
|
100
109
|
"fileLevelSuggestions": [{
|
|
101
110
|
"file": "path/to/file",
|
|
102
111
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
112
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
103
113
|
"title": "File-level concern",
|
|
104
114
|
"description": "File-level observation",
|
|
105
115
|
"suggestion": "How to address (omit for praise)",
|
|
@@ -146,6 +156,7 @@ const sections = [
|
|
|
146
156
|
{ name: 'analysis-process', required: true, tier: ['fast'] },
|
|
147
157
|
{ name: 'focus-areas', required: true, tier: ['fast'] },
|
|
148
158
|
{ name: 'available-commands', required: true, tier: ['fast'] },
|
|
159
|
+
{ name: 'severity-classification', required: true, tier: ['fast'] },
|
|
149
160
|
{ name: 'output-schema', locked: true },
|
|
150
161
|
{ name: 'diff-instructions', required: true, tier: ['fast'] },
|
|
151
162
|
{ name: 'guidelines', required: true, tier: ['fast'] }
|
|
@@ -166,6 +177,7 @@ const defaultOrder = [
|
|
|
166
177
|
'analysis-process',
|
|
167
178
|
'focus-areas',
|
|
168
179
|
'available-commands',
|
|
180
|
+
'severity-classification',
|
|
169
181
|
'output-schema',
|
|
170
182
|
'diff-instructions',
|
|
171
183
|
'guidelines'
|
|
@@ -171,6 +171,15 @@ Your role is strictly to analyze and report findings.
|
|
|
171
171
|
Note: You may optionally use parallel read-only Tasks to examine multiple files simultaneously if that would be helpful for a thorough analysis.
|
|
172
172
|
</section>
|
|
173
173
|
|
|
174
|
+
<section name="severity-classification" required="true">
|
|
175
|
+
### Severity Classification
|
|
176
|
+
Assign a severity to each suggestion (except praise):
|
|
177
|
+
- **critical**: Production incidents, system failures, or security vulnerabilities — runtime crashes, data corruption or loss, race conditions, deadlocks, breaking changes, changes that will cause existing tests to fail
|
|
178
|
+
- **medium**: Degraded functionality or reliability — missing error handling, N+1 queries, missing validation, missing or poor test coverage for new functionality
|
|
179
|
+
- **minor**: Code quality concerns — documentation gaps, minor optimizations, style inconsistencies
|
|
180
|
+
Omit severity for praise items.
|
|
181
|
+
</section>
|
|
182
|
+
|
|
174
183
|
<section name="output-schema" locked="true">
|
|
175
184
|
## Output Format
|
|
176
185
|
|
|
@@ -184,6 +193,7 @@ Output JSON with this structure:
|
|
|
184
193
|
"line": 42,
|
|
185
194
|
"old_or_new": "NEW",
|
|
186
195
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
196
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
187
197
|
"title": "Brief title",
|
|
188
198
|
"description": "Detailed explanation mentioning why full file context was needed",
|
|
189
199
|
"suggestion": "How to fix/improve based on file context (omit for praise items)",
|
|
@@ -193,6 +203,7 @@ Output JSON with this structure:
|
|
|
193
203
|
"fileLevelSuggestions": [{
|
|
194
204
|
"file": "path/to/file",
|
|
195
205
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
206
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
196
207
|
"title": "Brief title describing file-level concern",
|
|
197
208
|
"description": "Explanation of the file-level observation (architecture, organization, naming, etc.)",
|
|
198
209
|
"suggestion": "How to address the file-level concern (omit for praise items)",
|
|
@@ -307,6 +318,7 @@ const sections = [
|
|
|
307
318
|
{ name: 'analysis-process', required: true, tier: ['thorough'] },
|
|
308
319
|
{ name: 'focus-areas', required: true, tier: ['thorough'] },
|
|
309
320
|
{ name: 'available-commands', required: true, tier: ['thorough'] },
|
|
321
|
+
{ name: 'severity-classification', required: true },
|
|
310
322
|
{ name: 'output-schema', locked: true },
|
|
311
323
|
{ name: 'diff-instructions', required: true, tier: ['thorough'] },
|
|
312
324
|
{ name: 'confidence-guidance', required: true, tier: ['thorough'] },
|
|
@@ -331,6 +343,7 @@ const defaultOrder = [
|
|
|
331
343
|
'analysis-process',
|
|
332
344
|
'focus-areas',
|
|
333
345
|
'available-commands',
|
|
346
|
+
'severity-classification',
|
|
334
347
|
'output-schema',
|
|
335
348
|
'diff-instructions',
|
|
336
349
|
'confidence-guidance',
|
|
@@ -120,6 +120,15 @@ You may use parallel read-only Tasks to explore different areas of the codebase
|
|
|
120
120
|
{{sparseCheckoutGuidance}}
|
|
121
121
|
</section>
|
|
122
122
|
|
|
123
|
+
<section name="severity-classification" required="true">
|
|
124
|
+
### Severity Classification
|
|
125
|
+
Assign a severity to each suggestion (except praise):
|
|
126
|
+
- **critical**: Production incidents, system failures, or security vulnerabilities — runtime crashes, data corruption or loss, race conditions, deadlocks, breaking changes, changes that will cause existing tests to fail
|
|
127
|
+
- **medium**: Degraded functionality or reliability — missing error handling, N+1 queries, missing validation, missing or poor test coverage for new functionality
|
|
128
|
+
- **minor**: Code quality concerns — documentation gaps, minor optimizations, style inconsistencies
|
|
129
|
+
Omit severity for praise items.
|
|
130
|
+
</section>
|
|
131
|
+
|
|
123
132
|
<section name="output-schema" locked="true">
|
|
124
133
|
## Output Format
|
|
125
134
|
|
|
@@ -133,6 +142,7 @@ Output JSON with this structure:
|
|
|
133
142
|
"line": 42,
|
|
134
143
|
"old_or_new": "NEW",
|
|
135
144
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
145
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
136
146
|
"title": "Brief title",
|
|
137
147
|
"description": "Detailed explanation mentioning why codebase context was needed",
|
|
138
148
|
"suggestion": "How to fix/improve based on codebase context (omit for praise items)",
|
|
@@ -142,6 +152,7 @@ Output JSON with this structure:
|
|
|
142
152
|
"fileLevelSuggestions": [{
|
|
143
153
|
"file": "path/to/file",
|
|
144
154
|
"type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
|
|
155
|
+
"severity": "critical|medium|minor (omit for praise)",
|
|
145
156
|
"title": "Brief title describing file-level concern",
|
|
146
157
|
"description": "Explanation of the file-level observation from codebase perspective",
|
|
147
158
|
"suggestion": "How to address the file-level concern (omit for praise items)",
|
|
@@ -230,6 +241,7 @@ const sections = [
|
|
|
230
241
|
{ name: 'focus-areas', required: true, tier: ['balanced'] },
|
|
231
242
|
{ name: 'available-commands', required: true, tier: ['balanced'] },
|
|
232
243
|
{ name: 'sparse-checkout', optional: true, tier: ['fast', 'balanced', 'thorough'] },
|
|
244
|
+
{ name: 'severity-classification', required: true },
|
|
233
245
|
{ name: 'output-schema', locked: true },
|
|
234
246
|
{ name: 'diff-instructions', required: true, tier: ['balanced'] },
|
|
235
247
|
{ name: 'file-level-guidance', optional: true, tier: ['balanced', 'thorough'] },
|
|
@@ -252,6 +264,7 @@ const defaultOrder = [
|
|
|
252
264
|
'focus-areas',
|
|
253
265
|
'available-commands',
|
|
254
266
|
'sparse-checkout',
|
|
267
|
+
'severity-classification',
|
|
255
268
|
'output-schema',
|
|
256
269
|
'diff-instructions',
|
|
257
270
|
'file-level-guidance',
|