use boolean

Author: Lightning00Blade

src/McpContext.ts

@@ -128,7 +128,7 @@ export class McpContext implements Context {
     return context;
   }
 
-  getNetworkRequests(includePreviousNavigations?: number): HTTPRequest[] {
+  getNetworkRequests(includePreviousNavigations?: boolean): HTTPRequest[] {
     const page = this.getSelectedPage();
     return this.#networkCollector.getData(page, includePreviousNavigations);
   }

src/McpResponse.ts

@@ -40,7 +40,7 @@ export class McpResponse implements Response {
     include: boolean;
     pagination?: PaginationOptions;
     resourceTypes?: ResourceType[];
-    includePreviousNavigations?: number;
+    includePreviousNavigations?: boolean;
   };
   #consoleDataOptions?: {
     include: boolean;
@@ -61,7 +61,7 @@ export class McpResponse implements Response {
     value: boolean,
     options?: PaginationOptions & {
       resourceTypes?: ResourceType[];
-      includePreviousNavigations?: number;
+      includePreviousNavigations?: boolean;
     },
   ): void {
     if (!value) {

src/PageCollector.ts

@@ -133,14 +133,18 @@ export class PageCollector<T> {
     this.storage.delete(page);
   }
 
-  getData(page: Page, includePreviousNavigations?: number): T[] {
+  getData(page: Page, includePreviousNavigations?: boolean): T[] {
     const navigations = this.storage.get(page);
     if (!navigations) {
       return [];
     }
 
+    if (!includePreviousNavigations) {
+      return navigations[0];
+    }
+
     const data: T[] = [];
-    for (let index = includePreviousNavigations ?? 0; index >= 0; index--) {
+    for (let index = this.#maxNavigationSaved; index >= 0; index--) {
       data.push(...navigations[index]);
     }
     return data;

src/tools/ToolDefinition.ts

@@ -50,7 +50,7 @@ export interface Response {
     value: boolean,
     options?: PaginationOptions & {
       resourceTypes?: string[];
-      includePreviousNavigations?: number;
+      includePreviousNavigations?: boolean;
     },
   ): void;
   setIncludeConsoleData(

src/tools/network.ts

@@ -64,10 +64,8 @@ export const listNetworkRequests = defineTool({
         'Filter requests to only return requests of the specified resource types. When omitted or empty, returns all requests.',
       ),
     includePreviousNavigations: zod
-      .number()
-      .int()
-      .positive()
-      .max(2)
+      .boolean()
+      .default(false)
       .optional()
       .describe(
         'Number of previous navigations to include. Current navigation is always included.',

tests/tools/network.test.js.snapshot

@@ -8,7 +8,8 @@ reqid=3 GET data:text/html,<div>Hello 3</div> [success - 200]
 exports[`network > network_list_requests > list requests from previous navigations 1`] = `
 # list_request response
 ## Network requests
-Showing 1-2 of 2 (Page 1 of 1).
+Showing 1-3 of 3 (Page 1 of 1).
+reqid=1 GET data:text/html,<div>Hello 1</div> [success - 200]
 reqid=2 GET data:text/html,<div>Hello 2</div> [success - 200]
 reqid=3 GET data:text/html,<div>Hello 3</div> [success - 200]
 `;

tests/tools/network.test.ts

@@ -49,7 +49,7 @@ describe('network', () => {
         await listNetworkRequests.handler(
           {
             params: {
-              includePreviousNavigations: 1,
+              includePreviousNavigations: true,
             },
           },
           response,