fix: avoid reporting page close errors as errors

Author: OrKoN

src/McpContext.ts

@@ -134,7 +134,7 @@ export class McpContext implements Context {
   async closePage(pageIdx: number): Promise<void> {
     if (this.#pages.length === 1) {
       throw new Error(
-        'Unable to close the last page in the browser. It is fine to keep the last page open.',
+        'The last open page cannot be closed. It is fine to keep it open.',
       );
     }
     const page = this.getPageByIdx(pageIdx);

src/tools/pages.ts

@@ -58,7 +58,15 @@ export const closePage = defineTool({
       ),
   },
   handler: async (request, response, context) => {
-    await context.closePage(request.params.pageIdx);
+    try {
+      await context.closePage(request.params.pageIdx);
+    } catch (err) {
+      if (err.message.startsWith('The last open page cannot be closed.')) {
+        response.appendResponseLine(err.message);
+      } else {
+        throw err;
+      }
+    }
     response.setIncludePages(true);
   },
 });

tests/tools/pages.test.ts

@@ -56,15 +56,12 @@ describe('pages', () => {
     it('cannot close the last page', async () => {
       await withBrowser(async (response, context) => {
         const page = context.getSelectedPage();
-        try {
-          await closePage.handler({params: {pageIdx: 0}}, response, context);
-          assert.fail('not reached');
-        } catch (err) {
-          assert.strictEqual(
-            err.message,
-            'Unable to close the last page in the browser. It is fine to keep the last page open.',
-          );
-        }
+        await closePage.handler({params: {pageIdx: 0}}, response, context);
+        assert.deepStrictEqual(
+          response.responseLines[0],
+          `The last open page cannot be closed. It is fine to keep it open.`,
+        );
+        assert.ok(response.includePages);
         assert.ok(!page.isClosed());
       });
     });