Use SOURCE retention for immutables style annotations (#691)

Author: pkoenig10

tracing-api/src/main/java/com/palantir/tracing/api/ImmutablesStyle.java

@@ -0,0 +1,28 @@
+/*
+ * (c) Copyright 2021 Palantir Technologies Inc. All rights reserved.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.palantir.tracing.api;
+
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
+import org.immutables.value.Value;
+
+@Target({ElementType.PACKAGE, ElementType.TYPE})
+@Retention(RetentionPolicy.SOURCE)
+@Value.Style(visibility = Value.Style.ImplementationVisibility.PACKAGE)
+@interface ImmutablesStyle {}

tracing-api/src/main/java/com/palantir/tracing/api/OpenSpan.java

@@ -26,7 +26,7 @@
  * {@link Span} object.
  */
 @Value.Immutable
-@Value.Style(visibility = Value.Style.ImplementationVisibility.PACKAGE)
+@ImmutablesStyle
 public abstract class OpenSpan {
     private static final Clock CLOCK = Clock.systemUTC();
 

tracing-api/src/main/java/com/palantir/tracing/api/Span.java

@@ -22,7 +22,7 @@
 
 /** A value class representing a completed Span, see {@link OpenSpan} for a description of the fields. */
 @Value.Immutable
-@Value.Style(visibility = Value.Style.ImplementationVisibility.PACKAGE)
+@ImmutablesStyle
 public abstract class Span {
 
     public abstract String getTraceId();

tracing-jaxrs/build.gradle

@@ -28,8 +28,4 @@ dependencies {
     testImplementation "org.assertj:assertj-core"
     testImplementation "org.jmock:jmock"
     testImplementation "org.mockito:mockito-core"
-
-    annotationProcessor "org.immutables:value"
-    compileOnly "org.immutables:value::annotations"
-    testCompileOnly "org.immutables:value::annotations"
 }

tracing-jersey/build.gradle

@@ -30,7 +30,4 @@ dependencies {
     testImplementation "org.assertj:assertj-core"
     testImplementation "org.hamcrest:hamcrest-all"
     testImplementation "org.mockito:mockito-core"
-
-    compileOnly "org.immutables:value::annotations"
-    testCompileOnly "org.immutables:value::annotations"
 }

tracing-okhttp3/build.gradle

@@ -26,7 +26,4 @@ dependencies {
     testImplementation "junit:junit"
     testImplementation "org.assertj:assertj-core"
     testImplementation "org.mockito:mockito-core"
-
-    compileOnly "org.immutables:value::annotations"
-    testCompileOnly "org.immutables:value::annotations"
 }

tracing-servlet/build.gradle

@@ -29,7 +29,4 @@ dependencies {
     testImplementation "io.dropwizard:dropwizard-testing"
     testImplementation "junit:junit"
     testImplementation 'org.assertj:assertj-core'
-
-    compileOnly "org.immutables:value::annotations"
-    testCompileOnly "org.immutables:value::annotations"
 }

tracing-undertow/build.gradle

@@ -31,7 +31,4 @@ dependencies {
     testImplementation 'junit:junit'
     testImplementation 'org.assertj:assertj-core'
     testImplementation 'org.mockito:mockito-core'
-
-    compileOnly "org.immutables:value::annotations"
-    testCompileOnly "org.immutables:value::annotations"
 }

tracing/build.gradle

@@ -42,5 +42,4 @@ dependencies {
 
     annotationProcessor "org.immutables:value"
     compileOnly "org.immutables:value::annotations"
-    testCompileOnly "org.immutables:value::annotations"
 }

tracing/src/main/java/com/palantir/tracing/AsyncSlf4jSpanObserver.java

@@ -58,7 +58,7 @@ public final class AsyncSlf4jSpanObserver extends AsyncSpanObserver {
 
     @JsonSerialize(as = ImmutableZipkinCompatSpan.class)
     @Value.Immutable
-    @Value.Style(visibility = Value.Style.ImplementationVisibility.PACKAGE)
+    @ImmutablesStyle
     abstract static class ZipkinCompatSpan {
 
         abstract String getTraceId();
@@ -141,7 +141,7 @@ String toJson() {
 
     @JsonSerialize(as = ImmutableZipkinCompatAnnotation.class)
     @Value.Immutable
-    @Value.Style(visibility = Value.Style.ImplementationVisibility.PACKAGE)
+    @ImmutablesStyle
     abstract static class ZipkinCompatAnnotation {
         abstract long timestamp(); // epoch microseconds
 
@@ -160,7 +160,7 @@ static ZipkinCompatAnnotation of(long timestamp, String value, ZipkinCompatEndpo
 
     @JsonSerialize(as = ImmutableZipkinCompatBinaryAnnotation.class)
     @Value.Immutable
-    @Value.Style(visibility = Value.Style.ImplementationVisibility.PACKAGE)
+    @ImmutablesStyle
     abstract static class ZipkinCompatBinaryAnnotation {
 
         abstract String key();
@@ -180,7 +180,7 @@ static ZipkinCompatBinaryAnnotation of(String key, String value, ZipkinCompatEnd
 
     @JsonSerialize(as = ImmutableZipkinCompatEndpoint.class)
     @Value.Immutable
-    @Value.Style(visibility = Value.Style.ImplementationVisibility.PACKAGE)
+    @ImmutablesStyle
     @JsonInclude(JsonInclude.Include.NON_NULL)
     abstract static class ZipkinCompatEndpoint {
         abstract String serviceName();