Remove duplicate parsing logic from mcp inspect commands

[Copilot]

The mcp inspect and mcp inspect --inspector commands reimplemented frontmatter parsing, import processing, and MCP server merging logic that already exists in the compiler.

Changes

• mcp_inspect.go: Replace manual parsing pipeline with compiler.ParseWorkflowFile()
• mcp_inspect_inspector.go: Same compiler-based approach
• mcp_inspect_imports.go: Removed (78 lines) - applyImportsToFrontmatter() no longer needed
• mcp_inspect_test.go: Removed obsolete test (103 lines)

Added buildFrontmatterFromWorkflowData() helper that uses ParsedFrontmatter.ToMap() to reconstruct frontmatter for MCP extraction.

Before/After

Before:

// Manual parsing and merging
content, _ := os.ReadFile(workflowPath)
parsedData, _ := parser.ExtractFrontmatterFromContent(string(content))
importsResult, _ := parser.ProcessImportsFromFrontmatterWithManifest(...)
frontmatterWithImports, _ := applyImportsToFrontmatter(parsedData.Frontmatter, importsResult)
mcpConfigs, _ := parser.ExtractMCPConfigurations(frontmatterWithImports, serverFilter)

After:

// Compiler handles parsing, imports, and merging
compiler := workflow.NewCompiler(verbose, "", "")
workflowData, _ := compiler.ParseWorkflowFile(workflowPath)
frontmatter := buildFrontmatterFromWorkflowData(workflowData)
mcpConfigs, _ := parser.ExtractMCPConfigurations(frontmatter, serverFilter)

This ensures consistent behavior between compilation and inspection commands, eliminating ~220 lines of duplicate code.

Original prompt

Review mcp_inspect workflow parsing logic and remove duplicate custom parsing/merging that should be been done by the compiler. The compiler functions should be used to generate the mcp server config.

---

💡 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.