Skip to content

EPMRPP-106343 || MCP. Tool. Fix filter and tool names#29

Merged
AleksandrParamonoff merged 1 commit into
mainfrom
fix-tools-filter-names
Aug 1, 2025
Merged

EPMRPP-106343 || MCP. Tool. Fix filter and tool names#29
AleksandrParamonoff merged 1 commit into
mainfrom
fix-tools-filter-names

Conversation

@AleksandrParamonoff
Copy link
Copy Markdown
Contributor

No description provided.

@AleksandrParamonoff AleksandrParamonoff force-pushed the add-new-tool-get-suites-by-filters branch from 6d51141 to 8d9e052 Compare August 1, 2025 16:26
@AleksandrParamonoff AleksandrParamonoff force-pushed the fix-tools-filter-names branch 2 times, most recently from be9682a to 6c96620 Compare August 1, 2025 17:11
@AleksandrParamonoff AleksandrParamonoff force-pushed the add-new-tool-get-suites-by-filters branch 2 times, most recently from 54ce2b6 to 56681e5 Compare August 1, 2025 17:42
Base automatically changed from add-new-tool-get-suites-by-filters to main August 1, 2025 17:45
Copy link
Copy Markdown
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 standardizes parameter and tool naming conventions in an MCP (Model Context Protocol) tool for ReportPortal integration. The changes replace dots with hyphens in parameter names and update tool names from "list_" to "get_" prefix for consistency.

Key changes:

  • Standardized parameter naming by replacing dots with hyphens (e.g., "page.size" → "page-size", "filter.cnt.name" → "filter-cnt-name")
  • Updated tool names from "list_" to "get_" prefix for consistency ("list_launches" → "get_launches", "list_test_items_by_filter" → "get_test_items_by_filter")
  • Updated corresponding test cases and analytics references to match new naming

Reviewed Changes

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

Show a summary per file
File Description
internal/reportportal/utils.go Updates pagination parameter names from dot notation to hyphen notation
internal/reportportal/server.go Changes tool registration calls to use new "get_" prefixed tool names
internal/reportportal/launches_test.go Updates test to use new tool name "get_launches"
internal/reportportal/launches.go Renames tool method and updates all filter parameter names to use hyphens
internal/reportportal/items.go Renames tool method and updates all filter parameter names to use hyphens
internal/reportportal/analytics_test.go Updates test assertions to use new tool names
Taskfile.yaml Removes hardcoded port mapping and uses variable for inspector command

@AleksandrParamonoff AleksandrParamonoff merged commit 488319f into main Aug 1, 2025
@AleksandrParamonoff AleksandrParamonoff deleted the fix-tools-filter-names branch August 1, 2025 17:48
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.

3 participants