rubocop-bridgetown 0.4.1 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c1c55c379762ea7310036dd92c1c6133257af4b161e3ad41d722e2acded33082
-  data.tar.gz: 464b333e864028797b1975bbc21edb685043faf6449536190f05f163910a1bf4
+  metadata.gz: 55ea7b4b8b72d1401042de211382cae15ca378f6bcc70cd0d285de5a04db69e9
+  data.tar.gz: ba2a83d81067a2bf70c1c5b9477471b0127858ff44d1748be926149ed1dd98a6
 SHA512:
-  metadata.gz: '094a43e9ab0e00667fe83e775bf9bc85a487df3308bcdc1692dbded1c3fe94599c9a278f0d6a283e4a4cc8da7db7af0fe2e1a688196b3816071b777330400ac9'
-  data.tar.gz: 2870d7b4a58f7ab628978dcc3ed4e4ecebb1f37ca0e003e670d69764fd61ba3161224234e9312c5c1527b20226723fee958fa65609c084b422aaecb86abf1186
+  metadata.gz: cd922c7f02e8f8f4ef9ef46d5ae1252c62dbf69df305638856870dc17aba727c0bcbcfc24bf6995c0e1f1a98ace36cf1ccfadeb4c4dd11f5e76b7c7b356c564f
+  data.tar.gz: 96beab10089b0ab81d2381e75080ab4e650eca57b86486aa61be69bfb839bb454da7bce1b2d3b77be47321ca3d5386a84062f94c88bd19d40f9e829189878fed

data/.rubocop.yml

@@ -2,7 +2,7 @@ require:
   - rubocop-performance
 
 AllCops:
-  TargetRubyVersion: 2.5
+  TargetRubyVersion: 3.1
   NewCops: enable
   SuggestExtensions: false
   Exclude:
@@ -26,6 +26,8 @@ Layout/HashAlignment:
   EnforcedHashRocketStyle: table
 Layout/IndentationWidth:
   Severity: error
+Layout/LeadingCommentSpace:
+  Enabled: false
 Layout/MultilineMethodCallIndentation:
   EnforcedStyle: indented
 Layout/MultilineOperationIndentation:
@@ -78,7 +80,7 @@ Style/ModuleFunction:
 Style/MultilineBlockChain:
   Enabled: false
 Style/MultilineTernaryOperator:
-  Severity: error
+  Enabled: false
 Style/ParallelAssignment:
   Enabled: false
 Style/PercentLiteralDelimiters:

data/README.md

@@ -40,6 +40,8 @@ inherit_gem:
 
 Running `bundle exec rubocop` will now automatically load the `rubocop-bridgetown` cops together with the standard cops.
 
+**Note:** if you want just the extra cops from this gem such as `Bridgetown/HTMLEscapedHeredoc` but wish to use your own configuration otherwise, you can omit the `inherit_gem` section entirely.
+
 You can also add a `rubocop` task to your `Rakefile`.
 
 ```ruby
@@ -72,8 +74,10 @@ You can override any settings inherited from the extension by configuring cops i
 
 Besides cops which are provided directly by RuboCop and `rubocop-performance`, there are a few additional cops provided by this plugin:
 
-* `Bridgetown/HTMLEscapedHeredoc`: this will monitor any heredocs in your code for potential XSS issues inside of any string interpolations. To avoid linting errors, you will need to wrap any interpolated code inside of one of the following method names: `html`, `html_map`, `html_attributes`, `text`, or `render`. These methods are provided by the [Streamlined](https://github.com/bridgetownrb/streamlined) gem, bundled in Bridgetown 1.4 by default (but you can use them in any Ruby application including Rails).
+* `Bridgetown/InsecureHeredoc`: this will monitor any heredocs in your code starting with `HTML` or `MARKDOWN` for potential XSS issues inside of any string interpolations. To avoid linting errors, you will need to wrap any interpolated code in the string with one of the following method names: `html`, `html_map`, `html_attributes`, `text`, or `render`. These methods are provided by the [Streamlined](https://github.com/bridgetownrb/streamlined) gem, bundled in Bridgetown 2.0 by default (but you can use them in any Ruby application including Rails).
 * `Bridgetown/NoPAllowed`: this encourages using your framework's logger rather than `p` to output debugging information.
 * `Bridgetown/NoPutsAllowed`: this encourages using your framework's logger rather than `puts` to output debugging information.
 
 You can disable any of these cops in specific parts of your codebase as needed, or by setting `Enabled: false` for any particular cop in your `.rubocop.yml`.
+
+Regarding recommended Streamlined syntax, you may want to exclude `Layout/SpaceBeforeFirstArg` and `Layout/SpaceBeforeBlockBraces` in the folders you write Streamlined components and helpers. This is so you can write `text->{ ... }`, `html->{ ... }`, etc. (Otherwise RuboCop will require you to write `text -> { ... }`, etc. which is more verbose.)

data/lib/rubocop/cop/bridgetown/{html_escaped_heredoc.rb → insecure_heredoc.rb}

@@ -3,14 +3,17 @@
 module RuboCop
   module Cop
     module Bridgetown
-      class HTMLEscapedHeredoc < Cop
+      class InsecureHeredoc < Cop
         include Heredoc
 
-        MSG = "Insecure heredoc detected. Use `html`, `html_map`, `html_attributes`, `text`, or `render` inside interpolations."
+        MSG = "Insecure heredoc detected. Use `html`, `html_map`, `html_attributes`, `text`, " \
+              "or `render` inside interpolations."
 
         def on_heredoc(node)
           return unless node.source.match?(%r!(HTML|MARKDOWN)$!) &&
-            heredoc_body(node).match?(%r%[^\\]#\{(?!\s*?(html|html_map|html_attributes|text|render)[ \-\(])%)
+            heredoc_body(node).match?(
+              %r%[^\\]#\{(?!\s*?(html|html_map|html_attributes|text|render)[ \-\(])%
+            )
 
           add_offense(node, message: MSG)
         end

data/lib/rubocop-bridgetown.rb

@@ -3,4 +3,4 @@
 require "rubocop"
 
 path_to_cops = File.join(File.expand_path("rubocop", __dir__), "cop", "**", "*.rb")
-Dir[path_to_cops].sort.each { |cop| require cop }
+Dir[path_to_cops].each { |cop| require cop }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-bridgetown
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.5.0
 platform: ruby
 authors:
 - Bridgetown Team
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-11-11 00:00:00.000000000 Z
+date: 2024-04-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -38,36 +38,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.12'
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '12.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '12.0'
-description: A RuboCop extension to enforce common code style in Bridgetown projects
-  and beyond
+description: A RuboCop extension to enforce common code style in Bridgetown and beyond
 email:
 - maintainers@bridgetownrb.com
 executables: []
@@ -78,13 +49,14 @@ files:
 - LICENSE
 - README.md
 - lib/rubocop-bridgetown.rb
-- lib/rubocop/cop/bridgetown/html_escaped_heredoc.rb
+- lib/rubocop/cop/bridgetown/insecure_heredoc.rb
 - lib/rubocop/cop/bridgetown/no_p_allowed.rb
 - lib/rubocop/cop/bridgetown/no_puts_allowed.rb
 homepage: https://github.com/bridgetownrb/rubocop-bridgetown
 licenses:
 - MIT
-metadata: {}
+metadata:
+  rubygems_mfa_required: 'true'
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -93,14 +65,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 3.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.3
+rubygems_version: 3.5.3
 signing_key:
 specification_version: 4
 summary: Code style check for Bridgetown projects