From 070089049a04a8333485288207d7bf93660d02ef Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 28 Feb 2026 18:00:12 +0000
Subject: [PATCH 1/3] Initial plan


From ab730c2e01801207c51263ba6f29d69423f7f798 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 28 Feb 2026 18:03:37 +0000
Subject: [PATCH 2/3] Add comments explaining PrivateAssets and IncludeAssets
 on analyzer/tool packages

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
---
 src/TemplateDotNetLibrary/TemplateDotNetLibrary.csproj   | 5 +++++
 .../TemplateDotNetLibrary.Tests.csproj                   | 9 +++++++++
 2 files changed, 14 insertions(+)

diff --git a/src/TemplateDotNetLibrary/TemplateDotNetLibrary.csproj b/src/TemplateDotNetLibrary/TemplateDotNetLibrary.csproj
index a067882..8814ef4 100644
--- a/src/TemplateDotNetLibrary/TemplateDotNetLibrary.csproj
+++ b/src/TemplateDotNetLibrary/TemplateDotNetLibrary.csproj
@@ -50,6 +50,11 @@
     <PackageReference Include="Microsoft.Sbom.Targets" Version="4.1.5" PrivateAssets="All" />
     <PackageReference Include="Polyfill" Version="9.12.0" PrivateAssets="All" />
     <PackageReference Include="Microsoft.SourceLink.GitHub" Version="10.0.103" PrivateAssets="All" />
+    <!-- Analyzer packages use child-element form to configure both PrivateAssets and IncludeAssets:
+         - PrivateAssets="all" prevents these build-time analyzers from becoming transitive dependencies
+           in packages that consume this library.
+         - IncludeAssets lists all asset types (including 'analyzers' and 'buildtransitive') to ensure
+           Roslyn analyzers and MSBuild targets are fully activated during the build. -->
     <PackageReference Include="Microsoft.CodeAnalysis.NetAnalyzers" Version="10.0.103">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
diff --git a/test/TemplateDotNetLibrary.Tests/TemplateDotNetLibrary.Tests.csproj b/test/TemplateDotNetLibrary.Tests/TemplateDotNetLibrary.Tests.csproj
index 15bfdf1..c3004ba 100644
--- a/test/TemplateDotNetLibrary.Tests/TemplateDotNetLibrary.Tests.csproj
+++ b/test/TemplateDotNetLibrary.Tests/TemplateDotNetLibrary.Tests.csproj
@@ -21,6 +21,10 @@
 
   <!-- Test Framework Dependencies -->
   <ItemGroup>
+    <!-- coverlet.collector uses child-element form to configure both PrivateAssets and IncludeAssets:
+         - PrivateAssets="all" keeps this test-coverage tool out of any consuming project's dependencies.
+         - IncludeAssets lists all asset types (including 'build' and 'buildtransitive') to ensure the
+           data collector MSBuild targets are activated so coverage is collected during test runs. -->
     <PackageReference Include="coverlet.collector" Version="8.0.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
@@ -32,6 +36,11 @@
 
   <!-- Code Analysis Dependencies -->
   <ItemGroup>
+    <!-- Analyzer packages use child-element form to configure both PrivateAssets and IncludeAssets:
+         - PrivateAssets="all" prevents these build-time analyzers from becoming transitive dependencies
+           in any project that references this test project.
+         - IncludeAssets lists all asset types (including 'analyzers' and 'buildtransitive') to ensure
+           Roslyn analyzers and MSBuild targets are fully activated during the build. -->
     <PackageReference Include="Microsoft.CodeAnalysis.NetAnalyzers" Version="10.0.103">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>

From bd34a76ea6958a266e4a8f6d3f002141440e6698 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 28 Feb 2026 18:20:46 +0000
Subject: [PATCH 3/3] Split PackageReference ItemGroup into typed sections in
 library csproj

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
---
 src/TemplateDotNetLibrary/TemplateDotNetLibrary.csproj | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/TemplateDotNetLibrary/TemplateDotNetLibrary.csproj b/src/TemplateDotNetLibrary/TemplateDotNetLibrary.csproj
index 8814ef4..c4ab24d 100644
--- a/src/TemplateDotNetLibrary/TemplateDotNetLibrary.csproj
+++ b/src/TemplateDotNetLibrary/TemplateDotNetLibrary.csproj
@@ -46,10 +46,15 @@
     <SBOMPackageSupplier>Organization: $(Company)</SBOMPackageSupplier>
   </PropertyGroup>
 
+  <!-- Build Tool Dependencies -->
   <ItemGroup>
     <PackageReference Include="Microsoft.Sbom.Targets" Version="4.1.5" PrivateAssets="All" />
     <PackageReference Include="Polyfill" Version="9.12.0" PrivateAssets="All" />
     <PackageReference Include="Microsoft.SourceLink.GitHub" Version="10.0.103" PrivateAssets="All" />
+  </ItemGroup>
+
+  <!-- Code Analysis Dependencies -->
+  <ItemGroup>
     <!-- Analyzer packages use child-element form to configure both PrivateAssets and IncludeAssets:
          - PrivateAssets="all" prevents these build-time analyzers from becoming transitive dependencies
            in packages that consume this library.