Fix double registration in MSTest.AotReflection.SourceGeneration PoC and make it packable (non-shipping)

[Amaury Levé (Evangelink)]

What

Two fixes to the experimental MSTest.AotReflection.SourceGeneration PoC:

1. Fix double test registration. The PoC .csproj<Compile Include>-ed MSTest.SourceGeneration's ReflectionMetadataGenerator.cs — which is itself decorated with [Generator]. As a result, the PoC analyzer assembly ran two source generators, and each emitted a [ModuleInitializer] that called ReflectionMetadataHook.Register(...) for the same assembly:

   • MSTestSourceGeneratedReflectionMetadata (shipping generator, 3-arg Register)
   • MSTestAotSourceGeneratedReflectionMetadata (PoC generator, 5-arg Register)

   CompositeSourceGeneratedReflectionDataProvider appends both providers with no de-duplication, so every test class was discovered twice. A trivial 2-test project reported 4 passing tests.

   The PoC's own RuntimeRegistrationEmitter already emits a complete and strictly richer registration (5-arg overload carrying materialized type/assembly attributes) plus [DynamicDependency] rooting, so the shared shipping generator was redundant. The three shared Compile includes (ReflectionMetadataGenerator.cs, ReflectionMetadataEmitter.cs, TestAssemblyMetadata.cs) — none of which were referenced by any PoC source — are removed. Only one [ModuleInitializer] is emitted now.

2. Make the project packable as a non-shipping package (IsPackable=true, IsShipping=false) so it can be consumed locally for experimentation. The generator dll is packed under analyzers/dotnet/cs. The package lands in artifacts/packages/<config>/NonShipping and is not published.

Verification

• MSTest.AotReflection.SourceGeneration.UnitTests: 66/66 pass.
• Stood up a minimal consuming project that references the generator as an analyzer:
  • Before: a 2-test project reported total: 4 (each test discovered twice).
  • After: reports total: 2, and only the PoC's generated files are emitted (the duplicate Sample.MSTestReflectionMetadata.g.cs from the shipping generator is gone).
• dotnet pack produces MSTest.AotReflection.SourceGeneration.<ver>.nupkg in NonShipping, with the analyzer dll at analyzers/dotnet/cs/.

Notes

This remains a PoC tracked by #1837. The reflection-free registry it emits is still not consumed by the adapter (runtime discovery continues to go through ReflectionMetadataHook + [DynamicDependency]), which is out of scope for this change.

[Copilot]

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

This PR fixes duplicate MSTest registration in the experimental MSTest.AotReflection.SourceGeneration generator by removing shared “shipping” generator sources, and makes the PoC packable as a non-shipping NuGet analyzer for local experimentation.

Changes:

• Remove shared MSTest.SourceGeneration generator/emitters/models Compile Includes to prevent double [ModuleInitializer] registration.
• Enable packing (IsPackable=true, IsShipping=false) and add a pack rule to place the analyzer DLL under analyzers/dotnet/cs.
• Update package description metadata to a dedicated PackageDescription.

Show a summary per file

File	Description
src/Analyzers/MSTest.AotReflection.SourceGeneration/MSTest.AotReflection.SourceGeneration.csproj	Removes shared generator sources to prevent double registration; enables non-shipping packing and adds analyzer packing entry.

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 3

[Amaury Levé (Evangelink)]

What

Two fixes to the experimental MSTest.AotReflection.SourceGeneration PoC:

1. Fix double test registration. The PoC .csproj<Compile Include>-ed MSTest.SourceGeneration's ReflectionMetadataGenerator.cs — which is itself decorated with [Generator]. As a result, the PoC analyzer assembly ran two source generators, and each emitted a [ModuleInitializer] that called ReflectionMetadataHook.Register(...) for the same assembly:

   • MSTestSourceGeneratedReflectionMetadata (shipping generator, 3-arg Register)
   • MSTestAotSourceGeneratedReflectionMetadata (PoC generator, 5-arg Register)

   CompositeSourceGeneratedReflectionDataProvider appends both providers with no de-duplication, so every test class was discovered twice. A trivial 2-test project reported 4 passing tests.

   The PoC's own RuntimeRegistrationEmitter already emits a complete and strictly richer registration (5-arg overload carrying materialized type/assembly attributes) plus [DynamicDependency] rooting, so the shared shipping generator was redundant. The three shared Compile includes (ReflectionMetadataGenerator.cs, ReflectionMetadataEmitter.cs, TestAssemblyMetadata.cs) — none of which were referenced by any PoC source — are removed. Only one [ModuleInitializer] is emitted now.

2. Make the project packable as a non-shipping package (IsPackable=true, IsShipping=false) so it can be consumed locally for experimentation. The generator dll is packed under analyzers/dotnet/cs. The package lands in artifacts/packages/<config>/NonShipping and is not published.

Verification

• MSTest.AotReflection.SourceGeneration.UnitTests: 66/66 pass.
• Stood up a minimal consuming project that references the generator as an analyzer:
  • Before: a 2-test project reported total: 4 (each test discovered twice).
  • After: reports total: 2, and only the PoC's generated files are emitted (the duplicate Sample.MSTestReflectionMetadata.g.cs from the shipping generator is gone).
• dotnet pack produces MSTest.AotReflection.SourceGeneration.<ver>.nupkg in NonShipping, with the analyzer dll at analyzers/dotnet/cs/.

Notes

This remains a PoC tracked by #1837. The reflection-free registry it emits is still not consumed by the adapter (runtime discovery continues to go through ReflectionMetadataHook + [DynamicDependency]), which is out of scope for this change.

[Copilot]

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

This PR fixes duplicate MSTest registration in the experimental MSTest.AotReflection.SourceGeneration generator by removing shared “shipping” generator sources, and makes the PoC packable as a non-shipping NuGet analyzer for local experimentation.

Changes:

• Remove shared MSTest.SourceGeneration generator/emitters/models Compile Includes to prevent double [ModuleInitializer] registration.
• Enable packing (IsPackable=true, IsShipping=false) and add a pack rule to place the analyzer DLL under analyzers/dotnet/cs.
• Update package description metadata to a dedicated PackageDescription.

Show a summary per file

File	Description
src/Analyzers/MSTest.AotReflection.SourceGeneration/MSTest.AotReflection.SourceGeneration.csproj	Removes shared generator sources to prevent double registration; enables non-shipping packing and adds analyzer packing entry.

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 3

[Amaury Levé (Evangelink)]

What

Two fixes to the experimental MSTest.AotReflection.SourceGeneration PoC:

1. Fix double test registration. The PoC .csproj<Compile Include>-ed MSTest.SourceGeneration's ReflectionMetadataGenerator.cs — which is itself decorated with [Generator]. As a result, the PoC analyzer assembly ran two source generators, and each emitted a [ModuleInitializer] that called ReflectionMetadataHook.Register(...) for the same assembly:

   • MSTestSourceGeneratedReflectionMetadata (shipping generator, 3-arg Register)
   • MSTestAotSourceGeneratedReflectionMetadata (PoC generator, 5-arg Register)

   CompositeSourceGeneratedReflectionDataProvider appends both providers with no de-duplication, so every test class was discovered twice. A trivial 2-test project reported 4 passing tests.

   The PoC's own RuntimeRegistrationEmitter already emits a complete and strictly richer registration (5-arg overload carrying materialized type/assembly attributes) plus [DynamicDependency] rooting, so the shared shipping generator was redundant. The three shared Compile includes (ReflectionMetadataGenerator.cs, ReflectionMetadataEmitter.cs, TestAssemblyMetadata.cs) — none of which were referenced by any PoC source — are removed. Only one [ModuleInitializer] is emitted now.

2. Make the project packable as a non-shipping package (IsPackable=true, IsShipping=false) so it can be consumed locally for experimentation. The generator dll is packed under analyzers/dotnet/cs. The package lands in artifacts/packages/<config>/NonShipping and is not published.

Verification

• MSTest.AotReflection.SourceGeneration.UnitTests: 66/66 pass.
• Stood up a minimal consuming project that references the generator as an analyzer:
  • Before: a 2-test project reported total: 4 (each test discovered twice).
  • After: reports total: 2, and only the PoC's generated files are emitted (the duplicate Sample.MSTestReflectionMetadata.g.cs from the shipping generator is gone).
• dotnet pack produces MSTest.AotReflection.SourceGeneration.<ver>.nupkg in NonShipping, with the analyzer dll at analyzers/dotnet/cs/.

Notes

This remains a PoC tracked by #1837. The reflection-free registry it emits is still not consumed by the adapter (runtime discovery continues to go through ReflectionMetadataHook + [DynamicDependency]), which is out of scope for this change.

[Copilot]

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

This PR fixes duplicate MSTest registration in the experimental MSTest.AotReflection.SourceGeneration generator by removing shared “shipping” generator sources, and makes the PoC packable as a non-shipping NuGet analyzer for local experimentation.

Changes:

• Remove shared MSTest.SourceGeneration generator/emitters/models Compile Includes to prevent double [ModuleInitializer] registration.
• Enable packing (IsPackable=true, IsShipping=false) and add a pack rule to place the analyzer DLL under analyzers/dotnet/cs.
• Update package description metadata to a dedicated PackageDescription.

Show a summary per file

File	Description
src/Analyzers/MSTest.AotReflection.SourceGeneration/MSTest.AotReflection.SourceGeneration.csproj	Removes shared generator sources to prevent double registration; enables non-shipping packing and adds analyzer packing entry.

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 3

[Amaury Levé (Evangelink)]

What

Two fixes to the experimental MSTest.AotReflection.SourceGeneration PoC:

1. Fix double test registration. The PoC .csproj<Compile Include>-ed MSTest.SourceGeneration's ReflectionMetadataGenerator.cs — which is itself decorated with [Generator]. As a result, the PoC analyzer assembly ran two source generators, and each emitted a [ModuleInitializer] that called ReflectionMetadataHook.Register(...) for the same assembly:

   • MSTestSourceGeneratedReflectionMetadata (shipping generator, 3-arg Register)
   • MSTestAotSourceGeneratedReflectionMetadata (PoC generator, 5-arg Register)

   CompositeSourceGeneratedReflectionDataProvider appends both providers with no de-duplication, so every test class was discovered twice. A trivial 2-test project reported 4 passing tests.

   The PoC's own RuntimeRegistrationEmitter already emits a complete and strictly richer registration (5-arg overload carrying materialized type/assembly attributes) plus [DynamicDependency] rooting, so the shared shipping generator was redundant. The three shared Compile includes (ReflectionMetadataGenerator.cs, ReflectionMetadataEmitter.cs, TestAssemblyMetadata.cs) — none of which were referenced by any PoC source — are removed. Only one [ModuleInitializer] is emitted now.

2. Make the project packable as a non-shipping package (IsPackable=true, IsShipping=false) so it can be consumed locally for experimentation. The generator dll is packed under analyzers/dotnet/cs. The package lands in artifacts/packages/<config>/NonShipping and is not published.

Verification

• MSTest.AotReflection.SourceGeneration.UnitTests: 66/66 pass.
• Stood up a minimal consuming project that references the generator as an analyzer:
  • Before: a 2-test project reported total: 4 (each test discovered twice).
  • After: reports total: 2, and only the PoC's generated files are emitted (the duplicate Sample.MSTestReflectionMetadata.g.cs from the shipping generator is gone).
• dotnet pack produces MSTest.AotReflection.SourceGeneration.<ver>.nupkg in NonShipping, with the analyzer dll at analyzers/dotnet/cs/.

Notes

This remains a PoC tracked by #1837. The reflection-free registry it emits is still not consumed by the adapter (runtime discovery continues to go through ReflectionMetadataHook + [DynamicDependency]), which is out of scope for this change.

[Copilot]

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

This PR fixes duplicate MSTest registration in the experimental MSTest.AotReflection.SourceGeneration generator by removing shared “shipping” generator sources, and makes the PoC packable as a non-shipping NuGet analyzer for local experimentation.

Changes:

• Remove shared MSTest.SourceGeneration generator/emitters/models Compile Includes to prevent double [ModuleInitializer] registration.
• Enable packing (IsPackable=true, IsShipping=false) and add a pack rule to place the analyzer DLL under analyzers/dotnet/cs.
• Update package description metadata to a dedicated PackageDescription.

Show a summary per file

File	Description
src/Analyzers/MSTest.AotReflection.SourceGeneration/MSTest.AotReflection.SourceGeneration.csproj	Removes shared generator sources to prevent double registration; enables non-shipping packing and adds analyzer packing entry.

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 3

[Amaury Levé (Evangelink)]

What

Two fixes to the experimental MSTest.AotReflection.SourceGeneration PoC:

1. Fix double test registration. The PoC .csproj<Compile Include>-ed MSTest.SourceGeneration's ReflectionMetadataGenerator.cs — which is itself decorated with [Generator]. As a result, the PoC analyzer assembly ran two source generators, and each emitted a [ModuleInitializer] that called ReflectionMetadataHook.Register(...) for the same assembly:

   • MSTestSourceGeneratedReflectionMetadata (shipping generator, 3-arg Register)
   • MSTestAotSourceGeneratedReflectionMetadata (PoC generator, 5-arg Register)

   CompositeSourceGeneratedReflectionDataProvider appends both providers with no de-duplication, so every test class was discovered twice. A trivial 2-test project reported 4 passing tests.

   The PoC's own RuntimeRegistrationEmitter already emits a complete and strictly richer registration (5-arg overload carrying materialized type/assembly attributes) plus [DynamicDependency] rooting, so the shared shipping generator was redundant. The three shared Compile includes (ReflectionMetadataGenerator.cs, ReflectionMetadataEmitter.cs, TestAssemblyMetadata.cs) — none of which were referenced by any PoC source — are removed. Only one [ModuleInitializer] is emitted now.

2. Make the project packable as a non-shipping package (IsPackable=true, IsShipping=false) so it can be consumed locally for experimentation. The generator dll is packed under analyzers/dotnet/cs. The package lands in artifacts/packages/<config>/NonShipping and is not published.

Verification

• MSTest.AotReflection.SourceGeneration.UnitTests: 66/66 pass.
• Stood up a minimal consuming project that references the generator as an analyzer:
  • Before: a 2-test project reported total: 4 (each test discovered twice).
  • After: reports total: 2, and only the PoC's generated files are emitted (the duplicate Sample.MSTestReflectionMetadata.g.cs from the shipping generator is gone).
• dotnet pack produces MSTest.AotReflection.SourceGeneration.<ver>.nupkg in NonShipping, with the analyzer dll at analyzers/dotnet/cs/.

Notes

This remains a PoC tracked by #1837. The reflection-free registry it emits is still not consumed by the adapter (runtime discovery continues to go through ReflectionMetadataHook + [DynamicDependency]), which is out of scope for this change.

[Copilot]

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

This PR fixes duplicate MSTest registration in the experimental MSTest.AotReflection.SourceGeneration generator by removing shared “shipping” generator sources, and makes the PoC packable as a non-shipping NuGet analyzer for local experimentation.

Changes:

• Remove shared MSTest.SourceGeneration generator/emitters/models Compile Includes to prevent double [ModuleInitializer] registration.
• Enable packing (IsPackable=true, IsShipping=false) and add a pack rule to place the analyzer DLL under analyzers/dotnet/cs.
• Update package description metadata to a dedicated PackageDescription.

Show a summary per file

File	Description
src/Analyzers/MSTest.AotReflection.SourceGeneration/MSTest.AotReflection.SourceGeneration.csproj	Removes shared generator sources to prevent double registration; enables non-shipping packing and adds analyzer packing entry.

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 3

[Amaury Levé (Evangelink)]

What

Two fixes to the experimental MSTest.AotReflection.SourceGeneration PoC:

1. Fix double test registration. The PoC .csproj<Compile Include>-ed MSTest.SourceGeneration's ReflectionMetadataGenerator.cs — which is itself decorated with [Generator]. As a result, the PoC analyzer assembly ran two source generators, and each emitted a [ModuleInitializer] that called ReflectionMetadataHook.Register(...) for the same assembly:

   • MSTestSourceGeneratedReflectionMetadata (shipping generator, 3-arg Register)
   • MSTestAotSourceGeneratedReflectionMetadata (PoC generator, 5-arg Register)

   CompositeSourceGeneratedReflectionDataProvider appends both providers with no de-duplication, so every test class was discovered twice. A trivial 2-test project reported 4 passing tests.

   The PoC's own RuntimeRegistrationEmitter already emits a complete and strictly richer registration (5-arg overload carrying materialized type/assembly attributes) plus [DynamicDependency] rooting, so the shared shipping generator was redundant. The three shared Compile includes (ReflectionMetadataGenerator.cs, ReflectionMetadataEmitter.cs, TestAssemblyMetadata.cs) — none of which were referenced by any PoC source — are removed. Only one [ModuleInitializer] is emitted now.

2. Make the project packable as a non-shipping package (IsPackable=true, IsShipping=false) so it can be consumed locally for experimentation. The generator dll is packed under analyzers/dotnet/cs. The package lands in artifacts/packages/<config>/NonShipping and is not published.

Verification

• MSTest.AotReflection.SourceGeneration.UnitTests: 66/66 pass.
• Stood up a minimal consuming project that references the generator as an analyzer:
  • Before: a 2-test project reported total: 4 (each test discovered twice).
  • After: reports total: 2, and only the PoC's generated files are emitted (the duplicate Sample.MSTestReflectionMetadata.g.cs from the shipping generator is gone).
• dotnet pack produces MSTest.AotReflection.SourceGeneration.<ver>.nupkg in NonShipping, with the analyzer dll at analyzers/dotnet/cs/.

Notes

This remains a PoC tracked by #1837. The reflection-free registry it emits is still not consumed by the adapter (runtime discovery continues to go through ReflectionMetadataHook + [DynamicDependency]), which is out of scope for this change.

[Copilot]

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

This PR fixes duplicate MSTest registration in the experimental MSTest.AotReflection.SourceGeneration generator by removing shared “shipping” generator sources, and makes the PoC packable as a non-shipping NuGet analyzer for local experimentation.

Changes:

• Remove shared MSTest.SourceGeneration generator/emitters/models Compile Includes to prevent double [ModuleInitializer] registration.
• Enable packing (IsPackable=true, IsShipping=false) and add a pack rule to place the analyzer DLL under analyzers/dotnet/cs.
• Update package description metadata to a dedicated PackageDescription.

Show a summary per file

File	Description
src/Analyzers/MSTest.AotReflection.SourceGeneration/MSTest.AotReflection.SourceGeneration.csproj	Removes shared generator sources to prevent double registration; enables non-shipping packing and adds analyzer packing entry.

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 3

[Amaury Levé (Evangelink)]

What

Two fixes to the experimental MSTest.AotReflection.SourceGeneration PoC:

1. Fix double test registration. The PoC .csproj<Compile Include>-ed MSTest.SourceGeneration's ReflectionMetadataGenerator.cs — which is itself decorated with [Generator]. As a result, the PoC analyzer assembly ran two source generators, and each emitted a [ModuleInitializer] that called ReflectionMetadataHook.Register(...) for the same assembly:

   • MSTestSourceGeneratedReflectionMetadata (shipping generator, 3-arg Register)
   • MSTestAotSourceGeneratedReflectionMetadata (PoC generator, 5-arg Register)

   CompositeSourceGeneratedReflectionDataProvider appends both providers with no de-duplication, so every test class was discovered twice. A trivial 2-test project reported 4 passing tests.

   The PoC's own RuntimeRegistrationEmitter already emits a complete and strictly richer registration (5-arg overload carrying materialized type/assembly attributes) plus [DynamicDependency] rooting, so the shared shipping generator was redundant. The three shared Compile includes (ReflectionMetadataGenerator.cs, ReflectionMetadataEmitter.cs, TestAssemblyMetadata.cs) — none of which were referenced by any PoC source — are removed. Only one [ModuleInitializer] is emitted now.

2. Make the project packable as a non-shipping package (IsPackable=true, IsShipping=false) so it can be consumed locally for experimentation. The generator dll is packed under analyzers/dotnet/cs. The package lands in artifacts/packages/<config>/NonShipping and is not published.

Verification

• MSTest.AotReflection.SourceGeneration.UnitTests: 66/66 pass.
• Stood up a minimal consuming project that references the generator as an analyzer:
  • Before: a 2-test project reported total: 4 (each test discovered twice).
  • After: reports total: 2, and only the PoC's generated files are emitted (the duplicate Sample.MSTestReflectionMetadata.g.cs from the shipping generator is gone).
• dotnet pack produces MSTest.AotReflection.SourceGeneration.<ver>.nupkg in NonShipping, with the analyzer dll at analyzers/dotnet/cs/.

Notes

This remains a PoC tracked by #1837. The reflection-free registry it emits is still not consumed by the adapter (runtime discovery continues to go through ReflectionMetadataHook + [DynamicDependency]), which is out of scope for this change.

[Copilot]

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

This PR fixes duplicate MSTest registration in the experimental MSTest.AotReflection.SourceGeneration generator by removing shared “shipping” generator sources, and makes the PoC packable as a non-shipping NuGet analyzer for local experimentation.

Changes:

• Remove shared MSTest.SourceGeneration generator/emitters/models Compile Includes to prevent double [ModuleInitializer] registration.
• Enable packing (IsPackable=true, IsShipping=false) and add a pack rule to place the analyzer DLL under analyzers/dotnet/cs.
• Update package description metadata to a dedicated PackageDescription.

Show a summary per file

File	Description
src/Analyzers/MSTest.AotReflection.SourceGeneration/MSTest.AotReflection.SourceGeneration.csproj	Removes shared generator sources to prevent double registration; enables non-shipping packing and adds analyzer packing entry.

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 3

[Amaury Levé (Evangelink)]

What

Two fixes to the experimental MSTest.AotReflection.SourceGeneration PoC:

1. Fix double test registration. The PoC .csproj<Compile Include>-ed MSTest.SourceGeneration's ReflectionMetadataGenerator.cs — which is itself decorated with [Generator]. As a result, the PoC analyzer assembly ran two source generators, and each emitted a [ModuleInitializer] that called ReflectionMetadataHook.Register(...) for the same assembly:

   • MSTestSourceGeneratedReflectionMetadata (shipping generator, 3-arg Register)
   • MSTestAotSourceGeneratedReflectionMetadata (PoC generator, 5-arg Register)

   CompositeSourceGeneratedReflectionDataProvider appends both providers with no de-duplication, so every test class was discovered twice. A trivial 2-test project reported 4 passing tests.

   The PoC's own RuntimeRegistrationEmitter already emits a complete and strictly richer registration (5-arg overload carrying materialized type/assembly attributes) plus [DynamicDependency] rooting, so the shared shipping generator was redundant. The three shared Compile includes (ReflectionMetadataGenerator.cs, ReflectionMetadataEmitter.cs, TestAssemblyMetadata.cs) — none of which were referenced by any PoC source — are removed. Only one [ModuleInitializer] is emitted now.

2. Make the project packable as a non-shipping package (IsPackable=true, IsShipping=false) so it can be consumed locally for experimentation. The generator dll is packed under analyzers/dotnet/cs. The package lands in artifacts/packages/<config>/NonShipping and is not published.

Verification

• MSTest.AotReflection.SourceGeneration.UnitTests: 66/66 pass.
• Stood up a minimal consuming project that references the generator as an analyzer:
  • Before: a 2-test project reported total: 4 (each test discovered twice).
  • After: reports total: 2, and only the PoC's generated files are emitted (the duplicate Sample.MSTestReflectionMetadata.g.cs from the shipping generator is gone).
• dotnet pack produces MSTest.AotReflection.SourceGeneration.<ver>.nupkg in NonShipping, with the analyzer dll at analyzers/dotnet/cs/.

Notes

This remains a PoC tracked by #1837. The reflection-free registry it emits is still not consumed by the adapter (runtime discovery continues to go through ReflectionMetadataHook + [DynamicDependency]), which is out of scope for this change.

[Copilot]

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

This PR fixes duplicate MSTest registration in the experimental MSTest.AotReflection.SourceGeneration generator by removing shared “shipping” generator sources, and makes the PoC packable as a non-shipping NuGet analyzer for local experimentation.

Changes:

• Remove shared MSTest.SourceGeneration generator/emitters/models Compile Includes to prevent double [ModuleInitializer] registration.
• Enable packing (IsPackable=true, IsShipping=false) and add a pack rule to place the analyzer DLL under analyzers/dotnet/cs.
• Update package description metadata to a dedicated PackageDescription.

Show a summary per file

File	Description
src/Analyzers/MSTest.AotReflection.SourceGeneration/MSTest.AotReflection.SourceGeneration.csproj	Removes shared generator sources to prevent double registration; enables non-shipping packing and adds analyzer packing entry.

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 3