McpInspector - Path Encoding And Dosn't preserve References

[tjwald]

Describe the bug

The McpInspector integration doesn't supported relative paths with multiple '/' in the path.
In addition, the integration doesn't add relationships to the mcp servers it is attached to.

Regression

No response

Steps to reproduce

var server = builder.AddContainer('mcp-server', ...);

builder.AddMcpInspector("mcp-inspector")
    .WithMcpServer(server, path: "/route/mcp");


Will produce no references between server and inspector.

In addition the url in the configuration file will be url encoded: `/route%2Fmcp`

Expected behavior

Have a relationship between the mcp-inspector and the mcp server.

Have the custom path encode correctly.

Screenshots

No response

IDE and version

Rider

IDE version

2025.2

Nuget packages

CommunityToolkit.Aspire.Hosting.McpInspector Version="9.7.2"

Additional context

No response

Help us help you

No, just wanted to report this

[aaronpowell]

Nice catch, I must not have had enough test coverage on there. Let me see if I can get a hotfix out on that.

[aaronpowell]

@tjwald can you verify the 9.8.0 beta that's on NuGet, it should contain this fix. If so, I could backport it to 9.7.x

[tjwald]

@aaronpowell The configuration now is correct!
I was surprised that the reference now is from the mcp server to the inspector, since that would mean that the mcp server "knows" the inspector and that the inspector is agnostic, but the opposite is true.
If the reference direction can be switched then that is better -but not a deal breaker.
Also, I see that there is a new endpoint? What is it supposed to do? - I am getting an error when opening it.

[aaronpowell]

I've added a few more annotations to describe the relationships a bit better.

[github-actions]

We have noticed this issue has not been updated within 21 days. If there is no action on this issue in the next 14 days, we will automatically close it. You can use /stale-extend to extend the window.