Revert "fix(critical): Complete 4-bug fix for report quality issues"

This reverts commit 64f7180.

Author: alpsla

packages/agents/src/two-branch/analyzers/v9-grouped-report-formatter.ts

@@ -1370,8 +1370,7 @@ ${(() => {
   const autoFixPercent = issues.length > 0 ? Math.round((autoFixableCount / issues.length) * 100) : 0;
   
   if (autoFixableCount > 0) {
-    return `\n> 🚀 **Quick Win**: ${autoFixableCount.toLocaleString()} issues (${autoFixPercent}%) can be automatically fixed using IDE tools!
-> ⚠️  **Note**: While all issues have automated fixes available, we recommend manually reviewing critical (${issues.filter(i => i.severity === 'critical').length}) and high (${issues.filter(i => i.severity === 'high').length}) severity issues before applying fixes.\n`;
+    return `\n> 🚀 **Quick Win**: ${autoFixableCount.toLocaleString()} issues (${autoFixPercent}%) can be automatically fixed using the attached manifest file!\n`;
   }
   return '';
 })()}

packages/agents/src/two-branch/analyzers/v9-skill-score-manager.ts

@@ -18,7 +18,6 @@ export interface SkillScoreData {
   repository: string;  // Will be stored as 'repo_name' in database
   prNumber: number;
   branch?: string;
-  commitHash?: string;  // BUG #4 FIX: Track commit to prevent duplicate trend entries
   overallScore: number;
   qualityScore?: number;
   categoryScores: {
@@ -94,11 +93,11 @@ export class SkillScoreManager {
     try {
       const { data, error } = await this.supabase
         .from('skill_scores')
-        .select('overall_score, commit_hash')  // BUG #4 FIX: Fetch commit_hash too
+        .select('overall_score')
         .eq('developer_email', developerEmail)
         .eq('repo_name', repository)  // Fixed: use 'repo_name' column
         .order('analyzed_at', { ascending: true })
-        .limit(limit * 2);  // Fetch more to account for duplicates
+        .limit(limit);
 
       if (error) {
         console.warn('[SkillScoreManager] Error fetching trend:', error.message);
@@ -109,21 +108,8 @@ export class SkillScoreManager {
         return [];
       }
 
-      // BUG #4 FIX: Remove duplicate commits, keep only latest analysis per commit
-      const seenCommits = new Set<string>();
-      const uniqueScores: number[] = [];
-
-      for (const record of data) {
-        const commitHash = record.commit_hash || `pr-${Math.random()}`; // Fallback for old records
-        if (!seenCommits.has(commitHash)) {
-          seenCommits.add(commitHash);
-          uniqueScores.push(record.overall_score);
-          if (uniqueScores.length >= limit) break;
-        }
-      }
-
-      const trend = uniqueScores.slice(0, limit);
-      console.log(`[SkillScoreManager] Trend for ${developerEmail}: [${trend.join(', ')}] (${seenCommits.size} unique commits)`);
+      const trend = data.map(r => r.overall_score);
+      console.log(`[SkillScoreManager] Trend for ${developerEmail}: [${trend.join(', ')}]`);
       return trend;
     } catch (error) {
       console.error('[SkillScoreManager] Unexpected error fetching trend:', error);
@@ -145,7 +131,6 @@ export class SkillScoreManager {
         repo_name: scoreData.repository,  // Fixed: use 'repo_name' column
         pr_number: scoreData.prNumber,
         branch: scoreData.branch,
-        commit_hash: scoreData.commitHash,  // BUG #4 FIX: Store commit hash
         overall_score: scoreData.overallScore,
         quality_score: scoreData.qualityScore,
         security_score: scoreData.categoryScores.security,

packages/agents/src/two-branch/report/business-impact.ts

@@ -193,11 +193,7 @@ export function generateBusinessImpact(issues: EnrichedIssue[], groups: IssueGro
 
   const immediateRisk = blocking.length > 0 ? '🔴 High' : '🟢 Low';
 
-  // SESSION 13 FIX #3 + BUG #2 FIX: Detect if most/all blocking issues are auto-fixable
-  // This accounts for CheckStyle HIGH issues that are actually auto-fixable
-  // Note: totalFixCost (line 156) already includes auto-fix adjustments (lines 137-153)
-  // After fixing BUG #1 (severity classifier), most CheckStyle issues will be LOW,
-  // which eliminates the contradiction between "100% auto-fixable" and "high manual cost"
+  // SESSION 13 FIX #3: Detect if most/all blocking issues are auto-fixable
   const blockingAutoFixableGroups = autoFixableGroups.filter(g =>
     blocking.some(i => i.rule === g.rule && i.tool === g.tool && i.severity === g.severity)
   );

packages/agents/src/two-branch/report/metadata-footer.ts

@@ -81,15 +81,14 @@ export function generateAnalysisMetadata(
   // Add Agent Performance if available (optional)
   if (showAgentPerformance && metadata.agentPerformance && Array.isArray(metadata.agentPerformance) && metadata.agentPerformance.length > 0) {
     content += `\n### Agent Performance
-| Agent | Files Analyzed | Issues Found | Time | Cost | Model |
-|-------|----------------|--------------|------|------|-------|
+| Agent | Files Analyzed | Issues Found | Time | Cost |
+|-------|----------------|--------------|------|------|
 `;
     metadata.agentPerformance.forEach((agent: any) => {
       const issues = agent.issuesFound || agent.issues || 0;
       const time = agent.duration ? (agent.duration / 1000).toFixed(1) + 's' : 'N/A';
       const cost = agent.cost ? '$' + agent.cost.toFixed(4) : (issues === 0 ? 'N/A' : '$0.0000');
-      const model = agent.model || agent.modelName || 'N/A';
-      content += `| ${agent.name || agent.agent} | ${agent.filesAnalyzed || agent.files || 'N/A'} | ${issues} | ${time} | ${cost} | ${model} |\n`;
+      content += `| ${agent.name || agent.agent} | ${agent.filesAnalyzed || agent.files || 'N/A'} | ${issues} | ${time} | ${cost} |\n`;
     });
   }
 

packages/agents/src/two-branch/services/ai-severity-classifier.ts

@@ -84,36 +84,17 @@ IMPORTANT:
 - SpotBugs HIGH priority = usually HIGH or CRITICAL (actual bugs)
 - Semgrep security rules = usually HIGH or CRITICAL
 
-⚠️ CRITICAL: CHECKSTYLE RULES ARE ALWAYS LOW - NO EXCEPTIONS ⚠️
-
-ALL CheckStyle rules MUST be classified as LOW severity.
-CheckStyle ONLY detects style, formatting, and documentation issues.
-CheckStyle CANNOT detect security vulnerabilities or runtime bugs.
-
-DO NOT upgrade CheckStyle rules to HIGH/CRITICAL under ANY circumstances.
-If you see tool="checkstyle", ALWAYS return severity="low".
-
-Common CheckStyle rules (ALL LOW):
-- DesignForExtensionCheck → LOW (documentation/design guideline for extension)
-- LocalVariableNameCheck → LOW (naming convention - camelCase)
-- ParameterNameCheck → LOW (naming convention)
-- MemberNameCheck → LOW (naming convention)
-- MethodNameCheck → LOW (naming convention)
+CHECKSTYLE RULES (ALWAYS LOW unless security-related):
 - LineLengthCheck → LOW (line length is purely style)
 - JavadocPackageCheck → LOW (documentation is not runtime-critical)
 - JavadocMethodCheck → LOW (documentation preference)
-- JavadocVariableCheck → LOW (documentation preference)
 - MissingJavadocMethod → LOW (documentation preference)
 - IndentationCheck → LOW (formatting only)
 - WhitespaceAfter/Before → LOW (formatting only)
 - ImportOrder → LOW (import organization)
 - UnusedImports → LOW (cleanup, no runtime impact)
 - NeedBraces → LOW (style preference)
-- VisibilityModifierCheck → LOW (encapsulation guideline)
-- FinalParametersCheck → LOW (immutability guideline)
-- NewlineAtEndOfFileCheck → LOW (formatting convention)
-
-EXCEPTION: Only classify as HIGH if the rule detects actual security issues (rare)
+- EXCEPTION: Only classify as HIGH if the rule detects actual security issues (rare)
 
 Output ONLY this JSON structure:
 {
@@ -134,16 +115,6 @@ export async function classifyIssueSeverity(
   modelOverride?: string
 ): Promise<SeverityClassificationResult> {
 
-  // BUG #1 FIX: FORCE CheckStyle to LOW - no AI classification needed
-  // CheckStyle ONLY detects style/formatting/documentation - never security or bugs
-  if (input.tool.toLowerCase() === 'checkstyle') {
-    return {
-      severity: 'low',
-      reasoning: 'CheckStyle only detects style, formatting, and documentation issues (not runtime bugs or security vulnerabilities)',
-      confidence: 'high'
-    };
-  }
-
   // Build user prompt with issue details
   const userPrompt = buildClassificationPrompt(input);