tasker-rb 0.1.7 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a8c5ed3385bea52ede9378bb4cd2f35c4fb03e285a0b811698fa34042b2a0c8a
-  data.tar.gz: a9ce3c0d4d390ae8f0263ca60794c8a0997d99ca6272c52ccd7a8cea579de6ae
+  metadata.gz: efbbc9f01809d4ac82732ea8b7d7f906f57fb939879c78f3ca448447211abd21
+  data.tar.gz: 7b7b1e5a38af3a78d5f090a725c5db85c49e4193e12440ed9a9b802972718112
 SHA512:
-  metadata.gz: d7957c47e7f5403a0adc3cbfc917b49196e44e21438276f806809662b0bd3abaa2ff14408e69233dd06e1521fece83cc0f320d3601d152e8ee69730212f3ca58
-  data.tar.gz: e7844fe1ee414c7f3887db916b084c15ac44768885516792964bce9afff6cd807a8b847335428cb367fb7e0042730b8bc95a098e89cc7a24851ca0b96bc8b5db
+  metadata.gz: 05f12f661859b22dd1446bd6c5b162523c74f5640cbeeff1e4027416c07bca1d05160fd6700405d54453b34bc0acab5bad0b4e156e89bb7e1510462593b00ff1
+  data.tar.gz: 418b0a95010e8b500630769702357c63fcbba996918b35aace2b65364b73d6126a7b34cd7282e97c82e4b802be31b30ea2dac3f92f1faa08560d2cf6e208df26

data/ext/tasker_core/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "tasker-rb"
-version = "0.1.5"
+version = "0.1.6"
 edition = "2021"
 description = "Ruby bindings for tasker-core: High-performance workflow orchestration"
 readme = "../../../../README.md"
@@ -66,8 +66,8 @@ sqlx = { version = "0.8", features = [
 # Both path (for workspace builds) and version (for standalone/published builds).
 # cargo publish strips the path field, leaving only the version for crates.io resolution.
 # Version pins are updated by scripts/release/update-versions.sh during release-prepare.
-tasker-shared = { path = "../../../../tasker-shared", version = "=0.1.5" }
-tasker-worker = { path = "../../../../tasker-worker", version = "=0.1.5" }
+tasker-shared = { path = "../../../../tasker-shared", version = "=0.1.6" }
+tasker-worker = { path = "../../../../tasker-worker", version = "=0.1.6" }
 # Error handling
 thiserror = "2.0"
 # Async runtime for blocking on futures in FFI
@@ -80,7 +80,7 @@ uuid = { version = "1.11", features = ["serde", "v4", "v7"] }
 workspace_tools = { version = "0.11.0", features = ["full"] }
 
 [dev-dependencies]
-tasker-core = { package = "tasker-core", path = "../../../../", version = "=0.1.5" }
+tasker-core = { package = "tasker-core", path = "../../../../", version = "=0.1.6" }
 
 [features]
 default = []

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: ruby
 authors:
 - Pete Taylor
@@ -201,7 +201,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:
@@ -293,15 +293,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