Skip to content

EPMHLM-446 || MCP. Tools. Add save path to selector tool - #8

Merged
AleksandrParamonoff merged 1 commit into
mainfrom
add-tool-save-path-to-selector
Dec 16, 2025
Merged

AleksandrParamonoff merged 1 commit into
mainfrom
add-tool-save-path-to-selector

Conversation

@AleksandrParamonoff

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

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 a new MCP tool called saveSelectorPath that allows saving paths to selectors in Healenium by associating healing results with their declaring classes for a specific selector.

Key Changes:

  • Added saveSelectorPathTool implementation with comprehensive input validation
  • Registered the new tool in the MCP server
  • Added extensive test coverage including integration tests and validation error tests

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
internal/mcp_handlers/tools.go Implements the new saveSelectorPathTool function with validation for id (MD5 hash format) and data array (healing result IDs and declaring classes), includes POST request to /healenium/selector/save/path endpoint
internal/mcp_handlers/tools_test.go Adds comprehensive test coverage including integration tests, validation error tests, and helper functions (createMockSaveSelectorPathServer, convertToAnySlice)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread internal/mcp_handlers/tools_test.go Outdated
Comment thread internal/mcp_handlers/tools_test.go Outdated
Comment thread internal/mcp_handlers/tools_test.go
Comment thread internal/mcp_handlers/tools_test.go

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread internal/mcp_handlers/tools.go
Comment thread internal/mcp_handlers/tools_test.go
@AleksandrParamonoff
AleksandrParamonoff merged commit ed3b8d6 into main Dec 16, 2025
7 checks passed
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