collavre 0.8.2 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f586b6b08d3de11a9c9212f4ff7e11113143bdf0dbbd5963668e9613041b3f02
-  data.tar.gz: 5ad7d024a1c53506c0ef1ee08731041de3a66842de48f26912b5eb56051c7029
+  metadata.gz: 656a876df08e2034398807dd976a0da67f7b4e5120dd73301ea521dd1291eb8b
+  data.tar.gz: c17a5821354531dfaaafd6dd51794114a132c42a3535d612410a838d75499050
 SHA512:
-  metadata.gz: 187df2c9024426107e264eb14d7af711cc10255f87eec3380287465c2f0b19198b4a67f50b8d8e3920105166d268af57e1a9a525cb9c49ea84e11a2031ac9503
-  data.tar.gz: eb58fba86ff137079da0f834c3f1b82e2fbdede8c5e2d930ebff1602b6c38e45744d6a31bb01d568142b4bee42b5bd0bc68a5bf6e77f6187993f901bee3449ad
+  metadata.gz: 9547afa6f502aa818ef3a565d3e8991f25298623dffd980a2d63f3383debd2eef08ce0440285d5fe7c1f8dc7810245cb1aea8f252bf44d8cdd1400460b280fa1
+  data.tar.gz: 336344900a57853600abb3d8f879741d52f8673238a63b90b5054fe5d5f8b412fb54aced24dd9c93ea07a1c66175fe5518a58bd6011f9aae363a0a82720e77ef

data/app/assets/stylesheets/collavre/activity_logs.css

@@ -9,7 +9,7 @@
 }
 
 .comment-activity-log-block details>summary::marker {
-    color: var(--border-color);
+    color: var(--text-muted);
 }
 
 /* List container */

data/app/assets/stylesheets/collavre/comment_versions.css

@@ -14,7 +14,8 @@
   padding: 0 var(--space-2, 0.25rem);
   font-size: var(--text-xs, 0.75rem);
   line-height: 1.6;
-  color: var(--text-color, #333);
+  color: var(--text-primary, #202123);
+  font-weight: bold;
 }
 
 .comment-version-btn:hover:not(:disabled) {
@@ -35,16 +36,18 @@
 
 .comment-version-delete-btn {
   background: none;
-  border: none;
+  border: 1px solid var(--border-color, #ddd);
+  border-radius: var(--radius-1, 4px);
   cursor: pointer;
-  color: var(--text-muted, #888);
+  color: var(--text-primary, #202123);
   font-size: var(--text-xs, 0.75rem);
-  padding: 0 var(--space-1, 0.125rem);
+  padding: 0 var(--space-2, 0.25rem);
   margin-left: var(--space-1, 0.125rem);
+  line-height: 1.6;
 }
 
-.comment-version-delete-btn:hover {
-  color: var(--danger-color, #e53e3e);
+.comment-version-delete-btn:hover:not(:disabled) {
+  background: var(--surface-hover, #f0f0f0);
 }
 
 .comment-version-selected {
@@ -59,6 +62,7 @@
   cursor: pointer;
   color: var(--primary-color, #4a90d9);
   font-size: var(--text-xs, 0.75rem);
+  font-weight: bold;
   padding: 0 var(--space-2, 0.25rem);
   margin-left: var(--space-1, 0.125rem);
   line-height: 1.6;

data/app/assets/stylesheets/collavre/comments_popup.css

@@ -10,9 +10,11 @@
     min-width: 200px;
     min-height: 200px;
     background: var(--surface-bg);
-    border: 1px solid var(--border-color);
+    border: var(--border-1) solid var(--color-active);
     border-radius: var(--radius-3);
-    box-shadow: var(--shadow-3);
+    box-shadow: var(--shadow-3),
+                0 0 6px color-mix(in srgb, var(--color-active) 25%, transparent),
+                0 0 14px color-mix(in srgb, var(--color-active) 12%, transparent);
     padding: 1em;
     flex-direction: column;
     transition: top 0.25s ease, left 0.25s ease, width 0.25s ease, height 0.25s ease,
@@ -982,6 +984,12 @@ body.chat-fullscreen {
     margin-right: -1px;
 }
 
+.comment-contexts-list.context-drop-active {
+    outline: 2px dashed var(--color-link);
+    outline-offset: -2px;
+    background-color: color-mix(in srgb, var(--color-link) 8%, transparent);
+}
+
 .context-chip[draggable="true"] {
     cursor: grab;
 }
@@ -1098,6 +1106,30 @@ body.chat-fullscreen {
     box-sizing: border-box;
 }
 
+.topic-agent-avatar {
+    width: 16px;
+    height: 16px;
+    border-radius: 50%;
+    object-fit: cover;
+    flex-shrink: 0;
+}
+
+.ai-agent-draggable {
+    cursor: grab;
+}
+
+.ai-agent-draggable:active,
+.ai-agent-draggable.dragging {
+    cursor: grabbing;
+    opacity: 0.5;
+}
+
+.topic-creation-container.drag-over {
+    background: var(--color-active);
+    border-radius: 12px;
+    opacity: 0.8;
+}
+
 .topic-tag:hover {
     background: var(--surface-hover);
 }
@@ -1342,6 +1374,23 @@ body.chat-fullscreen {
     flex-wrap: wrap;
 }
 
+.selection-action-bar-select-all {
+    display: flex;
+    align-items: center;
+    gap: var(--space-1);
+    font-size: 0.85em;
+    font-weight: bold;
+    color: var(--color-text);
+    cursor: pointer;
+    user-select: none;
+    white-space: nowrap;
+}
+
+.selection-action-bar-select-all-checkbox {
+    cursor: pointer;
+    margin: 0;
+}
+
 .selection-action-bar-count {
     font-size: 0.85em;
     font-weight: bold;
@@ -1365,6 +1414,15 @@ body.chat-fullscreen {
     background: var(--surface-hover);
 }
 
+.selection-action-bar-btn:disabled {
+    opacity: 0.4;
+    cursor: not-allowed;
+}
+
+.selection-action-bar-btn:disabled:hover {
+    background: none;
+}
+
 .selection-action-delete:hover {
     color: var(--color-danger);
 }

data/app/assets/stylesheets/collavre/creatives.css

@@ -164,10 +164,6 @@ creative-tree-row.show-edit .creative-row {
 }
 
 .creative-row:hover .creative-content {
-    text-decoration: underline;
-    text-decoration-color: var(--color-link);
-    text-decoration-thickness: 2px;
-    text-underline-offset: 2px;
     cursor: pointer;
 }
 
@@ -440,6 +436,13 @@ creative-tree-row:not([expanded]) .creative-toggle-btn {
     background-color: var(--border-drag-over);
 }
 
+creative-tree-row.chat-active .creative-row {
+    border: var(--border-1) solid var(--color-active);
+    border-radius: var(--radius-2);
+    box-shadow: 0 0 6px color-mix(in srgb, var(--color-active) 25%, transparent),
+                0 0 14px color-mix(in srgb, var(--color-active) 12%, transparent);
+}
+
 .creative-row:hover,
 .creative-row.level-1:hover,
 .creative-row.level-2:hover,

data/app/assets/stylesheets/collavre/dark_mode.css

@@ -79,12 +79,7 @@ body.dark-mode {
 /* ============================================================================
  * BASE ELEMENT STYLES
  * ============================================================================ */
-html {
-  scroll-behavior: smooth;
-}
-
 @media (prefers-reduced-motion: reduce) {
-  html { scroll-behavior: auto; }
 
   /* Collapse decorative transitions to instant; state changes still
      apply but without visible motion (hover bg, theme switch, etc.). */
@@ -302,3 +297,28 @@ input[type="datetime-local"]::-webkit-datetime-edit-minute-field {
 .inline-block {
   display: inline-block;
 }
+
+/* --- Dark mode: stronger glow for visibility on dark backgrounds --- */
+body.dark-mode #comments-popup {
+  box-shadow: var(--shadow-3),
+              0 0 8px color-mix(in srgb, var(--color-active) 45%, transparent),
+              0 0 20px color-mix(in srgb, var(--color-active) 25%, transparent);
+}
+
+body.dark-mode creative-tree-row.chat-active .creative-row {
+  box-shadow: 0 0 8px color-mix(in srgb, var(--color-active) 45%, transparent),
+              0 0 20px color-mix(in srgb, var(--color-active) 25%, transparent);
+}
+
+@media (prefers-color-scheme: dark) {
+  body:not(.light-mode):not(.dark-mode) #comments-popup {
+    box-shadow: var(--shadow-3),
+                0 0 8px color-mix(in srgb, var(--color-active) 45%, transparent),
+                0 0 20px color-mix(in srgb, var(--color-active) 25%, transparent);
+  }
+
+  body:not(.light-mode):not(.dark-mode) creative-tree-row.chat-active .creative-row {
+    box-shadow: 0 0 8px color-mix(in srgb, var(--color-active) 45%, transparent),
+                0 0 20px color-mix(in srgb, var(--color-active) 25%, transparent);
+  }
+}

data/app/controllers/collavre/comments_controller.rb

@@ -281,7 +281,8 @@ module Collavre
           name: u.display_name,
           avatar_url: view_context.user_avatar_url(u, size: 20),
           default_avatar: !u.avatar.attached? && u.avatar_url.blank?,
-          initial: u.display_name[0].upcase
+          initial: u.display_name[0].upcase,
+          ai_user: u.ai_user?
         }
       end
       render json: {

data/app/controllers/collavre/topics_controller.rb

@@ -7,11 +7,12 @@ module Collavre
       can_manage = @creative.has_permission?(Current.user, :admin) || is_owner
       can_create_topic = can_manage || @creative.has_permission?(Current.user, :write)
 
-      active_topics = @creative.topics.active.order(:created_at)
+      active_topics = @creative.topics.active.order(:created_at).to_a
+      preload_primary_agents(active_topics)
       archived_topics = @creative.topics.archived.order(:created_at)
 
       render json: {
-        topics: active_topics,
+        topics: active_topics.map { |t| topic_json(t) },
         archived_topics: archived_topics,
         can_manage: can_manage,
         can_create_topic: can_create_topic
@@ -27,9 +28,16 @@ module Collavre
       topic.user = Current.user
 
       if topic.save
+        agent = nil
+        if params[:agent_id].present?
+          agent = User.find_by(id: params[:agent_id])
+          topic.set_primary_agent!(agent) if agent&.ai_user?
+        end
+
+        broadcast_data = agent ? topic_json_with_agent(topic, agent) : topic.slice(:id, :name)
         TopicsChannel.broadcast_to(
           @creative,
-          { action: "created", topic: topic.slice(:id, :name) }
+          { action: "created", topic: broadcast_data, user_id: Current.user.id }
         )
         render json: topic, status: :created
       else
@@ -159,6 +167,31 @@ module Collavre
       render json: { success: true }
     end
 
+    def set_primary_agent
+      unless @creative.has_permission?(Current.user, :write) || @creative.user == Current.user
+        render json: { error: I18n.t("collavre.topics.no_permission") }, status: :forbidden and return
+      end
+
+      topic = @creative.topics.find(params[:id])
+      agent = User.find_by(id: params[:agent_id])
+
+      unless agent&.ai_user?
+        render json: { error: I18n.t("collavre.topics.not_ai_agent") }, status: :unprocessable_entity and return
+      end
+
+      topic.set_primary_agent!(agent)
+
+      TopicsChannel.broadcast_to(
+        @creative,
+        {
+          action: "updated",
+          topic: topic_json_with_agent(topic, agent)
+        }
+      )
+
+      render json: { success: true, topic: topic_json_with_agent(topic, agent) }
+    end
+
     private
 
     def set_creative
@@ -168,5 +201,49 @@ module Collavre
     def topic_params
       params.require(:topic).permit(:name)
     end
+
+    # Batch-load primary agents for all topics to avoid N+1 queries
+    def preload_primary_agents(topics)
+      topic_ids = topics.map(&:id)
+      return if topic_ids.empty?
+
+      policies = OrchestratorPolicy.where(
+        policy_type: "arbitration",
+        scope_type: "Topic",
+        scope_id: topic_ids
+      ).index_by { |p| p.scope_id.to_i }
+
+      agent_ids = policies.values.filter_map { |p| p.config&.dig("primary_agent_id") }
+      agents = agent_ids.present? ? User.where(id: agent_ids).includes(avatar_attachment: :blob).index_by(&:id) : {}
+
+      topics.each do |topic|
+        policy = policies[topic.id]
+        agent_id = policy&.config&.dig("primary_agent_id")
+        topic.instance_variable_set(:@_primary_agent, agents&.dig(agent_id))
+      end
+    end
+
+    def topic_json(topic)
+      data = topic.slice(:id, :name)
+      agent = topic.instance_variable_get(:@_primary_agent) || topic.primary_agent
+      if agent
+        data[:primary_agent] = agent_json(agent)
+      end
+      data
+    end
+
+    def topic_json_with_agent(topic, agent)
+      data = topic.slice(:id, :name)
+      data[:primary_agent] = agent_json(agent)
+      data
+    end
+
+    def agent_json(agent)
+      {
+        id: agent.id,
+        name: agent.display_name,
+        avatar_url: view_context.user_avatar_url(agent, size: 20)
+      }
+    end
   end
 end

data/app/controllers/concerns/collavre/comments/approval_actions.rb

@@ -53,6 +53,11 @@ module Collavre
             end
           end
 
+          # Creative admins can also edit actions even if not the designated approver
+          if status_in_lock != :ok && @creative.has_permission?(Current.user, :admin)
+            status_in_lock = :ok
+          end
+
           if status_in_lock != :ok
             approver_mismatch_error = true
             status_error_key = case status_in_lock

data/app/controllers/concerns/collavre/comments/batch_operations.rb

@@ -37,6 +37,38 @@ module Collavre
         head :no_content
       end
 
+      def merge
+        comment_ids = Array(params[:comment_ids]).map(&:to_i).uniq
+        if comment_ids.size < 2
+          render json: { error: I18n.t("collavre.comments.merge.minimum_required") }, status: :unprocessable_entity and return
+        end
+
+        unless @creative.has_permission?(Current.user, :feedback)
+          render json: { error: I18n.t("collavre.comments.merge.not_authorized") }, status: :forbidden and return
+        end
+
+        visible_scope = @creative.comments.where(
+          "comments.private = ? OR comments.user_id = ? OR comments.approver_id = ?",
+          false, Current.user.id, Current.user.id
+        )
+        comments = visible_scope.where(id: comment_ids).to_a
+
+        if comments.length != comment_ids.length
+          render json: { error: I18n.t("collavre.comments.batch_delete_not_found") }, status: :not_found and return
+        end
+
+        # All comments must belong to the current user or their AI agents
+        my_ai_agent_ids = Current.user.created_ai_users.pluck(:id)
+        allowed_user_ids = [ Current.user.id ] + my_ai_agent_ids
+        unauthorized = comments.reject { |c| allowed_user_ids.include?(c.user_id) }
+        if unauthorized.any?
+          render json: { error: I18n.t("collavre.comments.merge.own_messages_only") }, status: :forbidden and return
+        end
+
+        MergeCommentsJob.perform_later(@creative.id, comment_ids, Current.user.id)
+        render json: { message: I18n.t("collavre.comments.merge.started") }, status: :accepted
+      end
+
       def move
         result = CommentMoveService.new(creative: @creative, user: Current.user).call(
           comment_ids: params[:comment_ids],

data/app/javascript/controllers/comment_version_controller.js

@@ -1,4 +1,5 @@
 import { Controller } from "@hotwired/stimulus"
+import { renderCommentMarkdown } from "../lib/utils/markdown"
 
 export default class extends Controller {
   static targets = ["prevBtn", "nextBtn", "indicator", "deleteBtn", "selectBtn"]
@@ -123,7 +124,10 @@ export default class extends Controller {
   setContentText(text) {
     const el = document.getElementById(this.contentTargetValue)
     const target = el?.querySelector(".comment-content") || el?.querySelector("[data-comment-target='content']")
-    if (target) target.textContent = text
+    if (target) {
+      target.innerHTML = renderCommentMarkdown(text)
+      target.dataset.rendered = "true"
+    }
   }
 
   updateButtons() {

data/app/javascript/controllers/comments/contexts_controller.js

@@ -1,5 +1,7 @@
 import { Controller } from "@hotwired/stimulus"
 
+const CREATIVE_MIME_TYPE = 'application/x-collavre-creative'
+
 export default class extends Controller {
     static targets = ["list", "toggleButton"]
 
@@ -11,6 +13,7 @@ export default class extends Controller {
         this.canManage = false
         this.draggingContextId = null
         this.listVisible = false
+        // External drop zone handlers are now Stimulus actions on the list target
     }
 
     get creativeId() {
@@ -22,6 +25,7 @@ export default class extends Controller {
         this.listVisible = false
         this._updateListVisibility()
         await this.loadContexts()
+        this._bindPopupDragDetection()
     }
 
     onPopupClosed() {
@@ -30,6 +34,7 @@ export default class extends Controller {
         if (this.hasListTarget) {
             this.listTarget.innerHTML = ''
         }
+        this._unbindPopupDragDetection()
     }
 
     async loadContexts() {
@@ -97,6 +102,7 @@ export default class extends Controller {
         if (!this.hasListTarget) return
 
         this._updateToggleButton()
+        // Drop zone is handled by Stimulus data-action on the list element
 
         const dragActions = this.canManage
             ? 'dragstart->comments--contexts#handleDragStart dragend->comments--contexts#handleDragEnd'
@@ -295,11 +301,15 @@ export default class extends Controller {
     }
 
     async handleReorderDrop(event) {
-        event.preventDefault()
-
         const targetEl = event.currentTarget
         targetEl.classList.remove('context-drag-over-left', 'context-drag-over-right')
 
+        // If this is a creative drag (not a context reorder), let it bubble to the list handler
+        if (!event.dataTransfer.types.includes('application/x-context-id')) return
+
+        event.preventDefault()
+        event.stopPropagation()
+
         const draggedId = parseInt(event.dataTransfer.getData('application/x-context-id'))
         const targetId = parseInt(targetEl.dataset.contextId)
 
@@ -345,7 +355,7 @@ export default class extends Controller {
                 method: 'PATCH',
                 headers: {
                     'Content-Type': 'application/json',
-                    'X-CSRF-Token': document.querySelector('meta[name="csrf-token"]').content
+                    'X-CSRF-Token': document.querySelector('meta[name="csrf-token"]')?.content || ''
                 },
                 body: JSON.stringify(params)
             })
@@ -358,6 +368,111 @@ export default class extends Controller {
         }
     }
 
+    // --- Auto-show context list when dragging creative over popup ---
+    _bindPopupDragDetection() {
+        const popup = this.element.closest('#comments-popup')
+        if (!popup) return
+        // Unbind any existing handlers first to prevent accumulation across opens
+        this._unbindPopupDragDetection()
+        this._popupEl = popup
+        this._boundPopupDragOver = this._handlePopupDragOver.bind(this)
+        this._boundPopupDragLeave = this._handlePopupDragLeave.bind(this)
+        this._boundPopupDrop = this.handleExternalDrop.bind(this)
+        popup.addEventListener('dragover', this._boundPopupDragOver)
+        popup.addEventListener('dragleave', this._boundPopupDragLeave)
+        popup.addEventListener('drop', this._boundPopupDrop)
+    }
+
+    _unbindPopupDragDetection() {
+        if (!this._popupEl) return
+        this._popupEl.removeEventListener('dragover', this._boundPopupDragOver)
+        this._popupEl.removeEventListener('dragleave', this._boundPopupDragLeave)
+        this._popupEl.removeEventListener('drop', this._boundPopupDrop)
+        this._popupEl = null
+    }
+
+    _handlePopupDragOver(event) {
+        if (this._isInternalReorder(event)) return
+        if (!this._isCreativeDrag(event)) return
+        if (!this.canManage) return
+
+        // Must preventDefault to allow drop on the popup
+        event.preventDefault()
+        event.dataTransfer.dropEffect = 'move'
+
+        if (!this.listVisible) {
+            // Auto-show context list when dragging a creative over the popup
+            this.listVisible = true
+            this._updateListVisibility()
+        }
+    }
+
+    _handlePopupDragLeave(event) {
+        if (!this._popupEl) return
+        // Only hide if leaving the popup entirely
+        if (this._popupEl.contains(event.relatedTarget)) return
+        if (this._hasBeenManuallyToggled) return
+
+        // Restore original state if no contexts
+        if (this.contexts.length === 0) {
+            this.listVisible = false
+            this._updateListVisibility()
+        }
+    }
+
+    // --- Drop zone for adding creatives from tree ---
+
+    _isCreativeDrag(event) {
+        return event.dataTransfer.types.includes(CREATIVE_MIME_TYPE)
+    }
+
+    _isInternalReorder(event) {
+        return event.dataTransfer.types.includes('application/x-context-id')
+    }
+
+    handleExternalDragOver(event) {
+        if (this._isInternalReorder(event)) return
+        if (!this._isCreativeDrag(event)) return
+        if (!this.canManage) return
+
+        event.preventDefault()
+        event.dataTransfer.dropEffect = 'move'
+        this.listTarget.classList.add('context-drop-active')
+    }
+
+    handleExternalDragLeave(event) {
+        // Only remove highlight if truly leaving the list area
+        if (!this.listTarget.contains(event.relatedTarget)) {
+            this.listTarget.classList.remove('context-drop-active')
+        }
+    }
+
+    async handleExternalDrop(event) {
+        if (this._isInternalReorder(event)) return
+        if (!this._isCreativeDrag(event)) return
+        if (!this.canManage) return
+
+        // Always prevent default for creative drags to avoid browser navigation
+        event.preventDefault()
+        event.stopPropagation()
+
+        this.listTarget.classList.remove('context-drop-active')
+
+        let creativeId = null
+
+        const rawData = event.dataTransfer.getData(CREATIVE_MIME_TYPE)
+        if (rawData) {
+            try {
+                const parsed = JSON.parse(rawData)
+                creativeId = parseInt(parsed.creativeId)
+            } catch (e) { /* ignore */ }
+        }
+
+        if (!creativeId) return
+
+        await this._addContextId(creativeId)
+    }
+
     _escapeHtml(text) {
         const div = document.createElement('div')
         div.textContent = text