checkoff 0.148.0 → 0.150.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 59a19680baee0ffb1a81101fe640019611e61fcb2f19928a7438a720bc2cc577
-  data.tar.gz: 8cdc27dce1e2e1d171ef3d8bb1b61f4978f8dddf53fbf477c31881eed36e81c8
+  metadata.gz: ff26e842716595b6b388bce568c5e65646535aa4ea7a88db6e375da53b0dd108
+  data.tar.gz: c7195465fa94bea1970610c90314a966cea5523fcd285693a5f66c4906d1a818
 SHA512:
-  metadata.gz: 298bf510b836223c6bb61424011c019cc54ab600c08baba89db73b5166b3ffabb74e1f0cbdf4faccc31cf70439411987445763b62c7422d3d46afc97508d506d
-  data.tar.gz: fcd68e91155a1058af51f3d75e32064d0afc9f45be9277d3cf3a4bf53e8df966acfadbb6f440330899e879dcc2e85348eced0e29fbe4cf7819a55214b980be7d
+  metadata.gz: cd5fec3fc1dfe9e9237fb887fc069f783ed1d321c96e0ff5a031d9e377a7d34d27c06f385884062a0f11284f96d5b88e50c9ab9dc82e570805136a902f864120
+  data.tar.gz: 24fbfffbf84845740afa19d6d4634d4deff06e86b9eadd23b1e978fb5a7be785c504e5cfab668903e0acfecb6454dc1d5c8c7fb75c604b3dfe3cb023746af0de

data/Gemfile.lock

@@ -12,7 +12,7 @@ GIT
 PATH
   remote: .
   specs:
-    checkoff (0.148.0)
+    checkoff (0.150.0)
       activesupport
       asana (> 0.10.0)
       cache_method

data/lib/checkoff/projects.rb

@@ -80,7 +80,7 @@ module Checkoff
         end
       end
     end
-    cache_method :project, LONG_CACHE_TIME
+    cache_method :project, REALLY_LONG_CACHE_TIME
 
     # @param workspace_name [String]
     # @param project_name [String,Symbol<:my_tasks>]
@@ -93,7 +93,7 @@ module Checkoff
 
       p
     end
-    cache_method :project_or_raise, LONG_CACHE_TIME
+    cache_method :project_or_raise, REALLY_LONG_CACHE_TIME
 
     # @param gid [String]
     # @param [Array<String>] extra_fields
@@ -102,7 +102,7 @@ module Checkoff
     def project_by_gid(gid, extra_fields: [])
       projects.find_by_id(gid, options: { fields: %w[name] + extra_fields })
     end
-    cache_method :project_by_gid, LONG_CACHE_TIME
+    cache_method :project_by_gid, REALLY_LONG_CACHE_TIME
 
     # find uncompleted tasks in a list
     # @param [Enumerable<Asana::Resources::Task>] tasks
@@ -133,10 +133,12 @@ module Checkoff
     def projects_by_workspace_name(workspace_name, extra_fields: [])
       workspace = @workspaces.workspace_or_raise(workspace_name)
       options = { fields: %w[name] + extra_fields }
-      projects.find_by_workspace(workspace: workspace.gid, per_page: 100, options: options)
+      # 15 minute cache resulted in 'Your pagination token has
+      # expired', so let's cache this a super long time and force
+      # evaluation
+      projects.find_by_workspace(workspace: workspace.gid, per_page: 100, options: options).to_a
     end
-    # 15 minute cache resulted in 'Your pagination token has expired'
-    cache_method :projects_by_workspace_name, MEDIUM_CACHE_TIME
+    cache_method :projects_by_workspace_name, REALLY_LONG_CACHE_TIME
 
     # @param project_obj [Asana::Resources::Project]
     # @param project [String, Symbol<:not_specified, :my_tasks>]
@@ -190,6 +192,7 @@ module Checkoff
     cache_method :projects, LONG_CACHE_TIME
 
     # @param [String] workspace_name
+    #
     # @return [Asana::Resources::Project]
     def my_tasks(workspace_name)
       workspace = @workspaces.workspace_or_raise(workspace_name)

data/lib/checkoff/tasks.rb

@@ -8,6 +8,7 @@ require_relative 'workspaces'
 require_relative 'internal/config_loader'
 require_relative 'internal/task_timing'
 require_relative 'internal/task_hashes'
+require_relative 'internal/logging'
 require 'asana'
 
 module Checkoff
@@ -16,6 +17,8 @@ module Checkoff
     # @!parse
     #   extend CacheMethod::ClassMethods
 
+    include Logging
+
     MINUTE = 60
     HOUR = MINUTE * 60
     DAY = 24 * HOUR
@@ -167,6 +170,8 @@ module Checkoff
 
     # True if any of the task's dependencies are marked incomplete
     #
+    # Include 'dependencies.gid' in extra_fields of task passed in.
+    #
     # @param task [Asana::Resources::Task]
     def incomplete_dependencies?(task)
       # Avoid a redundant fetch.  Unfortunately, Ruby SDK allows
@@ -179,8 +184,7 @@ module Checkoff
 
       # @sg-ignore
       # @type [Enumerable<Asana::Resources::Task>, nil]
-      dependencies = task.instance_variable_get(:@dependencies)
-      dependencies = task.dependencies.map { |dependency| { 'gid' => dependency.gid } } if dependencies.nil?
+      dependencies = task.instance_variable_get(:@dependencies) || []
 
       dependencies.any? do |parent_task_info|
         # the real bummer though is that asana doesn't let you fetch
@@ -188,20 +192,36 @@ module Checkoff
         # regardless:
         parent_task_gid = parent_task_info.fetch('gid')
 
-        parent_task = task_by_gid(parent_task_gid,
-                                  only_uncompleted: false)
+        parent_task = task_by_gid(parent_task_gid, only_uncompleted: false)
         parent_task.completed_at.nil?
       end
     end
 
     # @param task [Asana::Resources::Task]
     #
-    # @return [Array<Asana::Resources::Task>]
+    # @return [Array<Hash>]
     def all_dependent_tasks(task)
       dependent_tasks = []
-      task.dependents.each do |dependent_task|
-        dependent_tasks << dependent_task
-        dependent_tasks += all_dependent_tasks(dependent_task)
+      # See note above - same applies as does in @dependencies
+      #
+      # @type [Array<Hash>]
+      dependents = task.instance_variable_get(:@dependents) || []
+      dependents.each do |dependent_task_hash_or_obj|
+        # seems like if we ever .inspect the task, it stashes the task
+        # object instead of the hash.  Try to avoid this - but maybe we
+        # need to convert if it does happen.
+        raise 'Found dependent task object!' if dependent_task_hash_or_obj.is_a?(Asana::Resources::Task)
+
+        dependent_task_hash = dependent_task_hash_or_obj
+
+        dependent_task = task_by_gid(dependent_task_hash.fetch('gid'),
+                                     only_uncompleted: true,
+                                     extra_fields: ['dependents'])
+        debug { "#{task.name} has dependent task #{dependent_task.name}" }
+        unless dependent_task.nil?
+          dependent_tasks << dependent_task
+          dependent_tasks += all_dependent_tasks(dependent_task)
+        end
       end
       dependent_tasks
     end
@@ -248,6 +268,11 @@ module Checkoff
       end
     end
 
+    # @return [Hash]
+    def as_cache_key
+      {}
+    end
+
     private
 
     # @return [Checkoff::Internal::TaskTiming]

data/lib/checkoff/timelines.rb

@@ -69,7 +69,7 @@ module Checkoff
                                                               limit_to_portfolio_name)
       end
 
-      all_dependent_task_gids = @tasks.all_dependent_tasks(task).map(&:gid)
+      all_dependent_task_gids = nil
       task.memberships.all? do |membership_data|
         unless limit_to_portfolio_name.nil?
           project_gid = membership_data.fetch('project').fetch('gid')
@@ -86,6 +86,8 @@ module Checkoff
 
         next true if last_milestone.gid == task.gid
 
+        all_dependent_task_gids ||= @tasks.all_dependent_tasks(task).map(&:gid)
+
         all_dependent_task_gids.include? last_milestone.gid
       end
     end

data/lib/checkoff/version.rb

@@ -3,5 +3,5 @@
 # Command-line and gem client for Asana (unofficial)
 module Checkoff
   # Version of library
-  VERSION = '0.148.0'
+  VERSION = '0.150.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: checkoff
 version: !ruby/object:Gem::Version
-  version: 0.148.0
+  version: 0.150.0
 platform: ruby
 authors:
 - Vince Broz
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-12-05 00:00:00.000000000 Z
+date: 2023-12-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport