Handle null issues messages

Author: Natallia Harshunova

src/McpResponse.ts

@@ -252,10 +252,11 @@ export class McpResponse implements Response {
           ),
         };
       } else if (message instanceof AggregatedIssue) {
-        const result = mapIssueToMessageObject(message);
+        const mappedIssueMessage = mapIssueToMessageObject(message);
+        if (!mappedIssueMessage) throw new Error('Can\'t prpovide detals for the msgid ' + consoleMessageStableId);
         consoleData = {
           consoleMessageStableId,
-          ...result,
+          ...mappedIssueMessage,
         };
       } else {
         consoleData = {
@@ -310,10 +311,11 @@ export class McpResponse implements Response {
               };
             }
             if (item instanceof AggregatedIssue) {
-              const message = mapIssueToMessageObject(item);
+              const mappedIssueMessage = mapIssueToMessageObject(item);
+              if (!mappedIssueMessage) return null;
               return {
                 consoleMessageStableId,
-                ...message,
+                ...mappedIssueMessage,
               };
             }
             return {

tests/McpResponse.test.ts

@@ -10,6 +10,10 @@ import {tmpdir} from 'node:os';
 import {join} from 'node:path';
 import {describe, it} from 'node:test';
 
+import sinon from 'sinon';
+
+import { AggregatedIssue } from '../node_modules/chrome-devtools-frontend/mcp/mcp.js';
+
 import {
   getMockRequest,
   getMockResponse,
@@ -295,6 +299,46 @@ describe('McpResponse', () => {
       t.assert.snapshot?.(result[0].text);
     });
   });
+
+  it('doesn\'t list the issue message if mapping returns null', async () => {
+    await withBrowser(async (response, context) => {
+      const mockAggregatedIssue = sinon.createStubInstance(AggregatedIssue);
+          const mockDescription = {
+            file: 'not-existing-description-file.md',
+            links: [],
+          };
+      mockAggregatedIssue.getDescription.returns(mockDescription);
+      response.setIncludeConsoleData(true);
+      context.getConsoleData = () => {
+        return [mockAggregatedIssue];
+      };
+
+      const result = await response.handle('test', context);
+      const text = (result[0].text as string).toString();
+      assert.ok(text.includes('<no console messages found>'));
+    });
+  });
+
+  it('throws error if mapping returns null on get issue details', async () => {
+    await withBrowser(async (response, context) => {
+      const mockAggregatedIssue = sinon.createStubInstance(AggregatedIssue);
+          const mockDescription = {
+            file: 'not-existing-description-file.md',
+            links: [],
+          };
+      mockAggregatedIssue.getDescription.returns(mockDescription);
+      response.attachConsoleMessage(1);
+      context.getConsoleMessageById = () => {
+        return mockAggregatedIssue;
+      };
+
+      try {
+      await response.handle('test', context);
+      } catch (e) {
+        assert.ok(e.message.includes('Can\'t prpovide detals for the msgid 1'));
+      }
+    });
+  });
 });
 
 describe('McpResponse network request filtering', () => {