rubocop-isucon 0.1.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c235faec7d2edaadc48681992ad139ffc81d0745e010d923132a509f8012daa3
-  data.tar.gz: eecb16448d52f4a48860171045e9fc6b6108ce68db7d5aa4601a6fcc865828b0
+  metadata.gz: 6bd0eac249a23368414a86aa1ab08826542cf644fa50b19017a2bcfd6648d594
+  data.tar.gz: 18e829ac97ac7c7cb9696a1078f36b8a958fd340ddbb9bc1b232b2630e683df2
 SHA512:
-  metadata.gz: 2a82eca957548836926ef797d18ed5e014ab26c03407b1bdfd82a659b082479f1d5ede81c05baacd0c172b5856b38b839262fb9ba238618b03597f524f2ddeda
-  data.tar.gz: 3571157d23e59636c0af87617111d0146c00dedfe3da74f4b64fd4e4e108886ef32d1fa56f65f531578be251eca6f0c8182a263cec18c098c497cd9fcc8be6a3
+  metadata.gz: 58238a855da0596f653d47436977b44cc7722982a1b59e60850f79003b356d85071cba061cca6adae2b4cfa4b63ab489e121135e3c2954f366f5ca1c375e1bbf
+  data.tar.gz: b7747989e1219c7fee4f834b8811e419d3c9f6083336d5fab7e7b19a3f8e1f0dad86644e708b18b926b9de3a678edf8b31495097e319bff970599b80d0ab21d9

data/.github/dependabot.yml

@@ -0,0 +1,10 @@
+# c.f. https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file
+version: 2
+
+updates:
+  - package-ecosystem: github-actions
+    directory: /
+    schedule:
+      interval: weekly
+    assignees:
+      - sue445

data/.github/workflows/pages.yml

@@ -0,0 +1,68 @@
+# Simple workflow for deploying static content to GitHub Pages
+name: Deploy static content to Pages
+
+on:
+  # Runs on pushes targeting the default branch
+  push:
+    branches:
+      - main
+
+  # Allows you to run this workflow manually from the Actions tab
+  workflow_dispatch:
+
+# Sets permissions of the GITHUB_TOKEN to allow deployment to GitHub Pages
+permissions:
+  contents: read
+  pages: write
+  id-token: write
+
+# Allow one concurrent deployment
+concurrency:
+  group: "pages"
+  cancel-in-progress: true
+
+jobs:
+  # Single deploy job since we're just deploying
+  deploy:
+    environment:
+      name: github-pages
+      url: ${{ steps.deployment.outputs.page_url }}
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Install packages
+        run: |
+          set -xe
+          sudo apt-get update
+          sudo apt-get install -y libgda-5.0
+
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ruby
+          bundler-cache: true
+
+      - run: bundle exec yard
+
+      - name: Setup Pages
+        uses: actions/configure-pages@v3
+      - name: Upload artifact
+        uses: actions/upload-pages-artifact@v2
+        with:
+          # Upload entire repository
+          path: './doc'
+      - name: Deploy to GitHub Pages
+        id: deployment
+        uses: actions/deploy-pages@main
+
+      - name: Slack Notification (not success)
+        uses: lazy-actions/slatify@master
+        if: "! success()"
+        continue-on-error: true
+        with:
+          job_name: "*pages*"
+          type: ${{ job.status }}
+          icon_emoji: ":octocat:"
+          url: ${{ secrets.SLACK_WEBHOOK }}
+          token: ${{ secrets.GITHUB_TOKEN }}

data/.github/workflows/test.yml

@@ -21,23 +21,19 @@ jobs:
 
       matrix:
         ruby:
-          - "2.6"
-          - "2.7"
           - "3.0"
           - "3.1"
+          - "3.2"
         gemfile:
           - activerecord_6_1
           - activerecord_7_0
-        exclude:
-          # activerecord 7.0+ requires Ruby 2.7+
-          - ruby:    "2.6"
-            gemfile: activerecord_7_0
+          - activerecord_7_1
 
     env:
       BUNDLE_GEMFILE: gemfiles/${{ matrix.gemfile }}.gemfile
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
 
       - name: Install packages
         run: |

data/.rubocop.yml

@@ -1,10 +1,11 @@
 require:
   - rubocop-performance
+  - rubocop-yard
 
 AllCops:
   NewCops: enable
   SuggestExtensions: false
-  TargetRubyVersion: 2.6
+  TargetRubyVersion: 3.0
   Exclude:
     - 'gemfiles/vendor/**/*'
     - 'benchmark/**/*'
@@ -15,6 +16,9 @@ AllCops:
     - 'vendor/**/*'
     - '.git/**/*'
 
+Gemspec/DevelopmentDependencies:
+  EnforcedStyle: gemspec
+
 Layout/DotPosition:
   EnforcedStyle: trailing
 

data/CHANGELOG.md

@@ -1,5 +1,25 @@
 ## [Unreleased]
-[full changelog](http://github.com/sue445/rubocop-isucon/compare/v0.1.0...main)
+[full changelog](http://github.com/sue445/rubocop-isucon/compare/v1.0.0...main)
+
+## [1.0.0] - 2023-10-29
+[full changelog](http://github.com/sue445/rubocop-isucon/compare/v0.2.0...v1.0.0)
+
+* Requires Ruby 3.0+
+  * https://github.com/sue445/rubocop-isucon/pull/236
+
+## [0.2.0] - 2022-07-31
+[full changelog](http://github.com/sue445/rubocop-isucon/compare/v0.1.0...v0.2.0)
+
+* Add `Isucon/Sqlite3/SelectAsterisk`
+  * https://github.com/sue445/rubocop-isucon/pull/202
+* Add `Isucon/Sqlite3/WhereWithoutIndex`
+  * https://github.com/sue445/rubocop-isucon/pull/207
+* Add `Isucon/Sqlite3/JoinWithoutIndex`
+  * https://github.com/sue445/rubocop-isucon/pull/208
+* Add `Isucon/Sqlite3/ManyJoinTable`
+  * https://github.com/sue445/rubocop-isucon/pull/209
+* Add `Isucon/Sqlite3/NPlusOneQuery`
+  * https://github.com/sue445/rubocop-isucon/pull/210
 
 ## [0.1.0] - 2022-07-23
 

data/README.md

@@ -1,6 +1,7 @@
 # RuboCop ISUCON
 RuboCop plugin for ruby reference implementation of [ISUCON](https://github.com/isucon)
 
+[![Gem Version](https://badge.fury.io/rb/rubocop-isucon.svg)](https://badge.fury.io/rb/rubocop-isucon)
 [![Build Status](https://github.com/sue445/rubocop-isucon/workflows/test/badge.svg?branch=main)](https://github.com/sue445/rubocop-isucon/actions?query=workflow%3Atest)
 
 ## Installation
@@ -72,16 +73,25 @@ Isucon/Mysql2:
     password: isucon
     encoding: utf8
     port: 3306
+
+Isucon/Sqlite3:
+  Database:
+    adapter: sqlite3
+    database: # TODO: Fix this
 ```
 
 `Database` isn't configured in `.rubocop.yml`, some cops doesn't work
 
-| cop                               | offense detection          | auto-correct               |
-|-----------------------------------|----------------------------|----------------------------|
-| `Isucon/Mysql2/JoinWithoutIndex`  | `Database` is **required** | Not supported              |
-| `Isucon/Mysql2/NPlusOneQuery`     | `Database` is optional     | `Database` is **required** |
-| `Isucon/Mysql2/SelectAsterisk`    | `Database` is optional     | `Database` is **required** |
-| `Isucon/Mysql2/WhereWithoutIndex` | `Database` is **required** | Not supported              |
+| cop                                | offense detection          | auto-correct               |
+|------------------------------------|----------------------------|----------------------------|
+| `Isucon/Mysql2/JoinWithoutIndex`   | `Database` is **required** | Not supported              |
+| `Isucon/Mysql2/NPlusOneQuery`      | `Database` is optional     | `Database` is **required** |
+| `Isucon/Mysql2/SelectAsterisk`     | `Database` is optional     | `Database` is **required** |
+| `Isucon/Mysql2/WhereWithoutIndex`  | `Database` is **required** | Not supported              |
+| `Isucon/Sqlite3/JoinWithoutIndex`  | `Database` is **required** | Not supported              |
+| `Isucon/Sqlite3/NPlusOneQuery`     | `Database` is optional     | `Database` is **required** |
+| `Isucon/Sqlite3/SelectAsterisk`    | `Database` is optional     | `Database` is **required** |
+| `Isucon/Sqlite3/WhereWithoutIndex` | `Database` is **required** | Not supported              |
 
 ## Documentation
 See. https://sue445.github.io/rubocop-isucon/
@@ -89,6 +99,7 @@ See. https://sue445.github.io/rubocop-isucon/
 * `Isucon/Mysql2` department docs : https://sue445.github.io/rubocop-isucon/RuboCop/Cop/Isucon/Mysql2.html
 * `Isucon/Shell` department docs : https://sue445.github.io/rubocop-isucon/RuboCop/Cop/Isucon/Shell.html
 * `Isucon/Sinatra` department docs : https://sue445.github.io/rubocop-isucon/RuboCop/Cop/Isucon/Sinatra.html
+* `Isucon/Sqlite3` department docs : https://sue445.github.io/rubocop-isucon/RuboCop/Cop/Isucon/Sqlite3.html
 
 ## Benchmark
 See [benchmark/](benchmark/)
@@ -106,3 +117,10 @@ Bug reports and pull requests are welcome on GitHub at https://github.com/sue445
 ## License
 
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).
+
+ISUCON is a trademark or registered trademark of LINE Corporation.
+
+https://isucon.net
+
+## Presentation
+* [Fix SQL N\+1 queries with RuboCop](https://speakerdeck.com/sue445/fix-sql-n-plus-one-queries-with-rubocop) at [RubyKaigi 2023](https://rubykaigi.org/2013/) :gem:

data/config/default.yml

@@ -16,7 +16,7 @@ Isucon/Mysql2/ManyJoinTable:
   CountTables: 3
 
 Isucon/Mysql2/NPlusOneQuery:
-  Description: 'Checks that N+1 query is not used'
+  Description: 'Checks that there’s no N+1 query'
   Enabled: true
   VersionAdded: '0.1.0'
   SafeAutoCorrect: false
@@ -81,3 +81,39 @@ Isucon/Sinatra/ServeStaticFile:
   Enabled: true
   VersionAdded: '0.1.0'
   StyleGuide: ServeStaticFile.html
+
+Isucon/Sqlite3:
+  StyleGuideBaseURL: https://sue445.github.io/rubocop-isucon/RuboCop/Cop/Isucon/Sqlite3/
+  Database:
+
+Isucon/Sqlite3/JoinWithoutIndex:
+  Description: 'Check for `JOIN` without index'
+  Enabled: true
+  VersionAdded: '0.2.0'
+  StyleGuide: JoinWithoutIndex.html
+
+Isucon/Sqlite3/ManyJoinTable:
+  Description: 'Check if SQL contains many JOINs'
+  Enabled: true
+  VersionAdded: '0.2.0'
+  StyleGuide: ManyJoinTable.html
+  CountTables: 3
+
+Isucon/Sqlite3/NPlusOneQuery:
+  Description: 'Checks that there’s no N+1 query'
+  Enabled: true
+  VersionAdded: '0.2.0'
+  SafeAutoCorrect: false
+  StyleGuide: NPlusOneQuery.html
+
+Isucon/Sqlite3/SelectAsterisk:
+  Description: 'Avoid `SELECT *` in `db.execute`'
+  Enabled: true
+  VersionAdded: '0.2.0'
+  StyleGuide: SelectAsterisk.html
+
+Isucon/Sqlite3/WhereWithoutIndex:
+  Description: 'Check for `WHERE` without index'
+  Enabled: true
+  VersionAdded: '0.2.0'
+  StyleGuide: WhereWithoutIndex.html

data/gemfiles/activerecord_7_1.gemfile

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+source "https://rubygems.org"
+
+gem "activerecord", "~> 7.1.0"
+
+group :sqlite3 do
+  # c.f. https://github.com/rails/rails/blob/v7.1.0/activerecord/lib/active_record/connection_adapters/sqlite3_adapter.rb#L14
+  gem "sqlite3", "~> 1.4"
+end
+
+# eval_gemfile "#{__dir__}/common.gemfile"
+
+gemspec path: "../"

data/lib/rubocop/cop/isucon/correctors/{mysql2_n_plus_one_query_corrector → n_plus_one_query_corrector}/correctable_methods.rb

@@ -4,7 +4,7 @@ module RuboCop
   module Cop
     module Isucon
       module Correctors
-        class Mysql2NPlusOneQueryCorrector
+        class NPlusOneQueryCorrector
           # Check whether can correct
           module CorrectableMethods
             # @return [Boolean]

data/lib/rubocop/cop/isucon/correctors/{mysql2_n_plus_one_query_corrector → n_plus_one_query_corrector}/replace_methods.rb

@@ -4,7 +4,7 @@ module RuboCop
   module Cop
     module Isucon
       module Correctors
-        class Mysql2NPlusOneQueryCorrector
+        class NPlusOneQueryCorrector
           # replace ast
           module ReplaceMethods
             def replace

data/lib/rubocop/cop/isucon/correctors/{mysql2_n_plus_one_query_corrector.rb → n_plus_one_query_corrector.rb}

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative "mysql2_n_plus_one_query_corrector/correctable_methods"
-require_relative "mysql2_n_plus_one_query_corrector/replace_methods"
+require_relative "n_plus_one_query_corrector/correctable_methods"
+require_relative "n_plus_one_query_corrector/replace_methods"
 
 module RuboCop
   module Cop
@@ -9,7 +9,7 @@ module RuboCop
       module Correctors
         # rubocop:disable Layout/LineLength
 
-        # Corrector for {RuboCop::Cop::Isucon::Mysql2::NPlusOneQuery}
+        # Corrector for {RuboCop::Cop::Isucon::Mysql2::NPlusOneQuery} and {RuboCop::Cop::Isucon::Sqlite3::NPlusOneQuery}
         #
         # @example Before
         #   courses.map do |course|
@@ -21,7 +21,7 @@ module RuboCop
         #     @users_by_id ||= db.xquery('SELECT * FROM `users` WHERE `id` IN (?)', courses.map { |course| course[:teacher_id] }).each_with_object({}) { |v, hash| hash[v[:id]] = v }
         #     teacher = @users_by_id[course[:teacher_id]]
         #   end
-        class Mysql2NPlusOneQueryCorrector
+        class NPlusOneQueryCorrector
           # rubocop:enable Layout/LineLength
 
           include Mixin::Mysql2XqueryMethods
@@ -46,19 +46,24 @@ module RuboCop
           # @return [RuboCop::Isucon::DatabaseConnection]
           attr_reader :connection
 
+          # @return [Boolean]
+          attr_reader :is_array_arg
+
           # @param corrector [RuboCop::Cop::Corrector]
           # @param current_node [RuboCop::AST::Node]
           # @param parent_node [RuboCop::AST::Node]
           # @param type [Symbol] Node type. one of `:str`, `:dstr`
           # @param gda [RuboCop::Isucon::GDA::Client]
           # @param connection [RuboCop::Isucon::DatabaseConnection]
-          def initialize(corrector:, current_node:, parent_node:, type:, gda:, connection:) # rubocop:disable Metrics/ParameterLists
+          # @param is_array_arg [Boolean]
+          def initialize(corrector:, current_node:, parent_node:, type:, gda:, connection:, is_array_arg:) # rubocop:disable Metrics/ParameterLists
             @corrector = corrector
             @current_node = current_node
             @parent_node = parent_node
             @type = type
             @gda = gda
             @connection = connection
+            @is_array_arg = is_array_arg
           end
 
           def correct
@@ -67,6 +72,10 @@ module RuboCop
 
           private
 
+          def array_arg?
+            is_array_arg
+          end
+
           # @return [RuboCop::AST::Node,nil]
           def parent_receiver
             parent_node.child_nodes&.first&.receiver
@@ -88,6 +97,8 @@ module RuboCop
 
           # @return [RuboCop::AST::Node]
           def xquery_arg
+            return current_node.child_nodes[2].child_nodes[0] if array_arg? && current_node.child_nodes[2]&.array_type?
+
             current_node.child_nodes[2]
           end
 

data/lib/rubocop/cop/isucon/mixin/join_without_index_methods.rb

@@ -0,0 +1,87 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Isucon
+      module Mixin
+        # Common methods for {RuboCop::Cop::Isucon::Mysql2::JoinWithoutIndex}
+        # and {RuboCop::Cop::Isucon::Sqlite3::JoinWithoutIndex}
+        module JoinWithoutIndexMethods
+          include Mixin::DatabaseMethods
+
+          # @param node [RuboCop::AST::Node]
+          def on_send(node)
+            with_error_handling(node) do
+              return unless enabled_database?
+
+              with_db_query(node) do |type, root_gda|
+                check_and_register_offence(type: type, root_gda: root_gda, node: node)
+              end
+            end
+          end
+
+          private
+
+          # @param type [Symbol] Node type. one of `:str`, `:dstr`
+          # @param root_gda [RuboCop::Isucon::GDA::Client]
+          # @param node [RuboCop::AST::Node]
+          def check_and_register_offence(type:, root_gda:, node:)
+            return unless root_gda
+
+            root_gda.visit_all do |gda|
+              gda.join_conditions.each do |join_condition|
+                join_operand = join_operand_without_index(join_condition)
+                next unless join_operand
+
+                register_offense(type: type, node: node, join_operand: join_operand)
+              end
+            end
+          end
+
+          # @param join_condition [RuboCop::Isucon::GDA::JoinCondition]
+          # @return [RuboCop::Isucon::GDA::JoinOperand,nil]
+          def join_operand_without_index(join_condition)
+            join_condition.operands.each do |join_operand|
+              next unless join_operand.table_name
+
+              unless indexed_column?(table_name: join_operand.table_name, column_name: join_operand.column_name)
+                return join_operand
+              end
+            end
+
+            nil
+          end
+
+          # @param table_name [String]
+          # @param column_name [String]
+          # @return [Boolean]
+          def indexed_column?(table_name:, column_name:)
+            primary_keys = connection.primary_keys(table_name)
+
+            return true if primary_keys&.first == column_name
+
+            indexes = connection.indexes(table_name)
+            index_first_columns = indexes.map { |index| index.columns[0] }
+            index_first_columns.include?(column_name)
+          end
+
+          # @param type [Symbol] Node type. one of `:str`, `:dstr`
+          # @param node [RuboCop::AST::Node]
+          # @param join_operand [RuboCop::Isucon::GDA::JoinOperand]
+          def register_offense(type:, node:, join_operand:)
+            loc = offense_location(type: type, node: node, gda_location: join_operand.node.location)
+            return unless loc
+
+            message = offense_message(join_operand)
+            add_offense(loc, message: message)
+          end
+
+          # @param join_operand [RuboCop::Isucon::GDA::JoinOperand]
+          def offense_message(join_operand)
+            generate_offense_message(table_name: join_operand.table_name, column_name: join_operand.column_name)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/isucon/mixin/many_join_table_methods.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Isucon
+      module Mixin
+        # Common methods for {RuboCop::Cop::Isucon::Mysql2::ManyJoinTable}
+        # and {RuboCop::Cop::Isucon::Sqlite3::ManyJoinTable}
+        module ManyJoinTableMethods
+          MSG = "Avoid SQL with lots of JOINs"
+
+          # @param node [RuboCop::AST::Node]
+          def on_send(node)
+            with_db_query(node) do |_, root_gda|
+              check_and_register_offence(root_gda: root_gda, node: node)
+            end
+          end
+
+          private
+
+          # @param root_gda [RuboCop::Isucon::GDA::Client]
+          # @param node [RuboCop::AST::Node]
+          def check_and_register_offence(root_gda:, node:)
+            return unless root_gda
+
+            root_gda.visit_all do |gda|
+              add_offense(node) if gda.table_names.count > count_tables
+            end
+          end
+
+          # @return [Integer]
+          def count_tables
+            cop_config["CountTables"]
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/isucon/mixin/mysql2_xquery_methods.rb

@@ -8,6 +8,8 @@ module RuboCop
         module Mysql2XqueryMethods
           extend NodePattern::Macros
 
+          include OffenceLocationMethods
+
           # @!method find_xquery(node)
           #   @param node [RuboCop::AST::Node]
           def_node_search :find_xquery, <<~PATTERN
@@ -22,7 +24,7 @@ module RuboCop
           # @yieldparam root_gda [RuboCop::Isucon::GDA::Client,nil]
           #
           # @note If arguments of `db.xquery` isn't string, `root_gda` is `nil`
-          def with_xquery(node)
+          def with_db_query(node)
             find_xquery(node) do |type, params|
               sql = xquery_param(type: type, params: params)
 
@@ -36,22 +38,13 @@ module RuboCop
             end
           end
 
-          # @param type [Symbol] Node type. one of `:str`, `:dstr`
-          # @param node [RuboCop::AST::Node]
-          # @param gda_location [RuboCop::Isucon::GDA::NodeLocation]
-          # @return [Parser::Source::Range,nil]
-          def offense_location(type:, node:, gda_location:)
-            return nil unless gda_location
-
-            begin_pos = begin_position_from_gda_location(type: type, node: node, gda_location: gda_location)
-            return nil unless begin_pos
+          private
 
-            end_pos = begin_pos + gda_location.length
-            Parser::Source::Range.new(node.loc.expression.source_buffer, begin_pos, end_pos)
+          # @return [Array<Symbol>]
+          def db_query_methods
+            %i[xquery query]
           end
 
-          private
-
           # @param type [Symbol] Node type. one of `:str`, `:dstr`
           # @param params [Array<RuboCop::AST::Node>]
           # @return [String,nil]
@@ -67,108 +60,6 @@ module RuboCop
             end
             nil
           end
-
-          # @param type [Symbol] Node type. one of `:str`, `:dstr`
-          # @param node [RuboCop::AST::Node]
-          # @param gda_location [RuboCop::Isucon::GDA::NodeLocation]
-          # @return [Integer,nil]
-          def begin_position_from_gda_location(type:, node:, gda_location:)
-            case type
-            when :str
-              return begin_position_from_gda_location_for_str(node: node, gda_location: gda_location)
-            when :dstr
-              return begin_position_from_gda_location_for_dstr(node: node, gda_location: gda_location)
-            end
-
-            nil
-          end
-
-          # @param node [RuboCop::AST::Node]
-          # @param gda_location [RuboCop::Isucon::GDA::NodeLocation]
-          # @return [Integer,nil]
-          def begin_position_from_gda_location_for_str(node:, gda_location:)
-            str_node = node.child_nodes[1]
-            return nil unless str_node&.str_type?
-
-            str_node.loc.begin.end_pos + gda_location.begin_pos
-          end
-
-          # @param node [RuboCop::AST::Node]
-          # @param gda_location [RuboCop::Isucon::GDA::NodeLocation]
-          # @return [Integer,nil]
-          def begin_position_from_gda_location_for_dstr(node:, gda_location:)
-            dstr_node = node.child_nodes[1]
-            return nil unless dstr_node&.dstr_type?
-
-            str_node = find_str_node_from_gda_location(dstr_node: dstr_node, gda_location: gda_location)
-            index = str_node.value.index(gda_location.body)
-            return nil unless index
-
-            str_node_begin_pos(str_node) + index + heredoc_indent_level(node)
-          end
-
-          # @param str_node [RuboCop::AST::StrNode]
-          # @return [Integer]
-          def str_node_begin_pos(str_node)
-            begin_pos = str_node.loc.expression.begin_pos
-
-            # e.g.
-            #   db.xquery(
-            #     "SELECT * " \
-            #     "FROM users " \
-            #     "LIMIT 10"
-            #   )
-            return begin_pos + 1 if str_node.loc.expression.source_buffer.source[begin_pos] == '"'
-
-            begin_pos
-          end
-
-          # @param dstr_node [RuboCop::AST::DstrNode]
-          # @param gda_location [RuboCop::Isucon::GDA::NodeLocation]
-          # @return [RuboCop::AST::StrNode,nil]
-          def find_str_node_from_gda_location(dstr_node:, gda_location:)
-            return nil unless dstr_node
-
-            begin_pos = 0
-            dstr_node.child_nodes.each do |str_node|
-              return str_node if begin_pos <= gda_location.begin_pos && gda_location.begin_pos < begin_pos + str_node.value.length
-
-              begin_pos += str_node.value.length
-            end
-            nil
-          end
-
-          # @param node [RuboCop::AST::Node]
-          # @return [Integer]
-          def heredoc_indent_level(node)
-            dstr_node = node.child_nodes[1]
-            return 0 unless dstr_node&.dstr_type?
-
-            heredoc_indent_type = heredoc_indent_type(node)
-            return 0 unless heredoc_indent_type == "~"
-
-            heredoc_body = dstr_node.loc.heredoc_body.source
-            indent_level(heredoc_body)
-          end
-
-          # @param str [String]
-          # @return [Integer]
-          # @see https://github.com/rubocop/rubocop/blob/v1.21.0/lib/rubocop/cop/mixin/heredoc.rb#L23-L28
-          def indent_level(str)
-            indentations = str.lines.
-                           map { |line| line[/^\s*/] }.
-                           reject { |line| line.end_with?("\n") }
-            indentations.empty? ? 0 : indentations.min_by(&:size).size
-          end
-
-          # Returns '~', '-' or nil
-          #
-          # @param node [RuboCop::AST::Node]
-          # @return [String,nil] '~', '-' or `nil`
-          # @see https://github.com/rubocop/rubocop/blob/v1.21.0/lib/rubocop/cop/layout/heredoc_indentation.rb#L146-L149
-          def heredoc_indent_type(node)
-            node.source[/<<([~-])/, 1]
-          end
         end
       end
     end