Fix mcp inspect to apply imports before extracting MCP configurations

[Copilot]

Problem

The mcp inspect command was not processing the imports: field in workflow frontmatter before extracting MCP server configurations. This meant that MCP servers defined in imported files were invisible to the inspect command, even though they would be properly included during workflow compilation.

For example, the pdf-summary.md workflow imports shared/markitdown-mcp.md which defines a markitdown MCP server:

# .github/workflows/pdf-summary.md
---
imports:
  - shared/markitdown-mcp.md
---

# .github/workflows/shared/markitdown-mcp.md
---
mcp-servers:
  markitdown:
    container: "ghcr.io/microsoft/markitdown"
    allowed: ["*"]
---

Running gh aw mcp inspect pdf-summary would fail to find the markitdown MCP server because imports were not being processed.

Solution

Updated both InspectWorkflowMCP and spawnMCPInspector functions to:

1. Process imports from frontmatter using parser.ProcessImportsFromFrontmatterWithManifest (the same function used by the workflow compiler)
2. Merge imported MCP servers and tools with top-level configurations
3. Extract MCP configurations from the merged result

This ensures mcp inspect sees the complete MCP server configuration, including servers defined in imported files.

Key Changes

• Exported merge methods (MergeTools, MergeMCPServers) in pkg/workflow/imports.go for reuse across the codebase
• Added applyImportsToFrontmatter helper to merge imported configurations into frontmatter before extraction
• Updated both inspect functions to process imports before extracting MCP configurations

Testing

• Added unit test TestApplyImportsToFrontmatter to verify import merging logic
• Verified all 60 existing workflows compile successfully
• Integration testing confirms:
  • ✅ Workflows with imports (e.g., pdf-summary) correctly find imported MCP servers
  • ✅ Workflows without imports continue to work normally
  • ✅ Filtering by server name works for imported MCP servers

Now running gh aw mcp inspect pdf-summary correctly finds and inspects the markitdown MCP server imported from shared/markitdown-mcp.md.

Original prompt

mcp inspect must apply imports before trying to launch mcps. Reuse existing helper function to parse, apply the imports.

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.