tasker-rb 0.1.7-x86_64-linux → 0.1.8-x86_64-linux

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ef76abda3b0f0b478e4eaa31e4dfda4730971f67f32834bb94529af8139a7578
-  data.tar.gz: 3b6abe2f8fe5560df69c4339b53cd2f7226b4bafcb9457e6a598bf594818e700
+  metadata.gz: 94ad33456f400b47ae52c1ed366515e8b689829a1242cf7e7552bf12f9851957
+  data.tar.gz: 8e88e9b23228defefdd04878dcfcbe23873ad3d40a986936d82388d827a6c7b0
 SHA512:
-  metadata.gz: b818d34abcfcdbe664844373dc0a28efc48df10a29858553288d65371d2def98532d5e189b453c4582d173f0e1d169c996e16271ddfb8f1952b5592f2eedbfe8
-  data.tar.gz: afec9ba13cf2f9e9957ab521baed47f865f12af391d4cc1a9545f295c7b45f93b296048614d8c22f37e8caaebfd5bfef977d4105a5d4b1a79e599e1a253c0bd8
+  metadata.gz: 10c1052fd139cb8207884b83c63f4dd891a37ad86017b7eaf087666d065bc70774787da888ae7eb8c7cdcb6ea55546e180eb181145618fb740a98d78c8b7e92b
+  data.tar.gz: 0e8e84861132fa0bc80631a65dc21ea5c075f8e5f86a229c80c162ad9b6ae68637b4fc0ba41aa503c0968d3365130571848373e972b60603099bf36a8ac89184

data/lib/tasker_core/event_bridge.rb

@@ -194,21 +194,54 @@ module TaskerCore
 
       # Send completion event back to Rust
       # Called by StepExecutionSubscriber after handler execution
+      #
+      # serde_magnus requires string keys for deserialization into Rust structs.
+      # Ruby hashes built with symbol literals (e.g. { success: true }) must be
+      # deep-stringified before crossing the FFI boundary.
+      #
+      # If the FFI call still fails (e.g. unexpected nil, type mismatch), the
+      # fallback path constructs a guaranteed-safe failure result so the step is
+      # marked as permanently failed rather than silently lost.
       def publish_step_completion(completion_data)
         return unless active?
 
-        logger.debug "Sending step completion to Rust: #{completion_data[:event_id]}"
+        event_id_str = completion_data[:event_id].to_s
+        logger.debug "Sending step completion to Rust: #{event_id_str}"
 
         # Validate completion data
         validate_completion!(completion_data)
 
-        # Send to Rust via FFI (TAS-67: complete_step_event takes event_id and completion_data)
-        TaskerCore::FFI.complete_step_event(completion_data[:event_id].to_s, completion_data)
-
-        # Also publish locally for monitoring/debugging
-        publish('step.completion.sent', completion_data)
-
-        logger.debug 'Step completion sent to Rust'
+        # serde_magnus expects string keys — convert symbol keys at the FFI boundary
+        stringified = completion_data.deep_stringify_keys
+
+        begin
+          TaskerCore::FFI.complete_step_event(event_id_str, stringified)
+          # Only publish monitoring event after successful primary FFI
+          publish('step.completion.sent', completion_data)
+          logger.debug 'Step completion sent to Rust'
+        rescue StandardError => e
+          logger.error "FFI serialization failed for event #{event_id_str}: #{e.message}"
+          logger.error e.backtrace&.first(5)&.join("\n")
+
+          # Submit a minimal failure result that is guaranteed to deserialize
+          fallback = build_ffi_safe_failure(completion_data, e)
+          begin
+            TaskerCore::FFI.complete_step_event(event_id_str, fallback)
+            logger.warn "FFI fallback failure submitted for event #{event_id_str}"
+            begin
+              publish('step.completion.sent', fallback)
+            rescue StandardError => pub_err
+              logger.warn "Monitoring publish failed after fallback: #{pub_err.message}"
+            end
+          rescue StandardError => fallback_error
+            logger.error "FFI fallback also failed for event #{event_id_str}: " \
+                         "#{fallback_error.message} (original error: #{e.message})"
+            raise fallback_error
+          end
+        end
+      rescue ArgumentError
+        # Validation errors from validate_completion! should propagate
+        raise
       rescue StandardError => e
         logger.error "Failed to send step completion: #{e.message}"
         logger.error e.backtrace.join("\n")
@@ -246,10 +279,13 @@ module TaskerCore
         # Validate checkpoint data
         validate_checkpoint_yield!(checkpoint_data)
 
+        # serde_magnus expects string keys — convert symbol keys at the FFI boundary
+        stringified = checkpoint_data.deep_stringify_keys
+
         # Send to Rust via FFI (TAS-125)
         success = TaskerCore::FFI.checkpoint_yield_step_event(
           checkpoint_data[:event_id].to_s,
-          checkpoint_data
+          stringified
         )
 
         if success
@@ -311,6 +347,34 @@ module TaskerCore
         completion_data[:completed_at] ||= Time.now.utc.iso8601
       end
 
+      # Build a minimal StepExecutionResult-shaped hash that is guaranteed to
+      # deserialize through serde_magnus. Uses only string keys and primitive
+      # values so there is zero chance of a secondary serialization failure.
+      def build_ffi_safe_failure(original_data, error)
+        {
+          'step_uuid' => original_data[:step_uuid].to_s,
+          'task_uuid' => original_data[:task_uuid].to_s,
+          'success' => false,
+          'result' => {},
+          'status' => 'error',
+          'metadata' => {
+            'execution_time_ms' => 0,
+            'retryable' => false,
+            'completed_at' => Time.now.utc.iso8601,
+            'worker_id' => 'ruby_worker',
+            'custom' => {
+              'ffi_serialization_error' => error.message.to_s[0, 500],
+              'original_success' => original_data[:success].to_s
+            }
+          },
+          'error' => {
+            'message' => "FFI serialization failed: #{error.message}"[0, 500],
+            'error_type' => 'FFI_SERIALIZATION_ERROR',
+            'retryable' => false
+          }
+        }
+      end
+
       # TAS-125: Validate checkpoint yield data before sending to Rust
       def validate_checkpoint_yield!(checkpoint_data)
         required_fields = %i[event_id step_uuid cursor items_processed]

data/lib/tasker_core/version.rb

@@ -3,7 +3,7 @@
 module TaskerCore
   # Version synchronization with the core Rust crate
   # This should be kept in sync with the Cargo.toml version
-  VERSION = '0.1.7'
+  VERSION = '0.1.8'
 
   def self.version_info
     {

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tasker-rb
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
 platform: x86_64-linux
 authors:
 - Pete Taylor
@@ -187,7 +187,7 @@ description: |
   Ruby FFI bindings for tasker-core, providing 10-100x performance improvements
   for workflow orchestration, dependency resolution, and state management.
 
-  This gem enables Rails applications using the Tasker engine to leverage
+  This gem enables Ruby applications using the Tasker engine to leverage
   Rust's performance for computationally intensive orchestration operations
   while maintaining Ruby's flexibility for business logic.
 email:
@@ -263,15 +263,14 @@ licenses:
 metadata:
   homepage_uri: https://github.com/tasker-systems/tasker-core
   source_code_uri: https://github.com/tasker-systems/tasker-core/tree/main/workers/ruby
-  changelog_uri: https://github.com/tasker-systems/tasker-core/blob/main/workers/ruby/CHANGELOG.md
-  documentation_uri: https://github.com/tasker-systems/tasker-core/blob/main/docs/RUBY.md
+  changelog_uri: https://github.com/tasker-systems/tasker-core/blob/main/CHANGELOG.md
+  documentation_uri: https://docs.tasker.systems
   bug_tracker_uri: https://github.com/tasker-systems/tasker-core/issues
   allowed_push_host: https://rubygems.org
   rubygems_mfa_required: 'true'
 post_install_message: "\n\U0001F980 tasker-rb successfully installed!\n\nThis gem
   provides high-performance Rust-powered workflow orchestration.\n\nDocumentation:
-  https://github.com/tasker-systems/tasker-core/blob/main/docs/workers/ruby.md\n\nFor
-  Rails integration, see the tasker-engine gem documentation.\n\n"
+  https://docs.tasker.systems\n\n"
 rdoc_options: []
 require_paths:
 - lib