yard-lint 1.6.1 → 1.8.0

data/lib/yard/lint/git.rb

@@ -71,18 +71,33 @@ module Yard
         def uncommitted_files(path)
           ensure_git_repository!
 
-          # Get both staged and unstaged changes
+          # Get both staged and unstaged changes to tracked files
           stdout, stderr, status = Open3.capture3('git', 'diff', '--name-only', 'HEAD')
 
           unless status.success?
             raise Error, "Git diff failed: #{stderr.strip}"
           end
 
-          filter_ruby_files(stdout.split("\n"), path)
+          files = stdout.split("\n")
+
+          # Also include untracked (but not git-ignored) files - "all changes in
+          # the working directory" should cover newly added, not-yet-staged files.
+          others_stdout, _stderr, others_status =
+            Open3.capture3('git', 'ls-files', '--others', '--exclude-standard')
+          files += others_stdout.split("\n") if others_status.success?
+
+          filter_ruby_files(files.uniq, path)
         end
 
         private
 
+        # Absolute path to the repository root (git reports paths relative to it)
+        # @return [String] repo root, or the current directory if it can't be determined
+        def repository_root
+          stdout, _stderr, status = Open3.capture3('git', 'rev-parse', '--show-toplevel')
+          status.success? ? stdout.strip : Dir.pwd
+        end
+
         # Ensure we're in a git repository
         # @raise [Error] if not in a git repository
         def ensure_git_repository!
@@ -99,14 +114,40 @@ module Yard
         # @return [Array<String>] absolute paths to Ruby files that exist
         def filter_ruby_files(files, path)
           base_path = File.expand_path(path)
+          # git reports paths relative to the repository root, which is not
+          # necessarily the current working directory, so expand against the
+          # repo root - otherwise running from a subdirectory finds nothing.
+          root = repository_root
 
           files
+            .map { |f| unquote_git_path(f) }
             .select { |f| f.end_with?('.rb') }
-            .map { |f| File.expand_path(f) }
+            .map { |f| File.expand_path(f, root) }
             .select { |f| File.exist?(f) } # Skip deleted files
             .select { |f| file_within_path?(f, base_path) }
         end
 
+        # Unquotes a path as git emits it with the default core.quotepath=true:
+        # paths containing non-ASCII (or special) bytes are wrapped in double
+        # quotes with C-style escapes (e.g. "caf\303\251.rb"). Without this such
+        # files end with `"` rather than `.rb` and are silently dropped.
+        # @param path [String] a path from git output
+        # @return [String] the unquoted path
+        def unquote_git_path(path)
+          return path unless path.start_with?('"') && path.end_with?('"')
+
+          inner = path[1..-2]
+          unescaped = inner.gsub(/\\(?:(\d{3})|(.))/) do
+            octal = Regexp.last_match(1)
+            if octal
+              octal.to_i(8).chr
+            else
+              { 't' => "\t", 'n' => "\n", 'r' => "\r" }.fetch(Regexp.last_match(2), Regexp.last_match(2))
+            end
+          end
+          unescaped.force_encoding(Encoding::UTF_8)
+        end
+
         # Check if file is within the specified path
         # @param file [String] absolute file path
         # @param base_path [String] absolute base path

data/lib/yard/lint/path_grouper.rb

@@ -33,7 +33,10 @@ module Yard
 
           by_dir.each do |dir, dir_files|
             if should_group_directory?(dir, dir_files.uniq, limit)
-              result << "#{dir}/**/*"
+              # For root-level files File.dirname is ".", and "./**/*" matches
+              # nothing under File.fnmatch (FNM_PATHNAME), so the generated todo
+              # file would fail to exclude them. Use a plain recursive glob.
+              result << (dir == '.' ? '**/*' : "#{dir}/**/*")
             else
               result.concat(dir_files.uniq)
             end

data/lib/yard/lint/results/aggregate.rb

@@ -26,9 +26,14 @@ module Yard
         end
 
         # Get all offenses from all validators
-        # @return [Array<Hash>] flattened array of all offenses
+        # Identical offenses are reported once: objects sharing one docstring
+        # (e.g. the reader and writer generated by attr_accessor) produce the
+        # same offense for each generated method
+        # @return [Array<Hash>] flattened array of unique offenses
         def offenses
-          @results.flat_map(&:offenses)
+          @offenses ||= @results.flat_map(&:offenses).uniq do |offense|
+            offense.values_at(:validator, :name, :location, :location_line, :message)
+          end
         end
 
         # Total number of offenses
@@ -78,10 +83,11 @@ module Yard
         # @return [Integer] 0 for success, 1 for failure
         def exit_code
           # Check minimum coverage requirement first
-          if @config&.min_coverage &&
-             documentation_coverage &&
-             documentation_coverage[:coverage] < @config.min_coverage
-            return 1
+          if @config&.min_coverage
+            coverage = documentation_coverage
+            # Fail safe: if a minimum is required but coverage could not be
+            # determined (e.g. the YARD stats subprocess failed), do not pass.
+            return 1 if coverage.nil? || coverage[:coverage] < @config.min_coverage
           end
 
           return 0 if offenses.empty?
@@ -95,7 +101,9 @@ module Yard
           when SEVERITY_WARNING
             (statistics[:error] + statistics[:warning]).positive? ? 1 : 0
           when SEVERITY_CONVENTION
-            offenses.any? ? 1 : 0
+            # Exclude 'never'-severity offenses, which are meant to run without
+            # ever failing the build (they are also omitted from #statistics).
+            (statistics[:error] + statistics[:warning] + statistics[:convention]).positive? ? 1 : 0
           else
             0
           end

data/lib/yard/lint/stats_calculator.rb

@@ -20,6 +20,10 @@ module Yard
         return default_stats if files.empty?
 
         raw_stats = run_yard_stats_query
+        # nil means the YARD subprocess failed - return nil (coverage unknown)
+        # rather than default_stats, which would falsely report 100% coverage
+        # and let a MinCoverage gate pass.
+        return nil if raw_stats.nil?
         return default_stats if raw_stats.empty?
 
         parsed_stats = parse_stats_output(raw_stats)
@@ -52,8 +56,10 @@ module Yard
 
             stdout, _stderr, status = Open3.capture3(cmd)
 
-            # Return empty string if YARD command fails
-            return '' unless status.exitstatus.zero?
+            # Return nil if the YARD command fails (non-zero exit, or killed by
+            # a signal so exitstatus is nil) - distinct from genuinely empty
+            # output, so coverage is reported as unknown rather than 100%.
+            return nil unless status.exitstatus&.zero?
 
             stdout
           end

data/lib/yard/lint/templates/default_config.yml

@@ -44,6 +44,12 @@ Documentation/UndocumentedMethodArguments:
   Description: 'Checks for method parameters without @param tags.'
   Enabled: true
   Severity: warning
+  # Match each parameter to a @param tag by name (catches a misnamed @param).
+  # Set to false to fall back to a lenient count-only comparison.
+  CheckParameterNames: true
+  # Opt-in: skip methods with no documentation at all (reported by
+  # Documentation/UndocumentedObjects) instead of flagging them here too.
+  SkipFullyUndocumented: false
   # AllowedParentClasses:
   #   - StandardError
   # AllowedMethods: skip @param checks for specific methods (exact name, name/arity, /regex/)
@@ -108,6 +114,21 @@ Documentation/LineLength:
   Severity: convention
   MaxLength: 120
 
+Documentation/UnderfilledLines:
+  Description: 'Detects documentation prose that wraps before using the available line width.'
+  Enabled: false  # Opt-in validator (the inverse of LineLength)
+  Severity: convention
+  MaxLength: 120
+  # Only flag when the widest non-final line wastes at least this many columns.
+  MinTrailingSpace: 20
+  # Single-line descriptions are never flagged.
+  MinParagraphLines: 2
+  # A non-final line ending in one of these is a deliberate break (skipped).
+  # Add ',' to also respect comma breaks (suppresses more, catches less).
+  SentenceEndChars: ['.', '?', '!', ':', ';']
+  # Skip paragraphs with non-ASCII text (String#length is not display width).
+  SkipNonAscii: true
+
 Documentation/TextSubstitution:
   Description: 'Detects forbidden characters or strings in documentation and suggests replacements.'
   Enabled: false  # Opt-in validator
@@ -142,6 +163,13 @@ Tags/InvalidTypes:
     - param
     - option
     - return
+    - yieldreturn
+    - yieldparam
+    - raise
+  # Opt-in: flag CamelCase type names that are neither loaded Ruby constants
+  # nor defined in the analyzed codebase (catches typos like `Strng`). Off by
+  # default because types from un-analyzed dependencies would also be flagged.
+  StrictConstantNames: false
 
 Tags/TypeSyntax:
   Description: 'Validates YARD type syntax using YARD parser.'
@@ -152,6 +180,8 @@ Tags/TypeSyntax:
     - option
     - return
     - yieldreturn
+    - yieldparam
+    - raise
 
 Tags/MeaninglessTag:
   Description: 'Detects @param/@option tags on classes, modules, or constants.'
@@ -180,6 +210,8 @@ Tags/CollectionType:
     - option
     - return
     - yieldreturn
+    - yieldparam
+    - raise
 
 Tags/TagTypePosition:
   Description: 'Validates type annotation position in tags.'
@@ -210,6 +242,10 @@ Tags/ExampleSyntax:
   Description: 'Validates Ruby syntax in @example tags.'
   Enabled: true
   Severity: warning
+  # Opt-in: skip @example blocks that are interactive console transcripts
+  # (irb/pry sessions, their `=>` output, or shell `$` prompts) rather than
+  # runnable Ruby. Off by default so a real syntax error is not hidden.
+  SkipNonRuby: false
 
 Tags/ExampleStyle:
   Description: 'Validates code style in @example tags using RuboCop/StandardRB.'
@@ -258,14 +294,20 @@ Tags/InformalNotation:
   Severity: warning
   CaseSensitive: false
   RequireStartOfLine: true
+  # Opt-in: also skip 4-space/tab indented Markdown code blocks (not just
+  # fenced ``` blocks). Off by default - indented text is also used for list
+  # continuations and wrapped prose, which would then be skipped too.
+  SkipIndentedCodeBlocks: false
   Patterns:
     Note: '@note'
+    IMPORTANT: '@note'
+    Important: '@note'
     Todo: '@todo'
     TODO: '@todo'
     FIXME: '@todo'
     See: '@see'
     See also: '@see'
-    Warning: '@deprecated'
+    Warning: '@note'
     Deprecated: '@deprecated'
     Author: '@author'
     Version: '@version'
@@ -284,6 +326,7 @@ Tags/NonAsciiType:
     - return
     - yieldreturn
     - yieldparam
+    - raise
 
 Tags/TagGroupSeparator:
   Description: 'Enforces blank line separators between different YARD tag groups.'
@@ -313,6 +356,11 @@ Tags/ForbiddenTags:
   # - Tag: api  # Forbids @api tag entirely (no Types = any occurrence)
 
 # Warnings validators - catches YARD parser errors
+Warnings/SyntaxError:
+  Description: 'Detects Ruby files YARD cannot parse (syntax errors).'
+  Enabled: true
+  Severity: error
+
 Warnings/UnknownTag:
   Description: 'Detects unknown YARD tags.'
   Enabled: true

data/lib/yard/lint/templates/strict_config.yml

@@ -48,6 +48,11 @@ Documentation/UndocumentedMethodArguments:
   Description: 'Checks for method parameters without @param tags.'
   Enabled: true
   Severity: error
+  # Match each parameter to a @param tag by name (catches a misnamed @param).
+  CheckParameterNames: true
+  # Opt-in: skip methods with no documentation at all (reported by
+  # Documentation/UndocumentedObjects) instead of flagging them here too.
+  SkipFullyUndocumented: false
   # AllowedParentClasses:
   #   - StandardError
   # AllowedMethods: skip @param checks for specific methods (exact name, name/arity, /regex/)
@@ -112,6 +117,16 @@ Documentation/LineLength:
   Severity: error
   MaxLength: 120
 
+Documentation/UnderfilledLines:
+  Description: 'Detects documentation prose that wraps before using the available line width.'
+  Enabled: false  # Opt-in validator (heuristic; stays convention even in strict)
+  Severity: convention
+  MaxLength: 120
+  MinTrailingSpace: 20
+  MinParagraphLines: 2
+  SentenceEndChars: ['.', '?', '!', ':', ';']
+  SkipNonAscii: true
+
 Documentation/TextSubstitution:
   Description: 'Detects forbidden characters or strings in documentation and suggests replacements.'
   Enabled: true
@@ -146,6 +161,13 @@ Tags/InvalidTypes:
     - param
     - option
     - return
+    - yieldreturn
+    - yieldparam
+    - raise
+  # The strict preset opts in: flag CamelCase type names that are neither loaded
+  # Ruby constants nor defined in the analyzed codebase (catches typos). Add any
+  # legitimate dependency types to ExtraTypes to silence false positives.
+  StrictConstantNames: true
 
 Tags/TypeSyntax:
   Description: 'Validates YARD type syntax using YARD parser.'
@@ -156,6 +178,8 @@ Tags/TypeSyntax:
     - option
     - return
     - yieldreturn
+    - yieldparam
+    - raise
 
 Tags/MeaninglessTag:
   Description: 'Detects @param/@option tags on classes, modules, or constants.'
@@ -184,6 +208,8 @@ Tags/CollectionType:
     - option
     - return
     - yieldreturn
+    - yieldparam
+    - raise
 
 Tags/TagTypePosition:
   Description: 'Validates type annotation position in tags.'
@@ -214,6 +240,10 @@ Tags/ExampleSyntax:
   Description: 'Validates Ruby syntax in @example tags.'
   Enabled: true
   Severity: error
+  # Opt-in: skip @example blocks that are interactive console transcripts
+  # (irb/pry sessions, their `=>` output, or shell `$` prompts) rather than
+  # runnable Ruby. Off by default so a real syntax error is not hidden.
+  SkipNonRuby: false
 
 Tags/ExampleStyle:
   Description: 'Validates code style in @example tags using RuboCop/StandardRB.'
@@ -254,6 +284,7 @@ Tags/RedundantParamDescription:
     IdPattern: true
     DirectionalDate: true
     TypeGeneric: true
+    ArticleParamPhrase: true
 
 Tags/InformalNotation:
   Description: 'Detects informal tag notation patterns like "Note:" instead of @note.'
@@ -261,14 +292,20 @@ Tags/InformalNotation:
   Severity: error
   CaseSensitive: false
   RequireStartOfLine: true
+  # Opt-in: also skip 4-space/tab indented Markdown code blocks (not just
+  # fenced ``` blocks). Off by default - indented text is also used for list
+  # continuations and wrapped prose, which would then be skipped too.
+  SkipIndentedCodeBlocks: false
   Patterns:
     Note: '@note'
+    IMPORTANT: '@note'
+    Important: '@note'
     Todo: '@todo'
     TODO: '@todo'
     FIXME: '@todo'
     See: '@see'
     See also: '@see'
-    Warning: '@deprecated'
+    Warning: '@note'
     Deprecated: '@deprecated'
     Author: '@author'
     Version: '@version'
@@ -287,6 +324,7 @@ Tags/NonAsciiType:
     - return
     - yieldreturn
     - yieldparam
+    - raise
 
 Tags/TagGroupSeparator:
   Description: 'Enforces blank line separators between different YARD tag groups.'
@@ -316,6 +354,11 @@ Tags/ForbiddenTags:
   # - Tag: api  # Forbids @api tag entirely (no Types = any occurrence)
 
 # Warnings validators - catches YARD parser errors
+Warnings/SyntaxError:
+  Description: 'Detects Ruby files YARD cannot parse (syntax errors).'
+  Enabled: true
+  Severity: error
+
 Warnings/UnknownTag:
   Description: 'Detects unknown YARD tags.'
   Enabled: true

data/lib/yard/lint/todo_generator.rb

@@ -13,9 +13,12 @@ module Yard
         # @param config [Config] yard-lint configuration object with validator settings
         # @param force [Boolean] whether to overwrite existing todo file if present
         # @param exclude_limit [Integer] minimum files in directory before grouping into wildcard patterns
+        # @param config_path [String, nil] config file to link the todo into
+        #   (defaults to .yard-lint.yml in the current directory)
         # @return [Hash] result with :message, :offense_count, :validator_count
-        def generate(path:, config:, force: false, exclude_limit: DEFAULT_EXCLUDE_LIMIT)
-          new(path: path, config: config, force: force, exclude_limit: exclude_limit).generate
+        def generate(path:, config:, force: false, exclude_limit: DEFAULT_EXCLUDE_LIMIT, config_path: nil)
+          new(path: path, config: config, force: force, exclude_limit: exclude_limit,
+              config_path: config_path).generate
         end
       end
 
@@ -24,13 +27,14 @@ module Yard
       # @param config [Config] yard-lint configuration object
       # @param force [Boolean] whether to overwrite existing todo file
       # @param exclude_limit [Integer] minimum files before grouping into patterns
-      def initialize(path:, config:, force:, exclude_limit:)
+      # @param config_path [String, nil] config file to link the todo into
+      def initialize(path:, config:, force:, exclude_limit:, config_path: nil)
         @path = path
         @config = config
         @force = force
         @exclude_limit = exclude_limit
         @todo_path = File.join(Dir.pwd, '.yard-lint-todo.yml')
-        @config_path = File.join(Dir.pwd, Config::DEFAULT_CONFIG_FILE)
+        @config_path = config_path || File.join(Dir.pwd, Config::DEFAULT_CONFIG_FILE)
       end
 
       # Generate the .yard-lint-todo.yml file with exclusions for current violations
@@ -93,6 +97,12 @@ module Yard
           next unless @config.validator_enabled?(validator_name)
 
           validator_result = result_builder.build(validator_name, raw_results)
+          next unless validator_result
+
+          # Apply per-validator Exclude patterns (and drop no-location offenses)
+          # exactly as a normal run does, so the baseline does not silence files
+          # that are already excluded.
+          validator_result = runner.send(:filter_result_offenses, validator_name, validator_result)
           next unless validator_result && validator_result.offenses.any?
 
           # Extract file paths from offenses, skipping those without a location
@@ -206,17 +216,28 @@ module Yard
       # Update existing config file to add inherit_from todo file
       # @return [void]
       def update_existing_config
-        config_yaml = YAML.load_file(@config_path) || {}
-        inherit_from = Array(config_yaml['inherit_from'] || [])
-
-        # Add todo file to inherit_from if not already present
-        unless inherit_from.include?('.yard-lint-todo.yml')
-          inherit_from.unshift('.yard-lint-todo.yml')
-          config_yaml['inherit_from'] = inherit_from
+        config_yaml = ConfigLoader.load_yaml_file(@config_path)
+        return if Array(config_yaml['inherit_from']).include?('.yard-lint-todo.yml')
+
+        raw = File.read(@config_path)
+
+        # Edit the file textually rather than round-tripping through to_yaml,
+        # which would strip every comment and reformat the user's config.
+        updated =
+          if config_yaml.key?('inherit_from') && raw.match?(/^inherit_from:[ \t]*$/)
+            # Existing block list - prepend our entry as the first item.
+            raw.sub(/^inherit_from:[ \t]*$\n/) { "#{Regexp.last_match(0)}  - .yard-lint-todo.yml\n" }
+          elsif config_yaml.key?('inherit_from')
+            # Existing inline/array form (rare) - fall back to a structured
+            # rewrite; comments cannot be preserved in this case.
+            config_yaml['inherit_from'] = ['.yard-lint-todo.yml'] + Array(config_yaml['inherit_from'])
+            config_yaml.to_yaml
+          else
+            # No inherit_from yet - prepend a block, preserving the file verbatim.
+            "inherit_from:\n  - .yard-lint-todo.yml\n\n#{raw}"
+          end
 
-          # Write updated config
-          File.write(@config_path, config_yaml.to_yaml)
-        end
+        File.write(@config_path, updated)
       end
 
       # Create a minimal config file that inherits from todo file

data/lib/yard/lint/validators/base.rb

@@ -97,6 +97,52 @@ module Yard
           tags
         end
 
+        # Tracks docstring locations already processed by this validator
+        # instance. Objects generated from a single comment block (e.g. the
+        # reader and writer created by attr_accessor) share one docstring;
+        # content-scanning validators use this to report each docstring once.
+        # @param object [YARD::CodeObjects::Base] the code object to check
+        # @return [Boolean] true if this object's docstring was already seen
+        def duplicate_docstring?(object)
+          @scanned_docstrings ||= Set.new
+          key = [object.file, object.docstring.line_range&.first || object.line]
+
+          !@scanned_docstrings.add?(key)
+        end
+
+        # Converts a zero-based line offset within a docstring's text into an
+        # absolute line number in the source file, so offenses can point at
+        # the offending documentation line instead of the definition line.
+        # @param object [YARD::CodeObjects::Base] the documented object
+        # @param line_offset [Integer] zero-based offset within the docstring text
+        # @return [Integer] absolute source line number
+        def docstring_line(object, line_offset)
+          start_line = object.docstring.line_range&.first || object.line
+
+          start_line + line_offset
+        end
+
+        # Returns the lines of a source file, reading from disk only on the first
+        # call for each unique path. Invalid bytes are scrubbed so that callers
+        # matching regexes against the lines never raise Encoding::CompatibilityError
+        # on a non-UTF-8 source file.
+        # @param file [String] absolute path to the source file
+        # @return [Array<String>] lines of the file, memoized per path
+        def cached_lines(file)
+          @file_cache ||= {}
+          @file_cache[file] ||= File.readlines(file).map!(&:scrub)
+        end
+
+        # Returns the tag that actually carries a tag's types and description.
+        # For most tags that is the tag itself, but @option tags wrap their
+        # data in a nested pair tag - tag.types and tag.text are nil on the
+        # OptionTag itself, with the documented option living on tag.pair.
+        # @param tag [YARD::Tags::Tag] tag whose data holder should be resolved
+        # @return [YARD::Tags::Tag] the tag holding types/text data
+        def tag_data(tag)
+          tag.respond_to?(:pair) && tag.pair ? tag.pair : tag
+        end
+
         # Checks whether the object's enclosing class (or the object itself if it is
         # a class) has a superclass that appears in the validator's AllowedParentClasses
         # configuration list. When true, validators skip the object so that classes
@@ -236,7 +282,10 @@ module Yard
 
           return defaults[key] unless validator_name
 
-          config.validator_config(validator_name, key) || defaults[key]
+          value = config.validator_config(validator_name, key)
+          # A nil? check (not ||) so that explicitly configured false values
+          # are honored instead of falling back to a truthy default
+          value.nil? ? defaults[key] : value
         end
       end
     end

data/lib/yard/lint/validators/documentation/blank_line_before_definition/validator.rb

@@ -50,8 +50,11 @@ module Yard
                 if stripped.empty?
                   blank_count += 1
                 elsif stripped.start_with?('#')
-                  # Skip Ruby magic comments - they're not YARD documentation
-                  next if magic_comment?(stripped)
+                  # Skip lines that are not YARD documentation: magic comments,
+                  # shebangs, tool sigils/directives (Sorbet, RuboCop, Standard),
+                  # and bare `#` separators. Treating them as a doc block caused
+                  # spurious blank-line offenses for undocumented definitions.
+                  next if non_documentation_comment?(stripped)
 
                   has_doc_block = true
                   break
@@ -72,6 +75,18 @@ module Yard
               line.match?(/^#\s*(frozen[_-]string[_-]literal|encoding|warn[_-]indent|shareable[_-]constant[_-]value)\s*:/i)
             end
 
+            # Check if a comment line is not YARD documentation (magic comment,
+            # shebang, tool sigil/directive, or a bare `#` separator).
+            # @param line [String] stripped comment line
+            # @return [Boolean] true if the line should not count as documentation
+            def non_documentation_comment?(line)
+              magic_comment?(line) ||
+                line.start_with?('#!') ||                          # shebang
+                line.match?(/\A#\s*(rubocop|standard):/i) ||       # linter directives
+                line.match?(/\A#\s*typed:/i) ||                    # Sorbet sigil
+                line.match?(/\A#+\s*\z/)                           # bare # separator
+            end
+
             # Check if the given pattern is enabled in configuration
             # @param violation_type [String] 'single' or 'orphaned'
             # @return [Boolean] whether the pattern is enabled

data/lib/yard/lint/validators/documentation/empty_comment_line/validator.rb

@@ -33,8 +33,11 @@ module Yard
                 stripped = line.strip
 
                 if stripped.empty? && comment_end.nil?
-                  # Skip empty lines before finding comment block
-                  next
+                  # A blank line between the definition and any comment above
+                  # means there is no attached docstring (YARD only attaches a
+                  # comment immediately above the definition), so stop scanning -
+                  # otherwise a detached file header is mistaken for the doc.
+                  break
                 elsif stripped.start_with?('#')
                   comment_end ||= i
                   comment_start = i

data/lib/yard/lint/validators/documentation/line_length/validator.rb

@@ -23,6 +23,7 @@ module Yard
 
               line_range = object.docstring.line_range
               return unless line_range
+              return if duplicate_docstring?(object)
 
               max_length = config_or_default('MaxLength').to_i
               source_lines = cached_lines(object.file)
@@ -40,17 +41,6 @@ module Yard
               collector.puts "#{object.file}:#{object.line}: #{object.title}"
               collector.puts "#{max_length}|#{violations.join('|')}"
             end
-
-            private
-
-            # Returns the lines of a source file, reading from disk only on the first call
-            # for each unique path.
-            # @param file [String] absolute path to the source file
-            # @return [Array<String>] lines of the file, memoized per path
-            def cached_lines(file)
-              @file_cache ||= {}
-              @file_cache[file] ||= File.readlines(file)
-            end
           end
         end
       end