fix: address code review findings

- fetchUserSolutions: warn on truncated tree response
- fetchPRSubmissions: warn when 100-file cap is hit
- generateApproachAnalysis: wrap JSON.parse in try/catch
- progressBar: clamp filled to barLength with Math.min

Co-Authored-By: Soobin Bak <soobing@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 3 people

utils/learningComment.js

@@ -31,7 +31,7 @@ function progressBar(completed, total, barLength = 7) {
   if (total === 0) {
     return "□".repeat(barLength);
   }
-  const filled = Math.round((completed / total) * barLength);
+  const filled = Math.min(Math.round((completed / total) * barLength), barLength);
   return "■".repeat(filled) + "□".repeat(barLength - filled);
 }
 

utils/learningData.js

@@ -69,6 +69,12 @@ export async function fetchUserSolutions(
 
   const data = await response.json();
 
+  if (data.truncated) {
+    console.warn(
+      `[fetchUserSolutions] Tree response truncated for ${repoOwner}/${repoName}. Results may be incomplete.`
+    );
+  }
+
   // Pattern: {problem-name}/{username}.{ext}
   // The path must have exactly two segments and the filename must be username.ext
   const usernamePattern = new RegExp(
@@ -121,6 +127,12 @@ export async function fetchPRSubmissions(
 
   const files = await response.json();
 
+  if (files.length === 100) {
+    console.warn(
+      `[fetchPRSubmissions] PR #${prNumber} has 100+ files. Some submissions may be missed.`
+    );
+  }
+
   // Pattern: {problem-name}/{username}.{ext}
   const usernamePattern = new RegExp(
     `^([^/]+)/${escapeRegExp(username)}\\.[^/]+$`

utils/openai.js

@@ -244,7 +244,12 @@ ${truncatedContent}
     throw new Error("Empty response from OpenAI");
   }
 
-  const parsed = JSON.parse(content);
+  let parsed;
+  try {
+    parsed = JSON.parse(content);
+  } catch {
+    throw new Error(`OpenAI returned invalid JSON: ${content.slice(0, 200)}`);
+  }
 
   return {
     matches: parsed.matches === true,