DYN-9658: Correctly adds file information to pacman when publishing from custom node workspace

[dnenov]

Purpose

This PR addresses the issue where Custom nodes (.dyf files) published via "Publish → Publish Custom Node (.dyf)" menu didn't appear in the package contents in the publish wizard.

Cause
ShowNodePublishInfo() only populated CustomNodeDefinitions but not CustomDyfFilepaths. The wizard filters out CustomNode type items and only displays CustomNodePreview type items (created from CustomDyfFilepaths), resulting in empty package contents.

Solution

• Populate CustomDyfFilepaths with file paths from CustomNodeInfo before setting CustomNodeDefinitions (which triggers RefreshPackageContents())
• Added fallback to use workspace.FileName when CustomNodeInfo.Path is empty
• Enhanced OnDataContextChanged to check CustomDyfFilepaths count in addition to HasChanges for proper pending update detection

Changes

Declarations

Check these if you believe they are true

• Is documented according to the standards
• The level of testing this PR includes is appropriate
• Changes to the API follow Semantic Versioning and are documented in the API Changes document.

Release Notes

• now correctly refreshes the datacontext and triggers update to the UI

Reviewers

@zeusongit

FYIs

@benglin

[github-actions]

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

[Copilot]

Pull Request Overview

This PR fixes a bug where custom nodes (.dyf files) published via "Publish → Publish Custom Node (.dyf)" menu didn't appear in the package contents of the publish wizard. The issue was caused by the wizard only populating CustomNodeDefinitions without setting CustomDyfFilepaths, causing the UI to show empty package contents since it filters out CustomNode items and only displays CustomNodePreview items.

• Populate CustomDyfFilepaths before setting CustomNodeDefinitions to ensure proper UI display
• Add fallback to use workspace.FileName when CustomNodeInfo.Path is empty
• Enhance pending update detection logic to check for actual content availability

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
PackageManagerClientViewModel.cs	Adds logic to populate CustomDyfFilepaths dictionary and fallback for empty Path values
PublishPackageViewModel.cs	Changes CustomDyfFilepaths property visibility from private to internal
PackageManagerWizard.xaml.cs	Improves pending update detection by checking content availability and fixes typo in comment

[Copilot]

[nitpick] Creating a new CustomNodeInfo object to modify only the Path property is inefficient. Consider adding a constructor overload or a method to update the Path property directly, or use object initialization syntax to make the intent clearer.

Suggested change

	currentFunInfo = new CustomNodeInfo(
	currentFunInfo.FunctionId,
	currentFunInfo.Name,
	currentFunInfo.Category,
	currentFunInfo.Description,
	ws.FileName, // Use workspace FileName
	currentFunInfo.IsVisibleInDynamoLibrary);
	currentFunInfo.Path = ws.FileName; // Update Path directly