lyber-core 8.2.0 → 9.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36bd350f4d9c4ce74e6da529f8fe9dddd89a8f2c71daa79ccad91fdc9efa00f3
-  data.tar.gz: '0122852b4e4e24cef6844b8aef7026c75bcd9573947b6c0c002208bd61838315'
+  metadata.gz: aaaf768d39cc56a2ad9f40548a9734a1c03155a51cf5df81d8a224d3a773a562
+  data.tar.gz: 9ae8b44dc1843f6c7fda1e630851c8bd036f9537aebd610717543f0a9a4373c0
 SHA512:
-  metadata.gz: 84d64ade10a163e0092643301e0ead58e695f5506c38651e95965ebae44e128dd619990055c2b0a23cba201e4c816428d57d65f1b9db2c5955337ece3bcee148
-  data.tar.gz: 8b55154f52c0df55d7fa029ae22e1bd54747e10cb56763e0a71ffd44b1e3608e7d084f8d0c1feda15fc8a981c0d53a986b3b3ff36e5d8156fc9b391523db5c8b
+  metadata.gz: d1a9cb62f805fad5aa37ae205fcbaa8dc477d248184df92c114bef7451c05cc49bf379e2decb88b1d6a31f82a67c4fe5cf185e53ff48e97e0522f789771946bc
+  data.tar.gz: 27db999355b0bacf1e0f5bf76f33cb0abb03d197081481c9fd0ddf8dc391f5d2a8292eba43a64b07b40df196d6c1c3232f09c8509013e705a47d2f4845eaf63e

data/lib/lyber_core/robot.rb

@@ -52,13 +52,13 @@ module LyberCore
     # Calls the #perform_work method, then sets workflow to 'completed' or 'error' depending on success
     # rubocop:disable Metrics/AbcSize
     # rubocop:disable Metrics/MethodLength
-    def perform(druid, version = nil)
+    def perform(druid, version)
       @druid = druid
       @version = version
-      Honeybadger.context(druid:, process:, workflow_name:)
+      Honeybadger.context(druid:, process:, workflow_name:, version:)
 
       logger.info "#{druid} processing #{process} (#{workflow_name})"
-      return skip_for_superseded_version! if superseded_version?
+      return if skip_for_inactive_version?
       return unless check_item_queued_or_retry?
 
       # this is the default note to pass back to workflow service,
@@ -121,19 +121,15 @@ module LyberCore
       @workflow ||= Workflow.new(object_client:, workflow_name:, process:, version:)
     end
 
-    # @return [Boolean] true if a version was provided and it no longer matches the object's current version,
-    #   meaning a newer version has superseded the one this job was queued for
-    def superseded_version?
-      return false if version.nil?
+    def skip_for_inactive_version?
+      return false if workflow.active_version?
 
-      cocina_object.version.to_i != version.to_i
-    end
-
-    def skip_for_superseded_version!
-      msg = "Item #{druid} is queued for version #{version} but the current object version is " \
-            "#{cocina_object.version}. Skipping #{process} (#{workflow_name})."
+      msg = "Item #{druid} is queued for version #{version} of #{process} (#{workflow_name}), " \
+            'but that is not the active version. Skipping.'
       logger.warn(msg)
       workflow.skip!(msg)
+
+      true
     end
 
     def check_item_queued_or_retry? # rubocop:disable Metrics/AbcSize

data/lib/lyber_core/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LyberCore
-  VERSION = '8.2.0'
+  VERSION = '9.0.0'
 end

data/lib/lyber_core/workflow.rb

@@ -30,6 +30,11 @@ module LyberCore
       @process_response ||= workflow_response.process_for_recent_version(name: process)
     end
 
+    # @return [Boolean] true if this workflow step is for the active version of the workflow
+    def active_version?
+      workflow_response.process_for(name: process, version:).active_version?
+    end
+
     def start!(note)
       workflow_process.update(status: 'started', elapsed: 1.0, note:, version:)
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lyber-core
 version: !ruby/object:Gem::Version
-  version: 8.2.0
+  version: 9.0.0
 platform: ruby
 authors:
 - Alpana Pande