longer summary context and indicator when not full context · ellioth.co/summarizer-extension@c4fb588

+19

popup/popup.css

···

+155 -91

popup/popup.js

··································································

+7 -1

scripts/background.js

··· 6 6 const CONTENT_CACHE_PREFIX = "content_cache_"; 7 7 const CHAT_CACHE_PREFIX = "chat_cache_"; 8 8 9 + // ── Prompt templates ───────────────────────────────────────── 10 + const OLLAMA_CONTEXT_TEMPLATE = "Context:\n${context}\n\nUser: ${userMessage}"; 11 + const OLLAMA_SINGLE_MESSAGE_TEMPLATE = "${userMessage}"; 12 + 9 13 chrome.runtime.onInstalled.addListener(() => { 10 14 // Set default settings only if they don't already exist 11 15 chrome.storage.sync.get(["apiMode"]).then((result) => { ··· 144 148 .slice(0, -1) 145 149 .map((m) => `${m.role}: ${m.content}`) 146 150 .join("\n"); 147 - prompt = `Context:\n${context}\n\nUser: ${lastUserMsg?.content || ""}`; 151 + prompt = OLLAMA_CONTEXT_TEMPLATE 152 + .replace("${context}", context) 153 + .replace("${userMessage}", lastUserMsg?.content || ""); 148 154 } else { 149 155 prompt = lastUserMsg?.content || ""; 150 156 }

+49 -12

scripts/content.js

··· 18 18 'nav', 'aside', 'form', 'button', 'input' 19 19 ]; 20 20 21 + // Maximum characters to extract (increased to capture more content) 22 + const MAX_EXTRACTION_LENGTH = 50000; 23 + 21 24 function extractText() { 22 25 // Extract text with structure 23 26 let extractedText = ''; 27 + let wasTruncated = false; 24 28 25 29 // Get title 26 30 const title = document.title || ''; ··· 37 41 // Extract content from body, skipping noise elements 38 42 extractedText += extractTextFromElement(document.body); 39 43 40 - // Clean up the text 41 - extractedText = cleanText(extractedText); 44 + // Check if content exceeds limit before cleaning 45 + const originalLength = extractedText.length; 46 + 47 + // Clean up the text (without truncating yet) 48 + extractedText = cleanText(extractedText, false); 49 + 50 + // Track if we need to truncate 51 + if (extractedText.length > MAX_EXTRACTION_LENGTH) { 52 + wasTruncated = true; 53 + extractedText = extractedText.substring(0, MAX_EXTRACTION_LENGTH); 54 + } 42 55 43 56 // Fallback: if we got very little content, try brute force extraction 44 57 if (extractedText.length < 1000) { 45 - const fallbackText = extractTextFallback(); 58 + const fallbackResult = extractTextFallback(); 59 + const fallbackText = fallbackResult.text; 46 60 if (fallbackText.length > extractedText.length) { 47 61 extractedText = `Title: ${title}\n\n${fallbackText}`; 62 + wasTruncated = fallbackResult.wasTruncated; 48 63 } 49 64 } 50 65 51 - return extractedText; 66 + return { text: extractedText, wasTruncated }; 52 67 } 53 68 54 69 function extractTextFallback() { ··· 61 76 ]; 62 77 63 78 let text = ''; 79 + let wasTruncated = false; 64 80 const seen = new Set(); 65 81 66 82 for (const selector of selectors) { ··· 77 93 78 94 seen.add(content.substring(0, 100)); 79 95 text += content + '\n\n'; 96 + 97 + // Check if we're approaching the limit 98 + if (text.length > MAX_EXTRACTION_LENGTH) { 99 + wasTruncated = true; 100 + break; 101 + } 80 102 } 81 103 } catch (e) { 82 104 // Ignore invalid selectors 83 105 } 106 + if (wasTruncated) break; 84 107 } 85 108 86 109 // Last resort: get all paragraphs on the page 87 - if (text.length < 500) { 110 + if (text.length < 500 && !wasTruncated) { 88 111 const allParagraphs = document.querySelectorAll('p'); 89 112 for (const p of allParagraphs) { 90 113 const content = p.textContent.trim(); ··· 94 117 95 118 seen.add(content.substring(0, 100)); 96 119 text += content + '\n\n'; 120 + 121 + if (text.length > MAX_EXTRACTION_LENGTH) { 122 + wasTruncated = true; 123 + break; 124 + } 97 125 } 98 126 } 99 127 } 100 128 101 - return text.substring(0, 15000); 129 + return { text: text.substring(0, MAX_EXTRACTION_LENGTH), wasTruncated }; 102 130 } 103 131 104 132 function shouldSkipElement(el) { ··· 296 324 return text; 297 325 } 298 326 299 - function cleanText(text) { 300 - return text 327 + function cleanText(text, shouldTruncate = true) { 328 + let cleaned = text 301 329 .replace(/[^\S\n]+/g, ' ') // Collapse spaces/tabs but preserve newlines 302 330 .replace(/\n{3,}/g, '\n\n') // Collapse 3+ newlines to 2 303 - .replace(/^\s+|\s+$/g, '') // Trim 304 - .substring(0, 15000); // Limit length 331 + .replace(/^\s+|\s+$/g, ''); // Trim 332 + 333 + // Only truncate if explicitly requested (used for final output) 334 + if (shouldTruncate && cleaned.length > MAX_EXTRACTION_LENGTH) { 335 + cleaned = cleaned.substring(0, MAX_EXTRACTION_LENGTH); 336 + } 337 + 338 + return cleaned; 305 339 } 306 340 307 341 // Listen for messages from popup 308 342 chrome.runtime.onMessage.addListener((request, sender, sendResponse) => { 309 343 if (request.action === 'extract') { 310 - const content = extractText(); 311 - sendResponse({ content }); 344 + const result = extractText(); 345 + sendResponse({ 346 + content: result.text, 347 + wasTruncated: result.wasTruncated 348 + }); 312 349 } 313 350 return true; 314 351 });

Configure Feed

Configure Feed