User metadata (#223)

Fixes #217

Author: cretz

temporalio/Rakefile

@@ -94,7 +94,7 @@ Rake::Task[:build].enhance([:copy_parent_files]) do
 end
 
 task :rust_lint do
-  sh 'cargo', 'clippy', '--', '-Dwarnings'
+  sh 'cargo', 'clippy', '-p', 'temporalio_bridge', '--no-deps', '--', '-Dwarnings'
   sh 'cargo', 'fmt', '--check'
 end
 

temporalio/extra/proto_gen.rb

@@ -95,6 +95,7 @@ def generate_import_helper_files
         require 'temporalio/api/errordetails/v1/message'
         require 'temporalio/api/export/v1/message'
         require 'temporalio/api/operatorservice'
+        require 'temporalio/api/sdk/v1/workflow_metadata'
         require 'temporalio/api/workflowservice'
 
         module Temporalio

temporalio/lib/temporalio/api.rb

@@ -5,6 +5,7 @@
 require 'temporalio/api/errordetails/v1/message'
 require 'temporalio/api/export/v1/message'
 require 'temporalio/api/operatorservice'
+require 'temporalio/api/sdk/v1/workflow_metadata'
 require 'temporalio/api/workflowservice'
 
 module Temporalio

temporalio/lib/temporalio/client.rb

@@ -194,6 +194,12 @@ def operator_service
     # @param args [Array<Object>] Arguments to the workflow.
     # @param id [String] Unique identifier for the workflow execution.
     # @param task_queue [String] Task queue to run the workflow on.
+    # @param static_summary [String, nil] Fixed single-line summary for this workflow execution that may appear in
+    #   CLI/UI. This can be in single-line Temporal markdown format. This is currently experimental.
+    # @param static_details [String, nil] Fixed details for this workflow execution that may appear in CLI/UI. This can
+    #   be in Temporal markdown format and can be multiple lines. This is a fixed value on the workflow that cannot be
+    #   updated. For details that can be updated, use {Workflow.current_details=} within the workflow. This is currently
+    #   experimental.
     # @param execution_timeout [Float, nil] Total workflow execution timeout in seconds including retries and continue
     #   as new.
     # @param run_timeout [Float, nil] Timeout of a single workflow run in seconds.
@@ -220,6 +226,8 @@ def start_workflow(
       *args,
       id:,
       task_queue:,
+      static_summary: nil,
+      static_details: nil,
       execution_timeout: nil,
       run_timeout: nil,
       task_timeout: nil,
@@ -238,6 +246,8 @@ def start_workflow(
                              args:,
                              workflow_id: id,
                              task_queue:,
+                             static_summary:,
+                             static_details:,
                              execution_timeout:,
                              run_timeout:,
                              task_timeout:,
@@ -260,6 +270,12 @@ def start_workflow(
     # @param args [Array<Object>] Arguments to the workflow.
     # @param id [String] Unique identifier for the workflow execution.
     # @param task_queue [String] Task queue to run the workflow on.
+    # @param static_summary [String, nil] Fixed single-line summary for this workflow execution that may appear in
+    #   CLI/UI. This can be in single-line Temporal markdown format. This is currently experimental.
+    # @param static_details [String, nil] Fixed details for this workflow execution that may appear in CLI/UI. This can
+    #   be in Temporal markdown format and can be multiple lines. This is a fixed value on the workflow that cannot be
+    #   updated. For details that can be updated, use {Workflow.current_details=} within the workflow. This is currently
+    #   experimental.
     # @param execution_timeout [Float, nil] Total workflow execution timeout in seconds including retries and continue
     #   as new.
     # @param run_timeout [Float, nil] Timeout of a single workflow run in seconds.
@@ -287,6 +303,8 @@ def execute_workflow(
       *args,
       id:,
       task_queue:,
+      static_summary: nil,
+      static_details: nil,
       execution_timeout: nil,
       run_timeout: nil,
       task_timeout: nil,
@@ -305,6 +323,8 @@ def execute_workflow(
         *args,
         id:,
         task_queue:,
+        static_summary:,
+        static_details:,
         execution_timeout:,
         run_timeout:,
         task_timeout:,

temporalio/lib/temporalio/client/interceptor.rb

@@ -23,6 +23,8 @@ def intercept_client(next_interceptor)
         :args,
         :workflow_id,
         :task_queue,
+        :static_summary,
+        :static_details,
         :execution_timeout,
         :run_timeout,
         :task_timeout,

temporalio/lib/temporalio/client/schedule.rb

@@ -167,6 +167,8 @@ def _to_proto(data_converter)
           :args,
           :id,
           :task_queue,
+          :static_summary,
+          :static_details,
           :execution_timeout,
           :run_timeout,
           :task_timeout,
@@ -186,6 +188,14 @@ def _to_proto(data_converter)
         #   @return [String] Unique identifier for the workflow execution.
         # @!attribute task_queue
         #   @return [String] Task queue to run the workflow on.
+        # @!attribute static_summary
+        #   @return [String, nil] Fixed single-line summary for this workflow execution that may appear in CLI/UI.
+        #     This can be in single-line Temporal markdown format. This is currently experimental.
+        # @!attribute static_details
+        #   @return [String, nil] Fixed details for this workflow execution that may appear in CLI/UI. This can be in
+        #     Temporal markdown format and can be multiple lines. This is a fixed value on the workflow that cannot be
+        #     updated. For details that can be updated, use {Workflow.current_details=} within the workflow. This is
+        #     currently experimental.
         # @!attribute execution_timeout
         #   @return [Float, nil] Total workflow execution timeout in seconds including retries and continue as new.
         # @!attribute run_timeout
@@ -212,6 +222,12 @@ class << self
             # @param args [Array<Object>] Arguments to the workflow.
             # @param id [String] Unique identifier for the workflow execution.
             # @param task_queue [String] Task queue to run the workflow on.
+            # @param static_summary [String, nil] Fixed single-line summary for this workflow execution that may appear
+            #   in CLI/UI. This can be in single-line Temporal markdown format. This is currently experimental.
+            # @param static_details [String, nil] Fixed details for this workflow execution that may appear in CLI/UI.
+            #   This can be in Temporal markdown format and can be multiple lines. This is a fixed value on the workflow
+            #   that cannot be updated. For details that can be updated, use {Workflow.current_details=} within the
+            #   workflow. This is currently experimental.
             # @param execution_timeout [Float, nil] Total workflow execution timeout in seconds including retries and
             #   continue as new.
             # @param run_timeout [Float, nil] Timeout of a single workflow run in seconds.
@@ -225,6 +241,8 @@ def new(
               *args,
               id:,
               task_queue:,
+              static_summary: nil,
+              static_details: nil,
               execution_timeout: nil,
               run_timeout: nil,
               task_timeout: nil,
@@ -238,6 +256,8 @@ def new(
                 args:,
                 id:,
                 task_queue:,
+                static_summary:,
+                static_details:,
                 execution_timeout:,
                 run_timeout:,
                 task_timeout:,
@@ -251,11 +271,14 @@ def new(
 
           # @!visibility private
           def self._from_proto(raw_info, data_converter)
+            (summary, details) = Internal::ProtoUtils.from_user_metadata(raw_info.user_metadata, data_converter)
             StartWorkflow.new(
               raw_info.workflow_type.name,
               *data_converter.from_payloads(raw_info.input),
               id: raw_info.workflow_id,
               task_queue: raw_info.task_queue.name,
+              static_summary: summary,
+              static_details: details,
               execution_timeout: Internal::ProtoUtils.duration_to_seconds(raw_info.workflow_execution_timeout),
               run_timeout: Internal::ProtoUtils.duration_to_seconds(raw_info.workflow_run_timeout),
               task_timeout: Internal::ProtoUtils.duration_to_seconds(raw_info.workflow_task_timeout),
@@ -280,7 +303,8 @@ def _to_proto(data_converter)
                 retry_policy: retry_policy&._to_proto,
                 memo: Internal::ProtoUtils.memo_to_proto(memo, data_converter),
                 search_attributes: search_attributes&._to_proto,
-                header: Internal::ProtoUtils.headers_to_proto(headers, data_converter)
+                header: Internal::ProtoUtils.headers_to_proto(headers, data_converter),
+                user_metadata: Internal::ProtoUtils.to_user_metadata(static_summary, static_details, data_converter)
               )
             )
           end

temporalio/lib/temporalio/client/with_start_workflow_operation.rb

@@ -12,6 +12,8 @@ class WithStartWorkflowOperation
         :args,
         :id,
         :task_queue,
+        :static_summary,
+        :static_details,
         :execution_timeout,
         :run_timeout,
         :task_timeout,
@@ -41,6 +43,8 @@ def initialize(
         *args,
         id:,
         task_queue:,
+        static_summary: nil,
+        static_details: nil,
         execution_timeout: nil,
         run_timeout: nil,
         task_timeout: nil,
@@ -58,6 +62,8 @@ def initialize(
           args:,
           id:,
           task_queue:,
+          static_summary:,
+          static_details:,
           execution_timeout:,
           run_timeout:,
           task_timeout:,

temporalio/lib/temporalio/client/workflow_execution.rb

@@ -93,6 +93,25 @@ class Description < WorkflowExecution
         def initialize(raw_description, data_converter)
           super(raw_description.workflow_execution_info, data_converter)
           @raw_description = raw_description
+          @data_converter = data_converter
+        end
+
+        # @return [String, nil] Static summary configured on the workflow. This is currently experimental.
+        def static_summary
+          user_metadata.first
+        end
+
+        # @return [String, nil] Static details configured on the workflow. This is currently experimental.
+        def static_details
+          user_metadata.last
+        end
+
+        private
+
+        def user_metadata
+          @user_metadata ||= Internal::ProtoUtils.from_user_metadata(
+            @raw_description.execution_config&.user_metadata, @data_converter
+          )
         end
       end
     end

temporalio/lib/temporalio/internal/client/implementation.rb

@@ -69,6 +69,9 @@ def start_workflow(input)
             search_attributes: input.search_attributes&._to_proto,
             workflow_start_delay: ProtoUtils.seconds_to_duration(input.start_delay),
             request_eager_execution: input.request_eager_start,
+            user_metadata: ProtoUtils.to_user_metadata(
+              input.static_summary, input.static_details, @client.data_converter
+            ),
             header: ProtoUtils.headers_to_proto(input.headers, @client.data_converter)
           )
 
@@ -319,6 +322,9 @@ def _start_workflow_request_from_with_start_options(klass, start_options)
             memo: ProtoUtils.memo_to_proto(start_options.memo, @client.data_converter),
             search_attributes: start_options.search_attributes&._to_proto,
             workflow_start_delay: ProtoUtils.seconds_to_duration(start_options.start_delay),
+            user_metadata: ProtoUtils.to_user_metadata(
+              start_options.static_summary, start_options.static_details, @client.data_converter
+            ),
             header: ProtoUtils.headers_to_proto(start_options.headers, @client.data_converter)
           )
         end

temporalio/lib/temporalio/internal/proto_utils.rb

@@ -122,6 +122,22 @@ def self.reserved_name?(name)
         name.start_with?('__temporal_') || name == '__stack_trace' || name == '__enhanced_stack_trace'
       end
 
+      def self.to_user_metadata(summary, details, converter)
+        return nil if (!summary || summary.empty?) && (!details || details.empty?)
+
+        metadata = Temporalio::Api::Sdk::V1::UserMetadata.new
+        metadata.summary = converter.to_payload(summary) if summary && !summary.empty?
+        metadata.details = converter.to_payload(details) if details && !details.empty?
+        metadata
+      end
+
+      def self.from_user_metadata(metadata, converter)
+        [
+          (converter.from_payload(metadata.summary) if metadata&.summary), #: String?
+          (converter.from_payload(metadata.details) if metadata&.details) #: String?
+        ]
+      end
+
       class LazyMemo
         def initialize(raw_memo, converter)
           @raw_memo = raw_memo