From 726edb674d15dea82589378bccfd488e39873525 Mon Sep 17 00:00:00 2001
From: Kieron Lanning <kieronlanning@users.noreply.github.com>
Date: Thu, 3 Sep 2026 11:16:30 +0100
Subject: [PATCH] build: fix Pipeline pack

---
 build/PipelineCLI/Modules/PackModule.cs        |  9 +++------
 src/src/SourceGenerator/SourceGenerator.csproj | 12 ++++--------
 2 files changed, 7 insertions(+), 14 deletions(-)

diff --git a/build/PipelineCLI/Modules/PackModule.cs b/build/PipelineCLI/Modules/PackModule.cs
index 8752f8b9..779ad130 100644
--- a/build/PipelineCLI/Modules/PackModule.cs
+++ b/build/PipelineCLI/Modules/PackModule.cs
@@ -11,17 +11,14 @@ namespace Purview.Telemetry.SourceGenerator.PipelineCLI.Modules;
 [ModuleCategory("Build")]
 [DependsOn<RunTestsModule>]
 [DependsOn<VersionModule>]
-public sealed class PackModule(IOptions<BuildSettings> settings, IOptions<ReleaseSettings> releaseSettings)
-	: Module<CommandResult>
+public sealed class PackModule(IOptions<BuildSettings> settings) : Module<CommandResult>
 {
 	protected override ModuleConfiguration Configure() =>
 		ModuleConfiguration
 			.Create()
 			.WithSkipWhen(_ =>
-				!settings.Value.RunPack || releaseSettings.Value.Mode == ReleaseMode.None
-					? SkipDecision.Skip(
-						"Packing is disabled. Set Build__RunPack=true and Release__Mode to something other than None to enable it."
-					)
+				!settings.Value.RunPack
+					? SkipDecision.Skip("Packing is disabled. Set Build__RunPack=true to enable it.")
 					: SkipDecision.DoNotSkip
 			)
 			.Build();
diff --git a/src/src/SourceGenerator/SourceGenerator.csproj b/src/src/SourceGenerator/SourceGenerator.csproj
index 4dce14d2..1a622775 100644
--- a/src/src/SourceGenerator/SourceGenerator.csproj
+++ b/src/src/SourceGenerator/SourceGenerator.csproj
@@ -35,17 +35,13 @@
 		/>
 		<Compile Include="..\Shared\TelemetryAttributeNames.cs" Link="Shared\TelemetryAttributeNames.cs" />
 	</ItemGroup>
-	<!-- Package the build output into the "analyser" slot in the NuGet package -->
+	<!-- Package the build output into the "analyser" slot in the NuGet package.
+	     The generator assembly itself is packed by the SDK (Purview.DotNetProjectSdk)
+	     into analyzers/dotnet/cs; only the refactorings assembly is added here. -->
 	<ItemGroup>
-		<None Include="../../../assets/images/purview-logo.jpg" Pack="true" PackagePath="/Sdk/purview-logo.jpg" />
+		<None Include="../../../assets/images/purview-logo.jpg" Pack="true" PackagePath="/" />
 		<None Include="..\..\..\README.md" Pack="true" PackagePath="/Sdk/" />
 		<None Include="..\..\..\LICENSE.md" Pack="true" PackagePath="/Sdk/" />
-		<None
-			Include="$(OutputPath)\$(AssemblyName).dll"
-			Pack="true"
-			PackagePath="analyzers/dotnet/cs"
-			Visible="false"
-		/>
 		<None
 			Include="..\SourceGenerator.Refactorings\bin\$(Configuration)\netstandard2.0\Purview.Telemetry.SourceGenerator.Refactorings.dll"
 			Pack="true"