spoom 1.1.3 → 1.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b5782933918e2b2f81d28af1fe48f3727fdd81970bcaeed0549fa1b1fb594523
-  data.tar.gz: 5d980328f5d64277f3c03a2cccc589b067d2b4648aad29476d9a1f26b2175d5a
+  metadata.gz: fa8ac9092af39d9a951ffe28c217f9f7516d25e316483ba9398df6fbb220f10a
+  data.tar.gz: 5140065943452ad4ca4de375c13b27bd24003fa828ac4fe9594a7f66d11ac970
 SHA512:
-  metadata.gz: e80bccb9cad071234a117a66aa6c37bbdf8787a0399b436e0c07062521fa974b28ee9f94f9bf0c2964fcfe4c6689ddf551ad3e6f9b87e2fb6caa12ad1987a381
-  data.tar.gz: c635b37ef3dcadc64251d627e3574d7a8fef883afdc1dab8e32a68765f94a10ca7a5e696f88cee533369332efc69562eec005d7f032e7f02067871285b49ba64
+  metadata.gz: 1b706423681d2acdbdd167a3076229953df609036254c3dcbc470a828078857318c79c3bdbddfa422ad9f26ddeb01fe464d6ab3173949d47cdc490d799403b94
+  data.tar.gz: 13b7086057c38d3a1344f2920c42c8c9208d0f3c8c438eae02b3f0b32a21e0a35f526e6e5607c5235af2fad02d0aa0365678f9ae7ce88d13d7404f142890fe07

data/Gemfile

@@ -5,8 +5,9 @@ source "https://rubygems.org"
 
 gemspec
 
-group(:development) do
-  gem('rubocop-shopify', require: false)
-  gem('rubocop-sorbet', require: false)
-  gem('pry-byebug')
+group :development do
+  gem "pry-byebug"
+  gem "rubocop-shopify", require: false
+  gem "rubocop-sorbet", require: false
+  gem "tapioca", require: false
 end

data/README.md

@@ -358,4 +358,4 @@ The gem is available as open source under the terms of the [MIT License](https:/
 
 ## Code of Conduct
 
-Everyone interacting in the Spoom project’s codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/Shopify/spoom/blob/master/CODE_OF_CONDUCT.md).
+Everyone interacting in the Spoom project’s codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/Shopify/spoom/blob/main/CODE_OF_CONDUCT.md).

data/lib/spoom/cli/bump.rb

@@ -83,20 +83,30 @@ module Spoom
           exit(files_to_bump.empty?)
         end
 
-        output, no_errors = Sorbet.srb_tc(
+        error_url_base = Spoom::Sorbet::Errors::DEFAULT_ERROR_URL_BASE
+        output, status, exit_code = Sorbet.srb_tc(
           "--no-error-sections",
+          "--error-url-base=#{error_url_base}",
           path: exec_path,
           capture_err: true,
           sorbet_bin: options[:sorbet]
         )
 
-        if no_errors
+        check_sorbet_segfault(exit_code) do
+          say_error(<<~ERR, status: nil)
+            It means one of the file bumped to `typed: #{to}` made Sorbet crash.
+            Run `spoom bump -f` locally followed by `bundle exec srb tc` to investigate the problem.
+          ERR
+          undo_changes(files_to_bump, from)
+        end
+
+        if status
           print_changes(files_to_bump, command: cmd, from: from, to: to, dry: dry, path: exec_path)
           undo_changes(files_to_bump, from) if dry
           exit(files_to_bump.empty?)
         end
 
-        errors = Sorbet::Errors::Parser.parse_string(output)
+        errors = Sorbet::Errors::Parser.parse_string(output, error_url_base: error_url_base)
 
         files_with_errors = errors.map do |err|
           path = File.expand_path(err.file)

data/lib/spoom/cli/helper.rb

@@ -10,9 +10,10 @@ module Spoom
     module Helper
       extend T::Sig
       extend T::Helpers
-      include Thor::Shell
 
-      requires_ancestor Thor
+      include Colorize
+
+      requires_ancestor { Thor }
 
       # Print `message` on `$stdout`
       sig { params(message: String).void }
@@ -82,10 +83,24 @@ module Spoom
         Sorbet::Config.parse_file(sorbet_config_file)
       end
 
+      sig { params(exit_code: Integer, block: T.nilable(T.proc.void)).void }
+      def check_sorbet_segfault(exit_code, &block)
+        return unless exit_code == Spoom::Sorbet::SEGFAULT_CODE
+
+        say_error(<<~ERR, status: nil)
+          #{red("!!! Sorbet exited with code #{exit_code} - SEGFAULT !!!")}
+
+          This is most likely related to a bug in Sorbet.
+        ERR
+
+        block&.call
+        exit(exit_code)
+      end
+
       # Colors
 
       # Color used to highlight expressions in backticks
-      HIGHLIGHT_COLOR = :blue
+      HIGHLIGHT_COLOR = T.let(Spoom::Color::BLUE, Spoom::Color)
 
       # Is the `--color` option true?
       sig { returns(T::Boolean) }
@@ -117,35 +132,40 @@ module Spoom
       end
 
       # Colorize a string if `color?`
-      sig { params(string: String, color: Symbol).returns(String) }
-      def colorize(string, color)
+      sig { params(string: String, color: Color).returns(String) }
+      def colorize(string, *color)
         return string unless color?
-        string.colorize(color)
+        T.unsafe(self).set_color(string, *color)
       end
 
       sig { params(string: String).returns(String) }
       def blue(string)
-        colorize(string, :blue)
+        colorize(string, Color::BLUE)
+      end
+
+      sig { params(string: String).returns(String) }
+      def cyan(string)
+        colorize(string, Color::CYAN)
       end
 
       sig { params(string: String).returns(String) }
       def gray(string)
-        colorize(string, :light_black)
+        colorize(string, Color::LIGHT_BLACK)
       end
 
       sig { params(string: String).returns(String) }
       def green(string)
-        colorize(string, :green)
+        colorize(string, Color::GREEN)
       end
 
       sig { params(string: String).returns(String) }
       def red(string)
-        colorize(string, :red)
+        colorize(string, Color::RED)
       end
 
       sig { params(string: String).returns(String) }
       def yellow(string)
-        colorize(string, :yellow)
+        colorize(string, Color::YELLOW)
       end
     end
   end

data/lib/spoom/cli/run.rb

@@ -35,12 +35,27 @@ module Spoom
         sorbet = options[:sorbet]
 
         unless limit || code || sort
-          output, status = T.unsafe(Spoom::Sorbet).srb_tc(*arg, path: path, capture_err: false, sorbet_bin: sorbet)
+          output, status, exit_code = T.unsafe(Spoom::Sorbet).srb_tc(
+            *arg,
+            path: path,
+            capture_err: false,
+            sorbet_bin: sorbet
+          )
+
+          check_sorbet_segfault(exit_code)
           say_error(output, status: nil, nl: false)
           exit(status)
         end
 
-        output, status = T.unsafe(Spoom::Sorbet).srb_tc(*arg, path: path, capture_err: true, sorbet_bin: sorbet)
+        output, status, exit_code = T.unsafe(Spoom::Sorbet).srb_tc(
+          *arg,
+          path: path,
+          capture_err: true,
+          sorbet_bin: sorbet
+        )
+
+        check_sorbet_segfault(exit_code)
+
         if status
           say_error(output, status: nil, nl: false)
           exit(0)
@@ -99,7 +114,7 @@ module Spoom
               cyan = !cyan
               next
             end
-            word << (cyan ? c.cyan : c.red)
+            word << (cyan ? cyan(c) : red(c))
           end
           word.string
         end

data/lib/spoom/colors.rb

@@ -0,0 +1,45 @@
+# typed: strict
+# frozen_string_literal: true
+
+module Spoom
+  class Color < T::Enum
+    extend T::Sig
+
+    enums do
+      CLEAR           = new("\e[0m")
+      BOLD            = new("\e[1m")
+
+      BLACK           = new("\e[30m")
+      RED             = new("\e[31m")
+      GREEN           = new("\e[32m")
+      YELLOW          = new("\e[33m")
+      BLUE            = new("\e[34m")
+      MAGENTA         = new("\e[35m")
+      CYAN            = new("\e[36m")
+      WHITE           = new("\e[37m")
+
+      LIGHT_BLACK     = new("\e[90m")
+      LIGHT_RED       = new("\e[91m")
+      LIGHT_GREEN     = new("\e[92m")
+      LIGHT_YELLOW    = new("\e[93m")
+      LIGHT_BLUE      = new("\e[94m")
+      LIGHT_MAGENTA   = new("\e[95m")
+      LIGHT_CYAN      = new("\e[96m")
+      LIGHT_WHITE     = new("\e[97m")
+    end
+
+    sig { returns(String) }
+    def ansi_code
+      serialize
+    end
+  end
+
+  module Colorize
+    extend T::Sig
+
+    sig { params(string: String, color: Color).returns(String) }
+    def set_color(string, *color)
+      "#{color.map(&:ansi_code).join}#{string}#{Color::CLEAR.ansi_code}"
+    end
+  end
+end

data/lib/spoom/file_tree.rb

@@ -156,7 +156,7 @@ module Spoom
           end
           print("\n")
         else
-          print_colored(node.name, :blue)
+          print_colored(node.name, Color::BLUE)
           print("/")
           printn
           indent
@@ -180,15 +180,15 @@ module Spoom
         Spoom::Sorbet::Sigils.file_strictness(path)
       end
 
-      sig { params(strictness: T.nilable(String)).returns(Symbol) }
+      sig { params(strictness: T.nilable(String)).returns(Color) }
       def strictness_color(strictness)
         case strictness
         when "false"
-          :red
+          Color::RED
         when "true", "strict", "strong"
-          :green
+          Color::GREEN
         else
-          :uncolored
+          Color::CLEAR
         end
       end
     end

data/lib/spoom/git.rb

@@ -20,7 +20,7 @@ module Spoom
       err = e.read.to_s
       e.close
       i.close
-      [out, err, T.cast(s.value, Process::Status).success?]
+      [out, err, s.value.success?]
     end
 
     # Git commands

data/lib/spoom/printer.rb

@@ -8,6 +8,8 @@ module Spoom
     extend T::Sig
     extend T::Helpers
 
+    include Colorize
+
     abstract!
 
     sig { returns(T.any(IO, StringIO)) }
@@ -42,11 +44,10 @@ module Spoom
     # Print `string` colored with `color` into `out`
     #
     # Does not use colors unless `@colors`.
-    sig { params(string: T.nilable(String), color: Symbol, colors: Symbol).void }
-    def print_colored(string, color, *colors)
+    sig { params(string: T.nilable(String), color: Color).void }
+    def print_colored(string, *color)
       return unless string
-      string = colorize(string, color)
-      colors.each { |c| string = colorize(string, c) }
+      string = T.unsafe(self).colorize(string, *color)
       @out.print(string)
     end
 
@@ -72,9 +73,10 @@ module Spoom
     end
 
     # Colorize `string` with color if `@colors`
-    sig { params(string: String, color: Symbol).returns(String) }
-    def colorize(string, color)
-      @colors ? string.colorize(color) : string
+    sig { params(string: String, color: Spoom::Color).returns(String) }
+    def colorize(string, *color)
+      return string unless @colors
+      T.unsafe(self).set_color(string, *color)
     end
   end
 end

data/lib/spoom/sorbet/errors.rb

@@ -6,6 +6,8 @@ module Spoom
     module Errors
       extend T::Sig
 
+      DEFAULT_ERROR_URL_BASE = "https://srb.help/"
+
       # Parse errors from Sorbet output
       class Parser
         extend T::Sig
@@ -18,28 +20,16 @@ module Spoom
           "or set SORBET_SILENCE_DEV_MESSAGE=1 in your shell environment.",
         ]
 
-        ERROR_LINE_MATCH_REGEX = %r{
-          ^         # match beginning of line
-          (\S[^:]*) # capture filename as something that starts with a non-space character
-                    # followed by anything that is not a colon character
-          :         # match the filename - line number seperator
-          (\d+)     # capture the line number
-          :\s       # match the line number - error message separator
-          (.*)      # capture the error message
-          \shttps://srb.help/ # match the error code url prefix
-          (\d+)     # capture the error code
-          $         # match end of line
-        }x.freeze
-
-        sig { params(output: String).returns(T::Array[Error]) }
-        def self.parse_string(output)
-          parser = Spoom::Sorbet::Errors::Parser.new
+        sig { params(output: String, error_url_base: String).returns(T::Array[Error]) }
+        def self.parse_string(output, error_url_base: DEFAULT_ERROR_URL_BASE)
+          parser = Spoom::Sorbet::Errors::Parser.new(error_url_base: error_url_base)
           parser.parse(output)
         end
 
-        sig { void }
-        def initialize
+        sig { params(error_url_base: String).void }
+        def initialize(error_url_base: DEFAULT_ERROR_URL_BASE)
           @errors = []
+          @error_line_match_regex = error_line_match_regexp(error_url_base)
           @current_error = nil
         end
 
@@ -66,9 +56,26 @@ module Spoom
 
         private
 
+        sig { params(error_url_base: String).returns(Regexp) }
+        def error_line_match_regexp(error_url_base)
+          url = Regexp.escape(error_url_base)
+          %r{
+            ^         # match beginning of line
+            (\S[^:]*) # capture filename as something that starts with a non-space character
+                      # followed by anything that is not a colon character
+            :         # match the filename - line number seperator
+            (\d+)     # capture the line number
+            :\s       # match the line number - error message separator
+            (.*)      # capture the error message
+            \s#{url}  # match the error code url prefix
+            (\d+)     # capture the error code
+            $         # match end of line
+          }x
+        end
+
         sig { params(line: String).returns(T.nilable(Error)) }
         def match_error_line(line)
-          match = line.match(ERROR_LINE_MATCH_REGEX)
+          match = line.match(@error_line_match_regex)
           return unless match
 
           file, line, message, code = match.captures

data/lib/spoom/sorbet/lsp/structures.rb

@@ -2,6 +2,7 @@
 # frozen_string_literal: true
 
 require_relative "../../printer"
+require "set"
 
 module Spoom
   module LSP
@@ -56,7 +57,7 @@ module Spoom
 
       sig { override.params(printer: SymbolPrinter).void }
       def accept_printer(printer)
-        printer.print_colored("#{line}:#{char}", :light_black)
+        printer.print_colored("#{line}:#{char}", Color::LIGHT_BLACK)
       end
 
       def to_s
@@ -81,7 +82,7 @@ module Spoom
       sig { override.params(printer: SymbolPrinter).void }
       def accept_printer(printer)
         printer.print_object(start)
-        printer.print_colored("-", :light_black)
+        printer.print_colored("-", Color::LIGHT_BLACK)
         printer.print_object(self.end)
       end
 
@@ -106,7 +107,7 @@ module Spoom
 
       sig { override.params(printer: SymbolPrinter).void }
       def accept_printer(printer)
-        printer.print_colored("#{printer.clean_uri(uri)}:", :light_black)
+        printer.print_colored("#{printer.clean_uri(uri)}:", Color::LIGHT_BLACK)
         printer.print_object(range)
       end
 
@@ -203,14 +204,14 @@ module Spoom
         printer.printt
         printer.print(kind_string)
         printer.print(' ')
-        printer.print_colored(name, :blue, :bold)
-        printer.print_colored(' (', :light_black)
+        printer.print_colored(name, Color::BLUE, Color::BOLD)
+        printer.print_colored(' (', Color::LIGHT_BLACK)
         if range
           printer.print_object(range)
         elsif location
           printer.print_object(location)
         end
-        printer.print_colored(')', :light_black)
+        printer.print_colored(')', Color::LIGHT_BLACK)
         printer.printn
         unless children.empty?
           printer.indent

data/lib/spoom/sorbet.rb

@@ -15,6 +15,8 @@ module Spoom
     GEM_PATH = Gem::Specification.find_by_name("sorbet-static").full_gem_path
     BIN_PATH = (Pathname.new(GEM_PATH) / "libexec" / "sorbet").to_s
 
+    SEGFAULT_CODE = 139
+
     class << self
       extend T::Sig
 
@@ -24,7 +26,7 @@ module Spoom
           path: String,
           capture_err: T::Boolean,
           sorbet_bin: T.nilable(String)
-        ).returns([String, T::Boolean])
+        ).returns([String, T::Boolean, Integer])
       end
       def srb(*arg, path: '.', capture_err: false, sorbet_bin: nil)
         if sorbet_bin
@@ -41,7 +43,7 @@ module Spoom
           path: String,
           capture_err: T::Boolean,
           sorbet_bin: T.nilable(String)
-        ).returns([String, T::Boolean])
+        ).returns([String, T::Boolean, Integer])
       end
       def srb_tc(*arg, path: '.', capture_err: false, sorbet_bin: nil)
         arg.prepend("tc") unless sorbet_bin

data/lib/spoom/test_helpers/project.rb

@@ -82,7 +82,7 @@ module Spoom
         opts = {}
         opts[:chdir] = path
         out, err, status = Open3.capture3("bundle", "install", opts)
-        [out, err, status.success?]
+        [out, err, T.must(status.success?)]
       end
 
       # Run a command with `bundle exec` in this project
@@ -91,7 +91,7 @@ module Spoom
         opts = {}
         opts[:chdir] = path
         out, err, status = Open3.capture3(["bundle", "exec", cmd, *args].join(' '), opts)
-        [out, err, status.success?]
+        [out, err, T.must(status.success?)]
       end
 
       # Delete this project and its content

data/lib/spoom/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module Spoom
-  VERSION = "1.1.3"
+  VERSION = "1.1.7"
 end

data/lib/spoom.rb

@@ -1,8 +1,8 @@
 # typed: true
 # frozen_string_literal: true
 
-require "colorize"
 require "sorbet-runtime"
+require "pathname"
 
 module Spoom
   extend T::Sig
@@ -17,16 +17,19 @@ module Spoom
       arg: String,
       path: String,
       capture_err: T::Boolean
-    ).returns([String, T::Boolean])
+    ).returns([String, T::Boolean, Integer])
   end
   def self.exec(cmd, *arg, path: '.', capture_err: false)
     method = capture_err ? "popen2e" : "popen2"
-    Open3.send(method, [cmd, *arg].join(" "), chdir: path) do |_, o, t|
-      [o.read, T.cast(t.value, Process::Status).success?]
+    Open3.send(method, [cmd, *arg].join(" "), chdir: path) do |_, stdout, thread|
+      out = stdout.read
+      status = T.cast(thread.value, Process::Status)
+      [out, status.success?, status.exitstatus]
     end
   end
 end
 
+require "spoom/colors"
 require "spoom/sorbet"
 require "spoom/cli"
 require "spoom/version"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spoom
 version: !ruby/object:Gem::Version
-  version: 1.1.3
+  version: 1.1.7
 platform: ruby
 authors:
 - Alexandre Terrasa
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-09-28 00:00:00.000000000 Z
+date: 2021-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -53,13 +53,13 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '5.0'
 - !ruby/object:Gem::Dependency
-  name: sorbet-runtime
+  name: minitest-reporters
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -67,47 +67,47 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: sorbet
+  name: sorbet-runtime
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.5.6347
+        version: 0.5.9204
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.5.6347
+        version: 0.5.9204
 - !ruby/object:Gem::Dependency
-  name: thor
+  name: sorbet
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.19.2
+        version: 0.5.9204
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.19.2
+        version: 0.5.9204
 - !ruby/object:Gem::Dependency
-  name: colorize
+  name: thor
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.19.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.19.2
 description: 
 email:
 - ruby@shopify.com
@@ -128,6 +128,7 @@ files:
 - lib/spoom/cli/helper.rb
 - lib/spoom/cli/lsp.rb
 - lib/spoom/cli/run.rb
+- lib/spoom/colors.rb
 - lib/spoom/coverage.rb
 - lib/spoom/coverage/d3.rb
 - lib/spoom/coverage/d3/base.rb