refactor: use different format for reqid (#380)

Author: Lightning00Blade

src/formatters/networkFormatter.ts

@@ -15,7 +15,7 @@ export function getShortDescriptionForRequest(
   id: number,
 ): string {
   // TODO truncate the URL
-  return `reqid ${id} - ${request.url()} ${request.method()} ${getStatusFromRequest(request)}`;
+  return `reqid=${id} ${request.method()} ${request.url()} ${getStatusFromRequest(request)}`;
 }
 
 export function getStatusFromRequest(request: HTTPRequest): string {

tests/McpResponse.test.ts

@@ -210,8 +210,8 @@ Call handle_dialog to handle it before continuing.`,
         `# test response
 ## Network requests
 Showing 1-2 of 2 (Page 1 of 1).
-reqid 1 - http://example.com GET [pending]
-reqid 2 - http://example.com GET [pending]`,
+reqid=1 GET http://example.com [pending]
+reqid=2 GET http://example.com [pending]`,
       );
     });
   });
@@ -267,7 +267,7 @@ ${JSON.stringify({request: 'body'})}
 ${JSON.stringify({response: 'body'})}
 ## Network requests
 Showing 1-1 of 1 (Page 1 of 1).
-reqid 1 - http://example.com POST [success - 200]`,
+reqid=1 POST http://example.com [success - 200]`,
       );
     });
   });
@@ -290,7 +290,7 @@ Status:  [pending]
 - content-size:10
 ## Network requests
 Showing 1-1 of 1 (Page 1 of 1).
-reqid 1 - http://example.com GET [pending]`,
+reqid=1 GET http://example.com [pending]`,
       );
     });
   });
@@ -354,8 +354,8 @@ describe('McpResponse network request filtering', () => {
         `# test response
 ## Network requests
 Showing 1-2 of 2 (Page 1 of 1).
-reqid 1 - http://example.com GET [pending]
-reqid 1 - http://example.com GET [pending]`,
+reqid=1 GET http://example.com [pending]
+reqid=1 GET http://example.com [pending]`,
       );
     });
   });
@@ -378,7 +378,7 @@ reqid 1 - http://example.com GET [pending]`,
         `# test response
 ## Network requests
 Showing 1-1 of 1 (Page 1 of 1).
-reqid 1 - http://example.com GET [pending]`,
+reqid=1 GET http://example.com [pending]`,
       );
     });
   });
@@ -423,11 +423,11 @@ No requests found.`,
         `# test response
 ## Network requests
 Showing 1-5 of 5 (Page 1 of 1).
-reqid 1 - http://example.com GET [pending]
-reqid 1 - http://example.com GET [pending]
-reqid 1 - http://example.com GET [pending]
-reqid 1 - http://example.com GET [pending]
-reqid 1 - http://example.com GET [pending]`,
+reqid=1 GET http://example.com [pending]
+reqid=1 GET http://example.com [pending]
+reqid=1 GET http://example.com [pending]
+reqid=1 GET http://example.com [pending]
+reqid=1 GET http://example.com [pending]`,
       );
     });
   });
@@ -452,11 +452,11 @@ reqid 1 - http://example.com GET [pending]`,
         `# test response
 ## Network requests
 Showing 1-5 of 5 (Page 1 of 1).
-reqid 1 - http://example.com GET [pending]
-reqid 1 - http://example.com GET [pending]
-reqid 1 - http://example.com GET [pending]
-reqid 1 - http://example.com GET [pending]
-reqid 1 - http://example.com GET [pending]`,
+reqid=1 GET http://example.com [pending]
+reqid=1 GET http://example.com [pending]
+reqid=1 GET http://example.com [pending]
+reqid=1 GET http://example.com [pending]
+reqid=1 GET http://example.com [pending]`,
       );
     });
   });

tests/formatters/networkFormatter.test.ts

@@ -23,20 +23,20 @@ describe('networkFormatter', () => {
       const request = getMockRequest();
       const result = getShortDescriptionForRequest(request, 1);
 
-      assert.equal(result, 'reqid 1 - http://example.com GET [pending]');
+      assert.equal(result, 'reqid=1 GET http://example.com [pending]');
     });
     it('shows correct method', async () => {
       const request = getMockRequest({method: 'POST'});
       const result = getShortDescriptionForRequest(request, 1);
 
-      assert.equal(result, 'reqid 1 - http://example.com POST [pending]');
+      assert.equal(result, 'reqid=1 POST http://example.com [pending]');
     });
     it('shows correct status for request with response code in 200', async () => {
       const response = getMockResponse();
       const request = getMockRequest({response});
       const result = getShortDescriptionForRequest(request, 1);
 
-      assert.equal(result, 'reqid 1 - http://example.com GET [success - 200]');
+      assert.equal(result, 'reqid=1 GET http://example.com [success - 200]');
     });
     it('shows correct status for request with response code in 100', async () => {
       const response = getMockResponse({
@@ -45,7 +45,7 @@ describe('networkFormatter', () => {
       const request = getMockRequest({response});
       const result = getShortDescriptionForRequest(request, 1);
 
-      assert.equal(result, 'reqid 1 - http://example.com GET [failed - 199]');
+      assert.equal(result, 'reqid=1 GET http://example.com [failed - 199]');
     });
     it('shows correct status for request with response code above 200', async () => {
       const response = getMockResponse({
@@ -54,7 +54,7 @@ describe('networkFormatter', () => {
       const request = getMockRequest({response});
       const result = getShortDescriptionForRequest(request, 1);
 
-      assert.equal(result, 'reqid 1 - http://example.com GET [failed - 300]');
+      assert.equal(result, 'reqid=1 GET http://example.com [failed - 300]');
     });
     it('shows correct status for request that failed', async () => {
       const request = getMockRequest({
@@ -68,7 +68,7 @@ describe('networkFormatter', () => {
 
       assert.equal(
         result,
-        'reqid 1 - http://example.com GET [failed - Error in Network]',
+        'reqid=1 GET http://example.com [failed - Error in Network]',
       );
     });
   });