Skip to content

EPMHLM-445 || MCP. Tools. Set healing status - #6

Merged
AleksandrParamonoff merged 1 commit into
mainfrom
add-tool-update-healing-result
Dec 12, 2025
Merged

AleksandrParamonoff merged 1 commit into
mainfrom
add-tool-update-healing-result

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 updateHealingResult that allows updating the healing result status (successful or failed) for specific healing attempts in Healenium. The implementation includes comprehensive test coverage for integration scenarios, validation errors, and timeout behavior.

Key changes:

  • New tool function updateHealingResultTool() that sends POST requests to /healenium/healing/success endpoint
  • Comprehensive test suite including integration tests, validation tests, timeout tests, and mock server helpers
  • Registration of the new tool in the RegisterTools function

Reviewed changes

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

File Description
internal/mcp_handlers/tools.go Implements the updateHealingResultTool function with parameter validation, HTTP POST request handling, and success/error response formatting
internal/mcp_handlers/tools_test.go Adds comprehensive test coverage including integration tests (5 scenarios), validation error tests (4 scenarios), timeout test, and mock server helper function
healenium-mcp-server.exe Binary executable file added to repository (should not be committed)

💡 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.go Outdated
Comment thread internal/mcp_handlers/tools.go Outdated

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 4 changed files in this pull request and generated 6 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.go Outdated
Comment thread internal/mcp_handlers/tools_test.go
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 Outdated

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 4 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_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.

Pull request overview

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


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

Comment thread internal/mcp_handlers/tools.go
@AleksandrParamonoff
AleksandrParamonoff merged commit cb1c10b into main Dec 12, 2025
7 checks passed
@AleksandrParamonoff
AleksandrParamonoff deleted the add-tool-update-healing-result branch December 12, 2025 12:59
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