Remove Issue Triage feature - liability concerns

Author: Shaurya Singh

src/ai/generator.ts

@@ -4,12 +4,11 @@ import {
   buildContributorQuickstartPrompt,
   buildReleaseSummaryPrompt,
   buildGoodFirstIssuesPrompt,
-  buildIssueTriagePrompt,
 } from './prompts';
 import { RepoSnapshot } from '../github/fetchers';
 
 export interface AnalysisOutput {
-  type: 'maintainer_brief' | 'contributor_quickstart' | 'release_summary' | 'good_first_issues' | 'issue_triage';
+  type: 'maintainer_brief' | 'contributor_quickstart' | 'release_summary' | 'good_first_issues';
   content: string;
   confidence: number;
   sources: {
@@ -20,7 +19,7 @@ export interface AnalysisOutput {
 }
 
 /**
- * Generates all 5 analysis outputs for a repository
+ * Generates all 4 analysis outputs for a repository
  */
 export async function generateAllOutputs(
   snapshot: RepoSnapshot,
@@ -34,16 +33,15 @@ export async function generateAllOutputs(
   const sources = {
     commits: snapshot.commits.slice(0, 50).map(c => c.sha),
     prs: snapshot.prs.slice(0, 30).map(pr => pr.number),
-    issues: [...snapshot.issues.map(i => i.number), ...(snapshot.allIssues || []).map(i => i.number)],
+    issues: snapshot.issues.map(i => i.number),
   };
 
   // Generate all outputs in parallel
-  const [maintainerBrief, contributorQuickstart, releaseSummary, goodFirstIssues, issueTriage] = await Promise.all([
+  const [maintainerBrief, contributorQuickstart, releaseSummary, goodFirstIssues] = await Promise.all([
     generateMaintainerBrief(snapshot, tone),
     generateContributorQuickstart(snapshot, tone),
     generateReleaseSummary(snapshot, tone),
     generateGoodFirstIssues(snapshot, tone),
-    generateIssueTriage(snapshot, tone),
   ]);
 
   return [
@@ -71,12 +69,6 @@ export async function generateAllOutputs(
       confidence: calculateConfidence(snapshot, 'good_first_issues'),
       sources,
     },
-    {
-      type: 'issue_triage',
-      content: issueTriage,
-      confidence: calculateConfidence(snapshot, 'issue_triage'),
-      sources,
-    },
   ];
 }
 
@@ -112,14 +104,6 @@ async function generateGoodFirstIssues(
   return await generateText(prompt, { maxTokens: tone === 'detailed' ? 2500 : 1500 });
 }
 
-async function generateIssueTriage(
-  snapshot: RepoSnapshot,
-  tone: 'concise' | 'detailed'
-): Promise<string> {
-  const prompt = buildIssueTriagePrompt(snapshot, tone);
-  return await generateText(prompt, { maxTokens: tone === 'detailed' ? 3500 : 2500 });
-}
-
 /**
  * Calculates confidence score based on data quality
  */
@@ -151,13 +135,6 @@ function calculateConfidence(
     if (snapshot.releases.length > 0) score += 0.05;
   }
 
-  if (outputType === 'issue_triage') {
-    const allIssues = snapshot.allIssues || [];
-    if (allIssues.length >= 20) score += 0.2;
-    else if (allIssues.length >= 10) score += 0.1;
-    else if (allIssues.length === 0) score -= 0.3;
-  }
-
   // Clamp to [0, 1]
   return Math.max(0, Math.min(1, score));
 }

src/ai/prompts.ts

@@ -425,165 +425,6 @@ IMPORTANT RULES:
 7. Prefer stable areas over high-churn areas for safety`;
 }
 
-/**
- * Builds prompt for Issue Triage Report
- */
-export function buildIssueTriagePrompt(
-  snapshot: RepoSnapshot,
-  tone: 'concise' | 'detailed'
-): string {
-  const allIssues = snapshot.allIssues || [];
-  
-  if (allIssues.length === 0) {
-    return `No open issues found in repository ${snapshot.repo.full_name}. Generate a brief message explaining that there are no issues to triage.`;
-  }
-
-  // Calculate issue age
-  const now = new Date();
-  const issuesWithAge = allIssues.map(issue => {
-    const created = new Date(issue.created_at);
-    const ageInDays = Math.floor((now.getTime() - created.getTime()) / (1000 * 60 * 60 * 24));
-    return { ...issue, ageInDays };
-  });
-
-  // Sort by reactions + comments (engagement) for prioritization hints
-  const sortedByEngagement = [...issuesWithAge].sort((a, b) => 
-    ((b.reactions || 0) + b.comments) - ((a.reactions || 0) + a.comments)
-  );
-
-  // Find stale issues (>90 days with no recent activity)
-  const staleIssues = issuesWithAge.filter(i => i.ageInDays > 90);
-
-  // Recent commit patterns to understand current focus
-  const recentCommitMessages = snapshot.commits.slice(0, 20)
-    .map(c => c.message.split('\n')[0])
-    .join('\n');
-
-  return `You are an expert project manager analyzing GitHub issues to help maintainers prioritize their backlog.
-
-Repository: ${snapshot.repo.full_name}
-Language: ${snapshot.repo.language || 'Multiple'}
-Total Open Issues: ${allIssues.length}
-Stale Issues (>90 days): ${staleIssues.length}
-
-===== ALL OPEN ISSUES =====
-
-${issuesWithAge.slice(0, 50).map(issue => `
-#${issue.number}: ${issue.title}
-- Labels: ${issue.labels.length > 0 ? issue.labels.join(', ') : 'none'}
-- Age: ${issue.ageInDays} days
-- Comments: ${issue.comments}
-- Reactions: ${issue.reactions || 0}
-- Author: @${issue.author || 'unknown'}
-${issue.body ? `- Description: ${issue.body.slice(0, 300)}${issue.body.length > 300 ? '...' : ''}` : '- No description'}
-`).join('\n---\n')}
-
-===== RECENT DEVELOPMENT CONTEXT =====
-
-Recent commits show the team is working on:
-${recentCommitMessages}
-
-Recent PRs:
-${snapshot.prs.slice(0, 10).map(pr => `- #${pr.number}: ${pr.title}`).join('\n')}
-
-===== HIGHEST ENGAGEMENT ISSUES =====
-
-Top issues by community interest (reactions + comments):
-${sortedByEngagement.slice(0, 5).map(i => `- #${i.number}: ${i.title} (${(i.reactions || 0) + i.comments} engagement)`).join('\n')}
-
-===== GENERATE THIS EXACT FORMAT =====
-
-# Issue Triage Report
-
-> **${allIssues.length} open issues analyzed** • Generated ${new Date().toLocaleDateString()}
-
----
-
-## 🔴 Critical Priority
-[Issues that are blocking, security-related, or affecting many users]
-[For each issue, explain WHY it's critical]
-
-${tone === 'detailed' ? 'Include 3-5 issues with full reasoning' : 'Include 2-3 issues'}
-
-Format for each:
-### #[number]: [title]
-**Why Critical:** [1-2 sentences explaining impact]
-**Suggested Action:** [What maintainer should do]
-**Labels to Add:** [Suggest appropriate labels if missing]
-
----
-
-## 🟠 High Priority
-[Important issues that should be addressed soon - feature requests with high demand, significant bugs]
-
-${tone === 'detailed' ? 'Include 4-6 issues' : 'Include 2-4 issues'}
-
-Format for each:
-### #[number]: [title]
-**Why Important:** [1 sentence]
-**Effort Estimate:** Low/Medium/High
-**Suggested Action:** [Brief action]
-
----
-
-## 🟡 Medium Priority
-[Good to fix but not urgent - smaller bugs, enhancements, documentation]
-
-${tone === 'detailed' ? 'Include 5-8 issues' : 'Include 3-5 issues'}
-
-Format for each:
-- **#[number]**: [title] — [1 sentence summary] • Effort: [Low/Med/High]
-
----
-
-## ⚪ Low Priority / Backlog
-[Nice to have, long-term ideas, minor improvements]
-
-${tone === 'detailed' ? 'Include remaining issues' : 'Include 3-5 issues'}
-
-Format: Brief list with issue numbers and titles
-
----
-
-## 🗑️ Recommend Closing
-[Issues that should be closed - stale with no activity, duplicates, won't fix, already resolved]
-
-${staleIssues.length > 0 ? `
-Found ${staleIssues.length} stale issues (>90 days old). Review these for closure:
-` : 'No obviously stale issues found.'}
-
-For each:
-- **#[number]**: [title] — **Reason:** [duplicate of #X / stale / wont fix / resolved]
-
----
-
-## 📊 Backlog Health Summary
-
-| Metric | Value |
-|--------|-------|
-| Total Open | ${allIssues.length} |
-| Critical | [count] |
-| Stale (>90d) | ${staleIssues.length} |
-| Unlabeled | [count issues with no labels] |
-| Avg Age | [calculate] days |
-
-**Recommendations:**
-1. [Top recommendation based on analysis]
-2. [Second recommendation]
-3. [Third recommendation]
-
----
-
-IMPORTANT RULES:
-1. Every issue you mention MUST use the exact issue number from the data
-2. Base priority on: user impact, engagement (reactions+comments), alignment with recent development
-3. Be specific about WHY each issue has its priority level
-4. If an issue seems like a duplicate, mention it
-5. Consider issue age - very old issues with no activity may need closure
-6. Look at labels to understand existing categorization
-7. Suggest missing labels where appropriate`;
-}
-
 /**
  * Helper: Analyze file churn
  */

src/analysis/pipeline.ts

@@ -11,7 +11,6 @@ import {
   extractTldr, 
   extractRiskyChanges, 
   extractSuggestedActions,
-  extractCriticalIssues,
   WeeklyBriefData 
 } from '../notifications/templates/weekly-brief';
 import db from '../db/client';
@@ -152,17 +151,15 @@ async function sendJobNotifications(
     if (settings.notify_email && userEmail) {
       console.log(`📧 Email notifications enabled, preparing to send to ${userEmail}...`);
 
-      // Fetch the maintainer_brief and issue_triage outputs for this job
-      const outputsResult = await db.query(
-        `SELECT type, content_markdown FROM analysis_outputs 
-         WHERE job_id = $1 AND type IN ('maintainer_brief', 'issue_triage')`,
+      // Fetch the maintainer_brief output for this job
+      const briefResult = await db.query(
+        `SELECT content_markdown FROM analysis_outputs 
+         WHERE job_id = $1 AND type = 'maintainer_brief'`,
         [jobId]
       );
 
-      const maintainerBrief = outputsResult.rows.find(r => r.type === 'maintainer_brief')?.content_markdown || '';
-      const issueTriage = outputsResult.rows.find(r => r.type === 'issue_triage')?.content_markdown || '';
+      const maintainerBrief = briefResult.rows[0]?.content_markdown || '';
       console.log(`📧 Maintainer brief content length: ${maintainerBrief.length} chars`);
-      console.log(`📧 Issue triage content length: ${issueTriage.length} chars`);
 
       // Build rich email with extracted content
       const briefData: WeeklyBriefData = {
@@ -177,7 +174,6 @@ async function sendJobNotifications(
         tldr: extractTldr(maintainerBrief),
         riskyChanges: extractRiskyChanges(maintainerBrief),
         suggestedActions: extractSuggestedActions(maintainerBrief),
-        criticalIssues: extractCriticalIssues(issueTriage),
         schedule: settings.schedule || 'weekly',
       };
 

src/github/fetchers.ts

@@ -46,8 +46,6 @@ export interface GitHubIssue {
   labels: string[];
   created_at: string;
   comments: number;
-  reactions?: number;
-  author?: string;
 }
 
 export interface GitHubRelease {
@@ -61,8 +59,7 @@ export interface RepoSnapshot {
   repo: GitHubRepo;
   commits: GitHubCommit[];
   prs: GitHubPR[];
-  issues: GitHubIssue[];  // Good first issues (legacy)
-  allIssues: GitHubIssue[];  // ALL open issues for triage
+  issues: GitHubIssue[];
   releases: GitHubRelease[];
   readme: string | null;
   contributing: string | null;
@@ -231,48 +228,6 @@ export async function fetchGoodFirstIssues(
   });
 }
 
-/**
- * Fetches ALL open issues for triage analysis
- */
-export async function fetchAllOpenIssues(
-  octokit: any,
-  owner: string,
-  repo: string,
-  limit: number = 100
-): Promise<GitHubIssue[]> {
-  const octo = normalizeOctokit(octokit);
-  
-  try {
-    const result = await githubApiCall(async () => {
-      return await octo.issues.listForRepo({
-        owner,
-        repo,
-        state: 'open',
-        sort: 'updated',
-        direction: 'desc',
-        per_page: limit,
-      });
-    });
-
-    return result.data
-      .filter((issue: any) => !issue.pull_request) // Exclude PRs
-      .map((issue: any) => ({
-        number: issue.number,
-        title: issue.title,
-        state: issue.state,
-        body: issue.body || null,
-        labels: issue.labels.map((l: any) => typeof l === 'string' ? l : l.name || ''),
-        created_at: issue.created_at,
-        comments: issue.comments,
-        reactions: issue.reactions?.total_count || 0,
-        author: issue.user?.login || 'Unknown',
-      }));
-  } catch (error) {
-    console.warn('Failed to fetch all issues:', error);
-    return [];
-  }
-}
-
 /**
  * Fetches recent releases
  */
@@ -366,14 +321,12 @@ export async function fetchRepoSnapshot(
   ignorePaths: string[] = []
 ): Promise<RepoSnapshot> {
   const commitLimit = depth === 'deep' ? 200 : 50;
-  const issueLimit = depth === 'deep' ? 100 : 50;
 
   const [
     repoData,
     commits,
     prs,
     issues,
-    allIssues,
     releases,
     readme,
     contributing,
@@ -382,7 +335,6 @@ export async function fetchRepoSnapshot(
     fetchRecentCommits(octokit, owner, repo, branch, commitLimit, ignorePaths),
     fetchRecentPRs(octokit, owner, repo, depth === 'deep' ? 60 : 30),
     fetchGoodFirstIssues(octokit, owner, repo),
-    fetchAllOpenIssues(octokit, owner, repo, issueLimit),
     fetchRecentReleases(octokit, owner, repo),
     fetchReadme(octokit, owner, repo),
     fetchContributing(octokit, owner, repo),
@@ -393,7 +345,6 @@ export async function fetchRepoSnapshot(
     commits,
     prs,
     issues,
-    allIssues,
     releases,
     readme,
     contributing,