Merge pull request #313 from distantcam/hotfix/312

Author: distantcam

.github/workflows/build.yml

@@ -6,12 +6,10 @@ on:
     branches:
       - main
       - feature/*
+      - hotfix/*
     tags:
       - "[0-9]+.[0-9]+.[0-9]+"
       - "[0-9]+.[0-9]+.[0-9]+-**"
-  pull_request:
-    branches:
-      - main
 
 jobs:
   test:

src/AutoCtor.Shared/AttributeSourceGenerator/Emitter.cs

@@ -18,7 +18,7 @@ public static SourceText GenerateSource()
             using (source.StartBlock("namespace AutoCtor"))
             {
                 source.AddCompilerGeneratedAttribute().AddGeneratedCodeAttribute();
-                using (source.StartBlock("public enum GuardSetting"))
+                using (source.StartBlock("internal enum GuardSetting"))
                 {
                     source.AppendLine("Default,");
                     source.AppendLine("Disabled,");

src/AutoCtor.Tests/GeneratedAttributeTests.AttributeGeneratedCode#AutoConstructAttribute.g.verified.cs

@@ -9,7 +9,7 @@
 namespace AutoCtor
 {
 	[global::System.Runtime.CompilerServices.CompilerGeneratedAttribute]
-	public enum GuardSetting
+	internal enum GuardSetting
 	{
 		Default,
 		Disabled,

src/AutoCtor.Tests/GeneratedAttributeTests.cs

@@ -51,4 +51,31 @@ public async Task PreserveAttributesTest()
         Assert.Empty(diagnostics);
         Assert.Empty(outputCompilation.GetDiagnostics());
     }
+
+    [Fact]
+    public async Task EnsureGeneratedAttributesAreNotExternallyVisible()
+    {
+        // Issue 312
+        var compileBuilder = new CompilationBuilder()
+            .AddNetCoreReference()
+            .WithPreprocessorSymbols(["AUTOCTOR_EMBED_ATTRIBUTES"]);
+
+        var genDriver = new GeneratorDriverBuilder()
+            .AddGenerator(new AttributeSourceGenerator())
+            .Build(compileBuilder.ParseOptions);
+
+        var projectA = await compileBuilder
+            .Build("ProjectA");
+
+        genDriver = genDriver.RunGeneratorsAndUpdateCompilation(projectA, out var genProjectA, out _);
+
+        var projectB = await compileBuilder
+            .AddCompilationReference(genProjectA)
+            .Build("ProjectB");
+
+        genDriver.RunGeneratorsAndUpdateCompilation(projectB, out var outputCompilation, out var diagnostics);
+
+        Assert.Empty(diagnostics);
+        Assert.Empty(outputCompilation.GetDiagnostics());
+    }
 }

src/AutoCtor.Tests/Utilities/CompilationBuilder.cs

@@ -78,7 +78,7 @@ public CompilationBuilder AddAssemblyReference<T>()
         };
     }
 
-    public CompilationBuilder AddCompilationReference(CSharpCompilation compilation)
+    public CompilationBuilder AddCompilationReference(Compilation compilation)
     {
         return new(this)
         {