collavre 0.8.0 → 0.8.2

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

@@ -11,23 +11,33 @@
 
 /* Share popup specific tweaks */
 #share-creative-modal .popup-box {
-  max-height: 80vh;
+  max-height: calc(100vh - 4em);
   overflow-y: auto;
 }
 
 #share-creative-modal .share-grid {
-  max-height: 40vh;
+  max-height: 30vh;
   overflow-y: auto;
 }
 
+@media (max-width: 768px) {
+  #share-creative-modal .popup-box {
+    max-width: calc(100vw - 1em) !important;
+  }
+
+  #share-creative-modal .share-grid {
+    max-height: 25vh;
+  }
+}
+
 /* Ensure autocomplete popup inside share modal works without mention_menu.css */
 #share-creative-modal .common-popup {
   position: absolute;
-  z-index: var(--layer-modal, 100);
-  background: var(--surface-section, #fff);
-  border: 1px solid var(--border-color, #ddd);
-  box-shadow: var(--shadow-2, 0 2px 8px rgba(0,0,0,.15));
-  border-radius: 6px;
+  z-index: var(--layer-modal);
+  background: var(--surface-section);
+  border: 1px solid var(--border-color);
+  box-shadow: var(--shadow-2);
+  border-radius: var(--radius-2);
   padding: 0.35em;
   max-width: min(420px, 90vw);
 }
@@ -45,7 +55,7 @@
   cursor: pointer;
 }
 #share-creative-modal .common-popup-list li:hover {
-  background: var(--border-drag-over, #f0f0f0);
+  background: var(--surface-hover);
 }
 
 .share-grid {
@@ -89,7 +99,7 @@
   box-sizing: border-box;
   margin-bottom: 0.75em;
   padding: 0.5em;
-  border-radius: 6px;
+  border-radius: var(--radius-2);
   border: 1px solid var(--border-color);
   background: var(--surface-bg);
   color: var(--text-primary);
@@ -102,7 +112,7 @@
 }
 
 .popup-box input:not([type="radio"]):not([type="checkbox"]),
-.popup-box select {
+.popup-box select:not(.share-modal-permission-select) {
   width: 100%;
   box-sizing: border-box;
 }
@@ -151,27 +161,80 @@
   animation: share-modal-message-fade 4s ease-in-out;
 }
 .share-modal-message-success {
-  background: var(--success-bg, #d4edda);
-  color: var(--success-text, #155724);
-  border: 1px solid var(--success-border, #c3e6cb);
+  background: color-mix(in srgb, var(--color-success) 12%, transparent);
+  color: var(--color-success);
+  border: 1px solid color-mix(in srgb, var(--color-success) 30%, transparent);
 }
 .share-modal-message-error {
-  background: var(--danger-bg, #f8d7da);
-  color: var(--danger-text, #721c24);
-  border: 1px solid var(--danger-border, #f5c6cb);
+  background: color-mix(in srgb, var(--color-danger) 12%, transparent);
+  color: var(--color-danger);
+  border: 1px solid color-mix(in srgb, var(--color-danger) 30%, transparent);
 }
+
 @keyframes share-modal-message-fade {
   0%, 80% { opacity: 1; }
   100% { opacity: 0; }
 }
+
+.share-modal-permission-select {
+  width: auto;
+}
+
+.share-modal-permission-select[disabled] {
+  opacity: 0.7;
+  cursor: default;
+}
+
+.share-inherited-link {
+  font-size: var(--text-0);
+  color: var(--text-muted);
+  white-space: nowrap;
+}
+
+.share-inherited-link:hover {
+  color: var(--text-link);
+}
+
+.share-inherited-short {
+  display: none;
+}
+
+@media (max-width: 768px) {
+  .share-inherited-short {
+    display: inline;
+  }
+  .share-inherited-full {
+    display: none;
+  }
+}
+
+
 .share-modal-pending {
   opacity: 0.6;
 }
+
+.share-inherited-item {
+  opacity: 0.8;
+}
+
+.share-pending-item {
+  opacity: 0.7;
+}
+
+.share-pending-badge {
+  display: inline-block;
+  font-size: var(--text-00);
+  color: var(--color-warning);
+  font-weight: var(--weight-5);
+  margin-left: 0.3em;
+  vertical-align: middle;
+}
+
 .share-modal-spinner {
   display: inline-block;
   width: 12px;
   height: 12px;
-  border: 2px solid var(--text-muted, #999);
+  border: 2px solid var(--text-muted);
   border-top-color: transparent;
   border-radius: 50%;
   animation: share-spinner 0.6s linear infinite;
@@ -223,7 +286,8 @@
   background: var(--surface-section);
   border: 1px solid var(--border-color);
   padding: 0.5em;
-  box-shadow: 0 2px 8px var(--border-color);
+  box-shadow: var(--shadow-3);
+  border-radius: var(--radius-2);
   min-width: 220px;
   top: calc(100% + 4px);
   left: 0;

data/app/controllers/collavre/creative_shares_controller.rb

@@ -9,6 +9,9 @@ module Collavre
       @shared_list = CreativeShare.where(creative: @creative)
                                   .includes(user: [ avatar_attachment: :blob ])
 
+      # Build inherited shares from ancestor creatives
+      @inherited_shares = build_inherited_shares(@creative)
+
       @pending_invitations = Invitation.where(creative: @creative, accepted_at: nil)
                                        .where("expires_at > ?", Time.current)
                                        .order(created_at: :desc)
@@ -151,5 +154,29 @@ module Collavre
       def all_descendants(creative)
         creative.children.flat_map { |child| [ child ] + all_descendants(child) }
       end
+
+      # Returns inherited shares from ancestor creatives.
+      # Each entry is a hash with :share, :source_creative keys.
+      # Only includes the closest (most specific) share per user.
+      def build_inherited_shares(creative)
+        ancestors = creative.ancestors
+        return [] if ancestors.empty?
+
+        ancestor_ids = ancestors.pluck(:id)
+        direct_user_ids = CreativeShare.where(creative: creative).pluck(:user_id).compact
+
+        ancestor_shares = CreativeShare
+          .where(creative_id: ancestor_ids)
+          .where.not(user_id: direct_user_ids) # Exclude users who already have direct shares
+          .includes(:creative, user: [ avatar_attachment: :blob ])
+
+        # Group by user_id and pick the closest ancestor share
+        ancestor_shares.group_by(&:user_id).filter_map do |_user_id, shares|
+          closest = CreativeShare.closest_parent_share(ancestor_ids, shares)
+          next unless closest && closest.permission != "no_access"
+
+          { share: closest, source_creative: closest.creative }
+        end
+      end
   end
 end

data/app/javascript/components/creative_tree_row.js

@@ -1,4 +1,4 @@
-import { LitElement, html, nothing } from "lit";
+import { LitElement, html, svg, nothing } from "lit";
 import DOMPurify from "dompurify";
 import { unsafeHTML } from "lit/directives/unsafe-html.js";
 import { parseEmojis } from "../utils/emoji_parser";
@@ -412,7 +412,7 @@ class CreativeTreeRow extends LitElement {
     const classes = "before-link creative-toggle-btn creative-action-btn";
     if (this.hasChildren) {
       return html`
-        <div class=${classes} data-creative-id=${this.creativeId}>${this._toggleSymbol()}</div>
+        <div class=${classes} data-creative-id=${this.creativeId}>${this._toggleIcon()}</div>
       `;
     }
     return html`
@@ -424,8 +424,15 @@ class CreativeTreeRow extends LitElement {
     `;
   }
 
-  _toggleSymbol() {
-    return this.expanded ? "▼" : "▶";
+  _toggleIcon() {
+    if (this.expanded) {
+      return svg`<svg width="16" height="16" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round">
+        <path d="M6 9L12 15L18 9"/>
+      </svg>`;
+    }
+    return svg`<svg width="16" height="16" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round">
+      <path d="M9 6L15 12L9 18"/>
+    </svg>`;
   }
 
   _handleToggleClick(event) {

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

@@ -3,6 +3,9 @@ import { Controller } from "@hotwired/stimulus"
 export default class extends Controller {
     static targets = ["list", "toggleButton"]
 
+    static ICON_CONTEXT_LINK = '<svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M10 13a5 5 0 0 0 7.54.54l3-3a5 5 0 0 0-7.07-7.07l-1.72 1.71"/><path d="M14 11a5 5 0 0 0-7.54-.54l-3 3a5 5 0 0 0 7.07 7.07l1.71-1.71"/></svg>'
+    static ICON_CONTEXT_PIN = '<svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><line x1="12" y1="17" x2="12" y2="22"/><path d="M5 17h14v-1.76a2 2 0 0 0-1.11-1.79l-1.78-.9A2 2 0 0 1 15 10.76V6h1a2 2 0 0 0 0-4H8a2 2 0 0 0 0 4h1v4.76a2 2 0 0 1-1.11 1.79l-1.78.9A2 2 0 0 0 5 15.24Z"/></svg>'
+
     connect() {
         this.contexts = []
         this.canManage = false
@@ -75,9 +78,9 @@ export default class extends Controller {
             const activeLinked = this.contexts.filter(c => !c.disabled).length
             const selfActive = this.selfContextDisabled ? 0 : 1
             const total = activeLinked + selfActive
-            this.toggleButtonTarget.textContent = `🔗 ${total}`
+            this.toggleButtonTarget.innerHTML = `${this.constructor.ICON_CONTEXT_LINK} ${total}`
         } else {
-            this.toggleButtonTarget.textContent = '🔗'
+            this.toggleButtonTarget.innerHTML = this.constructor.ICON_CONTEXT_LINK
         }
 
         // Auto-show if linked contexts exist, otherwise keep hidden
@@ -111,7 +114,7 @@ export default class extends Controller {
         html += `<span class="context-chip context-self ${selfClass}"
                       data-action="click->comments--contexts#toggleSelfContext"
                       title="${this.selfContextLabel}">
-                    📌 ${selfLabel}
+                    ${this.constructor.ICON_CONTEXT_PIN} ${selfLabel}
                  </span>`
 
         this.contexts.forEach(ctx => {
@@ -123,7 +126,7 @@ export default class extends Controller {
                           data-action="click->comments--contexts#toggleContext ${dragActions} ${reorderActions}"
                           data-context-id="${ctx.id}"
                           title="${ctx.inherited ? this.inheritedLabel : ''}">
-                        🔗 ${this._escapeHtml(ctx.description)}`
+                        ${this.constructor.ICON_CONTEXT_LINK} ${this._escapeHtml(ctx.description)}`
 
             html += `<button class="navigate-context-btn" data-action="click->comments--contexts#navigateToContext" data-context-id="${ctx.id}" title="${this.navigateLabel}">\u2192</button>`
 

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

@@ -387,6 +387,12 @@ export default class extends Controller {
 
   handleTouchStart(event) {
     if (!this.isMobile()) return
+    // Ignore swipe when share modal is open
+    const shareModal = document.getElementById("share-creative-modal")
+    if (shareModal && shareModal.style.display === "flex") {
+      this.touchStartY = null
+      return
+    }
     if (!event.target.closest('#comments-list')) {
       this.touchStartY = event.touches[0].clientY
     } else {

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

@@ -1,6 +1,9 @@
 import { Controller } from "@hotwired/stimulus"
 import { createSubscription } from "../../services/cable"
 
+const ICON_ARCHIVE = `<svg width="12" height="12" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><rect x="2" y="3" width="20" height="5" rx="1"/><path d="M4 8v11a2 2 0 0 0 2 2h12a2 2 0 0 0 2-2V8"/><path d="M10 12h4"/></svg>`
+const ICON_RESTORE = `<svg width="12" height="12" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M3 7v6h6"/><path d="M21 17a9 9 0 0 0-9-9 9 9 0 0 0-6.69 3L3 13"/></svg>`
+
 export default class extends Controller {
     static targets = ["list"]
 
@@ -117,7 +120,7 @@ export default class extends Controller {
                         #${topic.name}`
 
             if (canManage) {
-                html += `<button class="archive-topic-btn" data-action="click->comments--topics#archiveTopic" data-id="${topic.id}" title="📦">📦</button>`
+                html += `<button class="archive-topic-btn" data-action="click->comments--topics#archiveTopic" data-id="${topic.id}" title="Archive">${ICON_ARCHIVE}</button>`
                 html += `<button class="delete-topic-btn" data-action="click->comments--topics#deleteTopic" data-id="${topic.id}">&times;</button>`
             }
 
@@ -127,13 +130,13 @@ export default class extends Controller {
         // Archived topics section
         if (this.archivedTopics && this.archivedTopics.length > 0) {
             html += `<span class="topic-archived-toggle" data-action="click->comments--topics#toggleArchivedTopics">
-                      📦 ${this.archivedTopics.length}
+                      ${ICON_ARCHIVE} ${this.archivedTopics.length}
                      </span>`
             if (this.showingArchived) {
                 this.archivedTopics.forEach(topic => {
                     html += `<span class="topic-tag topic-archived" data-id="${topic.id}">
                               #${topic.name}
-                              ${canManage ? `<button class="unarchive-topic-btn" data-action="click->comments--topics#unarchiveTopic" data-id="${topic.id}" title="Restore">↩</button>` : ''}
+                              ${canManage ? `<button class="unarchive-topic-btn" data-action="click->comments--topics#unarchiveTopic" data-id="${topic.id}" title="Restore">${ICON_RESTORE}</button>` : ''}
                              </span>`
                 })
             }
@@ -316,7 +319,7 @@ export default class extends Controller {
         const confirmText = this.listTarget.dataset.confirmDeleteText || "This will delete all messages in this topic. Are you sure?"
         if (!confirm(confirmText)) return
 
-        const topicId = event.target.dataset.id
+        const topicId = event.currentTarget.dataset.id
         if (!topicId) return
 
         try {
@@ -343,7 +346,7 @@ export default class extends Controller {
 
     async archiveTopic(event) {
         event.stopPropagation()
-        const topicId = event.target.dataset.id
+        const topicId = event.currentTarget.dataset.id
         if (!topicId) return
 
         try {
@@ -370,7 +373,7 @@ export default class extends Controller {
 
     async unarchiveTopic(event) {
         event.stopPropagation()
-        const topicId = event.target.dataset.id
+        const topicId = event.currentTarget.dataset.id
         if (!topicId) return
 
         try {

data/app/javascript/controllers/creatives/expansion_controller.js

@@ -78,10 +78,10 @@ export default class extends Controller {
   updateExpandButton() {
     if (!this.hasExpandTarget) return
     const button = this.expandTarget
-    const icon = button.querySelector('span')
-    if (icon) {
-      icon.textContent = this.allExpanded ? '▶' : '▼'
-    }
+    const expandIcon = button.querySelector('.icon-expand')
+    const collapseIcon = button.querySelector('.icon-collapse')
+    if (expandIcon) expandIcon.style.display = this.allExpanded ? 'none' : ''
+    if (collapseIcon) collapseIcon.style.display = this.allExpanded ? '' : 'none'
     const expandText = button.dataset.expandText
     const collapseText = button.dataset.collapseText
     button.ariaLabel = this.allExpanded ? collapseText : expandText

data/app/javascript/controllers/creatives/tree_controller.js

@@ -55,7 +55,9 @@ export default class extends Controller {
       // Update mobile button text
       if (mobileBtn) {
         const label = this._showingArchived ? (mobileBtn.dataset.hideText || '') : (mobileBtn.dataset.showText || '')
-        mobileBtn.textContent = '📦 ' + label
+        const iconSpan = mobileBtn.querySelector('span')
+        const iconHtml = iconSpan ? iconSpan.outerHTML + ' ' : ''
+        mobileBtn.innerHTML = iconHtml + label
       }
 
       const url = new URL(this.urlValue, window.location.origin)

data/app/javascript/controllers/share_modal_controller.js

@@ -99,6 +99,9 @@ export default class extends Controller {
     modal.style.display = "flex"
     document.body.classList.add("no-scroll")
 
+    // Constrain popup-box within viewport
+    this.#constrainModalHeight(modal)
+
     const closeBtn = document.getElementById("close-share-modal")
     if (closeBtn) {
       closeBtn.onclick = () => this.close()
@@ -108,11 +111,32 @@ export default class extends Controller {
       if (e.target === modal) this.close()
     }
 
+    // Prevent touch events from propagating to underlying elements (e.g., chat swipe-to-close)
+    modal.addEventListener("touchstart", (e) => e.stopPropagation(), { passive: true })
+    modal.addEventListener("touchmove", (e) => e.stopPropagation(), { passive: true })
+    modal.addEventListener("touchend", (e) => e.stopPropagation(), { passive: true })
+
     this.#initializeForm()
+    this.#initializePermissionSelects()
     this.#initializeDeleteButtons()
     this.#initializeInviteLink()
   }
 
+  #constrainModalHeight(modal) {
+    const popupBox = modal.querySelector(".popup-box")
+    if (!popupBox) return
+
+    // Ensure the overlay is properly styled for centering
+    modal.style.display = "flex"
+    modal.style.alignItems = "center"
+    modal.style.justifyContent = "center"
+
+    // Constrain popup-box height to viewport
+    const maxH = window.innerHeight - 32 // 16px margin top + bottom
+    popupBox.style.maxHeight = `${maxH}px`
+    popupBox.style.overflowY = "auto"
+  }
+
   #initializeForm() {
     const form = document.getElementById("share-creative-form")
     if (!form) return
@@ -211,6 +235,39 @@ export default class extends Controller {
     return li
   }
 
+  #initializePermissionSelects() {
+    const modal = document.getElementById("share-creative-modal")
+    if (!modal) return
+
+    const selects = modal.querySelectorAll(".share-modal-permission-select:not([disabled])")
+    selects.forEach(select => {
+      select.addEventListener("change", () => {
+        const url = select.dataset.updateUrl
+        const permission = select.value
+        const originalClass = select.className
+
+        // Update visual class immediately
+        select.className = select.className.replace(/org-chart-permission-\w+/g, "")
+        select.classList.add("org-chart-permission-select", "share-modal-permission-select", `org-chart-permission-${permission}`)
+
+        fetch(url, {
+          method: "PATCH",
+          headers: {
+            "Content-Type": "application/json",
+            "X-CSRF-Token": document.querySelector("meta[name='csrf-token']")?.content,
+            "Accept": "application/json"
+          },
+          body: JSON.stringify({ permission })
+        }).then(response => {
+          if (!response.ok) throw new Error("Failed")
+        }).catch(() => {
+          select.className = originalClass
+          this.#showMessage(this.#errorFallbackMessage, "error")
+        })
+      })
+    })
+  }
+
   #initializeDeleteButtons() {
     const modal = document.getElementById("share-creative-modal")
     if (!modal) return

data/app/services/collavre/ai_agent/approval_handler.rb

@@ -13,7 +13,7 @@ module Collavre
         @reply_comment = reply_comment
       end
 
-      def handle(error)
+      def handle(error, summary: nil)
         cleanup_placeholder
 
         broadcast_idle
@@ -22,7 +22,7 @@ module Collavre
 
         log_action(error)
 
-        create_approval_comment(error)
+        create_approval_comment(error, summary: summary)
       end
 
       private
@@ -64,7 +64,7 @@ module Collavre
         )
       end
 
-      def create_approval_comment(error)
+      def create_approval_comment(error, summary: nil)
         return unless @creative
 
         approver = @creative.user || User.find_by(id: @context.dig("comment", "user_id"))
@@ -92,6 +92,10 @@ module Collavre
           arguments: args_display
         )
 
+        if summary.present?
+          content += "\n#{I18n.t('collavre.ai_agent.approval.summary_header')}\n#{summary}\n"
+        end
+
         original_comment = Comment.find_by(id: @context.dig("comment", "id"))
         topic_id = original_comment&.topic_id
 

data/app/services/collavre/ai_agent_service.rb

@@ -48,8 +48,8 @@ module Collavre
         @lifecycle_manager.broadcast_status("thinking")
 
         # Execute AI chat with streaming
-        client = build_ai_client(system_prompt)
-        stream_response(client, messages)
+        @client = build_ai_client(system_prompt)
+        stream_response(@client, messages)
 
         log_action("completion", { response: @streamer.content })
 
@@ -62,10 +62,11 @@ module Collavre
         @streamer.content
       end
     rescue ApprovalPendingError => e
+      summary = generate_approval_summary(e)
       AiAgent::ApprovalHandler.new(
         task: @task, agent: @agent, context: @context,
         creative: @creative, reply_comment: @reply_comment
-      ).handle(e)
+      ).handle(e, summary: summary)
       raise
     rescue CancelledError
       handle_cancelled
@@ -194,6 +195,18 @@ module Collavre
       )
     end
 
+    def generate_approval_summary(error)
+      return nil unless @client
+
+      prompt = I18n.t(
+        "collavre.ai_agent.approval.summary_prompt",
+        tool_name: error.tool_name,
+        arguments: error.tool_arguments.present? ? JSON.pretty_generate(error.tool_arguments) : "(none)"
+      )
+
+      @client.ask(prompt)
+    end
+
     def build_agent_context(creative)
       return {} unless creative
 

data/app/services/collavre/ai_client.rb

@@ -57,10 +57,10 @@ module Collavre
         Rails.logger.warn "Unsupported LLM vendor '#{@vendor}'. Attempting to use default (google)."
       end
 
-      conversation = build_conversation(tools)
-      add_messages(conversation, contents)
+      @conversation = build_conversation(tools)
+      add_messages(@conversation, contents)
 
-      response = conversation.complete do |chunk|
+      response = @conversation.complete do |chunk|
         delta = extract_chunk_content(chunk)
         next if delta.blank?
 
@@ -83,7 +83,8 @@ module Collavre
 
       response_content.presence
     rescue ApprovalPendingError
-      raise # Re-raise approval errors without catching them
+      # Preserve conversation for follow-up (e.g. generating approval summary)
+      raise
     rescue CancelledError
       raise # Re-raise cancellation errors without catching them
     rescue StandardError => e
@@ -97,8 +98,8 @@ module Collavre
       @last_input_tokens = input_tokens || 0
       @last_output_tokens = output_tokens || 0
       log_interaction(
-        messages: conversation.messages.to_a || Array(contents),
-        tools: conversation.tools.to_a,
+        messages: @conversation&.messages&.to_a || Array(contents),
+        tools: @conversation&.tools&.to_a || [],
         response_content: response_content.presence,
         error_message: error_message,
         input_tokens: input_tokens,
@@ -106,6 +107,21 @@ module Collavre
       )
     end
 
+    # Ask a follow-up question using the existing conversation context.
+    # Used to generate approval summaries with full conversation history.
+    # Returns the response content string, or nil on failure.
+    def ask(prompt)
+      return nil unless @conversation
+
+      # Disable tool calls for summary generation to avoid recursive approval
+      @conversation.with_tools(replace: true)
+      response = @conversation.ask(prompt)
+      response&.content&.strip.presence
+    rescue StandardError => e
+      Rails.logger.warn("AiClient#ask failed: #{e.class} #{e.message}")
+      nil
+    end
+
     private
 
     attr_reader :vendor, :model, :system_prompt, :llm_api_key, :context

data/app/services/collavre/tools/creative_retrieval_service.rb

@@ -25,8 +25,8 @@ module Tools
         query: T.nilable(String),
         level: T.nilable(Integer),
         tags: T.nilable(String),
-        progress_min: T.nilable(Float),
-        progress_max: T.nilable(Float),
+        progress_min: T.nilable(Numeric),
+        progress_max: T.nilable(Numeric),
         updated_since: T.nilable(String),
         include_comments: T.nilable(T::Boolean),
         format: T.nilable(String)

data/app/services/collavre/tools/creative_update_service.rb

@@ -15,7 +15,7 @@ module Tools
     tool_param :progress, description: "Set to 1.0 to mark a leaf Creative as complete. Only 1.0 is allowed; partial progress and updates on parent Creatives are rejected.", required: false
     tool_param :parent_id, description: "New parent Creative ID to move this Creative under. Use null/0 to make it a root Creative.", required: false
 
-    sig { params(id: Integer, description: T.nilable(String), progress: T.nilable(Float), parent_id: T.nilable(Integer)).returns(T::Hash[Symbol, T.untyped]) }
+    sig { params(id: Integer, description: T.nilable(String), progress: T.nilable(Numeric), parent_id: T.nilable(Integer)).returns(T::Hash[Symbol, T.untyped]) }
     def call(id:, description: nil, progress: nil, parent_id: nil)
       raise "Current.user is required" unless Current.user
 

data/app/views/collavre/comments/_comments_popup.html.erb

@@ -60,7 +60,7 @@
               data-action="click->comments--contexts#toggleVisibility"
               type="button"
               title="<%= t('collavre.contexts.toggle_label', default: 'Contexts') %>"
-              style="display:none;">🔗</button>
+              style="display:none;"><%= svg_tag "context-link.svg", class: "comments-popup-action-icon", width: 16, height: 16 %></button>
       <button class="comments-popup-action comments-popup-fullscreen"
               data-comments--popup-target="fullscreenButton"
               data-action="click->comments--popup#toggleFullscreen"

data/app/views/collavre/creatives/_mobile_actions_menu.html.erb

@@ -26,7 +26,7 @@
       <%= button_to t('collavre.creatives.index.slide_view'), slide_view_creative_path(@parent_creative), method: :get, class: 'popup-menu-item' %>
     <% end %>
     <% if authenticated? %>
-      <button type="button" id="toggle-archived-btn-mobile" class="popup-menu-item" data-show-text="<%= t('collavre.creatives.index.show_archived') %>" data-hide-text="<%= t('collavre.creatives.index.hide_archived') %>">📦 <%= t('collavre.creatives.index.show_archived') %></button>
+      <button type="button" id="toggle-archived-btn-mobile" class="popup-menu-item" data-show-text="<%= t('collavre.creatives.index.show_archived') %>" data-hide-text="<%= t('collavre.creatives.index.hide_archived') %>"><span aria-hidden="true"><%= svg_tag 'archive.svg', width: 16, height: 16 %></span> <%= t('collavre.creatives.index.show_archived') %></button>
     <% end %>
     <% if authenticated? && (!params[:id] || (current_creative && current_creative.has_permission?(Current.user, :write))) %>
       <button type="button" class="popup-menu-item" data-controller="click-target" data-click-target-id-value="import-markdown-btn" data-action="click->click-target#trigger"><%= t('collavre.creatives.index.import_markdown') %></button>