prompt_navigator 2.0.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1519a5a4fc2a28bda0f3795d7ce91f15afeeab646db88d53888cce56ef031869
-  data.tar.gz: 6dd58cf6962f7db2fcdb51937147926588d69c8d2c9d71e32d7e923632038789
+  metadata.gz: f448c834c18a600ebf636fad362e51f7dd80c354484e1971388a884cdd9ffb80
+  data.tar.gz: d9a983063d1afe013001e7017b64ecc49f60467ea6e24a4179c20f37698d7eb4
 SHA512:
-  metadata.gz: cf79eba4420fc941941e99cc7701b3b0970227f62e171b23b43750ea5342b90b4600efdac5290a309f8d1773dfe71673719cfd7d11da9718b4e96e99a558afe0
-  data.tar.gz: 1146200b8a8eecd25555f3b83f858039d323b775832566cd4c8a2d80584417c224cb61fb7400e5637359f7b6071aa5257fa873c5dd0d3c7e352ee3afe1e76b25
+  metadata.gz: 7e3e53891c90ffa1fe59767eeb2de6f10b215eef872c4ca815384329b61e315aca9de871ed6f91761c22ef3c252d00415d6741b9158cf46b1ff51284b0c5db46
+  data.tar.gz: 35998b74c979da62e030e6bec1609bc35aea630fb205ffed49718f39a3b594879bb0d5c30340b0c9f5b6e38d8fc211b18c7f273d3d788f84a5e64843b09be3ba

data/app/assets/stylesheets/prompt_navigator/history.css

@@ -2,8 +2,8 @@
     position: relative;
     display: flex;
     flex-direction: column;
-    gap: 8px;
     padding-left: 32px; /* space for arrows */
+    margin-top: 20px;
 }
 
 .history-card {
@@ -14,6 +14,7 @@
     box-shadow: 0 1px 2px rgba(0, 0, 0, 0.06);
     position: relative;
     z-index: 1;
+    margin-bottom: 16px;
     transition:
             box-shadow 0.15s ease,
             transform 0.15s ease;
@@ -27,17 +28,70 @@
         box-shadow: 0 4px 12px rgba(0, 0, 0, 0.12);
         transform: translateY(-2px);
     }
+
+    &:last-child {
+        margin-bottom: 0;
+    }
+
+    &:has(+ .history-straight-arrow) {
+        margin-bottom: 0;
+    }
+}
+
+.history-card-row {
+    display: flex;
+    align-items: center;
+    gap: 6px;
+}
+
+.history-card-row .history-card-link {
+    flex: 1;
+    min-width: 0;
 }
 
 .history-card-link {
-    display: grid;
-    grid-template-columns: auto 1fr auto;
-    grid-gap: 8px;
+    display: block;
     text-decoration: none;
     color: #333;
+    min-width: 0;
+}
+
+.history-card-delete-form {
+    margin: 0;
+    display: flex;
     align-items: center;
 }
 
+.history-card-delete {
+    display: flex;
+    align-items: center;
+    justify-content: center;
+    width: 22px;
+    height: 22px;
+    padding: 0;
+    border: none;
+    background: transparent;
+    border-radius: 4px;
+    color: #888;
+    cursor: pointer;
+    flex-shrink: 0;
+    opacity: 0;
+    transition: opacity 0.15s ease, color 0.15s ease, background-color 0.15s ease;
+
+    i {
+        font-size: 12px;
+    }
+
+    &:hover {
+        color: #dc2626;
+        background-color: rgba(220, 38, 38, 0.1);
+    }
+}
+
+.history-card:hover .history-card-delete {
+    opacity: 1;
+}
+
 .history-card-number {
     font-size: 12px;
     font-weight: 600;
@@ -57,7 +111,6 @@
     font-size: 10px;
     font-weight: 700;
     letter-spacing: 0.02em;
-    margin-left: auto;
     flex-shrink: 0;
     padding: 2px 6px;
     border-radius: 4px;
@@ -85,9 +138,10 @@
     display: flex;
     justify-content: center;
     align-items: center;
-    height: 8px;
+    height: 16px;
     margin: 0;
-    font-size: 10px;
+    font-size: 16px;
+    font-weight: bold;
     color: #555;
     line-height: 1;
 }

data/app/javascript/controllers/history_controller.js

@@ -8,7 +8,13 @@ export default class extends Controller {
     #drawArrowsBound
     #markerId = "history-arrow-head"
     #startX = 32
-    #curveOffset = 40
+    // Curve offset scales with the vertical gap so arcs of different lengths
+    // nest instead of overlap. Bounded so short arcs don't collapse onto the
+    // stack and long arcs don't escape the sidebar pane (the stack only has
+    // `padding-left: 32px` of space before the chat area).
+    #minCurveOffset = 12
+    #maxCurveOffset = 28
+    #curveScale = 0.22
 
     connect() {
         this.#drawArrows()
@@ -102,13 +108,17 @@ export default class extends Controller {
         // Skip if cards are adjacent - straight arrow is rendered by helper
         if (verticalGap < 80) return
 
-        const path = this.#createCurvedArrowPath(startY, endY)
+        const path = this.#createCurvedArrowPath(startY, endY, verticalGap)
         svg.appendChild(path)
     }
 
-    #createCurvedArrowPath(startY, endY) {
+    #createCurvedArrowPath(startY, endY, verticalGap) {
         const startX = this.#startX // left edge margin
-        const curveX = startX - this.#curveOffset // curve outward to the left
+        const curveOffset = Math.max(
+            this.#minCurveOffset,
+            Math.min(verticalGap * this.#curveScale, this.#maxCurveOffset)
+        )
+        const curveX = startX - curveOffset // curve outward to the left
         const pathData = `M ${startX} ${startY} C ${curveX} ${startY}, ${curveX} ${endY}, ${startX} ${endY}`
 
         const path = document.createElementNS("http://www.w3.org/2000/svg", "path")

data/app/views/prompt_navigator/_history.html.erb

@@ -1,13 +1,16 @@
 <%
   active_uuid = locals[:active_uuid]
   card_path = locals[:card_path]
+  delete_path = locals[:delete_path]
+  # A PE is a leaf when no other PE in the chat references it as `previous`.
+  non_leaf_ids = @history.filter_map { |pe| pe.previous_id }
 %>
 
 <h2>History</h2>
 <% if @history.present? %>
   <div class="history-stack" data-controller="history">
     <% @history.each_with_index do |ann, idx| %>
-      <%= render 'prompt_navigator/history_card', locals: { ann: ann, next_ann: @history[idx + 1], is_active: ann.execution_id == active_uuid, card_path: card_path } %>
+      <%= render 'prompt_navigator/history_card', locals: { ann: ann, next_ann: @history[idx + 1], is_active: ann.execution_id == active_uuid, card_path: card_path, delete_path: delete_path, is_leaf: !non_leaf_ids.include?(ann.id) } %>
     <% end %>
     <svg class="history-arrows" data-history-target="svg"></svg>
   </div>

data/app/views/prompt_navigator/_history_card.html.erb

@@ -4,18 +4,44 @@
   is_active = locals[:is_active]
   parent_uuid = ann.previous&.execution_id
   card_path = locals[:card_path]
+  delete_path = locals[:delete_path]
+  is_leaf = locals[:is_leaf]
+
+  # Hide leading attached-image data URIs (`![](data:image/png;base64,…)`).
+  # The base64 blob isn't useful in a 30-char preview — replace each with a
+  # short marker so the rest of the prompt text is visible.
+  display_prompt = ann.prompt.to_s.gsub(/!\[[^\]]*\]\(data:[^)]+\)/m, "[image]").strip
 %>
 
 <div class="history-card<%= ' is-active' if is_active %>" data-history-target="cards" data-uuid="<%= ann.execution_id %>" data-parent-uuid="<%= parent_uuid %>">
-  <%= link_to card_path.call(ann.execution_id), class: "history-card-link" do %>
-    <div class="history-card-prompt"><%= truncate(ann.prompt.to_s, length: 30) %></div>
+  <div class="history-card-row">
+    <%= link_to card_path.call(ann.execution_id), class: "history-card-link" do %>
+      <div class="history-card-prompt"><%= truncate(display_prompt, length: 30) %></div>
+    <% end %>
+    <% if is_leaf && delete_path %>
+      <%= button_to delete_path.call(ann.execution_id),
+                    method: :delete,
+                    class: "history-card-delete",
+                    title: "Delete this prompt",
+                    form: { class: "history-card-delete-form" },
+                    data: { turbo_confirm: "Delete this prompt? This cannot be undone." } do %>
+        <i class="bi bi-trash"></i>
+      <% end %>
+    <% end %>
     <% if ann.llm_platform.present? %>
-      <span class="history-card-platform-label" data-platform="<%= ann.llm_platform %>"><%= { "openai" => "GPT", "anthropic" => "A", "google" => "G", "ollama" => "O" }[ann.llm_platform] || ann.llm_platform %></span>
+      <% platform_label = PromptNavigator.label_for(platform: ann.llm_platform, model: ann.model) %>
+      <% tooltip = ann.model.present? ? "#{ann.model} (#{ann.llm_platform})" : ann.llm_platform.to_s %>
+      <span class="history-card-platform-label" data-platform="<%= ann.llm_platform %>" title="<%= tooltip %>"><%= platform_label %></span>
     <% end %>
-  <% end %>
+  </div>
 </div>
 
-<%# Check !next_ann.nil? to prevent the "↑" arrow from appearing below the bottom history entry, as parent_uuid would be nil in that case. %>
-<% if !next_ann.nil? && parent_uuid == next_ann.execution_id %>
-  <div class="history-straight-arrow">↑</div>
-<% end %>
+<%# Direction-agnostic adjacency arrow: detect whether the next card is this card's
+    parent (newest-first ordering → "↑") or its child (oldest-first ordering → "↓"). %>
+<% if next_ann %>
+  <% if parent_uuid == next_ann.execution_id %>
+    <div class="history-straight-arrow">↑</div>
+  <% elsif next_ann.previous_id == ann.id %>
+    <div class="history-straight-arrow">↓</div>
+  <% end %>
+<% end %>

data/lib/prompt_navigator/configuration.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module PromptNavigator
+  # Host-overridable display config for the history sidebar.
+  #
+  # Usage from a host app initializer:
+  #
+  #   PromptNavigator.configure do |c|
+  #     c.platform_labels["openai"] = "OAI"           # override platform default
+  #     c.model_labels["claude-opus-4-7"] = "Opus"    # per-model override
+  #   end
+  class Configuration
+    DEFAULT_PLATFORM_LABELS = {
+      "openai"    => "GPT",
+      "anthropic" => "Claude",
+      "google"    => "Gemini",
+      "ollama"    => "Ollama"
+    }.freeze
+
+    attr_accessor :platform_labels, :model_labels
+
+    def initialize
+      @platform_labels = DEFAULT_PLATFORM_LABELS.dup
+      @model_labels    = {}
+    end
+  end
+
+  def self.config
+    @config ||= Configuration.new
+  end
+
+  def self.configure
+    yield config
+  end
+
+  # Resolution order: per-model override → per-platform → raw platform string.
+  # Returns "" only when both inputs are blank.
+  def self.label_for(platform:, model: nil)
+    model_key = model.to_s
+    return config.model_labels[model_key] if model_key != "" && config.model_labels.key?(model_key)
+    platform_key = platform.to_s
+    config.platform_labels[platform_key] || platform_key
+  end
+end

data/lib/prompt_navigator/helpers.rb

@@ -1,7 +1,11 @@
 module PromptNavigator
   module Helpers
-    def history_list(card_path, active_uuid: nil)
-      render "prompt_navigator/history", locals: { card_path: card_path, active_uuid: active_uuid }
+    def history_list(card_path, active_uuid: nil, delete_path: nil)
+      render "prompt_navigator/history", locals: {
+        card_path: card_path,
+        active_uuid: active_uuid,
+        delete_path: delete_path
+      }
     end
   end
 end

data/lib/prompt_navigator/version.rb

@@ -1,3 +1,3 @@
 module PromptNavigator
-  VERSION = "2.0.0"
+  VERSION = "2.2.0"
 end

data/lib/prompt_navigator.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
 require "prompt_navigator/version"
+require "prompt_navigator/configuration"
 require "prompt_navigator/engine"
 require "prompt_navigator/helpers"
 require_relative "../app/controllers/concerns/prompt_navigator/history_manageable"
 
 module PromptNavigator
-  # Your code goes here...
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: prompt_navigator
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.2.0
 platform: ruby
 authors:
 - dhq_boiler
@@ -60,6 +60,7 @@ files:
 - lib/generators/prompt_navigator/modeling/modeling_generator.rb
 - lib/generators/prompt_navigator/modeling/templates/db/migrate/20260129073026_create_prompt_navigator_prompt_executions.rb
 - lib/prompt_navigator.rb
+- lib/prompt_navigator/configuration.rb
 - lib/prompt_navigator/engine.rb
 - lib/prompt_navigator/helpers.rb
 - lib/prompt_navigator/version.rb