Style guide: clarify final for javaagent modules (option 2 - use final in javaagent modules)

docs/contributing/style-guide.md

@@ -80,8 +80,7 @@ methods.
 ### `final` keyword usage
 
 Public non-internal non-test classes should be declared `final` where possible.
-"Internal" here includes `.internal` packages **and** `javaagent/src/main/` classes — javaagent
-instrumentation code is not public API.
+"Internal" here means classes nested anywhere under a `.internal` package.
 "Test" here includes `src/test/` directories and any module whose directory name starts or ends
 with `testing` or `tests` (e.g., `testing/`, `testing-common/`, `testing-apps/`,
 `quarkus-2.0-testing/`, `smoke-tests/`).

instrumentation/activej-http-6.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/activejhttp/ActivejHttpServerInstrumentationModule.java

@@ -16,7 +16,7 @@
 import net.bytebuddy.matcher.ElementMatcher;
 
 @AutoService(InstrumentationModule.class)
-public class ActivejHttpServerInstrumentationModule extends InstrumentationModule
+public final class ActivejHttpServerInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
 
   public ActivejHttpServerInstrumentationModule() {

instrumentation/akka/akka-actor-2.3/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkaactor/AkkaActorInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class AkkaActorInstrumentationModule extends InstrumentationModule
+public final class AkkaActorInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public AkkaActorInstrumentationModule() {
     super("akka-actor", "akka-actor-2.3");

instrumentation/akka/akka-actor-2.3/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkaactor/AkkaIgnoredTypesConfigurer.java

@@ -10,7 +10,7 @@
 import io.opentelemetry.javaagent.extension.ignore.IgnoredTypesConfigurer;
 
 @AutoService(IgnoredTypesConfigurer.class)
-public class AkkaIgnoredTypesConfigurer implements IgnoredTypesConfigurer {
+public final class AkkaIgnoredTypesConfigurer implements IgnoredTypesConfigurer {
 
   @Override
   public void configure(IgnoredTypesBuilder builder) {

instrumentation/akka/akka-actor-2.3/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkaactor/VirtualFields.java

@@ -10,7 +10,7 @@
 import io.opentelemetry.instrumentation.api.util.VirtualField;
 import io.opentelemetry.javaagent.bootstrap.executors.PropagatedContext;
 
-public class VirtualFields {
+public final class VirtualFields {
 
   public static final VirtualField<Envelope, PropagatedContext> ENVELOPE_PROPAGATED_CONTEXT =
       VirtualField.find(Envelope.class, PropagatedContext.class);

instrumentation/akka/akka-actor-fork-join-2.5/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkaforkjoin/AkkaActorForkJoinInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class AkkaActorForkJoinInstrumentationModule extends InstrumentationModule
+public final class AkkaActorForkJoinInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public AkkaActorForkJoinInstrumentationModule() {
     super("akka-actor-fork-join", "akka-actor-fork-join-2.5", "akka-actor");

instrumentation/akka/akka-actor-fork-join-2.5/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkaforkjoin/VirtualFields.java

@@ -10,7 +10,7 @@
 import io.opentelemetry.javaagent.bootstrap.executors.PropagatedContext;
 import java.util.concurrent.Callable;
 
-public class VirtualFields {
+public final class VirtualFields {
 
   public static final VirtualField<ForkJoinTask<?>, PropagatedContext>
       FORK_JOIN_TASK_PROPAGATED_CONTEXT =

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/AkkaHttpUtil.java

@@ -13,7 +13,7 @@
 import java.util.List;
 import javax.annotation.Nullable;
 
-public class AkkaHttpUtil {
+public final class AkkaHttpUtil {
 
   private static final String INSTRUMENTATION_NAME = "io.opentelemetry.akka-http-10.0";
 

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/client/AkkaHttpClientInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class AkkaHttpClientInstrumentationModule extends InstrumentationModule
+public final class AkkaHttpClientInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public AkkaHttpClientInstrumentationModule() {
     super("akka-http", "akka-http-10.0", "akka-http-client");

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/client/AkkaHttpClientSingletons.java

@@ -12,7 +12,7 @@
 import io.opentelemetry.javaagent.bootstrap.internal.JavaagentHttpClientInstrumenters;
 import io.opentelemetry.javaagent.instrumentation.akkahttp.AkkaHttpUtil;
 
-public class AkkaHttpClientSingletons {
+public final class AkkaHttpClientSingletons {
 
   private static final HttpHeaderSetter SETTER;
   private static final Instrumenter<HttpRequest, HttpResponse> INSTRUMENTER;

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/client/HttpHeaderSetter.java

@@ -12,7 +12,7 @@
 import io.opentelemetry.context.propagation.TextMapSetter;
 import javax.annotation.Nullable;
 
-public class HttpHeaderSetter implements TextMapSetter<HttpHeaderSetter.AkkaHttpHeaders> {
+public final class HttpHeaderSetter implements TextMapSetter<HttpHeaderSetter.AkkaHttpHeaders> {
 
   private final ContextPropagators contextPropagators;
 

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/client/OnCompleteHandler.java

@@ -13,7 +13,7 @@
 import scala.runtime.AbstractFunction1;
 import scala.util.Try;
 
-public class OnCompleteHandler extends AbstractFunction1<Try<HttpResponse>, Void> {
+public final class OnCompleteHandler extends AbstractFunction1<Try<HttpResponse>, Void> {
   private final Context context;
   private final HttpRequest request;
 

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/server/AkkaFlowWrapper.java

@@ -31,7 +31,7 @@
 import java.util.List;
 import javax.annotation.Nullable;
 
-public class AkkaFlowWrapper
+public final class AkkaFlowWrapper
     extends GraphStage<BidiShape<HttpResponse, HttpResponse, HttpRequest, HttpRequest>> {
   private final Inlet<HttpRequest> requestIn = Inlet.create("otel.requestIn");
   private final Outlet<HttpRequest> requestOut = Outlet.create("otel.requestOut");

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/server/AkkaHttpServerInstrumentationModule.java

@@ -16,7 +16,7 @@
 import net.bytebuddy.matcher.ElementMatcher;
 
 @AutoService(InstrumentationModule.class)
-public class AkkaHttpServerInstrumentationModule extends InstrumentationModule
+public final class AkkaHttpServerInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public AkkaHttpServerInstrumentationModule() {
     super("akka-http", "akka-http-10.0", "akka-http-server");

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/server/AkkaServerIgnoredTypesConfigurer.java

@@ -10,7 +10,7 @@
 import io.opentelemetry.javaagent.extension.ignore.IgnoredTypesConfigurer;
 
 @AutoService(IgnoredTypesConfigurer.class)
-public class AkkaServerIgnoredTypesConfigurer implements IgnoredTypesConfigurer {
+public final class AkkaServerIgnoredTypesConfigurer implements IgnoredTypesConfigurer {
 
   @Override
   public void configure(IgnoredTypesBuilder builder) {

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/server/route/AkkaHttpServerRouteInstrumentationModule.java

@@ -18,7 +18,7 @@
  * AkkaHttpServerInstrumentationModule applies to classes in akka-http-core.jar
  */
 @AutoService(InstrumentationModule.class)
-public class AkkaHttpServerRouteInstrumentationModule extends InstrumentationModule
+public final class AkkaHttpServerRouteInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public AkkaHttpServerRouteInstrumentationModule() {
     super("akka-http", "akka-http-10.0", "akka-http-server", "akka-http-server-route");

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/server/route/AkkaRouteHolder.java

@@ -15,7 +15,7 @@
 import java.util.Deque;
 import javax.annotation.Nullable;
 
-public class AkkaRouteHolder implements ImplicitContextKeyed {
+public final class AkkaRouteHolder implements ImplicitContextKeyed {
   private static final ContextKey<AkkaRouteHolder> KEY = named("opentelemetry-akka-route");
 
   private StringBuilder route = new StringBuilder();

instrumentation/akka/akka-http-10.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/akkahttp/server/route/AkkaRouteWrapper.java

@@ -12,7 +12,7 @@
 import scala.concurrent.Future;
 import scala.runtime.AbstractFunction1;
 
-public class AkkaRouteWrapper extends AbstractFunction1<RequestContext, Future<RouteResult>> {
+public final class AkkaRouteWrapper extends AbstractFunction1<RequestContext, Future<RouteResult>> {
   private final Function1<RequestContext, Future<RouteResult>> route;
 
   public AkkaRouteWrapper(Function1<RequestContext, Future<RouteResult>> route) {

instrumentation/alibaba-druid-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/alibabadruid/v1_0/DruidInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class DruidInstrumentationModule extends InstrumentationModule
+public final class DruidInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
 
   public DruidInstrumentationModule() {

instrumentation/apache-dbcp-2.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apachedbcp/v2_0/ApacheDbcpInstrumentationModule.java

@@ -13,7 +13,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class ApacheDbcpInstrumentationModule extends InstrumentationModule {
+public final class ApacheDbcpInstrumentationModule extends InstrumentationModule {
   public ApacheDbcpInstrumentationModule() {
     super("apache-dbcp", "apache-dbcp-2.0");
   }

instrumentation/apache-dubbo-2.7/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apachedubbo/v2_7/DubboInstrumentationModule.java

@@ -21,7 +21,7 @@
 import net.bytebuddy.matcher.ElementMatcher;
 
 @AutoService(InstrumentationModule.class)
-public class DubboInstrumentationModule extends InstrumentationModule
+public final class DubboInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public DubboInstrumentationModule() {
     super("apache-dubbo", "apache-dubbo-2.7");

instrumentation/apache-elasticjob-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apacheelasticjob/v3_0/ElasticJobInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class ElasticJobInstrumentationModule extends InstrumentationModule
+public final class ElasticJobInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
 
   public ElasticJobInstrumentationModule() {

instrumentation/apache-httpasyncclient-4.1/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apachehttpasyncclient/ApacheHttpAsyncClientInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class ApacheHttpAsyncClientInstrumentationModule extends InstrumentationModule
+public final class ApacheHttpAsyncClientInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public ApacheHttpAsyncClientInstrumentationModule() {
     super("apache-httpasyncclient", "apache-httpasyncclient-4.1");

instrumentation/apache-httpclient/apache-httpclient-2.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apachehttpclient/v2_0/ApacheHttpClientInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class ApacheHttpClientInstrumentationModule extends InstrumentationModule
+public final class ApacheHttpClientInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
 
   public ApacheHttpClientInstrumentationModule() {

instrumentation/apache-httpclient/apache-httpclient-4.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apachehttpclient/v4_0/ApacheHttpClientInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class ApacheHttpClientInstrumentationModule extends InstrumentationModule
+public final class ApacheHttpClientInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
 
   public ApacheHttpClientInstrumentationModule() {

instrumentation/apache-httpclient/apache-httpclient-5.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apachehttpclient/v5_0/ApacheHttpClientInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class ApacheHttpClientInstrumentationModule extends InstrumentationModule
+public final class ApacheHttpClientInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
 
   public ApacheHttpClientInstrumentationModule() {

instrumentation/apache-httpclient/apache-httpclient-5.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apachehttpclient/v5_0/RequestWithHost.java

@@ -13,7 +13,7 @@
 import org.apache.hc.core5.http.message.HttpRequestWrapper;
 import org.apache.hc.core5.net.URIAuthority;
 
-public class RequestWithHost extends HttpRequestWrapper implements ClassicHttpRequest {
+public final class RequestWithHost extends HttpRequestWrapper implements ClassicHttpRequest {
 
   @Nullable private final String scheme;
   @Nullable private final URIAuthority authority;

instrumentation/apache-httpclient/apache-httpclient-5.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apachehttpclient/v5_0/WrappingStatusSettingResponseHandler.java

@@ -15,7 +15,7 @@
 import org.apache.hc.core5.http.HttpException;
 import org.apache.hc.core5.http.io.HttpClientResponseHandler;
 
-public class WrappingStatusSettingResponseHandler<T> implements HttpClientResponseHandler<T> {
+public final class WrappingStatusSettingResponseHandler<T> implements HttpClientResponseHandler<T> {
   private final Context context;
   private final Context parentContext;
   private final ClassicHttpRequest request;

instrumentation/apache-shenyu-2.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apacheshenyu/v2_4/ApacheShenYuInstrumentationModule.java

@@ -13,7 +13,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class ApacheShenYuInstrumentationModule extends InstrumentationModule {
+public final class ApacheShenYuInstrumentationModule extends InstrumentationModule {
   public ApacheShenYuInstrumentationModule() {
     super("apache-shenyu", "apache-shenyu-2.4");
   }

instrumentation/armeria/armeria-1.3/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/armeria/v1_3/ArmeriaInstrumentationModule.java

@@ -16,7 +16,7 @@
 import net.bytebuddy.matcher.ElementMatcher;
 
 @AutoService(InstrumentationModule.class)
-public class ArmeriaInstrumentationModule extends InstrumentationModule
+public final class ArmeriaInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public ArmeriaInstrumentationModule() {
     super("armeria", "armeria-1.3");

instrumentation/armeria/armeria-1.3/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/armeria/v1_3/SubscriberWrapper.java

@@ -10,7 +10,7 @@
 import org.reactivestreams.Subscriber;
 import org.reactivestreams.Subscription;
 
-public class SubscriberWrapper<T> implements Subscriber<T> {
+public final class SubscriberWrapper<T> implements Subscriber<T> {
   private static final Class<?> abortingSubscriberClass = getAbortingSubscriberClass();
   private static final Class<?> noopSubscriberClass = getNoopSubscriberClass();
 

instrumentation/armeria/armeria-grpc-1.14/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/armeria/grpc/v1_14/ArmeriaGrpcInstrumentationModule.java

@@ -13,7 +13,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class ArmeriaGrpcInstrumentationModule extends InstrumentationModule {
+public final class ArmeriaGrpcInstrumentationModule extends InstrumentationModule {
   public ArmeriaGrpcInstrumentationModule() {
     super("armeria-grpc", "armeria-grpc-1.14", "armeria", "armeria-1.14");
   }

instrumentation/async-http-client/async-http-client-1-common/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/asynchttpclient/common/ResponseInstrumentation.java

@@ -22,7 +22,7 @@
 import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.matcher.ElementMatcher;
 
-public class ResponseInstrumentation implements TypeInstrumentation {
+public final class ResponseInstrumentation implements TypeInstrumentation {
 
   @Override
   public ElementMatcher<ClassLoader> classLoaderOptimization() {

instrumentation/async-http-client/async-http-client-1.8/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/asynchttpclient/v1_8/AsyncHttpClientInstrumentationModule.java

@@ -17,7 +17,7 @@
 import net.bytebuddy.matcher.ElementMatcher;
 
 @AutoService(InstrumentationModule.class)
-public class AsyncHttpClientInstrumentationModule extends InstrumentationModule
+public final class AsyncHttpClientInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public AsyncHttpClientInstrumentationModule() {
     super("async-http-client", "async-http-client-1.8");

instrumentation/async-http-client/async-http-client-1.9/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/asynchttpclient/v1_9/AsyncHttpClientInstrumentationModule.java

@@ -17,7 +17,7 @@
 import net.bytebuddy.matcher.ElementMatcher;
 
 @AutoService(InstrumentationModule.class)
-public class AsyncHttpClientInstrumentationModule extends InstrumentationModule
+public final class AsyncHttpClientInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public AsyncHttpClientInstrumentationModule() {
     super("async-http-client", "async-http-client-1.9");

instrumentation/async-http-client/async-http-client-2.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/asynchttpclient/v2_0/AsyncHttpClientInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class AsyncHttpClientInstrumentationModule extends InstrumentationModule
+public final class AsyncHttpClientInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public AsyncHttpClientInstrumentationModule() {
     super("async-http-client", "async-http-client-2.0");

instrumentation/avaje-jex-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/avaje/jex/v3_0/JexInstrumentationModule.java

@@ -14,7 +14,7 @@
 
 @SuppressWarnings("unused")
 @AutoService(InstrumentationModule.class)
-public class JexInstrumentationModule extends InstrumentationModule {
+public final class JexInstrumentationModule extends InstrumentationModule {
 
   public JexInstrumentationModule() {
     super("avaje-jex", "avaje-jex-3.0");

instrumentation/aws-lambda/aws-lambda-core-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/awslambdacore/v1_0/AwsLambdaInstrumentationModule.java

@@ -17,7 +17,7 @@
 import net.bytebuddy.matcher.ElementMatcher;
 
 @AutoService(InstrumentationModule.class)
-public class AwsLambdaInstrumentationModule extends InstrumentationModule
+public final class AwsLambdaInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
 
   public AwsLambdaInstrumentationModule() {

instrumentation/aws-lambda/aws-lambda-events-2.2/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/awslambdaevents/v2_2/AwsLambdaInstrumentationModule.java

@@ -16,7 +16,7 @@
 import net.bytebuddy.matcher.ElementMatcher;
 
 @AutoService(InstrumentationModule.class)
-public class AwsLambdaInstrumentationModule extends InstrumentationModule
+public final class AwsLambdaInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public AwsLambdaInstrumentationModule() {
     super("aws-lambda-events", "aws-lambda-events-2.2", "aws-lambda");

instrumentation/aws-sdk/aws-sdk-1.11/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/awssdk/v1_11/AwsSdkInstrumentationModule.java

@@ -14,7 +14,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class AwsSdkInstrumentationModule extends InstrumentationModule
+public final class AwsSdkInstrumentationModule extends InstrumentationModule
     implements ExperimentalInstrumentationModule {
   public AwsSdkInstrumentationModule() {
     super("aws-sdk", "aws-sdk-1.11", "aws-sdk-1.11-core");

instrumentation/aws-sdk/aws-sdk-1.11/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/awssdk/v1_11/SqsInstrumentationModule.java

@@ -14,7 +14,7 @@
 import net.bytebuddy.asm.Advice;
 
 @AutoService(InstrumentationModule.class)
-public class SqsInstrumentationModule extends AbstractAwsSdkInstrumentationModule {
+public final class SqsInstrumentationModule extends AbstractAwsSdkInstrumentationModule {
 
   public SqsInstrumentationModule() {
     super("aws-sdk-1.11-sqs");

instrumentation/aws-sdk/aws-sdk-1.11/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/awssdk/v1_11/TracingRequestHandler.java

@@ -28,7 +28,7 @@
 // NB: If the error-handling workarounds stop working, we should consider introducing the same
 // x-amzn-request-id header check in Apache instrumentation for suppressing spans that we have in
 // Netty instrumentation.
-public class TracingRequestHandler extends RequestHandler2 {
+public final class TracingRequestHandler extends RequestHandler2 {
 
   public static final HandlerContextKey<Scope> SCOPE =
       new HandlerContextKey<>(Scope.class.getName());

instrumentation/aws-sdk/aws-sdk-2.2/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/awssdk/v2_2/AwsSdkInstrumentationModule.java

@@ -17,7 +17,7 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class AwsSdkInstrumentationModule extends AbstractAwsSdkInstrumentationModule {
+public final class AwsSdkInstrumentationModule extends AbstractAwsSdkInstrumentationModule {
   public AwsSdkInstrumentationModule() {
     super("aws-sdk-2.2-core");
   }

instrumentation/aws-sdk/aws-sdk-2.2/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/awssdk/v2_2/BedrockRuntimeInstrumentationModule.java

@@ -19,7 +19,7 @@
 import net.bytebuddy.matcher.ElementMatcher;
 
 @AutoService(InstrumentationModule.class)
-public class BedrockRuntimeInstrumentationModule extends AbstractAwsSdkInstrumentationModule {
+public final class BedrockRuntimeInstrumentationModule extends AbstractAwsSdkInstrumentationModule {
 
   public BedrockRuntimeInstrumentationModule() {
     super("aws-sdk-2.2-bedrock-runtime");