rubocop-sorbet 0.8.7 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8a44258e5997d8d76d11a35e921aaa0312daaadb5d159a2cf8425213baabdc1e
-  data.tar.gz: de2a3ed4f84838094981247acce55eb873c620fbcea454ed145c174008734cd7
+  metadata.gz: 743baa95aeb8e124bebb3d354867e42a397e874d43ab75e0f8fcc2b7d40e7757
+  data.tar.gz: 705349f2986c114712c99303de229cf95128a66c79dc97de54d274623ab9a785
 SHA512:
-  metadata.gz: af324556ad44f2eda2a26ba9cd006a92af766c8cc46872d761f1db769f9aa0b528d4d828b7d6a5e335a557cbce277f271da5a295f9c7d4a74f7babda35be6106
-  data.tar.gz: a927134808ef57e9ddaa7fb0798d61e2e00011b68c86f429e8c53d12f693072ef69368657963e587165693d84f0d814abdc2b1ff29531aa58fa36fb9ad849ae1
+  metadata.gz: 9fac22901d690391f93b24ceef20727e3b6e79a9de887792d7cdb50cd3237caa38ac5655162aec3c748d01e747dd838ed6ca9efa4c5b727ff5d218fe495867f6
+  data.tar.gz: c988d90e224c3cfd3c93d13190e8d100165d35d3e9968c15930319ca94f5c6d787484bbb9bff3a92cb7b84deb4643b86720bc841751f823560865cf429121c4c

data/.github/workflows/ci.yml

@@ -11,7 +11,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: ["3.1", "3.2", "3.3"]
+        ruby: ["3.1", "3.2", "3.3", "3.4"]
     name: Test Ruby ${{ matrix.ruby }}
     steps:
       - uses: actions/checkout@v4
@@ -42,5 +42,7 @@ jobs:
           bin/rubocop --config config/default.yml config -r rubocop-sorbet
       - name: Lint Ruby files
         run: bin/rubocop
+      - name: Verify cop examples
+        run: bundle exec rake documentation_syntax_check
       - name: Verify documentation is up to date
         run: bundle exec rake generate_cops_documentation

data/.rubocop.yml

@@ -24,3 +24,18 @@ Layout/LineLength:
 
 InternalAffairs/UndefinedConfig:
   Enabled: false # Bug in implementation fails to find our configs
+
+InternalAffairs/OnSendWithoutOnCSend:
+  Exclude:
+    # Cases where we don't expect a conditional send
+    - lib/rubocop/cop/sorbet/callback_conditionals_binding.rb
+    - lib/rubocop/cop/sorbet/forbid_include_const_literal.rb
+    - lib/rubocop/cop/sorbet/forbid_t_struct.rb
+    - lib/rubocop/cop/sorbet/forbid_t_untyped.rb
+    - lib/rubocop/cop/sorbet/implicit_conversion_method.rb
+    - lib/rubocop/cop/sorbet/rbi/forbid_extend_t_sig_helpers_in_shims.rb
+    - lib/rubocop/cop/sorbet/redundant_extend_t_sig.rb
+    - lib/rubocop/cop/sorbet/refinement.rb
+    - lib/rubocop/cop/sorbet/signatures/allow_incompatible_override.rb
+    - lib/rubocop/cop/sorbet/signatures/enforce_signatures.rb
+    - lib/rubocop/cop/sorbet/t_enum/forbid_comparable_t_enum.rb

data/.ruby-version

@@ -1 +1 @@
-3.3.3
+3.4.1

data/Gemfile.lock

@@ -1,36 +1,40 @@
 PATH
   remote: .
   specs:
-    rubocop-sorbet (0.8.7)
+    rubocop-sorbet (0.9.0)
+      lint_roller (~> 1.1)
       rubocop (>= 1)
 
 GEM
   remote: https://rubygems.org/
   specs:
     ast (2.4.2)
-    debug (1.9.2)
+    date (3.4.1)
+    debug (1.10.0)
       irb (~> 1.10)
       reline (>= 0.3.8)
     diff-lcs (1.5.1)
-    io-console (0.7.2)
-    irb (1.12.0)
-      rdoc
+    io-console (0.8.0)
+    irb (1.14.3)
+      rdoc (>= 4.0.0)
       reline (>= 0.4.2)
-    json (2.7.2)
-    language_server-protocol (3.17.0.3)
+    json (2.10.1)
+    language_server-protocol (3.17.0.4)
+    lint_roller (1.1.0)
     parallel (1.26.3)
-    parser (3.3.5.0)
+    parser (3.3.7.1)
       ast (~> 2.4.1)
       racc
-    psych (5.1.2)
+    psych (5.2.2)
+      date
       stringio
     racc (1.8.1)
     rainbow (3.1.1)
     rake (13.2.1)
-    rdoc (6.6.3.1)
+    rdoc (6.10.0)
       psych (>= 4.0.0)
-    regexp_parser (2.9.2)
-    reline (0.5.1)
+    regexp_parser (2.10.0)
+    reline (0.6.0)
       io-console (~> 0.5)
     rspec (3.13.0)
       rspec-core (~> 3.13.0)
@@ -45,23 +49,26 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
     rspec-support (3.13.1)
-    rubocop (1.67.0)
+    rubocop (1.73.2)
       json (~> 2.3)
-      language_server-protocol (>= 3.17.0)
+      language_server-protocol (~> 3.17.0.2)
+      lint_roller (~> 1.1.0)
       parallel (~> 1.10)
       parser (>= 3.3.0.2)
       rainbow (>= 2.2.2, < 4.0)
-      regexp_parser (>= 2.4, < 3.0)
-      rubocop-ast (>= 1.32.2, < 2.0)
+      regexp_parser (>= 2.9.3, < 3.0)
+      rubocop-ast (>= 1.38.0, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 2.4.0, < 3.0)
-    rubocop-ast (1.32.3)
+      unicode-display_width (>= 2.4.0, < 4.0)
+    rubocop-ast (1.38.1)
       parser (>= 3.3.1.0)
     rubocop-shopify (2.15.1)
       rubocop (~> 1.51)
     ruby-progressbar (1.13.0)
-    stringio (3.1.0)
-    unicode-display_width (2.6.0)
+    stringio (3.1.2)
+    unicode-display_width (3.1.4)
+      unicode-emoji (~> 4.0, >= 4.0.4)
+    unicode-emoji (4.0.4)
     yard (0.9.37)
 
 PLATFORMS

data/README.md

@@ -24,30 +24,33 @@ You need to tell RuboCop to load the Sorbet extension. There are three ways to d
 Put this into your `.rubocop.yml`:
 
 ```yaml
-require: rubocop-sorbet
+plugins: rubocop-sorbet
 ```
 
 Alternatively, use the following array notation when specifying multiple extensions:
 
 ```yaml
-require:
-  - rubocop-other-extension
+plugins:
   - rubocop-sorbet
+  - rubocop-other-extension
 ```
 
 Now you can run `rubocop` and it will automatically load the RuboCop Sorbet cops together with the standard cops.
 
+> [!NOTE]
+> The plugin system is supported in RuboCop 1.72+. In earlier versions, use `require` instead of `plugins`.
+
 ### Command line
 
 ```sh
-rubocop --require rubocop-sorbet
+rubocop --plugin rubocop-sorbet
 ```
 
 ### Rake task
 
 ```ruby
 RuboCop::RakeTask.new do |task|
-  task.requires << 'rubocop-sorbet'
+  task.plugins << 'rubocop-sorbet'
 end
 ```
 

data/config/default.yml

@@ -1,5 +1,5 @@
 Sorbet/Refinement:
-  Description: >- 
+  Description: >-
                   Checks for the use of Ruby Refinements library. Refinements add
                   complexity and incur a performance penalty that can be significant
                   for large code bases. They are also not supported by Sorbet.
@@ -102,6 +102,21 @@ Sorbet/ForbidTypeAliasedShapes:
   Enabled: false
   VersionAdded: 0.7.6
 
+Sorbet/ForbidSig:
+  Description: 'Forbid usage of sig'
+  Enabled: false
+  VersionAdded: <<next>>
+
+Sorbet/ForbidSigWithRuntime:
+  Description: 'Forbid usage of T::Sig.sig'
+  Enabled: false
+  VersionAdded: <<next>>
+
+Sorbet/ForbidSigWithoutRuntime:
+  Description: 'Forbid usage of T::Sig::WithoutRuntime.sig'
+  Enabled: false
+  VersionAdded: <<next>>
+
 Sorbet/ForbidSuperclassConstLiteral:
   Description: 'Forbid superclasses which are non-literal constants.'
   Enabled: false
@@ -110,6 +125,13 @@ Sorbet/ForbidSuperclassConstLiteral:
   Exclude:
   - db/migrate/*.rb
 
+Sorbet/ForbidTEnum:
+  Description: 'Forbid usage of T::Enum.'
+  Enabled: false
+  VersionAdded: <<next>>
+  VersionChanged: <<next>>
+  Safe: false
+
 Sorbet/ForbidTStruct:
   Description: 'Forbid usage of T::Struct.'
   Enabled: false
@@ -252,6 +274,7 @@ Sorbet/SingleLineRbiClassModuleDefinitions:
 Sorbet/StrictSigil:
   Description: 'All files must be at least at strictness `strict`.'
   Enabled: false
+  Safe: false
   SuggestedStrictness: "strict"
   VersionAdded: 0.3.3
   Include:

data/config/rbi.yml

@@ -103,11 +103,11 @@ Layout/InitialIndentation:
 
 Layout/LeadingCommentSpace:
   Enabled: true
+  AllowRBSInlineAnnotation: true
 
 Layout/LeadingEmptyLines:
   Enabled: true
 
-# TODO: make Tapioca break long lines?
 Layout/LineLength:
   Enabled: false
 
@@ -199,6 +199,9 @@ Lint/Syntax:
 
 ## Sorbet
 
+Sorbet/ForbidSigWithoutRuntime:
+  Enabled: true
+
 Sorbet/EnforceSigilOrder:
   Enabled: true
 

data/lib/rubocop/cop/sorbet/callback_conditionals_binding.rb

@@ -6,6 +6,7 @@ module RuboCop
       # Ensures that callback conditionals are bound to the right type
       # so that they are type checked properly.
       #
+      # @safety
       # Auto-correction is unsafe because other libraries define similar style callbacks as Rails, but don't always need
       # binding to the attached class. Auto-correcting those usages can lead to false positives and auto-correction
       # introduces new typing errors.

data/lib/rubocop/cop/sorbet/constants_from_strings.rb

@@ -41,6 +41,7 @@ module RuboCop
           :constantize,
           :constants,
           :const_get,
+          :safe_constantize,
         ].freeze
 
         def on_send(node)

data/lib/rubocop/cop/sorbet/forbid_include_const_literal.rb

@@ -50,14 +50,14 @@ module RuboCop
         private
 
         def neither_const_nor_self?(node)
-          !node.const_type? && !node.self_type?
+          !node.type?(:const, :self)
         end
 
         # Returns true if the node is within a module declaration that is not anonymous.
         def within_onymous_module?(node)
           parent = node.parent
-          parent = parent.parent while parent&.begin_type? || parent&.block_type?
-          parent && (parent.module_type? || parent.class_type? || parent.sclass_type?)
+          parent = parent.parent while parent&.type?(:begin, :block)
+          parent&.type?(:module, :class, :sclass)
         end
       end
     end

data/lib/rubocop/cop/sorbet/forbid_t_enum.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+require "rubocop"
+
+module RuboCop
+  module Cop
+    module Sorbet
+      # Disallow using `T::Enum`.
+      #
+      # @example
+      #
+      #   # bad
+      #   class MyEnum < T::Enum
+      #     enums do
+      #       A = new
+      #       B = new
+      #     end
+      #   end
+      #
+      #   # good
+      #   class MyEnum
+      #     A = "a"
+      #     B = "b"
+      #     C = "c"
+      #   end
+      class ForbidTEnum < RuboCop::Cop::Base
+        MSG = "Using `T::Enum` is deprecated in this codebase."
+
+        # @!method t_enum?(node)
+        def_node_matcher(:t_enum?, <<~PATTERN)
+          (const (const {nil? cbase} :T) :Enum)
+        PATTERN
+
+        def on_class(node)
+          add_offense(node) if t_enum?(node.parent_class)
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/sorbet/forbid_t_struct.rb

@@ -43,8 +43,8 @@ module RuboCop
 
         RESTRICT_ON_SEND = [:include, :prepend, :extend].freeze
 
-        MSG_STRUCT = "Using `T::Struct` or its variants is deprecated."
-        MSG_PROPS = "Using `T::Props` or its variants is deprecated."
+        MSG_STRUCT = "Using `T::Struct` or its variants is deprecated in this codebase."
+        MSG_PROPS = "Using `T::Props` or its variants is deprecated in this codebase."
 
         # This class walks down the class body of a T::Struct and collects all the properties that will need to be
         # translated into `attr_reader` and `attr_accessor` methods.

data/lib/rubocop/cop/sorbet/forbid_t_unsafe.rb

@@ -24,6 +24,7 @@ module RuboCop
         def on_send(node)
           add_offense(node) if t_unsafe?(node)
         end
+        alias_method :on_csend, :on_send
       end
     end
   end

data/lib/rubocop/cop/sorbet/mixin/signature_help.rb

@@ -9,21 +9,34 @@ module RuboCop
 
         # @!method signature?(node)
         def_node_matcher(:signature?, <<~PATTERN)
+          {#bare_sig? #sig_with_runtime? #sig_without_runtime?}
+        PATTERN
+
+        # @!method bare_sig?(node)
+        def_node_matcher(:bare_sig?, <<~PATTERN)
           (block (send
-            {nil? #with_runtime? #without_runtime?}
+            nil?
             :sig
             (sym :final)?
           ) (args) ...)
         PATTERN
 
-        # @!method with_runtime?(node)
-        def_node_matcher(:with_runtime?, <<~PATTERN)
-          (const (const {nil? cbase} :T) :Sig)
+        # @!method sig_with_runtime?(node)
+        def_node_matcher(:sig_with_runtime?, <<~PATTERN)
+          (block (send
+            (const (const {nil? cbase} :T) :Sig)
+            :sig
+            (sym :final)?
+          ) (args) ...)
         PATTERN
 
-        # @!method without_runtime?(node)
-        def_node_matcher(:without_runtime?, <<~PATTERN)
-          (const (const (const {nil? cbase} :T) :Sig) :WithoutRuntime)
+        # @!method sig_without_runtime?(node)
+        def_node_matcher(:sig_without_runtime?, <<~PATTERN)
+          (block (send
+            (const (const (const {nil? cbase} :T) :Sig) :WithoutRuntime)
+            :sig
+            (sym :final)?
+          ) (args) ...)
         PATTERN
 
         def on_block(node)

data/lib/rubocop/cop/sorbet/sigils/has_sigil.rb

@@ -13,6 +13,7 @@ module RuboCop
       # * `SuggestedStrictness`: Sorbet strictness level suggested in offense messages (default: 'false')
       # * `MinimumStrictness`: If set, make offense if the strictness level in the file is below this one
       #
+      # If a `SuggestedStrictness` level is specified, it will be used in autocorrect.
       # If a `MinimumStrictness` level is specified, it will be used in offense messages and autocorrect.
       class HasSigil < ValidSigil
         def require_sigil_on_all_files?

data/lib/rubocop/cop/sorbet/sigils/strict_sigil.rb

@@ -7,6 +7,21 @@ module RuboCop
   module Cop
     module Sorbet
       # Makes the Sorbet `strict` sigil mandatory in all files.
+      #
+      # @safety
+      #   This cop is unsafe because Sorbet sigils may not exist yet when it is run.
+      #
+      # @example
+      #
+      #   # bad
+      #   # typed: true
+      #
+      #   # bad
+      #   # typed: false
+      #
+      #   # good
+      #   # typed: strict
+      #
       class StrictSigil < HasSigil
         def minimum_strictness
           "strict"

data/lib/rubocop/cop/sorbet/sigils/valid_sigil.rb

@@ -16,6 +16,7 @@ module RuboCop
       # * `ExactStrictness`: If set, make offense if the strictness level in the file is different than this one
       #
       # If an `ExactStrictness` level is specified, it will be used in offense messages and autocorrect.
+      # If a `SuggestedStrictness` level is specified, it will be used in autocorrect.
       # Otherwise, if a `MinimumStrictness` level is specified, it will be used in offense messages and autocorrect.
       class ValidSigil < RuboCop::Cop::Base
         extend AutoCorrector

data/lib/rubocop/cop/sorbet/signatures/enforce_signatures.rb

@@ -28,6 +28,8 @@ module RuboCop
         extend AutoCorrector
         include SignatureHelp
 
+        RBS_COMMENT_REGEX = /^#:.*$/
+
         def initialize(config = nil, options = nil)
           super(config, options)
           @last_sig_for_scope = {}
@@ -65,7 +67,7 @@ module RuboCop
 
         def check_node(node)
           scope = self.scope(node)
-          unless @last_sig_for_scope[scope]
+          unless @last_sig_for_scope[scope] || has_rbs_comment?(node)
             add_offense(
               node,
               message: "Each method is required to have a signature.",
@@ -76,6 +78,22 @@ module RuboCop
           @last_sig_for_scope[scope] = nil
         end
 
+        def has_rbs_comment?(node)
+          return false unless cop_config["AllowRBS"] == true
+
+          node = node.parent while RuboCop::AST::SendNode === node.parent
+          return false if (comments = preceeding_comments(node)).empty?
+
+          last_comment = comments.last
+          return false if last_comment.loc.line + 1 < node.loc.line
+
+          comments.any? { |comment| RBS_COMMENT_REGEX.match?(comment.text) }
+        end
+
+        def preceeding_comments(node)
+          processed_source.ast_with_comments[node].select { |comment| comment.loc.line < node.loc.line }
+        end
+
         def autocorrect(corrector, node)
           suggest = SigSuggestion.new(node.loc.column, param_type_placeholder, return_type_placeholder)
 

data/lib/rubocop/cop/sorbet/signatures/forbid_sig.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+require "stringio"
+
+module RuboCop
+  module Cop
+    module Sorbet
+      # Check that definitions do not use a `sig` block.
+      #
+      # Good:
+      #
+      # ```
+      # #: -> void
+      # def foo; end
+      # ```
+      #
+      # Bad:
+      #
+      # ```
+      # sig { void }
+      # def foo; end
+      # ```
+      class ForbidSig < ::RuboCop::Cop::Base
+        include SignatureHelp
+
+        MSG = "Do not use `T::Sig`."
+
+        def on_signature(node)
+          add_offense(node) if bare_sig?(node)
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/sorbet/signatures/forbid_sig_with_runtime.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+require "stringio"
+
+module RuboCop
+  module Cop
+    module Sorbet
+      # Check that definitions do not use a `sig` block.
+      #
+      # Good:
+      #
+      # ```
+      # #: -> void
+      # def foo; end
+      # ```
+      #
+      # Bad:
+      #
+      # ```
+      # T::Sig.sig { void }
+      # def foo; end
+      # ```
+      class ForbidSigWithRuntime < ::RuboCop::Cop::Base
+        include SignatureHelp
+
+        MSG = "Do not use `T::Sig.sig`."
+
+        def on_signature(node)
+          add_offense(node) if sig_with_runtime?(node)
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/sorbet/signatures/forbid_sig_without_runtime.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+require "stringio"
+
+module RuboCop
+  module Cop
+    module Sorbet
+      # Check that `sig` is used instead of `T::Sig::WithoutRuntime.sig`.
+      #
+      # Good:
+      #
+      # ```
+      # sig { void }
+      # def foo; end
+      # ```
+      #
+      # Bad:
+      #
+      # ```
+      # T::Sig::WithoutRuntime.sig { void }
+      # def foo; end
+      # ```
+      class ForbidSigWithoutRuntime < ::RuboCop::Cop::Base
+        include SignatureHelp
+        extend AutoCorrector
+
+        MSG = "Do not use `T::Sig::WithoutRuntime.sig`."
+
+        def on_signature(node)
+          return unless sig_without_runtime?(node)
+
+          sig = node.children[0]
+          add_offense(sig) do |corrector|
+            corrector.replace(sig, sig.source.gsub(/T\s*::\s*Sig\s*::\s*WithoutRuntime\s*\.\s*/m, ""))
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/sorbet/signatures/keyword_argument_ordering.rb

@@ -35,7 +35,7 @@ module RuboCop
           out_of_kwoptarg = false
 
           parameters.reverse.each do |param|
-            out_of_kwoptarg = true unless param.kwoptarg_type? || param.blockarg_type? || param.kwrestarg_type?
+            out_of_kwoptarg = true unless param.type?(:kwoptarg, :blockarg, :kwrestarg)
 
             next unless param.kwoptarg_type? && out_of_kwoptarg
 

data/lib/rubocop/cop/sorbet/signatures/void_checked_tests.rb

@@ -35,7 +35,7 @@ module RuboCop
 
         # @!method checked_tests(node)
         def_node_search(:checked_tests, <<~PATTERN)
-          ({csend send} _ :checked (sym :tests))
+          (call _ :checked (sym :tests))
         PATTERN
 
         MESSAGE =

data/lib/rubocop/cop/sorbet_cops.rb

@@ -12,6 +12,7 @@ require_relative "sorbet/forbid_type_aliased_shapes"
 require_relative "sorbet/forbid_untyped_struct_props"
 require_relative "sorbet/implicit_conversion_method"
 require_relative "sorbet/callback_conditionals_binding"
+require_relative "sorbet/forbid_t_enum"
 require_relative "sorbet/forbid_t_struct"
 require_relative "sorbet/forbid_t_unsafe"
 require_relative "sorbet/forbid_t_untyped"
@@ -30,11 +31,14 @@ require_relative "sorbet/rbi_versioning/valid_gem_version_annotations"
 
 require_relative "sorbet/signatures/allow_incompatible_override"
 require_relative "sorbet/signatures/checked_true_in_signature"
-require_relative "sorbet/signatures/void_checked_tests"
+require_relative "sorbet/signatures/empty_line_after_sig"
+require_relative "sorbet/signatures/enforce_signatures"
+require_relative "sorbet/signatures/forbid_sig"
+require_relative "sorbet/signatures/forbid_sig_with_runtime"
+require_relative "sorbet/signatures/forbid_sig_without_runtime"
 require_relative "sorbet/signatures/keyword_argument_ordering"
 require_relative "sorbet/signatures/signature_build_order"
-require_relative "sorbet/signatures/enforce_signatures"
-require_relative "sorbet/signatures/empty_line_after_sig"
+require_relative "sorbet/signatures/void_checked_tests"
 
 require_relative "sorbet/sigils/valid_sigil"
 require_relative "sorbet/sigils/has_sigil"

data/lib/rubocop/sorbet/plugin.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+require "rubocop"
+require "lint_roller"
+require "pathname"
+
+module RuboCop
+  module Sorbet
+    # A plugin that integrates RuboCop Sorbet with RuboCop's plugin system.
+    class Plugin < LintRoller::Plugin
+      RUBOCOP_MIN_VERSION = "1.72.1"
+      SUPPORTED = Gem::Version.create(RuboCop::Version.version) < RUBOCOP_MIN_VERSION
+
+      def about
+        LintRoller::About.new(
+          name: "rubocop-sorbet",
+          version: VERSION,
+          homepage: "https://github.com/Shopify/rubocop-sorbet",
+          description: "A collection of Rubocop rules for Sorbet.",
+        )
+      end
+
+      def supported?(context)
+        context.engine == :rubocop
+      end
+
+      def rules(_context)
+        project_root = Pathname.new(__dir__).join("../../..")
+
+        ConfigObsoletion.files << project_root.join("config", "obsoletion.yml")
+
+        LintRoller::Rules.new(type: :path, config_format: :rubocop, value: project_root.join("config", "default.yml"))
+      end
+    end
+  end
+end

data/lib/rubocop/sorbet/version.rb

@@ -2,6 +2,6 @@
 
 module RuboCop
   module Sorbet
-    VERSION = "0.8.7"
+    VERSION = "0.9.0"
   end
 end

data/lib/rubocop/sorbet.rb

@@ -1,18 +1,23 @@
 # frozen_string_literal: true
 
+require "rubocop"
 require "rubocop/sorbet/version"
+require "rubocop/sorbet/plugin"
+require "pathname"
 require "yaml"
 
 module RuboCop
   module Sorbet
     class Error < StandardError; end
 
-    PROJECT_ROOT   = Pathname.new(__dir__).parent.parent.expand_path.freeze
-    CONFIG_DEFAULT = PROJECT_ROOT.join("config", "default.yml").freeze
-    CONFIG         = YAML.safe_load(CONFIG_DEFAULT.read).freeze
+    unless Plugin::SUPPORTED
+      PROJECT_ROOT   = Pathname.new(__dir__).parent.parent.expand_path.freeze
+      CONFIG_DEFAULT = PROJECT_ROOT.join("config", "default.yml").freeze
+      CONFIG         = YAML.safe_load(CONFIG_DEFAULT.read).freeze
 
-    private_constant(:CONFIG_DEFAULT, :PROJECT_ROOT)
+      private_constant(:CONFIG_DEFAULT, :PROJECT_ROOT)
 
-    ::RuboCop::ConfigObsoletion.files << PROJECT_ROOT.join("config", "obsoletion.yml")
+      ::RuboCop::ConfigObsoletion.files << PROJECT_ROOT.join("config", "obsoletion.yml")
+    end
   end
 end

data/lib/rubocop-sorbet.rb

@@ -4,8 +4,11 @@ require "rubocop"
 
 require_relative "rubocop/sorbet"
 require_relative "rubocop/sorbet/version"
-require_relative "rubocop/sorbet/inject"
+require_relative "rubocop/sorbet/plugin"
 
-RuboCop::Sorbet::Inject.defaults!
+unless RuboCop::Sorbet::Plugin::SUPPORTED
+  require_relative "rubocop/sorbet/inject"
+  RuboCop::Sorbet::Inject.defaults!
+end
 
 require_relative "rubocop/cop/sorbet_cops"

data/manual/cops.md

@@ -20,7 +20,11 @@ In the following section you find all available cops:
 * [Sorbet/ForbidExtendTSigHelpersInShims](cops_sorbet.md#sorbetforbidextendtsighelpersinshims)
 * [Sorbet/ForbidIncludeConstLiteral](cops_sorbet.md#sorbetforbidincludeconstliteral)
 * [Sorbet/ForbidRBIOutsideOfAllowedPaths](cops_sorbet.md#sorbetforbidrbioutsideofallowedpaths)
+* [Sorbet/ForbidSig](cops_sorbet.md#sorbetforbidsig)
+* [Sorbet/ForbidSigWithRuntime](cops_sorbet.md#sorbetforbidsigwithruntime)
+* [Sorbet/ForbidSigWithoutRuntime](cops_sorbet.md#sorbetforbidsigwithoutruntime)
 * [Sorbet/ForbidSuperclassConstLiteral](cops_sorbet.md#sorbetforbidsuperclassconstliteral)
+* [Sorbet/ForbidTEnum](cops_sorbet.md#sorbetforbidtenum)
 * [Sorbet/ForbidTStruct](cops_sorbet.md#sorbetforbidtstruct)
 * [Sorbet/ForbidTUnsafe](cops_sorbet.md#sorbetforbidtunsafe)
 * [Sorbet/ForbidTUntyped](cops_sorbet.md#sorbetforbidtuntyped)

data/manual/cops_sorbet.md

@@ -417,6 +417,72 @@ Name | Default value | Configurable values
 AllowedPaths | `rbi/**`, `sorbet/rbi/**` | Array
 Include | `**/*.rbi` | Array
 
+## Sorbet/ForbidSig
+
+Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
+--- | --- | --- | --- | ---
+Disabled | Yes | No | <<next>> | -
+
+Check that definitions do not use a `sig` block.
+
+Good:
+
+```
+#: -> void
+def foo; end
+```
+
+Bad:
+
+```
+sig { void }
+def foo; end
+```
+
+## Sorbet/ForbidSigWithRuntime
+
+Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
+--- | --- | --- | --- | ---
+Disabled | Yes | No | <<next>> | -
+
+Check that definitions do not use a `sig` block.
+
+Good:
+
+```
+#: -> void
+def foo; end
+```
+
+Bad:
+
+```
+T::Sig.sig { void }
+def foo; end
+```
+
+## Sorbet/ForbidSigWithoutRuntime
+
+Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
+--- | --- | --- | --- | ---
+Disabled | Yes | Yes  | <<next>> | -
+
+Check that `sig` is used instead of `T::Sig::WithoutRuntime.sig`.
+
+Good:
+
+```
+sig { void }
+def foo; end
+```
+
+Bad:
+
+```
+T::Sig::WithoutRuntime.sig { void }
+def foo; end
+```
+
 ## Sorbet/ForbidSuperclassConstLiteral
 
 Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
@@ -448,6 +514,33 @@ Name | Default value | Configurable values
 --- | --- | ---
 Exclude | `db/migrate/*.rb` | Array
 
+## Sorbet/ForbidTEnum
+
+Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
+--- | --- | --- | --- | ---
+Disabled | No | No | <<next>> | <<next>>
+
+Disallow using `T::Enum`.
+
+### Examples
+
+```ruby
+# bad
+class MyEnum < T::Enum
+  enums do
+    A = new
+    B = new
+  end
+end
+
+# good
+class MyEnum
+  A = "a"
+  B = "b"
+  C = "c"
+end
+```
+
 ## Sorbet/ForbidTStruct
 
 Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
@@ -588,6 +681,7 @@ Options:
 * `SuggestedStrictness`: Sorbet strictness level suggested in offense messages (default: 'false')
 * `MinimumStrictness`: If set, make offense if the strictness level in the file is below this one
 
+If a `SuggestedStrictness` level is specified, it will be used in autocorrect.
 If a `MinimumStrictness` level is specified, it will be used in offense messages and autocorrect.
 
 ### Configurable attributes
@@ -860,10 +954,23 @@ Include | `**/*.rbi` | Array
 
 Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
 --- | --- | --- | --- | ---
-Disabled | Yes | Yes  | 0.3.3 | -
+Disabled | No | Yes  | 0.3.3 | -
 
 Makes the Sorbet `strict` sigil mandatory in all files.
 
+### Examples
+
+```ruby
+# bad
+# typed: true
+
+# bad
+# typed: false
+
+# good
+# typed: strict
+```
+
 ### Configurable attributes
 
 Name | Default value | Configurable values
@@ -970,6 +1077,7 @@ Options:
 * `ExactStrictness`: If set, make offense if the strictness level in the file is different than this one
 
 If an `ExactStrictness` level is specified, it will be used in offense messages and autocorrect.
+If a `SuggestedStrictness` level is specified, it will be used in autocorrect.
 Otherwise, if a `MinimumStrictness` level is specified, it will be used in offense messages and autocorrect.
 
 ### Configurable attributes

data/rubocop-sorbet.gemspec

@@ -17,6 +17,7 @@ Gem::Specification.new do |spec|
   spec.metadata["allowed_push_host"] = "https://rubygems.org"
   spec.metadata["homepage_uri"] = spec.homepage
   spec.metadata["source_code_uri"] = "https://github.com/shopify/rubocop-sorbet"
+  spec.metadata["default_lint_roller_plugin"] = "RuboCop::Sorbet::Plugin"
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
@@ -27,5 +28,6 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
+  spec.add_runtime_dependency("lint_roller", "~> 1.1")
   spec.add_runtime_dependency("rubocop", ">= 1")
 end

metadata

@@ -1,18 +1,31 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-sorbet
 version: !ruby/object:Gem::Version
-  version: 0.8.7
+  version: 0.9.0
 platform: ruby
 authors:
 - Ufuk Kayserilioglu
 - Alan Wu
 - Alexandre Terrasa
 - Peter Zhu
-autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-11-01 00:00:00.000000000 Z
+date: 2025-03-10 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: lint_roller
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
@@ -27,7 +40,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '1'
-description:
 email:
 - ruby@shopify.com
 executables: []
@@ -68,6 +80,7 @@ files:
 - lib/rubocop/cop/sorbet/constants_from_strings.rb
 - lib/rubocop/cop/sorbet/forbid_include_const_literal.rb
 - lib/rubocop/cop/sorbet/forbid_superclass_const_literal.rb
+- lib/rubocop/cop/sorbet/forbid_t_enum.rb
 - lib/rubocop/cop/sorbet/forbid_t_struct.rb
 - lib/rubocop/cop/sorbet/forbid_t_unsafe.rb
 - lib/rubocop/cop/sorbet/forbid_t_untyped.rb
@@ -99,6 +112,9 @@ files:
 - lib/rubocop/cop/sorbet/signatures/checked_true_in_signature.rb
 - lib/rubocop/cop/sorbet/signatures/empty_line_after_sig.rb
 - lib/rubocop/cop/sorbet/signatures/enforce_signatures.rb
+- lib/rubocop/cop/sorbet/signatures/forbid_sig.rb
+- lib/rubocop/cop/sorbet/signatures/forbid_sig_with_runtime.rb
+- lib/rubocop/cop/sorbet/signatures/forbid_sig_without_runtime.rb
 - lib/rubocop/cop/sorbet/signatures/keyword_argument_ordering.rb
 - lib/rubocop/cop/sorbet/signatures/signature_build_order.rb
 - lib/rubocop/cop/sorbet/signatures/void_checked_tests.rb
@@ -108,6 +124,7 @@ files:
 - lib/rubocop/cop/sorbet_cops.rb
 - lib/rubocop/sorbet.rb
 - lib/rubocop/sorbet/inject.rb
+- lib/rubocop/sorbet/plugin.rb
 - lib/rubocop/sorbet/version.rb
 - manual/cops.md
 - manual/cops_sorbet.md
@@ -122,7 +139,7 @@ metadata:
   allowed_push_host: https://rubygems.org
   homepage_uri: https://github.com/shopify/rubocop-sorbet
   source_code_uri: https://github.com/shopify/rubocop-sorbet
-post_install_message:
+  default_lint_roller_plugin: RuboCop::Sorbet::Plugin
 rdoc_options: []
 require_paths:
 - lib
@@ -137,8 +154,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.22
-signing_key:
+rubygems_version: 3.6.5
 specification_version: 4
 summary: Automatic Sorbet code style checking tool.
 test_files: []