Add support for exact relative path resolution in mcp inspect command

[Copilot]

Summary

The gh aw mcp inspect command now supports exact relative path resolution for workflow files under .github/workflows, making it easier to inspect MCP server configurations in subdirectories.

Problem

Previously, mcp inspect could only resolve workflow files from the .github/workflows/ directory. Shared MCP configurations like serena.md, tavily.md, and other files in subdirectories like shared/mcp/ required users to provide full absolute or relative paths:

# Required before this change
gh aw mcp inspect .github/workflows/shared/mcp/serena.md

# Would fail with "workflow file not found"
gh aw mcp inspect shared/mcp/serena

Solution

Enhanced the ResolveWorkflowPath function to support exact relative path resolution under .github/workflows:

1. Exact path - If the path exists as-is, use it
2. Exact relative path - Try path under .github/workflows (e.g., shared/b.md → .github/workflows/shared/b.md)

Changes

• pkg/cli/resolver.go: Enhanced path resolution to support exact relative paths under .github/workflows
• pkg/cli/resolver_test.go: Added comprehensive tests covering full relative paths with and without .md extension
• pkg/cli/mcp_inspect.go: Updated inspector flag to use the enhanced path resolution

Examples

Users must provide the full relative path to access nested workflows:

# Full relative path required for nested workflows
gh aw mcp inspect shared/mcp/serena.md
gh aw mcp inspect shared/mcp/serena  # .md extension added automatically

# Main workflows still work with just the name
gh aw mcp inspect daily-news

# Launch inspector with shared workflow
gh aw mcp inspect shared/mcp/serena --inspector

All existing functionality remains unchanged:

• Main workflow files still work as before
• Absolute paths still work
• Listing workflows works correctly
• Error handling for nonexistent files works as expected

Testing

• ✅ All existing unit tests pass
• ✅ New tests added for exact relative path resolution
• ✅ Linting passes
• ✅ Manual testing confirms all use cases work correctly
• ✅ Backward compatibility maintained for main workflow files

Original prompt

Update the "mcp inspect" to support loading and inspect shared agentic workflows like serena.md without requiring a full agentic workflow

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[pelikhan]

@copilot search all workflows under .github/workflows as long as the subpath matches.

First full relative path match then

a.md -> .github/workflows/a.md
shared/b.md -> .github/workflows/b.md

There is already logic in the add command to do this kind of resolutions, refactor and reuse.

[Copilot]

@copilot search all workflows under .github/workflows as long as the subpath matches.

...

Refactored ResolveWorkflowPath to recursively search all workflows under .github/workflows with subpath matching support.

The function now supports:

• Full relative paths: shared/b.md → .github/workflows/shared/b.md
• Subpath matching: mcp/serena → .github/workflows/shared/mcp/serena.md
• Basename matching: serena → any serena.md in subdirectories

Resolution priority: exact path → exact relative path → subpath match → basename match

Commit: Refactor ResolveWorkflowPath to support recursive subpath matching

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• mcp.tavily.com
  • Triggering command: ./gh-aw mcp inspect daily-news (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[pelikhan]

@copilot remove recursive subpath matching

[Copilot]

@copilot remove recursive subpath matching

Removed recursive subpath matching. Now only supports exact relative paths:

• a.md → .github/workflows/a.md
• shared/b.md → .github/workflows/shared/b.md

Users must provide full relative path for nested workflows (e.g., shared/mcp/serena.md). Basename-only matching (serena) no longer works.

Commit: Remove recursive subpath matching from ResolveWorkflowPath

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• mcp.tavily.com
  • Triggering command: ./gh-aw mcp inspect daily-news (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[github-actions]

Agentic Changeset Generator triggered by this pull request