docscribe 1.3.1 → 1.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 911fd61018509316b42bf5a799232d24be36cb653d5997c4e170dab5db028884
-  data.tar.gz: 4ce0e7f0cc78c6b94ce366cb464726e083a6050a42f835e35995c378072507e6
+  metadata.gz: 765dd81d49b3cbf47c90f83f9e533a367da6590f84b15ca7e49de66c6140b5e1
+  data.tar.gz: b7c0607de2fb6e2344be24d4da58c1e03ab33755175783e1e6de8c934a8f2cdc
 SHA512:
-  metadata.gz: 107a67dd5c484b840ba5ab85918ec25d53b0ce4adda13e4ef612d3bfa99d4201e82b474cd636f79a60e5e40e5cbaeb8e15c899939b5e74df14f397f4188b4168
-  data.tar.gz: 9cac2bb4b0c37b29707f0d6beec99987ab8d0f3d9327b3f44e9684816694e74fced394d88fc493ee6f5dc19ce07b82fc295e38f9c0273faad09976d125103a9e
+  metadata.gz: f30b33860317c5b9914e6b6c633df75c35f11883d7524c3b832365431a5f47760eaaeb4daab3b20f358bad8cb088015b173d6848bf65b3d7cbba18a69fda5154
+  data.tar.gz: 66ac534ba8452880ee778ae1bf72ce571e9ea4bba481e5595172314df1f809eb62db87ec8b77cd4efb8c8cd37c85dd0c937b7fa8c263938ab46734ae03f5acda

data/lib/docscribe/cli/config_builder.rb

@@ -44,7 +44,7 @@ module Docscribe
         raw['filter']['files']['include'] = Array(raw['filter']['files']['include']) + options[:include_file]
         raw['filter']['files']['exclude'] = Array(raw['filter']['files']['exclude']) + options[:exclude_file]
 
-        if options[:rbs] || options[:sig_dirs].any?
+        if options[:rbs] || options[:rbs_collection] || options[:sig_dirs].any?
           raw['rbs'] ||= {}
           raw['rbs']['enabled'] = true
           raw['rbs']['sig_dirs'] = Array(raw['rbs']['sig_dirs']) + options[:sig_dirs] if options[:sig_dirs].any?

data/lib/docscribe/cli/run.rb

@@ -148,7 +148,9 @@ module Docscribe
             fail_paths: [],
             fail_changes: {},
             error_paths: [],
-            error_messages: {}
+            error_messages: {},
+            type_mismatch_paths: [],
+            type_mismatch_changes: {}
           }
         end
 
@@ -278,9 +280,18 @@ module Docscribe
         # @param [Hash] state shared processing state
         # @return [void]
         def handle_check_result(path, src:, out:, file_changes:, display_path:, options:, state:)
-          if out == src
-            options[:verbose] ? puts("OK #{display_path}") : print('.')
-            state[:checked_ok] += 1
+          type_mismatches = file_changes.select { |c| %i[updated_param updated_return].include?(c[:type]) }
+          has_real_changes = file_changes.any? { |c| !%i[updated_param updated_return].include?(c[:type]) }
+
+          if out == src && !has_real_changes
+            if type_mismatches.any?
+              state[:type_mismatch_paths] << path
+              state[:type_mismatch_changes][path] = type_mismatches
+              options[:verbose] ? puts("MT #{display_path}") : print('M')
+            else
+              state[:checked_ok] += 1
+              options[:verbose] ? puts("OK #{display_path}") : print('.')
+            end
             return
           end
 
@@ -350,13 +361,22 @@ module Docscribe
           puts
 
           checked_error = state[:error_paths].size
+          type_mismatch_count = state[:type_mismatch_paths].size
 
-          if state[:checked_fail].zero? && checked_error.zero?
+          if state[:checked_fail].zero? && checked_error.zero? && type_mismatch_count.zero?
             puts "Docscribe: OK (#{state[:checked_ok]} files checked)"
             return
           end
 
-          puts "Docscribe: FAILED (#{state[:checked_fail]} files need updates, #{checked_error} errors, #{state[:checked_ok]} ok)"
+          if state[:checked_fail].zero? && checked_error.zero?
+            puts "Docscribe: OK (#{state[:checked_ok]} files checked, #{type_mismatch_count} with type mismatches)"
+          else
+            parts = ["#{state[:checked_fail]} need updates"]
+            parts << "#{type_mismatch_count} type mismatches" if type_mismatch_count.positive?
+            parts << "#{checked_error} errors"
+            parts << "#{state[:checked_ok]} ok"
+            puts "Docscribe: FAILED (#{parts.join(', ')})"
+          end
 
           state[:fail_paths].each do |p|
             warn "Would update docs: #{p}"
@@ -367,6 +387,15 @@ module Docscribe
             end
           end
 
+          if options[:verbose] || options[:explain]
+            state[:type_mismatch_paths].each do |p|
+              warn "Type mismatches: #{p}"
+              Array(state[:type_mismatch_changes][p]).each do |change|
+                warn "  - #{format_change_reason(change)}"
+              end
+            end
+          end
+
           state[:error_paths].each do |p|
             warn "Error processing: #{p}"
             warn "  #{state[:error_messages][p]}" if state[:error_messages][p]

data/lib/docscribe/inline_rewriter/doc_builder.rb

@@ -289,10 +289,10 @@ module Docscribe
             if !info[:param_names].include?(pname)
               lines << "#{pl}\n"
               reasons << { type: :missing_param, message: "missing @param #{pname}", extra: { param: pname } }
-            elsif info[:param_types][pname] && strategy != :safe
+            elsif external_sig && info[:param_types][pname]
               new_type = extract_param_type_from_param_line(pl)
               if new_type && info[:param_types][pname] != new_type
-                lines << "#{pl}\n"
+                lines << "#{pl}\n" unless strategy == :safe
                 reasons << {
                   type: :updated_param,
                   message: "updated @param #{pname} from #{info[:param_types][pname]} to #{new_type}",
@@ -318,8 +318,8 @@ module Docscribe
           if !info[:has_return]
             lines << "#{indent}# @return [#{normal_type}]\n"
             reasons << { type: :missing_return, message: 'missing @return' }
-          elsif info[:return_type] && info[:return_type] != normal_type && strategy != :safe
-            lines << "#{indent}# @return [#{normal_type}]\n"
+          elsif external_sig && info[:return_type] && info[:return_type] != normal_type
+            lines << "#{indent}# @return [#{normal_type}]\n" unless strategy == :safe
             reasons << {
               type: :updated_return,
               message: "updated @return from #{info[:return_type]} to #{normal_type}"

data/lib/docscribe/inline_rewriter.rb

@@ -395,24 +395,28 @@ module Docscribe
               range = Parser::Source::Range.new(buffer, info[:start_pos], info[:end_pos])
               rewriter.replace(range, new_block)
 
-              reason_specs.each do |reason|
+              if existing_order_changed
                 add_change(
                   changes,
-                  type: reason[:type],
+                  type: :unsorted_tags,
                   insertion: insertion,
                   file: file,
-                  message: reason[:message],
-                  extra: reason[:extra] || {}
+                  message: 'unsorted tags'
                 )
               end
+            end
 
-              if existing_order_changed
+            type_mismatch_reasons = reason_specs.select { |r| %i[updated_param updated_return].include?(r[:type]) }
+
+            if new_block != old_block || type_mismatch_reasons.any?
+              reason_specs.each do |reason|
                 add_change(
                   changes,
-                  type: :unsorted_tags,
+                  type: reason[:type],
                   insertion: insertion,
                   file: file,
-                  message: 'unsorted tags'
+                  message: reason[:message],
+                  extra: reason[:extra] || {}
                 )
               end
             end

data/lib/docscribe/types/rbs/provider.rb

@@ -89,10 +89,27 @@ module Docscribe
         # @param [Symbol] scope
         # @return [Object]
         def definition_for(container:, scope:)
-          type_name = ::RBS::TypeName.parse(absolute_const(container))
+          type_name = parse_type_name(absolute_const(container))
           scope == :class ? @builder.build_singleton(type_name) : @builder.build_instance(type_name)
         end
 
+        # Parse a fully-qualified constant string into an RBS TypeName.
+        #
+        # Uses the lower-level constructor so it works across RBS versions
+        # that may not expose `TypeName.parse`.
+        #
+        # @private
+        # @param [String] string e.g. "::Irb::Autosuggestions"
+        # @return [::RBS::TypeName]
+        def parse_type_name(string)
+          absolute = string.start_with?('::')
+          *path, name = string.delete_prefix('::').split('::').map(&:to_sym)
+          ::RBS::TypeName.new(
+            name: name,
+            namespace: ::RBS::Namespace.new(path: path, absolute: absolute)
+          )
+        end
+
         # Normalize a container name into an absolute constant path.
         #
         # @private

data/lib/docscribe/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Docscribe
-  VERSION = '1.3.1'
+  VERSION = '1.3.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: docscribe
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.3.2
 platform: ruby
 authors:
 - unurgunite
@@ -213,7 +213,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 4.0.12
 specification_version: 4
 summary: Auto-generate inline YARD documentation for Ruby by analyzing code AST. Supports
   RBS and Sorbet type signatures.