chore: eslint errors

Author: devnook

eslint.config.mjs

@@ -37,6 +37,7 @@ export default defineConfig([
             'puppeteer.config.cjs',
             'eslint.config.mjs',
             'rollup.config.mjs',
+            'skills/memory-leak-debugging/references/compare_snapshots.js',
           ],
         },
       },

skills/memory-leak-debugging/SKILL.md

@@ -11,7 +11,7 @@ This skill provides expert guidance and workflows for finding, diagnosing, and f
 
 - **Prefer `memlab`:** Do NOT attempt to read raw `.heapsnapshot` files directly, as they are extremely large and will consume too many tokens. Always recommend and use `memlab` to process snapshots and identify leak traces.
 - **Isolate the Leak:** Determine if the leak is in the browser (client-side) or Node.js (server-side).
-- **Common Culprits:** Look for detached DOM nodes, unhandled closures, global variables, event listeners not being removed, and caches growing unbounded. *Note: Detached DOM nodes are sometimes intentional caches; always ask the user before nulling them.*
+- **Common Culprits:** Look for detached DOM nodes, unhandled closures, global variables, event listeners not being removed, and caches growing unbounded. _Note: Detached DOM nodes are sometimes intentional caches; always ask the user before nulling them._
 
 ## Workflows
 
@@ -42,7 +42,9 @@ When you have found a leak trace (e.g., via `memlab` output), you must identify
 If `memlab` is not available, you MUST use the fallback script in the references directory to compare two `.heapsnapshot` files and identify the top growing objects and common leak types.
 
 Run the script using Node.js:
+
 ```bash
 node compare_snapshots.js <baseline.heapsnapshot> <target.heapsnapshot>
 ```
+
 The script will analyze and output the top growing objects by size and highlight the 3 most common types of memory leaks (e.g., Detached DOM nodes, closures, Contexts) if they are present.

skills/memory-leak-debugging/references/compare_snapshots.js

@@ -1,4 +1,10 @@
-const fs = require('fs');
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+import * as fs from 'node:fs';
 
 function parseSnapshot(filePath) {
   console.log(`Loading ${filePath}...`);
@@ -7,36 +13,44 @@ function parseSnapshot(filePath) {
   const nodes = data.nodes;
   const nodeFields = data.snapshot.meta.node_fields;
   const nodeFieldCount = nodeFields.length;
-  
+
   const typeOffset = nodeFields.indexOf('type');
   const nameOffset = nodeFields.indexOf('name');
   const sizeOffset = nodeFields.indexOf('self_size');
-  
+
   const nodeTypes = data.snapshot.meta.node_types[typeOffset];
-  
+
   const counts = {};
   const sizes = {};
-  
+
   for (let i = 0; i < nodes.length; i += nodeFieldCount) {
     const typeIdx = nodes[i + typeOffset];
     const typeName = nodeTypes[typeIdx];
     const nameIdx = nodes[i + nameOffset];
     const name = typeof nameIdx === 'number' ? strings[nameIdx] : nameIdx;
     const size = nodes[i + sizeOffset];
-    
+
     // Ignore native primitives/arrays that clutter the output unless specifically looking for them
-    if (typeName === 'string' || typeName === 'number' || typeName === 'array') continue;
-    
+    if (
+      typeName === 'string' ||
+      typeName === 'number' ||
+      typeName === 'array'
+    ) {
+      continue;
+    }
+
     const key = `${typeName}::${name}`;
     counts[key] = (counts[key] || 0) + 1;
     sizes[key] = (sizes[key] || 0) + size;
   }
-  return { counts, sizes };
+  return {counts, sizes};
 }
 
-const [,, file1, file2] = process.argv;
+const [, , file1, file2] = process.argv;
 if (!file1 || !file2) {
-  console.error('Usage: node compare_snapshots.js <baseline.heapsnapshot> <target.heapsnapshot>');
+  console.error(
+    'Usage: node compare_snapshots.js <baseline.heapsnapshot> <target.heapsnapshot>',
+  );
   process.exit(1);
 }
 
@@ -50,12 +64,12 @@ try {
     const count2 = snap2.counts[key];
     const size1 = snap1.sizes[key] || 0;
     const size2 = snap2.sizes[key];
-    
+
     if (count2 > count1) {
       diffs.push({
         key,
         countDiff: count2 - count1,
-        sizeDiff: size2 - size1
+        sizeDiff: size2 - size1,
       });
     }
   }
@@ -68,12 +82,13 @@ try {
   });
 
   // Look for common leak indicators
-  const commonLeaks = diffs.filter(d => 
-    d.key.toLowerCase().includes('detached') || 
-    d.key.toLowerCase().includes('html') || 
-    d.key.toLowerCase().includes('eventlistener') ||
-    d.key.toLowerCase().includes('context') ||
-    d.key.toLowerCase().includes('closure')
+  const commonLeaks = diffs.filter(
+    d =>
+      d.key.toLowerCase().includes('detached') ||
+      d.key.toLowerCase().includes('html') ||
+      d.key.toLowerCase().includes('eventlistener') ||
+      d.key.toLowerCase().includes('context') ||
+      d.key.toLowerCase().includes('closure'),
   );
 
   commonLeaks.sort((a, b) => b.countDiff - a.countDiff);
@@ -87,6 +102,8 @@ try {
     });
   }
 } catch (error) {
-  console.error('Error parsing snapshots. They might be too large for JSON.parse or invalid.');
+  console.error(
+    'Error parsing snapshots. They might be too large for JSON.parse or invalid.',
+  );
   console.error(error.message);
-}
+}