checkoff 0.227.0 → 0.229.0

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

@@ -1,4 +1,4 @@
-# typed: false
+# typed: true
 # frozen_string_literal: true
 
 module Checkoff
@@ -9,10 +9,10 @@ module Checkoff
         # base class for handling different custom_field_#{gid}.variant params
         class CustomFieldVariant
           # @param [String] gid
-          # @param [Hash] remaining_params
+          # @param [Hash{String => Array<String>}] remaining_params
           def initialize(gid, remaining_params)
-            @gid = gid
-            @remaining_params = remaining_params
+            @gid = T.let(gid, String)
+            @remaining_params = T.let(remaining_params, T::Hash[String, T::Array[String]])
           end
 
           private
@@ -20,7 +20,7 @@ module Checkoff
           # @return [String]
           attr_reader :gid
 
-          # @return [Hash]
+          # @return [Hash{String => Array<String>}]
           attr_reader :remaining_params
 
           # @return [void]
@@ -32,18 +32,18 @@ module Checkoff
 
           # @param [String] param_name
           #
+          # @sg-ignore
           # @return [String]
           def fetch_solo_param(param_name)
             case remaining_params.keys
             when [param_name]
-              # @sg-ignore
-              # @type [Array<String>]
+              # @type param_values [Array<String>]
               param_values = remaining_params.fetch(param_name)
               unless param_values.length == 1
                 raise "Teach me how to handle these remaining keys for #{param_name}: #{remaining_params}"
               end
 
-              param_values[0]
+              param_values.fetch(0)
             else
               raise "Teach me how to handle #{remaining_params}"
             end
@@ -52,7 +52,7 @@ module Checkoff
 
         # custom_field_#{gid}.variant = 'less_than'
         class LessThan < CustomFieldVariant
-          # @return [Array<(Hash, Array)>]
+          # @return [Array(Hash{String => String}, Array<Array<String>,String>)]
           def convert
             max_value = fetch_solo_param("custom_field_#{gid}.max")
             empty_task_selector = []
@@ -62,7 +62,7 @@ module Checkoff
 
         # custom_field_#{gid}.variant = 'greater_than'
         class GreaterThan < CustomFieldVariant
-          # @return [Array<(Hash, Array)>]
+          # @return [Array(Hash{String => String}, Array<Array<String>,String>)]
           def convert
             max_value = fetch_solo_param("custom_field_#{gid}.min")
             empty_task_selector = []
@@ -72,7 +72,7 @@ module Checkoff
 
         # custom_field_#{gid}.variant = 'equals'
         class Equals < CustomFieldVariant
-          # @return [Array<(Hash, Array)>]
+          # @return [Array(Hash{String => String}, Array<Array<String>,String>)]
           def convert
             value = fetch_solo_param("custom_field_#{gid}.value")
             empty_task_selector = []
@@ -84,7 +84,7 @@ module Checkoff
         #
         # custom_field_#{gid}.variant = 'is_not'
         class IsNot < CustomFieldVariant
-          # @return [Array<(Hash, Array)>]
+          # @return [Array(Hash{String => String}, Array<Array<String>, String, Array>)]
           def convert
             selected_options = fetch_solo_param("custom_field_#{gid}.selected_options").split('~')
 
@@ -101,7 +101,7 @@ module Checkoff
         #
         # custom_field_#{gid}.variant = 'doesnt_contain_any'
         class DoesntContainAny < CustomFieldVariant
-          # @return [Array<(Hash, Array)>]
+          # @return [Array(Hash{String => String}, Array<Array<String>, String, Array>)]
           def convert
             selected_options = fetch_solo_param("custom_field_#{gid}.selected_options").split('~')
 
@@ -117,7 +117,7 @@ module Checkoff
         #
         # custom_field_#{gid}.variant = 'contains_any'
         class ContainsAny < CustomFieldVariant
-          # @return [Array<(Hash, Array)>]
+          # @return [Array(Hash{String => String}, Array<Array<String>, String, Array>)]
           def convert
             selected_options = fetch_solo_param("custom_field_#{gid}.selected_options").split('~')
 
@@ -132,7 +132,7 @@ module Checkoff
         #
         # custom_field_#{gid}.variant = 'contains_all'
         class ContainsAll < CustomFieldVariant
-          # @return [Array<(Hash, Array)>]
+          # @return [Array(Hash{String => String}, Array<Array<String>, String, Array>)]
           def convert
             selected_options = fetch_solo_param("custom_field_#{gid}.selected_options").split('~')
 
@@ -145,10 +145,11 @@ module Checkoff
 
         # custom_field_#{gid}.variant = 'no_value'
         class NoValue < CustomFieldVariant
-          # @return [Array<(Hash, Array)>]
+          # @return [Array(Hash{String => String}, Array<Array<String>, String, Array, Symbol>)]
           def convert
             ensure_no_remaining_params!
 
+            # @type [Hash{String => String}]
             api_params = { "custom_fields.#{gid}.is_set" => 'false' }
             # As of 2023-02, the 'is_set' => 'false' seems to not do
             # the intuitive thing on multi-select fields; it either
@@ -165,7 +166,7 @@ module Checkoff
         #
         # Not used for multi-select fields
         class AnyValue < CustomFieldVariant
-          # @return [Array<(Hash, Array)>]
+          # @return [Array(Hash{String => String}, Array<Array<String>, String, Array>)]
           def convert
             ensure_no_remaining_params!
 
@@ -177,7 +178,7 @@ module Checkoff
 
         # custom_field_#{gid}.variant = 'is'
         class Is < CustomFieldVariant
-          # @return [Array<(Hash, Array)>]
+          # @return [Array(Hash{String => String}, Array<Array<String>, String, Array>)]
           def convert
             selected_options = fetch_solo_param("custom_field_#{gid}.selected_options").split('~')
 

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

@@ -9,13 +9,13 @@ module Checkoff
       # Convert date parameters - ones where the param name itself
       # doesn't encode any parameters'
       class DateParamConverter
-        # @param date_url_params [Hash<String, Array<String>>] the simple params
+        # @param date_url_params [Hash{String => Array<String>}] the simple params
         def initialize(date_url_params:)
           @date_url_params = date_url_params
         end
 
         # @sg-ignore
-        # @return [Array(Hash<String, String>, Array<[Symbol, Array]>)]
+        # @return [Array(Hash{String => String}, Array<Symbol, Array>)]
         def convert
           return [{}, []] if date_url_params.empty?
 
@@ -35,9 +35,8 @@ module Checkoff
 
         private
 
-        # @sg-ignore
         # @param prefix [String]
-        # @return [Array(Hash<String, String>, Array<[Symbol, Array]>)]
+        # @return [Array(Hash{String => String}, Array<Symbol, Array>)]
         def convert_for_prefix(prefix)
           # example params:
           #   due_date.operator=through_next
@@ -70,9 +69,10 @@ module Checkoff
           'start_date' => 'start_on',
           'completion_date' => 'completed_on',
         }.freeze
+        private_constant :API_PREFIX
 
         # @param prefix [String]
-        # @return [Array(Hash<String, String>, Array<[Symbol, Array]>)] See https://developers.asana.com/docs/search-tasks-in-a-workspace
+        # @return [Array(Hash{String => String}, Array<Symbol, Array>)] See https://developers.asana.com/docs/search-tasks-in-a-workspace
         def handle_through_next(prefix)
           value = get_single_param("#{prefix}.value").to_i
 
@@ -89,7 +89,7 @@ module Checkoff
         end
 
         # @param prefix [String]
-        # @return [Array(Hash<String, String>, Array<[Symbol, Array]>)] See https://developers.asana.com/docs/search-tasks-in-a-workspace
+        # @return [Array(Hash{String => String}, Array<Symbol, Array>)] See https://developers.asana.com/docs/search-tasks-in-a-workspace
         def handle_between(prefix)
           after = get_single_param("#{prefix}.after")
           raise "Teach me how to handle #{prefix}.before" if date_url_params.key? "#{prefix}.before"
@@ -105,7 +105,7 @@ module Checkoff
         end
 
         # @param prefix [String]
-        # @return [Array(Hash<String, String>, Array<[Symbol, Array]>)] See https://developers.asana.com/docs/search-tasks-in-a-workspace
+        # @return [Array(Hash{String => String}, Array<Symbol, Array>)] See https://developers.asana.com/docs/search-tasks-in-a-workspace
         def handle_within_last(prefix)
           value = get_single_param("#{prefix}.value").to_i
 
@@ -119,7 +119,7 @@ module Checkoff
         end
 
         # @param prefix [String]
-        # @return [Array(Hash<String, String>, Array<[Symbol, Array]>)] See https://developers.asana.com/docs/search-tasks-in-a-workspace
+        # @return [Array(Hash{String => String}, Array<Symbol, Array>)] See https://developers.asana.com/docs/search-tasks-in-a-workspace
         def handle_within_next(prefix)
           value = get_single_param("#{prefix}.value").to_i
 
@@ -160,7 +160,7 @@ module Checkoff
           raise "Teach me how to handle other time units: #{unit}" unless unit == 'day'
         end
 
-        # @return [Hash<String, Array<String>>]
+        # @return [Hash{String => Array<String>}]
         attr_reader :date_url_params
       end
     end

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

@@ -1,5 +1,5 @@
 #!/usr/bin/env ruby
-# typed: false
+# typed: true
 
 # frozen_string_literal: true
 
@@ -22,21 +22,11 @@ module Checkoff
         end
 
         # @param url [String]
-        # @return [Array(Hash<String, String>, Hash<String, String>)]
+        # @return [Array(Hash{String => String}, Array)]
         def convert_params(url)
           url_params = CGI.parse(URI.parse(url).query)
-          # @type custom_field_params [Hash<String, Array<String>>]
-          # @type date_url_params [Hash<String, Array<String>>]
-          # @type simple_url_params [Hash<String, Array<String>>]
-          # @sg-ignore
           custom_field_params, date_url_params, simple_url_params = partition_url_params(url_params)
-          # @type custom_field_args [Hash<String, String>]
-          # @type custom_field_task_selector [Hash<String, String>]
-          # @sg-ignore
           custom_field_args, custom_field_task_selector = convert_custom_field_params(custom_field_params)
-          # @type date_args [Hash<String, String>]
-          # @type date_task_selector [Hash<String, String>]
-          # @sg-ignore
           date_url_args, date_task_selector = convert_date_params(date_url_params)
           simple_url_args = convert_simple_params(simple_url_params)
           # raise 'merge these things'
@@ -46,37 +36,37 @@ module Checkoff
 
         private
 
-        # @param date_url_params [Hash<String, Array<String>>]
-        # @return [Array(Hash<String, String>, Array<[Symbol, Array]>)]
+        # @param date_url_params [Hash{String => Array<String>}]
+        # @return [Array(Hash{String => String}, Array<Symbol, Array>)]
         def convert_date_params(date_url_params)
           DateParamConverter.new(date_url_params:).convert
         end
 
-        # @param simple_url_params [Hash<String, Array<String>>]
-        # @return [Hash<String, String>]
+        # @param simple_url_params [Hash{String => Array<String>}]
+        # @return [Hash{String => String}]
         def convert_simple_params(simple_url_params)
           SimpleParamConverter.new(simple_url_params:).convert
         end
 
-        # @param custom_field_params [Hash<String, Array<String>>]
-        # @return [Array(Hash<String, String>, Array<[Symbol, Array]>)]
+        # @param custom_field_params [Hash{String => Array<String>}]
+        # @return [Array(Hash{String => String}, Array<Symbol, Array>)]
         def convert_custom_field_params(custom_field_params)
           CustomFieldParamConverter.new(custom_field_params:).convert
         end
 
-        # @param url_params [Hash<String, String>]
-        # @return [Array(Hash<String, String>, Hash<String, String>, Hash<String, String>)]
+        # @param url_params [Hash{String => String}]
+        # @return [Array(Hash{String => Array<String>}, Hash{String => Array<String>}, Hash{String => Array<String>})]
         def partition_url_params(url_params)
-          groups = url_params.to_a.group_by do |key, _values|
-            if key.start_with? 'custom_field_'
-              :custom_field
-            elsif key.include? '_date'
-              :date
-            else
-              :simple
-            end
-          end.transform_values(&:to_h)
-          # @sg-ignore
+          groups = T.let(url_params.to_a.group_by do |key, _values|
+                           if key.start_with? 'custom_field_'
+                             :custom_field
+                           elsif key.include? '_date'
+                             :date
+                           else
+                             :simple
+                           end
+                         end.transform_values(&:to_h),
+                         T::Hash[Symbol, T::Hash[String, T::Array[String]]])
           [groups.fetch(:custom_field, {}), groups.fetch(:date, {}), groups.fetch(:simple, {})]
         end
       end

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

@@ -6,20 +6,18 @@ module Checkoff
     module SearchUrl
       # Merge task selectors and search API arguments
       class ResultsMerger
-        # @param args [Array<[Hash<String, String>]>]
-        # @return [Hash<String, String>
+        # @param args [Array<Hash{String => String}>]
+        # @return [Hash{String => String}]
         def self.merge_args(*args)
           # first element of args
-          # @sg-ignore
-          # @type [Hash<String, String>]
           f = args.fetch(0)
           # rest of args
           r = args.drop(0)
           f.merge(*r)
         end
 
-        # @param task_selectors [Array<Array<[Symbol, Array]>>]
-        # @return [Array<[Symbol, Array]>]
+        # @param task_selectors [Array<Symbol, Array>]
+        # @return [Array(Symbol, Array, Array)]
         def self.merge_task_selectors(*task_selectors)
           return [] if task_selectors.empty?
 

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

@@ -1,4 +1,4 @@
-# typed: false
+# typed: true
 # frozen_string_literal: true
 
 require_relative 'custom_field_param_converter'
@@ -50,6 +50,8 @@ module Checkoff
             single_value.split('~').each do |project_section_pair|
               # @sg-ignore
               project, section = project_section_pair.split('_column_')
+              raise "Invalid query string: #{project_section_pair}" if project.nil?
+
               if section.nil?
                 projects << project
               else
@@ -97,7 +99,7 @@ module Checkoff
 
         # Handle 'completion' search url param
         class Completion < SimpleParam
-          # @return [Array<String>]
+          # @return [Array<String,Boolean>]
           def convert
             case single_value
             when 'incomplete'
@@ -121,7 +123,7 @@ module Checkoff
 
         # handle 'subtask' search url param
         class Subtask < SimpleParam
-          # @return [Array<[String, Boolean]>]
+          # @return [Array<String, Boolean>]
           def convert
             case single_value
             when 'is_not_subtask'
@@ -185,16 +187,21 @@ module Checkoff
       # Convert simple parameters - ones where the param name itself
       # doesn't encode any parameters'
       class SimpleParamConverter
-        # @param simple_url_params [Hash<String, Array<String>>] the simple params
+        # @param simple_url_params [Hash{String => Array<String>}] the simple params
         def initialize(simple_url_params:)
           @simple_url_params = simple_url_params
         end
 
-        # @return [Hash<String, String>] the converted params
+        # @return [Hash{String => String}] the converted params
         def convert
-          out = simple_url_params.to_a.flat_map do |key, values|
-            convert_arg(key, values).each_slice(2).to_a
-          end.to_h
+          # @type [Array<Array(String, String)>]
+          arr_of_tuples = simple_url_params.to_a.flat_map do |key, values|
+            # @type
+            entry = convert_arg(key, values).each_slice(2).to_a
+            entry
+          end
+          # @type [Hash{String => String}]
+          out = T.cast(arr_of_tuples.to_h, T::Hash[String, String])
           unless out.include? 'sort_by'
             # keep results consistent between calls; API using default
             # sort_by does not seem to be.
@@ -205,7 +212,7 @@ module Checkoff
 
         private
 
-        # @type [Hash<String, Class<SimpleParam::SimpleParam>>] the mapping from param name to class
+        # @type [Hash{String => Class<SimpleParam::SimpleParam>}] the mapping from param name to class
         ARGS = {
           'portfolios.ids' => SimpleParam::PortfoliosIds,
           'any_projects.ids' => SimpleParam::AnyProjectsIds,
@@ -218,12 +225,13 @@ module Checkoff
           'milestone' => SimpleParam::Milestone,
           'searched_type' => SimpleParam::SearchedType,
         }.freeze
+        private_constant :ARGS
 
         # https://developers.asana.com/docs/search-tasks-in-a-workspace
         # @sg-ignore
         # @param key [String] the name of the search url param
         # @param values [Array<String>] the values of the search url param
-        # @return [Hash<String, String>] the converted params
+        # @return [Hash{String => String}] the converted params
         def convert_arg(key, values)
           # @type [Class<SimpleParam::SimpleParam>]
           clazz = ARGS.fetch(key)
@@ -234,7 +242,7 @@ module Checkoff
           obj.convert
         end
 
-        # @return [Hash<String, Array<String>>]
+        # @return [Hash{String => Array<String>}]
         attr_reader :simple_url_params
       end
     end

data/lib/checkoff/internal/section_selector_evaluator.rb

@@ -8,11 +8,12 @@ require_relative 'selector_evaluator'
 module Checkoff
   # Evaluates section selectors against a section
   class SectionSelectorEvaluator < SelectorEvaluator
-    # @param section [Asana::Resources::Project]
+    # @param section [Asana::Resources::Section]
     # @param client [Asana::Client]
     # @param projects [Checkoff::Projects]
     # @param sections [Checkoff::Sections]
     # @param custom_fields [Checkoff::CustomFields]
+    # @param _kwargs [Hash]
     def initialize(section:,
                    client:,
                    projects: Checkoff::Projects.new(client:),
@@ -32,16 +33,19 @@ module Checkoff
     COMMON_FUNCTION_EVALUATORS = (Checkoff::SelectorClasses::Common.constants.map do |const|
       Checkoff::SelectorClasses::Common.const_get(const)
     end - [Checkoff::SelectorClasses::Common::FunctionEvaluator]).freeze
+    private_constant :COMMON_FUNCTION_EVALUATORS
 
     SECTION_FUNCTION_EVALUATORS = (Checkoff::SelectorClasses::Section.constants.map do |const|
       Checkoff::SelectorClasses::Section.const_get(const)
     end - [Checkoff::SelectorClasses::Section::FunctionEvaluator]).freeze
+    private_constant :SECTION_FUNCTION_EVALUATORS
 
-    FUNCTION_EVALUTORS = (COMMON_FUNCTION_EVALUATORS + SECTION_FUNCTION_EVALUATORS).freeze
+    FUNCTION_EVALUATORS = (COMMON_FUNCTION_EVALUATORS + SECTION_FUNCTION_EVALUATORS).freeze
+    private_constant :FUNCTION_EVALUATORS
 
     # @return [Array<Class<Checkoff::SelectorClasses::Project::FunctionEvaluator>>]
     def function_evaluators
-      FUNCTION_EVALUTORS
+      FUNCTION_EVALUATORS
     end
 
     # @return [Hash]

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

@@ -8,8 +8,9 @@ module Checkoff
     module Common
       # Base class to evaluate a project selector function given fully evaluated arguments
       class FunctionEvaluator < ::Checkoff::SelectorClasses::FunctionEvaluator
-        # @param selector [Array<(Symbol, Array)>,String]
+        # @param selector [Array(Symbol, Array), String]
         # @param custom_fields [Checkoff::CustomFields]
+        # @param _kwargs [Hash]
         def initialize(selector:, custom_fields:, **_kwargs)
           @selector = selector
           @custom_fields = custom_fields
@@ -18,7 +19,7 @@ module Checkoff
 
         private
 
-        # @return [Array<(Symbol, Array)>]
+        # @return [Array(Symbol, Array), String]
         attr_reader :selector
       end
     end

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

@@ -1,4 +1,4 @@
-# typed: false
+# typed: true
 # frozen_string_literal: true
 
 require_relative 'common/function_evaluator'
@@ -224,12 +224,11 @@ module Checkoff
           false
         end
 
-        # @param resource [Asana::Resources::Task,Asana::Resources::Project]
+        # @param resource [Asana::Resources::Task, Asana::Resources::Project]
         # @param prefix [String]
-        # @return [Boolean]
-        # @sg-ignore
+        # @return [boolish]
         def evaluate(resource, prefix)
-          resource.name.start_with?(prefix)
+          resource.name&.start_with?(prefix)
         end
       end
 
@@ -243,7 +242,7 @@ module Checkoff
         # @param _resource [Asana::Resources::Task,Asana::Resources::Project]
         # @return [String]
         def evaluate(_resource)
-          selector
+          T.cast(selector, String)
         end
       end
     end

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

@@ -1,4 +1,4 @@
-# typed: strict
+# typed: true
 # frozen_string_literal: true
 
 require 'checkoff/internal/logging'
@@ -9,7 +9,6 @@ module Checkoff
     class FunctionEvaluator
       include Logging
 
-      # @sg-ignore
       # @param _index [Integer]
       def evaluate_arg?(_index)
         true
@@ -23,8 +22,8 @@ module Checkoff
 
       # @param _task [Asana::Resources::Task]
       # @param _args [Array<Object>]
-      # @return [Object]
       # @sg-ignore
+      # @return [Object]
       def evaluate(_task, *_args)
         raise 'Implement me!'
       end

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

@@ -8,10 +8,11 @@ module Checkoff
     module Project
       # Base class to evaluate a project selector function given fully evaluated arguments
       class FunctionEvaluator < ::Checkoff::SelectorClasses::FunctionEvaluator
-        # @param selector [Array<(Symbol, Array)>,String]
+        # @param selector [Array(Symbol, Array), String]
         # @param projects [Checkoff::Projects]
         # @param portfolios [Checkoff::Portfolios]
         # @param workspaces [Checkoff::Workspaces]
+        # @param _kwargs [Hash]
         def initialize(selector:,
                        projects:,
                        portfolios:,
@@ -26,7 +27,7 @@ module Checkoff
 
         private
 
-        # @return [Array<(Symbol, Array)>]
+        # @return [Array(Symbol, Array)]
         attr_reader :selector
       end
     end

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

@@ -8,7 +8,7 @@ module Checkoff
     module Section
       # Base class to evaluate a project selector function given fully evaluated arguments
       class FunctionEvaluator < ::Checkoff::SelectorClasses::FunctionEvaluator
-        # @param selector [Array<(Symbol, Array)>,String]
+        # @param selector [Array(Symbol, Array), String]
         # @param client [Asana::Client]
         # @param sections [Checkoff::Sections]
         # @param kwargs [Hash]
@@ -24,7 +24,7 @@ module Checkoff
 
         private
 
-        # @return [Array<(Symbol, Array)>]
+        # @return [Array(Symbol, Array)]
         attr_reader :selector
 
         # @return [Asana::Client]

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

@@ -1,4 +1,4 @@
-# typed: false
+# typed: true
 # frozen_string_literal: true
 
 require_relative 'section/function_evaluator'
@@ -24,7 +24,7 @@ module Checkoff
                                          per_page: 100,
                                          options: { fields: ['resource_subtype'] })
           # @sg-ignore
-          tasks.last&.resource_subtype == 'milestone'
+          T.unsafe(tasks).last&.resource_subtype == 'milestone'
         end
       end
 

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

@@ -8,10 +8,11 @@ module Checkoff
     module Task
       # Base class to evaluate a task selector function given fully evaluated arguments
       class FunctionEvaluator < ::Checkoff::SelectorClasses::FunctionEvaluator
-        # @param selector [Array<(Symbol, Array)>,String]
+        # @param selector [Array(Symbol, Array), String]
         # @param tasks [Checkoff::Tasks]
         # @param timelines [Checkoff::Timelines]
         # @param custom_fields [Checkoff::CustomFields]
+        # @param _kwargs [Hash]
         def initialize(selector:,
                        tasks:,
                        timelines:,
@@ -26,7 +27,7 @@ module Checkoff
 
         private
 
-        # @return [Array<(Symbol, Array)>]
+        # @return [Array(Symbol, Array)]
         attr_reader :selector
       end
     end

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

@@ -298,9 +298,8 @@ module Checkoff
         end
 
         # @param task [Asana::Resources::Task]
-        # @param project_name [String]
         # @param limit_to_portfolio_gid [String, nil] If specified,
-        # only projects in this portfolio will be evaluated.
+        #   only projects in this portfolio will be evaluated.
         #
         # @return [Boolean]
         def evaluate(task, limit_to_portfolio_gid: nil)

data/lib/checkoff/internal/selector_evaluator.rb

@@ -4,8 +4,8 @@
 module Checkoff
   # Base class to evaluate Asana resource selectors against an Asana resource
   class SelectorEvaluator
-    # @param selector [Array]
-    # @return [Boolean, Object, nil]
+    # @param selector [Symbol, Array<Symbol, Integer, Array>]
+    # @return [Boolean, Object, Array, nil]
     def evaluate(selector)
       return true if selector.empty?
 

data/lib/checkoff/internal/task_hashes.rb

@@ -46,7 +46,7 @@ module Checkoff
         task_hash['unwrapped']['custom_fields'] = unwrapped_custom_fields
       end
 
-      # @param [Hash<String, Hash, Array>] task_hash
+      # @param [Hash{String => String, Hash, Array}] task_hash
       # @param [Array<Hash>] memberships
       #
       # @return [void]
@@ -54,8 +54,8 @@ module Checkoff
         return unless task_hash.key? 'assignee_section'
 
         assignee_section = task_hash.fetch('assignee_section')
-        # @type [Hash]
-        assignee = task_hash.fetch('assignee')
+        # @type [Hash{String => String}]
+        assignee = T.cast(task_hash.fetch('assignee'), T::Hash[String, String])
         memberships << {
           'section' => assignee_section.dup,
           'project' => {