coderabbit-test
diff --git a/‎src/main/java/com/google/devtools/build/lib/rules/java/BUILD
Lines changed: 1 addition & 4 deletions b/‎src/main/java/com/google/devtools/build/lib/rules/java/BUILD
Lines changed: 1 addition & 4 deletions
diff --git a/‎src/main/java/com/google/devtools/build/lib/rules/java/JavaCompilationHelper.java
Lines changed: 3 additions & 33 deletions b/‎src/main/java/com/google/devtools/build/lib/rules/java/JavaCompilationHelper.java
Lines changed: 3 additions & 33 deletions
diff --git a/‎src/main/java/com/google/devtools/build/lib/rules/java/JavaInfoBuildHelper.java
Lines changed: 0 additions & 295 deletions b/‎src/main/java/com/google/devtools/build/lib/rules/java/JavaInfoBuildHelper.java
Lines changed: 0 additions & 295 deletions
@@ -38,6 +38,7 @@ java_library(
         "//src/main/java/com/google/devtools/build/lib/analysis:actions/custom_command_line",
         "//src/main/java/com/google/devtools/build/lib/analysis:analysis_cluster",
         "//src/main/java/com/google/devtools/build/lib/analysis:config/build_configuration",
+        "//src/main/java/com/google/devtools/build/lib/analysis:config/core_option_converters",
         "//src/main/java/com/google/devtools/build/lib/analysis:config/execution_transition_factory",
         "//src/main/java/com/google/devtools/build/lib/analysis:config/transitions/no_transition",
         "//src/main/java/com/google/devtools/build/lib/analysis:configured_target",
@@ -52,10 +53,8 @@ java_library(
         "//src/main/java/com/google/devtools/build/lib/collect/nestedset",
         "//src/main/java/com/google/devtools/build/lib/concurrent",
         "//src/main/java/com/google/devtools/build/lib/packages",
-        "//src/main/java/com/google/devtools/build/lib/packages:exec_group",
         "//src/main/java/com/google/devtools/build/lib/packages/semantics",
         "//src/main/java/com/google/devtools/build/lib/rules/cpp",
-        "//src/main/java/com/google/devtools/build/lib/starlarkbuildapi",
         "//src/main/java/com/google/devtools/build/lib/starlarkbuildapi/core",
         "//src/main/java/com/google/devtools/build/lib/starlarkbuildapi/cpp",
         "//src/main/java/com/google/devtools/build/lib/starlarkbuildapi/java",
@@ -95,8 +94,6 @@ java_library(
         "JavaHeaderCompileAction.java",
         "JavaHelper.java",
         "JavaInfo.java",
-        "JavaInfoBuildHelper.java",
-        "JavaLibraryHelper.java",
         "JavaModuleFlagsProvider.java",
         "JavaOptions.java",
         "JavaPackageConfigurationProvider.java",
 
@@ -603,7 +603,7 @@ public boolean usesAnnotationProcessing() {
     return getJavacOpts().contains("-processor") || attributes.plugins().hasProcessors();
   }
 
-  public void createGenJarAction(
+  private void createGenJarAction(
       Artifact classJar,
       Artifact manifestProto,
       Artifact genClassJar,
@@ -663,36 +663,6 @@ private void createResourceJarAction(Artifact resourceJar, ImmutableList<Artifac
         .build(semantics, ruleContext, execGroup);
   }
 
-  /**
-   * Creates an Action that packages the Java source files into a Jar. If {@code gensrcJar} is
-   * non-null, includes the contents of the {@code gensrcJar} with the output source jar.
-   *
-   * @param outputJar the Artifact to create with the Action
-   * @param gensrcJar the generated sources jar Artifact that should be included with the sources in
-   *     the output Artifact. May be null.
-   * @param javaToolchainProvider is used by SingleJarActionBuilder to retrieve jvm options
-   */
-  public void createSourceJarAction(
-      Artifact outputJar,
-      @Nullable Artifact gensrcJar,
-      JavaToolchainProvider javaToolchainProvider) {
-    JavaTargetAttributes attributes = getAttributes();
-    NestedSetBuilder<Artifact> resourceJars = NestedSetBuilder.stableOrder();
-    resourceJars.addAll(attributes.getSourceJars());
-    if (gensrcJar != null) {
-      resourceJars.add(gensrcJar);
-    }
-    SingleJarActionBuilder.createSourceJarAction(
-        ruleContext,
-        ruleContext,
-        semantics,
-        NestedSetBuilder.<Artifact>wrap(Order.STABLE_ORDER, attributes.getSourceFiles()),
-        resourceJars.build(),
-        outputJar,
-        javaToolchainProvider,
-        execGroup);
-  }
-
   public void createSourceJarAction(Artifact outputJar, @Nullable Artifact gensrcJar) {
     JavaTargetAttributes attributes = getAttributes();
     NestedSetBuilder<Artifact> resourceJars = NestedSetBuilder.stableOrder();
@@ -746,7 +716,7 @@ public Artifact createCompileTimeJarAction(
     return jar;
   }
 
-  public void createLocalOptimizationAction(
+  private void createLocalOptimizationAction(
       Artifact unoptimizedOutputJar,
       Artifact optimizedOutputJar,
       NestedSet<Artifact> classpath,
@@ -847,7 +817,7 @@ public void addLibrariesToAttributes(Collection<? extends TransitiveInfoCollecti
    *
    * @return filtered command line flag value, defaulting to ERROR
    */
-  public StrictDepsMode getStrictJavaDeps() {
+  private StrictDepsMode getStrictJavaDeps() {
     return strictJavaDeps;
   }