fix: correct navigate_page_history error handling

Per puppeteer/puppeteer#14160, null return from goBack/goForward
indicates successful navigation without network request, not failure.
Puppeteer throws error when no history exists.

Simplified to append error.message as suggested by @OrKoN.

Author: mithun50

src/tools/pages.ts

@@ -144,32 +144,15 @@ export const navigatePageHistory = defineTool({
     };
 
     try {
-      let result;
       if (request.params.navigate === 'back') {
-        result = await page.goBack(options);
+        await page.goBack(options);
       } else {
-        result = await page.goForward(options);
-      }
-
-      // If result is null, navigation wasn't possible (no history)
-      if (result === null) {
-        const direction =
-          request.params.navigate === 'back' ? 'previous' : 'next';
-        response.appendResponseLine(
-          `Cannot navigate ${request.params.navigate}, no ${direction} page in history.`,
-        );
+        await page.goForward(options);
       }
     } catch (error) {
-      // Provide more specific error messages based on the error type
-      if (error.message && error.message.includes('timeout')) {
-        response.appendResponseLine(
-          `Navigation ${request.params.navigate} timed out after ${request.params.timeout || 30000}ms.`,
-        );
-      } else {
-        response.appendResponseLine(
-          `Unable to navigate ${request.params.navigate}: ${error.message || 'Unknown error occurred'}`,
-        );
-      }
+      response.appendResponseLine(
+        `Unable to navigate ${request.params.navigate} in currently selected page. ${error.message}`,
+      );
     }
 
     response.setIncludePages(true);

tests/tools/pages.test.ts

@@ -163,9 +163,10 @@ describe('pages', () => {
           context,
         );
 
-        assert.equal(
-          response.responseLines.at(0),
-          'Unable to navigate forward in currently selected page.',
+        assert.ok(
+          response.responseLines
+            .at(0)
+            ?.startsWith('Unable to navigate forward in currently selected page.'),
         );
         assert.ok(response.includePages);
       });
@@ -178,9 +179,10 @@ describe('pages', () => {
           context,
         );
 
-        assert.equal(
-          response.responseLines.at(0),
-          'Unable to navigate back in currently selected page.',
+        assert.ok(
+          response.responseLines
+            .at(0)
+            ?.startsWith('Unable to navigate back in currently selected page.'),
         );
         assert.ok(response.includePages);
       });