Simplify Radio to single-component API, remove RadioCollection

Radio now works like Checkbox: call radio(value) to get a single
component with automatic name/value/checked binding. The user
controls iteration and HTML structure. Removes RadioCollection
and RadioOption which added dispatch complexity without value.

Author: bradgessler

README.md

@@ -417,15 +417,15 @@ class SignupForm < Components::Form
       end
     end
 
-    # Radio groups: pass [value, label] options and iterate. Superform
-    # handles the name, value, and checked state automatically.
+    # Radio groups: iterate your options and call radio(value) on the field.
+    # Superform handles the name, value, and checked state automatically.
     fieldset do
       legend { "Plan" }
-      Field(:plan_id).radio(*Plan.pluck(:id, :name)).each do |plan|
+      Plan.all.each do |plan|
         label do
-          plan.radio
+          Field(:plan_id).radio(plan.id)
           whitespace
-          plain plan.label
+          plain plan.name
         end
       end
     end

lib/superform/rails/components/radio.rb

@@ -2,19 +2,17 @@ module Superform
   module Rails
     module Components
       class Radio < Field
-        def initialize(field, value:, index: nil, attributes: {})
+        def initialize(field, value:, attributes: {})
           super(field, attributes: attributes)
           @value = value
-          @index = index
         end
 
         def view_template(&)
           input(type: :radio, **attributes)
         end
 
         def field_attributes
-          id = @index ? "#{dom.id}_#{@index}" : dom.id
-          { id: id, name: dom.name, value: @value, checked: field.value == @value }
+          { id: dom.id, name: dom.name, value: @value, checked: field.value == @value }
         end
       end
     end

lib/superform/rails/field.rb

@@ -143,15 +143,8 @@ def file(*, **)
         input(*, **, type: :file)
       end
 
-      def radio(*args, **attributes)
-        if args.length == 1 && !args.first.is_a?(Array)
-          # Single scalar value: Field(:gender).radio("male")
-          Components::Radio.new(field, value: args.first, attributes: attributes)
-        else
-          # Collection of options: Field(:status).radio("active", "inactive", "pending")
-          # or Field(:status).radio(["active", "Active"], ["inactive", "Inactive"])
-          RadioCollection.new(field: field, options: args)
-        end
+      def radio(value, **attributes)
+        Components::Radio.new(field, value: value, attributes: attributes)
       end
 
       # Rails compatibility aliases

lib/superform/rails/radio_collection.rb

@@ -1,89 +1,37 @@
 RSpec.describe Superform::Rails::Components::Radio, type: :view do
-  describe 'single radio' do
-    let(:object) { double('object', gender: "male") }
-    let(:field) do
-      Superform::Rails::Field.new(:gender, parent: nil, object: object)
-    end
-
-    it 'renders a checked radio when value matches' do
-      html = render(field.radio("male"))
-      expect(html).to include('type="radio"')
-      expect(html).to include('name="gender"')
-      expect(html).to include('value="male"')
-      expect(html).to include('checked')
-    end
-
-    it 'renders an unchecked radio when value does not match' do
-      html = render(field.radio("female"))
-      expect(html).to include('value="female"')
-      expect(html).not_to include('checked')
-    end
-
-    it 'accepts custom attributes' do
-      html = render(field.radio("male", class: "radio-input"))
-      expect(html).to include('class="radio-input"')
-    end
+  let(:object) { double('object', gender: "male") }
+  let(:field) do
+    Superform::Rails::Field.new(:gender, parent: nil, object: object)
   end
 
-  describe 'radio collection' do
-    let(:object) { double('object', status: "active") }
-    let(:field) do
-      Superform::Rails::Field.new(:status, parent: nil, object: object)
-    end
-
-    it 'returns an enumerable' do
-      collection = field.radio("active", "inactive", "pending")
-      expect(collection).to respond_to(:each)
-    end
-
-    it 'yields options with value, label, and radio' do
-      options = field.radio(["active", "Active"], ["inactive", "Inactive"]).to_a
-      expect(options.length).to eq(2)
-      expect(options.first.value).to eq("active")
-      expect(options.first.label).to eq("Active")
-      expect(options.first).to respond_to(:radio)
-    end
-
-    it 'renders radios with correct checked state' do
-      html = ""
-      field.radio(["active", "Active"], ["inactive", "Inactive"], ["pending", "Pending"]).each do |status|
-        html += render(status.radio)
-      end
-
-      expect(html.scan(/type="radio"/).count).to eq(3)
-      expect(html.scan(/name="status"/).count).to eq(3)
-      expect(html.scan(/checked/).count).to eq(1)
-      expect(html).to match(/<input[^>]*value="active"[^>]*checked/)
-      expect(html).not_to match(/<input[^>]*value="inactive"[^>]*checked/)
-      expect(html).not_to match(/<input[^>]*value="pending"[^>]*checked/)
-    end
-
-    it 'generates unique IDs for each radio' do
-      html = ""
-      field.radio("active", "inactive").each do |status|
-        html += render(status.radio)
-      end
+  it 'renders a checked radio when value matches' do
+    html = render(field.radio("male"))
+    expect(html).to include('type="radio"')
+    expect(html).to include('name="gender"')
+    expect(html).to include('value="male"')
+    expect(html).to include('checked')
+  end
 
-      expect(html).to include('id="status_1"')
-      expect(html).to include('id="status_2"')
-    end
+  it 'renders an unchecked radio when value does not match' do
+    html = render(field.radio("female"))
+    expect(html).to include('value="female"')
+    expect(html).not_to include('checked')
+  end
 
-    it 'works with the label pattern from the docs' do
-      html = ""
-      field.radio(["active", "Active"], ["inactive", "Inactive"]).each do |status|
-        # Simulating: label { status.radio; whitespace; plain status.value.humanize }
-        html += render(status.radio)
-      end
+  it 'accepts custom attributes' do
+    html = render(field.radio("male", class: "radio-input"))
+    expect(html).to include('class="radio-input"')
+  end
 
-      expect(html).to include('value="active"')
-      expect(html).to include('value="inactive"')
-      expect(html.scan(/checked/).count).to eq(1)
+  it 'renders a group with correct checked state' do
+    html = ""
+    ["male", "female", "other"].each do |value|
+      html += render(field.radio(value))
     end
 
-    it 'works with single-value options (value used as label)' do
-      options = field.radio("active", "inactive").to_a
-      expect(options.first.value).to eq("active")
-      expect(options.first.label).to eq("active")
-    end
+    expect(html.scan(/type="radio"/).count).to eq(3)
+    expect(html.scan(/name="gender"/).count).to eq(3)
+    expect(html.scan(/checked/).count).to eq(1)
+    expect(html).to match(/<input[^>]*value="male"[^>]*checked/)
   end
 end