checkoff 0.256.0 → 0.258.0

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

@@ -105,12 +105,9 @@ module Checkoff
         # @param custom_field_name [String]
         # @return [String, nil]
         def evaluate(resource, custom_field_name)
-          # @sg-ignore
-          custom_field = @custom_fields.resource_custom_field_by_name(resource, custom_field_name)
-          # @sg-ignore
+          custom_field = custom_fields.resource_custom_field_by_name(resource, custom_field_name)
           return nil if custom_field.nil?
 
-          # @sg-ignore
           custom_field['display_value']
         end
       end
@@ -125,17 +122,13 @@ module Checkoff
           false
         end
 
-        # @sg-ignore
         # @param resource [Asana::Resources::Task,Asana::Resources::Project]
-        # @sg-ignore
         # @param custom_field_gid [String]
         # @return [String, nil]
-        # @sg-ignore
         def evaluate(resource, custom_field_gid)
-          # @sg-ignore
-          custom_field = @custom_fields.resource_custom_field_by_gid_or_raise(resource, custom_field_gid)
-          # @sg-ignore
-          custom_field['display_value']
+          custom_field = custom_fields.resource_custom_field_by_gid_or_raise(resource, custom_field_gid)
+          display_value = custom_field['display_value']
+          display_value.nil? ? nil : T.cast(display_value, String)
         end
       end
 
@@ -152,17 +145,13 @@ module Checkoff
         end
 
         # @param resource [Asana::Resources::Task,Asana::Resources::Project]
-        # @sg-ignore
         # @param custom_field_gid [String]
         # @param custom_field_values_gids [Array<String>]
         # @return [Boolean]
-        # @sg-ignore
         def evaluate(resource, custom_field_gid, custom_field_values_gids)
-          # @sg-ignore
-          actual_custom_field_values_gids = @custom_fields.resource_custom_field_values_gids_or_raise(resource,
-                                                                                                      custom_field_gid)
+          actual_custom_field_values_gids = custom_fields.resource_custom_field_values_gids_or_raise(resource,
+                                                                                                     custom_field_gid)
 
-          # @sg-ignore
           actual_custom_field_values_gids.intersect?(custom_field_values_gids)
         end
       end
@@ -181,17 +170,13 @@ module Checkoff
 
         # @param resource [Asana::Resources::Task,Asana::Resources::Project]
         # @param custom_field_name [String]
-        # @sg-ignore
         # @param custom_field_value_names [Array<String>]
-        # @sg-ignore
         # @return [Boolean]
         def evaluate(resource, custom_field_name, custom_field_value_names)
           actual_custom_field_values_names =
-            # @sg-ignore
-            @custom_fields.resource_custom_field_values_names_by_name(resource,
-                                                                      custom_field_name)
+            custom_fields.resource_custom_field_values_names_by_name(resource,
+                                                                     custom_field_name)
 
-          # @sg-ignore
           actual_custom_field_values_names.intersect?(custom_field_value_names)
         end
       end
@@ -214,12 +199,10 @@ module Checkoff
         # @return [Boolean]
         def evaluate(resource, custom_field_gid, custom_field_values_gids)
           actual_custom_field_values_gids =
-            # @sg-ignore
-            @custom_fields.resource_custom_field_values_gids_or_raise(resource,
-                                                                      custom_field_gid)
+            custom_fields.resource_custom_field_values_gids_or_raise(resource,
+                                                                     custom_field_gid)
 
           custom_field_values_gids.all? do |custom_field_value|
-            # @sg-ignore
             actual_custom_field_values_gids.include?(custom_field_value)
           end
         end
@@ -238,9 +221,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
@@ -254,9 +238,9 @@ module Checkoff
 
         # @param _resource [Asana::Resources::Task,Asana::Resources::Project]
         # @return [String]
-        # @sg-ignore
+        # @sg-ignore selector is only String here per matches?'s is_a?(String) check, in another method
         def evaluate(_resource)
-          T.cast(selector, String)
+          selector
         end
       end
     end

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

@@ -9,21 +9,32 @@ module Checkoff
     class FunctionEvaluator
       include Logging
 
+      # Concrete subclasses each declare their own initialize with their
+      # specific keyword args (and set @selector themselves before calling
+      # super()); this signature exists only so Solargraph can resolve
+      # `evaluator_class.new(selector:, **kwargs)` call sites that hold a
+      # `Class<FunctionEvaluator>` reference rather than a concrete subclass -
+      # deliberately a no-op so a bare `super()` call from those subclasses
+      # can't clobber the @selector they already set.
+      # @param selector [Symbol, Array, String, nil]
+      # @param _kwargs [Hash]
+      def initialize(selector: nil, **_kwargs); end
+
       # @param _index [Integer]
       def evaluate_arg?(_index)
         true
       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/project/function_evaluator.rb

@@ -25,6 +25,15 @@ module Checkoff
           super()
         end
 
+        # @return [Checkoff::Projects]
+        attr_reader :projects
+
+        # @return [Checkoff::Portfolios]
+        attr_reader :portfolios
+
+        # @return [Checkoff::Workspaces]
+        attr_reader :workspaces
+
         private
 
         # @return [Array(Symbol, Array)]

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

@@ -17,7 +17,7 @@ module Checkoff
 
         # @param resource [Asana::Resources::Project]
         # @return [String, nil]
-        # @sg-ignore
+        # @sg-ignore nilable return from an @!parse-declared method isn't inferred
         def evaluate(resource)
           resource.due_date
         end
@@ -33,11 +33,9 @@ module Checkoff
 
         # @param project [Asana::Resources::Project]
         # @param period [Symbol] - :now_or_before or :this_week
-        # @sg-ignore
         # @return [Boolean]
         def evaluate(project, period = :now_or_before)
-          # @sg-ignore
-          @projects.project_ready?(project, period:)
+          projects.project_ready?(project, period:)
         end
       end
 
@@ -53,19 +51,14 @@ module Checkoff
         # @param portfolio_name [String]
         # @param workspace_name [String, nil]
         # @param extra_project_fields [Array<String>]
-        # @sg-ignore
         #
-        # @sg-ignore
         # @return [Boolean]
         def evaluate(project, portfolio_name, workspace_name: nil, extra_project_fields: [])
           workspace_name ||= project.workspace&.name
-          # @sg-ignore
-          workspace_name ||= @workspaces.default_workspace.name
-          # @sg-ignore
-          projects = @portfolios.projects_in_portfolio(workspace_name, portfolio_name,
-                                                       extra_project_fields:)
-          # @sg-ignore
-          projects.any? { |p| p.name == project.name }
+          workspace_name ||= workspaces.default_workspace.name
+          matching_projects = portfolios.projects_in_portfolio(T.must(workspace_name), portfolio_name,
+                                                               extra_project_fields:)
+          matching_projects.any? { |p| p.name == project.name }
         end
       end
     end

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

@@ -22,13 +22,16 @@ module Checkoff
           super()
         end
 
+        # @return [Checkoff::Sections]
+        attr_reader :sections
+
+        # @return [Asana::Client]
+        attr_reader :client
+
         private
 
         # @return [Array(Symbol, Array)]
         attr_reader :selector
-
-        # @return [Asana::Client]
-        attr_reader :client
       end
     end
   end

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

@@ -18,7 +18,7 @@ module Checkoff
         # @param section [Asana::Resources::Section]
         #
         # @return [Boolean]
-        # @sg-ignore
+        # @sg-ignore String, nil safe-nav chain isn't inferred as returning Boolean
         def evaluate(section)
           tasks = client.tasks.get_tasks(section: section.gid,
                                          per_page: 100,
@@ -37,11 +37,9 @@ module Checkoff
 
         # @param section [Asana::Resources::Section]
         #
-        # @sg-ignore
         # @return [Boolean]
         def evaluate(section)
-          # @sg-ignore
-          @sections.tasks_by_section_gid(section.gid).any?
+          sections.tasks_by_section_gid(section.gid).any?
         end
       end
     end

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

@@ -24,8 +24,12 @@ module Checkoff
           # @type [Hash{'unwrapped' => Hash}]
           task_data = tasks.task_to_h(task)
           # @type [Hash{'membership_by_project_name' => Hash}]
+          # @sg-ignore Hash#fetch generic<X> leak on rbs >= 4.1.0, fix in progress upstream
+          # https://github.com/castwide/solargraph/pull/1228
           unwrapped = task_data.fetch('unwrapped')
           # @type [Array]
+          # @sg-ignore Hash#fetch generic<X> leak on rbs >= 4.1.0, fix in progress upstream
+          # https://github.com/castwide/solargraph/pull/1228
           projects = unwrapped.fetch('membership_by_project_name').keys
           !(projects - [:my_tasks]).empty?
         end
@@ -45,10 +49,18 @@ module Checkoff
         # @param task [Asana::Resources::Task]
         # @param section_name_prefix [String]
         # @return [Boolean]
-        # @sg-ignore
         def evaluate(task, section_name_prefix)
+          # @type [Hash{'unwrapped' => Hash}]
           task_data = tasks.task_to_h(task)
-          task_data.fetch('unwrapped').fetch('membership_by_section_name').keys.any? do |section_name|
+          # @type [Hash{'membership_by_section_name' => Hash}]
+          # @sg-ignore Hash#fetch generic<X> leak on rbs >= 4.1.0, fix in progress upstream
+          # https://github.com/castwide/solargraph/pull/1228
+          unwrapped = task_data.fetch('unwrapped')
+          # @type [Array]
+          # @sg-ignore Hash#fetch generic<X> leak on rbs >= 4.1.0, fix in progress upstream
+          # https://github.com/castwide/solargraph/pull/1228
+          section_names = unwrapped.fetch('membership_by_section_name').keys
+          section_names.any? do |section_name|
             String(section_name).start_with?(section_name_prefix)
           end
         end
@@ -65,13 +77,21 @@ module Checkoff
           false
         end
 
-        # @sg-ignore
         # @param task [Asana::Resources::Task]
         # @param section_name [String]
         # @return [Boolean]
         def evaluate(task, section_name)
+          # @type [Hash{'unwrapped' => Hash}]
           task_data = tasks.task_to_h(task)
-          task_data.fetch('unwrapped').fetch('membership_by_section_name').keys.any?(section_name)
+          # @type [Hash{'membership_by_section_name' => Hash}]
+          # @sg-ignore Hash#fetch generic<X> leak on rbs >= 4.1.0, fix in progress upstream
+          # https://github.com/castwide/solargraph/pull/1228
+          unwrapped = task_data.fetch('unwrapped')
+          # @type [Array]
+          # @sg-ignore Hash#fetch generic<X> leak on rbs >= 4.1.0, fix in progress upstream
+          # https://github.com/castwide/solargraph/pull/1228
+          section_names = unwrapped.fetch('membership_by_section_name').keys
+          section_names.any?(section_name)
         end
       end
 
@@ -89,13 +109,15 @@ module Checkoff
         # @param task [Asana::Resources::Task]
         # @param project_name [String]
         # @return [Boolean]
-        # @sg-ignore
+        # @sg-ignore Hash#fetch generic<X> leak on rbs >= 4.1.0, fix in progress upstream
+        # https://github.com/castwide/solargraph/pull/1228
         def evaluate(task, project_name)
           project_names = task.memberships.map do |membership|
-            m = T.cast(membership, T::Hash[String, T.untyped])
-            T.cast(m.fetch('project'), T::Hash[String, T.untyped]).fetch('name')
+            # @sg-ignore Hash#fetch generic<X> leak on rbs >= 4.1.0, fix in progress upstream
+            # https://github.com/castwide/solargraph/pull/1228
+            membership.fetch('project').fetch('name')
           end
-          T.cast(project_names.include?(project_name), T::Boolean)
+          project_names.include?(project_name)
         end
       end
 
@@ -132,9 +154,11 @@ module Checkoff
         # @param task [Asana::Resources::Task]
         # @param tag_name [String]
         # @return [Boolean]
-        # @sg-ignore
         def evaluate(task, tag_name)
-          task.tags.map(&:name).include? tag_name
+          # @type [Array<Asana::Resources::Tag>]
+          tags = task.tags.to_a
+          tag_names = tags.map(&:name)
+          tag_names.include?(tag_name)
         end
       end
 
@@ -235,13 +259,13 @@ module Checkoff
 
           # @type [Array<Asana::Resources::Story>]
           stories = task.stories(per_page: 100).to_a.reject do |story|
-            # @sg-ignore
+            # @sg-ignore story's inferred block param type regressed on rbs >= 4.1.0, fix in progress upstream
+            # https://github.com/castwide/solargraph/pull/1228
             excluding_resource_subtypes.include? story.resource_subtype
           end
           return true if stories.empty? # no stories == infinitely old!
 
-          last_story = stories.last
-          # @sg-ignore
+          last_story = T.must(stories.last)
           last_story_created_at = Time.parse(last_story.created_at)
           n_days_ago = Time.at(Time.now.to_i - (num_days * 86_400))
           last_story_created_at < n_days_ago
@@ -362,11 +386,13 @@ 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?
+          resource_subtype = task.resource_subtype
+          raise 'Please add resource_subtype to extra_fields' if resource_subtype.nil?
 
-          task.resource_subtype == 'milestone'
+          resource_subtype == 'milestone'
         end
       end
 

data/lib/checkoff/internal/selector_evaluator.rb

@@ -11,13 +11,11 @@ module Checkoff
 
       function_evaluators.each do |evaluator_class|
         # @type [Checkoff::SelectorClasses::FunctionEvaluator]
-        # @sg-ignore
         evaluator = evaluator_class.new(selector:,
                                         **initializer_kwargs)
 
         next unless evaluator.matches?
 
-        # @sg-ignore
         return try_this_evaluator(selector, evaluator)
       end
 
@@ -32,18 +30,18 @@ module Checkoff
     end
 
     # @return [Array<Class<Checkoff::SelectorClasses::FunctionEvaluator>>]
-    # @sg-ignore
+    # @sg-ignore abstract method always raises; declared type documents the override contract, not this body
     def function_evaluators
       raise 'Implement me!'
     end
 
-    # @param selector [Array]
+    # @param selector [Symbol, Array]
     # @param evaluator [Checkoff::SelectorClasses::FunctionEvaluator]
     # @return [Array]
     def evaluate_args(selector, evaluator)
       return [] unless selector.is_a?(Array)
 
-      # @sg-ignore
+      # @sg-ignore Array#[] with an open-ended range is inferred as nilable even though it never is here
       selector[1..].map.with_index do |item, index|
         if evaluator.evaluate_arg?(index)
           evaluate(item)
@@ -53,7 +51,7 @@ module Checkoff
       end
     end
 
-    # @param selector [Array]
+    # @param selector [Symbol, Array]
     # @param evaluator [Checkoff::SelectorClasses::FunctionEvaluator]
     # @return [Boolean, Object, nil]
     def try_this_evaluator(selector, evaluator)

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>]
@@ -142,11 +142,10 @@ module Checkoff
     # @param section_name [String, nil]
     #
     # @return [Array<String>]
-    # @sg-ignore
+    # @sg-ignore Task#name is nilable; task names are assumed present here
     def section_task_names(workspace_name, project_name, section_name)
       task_array = tasks(workspace_name, project_name, section_name)
-      # @type [Array<String>]
-      T.cast(task_array.map(&:name), T::Array[String])
+      T.cast(task_array.map(&:name), Array)
     end
     cache_method :section_task_names, SHORT_CACHE_TIME
 
@@ -184,17 +183,15 @@ module Checkoff
     #
     # @return [Asana::Resources::Section, nil]
     def previous_section(section)
-      sections = sections_by_project_gid(section.project.fetch('gid'))
+      project_sections = sections_by_project_gid(section.project.fetch('gid'))
 
       # @type [Array<Asana::Resources::Section>]
-      # @sg-ignore
-      sections = sections.to_a
+      project_sections_arr = project_sections.to_a
 
-      index = sections.find_index { |s| s.gid == section.gid }
+      index = project_sections_arr.find_index { |s| s.gid == section.gid }
       return nil if index.nil? || index.zero?
 
-      # @sg-ignore
-      sections[index - 1]
+      project_sections_arr[index - 1]
     end
     cache_method :previous_section, SHORT_CACHE_TIME
 
@@ -295,7 +292,6 @@ module Checkoff
       # @type [String, nil]
       current_section = section_key(section_name)
 
-      # @sg-ignore
       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/task_selectors.rb

@@ -64,19 +64,19 @@ module Checkoff
     # :nocov:
     class << self
       # @return [String]
-      # @sg-ignore
+      # @sg-ignore `x || raise(...)` isn't inferred as narrowing away the nil branch
       def project_name
         ARGV[1] || raise('Please pass project name to pull tasks from as first argument')
       end
 
-      # @sg-ignore
       # @return [String]
+      # @sg-ignore `x || raise(...)` isn't inferred as narrowing away the nil branch
       def workspace_name
         ARGV[0] || raise('Please pass workspace name as first argument')
       end
 
       # @return [Array(Symbol, Array)]
-      # @sg-ignore
+      # @sg-ignore `x || raise(...)` isn't inferred as narrowing away the nil branch
       def task_selector
         task_selector_json = ARGV[2] || raise('Please pass task_selector in JSON form as third argument')