Fix pr comments

Natallia Harshunova · Natallia Harshunova · commit f53cc9068144 · 2026-01-15T16:33:47.000Z
diff --git a/src/tools/extensions.ts b/src/tools/extensions.ts
@@ -1,10 +1,9 @@
 /**
  * @license
- * Copyright 2025 Google LLC
+ * Copyright 2026 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import {logger} from '../logger.js';
 import {zod} from '../third_party/index.js';
 
 import {ToolCategory} from './categories.js';
@@ -23,13 +22,8 @@ export const installExtension = defineTool({
       .describe('Absolute path to the unpacked extension folder.'),
   },
   handler: async (request, response, context) => {
-    const {path} = request.params;
-    try {
-      const id = await context.installExtension(path);
-      response.appendResponseLine(`Extension installed. Id: ${id}`);
-    } catch (error) {
-      logger('Extension installation error: ', error);
-      throw error;
-    }
+    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
@@ -5,38 +5,19 @@
  */
 
 import assert from 'node:assert';
-import fs from 'node:fs';
-import os from 'node:os';
 import path from 'node:path';
-import {describe, it, after} 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';
 
-describe('extension', () => {
-  let tmpDir: string;
+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 () => {
-    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'extension-test-'));
-    fs.writeFileSync(
-      path.join(tmpDir, 'manifest.json'),
-      JSON.stringify({
-        manifest_version: 3,
-        name: 'Test Extension',
-        version: '1.0',
-        action: {
-          default_popup: 'popup.html',
-        },
-      }),
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'popup.html'),
-      '<!DOCTYPE html><html><body><h1>Test Popup</h1></body></html>',
-    );
-
     await withMcpContext(async (response, context) => {
       await installExtension.handler(
-        {params: {path: tmpDir}},
+        { params: { path: EXTENSION_PATH } },
         response,
         context,
       );
@@ -59,10 +40,4 @@ describe('extension', () => {
       );
     });
   });
-
-  after(() => {
-    if (tmpDir) {
-      fs.rmSync(tmpDir, {recursive: true, force: true});
-    }
-  });
 });
diff --git a/tests/tools/fixtures/extension/manifest.json b/tests/tools/fixtures/extension/manifest.json
@@ -0,0 +1,8 @@
+{
+  "manifest_version": 3,
+  "name": "Test Extension",
+  "version": "1.0",
+  "action": {
+    "default_popup": "popup.html"
+  }
+}
diff --git a/tests/tools/fixtures/extension/popup.html b/tests/tools/fixtures/extension/popup.html
@@ -0,0 +1 @@
+<!DOCTYPE html><html><body><h1>Test Popup</h1></body></html>

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+<!DOCTYPE html><html><body><h1>Test Popup</h1></body></html>`