Skip to content

DYN-10735: Align Autodesk.IDSDK wrapper with Revit's AdskIdentitySDK.dll version - #17270

Merged
jasonstratton merged 3 commits into
DynamoDS:masterfrom
jasonstratton:DYN-10735-align-idsdk-version-with-revit
Aug 6, 2026
Merged

DYN-10735: Align Autodesk.IDSDK wrapper with Revit's AdskIdentitySDK.dll version#17270
jasonstratton merged 3 commits into
DynamoDS:masterfrom
jasonstratton:DYN-10735-align-idsdk-version-with-revit

Conversation

@jasonstratton

@jasonstratton jasonstratton commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Purpose

DYN-10735: Dynamo bundles Autodesk.IDSDK 1.2.5 (native AdskIdentitySDK.dll 1.15.3.5). Revit 2027.3 ships AdskIdentitySDK.dll 1.16.4.7 at its root, so Dynamo's bundled copy under AddIns/DynamoForRevit/ is meaningfully behind the host. No Autodesk.IDSDK.Client wrapper release bundles Revit's exact 1.16.4.7, so this PR moves to the closest available artifact: wrapper 1.2.6, which bundles native DLL 1.16.5.1.

This has been confirmed acceptable by both the Identity team and Revit:

  • Identity team (Wilfred Ng): backward/forward compatibility across this range is supported.
  • Revit (Sorin Brinzaru): explicitly approved 1.16.5.1 as 'one of the FY27 ML-Final version we should align on.'

Note: the originally-reported Debug Warning dialog on Dynamo sign-out turned out to be an unrelated, pre-existing Revit-side bug (REVIT-255685) that reproduces even with DLL versions fully aligned — this PR does not (and cannot) fix that dialog. It closes the separate, real version gap that predates this ticket.

Declarations

Check these if you believe they are true

No public API surface changes — this is a single PackageReference version bump in src/DynamoCore/DynamoCore.csproj.

Testing: built locally, confirmed the harvested AdskIdentitySDK.dll is 1.16.5.1, deployed into a local Revit 2027.3 install via Dynamo.config redirect, and exercised sign-in/sign-out from the splash screen with no issues observed.

Release Notes

Updated the bundled Autodesk Identity SDK to a version closer to what Revit ships, reducing the native SDK version gap between Dynamo and its host application.

Reviewers

(TBD — Jason to assign)

FYIs

@casandra.armeanu, @sorin.brinzaru

…ntitySDK.dll

Revit 2027.3 ships AdskIdentitySDK.dll 1.16.4.7; Dynamo bundled 1.15.3.5
via Autodesk.IDSDK 1.2.5. No wrapper release carries the exact 1.16.4.x
DLL, so this moves to 1.2.6 (DLL 1.16.5.1), the closest available and
confirmed forward-compatible by the Identity team.
Copilot AI lite review requested due to automatic review settings August 5, 2026 16:15

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-10735

Copilot AI previously approved these changes Aug 5, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Ready to approve

The change is a straightforward dependency version bump scoped to a single PackageReference, with no other repo references to Autodesk.IDSDK found.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Pull request overview

This PR updates Dynamo’s bundled Autodesk Identity SDK wrapper dependency to better align with the native AdskIdentitySDK.dll version shipped by Revit 2027.3, reducing the version gap between DynamoForRevit and its host environment.

Changes:

  • Bumped Autodesk.IDSDK NuGet dependency from 1.2.5 to 1.2.6 in DynamoCore, which (per PR description) updates the bundled native SDK from 1.15.3.5 to 1.16.5.1.
File summaries
File Description
src/DynamoCore/DynamoCore.csproj Updates the Autodesk.IDSDK PackageReference version to 1.2.6.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

@jasonstratton jasonstratton added this to the 4.2 milestone Aug 5, 2026
@jasonstratton
jasonstratton requested review from a team and RobertGlobant20 and a lite review from Copilot August 5, 2026 16:57
Copilot AI dismissed their stale review, a newer Copilot review was requested August 5, 2026 16:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Ready to approve

The change is a single, scoped dependency version bump with no other code modifications and no additional references to update within the repo.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

Desktop Connector published 1.2.6 to nuget.org on 2026-08-06. Re-running
CI so the build actually resolves 1.16.5.1 instead of the NU1603 float
to 1.2.7 seen on the previous run.
Build DynamoDS#499 was aborted mid-run when its agent went offline, not due to
a test or code failure — all 5 test suites and Harmony had already
passed, and the compiler confirmed autodesk.idsdk 1.2.6 resolved
correctly. Retriggering for a clean run.
@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

@jasonstratton
jasonstratton merged commit 6b9bb9b into DynamoDS:master Aug 6, 2026
15 checks passed
@jasonstratton

Copy link
Copy Markdown
Contributor Author

Ongoing github actions outage. The cherry pick is unlikely to happen automatically or when issuing the command in a comment. So I will create a PR to RC4.2.0_master and cherry pick manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants