collavre 0.3.2 → 0.5.0

data/app/services/collavre/auto_theme_generator.rb

@@ -1,30 +1,55 @@
 module Collavre
   class AutoThemeGenerator
     REQUIRED_VARIABLES = %w[
-      --color-bg
-      --color-text
+      --surface-bg
+      --surface-nav
+      --surface-section
+      --surface-input
+      --surface-btn
+      --surface-secondary
+      --text-primary
+      --text-muted
+      --text-on-btn
+      --text-nav
+      --text-nav-btn
+      --text-chat-btn
+      --text-on-badge
+      --text-input
       --color-link
-      --color-nav-bg
-      --color-section-bg
-      --color-btn-bg
-      --color-btn-text
-      --color-border
-      --color-muted
-      --color-complete
-      --color-chip-bg
-      --color-drag-over
-      --color-drag-over-edge
-      --hover-brightness
+      --color-brand
+      --color-active
+      --color-danger
+      --color-success
+      --color-warning
+      --color-highlight
       --color-badge-bg
-      --color-badge-text
-      --color-secondary-active
-      --color-secondary-background
-      --color-nav-btn-text
-      --color-chat-btn-text
-      --color-input-bg
-      --color-input-text
-      --color-nav-text
+      --color-accent-border
+      --color-accent-text
+      --color-code-bg
+      --color-code-text
+      --border-color
+      --border-drag-over
+      --border-drag-edge
+      --hover-brightness
       --creative-loading-emojis
+      --creative-h1-size
+      --creative-h2-size
+      --creative-h3-size
+      --creative-h1-weight
+      --creative-h2-weight
+      --creative-h3-weight
+      --creative-h1-color
+      --creative-h2-color
+      --creative-h3-color
+      --creative-childless-size
+      --creative-childless-weight
+      --creative-bullet-size
+      --creative-bullet-color
+      --creative-tree-line-color
+      --creative-tree-line-opacity
+      --creative-h1-bg
+      --creative-h2-bg
+      --creative-h3-bg
     ].freeze
 
     def initialize(client: default_client)
@@ -33,35 +58,128 @@ module Collavre
 
     def generate(prompt)
       system_prompt = <<~PROMPT
-        You are an expert UI/UX designer specialized in creating color themes for web applications.
-        Your task is to generate a JSON object containing CSS variables.
-        Generate a CSS theme as a JSON object based on the prompt: "#{prompt}".
-        The JSON must strictly contain ONLY these keys: #{REQUIRED_VARIABLES.join(', ')}.
-
-        CRITICAL DESIGN RULES:
-        1. Use **only 'oklch()' color format** for all colors. Do not use hex, rgb, or hsl.
-        2. Ensure "--color-nav-btn-text" has High Contrast (WCAG AA/AAA) against "--color-bg" (which is used as the button background in the nav).
-        3. Ensure "--color-chat-btn-text" has High Contrast against "--color-section-bg" (where chat messages reside).
-        4. Ensure "--color-nav-text" has High Contrast against "--color-nav-bg".
-        5. Names of these text colors should be visually distinct from their background colors to ensure readability.
-        6. For "--creative-loading-emojis", provide a comma-separated string of exactly 6 emojis that match the theme mood (e.g., "🌵,🏜️,☀️,🦎,🌾,🐪").
-        7. Do not include any other keys or newlines.
-        8. Return valid JSON only.
-
-        The JSON object must strictly follow this structure:
-        {
-          "--color-bg": "oklch(95% 0.01 200)",
-          "--color-text": "oklch(20% 0.02 200)",
-          ...
-        }
-
-        REQUIRED VARIABLES:
-        #{REQUIRED_VARIABLES.join("\n")}
-
-        GUIDELINES:
-        - Ensure high contrast between text and background.
-        - Maintain a consistent aesthetic suitable for the description.
-        - Return ONLY the JSON object. No markdown formatting, no explanations.
+        You are a color theme designer for a workspace app.
+        Generate a JSON with ONLY these keys: #{REQUIRED_VARIABLES.join(', ')}.
+
+        FORMAT: hex colors (#rrggbb). --hover-brightness: "90%" (light) or "110%" (dark).
+        --creative-loading-emojis: 6 emojis. Return ONLY valid JSON, no markdown.
+
+        === REFERENCE THEMES (these are GOOD — match this quality) ===
+
+        "바나나" (yellow, light):
+        {"--surface-bg":"#fdf7d0","--surface-nav":"#e8d36b","--surface-section":"#feeec1",
+         "--surface-input":"#ffffff","--surface-btn":"#e3b831","--surface-secondary":"#f4d9bb",
+         "--text-primary":"#1a1200","--text-muted":"#5a4e00","--text-on-btn":"#241100",
+         "--text-nav":"#1a1200","--text-nav-btn":"#1a1200","--text-chat-btn":"#1a1200",
+         "--text-on-badge":"#fff8e0","--text-input":"#1a1200",
+         "--color-link":"#c69900","--color-brand":"#c69900","--color-active":"#c08500",
+         "--color-danger":"#cc3300","--color-success":"#4a8c00","--color-warning":"#cc8800",
+         "--color-highlight":"#fff3a0","--color-badge-bg":"#ca9600",
+         "--color-accent-border":"#a67a00","--color-accent-text":"#c69900",
+         "--color-code-bg":"#f5efc0","--color-code-text":"#1a1200",
+         "--border-color":"#c4b060","--border-drag-over":"#d4a800","--border-drag-edge":"#c69900",
+         "--hover-brightness":"90%","--creative-loading-emojis":"🍌🌻💛🍋✨🌼",
+         "--creative-h1-size":"1.3em","--creative-h2-size":"1.2em","--creative-h3-size":"1.1em",
+         "--creative-h1-weight":"700","--creative-h2-weight":"600","--creative-h3-weight":"500",
+         "--creative-h1-color":"#1a1200","--creative-h2-color":"#1a1200","--creative-h3-color":"#5a4e00",
+         "--creative-childless-size":"1em","--creative-childless-weight":"400",
+         "--creative-bullet-size":"5px","--creative-bullet-color":"#1a1200",
+         "--creative-tree-line-color":"#c4b060","--creative-tree-line-opacity":"0.5",
+         "--creative-h1-bg":"transparent","--creative-h2-bg":"transparent","--creative-h3-bg":"transparent"}
+
+        "숲속" (green, light):
+        {"--surface-bg":"#ecf4ef","--surface-nav":"#d5e2d7","--surface-section":"#e2f4e7",
+         "--surface-input":"#f6f9f7","--surface-btn":"#357153","--surface-secondary":"#dbe8e3",
+         "--text-primary":"#0a1f10","--text-muted":"#3a5040","--text-on-btn":"#f3fbf6",
+         "--text-nav":"#0a1f10","--text-nav-btn":"#0a1f10","--text-chat-btn":"#0a1f10",
+         "--text-on-badge":"#f3fbf6","--text-input":"#0a1f10",
+         "--color-link":"#25984d","--color-brand":"#25984d","--color-active":"#008a39",
+         "--color-danger":"#cc3300","--color-success":"#00880a","--color-warning":"#cc8800",
+         "--color-highlight":"#c8f0d0","--color-badge-bg":"#005734",
+         "--color-accent-border":"#337344","--color-accent-text":"#1a3520",
+         "--color-code-bg":"#e0ede4","--color-code-text":"#0a1f10",
+         "--border-color":"#a0bca6","--border-drag-over":"#40905a","--border-drag-edge":"#25984d",
+         "--hover-brightness":"90%","--creative-loading-emojis":"🌲🍃🌿🌱✨🦎",
+         "--creative-h1-size":"1.3em","--creative-h2-size":"1.15em","--creative-h3-size":"1.05em",
+         "--creative-h1-weight":"700","--creative-h2-weight":"600","--creative-h3-weight":"500",
+         "--creative-h1-color":"#0a1f10","--creative-h2-color":"#0a1f10","--creative-h3-color":"#3a5040",
+         "--creative-childless-size":"1em","--creative-childless-weight":"400",
+         "--creative-bullet-size":"5px","--creative-bullet-color":"#25984d",
+         "--creative-tree-line-color":"#a0bca6","--creative-tree-line-opacity":"0.5",
+         "--creative-h1-bg":"transparent","--creative-h2-bg":"transparent","--creative-h3-bg":"transparent"}
+
+        "토마토" (red/warm, light):
+        {"--surface-bg":"#fbefea","--surface-nav":"#f7ded6","--surface-section":"#f5e8e4",
+         "--surface-input":"#fff6f3","--surface-btn":"#cc0000","--surface-secondary":"#f0cfc4",
+         "--text-primary":"#1f0800","--text-muted":"#5a3020","--text-on-btn":"#fff0e8",
+         "--text-nav":"#1f0800","--text-nav-btn":"#1f0800","--text-chat-btn":"#1f0800",
+         "--text-on-badge":"#fff0e8","--text-input":"#1f0800",
+         "--color-link":"#d40924","--color-brand":"#d40924","--color-active":"#ff4040",
+         "--color-danger":"#cc0000","--color-success":"#4a8c00","--color-warning":"#cc8800",
+         "--color-highlight":"#ffe0d0","--color-badge-bg":"#ba0d01",
+         "--color-accent-border":"#c85b32","--color-accent-text":"#b22800",
+         "--color-code-bg":"#f0e4de","--color-code-text":"#1f0800",
+         "--border-color":"#d0a898","--border-drag-over":"#e04020","--border-drag-edge":"#cc0000",
+         "--hover-brightness":"90%","--creative-loading-emojis":"🍅🔴🌶️🫕✨🍝",
+         "--creative-h1-size":"1.3em","--creative-h2-size":"1.2em","--creative-h3-size":"1.1em",
+         "--creative-h1-weight":"700","--creative-h2-weight":"600","--creative-h3-weight":"500",
+         "--creative-h1-color":"#1f0800","--creative-h2-color":"#1f0800","--creative-h3-color":"#5a3020",
+         "--creative-childless-size":"1em","--creative-childless-weight":"400",
+         "--creative-bullet-size":"5px","--creative-bullet-color":"#d40924",
+         "--creative-tree-line-color":"#d0a898","--creative-tree-line-opacity":"0.5",
+         "--creative-h1-bg":"transparent","--creative-h2-bg":"transparent","--creative-h3-bg":"transparent"}
+
+        "사이버펑크" (neon, dark):
+        {"--surface-bg":"#070b14","--surface-nav":"#0f101f","--surface-section":"#12161f",
+         "--surface-input":"#02060d","--surface-btn":"#2f1d4a","--surface-secondary":"#181b1f",
+         "--text-primary":"#dcdde5","--text-muted":"#8a8c99","--text-on-btn":"#e0d0ff",
+         "--text-nav":"#dcdde5","--text-nav-btn":"#dcdde5","--text-chat-btn":"#dcdde5",
+         "--text-on-badge":"#dcdde5","--text-input":"#dcdde5",
+         "--color-link":"#0099f0","--color-brand":"#e749df","--color-active":"#0089e9",
+         "--color-danger":"#ff3050","--color-success":"#00cc66","--color-warning":"#ffaa00",
+         "--color-highlight":"#2a1848","--color-badge-bg":"#692278",
+         "--color-accent-border":"#0094c9","--color-accent-text":"#eb63c5",
+         "--color-code-bg":"#0e1220","--color-code-text":"#c0c4d0",
+         "--border-color":"#2a2e3a","--border-drag-over":"#6030a0","--border-drag-edge":"#e749df",
+         "--hover-brightness":"110%","--creative-loading-emojis":"🌃💜⚡🤖✨🎮",
+         "--creative-h1-size":"1.4em","--creative-h2-size":"1.2em","--creative-h3-size":"1.1em",
+         "--creative-h1-weight":"800","--creative-h2-weight":"600","--creative-h3-weight":"500",
+         "--creative-h1-color":"#e749df","--creative-h2-color":"#0099f0","--creative-h3-color":"#8a8c99",
+         "--creative-childless-size":"1em","--creative-childless-weight":"400",
+         "--creative-bullet-size":"4px","--creative-bullet-color":"#e749df",
+         "--creative-tree-line-color":"#2a2e3a","--creative-tree-line-opacity":"0.7",
+         "--creative-h1-bg":"#0f101f","--creative-h2-bg":"#12161f","--creative-h3-bg":"transparent"}
+
+        === RULES ===
+
+        1) SURFACES must be TINTED with the theme color — never gray or neutral.
+           bg is lightest (pastel), then input, section, secondary, nav (deeper), btn (deepest or vivid).
+        2) ALL text must be DARK (#0a-#2f range) on light themes, LIGHT (#cc-#ff range) on dark themes.
+           Exception: text-on-btn must contrast with surface-btn. If btn is dark, text-on-btn is light.
+        3) brand/link = vivid theme color. danger=red, success=green, warning=amber ALWAYS.
+        4) code-bg = same hue as surface-bg, slightly darker. NEVER a different hue family.
+        5) Multi-color prompts (e.g. "Google logo", "rainbow", "Italian flag"):
+           - Surfaces: use the DOMINANT color's hue (tinted, not gray)
+           - Distribute each distinct color to: brand, active, badge-bg, accent-border, accent-text
+           - Each color must be RECOGNIZABLE — never blend into one muddy tone
+        6) Match the vibe: "토마토" = warm reds/oranges, "바나나" = bright yellows, "숲" = rich greens.
+           The user should IMMEDIATELY recognize the theme from its colors.
+        7) CREATIVE TREE STYLING — the app has a hierarchical list (outliner) with levels:
+           - Level 1-3: headings (h1/h2/h3). Level 4+: bullet items.
+           - Items at level 1-3 WITHOUT children render as plain text (childless style).
+           - creative-h{1,2,3}-size: font size (em units). h1 largest, descending. Range: 1.0em-1.5em.
+           - creative-h{1,2,3}-weight: font weight. h1 boldest. Values: 400-800.
+           - creative-h{1,2,3}-color: heading colors. Use text-primary or theme accent for h1.
+             h3 can use muted color. MUST be readable on surface-bg.
+           - creative-childless-size: usually 1em (normal text size).
+           - creative-childless-weight: usually 400 (normal weight).
+           - creative-bullet-size: bullet dot diameter in px. Range: 3px-6px.
+           - creative-bullet-color: bullet color. Can match brand or text-primary.
+           - creative-tree-line-color: vertical guide line color. Usually border-color or muted.
+           - creative-tree-line-opacity: 0.3-0.8. Subtle but visible.
+           - creative-h{1,2,3}-bg: heading row background color. Usually "transparent".
+             For dark/accent themes, h1 can use a subtle darker surface. Must not clash with text.
+           - For playful themes, vary heading colors. For minimal themes, keep sizes uniform.
       PROMPT
 
       response = @client.chat([

data/app/services/collavre/command_menu_service.rb

@@ -0,0 +1,70 @@
+module Collavre
+  class CommandMenuService
+    def initialize(user:)
+      @user = user
+    end
+
+    def items
+      built_in_items + mcp_items
+    end
+
+    private
+
+    attr_reader :user
+
+    def built_in_items
+      [
+        {
+          name: "calendar",
+          label: "/calendar",
+          aliases: [ "/cal" ],
+          description: I18n.t("collavre.comments.command_menu.calendar_description"),
+          args: I18n.t("collavre.comments.command_menu.calendar_args")
+        },
+        {
+          name: "topic",
+          label: "/topic",
+          description: I18n.t("collavre.comments.command_menu.topic_description"),
+          args: I18n.t("collavre.comments.command_menu.topic_args")
+        }
+      ]
+    end
+
+    def mcp_items
+      Collavre::McpService.available_tools(user).filter_map do |tool|
+        tool_name = tool[:name] || tool["name"]
+        next unless tool_name
+
+        {
+          name: tool_name,
+          label: "/#{tool_name}",
+          description: tool[:description] || tool["description"],
+          args: format_args(tool[:params] || tool["params"])
+        }
+      end
+    end
+
+    def format_args(params)
+      return if params.blank?
+
+      if params.is_a?(Array)
+        return params.map do |param|
+          name = param[:name] || param["name"]
+          required = param[:required] || param["required"]
+          name.to_s + (required ? "*" : "")
+        end.join(", ")
+      end
+
+      properties = params[:properties] || params["properties"]
+      return unless properties.is_a?(Hash)
+
+      required = params[:required] || params["required"] || []
+      required = Array(required).map(&:to_s)
+
+      properties.keys.map do |key|
+        key = key.to_s
+        required.include?(key) ? "#{key}*" : key
+      end.join(", ")
+    end
+  end
+end

data/app/services/collavre/comment_move_service.rb

@@ -0,0 +1,94 @@
+module Collavre
+  class CommentMoveService
+    class MoveError < StandardError; end
+
+    def initialize(creative:, user:)
+      @creative = creative
+      @user = user
+    end
+
+    # Returns { success: true, moved_count: N }
+    # Raises MoveError on validation failures
+    def call(comment_ids:, target_creative_id: nil, target_topic_id: nil)
+      comment_ids = Array(comment_ids).map(&:presence).compact.map(&:to_i)
+      raise MoveError, I18n.t("collavre.comments.move_no_selection") if comment_ids.empty?
+
+      target_origin, new_topic_id = resolve_target(target_creative_id, target_topic_id)
+
+      validate_permissions!(target_origin)
+      comments = fetch_visible_comments(comment_ids)
+
+      moved_count = perform_move(comments, target_origin, new_topic_id)
+
+      Comment.broadcast_badges(@creative)
+      Comment.broadcast_badges(target_origin) unless target_origin == @creative
+
+      { success: true, moved_count: moved_count }
+    end
+
+    private
+
+    attr_reader :creative, :user
+
+    def resolve_target(target_creative_id, target_topic_id)
+      if target_creative_id.present?
+        target_creative = Creative.find_by(id: target_creative_id)
+        raise MoveError, I18n.t("collavre.comments.move_invalid_target") unless target_creative
+        [ target_creative.effective_origin, nil ]
+      elsif !target_topic_id.nil?
+        new_topic_id = target_topic_id.presence
+        if new_topic_id.present? && !creative.topics.exists?(id: new_topic_id)
+          raise MoveError, I18n.t("collavre.comments.move_invalid_topic", default: "Invalid topic")
+        end
+        [ creative, new_topic_id ]
+      else
+        raise MoveError, I18n.t("collavre.comments.move_invalid_target")
+      end
+    end
+
+    def validate_permissions!(target_origin)
+      unless creative.has_permission?(user, :feedback) && target_origin.has_permission?(user, :feedback)
+        raise MoveError, I18n.t("collavre.comments.move_not_allowed")
+      end
+    end
+
+    def fetch_visible_comments(comment_ids)
+      scope = creative.comments.where(
+        "comments.private = ? OR comments.user_id = ? OR comments.approver_id = ?",
+        false, user.id, user.id
+      )
+      comments = scope.where(id: comment_ids).to_a
+      raise MoveError, I18n.t("collavre.comments.move_not_allowed") if comments.length != comment_ids.length
+      comments
+    end
+
+    def perform_move(comments, target_origin, new_topic_id)
+      moved_count = 0
+      ActiveRecord::Base.transaction do
+        comments.each do |comment|
+          same_creative = comment.creative_id == target_origin.id
+          same_topic = comment.topic_id.to_s == new_topic_id.to_s
+          next if same_creative && same_topic
+
+          if same_creative
+            comment.update!(topic_id: new_topic_id)
+          else
+            comment.update!(creative: target_origin, topic_id: new_topic_id)
+            broadcast_move_removal(comment, comment.creative)
+          end
+          moved_count += 1
+        end
+      end
+      moved_count
+    end
+
+    def broadcast_move_removal(comment, original_creative)
+      return if comment.private?
+
+      Turbo::StreamsChannel.broadcast_remove_to(
+        [ original_creative, :comments ],
+        target: ActionView::RecordIdentifier.dom_id(comment)
+      )
+    end
+  end
+end

data/app/services/collavre/comments/action_executor.rb

@@ -73,6 +73,7 @@ module Collavre
         SUPPORTED_ACTIONS = {
           "create_creative" => :create_creative,
           "update_creative" => :update_creative,
+          "delete_creative" => :delete_creative,
           "approve_tool" => :approve_tool,
           "execute_tool" => :execute_tool
         }.freeze
@@ -150,6 +151,15 @@ module Collavre
           raise InvalidActionError, e.record.errors.full_messages.to_sentence
         end
 
+        def delete_creative(payload)
+          creative = find_target_creative(payload)
+          executor = comment.user || Current.user
+          unless creative.has_permission?(executor, :write)
+            raise InvalidActionError, I18n.t("collavre.comments.approve_no_write_permission")
+          end
+          creative.destroy!
+        end
+
         def approve_tool(payload)
           tool_name = payload["tool_name"]
           raise InvalidActionError, "Tool name is required" if tool_name.blank?

data/app/services/collavre/comments/mcp_command.rb

@@ -79,8 +79,7 @@ module Collavre
       end
 
       def format_response(response)
-        # TODO: i18n text
-        return "Error running /#{tool_name}: #{response[:error]}" if response[:error].present?
+        return I18n.t("collavre.comments.mcp_command.error_running", tool_name: tool_name, error: response[:error]) if response[:error].present?
 
         result = response[:result]
         content = case result

data/app/services/collavre/creatives/create_service.rb

@@ -0,0 +1,86 @@
+# frozen_string_literal: true
+
+module Collavre
+  module Creatives
+    # Handles creative creation including sequencing and tagging.
+    class CreateService
+      Result = Struct.new(:creative, :success?, :errors, keyword_init: true)
+
+      def initialize(creative_params:, user:, child_id: nil, before_id: nil, after_id: nil, tag_ids: nil)
+        @creative_params = creative_params
+        @user = user
+        @child_id = child_id
+        @before_id = before_id
+        @after_id = after_id
+        @tag_ids = Array(tag_ids).compact
+      end
+
+      def call
+        creative = Creative.new(@creative_params)
+        creative.user = creative.parent ? creative.parent.user : @user
+
+        unless creative.save
+          return Result.new(creative: creative, success?: false, errors: creative.errors.full_messages)
+        end
+
+        reparent_child(creative)
+        insert_at_position(creative)
+        apply_tags(creative)
+
+        Result.new(creative: creative, success?: true, errors: [])
+      end
+
+      private
+
+      def reparent_child(creative)
+        return unless @child_id.present?
+
+        child = Creative.find_by(id: @child_id)
+        child&.update(parent: creative)
+      end
+
+      def insert_at_position(creative)
+        if @before_id.present?
+          insert_before(creative, @before_id)
+        elsif @after_id.present?
+          insert_after(creative, @after_id)
+        end
+      end
+
+      def insert_before(creative, before_id)
+        before_creative = Creative.find_by(id: before_id)
+        return unless before_creative && before_creative.parent_id == creative.parent_id
+
+        siblings = fetch_siblings(creative)
+        index = siblings.index { |s| s.id == before_creative.id } || 0
+        siblings.insert(index, creative)
+        resequence(siblings)
+      end
+
+      def insert_after(creative, after_id)
+        after_creative = Creative.find_by(id: after_id)
+        return unless after_creative && after_creative.parent_id == creative.parent_id
+
+        siblings = fetch_siblings(creative)
+        index = siblings.index { |s| s.id == after_creative.id } || -1
+        siblings.insert(index + 1, creative)
+        resequence(siblings)
+      end
+
+      def fetch_siblings(creative)
+        collection = creative.parent ? creative.parent.children.order(:sequence) : Creative.roots.order(:sequence)
+        collection.to_a.reject { |s| s.id == creative.id }
+      end
+
+      def resequence(siblings)
+        siblings.each_with_index { |c, idx| c.update_column(:sequence, idx) }
+      end
+
+      def apply_tags(creative)
+        @tag_ids.each do |tag_id|
+          creative.tags.create(label_id: tag_id)
+        end
+      end
+    end
+  end
+end

data/app/services/collavre/creatives/destroy_service.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+module Collavre
+  module Creatives
+    # Handles creative destruction with optional recursive child deletion.
+    class DestroyService
+      def initialize(creative:, user:, delete_with_children: false)
+        @creative = creative
+        @user = user
+        @delete_with_children = delete_with_children
+      end
+
+      def call
+        if @delete_with_children
+          destroy_descendants_recursively(@creative)
+        else
+          reparent_children
+        end
+
+        CreativeShare.where(creative: @creative).destroy_all
+        @creative.destroy
+      end
+
+      private
+
+      def reparent_children
+        parent = @creative.parent
+        @creative.children.each { |child| child.update(parent: parent) }
+      end
+
+      def destroy_descendants_recursively(creative)
+        deletable_children = creative.children_with_permission(@user, :admin)
+        deletable_children.each do |child|
+          destroy_descendants_recursively(child)
+          CreativeShare.where(creative: child).destroy_all
+          child.destroy
+        end
+      end
+    end
+  end
+end

data/app/services/collavre/creatives/index_query.rb

@@ -82,6 +82,9 @@ module Creatives
         # Sort by comment updated_at for comment filter
         if params[:comment] == "true"
           matched_creatives = matched_creatives.sort_by { |c| c.comments.maximum(:updated_at) || c.updated_at }.reverse
+        elsif params[:search].present? && params[:simple].present?
+          # Sort by description length (shorter = more relevant match)
+          matched_creatives = matched_creatives.sort_by { |c| c.description.to_s.length }
         end
 
         parent = params[:id] ? Creative.find_by(id: params[:id]) : nil