Skip to content

fix(daily-cli-tools-tester): replace non-existent list MCP tool with status#31481

Merged
pelikhan merged 2 commits into
mainfrom
copilot/fix-daily-cli-tools-tester
May 11, 2026
Merged

fix(daily-cli-tools-tester): replace non-existent list MCP tool with status#31481
pelikhan merged 2 commits into
mainfrom
copilot/fix-daily-cli-tools-tester

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 11, 2026

Bug Fix

What was the bug?

daily-cli-tools-tester referenced a list tool on the agentic-workflows MCP server that doesn't exist, causing Error [-32602]: unknown tool "list" on every run. The status tool already provides identical functionality (returns all workflows with metadata).

How did you fix it?

Two changes to .github/workflows/daily-cli-tools-tester.md:

  • Tool inventory (line 66): Removed the bogus list entry from the MCP tool list — status already covered this.
  • Phase 1.2 instructions (line 90): Replaced the list command reference with status.
-  - `list` - List all workflows in the repository
   - `status` - Get status and metadata for workflows

-  Use the agentic-workflows MCP tool's "list" command to enumerate all workflows in the repository.
+  Use the agentic-workflows MCP tool's "status" command to enumerate all workflows in the repository.

…ster

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix daily-cli-tools-tester to utilize existing status tool fix(daily-cli-tools-tester): replace non-existent list MCP tool with status May 11, 2026
Copilot AI requested a review from pelikhan May 11, 2026 11:27
@pelikhan pelikhan marked this pull request as ready for review May 11, 2026 11:28
Copilot AI review requested due to automatic review settings May 11, 2026 11:28
@pelikhan pelikhan merged commit 95cfa25 into main May 11, 2026
1 check passed
@pelikhan pelikhan deleted the copilot/fix-daily-cli-tools-tester branch May 11, 2026 11:29
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

Fixes daily-cli-tools-tester instructions to stop calling a non-existent list tool on the agentic-workflows MCP server and use status instead.

Changes:

  • Remove the invalid list entry from the documented MCP tool inventory.
  • Update workflow discovery instructions to use the status tool for enumeration.
Show a summary per file
File Description
.github/workflows/daily-cli-tools-tester.md Replaces list with status in the MCP tool list and discovery instructions to avoid runtime “unknown tool” errors.
.github/workflows/issue-monster.lock.yml Unrelated edit to an auto-generated lock file; should be reverted or applied via the source .md + recompile.

Copilot's findings

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 1

Comment on lines 389 to 393
# return {
# number: issue.number,
# title: issue.title,
# labels: issue.labels.map(l => l.name), # Label filtering applied via job conditions
# labels: issue.labels.map(l => l.name),
# body: issue.body,
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.

[cli-tools-test] daily-cli-tools-tester: references non-existent list tool in agentic-workflows MCP

3 participants