Conversation
The MCP toggle button has no icon at all today, only text, and the running/stopped state is only ever shown in a one-off TaskDialog popup. This adds an ON/OFF icon to the ribbon button itself, kept in sync with Application.SocketService.IsRunning, plus a tooltip showing the listening port from ConfigManager.Instance.Settings.Port. Not build-verified locally: this machine lacks the .NET 10 SDK that Nice3point.Revit.Sdk 6.1.0's build task requires; only the runtime is installed. The change is small and mirrors the same pattern already shipped in a sibling project (revit-mcp-plugin), but please run CI/ a local build before merging. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
TaskDialogpopup, so there's no persistent visual indicator on the ribbon.LargeImage, kept in sync withApplication.SocketService.IsRunning.ConfigManager.Instance.Settings.Port).Note on CI
I see
check-pr.ymlrestricts fork PRs todomain/*.mdandGEMINI.mdonly, so this PR (touchingMCP/Application.cs,MCP/Commands/MCPCommands.cs,MCP/RevitMCP.csproj, and two new icon resources) is expected to fail that check. Opening it anyway per the repo owner's own request, so it's here for visibility/discretion rather than an auto-mergeable contribution — happy to close it, rework it into an Issue, or adjust however you'd prefer.Test plan
Nice3point.Revit.Sdk6.1.0's build task requires (dotnet build -c Release.R24fails withMSB4062loading the SDK's own task assembly). The change itself is small (twoBitmapImageswaps + a tooltip string), and mirrors the same pattern already applied and manually verified in a sibling project (revit-mcp-plugin).🤖 Generated with Claude Code