ruby_llm-agents 0.3.3 → 0.3.5

data/app/views/{rubyllm → ruby_llm}/agents/executions/_filters.html.erb

@@ -3,13 +3,15 @@
   selected_agents = params[:agent_types].present? ? (params[:agent_types].is_a?(Array) ? params[:agent_types] : params[:agent_types].split(",")) : []
   selected_statuses = params[:statuses].present? ? (params[:statuses].is_a?(Array) ? params[:statuses] : params[:statuses].split(",")) : []
   selected_models = params[:model_ids].present? ? (params[:model_ids].is_a?(Array) ? params[:model_ids] : params[:model_ids].split(",")) : []
+  selected_workflows = params[:workflow_types].present? ? (params[:workflow_types].is_a?(Array) ? params[:workflow_types] : params[:workflow_types].split(",")) : []
 
-  has_filters = selected_agents.any? || selected_statuses.any? || params[:days].present? || selected_models.any?
+  has_filters = selected_agents.any? || selected_statuses.any? || params[:days].present? || selected_models.any? || selected_workflows.any?
   active_filter_count = [
     selected_agents.any? ? 1 : 0,
     selected_statuses.any? ? 1 : 0,
     params[:days].present? ? 1 : 0,
-    selected_models.any? ? 1 : 0
+    selected_models.any? ? 1 : 0,
+    selected_workflows.any? ? 1 : 0
   ].sum
 
   status_options = [
@@ -33,11 +35,23 @@
     short_name = m.split("/").last.split(":").first # Handle "provider/model:version" format
     { value: m, label: short_name }
   end || []
+
+  # Build workflow type options with icons
+  workflow_type_labels = {
+    "pipeline" => { label: "Pipeline", icon: "→" },
+    "parallel" => { label: "Parallel", icon: "⫴" },
+    "router" => { label: "Router", icon: "⑂" }
+  }
+  workflow_options = [{ value: "single", label: "Single Agent" }]
+  workflow_options += (local_assigns[:workflow_types] || []).map do |wt|
+    info = workflow_type_labels[wt] || { label: wt.titleize, icon: "" }
+    { value: wt, label: "#{info[:icon]} #{info[:label]}".strip }
+  end
 %>
 <div id="filters-container" class="bg-white dark:bg-gray-800 rounded-xl border border-gray-200 dark:border-gray-700 p-4 mb-6"
      x-data="{ mobileOpen: false }">
   <%= form_with url: ruby_llm_agents.executions_path, method: :get,
-                data: { turbo_stream: true, turbo_action: "advance" },
+                local: true,
                 id: "filters-form" do |f| %>
 
     <%# Mobile: Toggle button (only shows on small screens) %>
@@ -71,7 +85,7 @@
       <div class="flex flex-col md:flex-row md:items-center gap-3">
         <%# Status Filter %>
         <div class="md:w-auto">
-          <%= render "rubyllm/agents/shared/filter_dropdown",
+          <%= render "ruby_llm/agents/shared/filter_dropdown",
               name: "statuses[]",
               filter_id: "statuses",
               label: "Status",
@@ -84,7 +98,7 @@
 
         <%# Time Range Filter %>
         <div class="md:w-auto">
-          <%= render "rubyllm/agents/shared/select_dropdown",
+          <%= render "ruby_llm/agents/shared/select_dropdown",
               name: "days",
               filter_id: "days",
               options: time_options,
@@ -97,7 +111,7 @@
         <%# Model Filter %>
         <% if model_options.any? %>
           <div class="md:w-auto">
-            <%= render "rubyllm/agents/shared/filter_dropdown",
+            <%= render "ruby_llm/agents/shared/filter_dropdown",
                 name: "model_ids[]",
                 filter_id: "model_ids",
                 label: "Model",
@@ -112,7 +126,7 @@
         <%# Agent Types Filter %>
         <% if agent_types.any? %>
           <div class="md:w-auto">
-            <%= render "rubyllm/agents/shared/filter_dropdown",
+            <%= render "ruby_llm/agents/shared/filter_dropdown",
                 name: "agent_types[]",
                 filter_id: "agent_types",
                 label: "Agents",
@@ -125,6 +139,22 @@
           </div>
         <% end %>
 
+        <%# Workflow Type Filter %>
+        <% if workflow_options.length > 1 %>
+          <div class="md:w-auto">
+            <%= render "ruby_llm/agents/shared/filter_dropdown",
+                name: "workflow_types[]",
+                filter_id: "workflow_types",
+                label: "Type",
+                all_label: "All Types",
+                options: workflow_options,
+                selected: selected_workflows,
+                icon: "M4 5a1 1 0 011-1h14a1 1 0 011 1v2a1 1 0 01-1 1H5a1 1 0 01-1-1V5zm0 8a1 1 0 011-1h6a1 1 0 011 1v6a1 1 0 01-1 1H5a1 1 0 01-1-1v-6zm12 0a1 1 0 011-1h2a1 1 0 011 1v6a1 1 0 01-1 1h-2a1 1 0 01-1-1v-6z",
+                width: "w-44",
+                full_width: true %>
+          </div>
+        <% end %>
+
         <%# Spacer (desktop only) %>
         <div class="hidden md:block flex-1"></div>
 
@@ -144,7 +174,6 @@
         <div class="flex items-center gap-2 md:gap-1">
           <% if has_filters %>
             <%= link_to ruby_llm_agents.executions_path,
-                data: { turbo_stream: true, turbo_action: "advance" },
                 class: "flex-1 md:flex-initial flex items-center justify-center gap-2 px-3 py-2 md:p-2 text-sm md:text-base text-red-600 md:text-gray-400 dark:text-red-400 md:dark:text-gray-400 bg-red-50 md:bg-transparent dark:bg-red-900/20 md:dark:bg-transparent hover:text-red-500 hover:bg-red-100 md:hover:bg-red-50 dark:hover:bg-red-900/30 md:dark:hover:bg-red-900/20 rounded-lg transition-colors",
                 title: "Clear filters" do %>
               <svg class="w-4 h-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
@@ -153,10 +182,9 @@
               <span class="md:hidden">Clear</span>
             <% end %>
           <% end %>
-          <%= link_to ruby_llm_agents.export_executions_path(agent_types: selected_agents.presence, statuses: selected_statuses.presence, days: params[:days].presence, model_ids: selected_models.presence),
+          <%= link_to ruby_llm_agents.export_executions_path(agent_types: selected_agents.presence, statuses: selected_statuses.presence, days: params[:days].presence, model_ids: selected_models.presence, workflow_types: selected_workflows.presence),
               class: "flex-1 md:flex-initial flex items-center justify-center gap-2 px-3 py-2 md:p-2 text-sm md:text-base text-gray-600 md:text-gray-400 dark:text-gray-300 md:dark:text-gray-400 bg-gray-50 md:bg-transparent dark:bg-gray-700 md:dark:bg-transparent hover:text-gray-600 md:hover:text-gray-600 dark:hover:text-gray-200 hover:bg-gray-100 dark:hover:bg-gray-600 md:dark:hover:bg-gray-700 rounded-lg transition-colors",
-              title: "Export CSV",
-              data: { turbo: false } do %>
+              title: "Export CSV" do %>
             <svg class="w-4 h-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
               <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M12 10v6m0 0l-3-3m3 3l3-3m2 8H7a2 2 0 01-2-2V5a2 2 0 012-2h5.586a1 1 0 01.707.293l5.414 5.414a1 1 0 01.293.707V19a2 2 0 01-2 2z"/>
             </svg>

data/app/views/{rubyllm → ruby_llm}/agents/executions/_list.html.erb

@@ -94,7 +94,7 @@
 
         <tbody class="bg-white dark:bg-gray-800 divide-y divide-gray-100 dark:divide-gray-700">
           <% executions.each do |execution| %>
-            <% has_attempts = execution.attempts.present? && execution.attempts.size > 0 %>
+            <% has_attempts = execution.respond_to?(:attempts) && execution.attempts.present? && execution.attempts.size > 0 %>
             <tr
               class="hover:bg-gray-50 dark:hover:bg-gray-700 transition-colors"
               id="execution-row-<%= execution.id %>"
@@ -118,13 +118,23 @@
               </td>
 
               <td class="px-4 py-3 whitespace-nowrap cursor-pointer" onclick="window.location='<%= ruby_llm_agents.execution_path(execution) %>'">
-                <span class="text-sm font-medium text-gray-900 dark:text-gray-100">
-                  <%= execution.agent_type.gsub(/Agent$/, '') %>
-                </span>
+                <div class="flex items-center gap-2">
+                  <% if execution.respond_to?(:workflow_type) && execution.workflow_type.present? %>
+                    <%= render "ruby_llm/agents/shared/workflow_type_badge", workflow_type: execution.workflow_type, size: :xs, show_label: false %>
+                  <% end %>
+                  <span class="text-sm font-medium text-gray-900 dark:text-gray-100">
+                    <%= execution.agent_type.gsub(/Agent$|Workflow$|Pipeline$|Parallel$|Router$/, '') %>
+                  </span>
+                </div>
+                <% if execution.respond_to?(:parent_execution_id) && execution.parent_execution_id.present? %>
+                  <span class="text-xs text-gray-400 dark:text-gray-500 ml-5 block">
+                    child of #<%= execution.parent_execution_id %>
+                  </span>
+                <% end %>
               </td>
 
               <td class="px-4 py-3 whitespace-nowrap cursor-pointer" onclick="window.location='<%= ruby_llm_agents.execution_path(execution) %>'">
-                <%= render "rubyllm/agents/shared/status_badge", status: execution.status %>
+                <%= render "ruby_llm/agents/shared/status_badge", status: execution.status %>
               </td>
 
               <td class="px-4 py-3 whitespace-nowrap text-sm text-gray-500 dark:text-gray-400 cursor-pointer" onclick="window.location='<%= ruby_llm_agents.execution_path(execution) %>'">
@@ -133,7 +143,7 @@
 
               <!-- Attempts Count -->
               <td class="px-4 py-3 whitespace-nowrap text-sm text-center cursor-pointer" onclick="window.location='<%= ruby_llm_agents.execution_path(execution) %>'">
-                <% attempts_count = execution.attempts_count || (execution.attempts&.size || 1) %>
+                <% attempts_count = (execution.respond_to?(:attempts_count) && execution.attempts_count) || (execution.respond_to?(:attempts) && execution.attempts&.size) || 1 %>
                 <% if attempts_count > 1 %>
                   <span class="inline-flex items-center px-2 py-0.5 rounded text-xs font-medium bg-yellow-100 text-yellow-800 dark:bg-yellow-900 dark:text-yellow-200">
                     <%= attempts_count %> attempts
@@ -257,7 +267,7 @@
 
         <nav class="flex items-center space-x-1">
           <% if current_page > 1 %>
-            <%= link_to "Previous", url_for(page: current_page - 1), data: { turbo_frame: "executions_content", turbo_action: "advance" }, class: "px-3 py-1.5 text-sm font-medium text-gray-700 dark:text-gray-200 bg-white dark:bg-gray-800 border border-gray-300 dark:border-gray-600 rounded-md hover:bg-gray-50 dark:hover:bg-gray-700" %>
+            <%= link_to "Previous", url_for(page: current_page - 1), class: "px-3 py-1.5 text-sm font-medium text-gray-700 dark:text-gray-200 bg-white dark:bg-gray-800 border border-gray-300 dark:border-gray-600 rounded-md hover:bg-gray-50 dark:hover:bg-gray-700" %>
           <% else %>
             <span
               class="
@@ -297,12 +307,12 @@
                 <%= page %>
               </span>
             <% else %>
-              <%= link_to page, url_for(page: page), data: { turbo_frame: "executions_content", turbo_action: "advance" }, class: "px-3 py-1.5 text-sm font-medium text-gray-700 dark:text-gray-200 bg-white dark:bg-gray-800 border border-gray-300 dark:border-gray-600 rounded-md hover:bg-gray-50 dark:hover:bg-gray-700" %>
+              <%= link_to page, url_for(page: page), class: "px-3 py-1.5 text-sm font-medium text-gray-700 dark:text-gray-200 bg-white dark:bg-gray-800 border border-gray-300 dark:border-gray-600 rounded-md hover:bg-gray-50 dark:hover:bg-gray-700" %>
             <% end %>
           <% end %>
 
           <% if current_page < total_pages %>
-            <%= link_to "Next", url_for(page: current_page + 1), data: { turbo_frame: "executions_content", turbo_action: "advance" }, class: "px-3 py-1.5 text-sm font-medium text-gray-700 dark:text-gray-200 bg-white dark:bg-gray-800 border border-gray-300 dark:border-gray-600 rounded-md hover:bg-gray-50 dark:hover:bg-gray-700" %>
+            <%= link_to "Next", url_for(page: current_page + 1), class: "px-3 py-1.5 text-sm font-medium text-gray-700 dark:text-gray-200 bg-white dark:bg-gray-800 border border-gray-300 dark:border-gray-600 rounded-md hover:bg-gray-50 dark:hover:bg-gray-700" %>
           <% else %>
             <span
               class="

data/app/views/ruby_llm/agents/executions/_workflow_summary.html.erb

@@ -0,0 +1,101 @@
+<%# Simplified Workflow Summary - Clean table approach %>
+<%# @param execution [RubyLLM::Agents::Execution] The workflow execution to display %>
+
+<% if execution.root_workflow? %>
+  <% stats = execution.workflow_aggregate_stats %>
+  <% steps = execution.workflow_steps.to_a %>
+
+  <div class="bg-white dark:bg-gray-800 border border-gray-200 dark:border-gray-700 rounded-lg overflow-hidden mb-6">
+    <!-- Header -->
+    <div class="px-4 py-3 border-b border-gray-200 dark:border-gray-700 flex items-center justify-between">
+      <div class="flex items-center gap-2">
+        <% case execution.workflow_type
+           when "pipeline" %>
+          <span class="text-indigo-600 dark:text-indigo-400">→</span>
+          <span class="font-medium text-gray-900 dark:text-gray-100">Pipeline</span>
+          <span class="text-gray-500 dark:text-gray-400">· <%= stats[:steps_count] %> steps</span>
+        <% when "parallel" %>
+          <span class="text-cyan-600 dark:text-cyan-400">⫴</span>
+          <span class="font-medium text-gray-900 dark:text-gray-100">Parallel</span>
+          <span class="text-gray-500 dark:text-gray-400">· <%= stats[:steps_count] %> branches</span>
+        <% when "router" %>
+          <span class="text-amber-600 dark:text-amber-400">⑂</span>
+          <span class="font-medium text-gray-900 dark:text-gray-100">Router</span>
+          <span class="text-gray-500 dark:text-gray-400">→ <%= execution.routed_to %></span>
+        <% end %>
+      </div>
+
+      <% overall_status = execution.workflow_overall_status %>
+      <%= render "ruby_llm/agents/shared/status_badge", status: overall_status.to_s, size: :sm %>
+    </div>
+
+    <!-- Table -->
+    <div class="overflow-x-auto">
+      <table class="min-w-full text-sm">
+        <thead class="bg-gray-50 dark:bg-gray-900/50">
+          <tr>
+            <th class="px-4 py-2 text-left text-xs font-medium text-gray-500 dark:text-gray-400 uppercase">
+              <%= execution.pipeline_workflow? ? "Step" : execution.parallel_workflow? ? "Branch" : "Route" %>
+            </th>
+            <th class="px-4 py-2 text-left text-xs font-medium text-gray-500 dark:text-gray-400 uppercase">Status</th>
+            <th class="px-4 py-2 text-right text-xs font-medium text-gray-500 dark:text-gray-400 uppercase">Duration</th>
+            <th class="px-4 py-2 text-right text-xs font-medium text-gray-500 dark:text-gray-400 uppercase">Tokens</th>
+            <th class="px-4 py-2 text-right text-xs font-medium text-gray-500 dark:text-gray-400 uppercase">Cost</th>
+          </tr>
+        </thead>
+        <tbody class="divide-y divide-gray-100 dark:divide-gray-700">
+          <% steps.each_with_index do |step, index| %>
+            <tr class="hover:bg-gray-50 dark:hover:bg-gray-800/50">
+              <td class="px-4 py-2">
+                <%= link_to ruby_llm_agents.execution_path(step.id), class: "text-blue-600 dark:text-blue-400 hover:underline" do %>
+                  <% if execution.pipeline_workflow? %>
+                    <span class="text-gray-400 dark:text-gray-500"><%= index + 1 %>.</span>
+                  <% end %>
+                  <%= step.workflow_step || step.agent_type.gsub(/Agent$/, "") %>
+                <% end %>
+              </td>
+              <td class="px-4 py-2">
+                <% case step.status
+                   when "success" %>
+                  <span class="text-green-600 dark:text-green-400">✓</span>
+                <% when "error" %>
+                  <span class="text-red-600 dark:text-red-400">✗</span>
+                <% when "timeout" %>
+                  <span class="text-orange-600 dark:text-orange-400">⏱</span>
+                <% when "running" %>
+                  <span class="text-blue-600 dark:text-blue-400 animate-pulse">●</span>
+                <% else %>
+                  <span class="text-gray-400">○</span>
+                <% end %>
+              </td>
+              <td class="px-4 py-2 text-right text-gray-600 dark:text-gray-300 tabular-nums">
+                <%= step.duration_ms ? "#{number_with_delimiter(step.duration_ms)}ms" : "-" %>
+              </td>
+              <td class="px-4 py-2 text-right text-gray-600 dark:text-gray-300 tabular-nums">
+                <%= number_with_delimiter(step.total_tokens || 0) %>
+              </td>
+              <td class="px-4 py-2 text-right text-gray-600 dark:text-gray-300 tabular-nums">
+                $<%= number_with_precision(step.total_cost || 0, precision: 4) %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+        <tfoot class="bg-gray-50 dark:bg-gray-900/50 font-medium">
+          <tr>
+            <td class="px-4 py-2 text-gray-700 dark:text-gray-300">Total</td>
+            <td class="px-4 py-2"></td>
+            <td class="px-4 py-2 text-right text-gray-700 dark:text-gray-300 tabular-nums">
+              <%= stats[:wall_clock_ms] ? "#{number_with_delimiter(stats[:wall_clock_ms])}ms" : "-" %>
+            </td>
+            <td class="px-4 py-2 text-right text-gray-700 dark:text-gray-300 tabular-nums">
+              <%= number_with_delimiter(stats[:total_tokens]) %>
+            </td>
+            <td class="px-4 py-2 text-right text-gray-700 dark:text-gray-300 tabular-nums">
+              $<%= number_with_precision(stats[:total_cost], precision: 4) %>
+            </td>
+          </tr>
+        </tfoot>
+      </table>
+    </div>
+  </div>
+<% end %>

data/app/views/ruby_llm/agents/executions/index.html.erb

@@ -0,0 +1,88 @@
+<h2 class="text-2xl font-bold text-gray-900 dark:text-gray-100 mb-6">Executions</h2>
+
+<div id="executions_content" x-data="{ activeTab: '<%= params[:execution_type] || 'all' %>', activeWorkflowType: '<%= params[:workflow_type_tab] %>' }">
+  <!-- Top-Level Execution Type Tabs -->
+  <div class="border-b border-gray-200 dark:border-gray-700 mb-4">
+    <nav class="-mb-px flex space-x-6" aria-label="Execution Types">
+      <button type="button" @click="activeTab = 'all'; activeWorkflowType = ''; submitFilters()"
+              :class="activeTab === 'all' ? 'border-blue-500 text-blue-600 dark:text-blue-400' : 'border-transparent text-gray-500 dark:text-gray-400 hover:text-gray-700 dark:hover:text-gray-300'"
+              class="whitespace-nowrap py-3 px-1 border-b-2 font-medium text-sm transition-colors">
+        All
+      </button>
+      <button type="button" @click="activeTab = 'agents'; activeWorkflowType = ''; submitFilters()"
+              :class="activeTab === 'agents' ? 'border-blue-500 text-blue-600 dark:text-blue-400' : 'border-transparent text-gray-500 dark:text-gray-400 hover:text-gray-700 dark:hover:text-gray-300'"
+              class="whitespace-nowrap py-3 px-1 border-b-2 font-medium text-sm transition-colors">
+        Agents Only
+      </button>
+      <button type="button" @click="activeTab = 'workflows'; activeWorkflowType = ''; submitFilters()"
+              :class="activeTab === 'workflows' ? 'border-blue-500 text-blue-600 dark:text-blue-400' : 'border-transparent text-gray-500 dark:text-gray-400 hover:text-gray-700 dark:hover:text-gray-300'"
+              class="whitespace-nowrap py-3 px-1 border-b-2 font-medium text-sm transition-colors">
+        Workflows
+      </button>
+    </nav>
+  </div>
+
+  <!-- Workflow Sub-tabs -->
+  <div x-show="activeTab === 'workflows'" x-cloak class="mb-4">
+    <div class="flex flex-wrap gap-2">
+      <button type="button" @click="activeWorkflowType = ''; submitFilters()"
+              :class="activeWorkflowType === '' ? 'bg-gray-900 dark:bg-gray-100 text-white dark:text-gray-900' : 'bg-gray-100 dark:bg-gray-700 text-gray-700 dark:text-gray-300 hover:bg-gray-200 dark:hover:bg-gray-600'"
+              class="px-3 py-1.5 rounded-md text-sm font-medium transition-colors">
+        All Workflows
+      </button>
+      <button type="button" @click="activeWorkflowType = 'pipeline'; submitFilters()"
+              :class="activeWorkflowType === 'pipeline' ? 'bg-indigo-600 text-white' : 'bg-indigo-100 dark:bg-indigo-900/50 text-indigo-700 dark:text-indigo-300 hover:bg-indigo-200 dark:hover:bg-indigo-800'"
+              class="px-3 py-1.5 rounded-md text-sm font-medium transition-colors inline-flex items-center gap-1.5">
+        <span aria-hidden="true">-></span> Pipeline
+      </button>
+      <button type="button" @click="activeWorkflowType = 'parallel'; submitFilters()"
+              :class="activeWorkflowType === 'parallel' ? 'bg-cyan-600 text-white' : 'bg-cyan-100 dark:bg-cyan-900/50 text-cyan-700 dark:text-cyan-300 hover:bg-cyan-200 dark:hover:bg-cyan-800'"
+              class="px-3 py-1.5 rounded-md text-sm font-medium transition-colors inline-flex items-center gap-1.5">
+        <span aria-hidden="true">//</span> Parallel
+      </button>
+      <button type="button" @click="activeWorkflowType = 'router'; submitFilters()"
+              :class="activeWorkflowType === 'router' ? 'bg-amber-600 text-white' : 'bg-amber-100 dark:bg-amber-900/50 text-amber-700 dark:text-amber-300 hover:bg-amber-200 dark:hover:bg-amber-800'"
+              class="px-3 py-1.5 rounded-md text-sm font-medium transition-colors inline-flex items-center gap-1.5">
+        <span aria-hidden="true">&lt;&gt;</span> Router
+      </button>
+    </div>
+  </div>
+
+  <!-- Hidden inputs for tab state -->
+  <input type="hidden" id="execution-type-filter" name="execution_type" x-model="activeTab" form="filters-form">
+  <input type="hidden" id="workflow-type-tab-filter" name="workflow_type_tab" x-model="activeWorkflowType" form="filters-form">
+
+  <%= render partial: "ruby_llm/agents/executions/filters", locals: { agent_types: @agent_types, model_ids: @model_ids, workflow_types: @workflow_types, filter_stats: @filter_stats } %>
+  <%= render partial: "ruby_llm/agents/executions/list", locals: { executions: @executions, pagination: @pagination, filter_stats: @filter_stats } %>
+</div>
+
+<script>
+  // Toggle attempts expansion (used by executions list)
+  function toggleAttempts(executionId) {
+    const attemptsRow = document.getElementById('attempts-row-' + executionId);
+    const expandBtn = document.querySelector(`button[data-execution-id="${executionId}"]`);
+
+    if (attemptsRow) {
+      const isHidden = attemptsRow.classList.contains('hidden');
+      attemptsRow.classList.toggle('hidden');
+
+      // Rotate the chevron
+      if (expandBtn) {
+        const svg = expandBtn.querySelector('svg');
+        if (svg) {
+          svg.classList.toggle('rotate-90', isHidden);
+        }
+        expandBtn.setAttribute('aria-expanded', isHidden ? 'true' : 'false');
+      }
+    }
+  }
+
+  // Submit filters when tabs change
+  function submitFilters() {
+    // Delay to allow Alpine to update x-model values
+    setTimeout(function() {
+      const form = document.getElementById('filters-form');
+      if (form) form.requestSubmit();
+    }, 50);
+  }
+</script>