DYN-10553: Bundle DynamoMCP as built-in package#17162
Closed
johnpierson wants to merge 2 commits into
Closed
Conversation
Ship the DynamoMCP view extension through the existing Built-In Packages mechanism so all Dynamo users receive MCP server support without installing from Package Manager.
johnpierson
changed the title
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-10553
Third-party and DynamoMCP-owned binaries in extern/DynamoMCP ship with their own assembly versions, matching the TuneUp built-in package exclusion pattern.
|
RobertGlobant20
left a comment
RobertGlobant20
left a comment
Contributor
There was a problem hiding this comment.
basically my only comment is that the mcp_server dlls are not signed
| "copyright_year": "2026", | ||
| "file_hash": null, | ||
| "name": "DynamoMCP", | ||
| "version": "7.7.7.7", |
Contributor
There was a problem hiding this comment.
what this version 7.7.7.7 means?
Contributor
There was a problem hiding this comment.
Not signed, I think you should use the ones from the DynamoMCP release
| "engine": "dynamo", | ||
| "engine_metadata": "", | ||
| "site_url": "https://dynamobim.org/", | ||
| "repository_url": "https://git.autodesk.com/Dynamo/DynamoMCP" |
Member
Author
|
Close in favor of #17165 |
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
2 participants
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.
Purpose
DYN-10553 — Bundle DynamoMCP as a Dynamo Built-In Package (epic DYN-10205), using the same
extern/+ MSBuild copy mechanism as TuneUp (DYN-10450 pattern), so MCP server support ships with Dynamo Core for all users (including those without Package Manager access).Delivery approach: This PR implements the TuneUp-style vendoring path (
extern/DynamoMCP/→AfterBuildOpscopy) rather than the NuGet-feed approach described in DYN-10553. That matches how TuneUp and the AA built-in work are integrated and avoids an additional NuGet dependency in the installer pipeline.Key changes:
extern/DynamoMCP/package layout (pkg.json,extra/MCP_ViewExtensionDefinition.xml, entry-point binaries underbin/andbin/mcp_server/)DynamoCoreWpfAfterBuildOps→Built-In Packages/packages/DynamoMCP/extern/.gitignoreto track runtime DLLs while excluding test/pdb/deps artifactsMCPExtension.dllto installer signing list and file-version check exclusions (mirrors TuneUp handling; prerequisite DYN-10551)This follows the existing built-in package loader path:
%BuiltInPackages%→PackageLoader→PackageManagerViewExtensionloads the view extension manifest fromextra/. Dynamo core already exposesInternalsVisibleTofor MCP assemblies and blocks the MCP view extension whenNoNetworkModeis enabled.Note:
MCPExtension.dllmust be Authenticode-signed before release (included inadditional-files-to-sign.txt). Binaries in this PR are from a local DynamoMCP Release build and will be re-signed in the ADSK pipeline.Follow-up (separate tickets): DYN-10552 (remove PM delivery path), DYN-10554 (integration verification), DYN-10570 (customer docs).
Declarations
Check these if you believe they are true
No public API changes. Packaging, build, and test updates only.
Release Notes
Dynamo now includes the DynamoMCP built-in package, exposing workspace operations through the Model Context Protocol (MCP) without a separate Package Manager install.
Reviewers
(FILL ME IN) Reviewer 1
FYIs
doc/integration_docs/4_builtinpackages.mddist/DynamoMCP/intoextern/DynamoMCP/