checkoff 0.256.0 → 0.257.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a6f30a4354aedf2688699c3c460f2495445ce9cc647e0da109586f9be561caf4
-  data.tar.gz: 159ce5f7020c4ed9d0c3bd2f6841e3da1c5e5fc3009a2efefc04af8ccc3b97b1
+  metadata.gz: 1896715fc41d51e76c6c1381eed1a4b3cd72ee304a75b36d1a26bfe42d0ee158
+  data.tar.gz: 07f5cc66320360ebc668d21adc062585fb6927b029872b99f0416f30b7986387
 SHA512:
-  metadata.gz: 473fa677a861c4d4d8d2d6b9c93feeaa4d3a3cd6d029298a3a8ac98bf1ec443316c08860e20f64406aa3775642cf8b08115163513b8b634fe6ddf9dddb8ae78e
-  data.tar.gz: c50cd2149b2465e59e6a84b7ab53ca5027c8fb24aceb1e01731a39dfa52e5e52964f8ee2a38bb3b69ded267a9f989c7a9cc806bbcd2b24ad2b75be70c6d25ad6
+  metadata.gz: 161d6849b466f214b62ee2c04d2188bd672323560ff37fd99709166efb920aea03aec8f5255835b2b938ee6c0036c847e40d57a509ccaeded3f03b5bda98d603
+  data.tar.gz: 16acaccaea9884fbfe513ac8dfe6f9bb30f72342deba8d88a335a3f391d8fb18fd61d8034731346b5e6015ef9c0fb125e3153599dfe2eeb76dfd4ed6ff0ccfe3

data/lib/checkoff/attachments.rb

@@ -54,7 +54,7 @@ module Checkoff
     end
 
     # @param url [String]
-    # @param resource [Asana::Resources::Resource]
+    # @param resource [Asana::Resources::Task]
     # @param attachment_name [String,nil]
     # @param just_the_url [Boolean]
     # @param verify_mode [Integer] - e.g., OpenSSL::SSL::VERIFY_NONE or OpenSSL::SSL::VERIFY_PEER
@@ -79,15 +79,15 @@ module Checkoff
     # extension as the URL using Net::HTTP, raising an exception if
     # not succesful
     #
-    # @param uri [URI]
+    # @param uri [URI::Generic]
     # @param verify_mode [Integer] - e.g., OpenSSL::SSL::VERIFY_NONE,OpenSSL::SSL::VERIFY_PEER
     #
     # @return [Object]
     def download_uri(uri, verify_mode: OpenSSL::SSL::VERIFY_PEER, &block)
       out = nil
-      # @sg-ignore
       Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https', verify_mode:) do |http|
-        # @sg-ignore
+        # @sg-ignore Unresolved constant Net::HTTP::Get / Unresolved call to request —
+        #   stdlib RBS gap on Net::HTTP block param types
         http.request(Net::HTTP::Get.new(uri)) do |response|
           raise("Unexpected response code: #{response.code}") unless response.code == '200'
 
@@ -120,7 +120,7 @@ module Checkoff
     end
 
     # @param url [String]
-    # @param resource [Asana::Resources::Resource]
+    # @param resource [Asana::Resources::Task]
     # @param attachment_name [String,nil]
     # @param verify_mode [Integer] - e.g., OpenSSL::SSL::VERIFY_NONE,OpenSSL::SSL::VERIFY_PEER
     #
@@ -130,25 +130,25 @@ module Checkoff
       uri = URI(url)
       attachment_name ||= File.basename(uri.path)
       download_uri(uri, verify_mode:) do |tempfile|
-        # @sg-ignore
+        # @sg-ignore attachment_name reassignment above isn't narrowed from the declared
+        #   [String, nil] param type — needs a fresh typed local, deferred to a follow-up code PR
         content_type ||= content_type_from_filename(attachment_name)
-        # @sg-ignore
+        # @sg-ignore URI::Generic#path can be nil
         content_type ||= content_type_from_filename(uri.path)
 
-        # @sg-ignore
+        # @sg-ignore same attachment_name/content_type narrowing gap as above
         resource.attach(filename: attachment_name, mime: content_type,
                         io: tempfile)
       end
     end
 
     # @param url [String]
-    # @param resource [Asana::Resources::Resource]
+    # @param resource [Asana::Resources::Task]
     # @param attachment_name [String,nil]
     #
     # @return [Asana::Resources::Attachment]
     def create_attachment_from_url_alone!(url, resource, attachment_name:)
       with_params = {
-        # @sg-ignore
         'parent' => resource.gid,
         'url' => url,
         'resource_subtype' => 'external',
@@ -196,7 +196,8 @@ module Checkoff
         tasks = Checkoff::Tasks.new
         attachments = Checkoff::Attachments.new
         task = tasks.task_by_gid(gid)
-        # @sg-ignore
+        # @sg-ignore task_by_gid can return nil; needs a raise-if-nil guard, deferred to a
+        #   follow-up code PR
         attachment = attachments.create_attachment_from_url!(url, task)
         puts "Results: #{attachment.inspect}"
       end

data/lib/checkoff/cli.rb

@@ -29,7 +29,6 @@ module Checkoff
       end
 
       @from_workspace_name = from_workspace_arg
-      # @sg-ignore
       @from_project_name = project_arg_to_name(from_project_arg)
       @from_section_name = from_section_arg
     end
@@ -41,7 +40,6 @@ module Checkoff
       if to_project_arg == :source_project
         from_project_name
       else
-        # @sg-ignore
         project_arg_to_name(to_project_arg)
       end
     end
@@ -146,7 +144,7 @@ module Checkoff
                 :to_workspace_name, :to_project_name, :to_section_name,
                 :projects, :sections
 
-    # @param project_arg [String]
+    # @param project_arg [Symbol, String]
     # @return [Symbol, String]
     def project_arg_to_name(project_arg)
       arg = project_arg.to_s
@@ -218,7 +216,6 @@ module Checkoff
     # @return [String]
     def run_on_project(workspace, project)
       tasks_by_section =
-        # @sg-ignore
         sections.tasks_by_section(workspace, project)
       tasks_by_section.update(tasks_by_section) do |_key, tasks|
         tasks_to_hash(tasks)

data/lib/checkoff/clients.rb

@@ -30,9 +30,9 @@ module Checkoff
     # @return [Asana::Client]
     def client
       @client ||= @asana_client_class.new do |c|
-        # @sg-ignore
+        # @sg-ignore Unresolved call to authentication — gem config DSL block, not covered by annotations
         c.authentication :access_token, @config.fetch(:personal_access_token)
-        # @sg-ignore
+        # @sg-ignore Unresolved call to default_headers — gem config DSL block, not covered by annotations
         c.default_headers 'asana-enable' =>
                           'new_project_templates,new_user_task_lists,new_memberships,new_goal_memberships'
       end

data/lib/checkoff/custom_fields.rb

@@ -41,7 +41,7 @@ module Checkoff
     # @param custom_field_name [String]
     #
     # @return [Asana::Resources::CustomField]
-    # @sg-ignore
+    # @sg-ignore nil check above is not flow-sensitive
     def custom_field_or_raise(workspace_name, custom_field_name)
       cf = custom_field(workspace_name, custom_field_name)
       raise "Could not find custom_field #{custom_field_name} under workspace #{workspace_name}." if cf.nil?
@@ -78,8 +78,10 @@ module Checkoff
 
     # @param resource [Asana::Resources::Project,Asana::Resources::Task]
     # @param custom_field_name [String]
-    # @sg-ignore
     # @return [Array<String>]
+    # @sg-ignore Declared return type Array<String> does not match inferred type Array,Array<Array> —
+    #   fixing cleanly needs binding enum_value.fetch('name') to a typed local; leaving for a
+    #   follow-up code PR to keep this one annotation-only.
     def resource_custom_field_values_names_by_name(resource, custom_field_name)
       custom_field = resource_custom_field_by_name(resource, custom_field_name)
       return [] if custom_field.nil?
@@ -108,9 +110,9 @@ module Checkoff
     end
 
     # @param resource [Asana::Resources::Task,Asana::Resources::Project]
-    # @sg-ignore
     # @param custom_field_name [String]
     # @return [Hash]
+    # @sg-ignore nil check above is not flow-sensitive
     def resource_custom_field_by_name_or_raise(resource, custom_field_name)
       custom_field = resource_custom_field_by_name(resource, custom_field_name)
       if custom_field.nil?
@@ -120,10 +122,10 @@ module Checkoff
       custom_field
     end
 
-    # @sg-ignore
     # @param resource [Asana::Resources::Project,Asana::Resources::Task]
     # @param custom_field_gid [String]
     # @return [Hash]
+    # @sg-ignore nil check above is not flow-sensitive
     def resource_custom_field_by_gid_or_raise(resource, custom_field_gid)
       # @type [Array<Hash>]
       custom_fields = resource.custom_fields
@@ -146,7 +148,9 @@ module Checkoff
     # @param custom_field [Hash{String => Hash,Array<Hash>}]
     #
     # @return [Array<Hash>]
-    # @sg-ignore
+    # @sg-ignore Declared return type Array<Hash> does not match inferred type Array,Hash,Array<Hash> —
+    #   fixing cleanly needs restructuring the case/when to bind each branch to a typed local; leaving
+    #   for a follow-up code PR to keep this one annotation-only.
     def resource_custom_field_enum_values(custom_field)
       resource_subtype = custom_field.fetch('resource_subtype')
       case resource_subtype

data/lib/checkoff/internal/asana_event_enrichment.rb

@@ -74,9 +74,11 @@ module Checkoff
         end
         resource = webhook_subscription&.fetch('resource', nil)
         name, resource_type = enrich_gid(resource) if resource
-        # @sg-ignore
+        # @sg-ignore webhook_subscription is declared [Hash, nil] but this line assumes non-nil
+        #   without the &. used elsewhere in this method — looks like a real nil-safety gap, not
+        #   just a typing gap; flagging for the follow-up code PR rather than silently widening
         webhook_subscription['checkoff:enriched:name'] = name if name
-        # @sg-ignore
+        # @sg-ignore same nil-safety gap as above
         webhook_subscription['checkoff:enriched:resource_type'] = resource_type if resource_type
       end
 
@@ -96,13 +98,11 @@ module Checkoff
 
       # @param filter [Hash{String => String, Array<String>}]
       #
-      # @sg-ignore
-      # @return [String, nil]
+      # @return [void]
       def enrich_filter_parent_gid!(filter)
-        parent_gid = filter['checkoff:parent.gid']
+        parent_gid = T.cast(filter['checkoff:parent.gid'], T.nilable(String))
         return unless parent_gid
 
-        # @sg-ignore
         name, resource_type = enrich_gid(parent_gid)
         filter['checkoff:enriched:parent.name'] = name if name
         filter['checkoff:enriched:parent.resource_type'] = resource_type if resource_type
@@ -112,11 +112,10 @@ module Checkoff
       #
       # @return [void]
       def enrich_filter_resource!(filter)
-        resource_gid = filter['checkoff:resource.gid']
+        resource_gid = T.cast(filter['checkoff:resource.gid'], T.nilable(String))
 
         return unless resource_gid
 
-        # @sg-ignore
         task = tasks.task_by_gid(resource_gid)
         task_name = task&.name
         filter['checkoff:enriched:resource.name'] = task_name if task_name
@@ -134,12 +133,11 @@ module Checkoff
         filter['checkoff:enriched:fetched.section.name'] = name if name
       end
 
-      # @param asana_event [Hash{'resource' => Hash}]
+      # @param asana_event [Hash]
       #
       # @return [void]
       def enrich_event_parent!(asana_event)
         # @type [Hash{String => String }]
-        # @sg-ignore
         parent = asana_event['parent']
 
         return unless parent
@@ -154,7 +152,7 @@ module Checkoff
         nil
       end
 
-      # @param asana_event [Hash{'resource' => Hash}]
+      # @param asana_event [Hash]
       #
       # @return [void]
       def enrich_event_resource!(asana_event)

data/lib/checkoff/internal/config_loader.rb

@@ -20,12 +20,12 @@ module Checkoff
 
       # @param key [Symbol]
       # @return [Object]
-      # @sg-ignore
+      # @sg-ignore return type could not be inferred — cascades from the ENV.fetch RBS gap below
       def [](key)
         config_value = @config[key]
         return config_value unless config_value.nil?
 
-        # @sg-ignore
+        # @sg-ignore Unresolved call to fetch on RBS::Unnamed::ENVClass, Class<ENV>
         ENV.fetch(envvar_name(key), nil)
       end
 

data/lib/checkoff/internal/logging.rb

@@ -57,10 +57,10 @@ module Logging
   private
 
   # @return [Symbol]
-  # @sg-ignore
+  # @sg-ignore return type could not be inferred — cascades from the ENV.fetch RBS gap below
   def log_level
-    # @sg-ignore
     # rubocop:disable Style/RedundantFetchBlock
+    # @sg-ignore Unresolved call to fetch on RBS::Unnamed::ENVClass, Class<ENV>
     ENV.fetch('LOG_LEVEL') { 'INFO' }.downcase.to_sym
     # rubocop:enable Style/RedundantFetchBlock
   end

data/lib/checkoff/internal/search_url/parser.rb

@@ -24,9 +24,10 @@ module Checkoff
         # @param url [String]
         # @return [Array(Hash{String => String}, Array)]
         def convert_params(url)
-          # @sg-ignore
+          # @sg-ignore URI#query can be nil for a URL with no query string; CGI.parse expects
+          #   String — looks like a real gap (unhandled no-query-string case), not just a typing
+          #   gap, flagging for the follow-up code PR rather than silently asserting non-nil
           url_params = CGI.parse(URI.parse(url).query)
-          # @sg-ignore
           custom_field_params, date_url_params, simple_url_params = partition_url_params(url_params)
           custom_field_args, custom_field_task_selector = convert_custom_field_params(custom_field_params)
           date_url_args, date_task_selector = convert_date_params(date_url_params)
@@ -56,14 +57,15 @@ module Checkoff
           CustomFieldParamConverter.new(custom_field_params:).convert
         end
 
-        # @param url_params [Hash{String => String}]
+        # @param url_params [Hash{String => Array<String>}]
         # @return [Array(Hash{String => Array<String>}, Hash{String => Array<String>}, Hash{String => Array<String>})]
         def partition_url_params(url_params)
           groups = T.let(url_params.to_a.group_by do |key, _values|
-                           # @sg-ignore
+                           # @sg-ignore Unresolved call to start_with? — T.let wrapping this
+                           #   chained group_by block seems to block block-param inference
                            if key.start_with? 'custom_field_'
                              :custom_field
-                           # @sg-ignore
+                           # @sg-ignore Unresolved call to include? — same T.let/block-param gap
                            elsif key.include? '_date'
                              :date
                            else

data/lib/checkoff/internal/search_url/results_merger.rb

@@ -17,8 +17,7 @@ module Checkoff
         end
 
         # @param task_selectors [Array<Symbol, Array>]
-        # @return [Array(Symbol, Array, Array)]
-        # @sg-ignore
+        # @return [Symbol, Array, Array(Symbol, Array, Array)]
         def self.merge_task_selectors(*task_selectors)
           return [] if task_selectors.empty?
 

data/lib/checkoff/internal/selector_classes/common.rb

@@ -238,9 +238,10 @@ module Checkoff
         end
 
         # @param resource [Asana::Resources::Task, Asana::Resources::Project]
-        # @sg-ignore
         # @param prefix [String]
         # @return [boolish]
+        # @sg-ignore Checkoff::SelectorClasses::Common::NameStartsWithPFunctionEvaluator#evaluate
+        #   return type could not be inferred
         def evaluate(resource, prefix)
           resource.name&.start_with?(prefix)
         end

data/lib/checkoff/internal/selector_classes/function_evaluator.rb

@@ -15,15 +15,15 @@ module Checkoff
       end
 
       # @return [Boolean]
-      # @sg-ignore
+      # @sg-ignore return type could not be inferred — abstract method body is only `raise`
       def matches?
         raise 'Override me!'
       end
 
       # @param _task [Asana::Resources::Task]
       # @param _args [Array<Object>]
-      # @sg-ignore
       # @return [Object]
+      # @sg-ignore return type could not be inferred — abstract method body is only `raise`
       def evaluate(_task, *_args)
         raise 'Implement me!'
       end

data/lib/checkoff/internal/selector_classes/task.rb

@@ -362,7 +362,8 @@ module Checkoff
         # @param task [Asana::Resources::Task]
         #
         # @return [Boolean]
-        # @sg-ignore
+        # @sg-ignore Checkoff::SelectorClasses::Task::MilestonePFunctionEvaluator#evaluate
+        #   return type could not be inferred
         def evaluate(task)
           raise 'Please add resource_subtype to extra_fields' if task.resource_subtype.nil?
 

data/lib/checkoff/internal/task_timing.rb

@@ -19,14 +19,16 @@ module Checkoff
 
       # @param task [Asana::Resources::Task]
       # @return [Time, nil]
-      # @sg-ignore
+      # @sg-ignore return type could not be inferred — #to_time called via &. on a Date|Time
+      #   union receiver
       def start_time(task)
         date_or_time_field_by_name(task, :start)&.to_time
       end
 
       # @param task [Asana::Resources::Task]
-      # @sg-ignore
       # @return [Time, nil]
+      # @sg-ignore return type could not be inferred — #to_time called via &. on a Date|Time
+      #   union receiver
       def due_time(task)
         date_or_time_field_by_name(task, :due)&.to_time
       end
@@ -35,8 +37,7 @@ module Checkoff
       #
       # @return [Date, Time, nil]
       def start_date_or_time(task)
-        # @sg-ignore
-        return @time_class.parse(task.start_at).localtime unless task.start_at.nil?
+        return @time_class.parse(T.must(task.start_at)).localtime unless task.start_at.nil?
 
         return @date_class.parse(task.start_on) unless task.start_on.nil?
 
@@ -58,7 +59,7 @@ module Checkoff
       #
       # @return [Time, nil]
       def modified_time(task)
-        @time_class.parse(task.modified_at).localtime unless task.modified_at.nil?
+        @time_class.parse(T.must(task.modified_at)).localtime unless task.modified_at.nil?
       end
 
       # @param task [Asana::Resources::Task]

data/lib/checkoff/internal/thread_local.rb

@@ -10,7 +10,8 @@ module Checkoff
       # @param value [Object,Boolean]
       # @yieldreturn [generic<T>]
       # @return [generic<T>]
-      # @sg-ignore
+      # @sg-ignore return type could not be inferred — Solargraph's generic/yield-return
+      #   inference gap for #with_thread_local_variable
       def with_thread_local_variable(name, value, &block)
         old_value = Thread.current[name]
         Thread.current[name] = value

data/lib/checkoff/my_tasks.rb

@@ -67,7 +67,9 @@ module Checkoff
       sections.each_entry { |section| by_section[section_key(section.name)] = [] }
       tasks.each do |task|
         assignee_section = task.assignee_section
-        # @sg-ignore
+        # @sg-ignore assignee_section can be nil (task has no My Tasks section) but is
+        #   dereferenced without a nil guard — looks like a real gap, flagging for the
+        #   follow-up code PR
         current_section = section_key(assignee_section.name)
         by_section[current_section] ||= []
         by_section[current_section] << task

data/lib/checkoff/portfolios.rb

@@ -47,7 +47,7 @@ module Checkoff
     # @param portfolio_name [String]
     #
     # @return [Asana::Resources::Portfolio]
-    # @sg-ignore
+    # @sg-ignore nil check above is not flow-sensitive
     def portfolio_or_raise(workspace_name, portfolio_name)
       portfolio_obj = portfolio(workspace_name, portfolio_name)
       raise "Could not find portfolio #{portfolio_name} under workspace #{workspace_name}." if portfolio_obj.nil?

data/lib/checkoff/projects.rb

@@ -108,8 +108,9 @@ module Checkoff
       else
         # @type [Enumerable<Asana::Resources::Project>]
         ps = projects_by_workspace_name(workspace_name, extra_fields:)
-        # @type <Asana::Resources::Project,nil>
-        # @sg-ignore
+        # @type [Asana::Resources::Project,nil]
+        # @sg-ignore Variable type could not be inferred / Unresolved call to _1 — Solargraph
+        #   doesn't fully support numbered block parameters
         project = ps.find { _1.name == project_name }
         project_by_gid(project.gid, extra_fields:) unless project.nil?
       end
@@ -120,8 +121,8 @@ module Checkoff
     # @param project_name [String,Symbol] - :my_tasks or a project name
     # @param [Array<String>] extra_fields
     #
-    # @sg-ignore
     # @return [Asana::Resources::Project]
+    # @sg-ignore nil check above is not flow-sensitive
     def project_or_raise(workspace_name, project_name, extra_fields: [])
       p = project(workspace_name, project_name, extra_fields:)
       raise "Could not find project #{project_name.inspect} under workspace #{workspace_name}." if p.nil?

data/lib/checkoff/sections.rb

@@ -80,7 +80,7 @@ module Checkoff
 
     # Given a workspace name and project name, then provide a Hash of
     # tasks with section name -> task list of the uncompleted tasks
-    # @param workspace_name [String]
+    # @param workspace_name [String, Symbol]
     # @param project_name [String, Symbol]
     # @param only_uncompleted [Boolean]
     # @param extra_fields [Array<String>]
@@ -295,7 +295,9 @@ module Checkoff
       # @type [String, nil]
       current_section = section_key(section_name)
 
-      # @sg-ignore
+      # @sg-ignore Hash#fetch arg0 expected String,NilClass received String,nil — Solargraph
+      #   treats the nil literal in a Hash{K,nil=>V} declaration and an inferred `nil` union
+      #   member as distinct types; no comment-only fix found
       by_section.fetch(current_section) << task
     end
 

data/lib/checkoff/subtasks.rb

@@ -89,7 +89,8 @@ module Checkoff
     # as memberships with a separate API within a task.
     #
     # @param subtask [Asana::Resources::Task]
-    # @sg-ignore
+    # @return [Boolean, nil]
+    # @sg-ignore Checkoff::Subtasks#subtask_section? return type could not be inferred
     def subtask_section?(subtask)
       subtask.is_rendered_as_separator
     end

data/lib/checkoff/tags.rb

@@ -79,7 +79,7 @@ module Checkoff
     # @param tag_name [String]
     #
     # @return [Asana::Resources::Tag]
-    # @sg-ignore
+    # @sg-ignore nil check above is not flow-sensitive
     def tag_or_raise(workspace_name, tag_name)
       t = tag(workspace_name, tag_name)
 
@@ -114,7 +114,8 @@ module Checkoff
     # @return [Hash{Symbol => Object}]
     def build_params(options)
       { limit: options[:per_page], completed_since: options[:completed_since] }.reject do |_, v|
-        # @sg-ignore
+        # @sg-ignore Unresolved call to nil? on Object — Solargraph doesn't treat a Hash's
+        #   declared `Object` value type as full Object here
         v.nil? || Array(v).empty?
       end
     end

data/lib/checkoff/task_searches.rb

@@ -73,7 +73,9 @@ module Checkoff
       workspace = workspaces.workspace_or_raise(workspace_name)
       api_params, task_selector = @search_url_parser.convert_params(url)
       debug { "Task search params: api_params=#{api_params}, task_selector=#{task_selector}" }
-      # @sg-ignore
+      # @sg-ignore task_selector expected Symbol,Array<Symbol,Integer,Array> received
+      #   Hash{String=>String} — Solargraph mis-assigns tuple-destructured types from
+      #   `api_params, task_selector = @search_url_parser.convert_params(url)` above
       raw_task_search(api_params, workspace_gid: workspace.gid, task_selector:,
                                   extra_fields:)
     end
@@ -186,8 +188,9 @@ module Checkoff
     end
 
     # @param [Array<String>] extra_fields
-    # @sg-ignore
-    # @return [Hash{Symbol => undefined}]
+    # @return [Hash{Symbol => Array<String>}]
+    # @sg-ignore return type could not be inferred — bracket access on a Hash{Symbol=>undefined}
+    #   local doesn't narrow even with a T.cast on the return expression
     def calculate_api_options(extra_fields)
       # @type [Hash{Symbol => undefined}]
       all_options = projects.task_options(extra_fields: ['custom_fields'] + extra_fields)

data/lib/checkoff/tasks.rb

@@ -122,8 +122,7 @@ module Checkoff
              only_uncompleted: true,
              extra_fields: [])
       thread_local = Checkoff::Internal::ThreadLocal.new
-      # @type [String]
-      # @sg-ignore
+      # @type [String, nil]
       task_gid = thread_local.with_thread_local_variable(:suppress_asana_webhook_watch_creation,
                                                          true) do
         gid_for_task(workspace_name, project_name, section_name, task_name)
@@ -179,9 +178,10 @@ module Checkoff
     # @param name [String]
     # @param workspace_gid [String]
     # @param assignee_gid [String]
-    # @sg-ignore
     #
     # @return [Asana::Resources::Task]
+    # @sg-ignore @asana_task.create return type could not be inferred — calling a class method
+    #   through a Class<T>-typed ivar is a known Solargraph dispatch limitation
     def add_task(name,
                  workspace_gid: @workspaces.default_workspace_gid,
                  assignee_gid: default_assignee_gid)
@@ -201,10 +201,10 @@ module Checkoff
 
     # True if any of the task's dependencies are marked incomplete
     #
-    # @sg-ignore
     # Include 'dependencies.gid' in extra_fields of task passed in.
     #
     # @param task [Asana::Resources::Task]
+    # @return [Boolean]
     def incomplete_dependencies?(task)
       # Avoid a redundant fetch.  Unfortunately, Ruby SDK allows
       # dependencies to be fetched along with other attributes--but
@@ -214,14 +214,13 @@ module Checkoff
       #
       # https://github.com/Asana/ruby-asana/issues/125
 
-      # @type [Enumerable<Asana::Resources::Task>, nil]
+      # @type [Array<Hash>]
       dependencies = task.instance_variable_get(:@dependencies) || []
 
       dependencies.any? do |parent_task_info|
         # the real bummer though is that asana doesn't let you fetch
         # the completion status of dependencies, so we need to do this
         # regardless:
-        # @sg-ignore
         parent_task_gid = parent_task_info.fetch('gid')
 
         parent_task = task_by_gid(parent_task_gid, only_uncompleted: false)
@@ -387,7 +386,9 @@ module Checkoff
     end
 
     # @return [String]
-    # @sg-ignore
+    # @sg-ignore @config.fetch return type could not be inferred — @config is intentionally
+    #   dual-typed [Hash, EnvFallbackConfigLoader] throughout this codebase; Solargraph can't
+    #   unify #fetch across that union
     def default_assignee_gid
       @config.fetch(:default_assignee_gid)
     end

data/lib/checkoff/timelines.rb

@@ -57,7 +57,9 @@ module Checkoff
         section_data = membership_data.fetch('section')
         section_gid = section_data.fetch('gid')
         section = @sections.section_by_gid(section_gid)
-        # @sg-ignore
+        # @sg-ignore section_by_gid can return nil on a gid lookup miss, and
+        #   task_data_dependent_on_previous_section_last_milestone? dereferences it immediately
+        #   without a nil guard — looks like a real gap, flagging for the follow-up code PR
         task_data_dependent_on_previous_section_last_milestone?(task_data, section)
       end
     end
@@ -95,7 +97,8 @@ module Checkoff
 
         all_dependent_task_gids ||= @tasks.all_dependent_tasks(task).map(&:gid)
 
-        # @sg-ignore
+        # @sg-ignore all_dependent_task_gids is nil-initialized then set via ||= above, but
+        #   Solargraph doesn't narrow it back to non-nil after the reassignment
         all_dependent_task_gids.include? last_milestone.gid
       end
     end
@@ -131,7 +134,8 @@ module Checkoff
             dependent_task.resource_subtype == 'milestone'
           end
 
-        # @sg-ignore
+        # @sg-ignore all_dependent_milestones is nil-initialized then set via ||= above, but
+        #   Solargraph doesn't narrow it back to non-nil after the reassignment
         all_dependent_milestones.any? do |milestone|
           milestone.memberships.any? do |milestone_membership_data|
             milestone_membership_data.fetch('project').fetch('gid') == project_gid