Skip to content

Conversation

@eyalzh
Copy link
Owner

@eyalzh eyalzh commented Aug 5, 2025

No description provided.

@eyalzh eyalzh requested a review from Copilot August 5, 2025 16:00
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds bookmark search functionality to the browser control MCP server, enabling AI assistants to access and search through browser bookmarks. The implementation includes both full bookmark tree retrieval and query-based bookmark searching.

  • Adds a new get-browser-bookmarks tool that can retrieve bookmarks with optional search functionality
  • Implements bookmark formatting as hierarchical text structure with proper indentation and numbering
  • Updates all package versions from 1.5.x to 1.6.0 across the project

Reviewed Changes

Copilot reviewed 14 out of 18 changed files in this pull request and generated no comments.

Show a summary per file
File Description
package.json Version bump to 1.6.0
mcp-server/server.ts Adds new bookmark retrieval tool and version update
mcp-server/package.json Version bump to 1.6.0
mcp-server/manifest.json Version update and tool registration
mcp-server/browser-api.ts Implements bookmark API communication method
firefox-extension/util.ts New utility for formatting bookmarks as hierarchical text
firefox-extension/package.json Version bump to 1.6.0
firefox-extension/message-handler.ts Handles bookmark retrieval requests and formatting
firefox-extension/manifest.json Adds bookmarks permission and version update
firefox-extension/extension-config.ts Registers new bookmark tool configuration
firefox-extension/tests/setup.ts Adds bookmark mock for testing
firefox-extension/tests/message-handler.test.ts Comprehensive tests for bookmark functionality
common/server-messages.ts Defines bookmark request message types
common/extension-messages.ts Defines bookmark response message types
Files not reviewed (2)
  • firefox-extension/package-lock.json: Language not supported
  • mcp-server/package-lock.json: Language not supported
Comments suppressed due to low confidence (1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants