Skip to content

Update package display format from package::version to package@version - #9415

Merged
marcpopMSFT merged 4 commits into
release/10.0.2xxfrom
copilot/update-templating-text-symbol
Oct 16, 2025
Merged

Update package display format from package::version to package@version#9415
marcpopMSFT merged 4 commits into
release/10.0.2xxfrom
copilot/update-templating-text-symbol

Conversation

Copilot AI commented Oct 10, 2025

Copy link
Copy Markdown
  • Explore repository structure and identify files to change
  • Update InstallRequest.DisplayName to use @ instead of ::
  • Update NuGetManagedTemplatePackage.DisplayName to use @ instead of ::
  • Update test expectations in NuGetInstallerTests.cs to use @ instead of ::
  • Update exception messages in NuGet exceptions to use @ instead of ::
  • Update logging messages in NugetApiPackageManager.cs to use @ instead of ::
  • Update logging messages in PackageManager.cs test helper to use @ instead of ::
  • Update console output in TemplateDiscovery tools to use @ instead of ::
  • Verify no other occurrences of package::version format remain
  • Fix test expectations in TemplateDiscoveryTests for @ format
  • Final review and validation

Summary of Changes

Updated all package identifier and version display formatting from the legacy package::version format to the new recommended package@version format across:

Core Properties:

  • InstallRequest.DisplayName in Microsoft.TemplateEngine.Abstractions
  • NuGetManagedTemplatePackage.DisplayName in Microsoft.TemplateEngine.Edge

Exception Messages:

  • PackageNotFoundException constructor
  • DownloadException constructors (3 overloads)

Logging Messages:

  • NugetApiPackageManager.cs (5 locations)
  • PackageManager.cs test helper (3 locations)

Console Output:

  • AdditionalDataExtensions.cs in TemplateDiscovery tools
  • PackSourceChecker.cs in TemplateDiscovery tools (12 locations)

Tests:

  • NuGetInstallerTests.cs (4 test assertions)
  • TemplateDiscoveryTests.cs (6 test assertions)

Total: 10 files modified, 37 changes across the codebase. All references consistently updated to use the @ symbol instead of :: for package@version formatting.

Original prompt

This section details on the original issue you should resolve

<issue_title>Update templating text to use @ instead of :: symbol</issue_title>
<issue_description>The package::version format is the legacy format and the recommendation now is to use @ instead. We have an SDK PR to update this but some of the strings come from the template engine as well. Both repos already support the new format so this is just a matter of ensuring all messaging recommends customers use the new format.

Here's the SDK PR: dotnet/sdk#51153

For the templating repo:

  • Update InstallRequest.DisplayName
  • Update TemplatePackage.DisplayName properties
  • identify any additional changes needed to strings or tests related to this transition

Both DisplayNames come from the Microsoft.TemplateEngine.Edge package</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #9414


ðŸ’Ą You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits October 10, 2025 18:44
â€Ķpackage@version

Co-authored-by: marcpopMSFT <12663534+marcpopMSFT@users.noreply.github.com>
Co-authored-by: marcpopMSFT <12663534+marcpopMSFT@users.noreply.github.com>
Copilot AI changed the title [WIP] Update templating text to use @ instead of :: symbol Update package display format from package::version to package@version Oct 10, 2025
Copilot AI requested a review from marcpopMSFT October 10, 2025 18:55
@marcpopMSFT

Copy link
Copy Markdown
Member

@copilot can you fix these three tests for the change in the :: output to @?

  • CanDetectRemovedPackagesInDiffMode
  • CanDetectUpdatedPackagesInDiffMode
  • CanDetectNewPackagesInDiffMode

â€ĶTests

Co-authored-by: marcpopMSFT <12663534+marcpopMSFT@users.noreply.github.com>

Copilot AI commented Oct 10, 2025

Copy link
Copy Markdown
Author

@copilot can you fix these three tests for the change in the :: output to @?

  • CanDetectRemovedPackagesInDiffMode
  • CanDetectUpdatedPackagesInDiffMode
    ...

Fixed the three tests to expect @ instead of :: in the output. Updated 6 test assertions across:

  • CanDetectNewPackagesInDiffMode (2 assertions)
  • CanDetectUpdatedPackagesInDiffMode (1 assertion)
  • CanDetectRemovedPackagesInDiffMode (3 assertions)

Commit: Will be included in the progress report.

@marcpopMSFT
marcpopMSFT marked this pull request as ready for review October 14, 2025 23:12
@marcpopMSFT
marcpopMSFT requested a review from a team as a code owner October 14, 2025 23:12
@marcpopMSFT
marcpopMSFT requested a review from MiYanni October 14, 2025 23:12
@marcpopMSFT

Copy link
Copy Markdown
Member

I looked through all instances of :: in the sources and this covers all the ones I would expect to update for this. I don't know how to test the template repo and am relying on the automated tests.

@MiYanni MiYanni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I can't verify if any of these changes are correct. But if it doesn't cause problems in the SDK repo, then it is likely fine. But we'd have to wait for this to flow there to verify that.

@marcpopMSFT
marcpopMSFT merged commit 0266032 into release/10.0.2xx Oct 16, 2025
9 checks passed
mmitche pushed a commit to dotnet/sdk that referenced this pull request Apr 2, 2026
â€Ķtemplating-text-symbol

Update package display format from package::version to package@version

Commit migrated from dotnet/templating@0266032
@MichaelSimons
MichaelSimons deleted the copilot/update-templating-text-symbol branch April 27, 2026 16:25
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