Fix NuGet restore loop with CPM for SqlDatabaseProjects

[almostchristian]

Closes #1065

Fixes a NuGet restore loop / circular dependency when using Central Package Management (Directory.Packages.props) with CommunityToolkit.Aspire.Hosting.SqlDatabaseProjects.

This updates the props file in CommunityToolkit.Aspire.Hosting.SqlDatabaseProjects to mark the PackageReferences as implicitly defined (IsImplicitlyDefined="true"), preventing CPM from treating them as explicit references and thrashing during restore.

Fixes #1065

PR Checklist

• Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
• Based off latest main branch of toolkit
• PR doesn't include merge commits (always rebase on top of our main, if needed)
• Code follows all style conventions

Other information

Discussion on the IsImplicitlyDefined property and its relationship with CPM NuGet/Home#13529

[github-actions]

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.sh | bash -s -- 1291

Or

• Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.ps1) } 1291"

[Copilot]

Pull request overview

Fixes a NuGet restore loop/circular dependency affecting CommunityToolkit.Aspire.Hosting.SqlDatabaseProjects when consumers use Central Package Management (Directory.Packages.props) by marking build-injected PackageReferences as implicitly defined.

Changes:

• Add IsImplicitlyDefined="true" to Microsoft.Build* package references injected via the integration’s .props file to prevent CPM from treating them as explicit references.

[ErikEJ]

LGTM - looking foward to @jmezach 's comments

[ErikEJ]

@aaronpowell Privateassets true is not a valid value