refactor tool output

Dockerfile

@@ -29,10 +29,6 @@ RUN bundle config set --local path 'vendor/bundle' && bundle install && gem clea
 
 FROM $BASE_IMAGE
 
-ARG GEM_FOLDER
-ENV GEM_HOME=$GEM_FOLDER
-ENV PATH $GEM_HOME/bin:$GEM_HOME/gems/bin:$PATH
-
 RUN apk add --no-cache ruby ruby-etc ruby-dev ruby-irb ruby-rake ruby-io-console ruby-bigdecimal ruby-json \ 
     openjdk17-jre \
     bash

Gemfile.lock

@@ -108,7 +108,7 @@ GEM
     git_diff_parser (3.2.0)
     globalid (1.2.1)
       activesupport (>= 6.1)
-    graphql (2.3.6)
+    graphql (2.3.7)
       base64
     haml (6.3.0)
       temple (>= 0.8.2)
@@ -143,7 +143,7 @@ GEM
     matrix (0.4.2)
     method_source (1.1.0)
     mini_mime (1.1.5)
-    minitest (5.24.0)
+    minitest (5.24.1)
     mocha (2.4.0)
       ruby2_keywords (>= 0.0.5)
     msgpack (1.7.2)
@@ -158,8 +158,18 @@ GEM
     net-smtp (0.5.0)
       net-protocol
     nio4r (2.7.3)
+    nokogiri (1.16.6-aarch64-linux)
+      racc (~> 1.4)
+    nokogiri (1.16.6-arm-linux)
+      racc (~> 1.4)
+    nokogiri (1.16.6-arm64-darwin)
+      racc (~> 1.4)
+    nokogiri (1.16.6-x86-linux)
+      racc (~> 1.4)
     nokogiri (1.16.6-x86_64-darwin)
       racc (~> 1.4)
+    nokogiri (1.16.6-x86_64-linux)
+      racc (~> 1.4)
     parallel (1.25.1)
     parser (3.3.3.0)
       ast (~> 2.4.1)
@@ -266,7 +276,7 @@ GEM
     rubocop-performance (1.21.1)
       rubocop (>= 1.48.1, < 2.0)
       rubocop-ast (>= 1.31.1, < 2.0)
-    rubocop-rails (2.25.0)
+    rubocop-rails (2.25.1)
       activesupport (>= 4.2.0)
       rack (>= 1.1)
       rubocop (>= 1.33.0, < 2.0)
@@ -313,7 +323,16 @@ GEM
       actionpack (>= 6.1)
       activesupport (>= 6.1)
       sprockets (>= 3.0.0)
+    sqlite3 (2.0.2-aarch64-linux-gnu)
+    sqlite3 (2.0.2-aarch64-linux-musl)
+    sqlite3 (2.0.2-arm-linux-gnu)
+    sqlite3 (2.0.2-arm-linux-musl)
+    sqlite3 (2.0.2-arm64-darwin)
+    sqlite3 (2.0.2-x86-linux-gnu)
+    sqlite3 (2.0.2-x86-linux-musl)
     sqlite3 (2.0.2-x86_64-darwin)
+    sqlite3 (2.0.2-x86_64-linux-gnu)
+    sqlite3 (2.0.2-x86_64-linux-musl)
     standard (1.39.0)
       language_server-protocol (~> 3.17.0.2)
       lint_roller (~> 1.0)
@@ -342,7 +361,7 @@ GEM
       syntax_tree (>= 2.0.1)
     temple (0.10.3)
     thor (1.3.1)
-    tilt (2.3.0)
+    tilt (2.4.0)
     timeout (0.4.1)
     turbo-rails (2.0.5)
       actionpack (>= 6.0.0)
@@ -368,7 +387,20 @@ GEM
     zeitwerk (2.6.16)
 
 PLATFORMS
-  x86_64-darwin-23
+  aarch64-linux
+  aarch64-linux-gnu
+  aarch64-linux-musl
+  arm-linux
+  arm-linux-gnu
+  arm-linux-musl
+  arm64-darwin
+  x86-linux
+  x86-linux-gnu
+  x86-linux-musl
+  x86_64-darwin
+  x86_64-linux
+  x86_64-linux-gnu
+  x86_64-linux-musl
 
 DEPENDENCIES
   actioncable
@@ -425,4 +457,4 @@ DEPENDENCIES
   yard
 
 BUNDLED WITH
-   2.5.7
+   2.5.14

docs/patterns.json

@@ -9816,6 +9816,10 @@
       "level": "Warning",
       "category": "ErrorProne",
       "parameters": [
+        {
+          "name": "Safe",
+          "default": "false"
+        },
         {
           "name": "ForbiddenMethods",
           "default": [
@@ -10155,6 +10159,12 @@
       "patternId": "Rails_WhereRange",
       "level": "Warning",
       "category": "ErrorProne",
+      "parameters": [
+        {
+          "name": "SafeAutoCorrect",
+          "default": "false"
+        }
+      ],
       "enabled": false
     },
     {

src/main/scala/codacy/rubocop/Rubocop.scala

@@ -45,11 +45,10 @@ object Rubocop extends Tool {
       files: Option[Set[api.Source.File]],
       options: Map[Options.Key, Options.Value]
   )(implicit specification: Tool.Specification): Try[List[Result]] = {
-
     val cmd = getCommandFor(Paths.get(source.path), configuration, files, specification, resultFilePath)
     CommandRunner.exec(cmd, Some(new File(source.path))) match {
 
-      case Right(resultFromTool) if resultFromTool.exitCode < 2 =>
+      case Right(resultFromTool) =>
         parseResult(resultFilePath.toFile) match {
           case s @ Success(_) => s
           case Failure(e) =>
@@ -63,15 +62,6 @@ object Rubocop extends Tool {
             Failure(new Exception(msg))
         }
 
-      case Right(resultFromTool) =>
-        val msg =
-          s"""
-             |Rubocop exited with code ${resultFromTool.exitCode}
-             |stdout: ${resultFromTool.stdout.mkString(Properties.lineSeparator)}
-             |stderr: ${resultFromTool.stderr.mkString(Properties.lineSeparator)}
-                """.stripMargin
-        Failure(new Exception(msg))
-
       case Left(e) =>
         Failure(e)
     }
@@ -140,7 +130,14 @@ object Rubocop extends Tool {
           file.toString
       }
 
-    List("rubocop", "--force-exclusion", "-f", "json", "-o", outputFilePath.toAbsolutePath.toString) ++ configFileOptions ++ patternsCmd ++ filesCmd
+    List(
+      "rubocop",
+      "--force-exclusion",
+      "-f",
+      "json",
+      "-o",
+      outputFilePath.toAbsolutePath.toString
+    ) ++ configFileOptions ++ patternsCmd ++ filesCmd
   }
 
   private[this] lazy val resultFilePath =