Fix Copy Thread bug: use extractComments() instead of broken DOM traversal

claude[bot] · claude · claude[bot] · commit fb64fd4a8178 · 2026-02-11T09:08:50.000Z
The Copy Thread feature was failing because it tried to use a non-existent selector '#story_comments > ol.comments' and fell back to the main comments container which includes the comment form. This broke the DOM traversal. Fixed by: - Using the existing extractComments() function which correctly identifies comments - Building comment hierarchy from parent-child relationships in the extracted data - Removing dependency on the non-existent #story_comments selector 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/lobsters-bookmarklet.js b/lobsters-bookmarklet.js
@@ -164,29 +164,34 @@
     const storyAuthor = storyEl ? getAuthor(storyEl) : null;
     result.push(`[1] ${storyAuthor || 'Unknown'}: ${storyTitle}`);
 
-    function processLevel(parentEl, pathPrefix) {
-      const subtrees = parentEl.querySelectorAll(':scope > .comments_subtree');
+    // Use extractComments() to get all comments and build hierarchy
+    const comments = extractComments();
+    
+    // Build a map of comment ID to comment data for quick lookup
+    const commentMap = new Map();
+    comments.forEach(c => commentMap.set(c.id, c));
+    
+    // Build hierarchy by processing top-level comments and their children
+    function buildHierarchy(parentId = null, pathPrefix = '1') {
       let counter = 0;
-      subtrees.forEach(subtree => {
-        const comment = subtree.querySelector(':scope > .comment[id^="c_"]');
-        if (!comment) return;
+      
+      // Find all comments that are direct children of parentId
+      const children = comments.filter(c => c.parentId === parentId);
+      
+      children.forEach(comment => {
         counter++;
-        const path = `${pathPrefix}.${counter}`;
-        const author = getAuthor(comment) || 'Anonymous';
-        const textEl = comment.querySelector('.comment_text');
+        const path = parentId === null ? `${pathPrefix}.${counter}` : `${pathPrefix}.${counter}`;
+        const author = comment.author || 'Anonymous';
+        const textEl = comment.element.querySelector('.comment_text');
         const text = textEl ? textEl.textContent.trim().replace(/\s+/g, ' ') : '';
         result.push(`[${path}] ${author}: ${text}`);
-        const nestedOl = subtree.querySelector(':scope > ol');
-        if (nestedOl) {
-          processLevel(nestedOl, path);
-        }
+        
+        // Recursively process children of this comment
+        buildHierarchy(comment.id, path);
       });
     }
-
-    // The actual comment tree is inside #story_comments > ol.comments,
-    // not the top-level ol.comments which also contains the comment form
-    const actualComments = document.querySelector('#story_comments > ol.comments') || commentsContainer;
-    processLevel(actualComments, '1');
+    
+    buildHierarchy();
     return result.join('\n\n');
   }
 
