Optimize queue fetching to prevent choppy playback with large queues

- Add skipQueue parameter to getNowPlaying() function
- Skip queue fetching during status polling (every 2s) to avoid choppy playback
- Queue is only fetched when needed (e.g., admin UI showing next tracks)
- Fixes issue with choppy playback when queue has 400+ tracks
- Minor formatting/whitespace cleanup in other files

Author: htilly

.github/agent/agent.mjs

@@ -1229,48 +1229,48 @@ This is ONE FILE ONLY. Generate the complete diff now:`;
 if (!filesToModify || allDiffs.length === 0) {
   console.log(`[AGENT] Using single-pass generation method...`);
   const apiStartTime = Date.now();
-  try {
-    output = await retryWithBackoff(async () => {
-      return await callAI(prompt);
-    }, 3, 1000);
-    const apiDuration = Date.now() - apiStartTime;
-    console.log(`[AGENT] API call completed in ${apiDuration}ms`);
-  } catch (error) {
-    const errorDetails = formatErrorDetails(error, {});
-    await handleError(error, `${provider.toUpperCase()} API Error`, { errorDetails });
-    // handleError calls process.exit(1), so we never reach here
+try {
+  output = await retryWithBackoff(async () => {
+    return await callAI(prompt);
+  }, 3, 1000);
+  const apiDuration = Date.now() - apiStartTime;
+  console.log(`[AGENT] API call completed in ${apiDuration}ms`);
+} catch (error) {
+  const errorDetails = formatErrorDetails(error, {});
+  await handleError(error, `${provider.toUpperCase()} API Error`, { errorDetails });
+  // handleError calls process.exit(1), so we never reach here
   }
 }
 
 // Extract diff from potential markdown code blocks (only if using fallback method)
 let diff = output.trim();
 if (!filesToModify || allDiffs.length === 0) {
   // Only do markdown extraction for fallback single-pass method
-  if (output.includes("```")) {
-    // Try to extract content between code fences
-    // Handle both single and multiple code blocks
-    const matches = output.matchAll(/```(?:diff)?\n([\s\S]*?)```/g);
-    const extractedDiffs = [];
-    for (const match of matches) {
-      extractedDiffs.push(match[1].trim());
-    }
-    // Use the longest extracted diff (likely the actual diff)
-    if (extractedDiffs.length > 0) {
-      diff = extractedDiffs.reduce((a, b) => a.length > b.length ? a : b);
-    }
-    
-    // If no code blocks found but output contains diff markers, use the whole output
-    if (!diff.includes("--- a/") && output.includes("--- a/")) {
-      // Extract everything after the first "--- a/" line
-      const diffStart = output.indexOf("--- a/");
-      diff = output.substring(diffStart).trim();
-      // Remove any trailing markdown or explanations
-      const diffEnd = diff.indexOf("\n\n```") !== -1 ? diff.indexOf("\n\n```") : 
-                      diff.indexOf("\n\n##") !== -1 ? diff.indexOf("\n\n##") :
-                      diff.indexOf("\n\n**") !== -1 ? diff.indexOf("\n\n**") :
-                      diff.length;
-      diff = diff.substring(0, diffEnd).trim();
-    }
+if (output.includes("```")) {
+  // Try to extract content between code fences
+  // Handle both single and multiple code blocks
+  const matches = output.matchAll(/```(?:diff)?\n([\s\S]*?)```/g);
+  const extractedDiffs = [];
+  for (const match of matches) {
+    extractedDiffs.push(match[1].trim());
+  }
+  // Use the longest extracted diff (likely the actual diff)
+  if (extractedDiffs.length > 0) {
+    diff = extractedDiffs.reduce((a, b) => a.length > b.length ? a : b);
+  }
+  
+  // If no code blocks found but output contains diff markers, use the whole output
+  if (!diff.includes("--- a/") && output.includes("--- a/")) {
+    // Extract everything after the first "--- a/" line
+    const diffStart = output.indexOf("--- a/");
+    diff = output.substring(diffStart).trim();
+    // Remove any trailing markdown or explanations
+    const diffEnd = diff.indexOf("\n\n```") !== -1 ? diff.indexOf("\n\n```") : 
+                    diff.indexOf("\n\n##") !== -1 ? diff.indexOf("\n\n##") :
+                    diff.indexOf("\n\n**") !== -1 ? diff.indexOf("\n\n**") :
+                    diff.length;
+    diff = diff.substring(0, diffEnd).trim();
+  }
   }
 } else {
   // For one-file-at-a-time, diff is already clean (output from combined diffs)
@@ -1372,9 +1372,9 @@ if (validationResult.errors.length > 0 || diff.match(/\+\+\+ b\/[^\n]*$/m)) {
       }
 
       const errorMsg = `${truncationReason}. The AI model may have generated an incomplete diff.\n\nErrors:\n${validationResult.errors.join('\n')}\n\nAPI Response Info:\n- Stop reason: ${stopReason}\n- Output tokens used: ${outputTokens}\n- Was truncated: ${wasTruncated}\n\nDiff preview (last 500 chars):\n\`\`\`\n${diff.substring(Math.max(0, diffLength - 500))}\n\`\`\`\n\nPossible causes:\n- Model hit output token limit (check max_tokens setting)\n- Input context too large, leaving insufficient room for output\n- Model stopped generating for other reasons\n\nSuggestions:\n- Reduce input context size (fewer files) - already reduced to 400KB\n- Break task into smaller parts\n- Verify max_tokens is sufficient (currently 180K)`;
-      const errorDetails = formatErrorDetails(new Error(errorMsg), { diff: diff.substring(Math.max(0, diffLength - 1000)), files: validationResult.stats.filesChanged });
+    const errorDetails = formatErrorDetails(new Error(errorMsg), { diff: diff.substring(Math.max(0, diffLength - 1000)), files: validationResult.stats.filesChanged });
       await handleError(new Error(errorMsg), "Incomplete Diff (Truncated)", { diff: diff.substring(Math.max(0, diffLength - 1000)), errorDetails });
-      // handleError calls process.exit(1), so we never reach here
+    // handleError calls process.exit(1), so we never reach here
     }
   }
 }

app.manifest.json

@@ -56,3 +56,5 @@
 
 
 
+
+

docs/DISCORD_DISCOVERY_CONTENT.md

@@ -109,3 +109,5 @@ Join our Discord server for:
 
 
 
+
+

index.js

@@ -2340,15 +2340,15 @@ async function getNowPlaying(options = {}) {
     // Only fetch queue if explicitly needed (skip for status polling to avoid choppy playback)
     if (!skipQueue) {
       promises.push(
-        sonos.getQueue().catch(err => {
-          // Ignore queue errors for now-playing
-          return null;
-        })
+      sonos.getQueue().catch(err => {
+        // Ignore queue errors for now-playing
+        return null;
+      })
       );
     } else {
       promises.push(Promise.resolve(null));
     }
-    
+
     const [state, volume, queue] = await Promise.all(promises);
 
     // Fetch queue (next tracks) - only if we have queue data

lib/slack-validator.js

@@ -99,3 +99,5 @@ module.exports = {
 
 
 
+
+

lib/sonos-discovery.js

@@ -147,3 +147,5 @@ module.exports = {
 
 
 
+
+

lib/spotify-validator.js

@@ -63,3 +63,5 @@ module.exports = {
 
 
 
+
+