rubocop-sketchup 1.3.0 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e9935671119993e41eef012a5f69fca2709a59d7db06d28c2e89ff9ea5721aa7
-  data.tar.gz: 05cba02d5fa27e8eab8b32cae46d586c68d5fec8922b959febf89ebd36660ced
+  metadata.gz: 14a5adab07c22fb2c09430a96c7f4376db73489c8015cb46445e71065cd377a0
+  data.tar.gz: d966dfb2e361f1f147357048d1d5db235d19ba3053ff5bd93f254436ef658612
 SHA512:
-  metadata.gz: 06e197b228db557e7d1b0ab9eeec6550e9a70a8e64fa4079f6f4e093249f0057390f9ce6f4270d15e1be956e36b684ff66f5e926a55d0a2b862e33d113a8a3c2
-  data.tar.gz: 54242ae1e4bb983394e9976f02893d98794ae46971755751bdbcea1b4ecdfd9e1c6e3c9e4dde6cd995e7977b788d8ec5fa3dab693723bdd84f30dd3717b8a590
+  metadata.gz: 1174647c4482bdc0d30cb55898c8d2ded2c0a3faee7ce8c174b6b1948ef81a880c32433bf09c116e40629ccfd56fbddd07cdafbc01bf5b84f794c24d6cc94ea5
+  data.tar.gz: f051f128770b9467ee64dedeb7acd2f92d4f645546519d944dd8782b6174d2ea465cd5bf444f09bc22757d00b9621cef5fd185a69b824c639e9192ead7e38e3d

data/Gemfile

@@ -5,7 +5,6 @@ source 'https://rubygems.org'
 gemspec
 
 group :test do
-  gem 'appveyor-worker', '~> 0.2', require: false
   gem 'rake', '~> 12.0', require: false
   gem 'rspec', '~> 3.7', require: false
   gem 'rubocop-performance', '~> 1.11.0', require: false
@@ -16,6 +15,7 @@ end
 
 group :development do
   gem 'bump', '~> 0.5', require: false
+  gem 'bundler', '>= 1.13', '< 3.0', require: false
   gem 'solargraph', '~> 0.34', require: false
   gem 'yard', '~> 0.9', require: false
 end

data/config/default.yml

@@ -142,6 +142,14 @@ SketchupRequirements/ApiNamespace:
   Reference: https://github.com/SketchUp/rubocop-sketchup/tree/main/manual/cops_requirements.md#apinamespace
   Enabled: true
 
+SketchupRequirements/DebugMode:
+  Description: Don't change the SketchUp debug mode in production.
+  Details: >-
+             Disabling the SketchUp debug mode hides warnings other extension
+             developers may rely on.
+  Reference: https://github.com/SketchUp/rubocop-sketchup/tree/main/manual/cops_requirements.md#debugmode
+  Enabled: true
+
 SketchupRequirements/Exit:
   Description: Exit attempts to kill the Ruby interpreter.
   Details: >-
@@ -201,6 +209,11 @@ SketchupRequirements/GlobalVariables:
   Reference: https://github.com/SketchUp/rubocop-sketchup/tree/main/manual/cops_requirements.md#globalvariables
   Enabled: true
 
+SketchupRequirements/InitializeEntity:
+  Description: Do not initialize SketchUp Entity objects with `new`.
+  Reference: https://github.com/SketchUp/rubocop-sketchup/tree/main/manual/cops_requirements.md#initializeentity
+  Enabled: true
+
 SketchupRequirements/LanguageHandlerGlobals:
   Description: Don't use other known global variables.
   Details: >-
@@ -352,6 +365,14 @@ SketchupSuggestions/SketchupFindSupportFile:
   Reference: https://github.com/SketchUp/rubocop-sketchup/tree/main/manual/cops_suggestions.md#sketchupfindsupportfile
   Enabled: true
 
+SketchupSuggestions/Sleep:
+  Description: Avoid kernel `sleep` as it freezes up SketchUp.
+  Details: >-
+             Prefer `UI.start_timer` or a callback from the resource you are
+             waiting for.
+  Reference: https://github.com/SketchUp/rubocop-sketchup/tree/main/manual/cops_suggestions.md#sleep
+  Enabled: true
+
 SketchupSuggestions/ToolDrawingBounds:
   Description: >-
                  When drawing to the viewport implement `getExtents` so drawn

data/lib/rubocop/sketchup/config.rb

@@ -19,7 +19,7 @@ module RuboCop
 
 
       def sketchup_config_key?(key)
-        return unless all_cops_config.key?('SketchUp')
+        return false unless all_cops_config.key?('SketchUp')
 
         all_cops_config.fetch('SketchUp').key?(key)
       end

data/lib/rubocop/sketchup/cop/bugs/material_name.rb

@@ -78,7 +78,7 @@ module RuboCop
           (send #material? :name= _)
         PATTERN
 
-        MSG_SET_NAME = '`material.name=` might add duplicate materials in '\
+        MSG_SET_NAME = '`material.name=` might add duplicate materials in ' \
                        'SU2017 and older versions.'
 
         def on_send(node)

data/lib/rubocop/sketchup/cop/bugs/render_mode.rb

@@ -42,10 +42,10 @@ module RuboCop
           (send _ :[]= (str "RenderMode") (int $_))
         PATTERN
 
-        MSG_OBSOLETE = 'Obsolete render mode will crash in SU2017 and '\
+        MSG_OBSOLETE = 'Obsolete render mode will crash in SU2017 and ' \
                        'newer versions.'
 
-        MSG_INVALID = 'Invalid render mode will crash in SU2017 and '\
+        MSG_INVALID = 'Invalid render mode will crash in SU2017 and ' \
                       'newer versions.'
 
         def on_send(node)

data/lib/rubocop/sketchup/cop/bugs/uniform_scaling.rb

@@ -20,7 +20,7 @@ module RuboCop
           (send (const (const nil? :Geom) :Transformation) :scaling _)
         PATTERN
 
-        MSG = 'Resulting transformation matrix might yield unexpected '\
+        MSG = 'Resulting transformation matrix might yield unexpected ' \
               'results.'
 
         def on_send(node)

data/lib/rubocop/sketchup/cop/deprecations/require_all.rb

@@ -8,7 +8,7 @@ module RuboCop
       # to inadvertently load the wrong file.
       class RequireAll < SketchUp::Cop
 
-        MSG = 'Method is deprecated because it adds the given path '\
+        MSG = 'Method is deprecated because it adds the given path ' \
               'to $LOAD_PATH.'
 
         def_node_matcher :require_all?, <<-PATTERN

data/lib/rubocop/sketchup/cop/deprecations/show_ruby_panel.rb

@@ -6,7 +6,7 @@ module RuboCop
       # Method is deprecated. Use `SKETCHUP_CONSOLE.show` instead.
       class ShowRubyPanel < SketchUp::Cop
 
-        MSG = 'Method is deprecated. Use `SKETCHUP_CONSOLE.show` '\
+        MSG = 'Method is deprecated. Use `SKETCHUP_CONSOLE.show` ' \
               'instead.'
 
         def_node_matcher :show_ruby_panel?, <<-PATTERN

data/lib/rubocop/sketchup/cop/performance/openssl.rb

@@ -15,8 +15,8 @@ module RuboCop
       # It also affects `Net::HTTP` if making HTTPS connections.
       class OpenSSL < SketchUp::Cop
 
-        MSG = 'Avoid use of OpenSSL within SketchUp due to severe performance '\
-              'issues.'
+        MSG = 'Avoid use of OpenSSL within SketchUp due to severe ' \
+              'performance issues.'
 
         # http://www.rubydoc.info/gems/rubocop/RuboCop/NodePattern
         # https://rubocop.readthedocs.io/en/latest/node_pattern/

data/lib/rubocop/sketchup/cop/performance/selection_bulk.rb

@@ -28,7 +28,7 @@ module RuboCop
 
         include RangeHelp
 
-        MSG = 'Prefer changing selection in bulk instead of modifying '\
+        MSG = 'Prefer changing selection in bulk instead of modifying ' \
               'selection within loops.'
 
         # http://www.rubydoc.info/gems/rubocop/RuboCop/NodePattern

data/lib/rubocop/sketchup/cop/performance/type_check.rb

@@ -30,7 +30,7 @@ module RuboCop
 
         include RangeHelp
 
-        MSG = 'String comparisons are very slow, prefer `.is_a?` '\
+        MSG = 'String comparisons are very slow, prefer `.is_a?` ' \
               'instead.'
 
         def_node_matcher :string_class_compare?, <<-PATTERN

data/lib/rubocop/sketchup/cop/requirements/debug_mode.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module SketchupRequirements
+      # Changing the global SketchUp debug mode can cause warnings from other
+      # extensions to be silenced.
+      # Don't let your extension change the debug mode in a production setting.
+      class DebugMode < SketchUp::Cop
+
+        def_node_matcher :sketchup_set_debug?, <<-PATTERN
+          (send (const nil? :Sketchup) :debug_mode= _)
+        PATTERN
+
+        MSG_SET_NAME = 'Changing `Sketchup.debug_mode=` may hide warnings ' \
+                       'other extension developers rely on.'
+
+        def on_send(node)
+          return unless sketchup_set_debug?(node)
+
+          add_offense(node, message: MSG_SET_NAME)
+        end
+
+      end
+    end
+  end
+end

data/lib/rubocop/sketchup/cop/requirements/exit.rb

@@ -13,7 +13,7 @@ module RuboCop
 
         include SketchUp::NoCommentDisable
 
-        MSG = '`%s` attempts to kill the Ruby interpreter. Use `return`, '\
+        MSG = '`%s` attempts to kill the Ruby interpreter. Use `return`, ' \
               '`next`, `break` or `raise` instead.'
 
         # Reference: http://rubocop.readthedocs.io/en/latest/development/

data/lib/rubocop/sketchup/cop/requirements/extension_namespace.rb

@@ -103,7 +103,7 @@ module RuboCop
 
         def message(node)
           namespace = Namespace.new(node.defined_module_name).from_root
-          format('Use a single root namespace. '\
+          format('Use a single root namespace. ' \
                  '(Found `%<found>s`; Previously found `%<expected>s`)',
                  found: namespace, expected: @@namespace)
         end

data/lib/rubocop/sketchup/cop/requirements/file_structure.rb

@@ -37,7 +37,7 @@ module RuboCop
 
           # Ensure there is only one root Ruby file.
           if root_ruby_files.size != 1
-            msg = 'Extensions must have exactly one root Ruby (.rb) file. '\
+            msg = 'Extensions must have exactly one root Ruby (.rb) file. ' \
                   'Found: %d'
             add_offense(nil,
                         location: range,
@@ -65,8 +65,8 @@ module RuboCop
           # Ensure support directory's name match the root Ruby file.
           support_directory = sub_folders.first
           unless support_directory.basename.to_s == extension_basename
-            msg = 'Extensions must have a support directory matching the name '\
-                  'of the root Ruby file. Expected %s, found %s'
+            msg = 'Extensions must have a support directory matching the ' \
+                  'name of the root Ruby file. Expected %s, found %s'
             msg = format(msg, extension_basename, support_directory.basename)
             add_offense(nil,
                         location: range,

data/lib/rubocop/sketchup/cop/requirements/gem_install.rb

@@ -25,7 +25,7 @@ module RuboCop
         include RangeHelp
         include SketchUp::NoCommentDisable
 
-        MSG = '`Gem.install` is unreliable in SketchUp, and can cause '\
+        MSG = '`Gem.install` is unreliable in SketchUp, and can cause ' \
               'extensions to clash.'
 
         # Reference: http://rubocop.readthedocs.io/en/latest/development/

data/lib/rubocop/sketchup/cop/requirements/get_extension_license.rb

@@ -16,7 +16,7 @@ module RuboCop
 
         MSG_INVALID = 'Invalid extension GUID'
 
-        MSG_WRONG_TYPE = 'Only pass in extension GUID from local string '\
+        MSG_WRONG_TYPE = 'Only pass in extension GUID from local string ' \
                          'literals.'
 
         MSG_TRAILING_SPACE = 'Extra space in extension GUID'

data/lib/rubocop/sketchup/cop/requirements/initialize_entity.rb

@@ -0,0 +1,76 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module SketchupRequirements
+      # SketchUp Entity objects (`Face`, `Edge`, `Group` etc) should not be
+      # initialized using the `new` constructor. These objects are managed by
+      # SketchUp. Instead use the `Sketchup::Entities#add_*` methods to create
+      # them.
+      class InitializeEntity < SketchUp::Cop
+
+        ENTITY_CLASSES = %i[
+          ComponentDefinition
+          ComponentInstance
+          ConstructionLine
+          ConstructionPoint
+          Dimension
+          DimensionLinear
+          DimensionRadial
+          Edge
+          Face
+          Group
+          Image
+          SectionPlane
+          Text
+          AttributeDictionaries
+          AttributeDictionary
+          Axes
+          Behavior
+          Curve
+          DefinitionList
+          Drawingelement
+          EdgeUse
+          Layer
+          LayerFolder
+          Layers
+          LineStyle
+          LineStyles
+          Loop
+          Material
+          Materials
+          Page
+          Pages
+          RenderingOptions
+          ShadowInfo
+          Style
+          Styles
+          Texture
+          Vertex
+        ].freeze
+
+        def_node_matcher :init_entity?, <<-PATTERN
+          (send (const (const nil? :Sketchup) #entity? ) :new ... )
+        PATTERN
+
+        MSG_INITIALIZE_ENTITY = 'Entity objects (Face, Edge, Group etc) are ' \
+                                'managed SketchUp. Instead of using the ' \
+                                '`new` constructor, use the ' \
+                                '`Sketchup::Entities#add_*` methods.'
+
+        def on_send(node)
+          return unless init_entity?(node)
+
+          add_offense(node, message: MSG_INITIALIZE_ENTITY)
+        end
+
+        private
+
+        def entity?(sym)
+          ENTITY_CLASSES.include?(sym)
+        end
+
+      end
+    end
+  end
+end

data/lib/rubocop/sketchup/cop/requirements/language_handler_globals.rb

@@ -10,8 +10,8 @@ module RuboCop
 
         include SketchUp::NoCommentDisable
 
-        MSG = 'Avoid globals in general, but especially these which are known '\
-              'to be in use.'
+        MSG = 'Avoid globals in general, but especially these which are ' \
+              'known to be in use.'
 
         LH_GLOBALS = %i[
           $dc_strings

data/lib/rubocop/sketchup/cop/requirements/minimal_registration.rb

@@ -32,7 +32,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include SketchUp::ExtensionProject
 
-        MSG = "Don't load extension files in the root file registering the "\
+        MSG = "Don't load extension files in the root file registering the " \
               'extension.'
 
         # Reference: http://rubocop.readthedocs.io/en/latest/node_pattern/

data/lib/rubocop/sketchup/cop/requirements/sketchup_require.rb

@@ -26,16 +26,17 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include RangeHelp
 
-        MSG_SKETCHUP_REQUIRE_EXT_NAME = 'Do not hard code file extensions '\
+        MSG_SKETCHUP_REQUIRE_EXT_NAME = 'Do not hard code file extensions ' \
                                         'with `Sketchup.require`.'
 
-        MSG_EXTENSION_NEW_EXT_NAME = 'Do not hard code file extensions '\
+        MSG_EXTENSION_NEW_EXT_NAME = 'Do not hard code file extensions ' \
                                      'with `SketchupExtension.new`.'
 
-        MSG_REQUIRE_FOR_BINARY = 'Use `require` instead of `Sketchup.require` '\
-                                 'to load binary Ruby libraries.'
+        MSG_REQUIRE_FOR_BINARY = 'Use `require` instead of ' \
+                                 '`Sketchup.require` to load binary Ruby ' \
+                                 'libraries.'
 
-        MSG_REQUIRE_ENCRYPTED = 'Use `Sketchup.require` when loading Ruby '\
+        MSG_REQUIRE_ENCRYPTED = 'Use `Sketchup.require` when loading Ruby ' \
                                 'files for encrypted extensions.'
 
         def_node_matcher :ruby_require, <<-PATTERN
@@ -88,8 +89,8 @@ module RuboCop
         private
 
         def binary_require?(filename)
-          return unless extension_binaries?
-          return if extension_binaries.empty?
+          return false unless extension_binaries?
+          return false if extension_binaries.empty?
 
           extension_binaries.include?(filename)
         end

data/lib/rubocop/sketchup/cop/suggestions/compatibility.rb

@@ -86,8 +86,8 @@ module RuboCop
         end
 
         def report(node, feature_name, feature_version, feature_type)
-          message = "The #{feature_type} `#{feature_name}` was added in "\
-                    "#{feature_version} which is incompatible with target "\
+          message = "The #{feature_type} `#{feature_name}` was added in " \
+                    "#{feature_version} which is incompatible with target " \
                     "#{sketchup_target_version}."
           location = find_node_location(node)
           add_offense(node, location: location, message: message)

data/lib/rubocop/sketchup/cop/suggestions/monkey_patched_api.rb

@@ -31,7 +31,7 @@ module RuboCop
           end
 
           path = dc_method[:path]
-          message = "#{path}##{name} is not part of the official API. "\
+          message = "#{path}##{name} is not part of the official API. " \
                     "It's a monkey-patched addition by Dynamic Components."
           add_offense(node,
                       location: :selector,

data/lib/rubocop/sketchup/cop/suggestions/sketchup_find_support_file.rb

@@ -15,7 +15,7 @@ module RuboCop
       # control.
       class SketchupFindSupportFile < SketchUp::Cop
 
-        MSG = 'Avoid `Sketchup.find_support_file` to find your '\
+        MSG = 'Avoid `Sketchup.find_support_file` to find your ' \
               "extension's files."
 
         # http://www.rubydoc.info/gems/rubocop/RuboCop/NodePattern

data/lib/rubocop/sketchup/cop/suggestions/sleep.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module SketchupSuggestions
+      # Avoid kernel `sleep` as it freezes up SketchUp.
+      # Prefer `UI.start_timer` or a callback from the resource you are waiting
+      # for.
+      class Sleep < SketchUp::Cop
+        MSG = '`sleep` freezes up SketchUp. Prefer `UI.start_timer` or a ' \
+              'callback for the resource you are waiting for.'
+
+        def_node_matcher :sleep?, <<-PATTERN
+          (send {(const nil? :Kernel) nil?} :sleep ...)
+        PATTERN
+
+        def on_send(node)
+          return unless sleep?(node)
+
+          add_offense(node, message: MSG)
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/sketchup/cop/suggestions/tool_drawing_bounds.rb

@@ -28,8 +28,8 @@ module RuboCop
 
         include SketchUp::ToolChecker
 
-        MSG_MISSING_GET_EXTENTS = 'When drawing to the viewport implement '\
-                                  '`getExtents` so drawn geometry is not '\
+        MSG_MISSING_GET_EXTENTS = 'When drawing to the viewport implement ' \
+                                  '`getExtents` so drawn geometry is not ' \
                                   'clipped.'
 
         def on_tool_class(class_node, body_methods)

data/lib/rubocop/sketchup/cop/suggestions/tool_invalidate.rb

@@ -30,12 +30,12 @@ module RuboCop
 
         include SketchUp::ToolChecker
 
-        MSG_MISSING_INVALIDATE_METHOD = 'When drawing to the viewport, make '\
-                                        'sure to `suspend` and `deactivate` '\
+        MSG_MISSING_INVALIDATE_METHOD = 'When drawing to the viewport, make ' \
+                                        'sure to `suspend` and `deactivate` ' \
                                         'calls `view.invalidate`.'
 
-        MSG_MISSING_INVALIDATE = 'When drawing to the viewport, make sure to '\
-                                 'call `view.invalidate` when the tool '\
+        MSG_MISSING_INVALIDATE = 'When drawing to the viewport, make sure to ' \
+                                 'call `view.invalidate` when the tool ' \
                                  'becomes inactive.'
 
         def_node_search :view_invalidate?, <<-PATTERN

data/lib/rubocop/sketchup/cop/suggestions/tool_user_input.rb

@@ -23,7 +23,7 @@ module RuboCop
 
         include SketchUp::ToolChecker
 
-        MSG_MISSING_ENABLE_VCB = 'When accepting user input, make sure to '\
+        MSG_MISSING_ENABLE_VCB = 'When accepting user input, make sure to ' \
                                  'define `enableVCB?`.'
 
         def on_tool_class(class_node, body_methods)

data/lib/rubocop/sketchup/cop/suggestions/toolbar_timer.rb

@@ -24,7 +24,7 @@ module RuboCop
 
         include RangeHelp
 
-        MSG = 'Wrapping `toolbar.restore` in `UI.start_timer` is '\
+        MSG = 'Wrapping `toolbar.restore` in `UI.start_timer` is ' \
               'redundant.'
 
         def_node_matcher :toolbar_new?, <<-PATTERN

data/lib/rubocop/sketchup/features.rb

@@ -41,6 +41,96 @@ module RuboCop
 
       FEATURES = [
 
+        {
+          version: 'SketchUp 2024.0',
+          types: {
+            method: [
+              'Sketchup::View#graphics_engine',
+            ],
+          },
+        },
+
+        {
+          version: 'LayOut 2023.1',
+          types: {
+            method: [
+              'Layout::Document#render_mode_override',
+              'Layout::Document#render_mode_override=',
+              'Layout::PageInfo#image_display_resolution',
+              'Layout::PageInfo#image_display_resolution=',
+              'Layout::PageInfo#image_output_resolution',
+              'Layout::PageInfo#image_output_resolution=',
+              'Layout::SketchUpModel#output_entities',
+            ],
+          },
+        },
+
+        {
+          version: 'SketchUp 2023.1',
+          types: {
+            method: [
+              'Sketchup::ArcCurve#circular?',
+              'UI.get_clipboard_data',
+              'UI.set_clipboard_data',
+            ],
+          },
+        },
+
+        {
+          version: 'LayOut 2023.0',
+          types: {
+            class: [
+              'Layout::ReferenceEntity',
+            ],
+            method: [
+              'Layout::AutoTextDefinition#sequence_type',
+              'Layout::AutoTextDefinition#sequence_type=',
+              'Layout::ReferenceEntity#clip_mask',
+              'Layout::ReferenceEntity#clip_mask=',
+              'Layout::ReferenceEntity#entities',
+            ],
+          },
+        },
+
+        {
+          version: 'SketchUp 2023.0',
+          types: {
+            class: [
+              'Sketchup::Overlay',
+              'Sketchup::OverlaysManager',
+            ],
+            method: [
+              'Sketchup.resize_viewport',
+              'Sketchup::ComponentDefinition#thumbnail_camera',
+              'Sketchup::ComponentDefinition#thumbnail_camera=',
+              'Sketchup::Model#overlays',
+              'Sketchup::Overlay#description',
+              'Sketchup::Overlay#description=',
+              'Sketchup::Overlay#draw',
+              'Sketchup::Overlay#enabled=',
+              'Sketchup::Overlay#enabled?',
+              'Sketchup::Overlay#getExtents',
+              'Sketchup::Overlay#initialize',
+              'Sketchup::Overlay#name',
+              'Sketchup::Overlay#onMouseEnter',
+              'Sketchup::Overlay#onMouseLeave',
+              'Sketchup::Overlay#onMouseMove',
+              'Sketchup::Overlay#overlay_id',
+              'Sketchup::Overlay#source',
+              'Sketchup::Overlay#start',
+              'Sketchup::Overlay#stop',
+              'Sketchup::Overlay#valid?',
+              'Sketchup::OverlaysManager#[]',
+              'Sketchup::OverlaysManager#add',
+              'Sketchup::OverlaysManager#at',
+              'Sketchup::OverlaysManager#each',
+              'Sketchup::OverlaysManager#length',
+              'Sketchup::OverlaysManager#remove',
+              'Sketchup::OverlaysManager#size',
+            ],
+          },
+        },
+
         {
           version: 'LayOut 2022.0',
           types: {
@@ -1302,6 +1392,7 @@ module RuboCop
         fov_is_height?
         georeferenced?
         get_classification_value
+        get_clipboard_data
         get_drawingelement_visibility
         get_glued_instances
         get_texture_projection
@@ -1364,8 +1455,10 @@ module RuboCop
         same_direction?
         scale_2d
         screen_scale_factor
+        sequence_type
         set_can_close
         set_classification_value
+        set_clipboard_data
         set_download_progress_callback
         set_drawingelement_visibility
         set_full_security
@@ -1383,6 +1476,7 @@ module RuboCop
         status_bar_text
         style_modified?
         text_bounds
+        thumbnail_camera
         unit_vector?
         unload_schema
         upper_left

data/lib/rubocop/sketchup/formatter/extension_review.rb

@@ -62,7 +62,7 @@ module RuboCop
         context = ERBContext.new(categories, files, summary)
 
         template = File.read(TEMPLATE_PATH, encoding: Encoding::UTF_8)
-        erb = ERB.new(template, nil, '-')
+        erb = ERB.new(template, trim_mode: '-')
         html = erb.result(context.binding)
 
         output.write html
@@ -242,7 +242,7 @@ module RuboCop
 
         def source_after_highlight(offense)
           source_line = offense.location.source_line
-          escape(source_line[offense.highlighted_area.end_pos..-1])
+          escape(source_line[offense.highlighted_area.end_pos..])
         end
 
         def possible_ellipses(location)

data/lib/rubocop/sketchup/generator.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+require 'rubocop'
+
+module RuboCop
+  module SketchUp
+    class Generator < RuboCop::Cop::Generator
+      def source_path
+        department = snake_case(badge.department.to_s).gsub(/^sketchup_/, '')
+        File.join(
+            'lib',
+            'rubocop',
+            'sketchup', # We have our cops nested one extra level.
+            'cop',
+            department,
+            "#{snake_case(badge.cop_name.to_s)}.rb"
+          )
+      end
+    end
+  end
+end

data/lib/rubocop/sketchup/sketchup_version.rb

@@ -67,6 +67,9 @@ module RuboCop
       #
       # The second item in the array is maintenance annotation
       VALID_VERSIONS = [
+        [2024.0, 0],
+        [2023.1, 0],
+        [2023.0, 0],
         [2022.0, 0],
         [2021.1, 0],
         [2021.0, 0],

data/lib/rubocop/sketchup/tool_checker.rb

@@ -13,11 +13,9 @@ module RuboCop
 
       private
 
-      # rubocop:disable Lint/UnusedMethodArgument
       def on_tool_class(class_node, body, body_methods)
         raise NotImplementedError, 'Implement this method'
       end
-      # rubocop:enable Lint/UnusedMethodArgument
 
       def body_methods(body)
         return [body] if body.def_type?

data/lib/rubocop/sketchup/version.rb

@@ -2,6 +2,6 @@
 
 module RuboCop
   module SketchUp
-    VERSION = '1.3.0'
+    VERSION = '1.5.0'
   end
 end

data/rubocop-sketchup.gemspec

@@ -13,7 +13,7 @@ Gem::Specification.new do |spec|
 
   spec.version = RuboCop::SketchUp::VERSION
   spec.platform = Gem::Platform::RUBY
-  spec.required_ruby_version = '>= 2.5.0'
+  spec.required_ruby_version = '>= 2.6.0'
 
   spec.require_paths = ['lib']
   spec.files = Dir[
@@ -25,5 +25,4 @@ Gem::Specification.new do |spec|
   ]
 
   spec.add_dependency 'rubocop', '>= 0.82', '< 2.0'
-  spec.add_development_dependency 'bundler', '>= 1.13', '< 3.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-sketchup
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.5.0
 platform: ruby
 authors:
 - Trimble Inc, SketchUp Team
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-03-07 00:00:00.000000000 Z
+date: 2024-04-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -30,28 +30,8 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '1.13'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '3.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '1.13'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '3.0'
 description: RuboCop rules for SketchUp extensions.
-email: 
+email:
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -79,6 +59,7 @@ files:
 - lib/rubocop/sketchup/cop/performance/type_check.rb
 - lib/rubocop/sketchup/cop/performance/typename.rb
 - lib/rubocop/sketchup/cop/requirements/api_namespace.rb
+- lib/rubocop/sketchup/cop/requirements/debug_mode.rb
 - lib/rubocop/sketchup/cop/requirements/exit.rb
 - lib/rubocop/sketchup/cop/requirements/extension_namespace.rb
 - lib/rubocop/sketchup/cop/requirements/file_structure.rb
@@ -88,6 +69,7 @@ files:
 - lib/rubocop/sketchup/cop/requirements/global_include.rb
 - lib/rubocop/sketchup/cop/requirements/global_methods.rb
 - lib/rubocop/sketchup/cop/requirements/global_variables.rb
+- lib/rubocop/sketchup/cop/requirements/initialize_entity.rb
 - lib/rubocop/sketchup/cop/requirements/language_handler_globals.rb
 - lib/rubocop/sketchup/cop/requirements/load_path.rb
 - lib/rubocop/sketchup/cop/requirements/minimal_registration.rb
@@ -106,6 +88,7 @@ files:
 - lib/rubocop/sketchup/cop/suggestions/monkey_patched_api.rb
 - lib/rubocop/sketchup/cop/suggestions/operation_name.rb
 - lib/rubocop/sketchup/cop/suggestions/sketchup_find_support_file.rb
+- lib/rubocop/sketchup/cop/suggestions/sleep.rb
 - lib/rubocop/sketchup/cop/suggestions/tool_drawing_bounds.rb
 - lib/rubocop/sketchup/cop/suggestions/tool_invalidate.rb
 - lib/rubocop/sketchup/cop/suggestions/tool_user_input.rb
@@ -115,6 +98,7 @@ files:
 - lib/rubocop/sketchup/extension_project.rb
 - lib/rubocop/sketchup/features.rb
 - lib/rubocop/sketchup/formatter/extension_review.rb
+- lib/rubocop/sketchup/generator.rb
 - lib/rubocop/sketchup/inject.rb
 - lib/rubocop/sketchup/namespace.rb
 - lib/rubocop/sketchup/namespace_checker.rb
@@ -129,7 +113,7 @@ homepage: https://github.com/sketchup/rubocop-sketchup
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -137,15 +121,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.15
-signing_key: 
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: RuboCop rules for SketchUp extensions.
 test_files: []