fix(memory): prevent unbounded growth of navigations array in PageCollector

Author: flobo3

src/PageCollector.ts

@@ -409,5 +409,6 @@ export class NetworkCollector extends PageCollector<HTTPRequest> {
     } else {
       navigations.unshift([]);
     }
+    navigations.splice(3); // Hardcoded to match #maxNavigationSaved since it's private in base class
   }
 }

tests/PageCollector.test.ts

@@ -284,6 +284,26 @@ describe('NetworkCollector', () => {
     page.emit('request', request);
     assert.equal(collector.getData(page, true).length, 3);
   });
+
+  it('should not grow beyond maxNavigationSaved', async () => {
+    const collector = new NetworkCollector(browser);
+    await collector.init([page]);
+
+    // Simulate 5 navigations (maxNavigationSaved is 3)
+    for (let i = 0; i < 5; i++) {
+      const req = getMockRequest({
+        url: `http://example.com/nav${i}`,
+        isNavigationRequest: true,
+        frame: mainFrame,
+      });
+      page.emit('request', req);
+      page.emit('framenavigated', mainFrame);
+    }
+
+    // We expect 4 arrays in navigations (current + 3 saved)
+    // Each navigation has 1 request, so total should be 4
+    assert.equal(collector.getData(page, true).length, 4);
+  });
 });
 
 describe('ConsoleCollector', () => {