Move extension test to fixtures

Natallia Harshunova · Natallia Harshunova · commit 3e0d5fa44993 · 2026-01-15T16:34:10.000Z
diff --git a/docs/tool-reference.md b/docs/tool-reference.md
@@ -283,12 +283,12 @@ so returned values have to JSON-serializable.
 **Parameters:**
 
 - **function** (string) **(required)**: A JavaScript function declaration to be executed by the tool in the currently selected page.
-Example without arguments: `() => {
+  Example without arguments: `() => {
   return document.title
 }` or `async () => {
   return await fetch("example.com")
 }`.
-Example with arguments: `(el) => {
+  Example with arguments: `(el) => {
   return el.innerText;
 }`
 
diff --git a/scripts/generate-docs.ts b/scripts/generate-docs.ts
@@ -278,15 +278,15 @@ async function generateToolDocumentation(): Promise<void> {
     // Convert ToolDefinitions to ToolWithAnnotations
     const toolsWithAnnotations: ToolWithAnnotations[] = tools
       .filter(tool => {
-        if (!tool.annotations.conditions) {
-          return true;
-        }
-
         // Filter out extension tools
         if (tool.name === 'install_extension') {
           return false;
         }
-        
+
+        if (!tool.annotations.conditions) {
+          return true;
+        }
+
         // Only include unconditional tools.
         return tool.annotations.conditions.length === 0;
       })
diff --git a/src/tools/extensions.ts b/src/tools/extensions.ts
@@ -22,7 +22,7 @@ export const installExtension = defineTool({
       .describe('Absolute path to the unpacked extension folder.'),
   },
   handler: async (request, response, context) => {
-    const { path } = request.params;
+    const {path} = request.params;
     const id = await context.installExtension(path);
     response.appendResponseLine(`Extension installed. Id: ${id}`);
   },
diff --git a/tests/tools/extensions.test.ts b/tests/tools/extensions.test.ts
@@ -6,18 +6,21 @@
 
 import assert from 'node:assert';
 import path from 'node:path';
-import { describe, it } from 'node:test';
+import {describe, it} from 'node:test';
 
-import { installExtension } from '../../src/tools/extensions.js';
-import { withMcpContext } from '../utils.js';
+import {installExtension} from '../../src/tools/extensions.js';
+import {withMcpContext} from '../utils.js';
 
-const EXTENSION_PATH = path.join(import.meta.dirname, '../../../tests/tools/fixtures/extension');
+const EXTENSION_PATH = path.join(
+  import.meta.dirname,
+  '../../../tests/tools/fixtures/extension',
+);
 
 describe('extension', () => {
   it('installs an extension and verifies it is listed in chrome://extensions', async () => {
     await withMcpContext(async (response, context) => {
       await installExtension.handler(
-        { params: { path: EXTENSION_PATH } },
+        {params: {path: EXTENSION_PATH}},
         response,
         context,
       );
diff --git a/tests/tools/fixtures/extension/popup.html b/tests/tools/fixtures/extension/popup.html
@@ -1 +1,6 @@
-<!DOCTYPE html><html><body><h1>Test Popup</h1></body></html>
+<!doctype html>
+<html
+  ><body
+    ><h1>Test Popup</h1></body
+  ></html
+>
