trailblazer-activity 0.11.4 → 0.11.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 05447b3e4a4957abae7548719bc6b33500621fc6801860c8aaeaa06c33532c21
-  data.tar.gz: d7f55a424263aaffb829a9772fd8cff3d2e93e15691ae091a1eb72570d2136b3
+  metadata.gz: c765281f7fa2902f9cc47fb4a2045b54de89fb48ef536d4696a497f947b3f9bb
+  data.tar.gz: 8a4f24aacb4da6b80e12a3edcd3698cb7583e014b489293d29938a003a27d084
 SHA512:
-  metadata.gz: 43fc2e829e8c1c3de9493a6b1ef8dd4bacc77a1136d8efbb1f9c08d2260e381debc733df7f6d5e4e1b53bb0cefaebf76d54caceeaf786b6056f56878adc1cc6d
-  data.tar.gz: 98ff3aca3a7b3fe102ce9bd368cdfc962cc68687c432c087750f64a36d34ef258a338abc32dc2d3d228d8982a0211b5c6de8efe051dbb4576bbe740bc860d13a
+  metadata.gz: 4388f3be8e6c4f902537b5febdec0b36b5a15e143f2da5f1a9b0aba4a0cb73e0066ef825ca4f285e040e36d25ff77ce9031114f85198e70779dee9359e75df2d
+  data.tar.gz: 71c58c25da986267e69c37e50a5a1b0de0083c58cf47def17b2c63eb44af7f8498d0f4f2b121be1d46b545b7c937bb85cf0156e32d642c73750e086232b96c98

data/CHANGES.md

@@ -1,3 +1,8 @@
+# 0.11.5
+
+* Bug fix: `:output` filter from `TaskWrap::VariableMapping` wasn't returning the correct `flow_options`. If the wrapped task changed
+  its `flow_options`, the original one was still returned from the taskWrap run, not the updated one.
+
 # 0.11.4
 
 * Introduce the `config_wrap:` option in `Intermediate.call(intermediate, implementation, config_merge: {})` to allow injecting data into the activity's `:config` field.

data/lib/trailblazer/activity/task_wrap/variable_mapping.rb

@@ -73,12 +73,14 @@ class Trailblazer::Activity
       def call(wrap_ctx, original_args)
         (original_ctx, original_flow_options), original_circuit_options = original_args
 
-        returned_ctx, _ = wrap_ctx[:return_args]  # this is the Context returned from {call}ing the wrapped user task.
-        original_ctx    = wrap_ctx[@id]           # grab the original ctx from before which was set in the {:input} filter.
+        return_args = wrap_ctx[:return_args]
+
+        returned_ctx, returned_flow_options = wrap_ctx[:return_args]  # this is the Context returned from {call}ing the wrapped user task.
+        original_ctx                        = wrap_ctx[@id]           # grab the original ctx from before which was set in the {:input} filter.
         # let user compute the output.
-        output_ctx = @filter.(returned_ctx, [original_ctx, original_flow_options], original_circuit_options)
+        output_ctx = @filter.(returned_ctx, [original_ctx, returned_flow_options], original_circuit_options) # FIXME: shouldn't we pass {returned_ctx} instead of {original_ctx}?
 
-        wrap_ctx = wrap_ctx.merge( return_args: [output_ctx, original_flow_options] )
+        wrap_ctx = wrap_ctx.merge( return_args: [output_ctx, returned_flow_options] )
 
         # and then pass on the "new" context.
         return wrap_ctx, original_args

data/lib/trailblazer/activity/version.rb

@@ -1,7 +1,7 @@
 module Trailblazer
   module Version
     module Activity
-      VERSION = '0.11.4'.freeze
+      VERSION = '0.11.5'.freeze
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: trailblazer-activity
 version: !ruby/object:Gem::Version
-  version: 0.11.4
+  version: 0.11.5
 platform: ruby
 authors:
 - Nick Sutterer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-02 00:00:00.000000000 Z
+date: 2021-02-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: trailblazer-context