fix: ignore unmapped PerformanceIssue events

Author: officialasishkumar

src/PageCollector.ts

@@ -342,6 +342,11 @@ class PageEventSubscriber {
   #onIssueAdded = (data: Protocol.Audits.IssueAddedEvent) => {
     try {
       const inspectorIssue = data.issue;
+      // DevTools currently defines this protocol issue code but has no
+      // IssuesManager handler for it, so calling into the mapper only warns.
+      if (String(inspectorIssue.code) === 'PerformanceIssue') {
+        return;
+      }
       const issue = DevTools.createIssuesFromProtocolIssue(
         null,
         // @ts-expect-error Protocol types diverge.

tests/PageCollector.test.ts

@@ -385,6 +385,37 @@ describe('ConsoleCollector', () => {
     assert.equal(data.length, 2);
   });
 
+  it('silently ignores unmapped PerformanceIssue events', async () => {
+    const browser = getMockBrowser();
+    const page = (await browser.pages())[0];
+    // @ts-expect-error internal API.
+    const cdpSession = page._client();
+    const warnStub = sinon.stub(console, 'warn');
+
+    const collector = new ConsoleCollector(browser, collect => {
+      return {
+        issue: issue => {
+          collect(issue as DevTools.AggregatedIssue);
+        },
+      } as ListenerMap;
+    });
+    await collector.init([page]);
+
+    const performanceIssue = {
+      code: 'PerformanceIssue',
+      details: {
+        performanceIssueDetails: {
+          performanceIssueType: 'DocumentCookie',
+        },
+      },
+    } as unknown as Protocol.Audits.InspectorIssue;
+
+    cdpSession.emit('Audits.issueAdded', {issue: performanceIssue});
+
+    assert.equal(collector.getData(page).length, 0);
+    sinon.assert.notCalled(warnStub);
+  });
+
   it('filters duplicated issues', async () => {
     const browser = getMockBrowser();
     const page = (await browser.pages())[0];