senren-ui 0.1.4 → 0.1.6

data/lib/senren/rails/llms_writer.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-require 'yaml'
-
 module Senren
   module Rails
-    # Generates public/llms.txt and public/llms-full.txt for AI discoverability.
-    # The entire content of both files is owned by this writer; do not edit by hand.
+    # Backward-compatible wrapper around AgentRulesWriter.
+    #
+    # Legacy llms generation now maps to agent rules synchronization and no longer
+    # writes public llms files.
     class LlmsWriter
       attr_reader :registry, :paths
 
@@ -15,134 +15,7 @@ module Senren
       end
 
       def generate!
-        paths.llms_short.parent.mkpath
-        paths.llms_full.parent.mkpath
-        atomic_write(paths.llms_short, render_short)
-        atomic_write(paths.llms_full,  render_full)
-        [paths.llms_short, paths.llms_full]
-      end
-
-      private
-
-      def installed
-        path = paths.installed_components
-        return [] unless path.exist?
-
-        ledger = YAML.safe_load_file(path) || {}
-        Array(ledger['installed']).filter_map { |e| registry.find(e['name']) }
-      end
-
-      def render_short
-        names = installed.map(&:name).sort
-        <<~TXT
-          # Senren UI
-
-          Generated by senren-ui. Do not edit by hand. Run
-          `bin/rails senren:llms:generate` to regenerate.
-
-          Senren UI is the local Rails UI system used by this application.
-          Use `.senren/skill.md` as the primary AI Agent guide.
-
-          ## Hard Rules
-
-          - Use Senren components before writing custom HTML.
-          - Use ViewComponent for reusable UI.
-          - Use Turbo for server state.
-          - Use Stimulus only for local behavior.
-          - Do not introduce React, Vue, Alpine, or any external state framework.
-          - Do not hard-code colors; use semantic Tailwind tokens.
-
-          ## Important Files
-
-          - `.senren/skill.md`              - centralized AI agent guide
-          - `.senren/registry.yml`          - mirror of installable components
-          - `.senren/installed_components.yml` - what is currently installed
-          - `.senren/conventions.md`        - Senren conventions for humans and agents
-
-          ## Installed Components (#{names.size})
-
-          #{names.map { |n| "- #{n}" }.join("\n")}
-        TXT
-      end
-
-      def render_full
-        comps = installed.sort_by(&:name)
-
-        out = []
-        out << '# Senren UI - Full Snapshot'
-        out << ''
-        out << 'Generated by senren-ui. Do not edit by hand. Run'
-        out << '`bin/rails senren:llms:generate` to regenerate.'
-        out << ''
-        out << '## Hard Rules'
-        out << ''
-        out << '- Use Senren components before writing custom HTML.'
-        out << '- Server-rendered HTML first; ViewComponent for reusable UI.'
-        out << '- Hotwire (Turbo + Stimulus) is the only client runtime.'
-        out << '- TailwindCSS with semantic tokens (`bg-background`, `text-foreground`, ...).'
-        out << '- Do not introduce React, Vue, Alpine, or external state frameworks.'
-        out << '- Components copied into `app/components/senren/` are owned by this app; edit them directly.'
-        out << ''
-        out << '## Installed Component Inventory'
-        out << ''
-
-        registry.groups.each do |group|
-          group_comps = comps.select { |c| c.category == group['id'] }
-          next if group_comps.empty?
-
-          out << "### #{group['title']}"
-          out << ''
-          out << group['description'].to_s
-          out << ''
-          group_comps.each { |c| out << render_component(c) }
-        end
-
-        out << '## Recipes'
-        out << ''
-        registry.recipes.each do |id, recipe|
-          out << "### #{id}"
-          out << ''
-          out << recipe['description'].to_s
-          out << ''
-          out << 'Components:'
-          recipe['components'].each { |c| out << "- #{c}" }
-          out << ''
-        end
-
-        out.join("\n")
-      end
-
-      def render_component(comp)
-        ruby_class = "Senren::#{comp.name.split('_').map { |w| w[0].upcase + w[1..] }.join}Component"
-        s = []
-        s << "#### #{comp.name}#{'  (stub)' if comp.stub?}"
-        s << ''
-        s << "Category: #{comp.category}.  " \
-             "Client: #{comp.client? ? "yes (#{comp.controller})" : 'no'}.  " \
-             "Variants: #{comp.variants.empty? ? 'none' : comp.variants.join(', ')}."
-        s << ''
-        s << "Use for: #{comp.use_for.join('; ')}." unless comp.use_for.empty?
-        s << "Avoid: #{comp.avoid.join('; ')}." unless comp.avoid.empty?
-        s << ''
-        s << 'Rails usage:'
-        s << ''
-        s << '```erb'
-        s << if comp.variants.any?
-               "<%= render #{ruby_class}.new(variant: :#{comp.variants.first}) do %>"
-             else
-               "<%= render #{ruby_class}.new do %>"
-             end
-        s << '  ...'
-        s << '<% end %>'
-        s << '```'
-        s << ''
-        s.join("\n")
-      end
-
-      def atomic_write(path, content)
-        tmp = "#{path}.tmp"
-        File.write(tmp, content)
-        File.rename(tmp, path)
+        AgentRulesWriter.new(registry: registry, paths: paths).sync!
       end
     end
   end

data/lib/senren/rails/registry.rb

@@ -3,23 +3,16 @@ require 'yaml'
 module Senren
   module Rails
     # Loads, validates, and queries the Senren component registry.
-    #
-    #   reg = Senren::Rails::Registry.load!
-    #   reg.find("button")           # => Component struct
-    #   reg.dependencies("dialog")    # => [<button>]
-    #   reg.group("forms")            # => [<form>, <input>, ...]
     class Registry
       include Enumerable
 
       REQUIRED_KEYS = %w[category client can_have_client files depends_on pairs_with variants accessibility ai].freeze
+      OPTIONAL_KEYS = %w[controller stub].freeze
+      ALLOWED_KEYS = (REQUIRED_KEYS + OPTIONAL_KEYS).freeze
       VALID_CATEGORIES = %w[actions forms overlays navigation layout data saas rich].freeze
 
-      Component = Struct.new(
-        :name, :category, :client, :can_have_client, :controller, :stub,
-        :files, :depends_on, :pairs_with, :variants, :accessibility,
-        :use_for, :avoid,
-        keyword_init: true
-      ) do
+      Component = Struct.new(:name, :category, :client, :can_have_client, :controller, :stub, :files, :depends_on,
+                             :pairs_with, :variants, :accessibility, :use_for, :avoid, keyword_init: true) do
         def stub? = stub == true
         def client? = client == true
 
@@ -43,6 +36,7 @@ module Senren
       end
 
       def initialize(components_yaml, groups_yaml, recipes_yaml)
+        @raw_components = (components_yaml || {}).fetch('components', {})
         @components = parse_components(components_yaml)
         @groups     = (groups_yaml || {}).fetch('groups', [])
         @recipes    = (recipes_yaml || {}).fetch('recipes', {})
@@ -57,21 +51,11 @@ module Senren
                                            "Known: #{@components.keys.sort.join(', ')}"
       end
 
-      def all
-        @components.values
-      end
-
-      def each(&)
-        all.each(&)
-      end
+      def all = @components.values
+      def each(&) = all.each(&)
+      alias find_each each
 
-      def find_each(&)
-        each(&)
-      end
-
-      def names
-        @components.keys
-      end
+      def names = @components.keys
 
       def group(category_id)
         @components.values.select { |c| c.category == category_id.to_s }
@@ -118,12 +102,48 @@ module Senren
 
       def validate_components(errors)
         @components.each do |name, comp|
-          errors << "#{name}: invalid category #{comp.category.inspect}" unless VALID_CATEGORIES.include?(comp.category)
-          comp.depends_on.each do |dep|
-            errors << "#{name}: depends_on unknown component #{dep.inspect}" unless @components.key?(dep)
-          end
-          errors << "#{name}: client=true but can_have_client=false" if comp.client && !comp.can_have_client
-          errors << "#{name}: client=true requires a controller identifier" if comp.client && comp.controller.nil?
+          validate_component(name, comp, errors)
+        end
+      end
+
+      def validate_component(name, comp, errors)
+        validate_component_keys(name, errors)
+        validate_component_category(name, comp, errors)
+        validate_component_dependencies(name, comp, errors)
+        validate_component_client_contract(name, comp, errors)
+        validate_component_file_paths(name, comp, errors)
+      end
+
+      def validate_component_keys(name, errors)
+        extra_keys = @raw_components.fetch(name).keys - ALLOWED_KEYS
+        errors << "#{name}: unknown keys #{extra_keys.sort.join(', ')}" if extra_keys.any?
+      end
+
+      def validate_component_category(name, comp, errors)
+        return if VALID_CATEGORIES.include?(comp.category)
+
+        errors << "#{name}: invalid category #{comp.category.inspect}"
+      end
+
+      def validate_component_dependencies(name, comp, errors)
+        comp.depends_on.each do |dep|
+          errors << "#{name}: depends_on unknown component #{dep.inspect}" unless @components.key?(dep)
+        end
+      end
+
+      def validate_component_client_contract(name, comp, errors)
+        errors << "#{name}: client=true but can_have_client=false" if comp.client && !comp.can_have_client
+        errors << "#{name}: client=true requires a controller identifier" if comp.client && comp.controller.nil?
+        return unless comp.client && !controller_file?(name, comp)
+
+        errors << "#{name}: client=true requires a Stimulus controller file"
+      end
+
+      def validate_component_file_paths(name, comp, errors)
+        comp.files.each do |path|
+          next if allowed_component_file?(name, path)
+
+          errors << "#{name}: invalid file path #{path.inspect}"
         end
       end
 
@@ -156,6 +176,18 @@ module Senren
           ).freeze
         end
       end
+
+      def controller_file?(name, comp)
+        comp.files.include?("app/javascript/controllers/senren/#{name}_controller.js")
+      end
+
+      def allowed_component_file?(name, path)
+        [
+          "app/components/senren/#{name}_component.rb",
+          "app/components/senren/#{name}_component.html.erb",
+          "app/javascript/controllers/senren/#{name}_controller.js"
+        ].include?(path)
+      end
     end
   end
 end

data/lib/senren/rails/skill_writer.rb

@@ -50,7 +50,7 @@ module Senren
 
       def render(installed_names)
         if installed_names.empty?
-          return '_No Senren components installed yet. Run `bin/rails senren:add <name>...` to install components._'
+          return '_No Senren components installed yet. Run `bin/rails senren:add button` to install components._'
         end
 
         lines = []

data/lib/senren/rails/version.rb

@@ -2,6 +2,6 @@
 
 module Senren
   module Rails
-    VERSION = '0.1.4'
+    VERSION = '0.1.6'
   end
 end

data/lib/senren/rails.rb

@@ -8,7 +8,9 @@ module Senren
 
     autoload :Registry,         'senren/rails/registry'
     autoload :ComponentCopier,  'senren/rails/component_copier'
+    autoload :ComponentInstaller, 'senren/rails/component_installer'
     autoload :SkillWriter,      'senren/rails/skill_writer'
+    autoload :AgentRulesWriter, 'senren/rails/agent_rules_writer'
     autoload :LlmsWriter,       'senren/rails/llms_writer'
     autoload :Installer,        'senren/rails/installer'
     autoload :Doctor,           'senren/rails/doctor'

data/lib/tasks/senren.rake

@@ -3,40 +3,49 @@
 require 'senren/rails'
 
 namespace :senren do
-  desc 'Install one or more Senren components: rake senren:add[button,dialog] or bin/rails senren:add button dialog'
-  task :add, [:names] => :environment do |_t, args|
+  desc 'Install one or more Senren components. Preferred: bin/rails senren:add button dialog. ' \
+       "Legacy: bin/rails 'senren:add[button,dialog]'"
+  task :add, [:names] do |_t, args|
     names = parse_names(args)
     options = parse_options
-    abort 'Usage: bin/rails senren:add NAME [NAME...] [--client | --no-client]' if names.empty?
 
-    paths    = Senren::Rails::HostPaths.new
-    registry = Senren::Rails::Registry.load!
-    copier   = Senren::Rails::ComponentCopier.new(registry: registry, paths: paths)
-
-    installed = copier.install(names, client_override: options[:client_override], force: options[:force])
-
-    Senren::Rails::SkillWriter.new(registry: registry, paths: paths).sync!
-    Senren::Rails::LlmsWriter.new(registry: registry, paths: paths).generate!
-
-    puts "Installed: #{installed.join(', ')}"
+    Senren::Rails::ComponentInstaller.new.install(
+      names: names,
+      client_override: options[:client_override],
+      force: options[:force]
+    )
+  rescue ArgumentError => e
+    abort e.message
   end
 
   namespace :skill do
-    desc 'Rebuild .senren/skill.md from the registry and installed_components ledger.'
+    desc 'Rebuild .senren/skill.md and refresh agent instruction adapters.'
     task sync: :environment do
       paths    = Senren::Rails::HostPaths.new
       registry = Senren::Rails::Registry.load!
       file = Senren::Rails::SkillWriter.new(registry: registry, paths: paths).sync!
       puts "Wrote #{file}"
+      Senren::Rails::AgentRulesWriter.new(registry: registry, paths: paths).sync!
+    end
+  end
+
+  namespace :agents do
+    desc 'Regenerate .senren/agent-rules.md and adapter instruction files.'
+    task sync: :environment do
+      paths    = Senren::Rails::HostPaths.new
+      registry = Senren::Rails::Registry.load!
+      files = Senren::Rails::AgentRulesWriter.new(registry: registry, paths: paths).sync!
+      files.each { |f| puts "Wrote #{f}" }
     end
   end
 
   namespace :llms do
-    desc 'Regenerate public/llms.txt and public/llms-full.txt.'
+    desc 'Deprecated alias for senren:agents:sync.'
     task generate: :environment do
+      puts 'senren:llms:generate is deprecated. Running senren:agents:sync instead.'
       paths    = Senren::Rails::HostPaths.new
       registry = Senren::Rails::Registry.load!
-      files = Senren::Rails::LlmsWriter.new(registry: registry, paths: paths).generate!
+      files = Senren::Rails::AgentRulesWriter.new(registry: registry, paths: paths).sync!
       files.each { |f| puts "Wrote #{f}" }
     end
   end
@@ -55,11 +64,7 @@ def parse_names(args)
   raw.concat(args.extras)
   raw << args[:names] if args[:names]
   raw.concat(ARGV.drop_while { |a| !a.start_with?('senren:') }.drop(1))
-  raw
-    .flatten
-    .flat_map { |s| s.to_s.split(/[,\s]+/) }
-    .reject { |s| s.empty? || s.start_with?('-') }
-    .uniq
+  Senren::Rails::ComponentInstaller.normalize_names(raw)
 end
 
 def parse_options

data/templates/components/billing_plan_card/billing_plan_card_component.html.erb

@@ -22,7 +22,7 @@
       </ul>
     <% end %>
     <% if cta_label.present? %>
-      <%= link_to cta_label, cta_href || "#", class: "mt-6 inline-flex h-10 w-full items-center justify-center rounded-(--senren-radius) bg-[hsl(var(--senren-primary))] px-4 text-sm font-medium text-[hsl(var(--senren-primary-foreground))] hover:opacity-90" %>
+      <%= link_to cta_label, safe_url(cta_href), class: "mt-6 inline-flex h-10 w-full items-center justify-center rounded-(--senren-radius) bg-[hsl(var(--senren-primary))] px-4 text-sm font-medium text-[hsl(var(--senren-primary-foreground))] hover:opacity-90" %>
     <% end %>
   <% end %>
 <% end %>

data/templates/components/breadcrumb/breadcrumb_component.rb

@@ -19,10 +19,10 @@ module Senren
     def normalize_items(items)
       Array(items).map do |item|
         if item.is_a?(Hash)
-          { label: item.fetch(:label) { item.fetch('label') }, href: item[:href] || item['href'] }
+          { label: item.fetch(:label) { item.fetch('label') }, href: safe_url(item[:href] || item['href'], fallback: nil) }
         else
           label, href = item
-          { label: label, href: href }
+          { label: label, href: safe_url(href, fallback: nil) }
         end
       end
     end

data/templates/components/button/button_component.html.erb

@@ -1,6 +1,6 @@
 <% base = "inline-flex items-center justify-center gap-2 whitespace-nowrap rounded-(--senren-radius) font-medium transition-colors cursor-pointer focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-[hsl(var(--senren-ring))] disabled:opacity-50 disabled:pointer-events-none disabled:cursor-not-allowed" %>
 <% if as == :a %>
-  <%= tag.a content, **root_attrs(base, href: href) %>
+  <%= tag.a content, **root_attrs(base, href: safe_url(href)) %>
 <% else %>
   <%= tag.button content, **root_attrs(base, type: type) %>
 <% end %>

data/templates/components/carousel/carousel_component.rb

@@ -21,7 +21,7 @@ module Senren
           {
             title: slide[:title] || slide['title'],
             description: slide[:description] || slide['description'],
-            image_url: slide[:image_url] || slide['image_url'],
+            image_url: safe_media_url(slide[:image_url] || slide['image_url']),
             alt: slide[:alt] || slide['alt'],
             badge: slide[:badge] || slide['badge']
           }

data/templates/components/command/command_component.rb

@@ -29,7 +29,7 @@ module Senren
           id: data[:id] || data['id'] || "#{dom_id}-option-#{index}",
           label: label,
           description: description,
-          href: data[:href] || data['href'],
+          href: safe_url(data[:href] || data['href'], fallback: nil),
           keywords: [label, description, keywords].flatten.compact.join(' ')
         }
       end

data/templates/components/dropdown_menu/dropdown_menu_component.rb

@@ -12,23 +12,26 @@ module Senren
       super(variant: :default, size: :md, class_name: class_name, **html)
     end
 
-    class ItemTag < ViewComponent::Base
-      def initialize(href: nil, method: nil, destructive: false, **opts)
+    class ItemTag < BaseComponent
+      VARIANTS = { default: '' }.freeze
+      SIZES = { md: '' }.freeze
+      ITEM_ACTION = 'click->senren--dropdown-menu#close keydown->senren--dropdown-menu#onItemKey'
+
+      def initialize(href: nil, method: nil, destructive: false, class_name: nil, **)
+        super(variant: :default, size: :md, class_name: class_name, **)
         @href = href
         @method = method
         @destructive = destructive
-        @opts = opts
       end
 
       def call
         klass = 'block w-full text-left px-3 py-2 text-sm rounded-sm hover:bg-[hsl(var(--senren-accent))] focus:bg-[hsl(var(--senren-accent))] outline-none cursor-pointer'
+        klass += " #{class_name}" if class_name.present?
         klass += ' text-[hsl(var(--senren-destructive))]' if @destructive
         if @href
-          link_to(content, @href, role: 'menuitem', method: @method, class: klass,
-                                  data: { action: 'click->senren--dropdown-menu#close keydown->senren--dropdown-menu#onItemKey' })
+          link_to(content, safe_url(@href), role: 'menuitem', method: @method, class: klass, data: { action: ITEM_ACTION }, **html_attrs)
         else
-          tag.button(content, type: 'button', role: 'menuitem', class: klass,
-                              data: { action: 'click->senren--dropdown-menu#close keydown->senren--dropdown-menu#onItemKey' }, **@opts)
+          tag.button(content, type: 'button', role: 'menuitem', class: klass, data: { action: ITEM_ACTION }, **html_attrs)
         end
       end
     end

data/templates/components/form/form_component.html.erb

@@ -1,3 +1,10 @@
-<%= form_with(model: model, url: url, method: method, multipart: multipart, **root_attrs("space-y-4")) do |f| %>
+<%
+  form_opts = {}
+  form_opts[:model]     = model     if model
+  form_opts[:url]       = url       if url
+  form_opts[:method]    = method    if method
+  form_opts[:multipart] = multipart if multipart
+%>
+<%= form_with(**form_opts, **root_attrs("space-y-4")) do |f| %>
   <%= content || capture(f, &Proc.new) %>
 <% end %>

data/templates/components/form/form_component.rb

@@ -5,7 +5,9 @@ module Senren
     VARIANTS = { default: '' }.freeze
     SIZES    = { md: '' }.freeze
 
-    def initialize(model: nil, url: nil, method: :post, multipart: false, class_name: nil, **html)
+    # method: defaults to nil so Rails can infer PATCH for persisted models.
+    # Pass method: :post / :patch / :delete explicitly only when needed.
+    def initialize(model: nil, url: nil, method: nil, multipart: false, class_name: nil, **html)
       super(variant: :default, size: :md, class_name: class_name, **html)
       @model = model
       @url   = url

data/templates/components/input/input_component.html.erb

@@ -1 +1 @@
-<%= tag.input(**root_attrs("flex w-full rounded-(--senren-radius) border bg-[hsl(var(--senren-background))] text-[hsl(var(--senren-foreground))] file:border-0 file:bg-transparent file:text-sm file:font-medium placeholder:text-[hsl(var(--senren-muted-foreground))] focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-offset-0 disabled:cursor-not-allowed disabled:opacity-50", id: id, name: name, type: type, value: value, placeholder: placeholder, "aria-invalid": variant == :error)) %>
+<%= tag.input(**root_attrs(input_classes, id: id, name: name, type: type, value: value, placeholder: placeholder, "aria-invalid": variant == :error)) %>

data/templates/components/input/input_component.rb

@@ -13,6 +13,17 @@ module Senren
       lg: 'h-12 text-base px-4'
     }.freeze
 
+    # Base classes shared by all input types.
+    # NOTE: `flex` is intentionally omitted — it breaks browser-native
+    #       date/datetime-local/time picker UI on some engines.
+    BASE_CLASSES = 'w-full rounded-(--senren-radius) border bg-[hsl(var(--senren-background))] text-[hsl(var(--senren-foreground))] placeholder:text-[hsl(var(--senren-muted-foreground))] focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-offset-0 disabled:cursor-not-allowed disabled:opacity-50'
+
+    # File-type inputs get a styled button instead of unstyled browser defaults.
+    FILE_CLASSES = 'file:mr-3 file:h-full file:cursor-pointer file:border-0 file:border-r file:border-solid file:border-[hsl(var(--senren-border))] file:bg-[hsl(var(--senren-muted))] file:px-3 file:text-sm file:font-medium file:text-[hsl(var(--senren-foreground))]'
+
+    # Non-file inputs get standard font styling for the placeholder.
+    TEXT_FILE_CLASSES = 'file:border-0 file:bg-transparent file:text-sm file:font-medium'
+
     def initialize(name:, type: 'text', value: nil, placeholder: nil, id: nil, variant: :default, size: :md,
                    class_name: nil, **html)
       super(variant: variant, size: size, class_name: class_name, **html)
@@ -24,5 +35,13 @@ module Senren
     end
 
     attr_reader :name, :type, :value, :placeholder, :id
+
+    def input_classes
+      file_type? ? "#{BASE_CLASSES} #{FILE_CLASSES}" : "#{BASE_CLASSES} #{TEXT_FILE_CLASSES}"
+    end
+
+    def file_type?
+      type.to_s == 'file'
+    end
   end
 end

data/templates/components/label/label_component.html.erb

@@ -1,4 +1,3 @@
 <%= tag.label(**root_attrs("text-sm font-medium leading-none text-[hsl(var(--senren-foreground))] peer-disabled:cursor-not-allowed peer-disabled:opacity-70", for: for_field)) do %>
-  <%= content %>
-  <% if variant == :required %><span class="text-[hsl(var(--senren-destructive))]" aria-hidden="true"> *</span><% end %>
+  <%= label_text %><% if variant == :required %><span class="text-[hsl(var(--senren-destructive))]" aria-hidden="true"> *</span><% end %>
 <% end %>

data/templates/components/label/label_component.rb

@@ -9,11 +9,21 @@ module Senren
 
     SIZES = { md: '' }.freeze
 
-    def initialize(for_field:, variant: :default, class_name: nil, **html)
+    # Accept optional text: param as fallback for inline block syntax.
+    # This ensures both patterns work reliably:
+    #   render(Senren::LabelComponent.new(for_field: "name", text: "Name"))
+    #   render(Senren::LabelComponent.new(for_field: "name")) { "Name" }
+    def initialize(for_field:, text: nil, variant: :default, class_name: nil, **html)
       super(variant: variant, size: :md, class_name: class_name, **html)
       @for_field = for_field
+      @text = text
     end
 
-    attr_reader :for_field
+    attr_reader :for_field, :text
+
+    # Resolve label text: content block > text param > empty string
+    def label_text
+      content.presence || @text || ''
+    end
   end
 end

data/templates/components/link/link_component.html.erb

@@ -1 +1 @@
-<%= tag.a content, **root_attrs("inline-flex items-center gap-1 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-[hsl(var(--senren-ring))] rounded-sm", href: href, **external_attrs) %>
+<%= tag.a content, **root_attrs("inline-flex items-center gap-1 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-[hsl(var(--senren-ring))] rounded-sm", href: safe_url(href), **external_attrs) %>

data/templates/components/native_select/native_select_component.html.erb

@@ -1,5 +1,6 @@
-<%= tag.div(**wrapper_attrs) do %>
-  <%= tag.select(**select_attrs) do %>
+<% if native_arrow? %>
+  <%# Native mode: no wrapper div, no SVG arrow — let the browser render its own. %>
+  <%= tag.select(**root_select_attrs) do %>
     <% if prompt %>
       <option value=""><%= prompt %></option>
     <% end %>
@@ -8,7 +9,20 @@
       <option value="<%= value %>" <%= "selected" if selected.to_s == value.to_s %>><%= label %></option>
     <% end %>
   <% end %>
-  <svg aria-hidden="true" viewBox="0 0 20 20" fill="none" stroke="currentColor" stroke-width="1.8" stroke-linecap="round" stroke-linejoin="round" class="pointer-events-none absolute right-3 top-1/2 h-4 w-4 -translate-y-1/2 text-[hsl(var(--senren-muted-foreground))] transition-transform duration-150 group-focus-within:rotate-180">
-    <path d="m5 8 5 5 5-5"></path>
-  </svg>
+<% else %>
+  <%# Custom arrow mode: wrapper div + positioned SVG. %>
+  <%= tag.div(class: "group relative w-full", data: { senren_component: senren_component_name }) do %>
+    <%= tag.select(**select_attrs) do %>
+      <% if prompt %>
+        <option value=""><%= prompt %></option>
+      <% end %>
+      <% options.each do |opt| %>
+        <% value, label = Array === opt ? opt : [opt, opt] %>
+        <option value="<%= value %>" <%= "selected" if selected.to_s == value.to_s %>><%= label %></option>
+      <% end %>
+    <% end %>
+    <svg aria-hidden="true" viewBox="0 0 20 20" fill="none" stroke="currentColor" stroke-width="1.8" stroke-linecap="round" stroke-linejoin="round" class="pointer-events-none absolute right-3 top-1/2 h-4 w-4 -translate-y-1/2 text-[hsl(var(--senren-muted-foreground))] transition-transform duration-150 group-focus-within:rotate-180">
+      <path d="m5 8 5 5 5-5"></path>
+    </svg>
+  <% end %>
 <% end %>