@in-the-loop-labs/pair-review 1.4.3 → 1.5.0

package/src/ai/claude-cli.js

@@ -37,9 +37,9 @@ class ClaudeCLI {
    */
   async execute(prompt, options = {}) {
     return new Promise((resolve, reject) => {
-      const { cwd = process.cwd(), timeout = 300000, level = 'unknown' } = options; // 5 minute default timeout
+      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', logPrefix } = options; // 5 minute default timeout
 
-      const levelPrefix = `[Level ${level}]`;
+      const levelPrefix = logPrefix || `[Level ${level}]`;
       logger.info(`${levelPrefix} Executing Claude CLI...`);
       logger.info(`${levelPrefix} Writing prompt: ${prompt.length} bytes`);
 

package/src/ai/claude-provider.js

@@ -241,9 +241,9 @@ class ClaudeProvider extends AIProvider {
    */
   async execute(prompt, options = {}) {
     return new Promise((resolve, reject) => {
-      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent } = options;
+      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent, logPrefix } = options;
 
-      const levelPrefix = `[Level ${level}]`;
+      const levelPrefix = logPrefix || `[Level ${level}]`;
       logger.info(`${levelPrefix} Executing Claude CLI...`);
       logger.info(`${levelPrefix} Writing prompt: ${prompt.length} bytes`);
 
@@ -359,7 +359,7 @@ class ClaudeProvider extends AIProvider {
         logger.info(`${levelPrefix} Claude CLI completed: ${lineCount} JSONL events received`);
 
         // Parse the Claude JSONL stream response
-        const parsed = this.parseClaudeResponse(stdout, level);
+        const parsed = this.parseClaudeResponse(stdout, level, levelPrefix);
         if (parsed.success) {
           logger.success(`${levelPrefix} Successfully parsed JSON response`);
           // Dump the parsed data for debugging
@@ -384,7 +384,7 @@ class ClaudeProvider extends AIProvider {
           // Use async IIFE to handle the async LLM extraction
           (async () => {
             try {
-              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess });
+              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess, logPrefix: levelPrefix });
               if (llmExtracted.success) {
                 logger.success(`${levelPrefix} LLM extraction fallback succeeded`);
                 settle(resolve, llmExtracted.data);
@@ -633,8 +633,8 @@ class ClaudeProvider extends AIProvider {
    * @param {string|number} level - Analysis level for logging
    * @returns {{success: boolean, data?: Object, error?: string}}
    */
-  parseClaudeResponse(stdout, level) {
-    const levelPrefix = `[Level ${level}]`;
+  parseClaudeResponse(stdout, level, logPrefix) {
+    const levelPrefix = logPrefix || `[Level ${level}]`;
 
     try {
       // Split by newlines and parse each JSON line

package/src/ai/codex-provider.js

@@ -132,9 +132,9 @@ class CodexProvider extends AIProvider {
    */
   async execute(prompt, options = {}) {
     return new Promise((resolve, reject) => {
-      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent } = options;
+      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent, logPrefix } = options;
 
-      const levelPrefix = `[Level ${level}]`;
+      const levelPrefix = logPrefix || `[Level ${level}]`;
       logger.info(`${levelPrefix} Executing Codex CLI...`);
       logger.info(`${levelPrefix} Writing prompt: ${prompt.length} bytes`);
 
@@ -256,7 +256,7 @@ class CodexProvider extends AIProvider {
         }
 
         // Parse the Codex JSONL response
-        const parsed = this.parseCodexResponse(stdout, level);
+        const parsed = this.parseCodexResponse(stdout, level, levelPrefix);
         if (parsed.success) {
           logger.success(`${levelPrefix} Successfully parsed JSON response`);
           // Dump the parsed data for debugging
@@ -278,7 +278,7 @@ class CodexProvider extends AIProvider {
           // Use async IIFE to handle the async LLM extraction
           (async () => {
             try {
-              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess });
+              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess, logPrefix: levelPrefix });
               if (llmExtracted.success) {
                 logger.success(`${levelPrefix} LLM extraction fallback succeeded`);
                 settle(resolve, llmExtracted.data);
@@ -337,8 +337,8 @@ class CodexProvider extends AIProvider {
    * @param {string|number} level - Analysis level for logging
    * @returns {{success: boolean, data?: Object, error?: string}}
    */
-  parseCodexResponse(stdout, level) {
-    const levelPrefix = `[Level ${level}]`;
+  parseCodexResponse(stdout, level, logPrefix) {
+    const levelPrefix = logPrefix || `[Level ${level}]`;
 
     try {
       // Split by newlines and parse each JSON line

package/src/ai/copilot-provider.js

@@ -197,9 +197,9 @@ class CopilotProvider extends AIProvider {
     return new Promise((resolve, reject) => {
       // Note: Copilot does not support streaming — output is plain text returned on process exit, not JSONL.
       // onStreamEvent is therefore not destructured here (no StreamParser integration).
-      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess } = options;
+      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, logPrefix } = options;
 
-      const levelPrefix = `[Level ${level}]`;
+      const levelPrefix = logPrefix || `[Level ${level}]`;
       logger.info(`${levelPrefix} Executing Copilot CLI...`);
       logger.info(`${levelPrefix} Writing prompt: ${prompt.length} bytes`);
 
@@ -310,7 +310,7 @@ class CopilotProvider extends AIProvider {
           // Use async IIFE to handle the async LLM extraction
           (async () => {
             try {
-              const llmExtracted = await this.extractJSONWithLLM(stdout, { level, analysisId, registerProcess });
+              const llmExtracted = await this.extractJSONWithLLM(stdout, { level, analysisId, registerProcess, logPrefix: levelPrefix });
               if (llmExtracted.success) {
                 logger.success(`${levelPrefix} LLM extraction fallback succeeded`);
                 settle(resolve, llmExtracted.data);

package/src/ai/cursor-agent-provider.js

@@ -211,9 +211,9 @@ class CursorAgentProvider extends AIProvider {
    */
   async execute(prompt, options = {}) {
     return new Promise((resolve, reject) => {
-      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent } = options;
+      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent, logPrefix } = options;
 
-      const levelPrefix = `[Level ${level}]`;
+      const levelPrefix = logPrefix || `[Level ${level}]`;
       logger.info(`${levelPrefix} Executing Cursor Agent CLI...`);
       logger.info(`${levelPrefix} Writing prompt: ${prompt.length} chars`);
 
@@ -335,7 +335,7 @@ class CursorAgentProvider extends AIProvider {
         logger.info(`${levelPrefix} Cursor Agent CLI completed: ${lineCount} JSONL events received`);
 
         // Parse the Cursor Agent JSONL stream response
-        const parsed = this.parseCursorAgentResponse(stdout, level);
+        const parsed = this.parseCursorAgentResponse(stdout, level, levelPrefix);
         if (parsed.success) {
           logger.success(`${levelPrefix} Successfully parsed JSON response`);
           // Dump the parsed data for debugging
@@ -361,7 +361,7 @@ class CursorAgentProvider extends AIProvider {
               // orphan processes if timeout fired between close-handler entry
               // and reaching this point.
               if (settled) return;
-              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess });
+              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess, logPrefix: levelPrefix });
               if (llmExtracted.success) {
                 logger.success(`${levelPrefix} LLM extraction fallback succeeded`);
                 settle(resolve, llmExtracted.data);
@@ -425,8 +425,8 @@ class CursorAgentProvider extends AIProvider {
    * @param {string|number} level - Analysis level for logging
    * @returns {{success: boolean, data?: Object, error?: string}}
    */
-  parseCursorAgentResponse(stdout, level) {
-    const levelPrefix = `[Level ${level}]`;
+  parseCursorAgentResponse(stdout, level, logPrefix) {
+    const levelPrefix = logPrefix || `[Level ${level}]`;
 
     try {
       // Split by newlines and parse each JSON line

package/src/ai/gemini-provider.js

@@ -174,9 +174,9 @@ class GeminiProvider extends AIProvider {
    */
   async execute(prompt, options = {}) {
     return new Promise((resolve, reject) => {
-      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent } = options;
+      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent, logPrefix } = options;
 
-      const levelPrefix = `[Level ${level}]`;
+      const levelPrefix = logPrefix || `[Level ${level}]`;
       logger.info(`${levelPrefix} Executing Gemini CLI...`);
       logger.info(`${levelPrefix} Writing prompt: ${prompt.length} bytes`);
 
@@ -298,7 +298,7 @@ class GeminiProvider extends AIProvider {
         }
 
         // Parse the Gemini JSONL stream response
-        const parsed = this.parseGeminiResponse(stdout, level);
+        const parsed = this.parseGeminiResponse(stdout, level, levelPrefix);
         if (parsed.success) {
           logger.success(`${levelPrefix} Successfully parsed JSON response`);
           // Dump the parsed data for debugging
@@ -320,7 +320,7 @@ class GeminiProvider extends AIProvider {
           // Use async IIFE to handle the async LLM extraction
           (async () => {
             try {
-              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess });
+              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess, logPrefix: levelPrefix });
               if (llmExtracted.success) {
                 logger.success(`${levelPrefix} LLM extraction fallback succeeded`);
                 settle(resolve, llmExtracted.data);
@@ -382,8 +382,8 @@ class GeminiProvider extends AIProvider {
    * @param {string|number} level - Analysis level for logging
    * @returns {{success: boolean, data?: Object, error?: string}}
    */
-  parseGeminiResponse(stdout, level) {
-    const levelPrefix = `[Level ${level}]`;
+  parseGeminiResponse(stdout, level, logPrefix) {
+    const levelPrefix = logPrefix || `[Level ${level}]`;
 
     try {
       // Split by newlines and parse each JSON line

package/src/ai/opencode-provider.js

@@ -102,9 +102,9 @@ class OpenCodeProvider extends AIProvider {
    */
   async execute(prompt, options = {}) {
     return new Promise((resolve, reject) => {
-      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent } = options;
+      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent, logPrefix } = options;
 
-      const levelPrefix = `[Level ${level}]`;
+      const levelPrefix = logPrefix || `[Level ${level}]`;
       logger.info(`${levelPrefix} Executing OpenCode CLI...`);
       logger.info(`${levelPrefix} Writing prompt via stdin: ${prompt.length} bytes`);
 
@@ -238,7 +238,7 @@ class OpenCodeProvider extends AIProvider {
         logger.info(`${levelPrefix} OpenCode CLI completed - received ${lineCount} JSONL events`);
 
         // Parse the OpenCode JSONL response
-        const parsed = this.parseOpenCodeResponse(stdout, level);
+        const parsed = this.parseOpenCodeResponse(stdout, level, levelPrefix);
         if (parsed.success) {
           logger.success(`${levelPrefix} Successfully parsed JSON response`);
 
@@ -262,7 +262,7 @@ class OpenCodeProvider extends AIProvider {
           // Use async IIFE to handle the async LLM extraction
           (async () => {
             try {
-              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess });
+              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess, logPrefix: levelPrefix });
               if (llmExtracted.success) {
                 logger.success(`${levelPrefix} LLM extraction fallback succeeded`);
                 settle(resolve, llmExtracted.data);
@@ -442,8 +442,8 @@ class OpenCodeProvider extends AIProvider {
    * @param {string|number} level - Analysis level for logging
    * @returns {{success: boolean, data?: Object, error?: string}}
    */
-  parseOpenCodeResponse(stdout, level) {
-    const levelPrefix = `[Level ${level}]`;
+  parseOpenCodeResponse(stdout, level, logPrefix) {
+    const levelPrefix = logPrefix || `[Level ${level}]`;
 
     try {
       // Split by newlines and parse each JSON line

package/src/ai/pi-provider.js

@@ -241,9 +241,9 @@ class PiProvider extends AIProvider {
    */
   async execute(prompt, options = {}) {
     return new Promise((resolve, reject) => {
-      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent } = options;
+      const { cwd = process.cwd(), timeout = 300000, level = 'unknown', analysisId, registerProcess, onStreamEvent, logPrefix } = options;
 
-      const levelPrefix = `[Level ${level}]`;
+      const levelPrefix = logPrefix || `[Level ${level}]`;
       logger.info(`${levelPrefix} Executing Pi CLI...`);
       logger.info(`${levelPrefix} Writing prompt via stdin: ${prompt.length} bytes`);
 
@@ -388,7 +388,7 @@ class PiProvider extends AIProvider {
         logger.info(`${levelPrefix} Pi CLI completed - received ${lineCount} JSONL events`);
 
         // Parse the Pi JSONL response
-        const parsed = this.parsePiResponse(stdout, level);
+        const parsed = this.parsePiResponse(stdout, level, levelPrefix);
         if (parsed.success) {
           logger.success(`${levelPrefix} Successfully parsed JSON response`);
 
@@ -419,7 +419,7 @@ class PiProvider extends AIProvider {
             if (settled) return;
 
             try {
-              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess });
+              const llmExtracted = await this.extractJSONWithLLM(llmFallbackInput, { level, analysisId, registerProcess, logPrefix: levelPrefix });
               if (llmExtracted.success) {
                 logger.success(`${levelPrefix} LLM extraction fallback succeeded`);
                 settle(resolve, llmExtracted.data);
@@ -612,8 +612,8 @@ class PiProvider extends AIProvider {
    * @param {string|number} level - Analysis level for logging
    * @returns {{success: boolean, data?: Object, error?: string}}
    */
-  parsePiResponse(stdout, level) {
-    const levelPrefix = `[Level ${level}]`;
+  parsePiResponse(stdout, level, logPrefix) {
+    const levelPrefix = logPrefix || `[Level ${level}]`;
 
     try {
       // Split by newlines and parse each JSON line

package/src/ai/prompts/baseline/consolidation/balanced.js

@@ -0,0 +1,208 @@
+// SPDX-License-Identifier: GPL-3.0-or-later
+/**
+ * Consolidation Balanced Prompt - Cross-Reviewer Suggestion Merging
+ *
+ * This is the canonical baseline prompt for Consolidation analysis.
+ * It merges suggestions from multiple independent AI reviewers who
+ * analyzed the same code changes, deduplicating and resolving conflicts.
+ *
+ * Unlike Orchestration (which merges across analysis levels 1/2/3),
+ * Consolidation merges across reviewers within the same scope.
+ *
+ * Section categories:
+ * - locked: Cannot be modified by variants (data integrity)
+ * - required: Must be present, content can be rephrased
+ * - optional: Can be removed entirely if unhelpful
+ */
+
+/**
+ * Tagged prompt template for Consolidation Balanced analysis
+ *
+ * Placeholders:
+ * - {{reviewIntro}} - Review introduction line
+ * - {{lineNumberGuidance}} - Line number guidance section
+ * - {{customInstructions}} - Custom instructions section (optional)
+ * - {{reviewerSuggestions}} - Formatted reviewer suggestions input
+ * - {{suggestionCount}} - Total number of input suggestions
+ * - {{reviewerCount}} - Number of reviewers being consolidated
+ */
+const taggedPrompt = `<section name="role" required="true">
+{{reviewIntro}}
+</section>
+
+<section name="task-header" required="true">
+# Cross-Reviewer Consolidation Task
+</section>
+
+<section name="line-number-guidance" required="true">
+{{lineNumberGuidance}}
+</section>
+
+<section name="critical-output" locked="true">
+**>>> CRITICAL: Output ONLY valid JSON. No markdown, no \`\`\`json blocks. Start with { end with }. <<<**
+</section>
+
+<section name="role-description" required="true">
+## Your Role
+Multiple independent AI reviewers have analyzed the same code changes. Your job is to merge their findings into a single, high-quality set of suggestions by deduplicating, resolving conflicts, and preserving unique insights.
+</section>
+
+<section name="custom-instructions" optional="true">
+{{customInstructions}}
+</section>
+
+<section name="input-suggestions" locked="true">
+## Input: {{reviewerCount}} Reviewer(s), {{suggestionCount}} Total Suggestions
+
+{{reviewerSuggestions}}
+</section>
+
+<section name="consolidation-rules" required="true">
+## Consolidation Guidelines
+
+### 1. Deduplication
+- **Merge suggestions** that identify the same issue at the same location
+- When merging, combine the best elements from each reviewer's description
+- Use the most specific and actionable framing
+
+### 2. Conflict Resolution
+- When reviewers **disagree**, prefer the analysis with stronger evidence
+- If genuinely uncertain, keep the suggestion with reduced confidence
+- Consider whether one reviewer had context the other missed
+
+### 3. Unique Insights
+- **Preserve suggestions** that only one reviewer noticed
+- A unique finding from one reviewer can be the most valuable insight
+- Don't discard something just because only one reviewer flagged it
+
+### 4. Quality Filter
+- Drop suggestions with very low confidence (< 0.3) unless multiple reviewers agree
+- Boost confidence when multiple reviewers independently identify the same issue
+</section>
+
+<section name="consensus-handling" required="true">
+### 5. Consensus Handling
+- **Agreement**: When multiple reviewers flag the same issue, increase confidence by 0.1-0.2 (cap at 1.0)
+- **Partial overlap**: Merge related but distinct observations into a richer suggestion
+- **Contradiction**: Use your judgment; prefer the more actionable analysis
+</section>
+
+<section name="output-schema" locked="true">
+## Output Format
+
+**>>> CRITICAL: Output ONLY valid JSON. No markdown, no \`\`\`json blocks. Start with { end with }. <<<**
+
+Output JSON with this structure:
+{
+  "suggestions": [
+    {
+      "file": "path/to/file",
+      "line": 42,
+      "old_or_new": "NEW",
+      "type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
+      "title": "Brief title",
+      "description": "Detailed explanation",
+      "suggestion": "How to fix/improve (omit for praise)",
+      "confidence": 0.0-1.0
+    }
+  ],
+  "fileLevelSuggestions": [{
+    "file": "path/to/file",
+    "type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
+    "title": "Brief title describing file-level concern",
+    "description": "Explanation of the file-level observation",
+    "suggestion": "How to address the file-level concern (omit for praise items)",
+    "confidence": 0.0-1.0
+  }],
+  "summary": "Brief consolidation summary. Write as if a single reviewer produced this analysis — do NOT mention 'consolidation', 'merging', or 'multiple reviewers'."
+}
+</section>
+
+<section name="diff-instructions" required="true">
+## Line Number Reference (old_or_new field)
+- **"NEW"** (default): For added [+] and context lines
+- **"OLD"**: ONLY for deleted [-] lines
+Preserve the old_or_new value from input suggestions when merging.
+</section>
+
+<section name="guidelines" required="true">
+## Important Notes
+- **Quality over quantity** — better to have fewer excellent suggestions than many mediocre ones
+- **Cross-reviewer agreement** increases confidence significantly
+- **Preserve actionability** — every suggestion should give clear next steps
+- **Only include modified files** — discard suggestions for unmodified files
+</section>`;
+
+/**
+ * Section definitions with metadata
+ * Used for parsing and validation
+ */
+const sections = [
+  { name: 'role', required: true },
+  { name: 'task-header', required: true },
+  { name: 'line-number-guidance', required: true },
+  { name: 'critical-output', locked: true },
+  { name: 'role-description', required: true },
+  { name: 'custom-instructions', optional: true },
+  { name: 'input-suggestions', locked: true },
+  { name: 'consolidation-rules', required: true },
+  { name: 'consensus-handling', required: true },
+  { name: 'output-schema', locked: true },
+  { name: 'diff-instructions', required: true },
+  { name: 'guidelines', required: true }
+];
+
+/**
+ * Default section order for Consolidation Balanced
+ */
+const defaultOrder = [
+  'role',
+  'task-header',
+  'line-number-guidance',
+  'critical-output',
+  'role-description',
+  'custom-instructions',
+  'input-suggestions',
+  'consolidation-rules',
+  'consensus-handling',
+  'output-schema',
+  'diff-instructions',
+  'guidelines'
+];
+
+/**
+ * Parse the tagged prompt into section objects
+ * @returns {Array<Object>} Array of section objects with name, attributes, and content
+ */
+function parseSections() {
+  const sectionRegex = /<section\s+name="([^"]+)"([^>]*)>([\s\S]*?)<\/section>/g;
+  const parsed = [];
+  let match;
+
+  while ((match = sectionRegex.exec(taggedPrompt)) !== null) {
+    const [, name, attrs, content] = match;
+    const section = {
+      name,
+      content: content.trim(),
+      locked: attrs.includes('locked="true"'),
+      required: attrs.includes('required="true"'),
+      optional: attrs.includes('optional="true"')
+    };
+
+    const tierMatch = attrs.match(/tier="([^"]+)"/);
+    if (tierMatch) {
+      section.tier = tierMatch[1].split(',').map(t => t.trim());
+    }
+
+    parsed.push(section);
+  }
+
+  return parsed;
+}
+
+module.exports = {
+  taggedPrompt,
+  sections,
+  defaultOrder,
+  parseSections
+};

package/src/ai/prompts/baseline/consolidation/fast.js

@@ -0,0 +1,175 @@
+// SPDX-License-Identifier: GPL-3.0-or-later
+/**
+ * Consolidation Fast Prompt - Quick Cross-Reviewer Suggestion Merging
+ *
+ * This is the fast tier variant of Consolidation analysis. It is optimized
+ * for speed with shorter, more directive prompts.
+ *
+ * Tier-specific optimizations applied:
+ * - Simplified: consolidation-rules to essential directives
+ * - Removed: consensus-handling section (folded into rules)
+ * - Shortened: guidelines to essentials
+ *
+ * Section categories:
+ * - locked: Cannot be modified by variants (data integrity)
+ * - required: Must be present, content can be rephrased
+ * - optional: Can be removed entirely if unhelpful
+ */
+
+/**
+ * Tagged prompt template for Consolidation Fast analysis
+ *
+ * Placeholders:
+ * - {{reviewIntro}} - Review introduction line
+ * - {{lineNumberGuidance}} - Line number guidance section
+ * - {{customInstructions}} - Custom instructions section (optional)
+ * - {{reviewerSuggestions}} - Formatted reviewer suggestions input
+ * - {{suggestionCount}} - Total number of input suggestions
+ * - {{reviewerCount}} - Number of reviewers being consolidated
+ */
+const taggedPrompt = `<section name="role" required="true" tier="fast">
+{{reviewIntro}}
+</section>
+
+<section name="task-header" required="true" tier="fast">
+# Cross-Reviewer Consolidation
+</section>
+
+<section name="line-number-guidance" required="true">
+{{lineNumberGuidance}}
+</section>
+
+<section name="critical-output" locked="true">
+**>>> CRITICAL: Output ONLY valid JSON. No markdown, no \`\`\`json blocks. Start with { end with }. <<<**
+</section>
+
+<section name="role-description" required="true" tier="fast">
+## Task
+Merge suggestions from multiple AI reviewers. Deduplicate. Resolve conflicts. Keep high-value items only.
+</section>
+
+<section name="custom-instructions" optional="true" tier="fast,balanced,thorough">
+{{customInstructions}}
+</section>
+
+<section name="input-suggestions" locked="true">
+## Input: {{reviewerCount}} Reviewer(s), {{suggestionCount}} Total Suggestions
+
+{{reviewerSuggestions}}
+</section>
+
+<section name="consolidation-rules" required="true" tier="fast">
+## Rules
+- Merge duplicate suggestions (same file/line/issue). Boost confidence for consensus.
+- When reviewers disagree, keep the more specific analysis.
+- Preserve unique insights from individual reviewers.
+- Drop very low confidence (< 0.3) items unless multiple reviewers agree.
+</section>
+
+<section name="output-schema" locked="true">
+## JSON Schema
+{
+  "suggestions": [{
+    "file": "path/to/file",
+    "line": 42,
+    "old_or_new": "NEW",
+    "type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
+    "title": "Brief title",
+    "description": "Detailed explanation",
+    "suggestion": "How to fix/improve (omit for praise)",
+    "confidence": 0.0-1.0
+  }],
+  "fileLevelSuggestions": [{
+    "file": "path/to/file",
+    "type": "bug|improvement|praise|suggestion|design|performance|security|code-style",
+    "title": "Brief title describing file-level concern",
+    "description": "Explanation of the file-level observation",
+    "suggestion": "How to address the file-level concern (omit for praise items)",
+    "confidence": 0.0-1.0
+  }],
+  "summary": "Key findings as if from single reviewer (no mention of consolidation/merging)"
+}
+</section>
+
+<section name="diff-instructions" required="true" tier="fast">
+## old_or_new
+"NEW" (default): added [+] and context lines. "OLD": deleted [-] only. Preserve from input.
+</section>
+
+<section name="guidelines" required="true" tier="fast">
+## Notes
+Quality over quantity. Higher confidence for multi-reviewer agreement. Only modified files. Omit uncertain suggestions.
+</section>`;
+
+/**
+ * Section definitions with metadata
+ * Used for parsing and validation
+ */
+const sections = [
+  { name: 'role', required: true, tier: ['fast'] },
+  { name: 'task-header', required: true, tier: ['fast'] },
+  { name: 'line-number-guidance', required: true },
+  { name: 'critical-output', locked: true },
+  { name: 'role-description', required: true, tier: ['fast'] },
+  { name: 'custom-instructions', optional: true, tier: ['fast', 'balanced', 'thorough'] },
+  { name: 'input-suggestions', locked: true },
+  { name: 'consolidation-rules', required: true, tier: ['fast'] },
+  { name: 'output-schema', locked: true },
+  { name: 'diff-instructions', required: true, tier: ['fast'] },
+  { name: 'guidelines', required: true, tier: ['fast'] }
+];
+
+/**
+ * Default section order for Consolidation Fast
+ * Note: Removed consensus-handling section
+ */
+const defaultOrder = [
+  'role',
+  'task-header',
+  'line-number-guidance',
+  'critical-output',
+  'role-description',
+  'custom-instructions',
+  'input-suggestions',
+  'consolidation-rules',
+  'output-schema',
+  'diff-instructions',
+  'guidelines'
+];
+
+/**
+ * Parse the tagged prompt into section objects
+ * @returns {Array<Object>} Array of section objects with name, attributes, and content
+ */
+function parseSections() {
+  const sectionRegex = /<section\s+name="([^"]+)"([^>]*)>([\s\S]*?)<\/section>/g;
+  const parsed = [];
+  let match;
+
+  while ((match = sectionRegex.exec(taggedPrompt)) !== null) {
+    const [, name, attrs, content] = match;
+    const section = {
+      name,
+      content: content.trim(),
+      locked: attrs.includes('locked="true"'),
+      required: attrs.includes('required="true"'),
+      optional: attrs.includes('optional="true"')
+    };
+
+    const tierMatch = attrs.match(/tier="([^"]+)"/);
+    if (tierMatch) {
+      section.tier = tierMatch[1].split(',').map(t => t.trim());
+    }
+
+    parsed.push(section);
+  }
+
+  return parsed;
+}
+
+module.exports = {
+  taggedPrompt,
+  sections,
+  defaultOrder,
+  parseSections
+};