source_monitor 0.1.3 → 0.2.0

data/app/views/source_monitor/import_sessions/steps/_confirm.html.erb

@@ -0,0 +1,121 @@
+<div id="import_session_confirm" class="space-y-6">
+  <div class="rounded-lg border border-slate-200 bg-white p-4 shadow-sm">
+    <div class="flex flex-col gap-3 sm:flex-row sm:items-start sm:justify-between">
+      <div>
+        <h2 class="text-lg font-semibold text-slate-900">Review &amp; confirm</h2>
+        <p class="mt-1 text-sm text-slate-600">
+          Apply these settings to the <%= @selected_entries.size %> feeds you selected. Duplicates will be skipped automatically.
+        </p>
+      </div>
+      <div class="flex items-center gap-3 text-sm">
+        <span class="inline-flex items-center rounded-full bg-green-50 px-3 py-1 font-semibold text-green-700">
+          Selected: <%= @selected_entries.size %>
+        </span>
+        <span class="inline-flex items-center rounded-full bg-amber-50 px-3 py-1 font-semibold text-amber-800">
+          Skipped duplicates: <%= @selected_entries.count { |entry| entry[:duplicate] } %>
+        </span>
+      </div>
+    </div>
+    <% if @selection_error.present? %>
+      <div class="mt-3 rounded-md bg-red-50 px-3 py-2 text-sm text-red-800">
+        <%= @selection_error %>
+      </div>
+    <% end %>
+  </div>
+
+  <div class="rounded-lg border border-slate-200 bg-white shadow-sm">
+    <div class="flex items-center justify-between border-b border-slate-100 px-4 py-3">
+      <div>
+        <h3 class="text-base font-semibold text-slate-900">Sources to import</h3>
+        <p class="text-sm text-slate-600">Each feed will be created individually; duplicates are skipped.</p>
+      </div>
+      <span class="rounded-full bg-slate-100 px-3 py-1 text-xs font-semibold uppercase tracking-wide text-slate-600">
+        <%= @selected_entries.size %> total
+      </span>
+    </div>
+    <div class="overflow-x-auto">
+      <table class="min-w-full divide-y divide-slate-200 text-left text-sm">
+        <thead class="bg-slate-50 text-xs font-semibold uppercase tracking-wide text-slate-500">
+          <tr>
+            <th scope="col" class="px-4 py-3">Title</th>
+            <th scope="col" class="px-4 py-3">Feed URL</th>
+            <th scope="col" class="px-4 py-3">Website</th>
+            <th scope="col" class="px-4 py-3">Status</th>
+          </tr>
+        </thead>
+        <tbody class="divide-y divide-slate-100 text-slate-700">
+          <% @selected_entries.each do |entry| %>
+            <tr>
+              <td class="px-4 py-3 align-top font-semibold text-slate-900">
+                <%= entry[:title].presence || entry[:feed_url] %>
+              </td>
+              <td class="px-4 py-3 align-top text-slate-700">
+                <div class="break-all font-mono text-xs"><%= entry[:feed_url] %></div>
+              </td>
+              <td class="px-4 py-3 align-top text-slate-700">
+                <% if entry[:website_url].present? %>
+                  <div class="break-all text-xs text-blue-700"><%= entry[:website_url] %></div>
+                <% else %>
+                  <span class="text-xs text-slate-400">—</span>
+                <% end %>
+              </td>
+              <td class="px-4 py-3 align-top">
+                <% if entry[:duplicate] %>
+                  <span class="inline-flex items-center rounded-full bg-amber-50 px-2 py-1 text-xs font-semibold uppercase tracking-wide text-amber-800">Duplicate</span>
+                <% else %>
+                  <span class="inline-flex items-center rounded-full bg-green-50 px-2 py-1 text-xs font-semibold uppercase tracking-wide text-green-700">Ready</span>
+                <% end %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+  </div>
+
+  <div class="rounded-lg border border-slate-200 bg-white shadow-sm">
+    <div class="flex items-center justify-between border-b border-slate-100 px-4 py-3">
+      <div>
+        <h3 class="text-base font-semibold text-slate-900">Bulk settings</h3>
+        <p class="text-sm text-slate-600">These options apply to every new source created by this import.</p>
+      </div>
+    </div>
+    <div class="px-4 py-4">
+      <% if @bulk_settings.present? %>
+        <dl class="grid gap-3 md:grid-cols-2">
+          <% @bulk_settings.each do |key, value| %>
+            <div class="rounded-md border border-slate-100 bg-slate-50 px-3 py-2">
+              <dt class="text-xs font-semibold uppercase tracking-wide text-slate-500"><%= key.to_s.humanize %></dt>
+              <dd class="mt-1 text-sm text-slate-800"><%= formatted_setting_value(value) %></dd>
+            </div>
+          <% end %>
+        </dl>
+      <% else %>
+        <p class="text-sm text-slate-600">No additional settings selected. Defaults will be used.</p>
+      <% end %>
+    </div>
+  </div>
+
+  <div class="flex items-center justify-between gap-3">
+    <%= form_with model: import_session,
+          url: source_monitor.step_import_session_path(import_session, step: "configure"),
+          method: :patch,
+          data: { turbo: false } do |form| %>
+      <%= hidden_field_tag :next_step, "configure", name: "import_session[next_step]" %>
+      <%= form.submit "Back", class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+    <% end %>
+
+    <div class="flex items-center gap-3">
+      <%= link_to "Cancel", source_monitor.import_session_path(import_session),
+            data: { turbo_method: :delete, action: "confirm-navigation#disable" },
+            class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+
+      <%= form_with model: import_session,
+            url: source_monitor.step_import_session_path(import_session, step: "confirm"),
+            method: :patch do |form| %>
+        <%= hidden_field_tag :next_step, "confirm", name: "import_session[next_step]" %>
+        <%= form.submit "Start import", class: "inline-flex items-center rounded-md bg-blue-600 px-4 py-2 text-sm font-semibold text-white shadow hover:bg-blue-500" %>
+      <% end %>
+    </div>
+  </div>
+</div>

data/app/views/source_monitor/import_sessions/steps/_health_check.html.erb

@@ -0,0 +1,82 @@
+<%= turbo_stream_from import_session.health_stream_name %>
+
+<div class="space-y-6">
+  <div class="rounded-lg border border-slate-200 bg-white shadow-sm">
+    <div class="flex flex-wrap items-center justify-between gap-3 border-b border-slate-100 px-4 py-3">
+      <div>
+        <h2 class="text-lg font-semibold text-slate-900">Run health checks</h2>
+        <p class="mt-1 text-sm text-slate-600">We check each selected feed and update results live. Unhealthy feeds are unselected by default.</p>
+      </div>
+      <%= render "source_monitor/import_sessions/health_check/progress", import_session: import_session, progress: @health_progress %>
+    </div>
+
+    <% if @selection_error.present? %>
+      <div class="border-b border-amber-200 bg-amber-50 px-4 py-3 text-sm text-amber-900">
+        <%= @selection_error %>
+      </div>
+    <% end %>
+
+    <% if @health_check_entries.blank? %>
+      <div class="px-4 py-6 text-sm text-slate-600">
+        No sources are selected for health checks. Go back to Preview to pick at least one feed.
+      </div>
+    <% else %>
+      <%= form_with model: import_session,
+            url: source_monitor.step_import_session_path(import_session, step: "health_check"),
+            method: :patch,
+            data: { turbo: false, action: "confirm-navigation#disable" },
+            html: { class: "block" } do |form| %>
+        <div class="overflow-x-auto">
+          <table class="min-w-full divide-y divide-slate-200 text-left text-sm" data-controller="select-all">
+            <thead class="bg-slate-50 text-xs font-semibold uppercase tracking-wide text-slate-500">
+              <tr>
+                <th scope="col" class="px-4 py-3 w-12 text-center">
+                  <div class="flex items-center justify-center gap-2">
+                    <span class="sr-only">Select all</span>
+                    <%= check_box_tag "select_all_master", "1", @health_check_entries.all? { |entry| entry[:selected] },
+                          data: { select_all_target: "master", action: "select-all#toggleAll" },
+                          class: "h-4 w-4 rounded border-slate-300 text-blue-600 focus:ring-blue-500" %>
+                  </div>
+                </th>
+                <th scope="col" class="px-4 py-3">Feed URL</th>
+                <th scope="col" class="px-4 py-3">Title</th>
+                <th scope="col" class="px-4 py-3">Health</th>
+              </tr>
+            </thead>
+            <tbody class="divide-y divide-slate-100 text-slate-800">
+              <%= render partial: "source_monitor/import_sessions/health_check/row",
+                    collection: @health_check_entries,
+                    as: :entry,
+                    locals: { import_session: import_session } %>
+            </tbody>
+          </table>
+        </div>
+
+        <div class="flex items-center justify-between gap-3 border-t border-slate-200 px-4 py-4">
+          <div>
+            <%= button_tag "Back",
+                  type: :submit,
+                  name: "import_session[next_step]",
+                  value: "preview",
+                  class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+          </div>
+          <div class="flex items-center gap-3">
+            <%= link_to "Cancel", source_monitor.import_session_path(import_session),
+                  data: { turbo_method: :delete, action: "confirm-navigation#disable" },
+                  class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+            <% continue_disabled = @selected_source_ids.blank? %>
+            <%= button_tag "Continue",
+                  type: :submit,
+                  name: "import_session[next_step]",
+                  value: "configure",
+                  disabled: continue_disabled,
+                  class: [
+                    "inline-flex items-center rounded-md px-4 py-2 text-sm font-semibold text-white shadow",
+                    continue_disabled ? "bg-slate-300 cursor-not-allowed" : "bg-blue-600 hover:bg-blue-500"
+                  ].join(" ") %>
+          </div>
+        </div>
+      <% end %>
+    <% end %>
+  </div>
+</div>

data/app/views/source_monitor/import_sessions/steps/_navigation.html.erb

@@ -0,0 +1,29 @@
+<% previous_step ||= nil %>
+<% next_step ||= nil %>
+<div class="flex items-center justify-between gap-3">
+  <div>
+    <% if previous_step.present? %>
+      <%= form_with model: import_session,
+            url: source_monitor.step_import_session_path(import_session, step: previous_step),
+            method: :patch,
+            data: { turbo: false } do |form| %>
+        <%= hidden_field_tag :next_step, previous_step, name: "import_session[next_step]" %>
+        <%= form.submit "Back", class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+      <% end %>
+    <% end %>
+  </div>
+  <div class="flex items-center gap-3">
+    <%= link_to "Cancel", source_monitor.import_session_path(import_session),
+          data: { turbo_method: :delete, action: "confirm-navigation#disable" },
+          class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+    <% if next_step.present? %>
+      <%= form_with model: import_session,
+            url: source_monitor.step_import_session_path(import_session, step: next_step),
+            method: :patch,
+            data: { turbo: false } do |form| %>
+        <%= hidden_field_tag :next_step, next_step, name: "import_session[next_step]" %>
+        <%= form.submit "Continue", class: "inline-flex items-center rounded-md bg-blue-600 px-4 py-2 text-sm font-semibold text-white shadow hover:bg-blue-500" %>
+      <% end %>
+    <% end %>
+  </div>
+</div>

data/app/views/source_monitor/import_sessions/steps/_preview.html.erb

@@ -0,0 +1,172 @@
+<div class="space-y-6">
+  <div class="rounded-lg border border-slate-200 bg-white shadow-sm">
+    <div class="flex flex-wrap items-center justify-between gap-3 border-b border-slate-100 px-4 py-3">
+      <div>
+        <h2 class="text-lg font-semibold text-slate-900">Preview sources</h2>
+        <p class="mt-1 text-sm text-slate-600">Select the feeds to import. Duplicates and malformed entries are disabled.</p>
+      </div>
+      <div class="flex items-center gap-2 text-xs font-semibold uppercase tracking-wide text-slate-500">
+        <span class="rounded-full bg-slate-100 px-2 py-1">Total: <%= @preview_entries.size %></span>
+        <span class="rounded-full bg-green-50 px-2 py-1 text-green-700">New: <%= @preview_entries.count { |e| e[:selectable] } %></span>
+        <span class="rounded-full bg-amber-50 px-2 py-1 text-amber-800">Existing: <%= @preview_entries.count { |e| e[:duplicate] } %></span>
+      </div>
+    </div>
+
+    <% if @preview_entries.blank? %>
+      <div class="px-4 py-6 text-sm text-slate-600">
+        No parsed entries found. Return to the Upload step and add a different OPML file.
+      </div>
+      <div class="flex items-center justify-between gap-3 border-t border-slate-200 px-4 py-4">
+      <%= form_with model: import_session,
+            url: source_monitor.step_import_session_path(import_session, step: "preview"),
+            method: :patch,
+            data: { turbo: false } do |form| %>
+          <%= button_tag "Back", type: :submit, name: "import_session[next_step]", value: "upload",
+                class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+        <% end %>
+        <div class="flex items-center gap-3">
+          <%= link_to "Cancel", source_monitor.import_session_path(import_session),
+                data: { turbo_method: :delete, action: "confirm-navigation#disable" },
+                class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+          <span class="inline-flex items-center rounded-md bg-slate-200 px-4 py-2 text-sm font-semibold text-white">Continue</span>
+        </div>
+      </div>
+    <% else %>
+      <div class="border-b border-slate-100 px-4 py-3">
+        <div class="flex items-center gap-2 text-sm font-medium text-slate-700">
+          <% filters = { "all" => "All", "new" => "New Sources", "existing" => "Existing Sources" } %>
+          <% filters.each do |key, label| %>
+            <% active = @filter == key %>
+            <%= link_to label,
+                  source_monitor.step_import_session_path(import_session, step: "preview", filter: key),
+                  data: { turbo_frame: "import_session_step" },
+                  class: [
+                    "rounded-md px-3 py-2 text-xs font-semibold",
+                    active ? "bg-blue-50 text-blue-700 border border-blue-200" : "border border-transparent text-slate-600 hover:text-slate-900 hover:border-slate-200"
+                  ].join(" ") %>
+          <% end %>
+        </div>
+      </div>
+
+      <% if defined?(@selection_error) && @selection_error.present? %>
+        <div class="border-b border-amber-200 bg-amber-50 px-4 py-3 text-sm text-amber-900">
+          <%= @selection_error %>
+        </div>
+      <% end %>
+
+      <%= form_with model: import_session,
+            url: source_monitor.step_import_session_path(import_session, step: "preview"),
+            method: :patch,
+            data: { turbo: false, action: "confirm-navigation#disable" },
+            html: { class: "block" } do |form| %>
+        <%= hidden_field_tag :filter, @filter %>
+        <div class="overflow-x-auto">
+          <table class="min-w-full divide-y divide-slate-200 text-left text-sm" data-controller="select-all">
+            <thead class="bg-slate-50 text-xs font-semibold uppercase tracking-wide text-slate-500">
+              <tr>
+                <th scope="col" class="px-4 py-3 w-12 text-center">
+                  <div class="flex items-center justify-center gap-2">
+                    <span class="sr-only">Select all</span>
+                    <%= check_box_tag "select_all_master", "1", @filtered_entries.all? { |e| e[:selected] || !e[:selectable] },
+                          data: { select_all_target: "master", action: "select-all#toggleAll" },
+                          class: "h-4 w-4 rounded border-slate-300 text-blue-600 focus:ring-blue-500" %>
+                  </div>
+                </th>
+                <th scope="col" class="px-4 py-3">Feed URL</th>
+                <th scope="col" class="px-4 py-3">Title</th>
+                <th scope="col" class="px-4 py-3">Status</th>
+              </tr>
+            </thead>
+            <tbody class="divide-y divide-slate-100 text-slate-800">
+              <% @paginated_entries.each do |entry| %>
+                <% disabled = !entry[:selectable] %>
+                <tr class="<%= disabled ? 'bg-slate-50' : '' %>">
+                  <td class="px-4 py-3 text-center align-middle">
+                    <%= check_box_tag "import_session[selected_source_ids][]",
+                          entry[:id],
+                          entry[:selected],
+                          disabled: disabled,
+                          data: { select_all_target: "item", action: "select-all#toggleItem" },
+                          class: "h-4 w-4 rounded border-slate-300 text-blue-600 focus:ring-blue-500 disabled:cursor-not-allowed disabled:border-slate-200 disabled:bg-slate-100" %>
+                  </td>
+                  <td class="px-4 py-3 align-top">
+                    <div class="max-w-xl break-words font-medium text-slate-900"><%= entry[:feed_url] %></div>
+                    <% if entry[:website_url].present? %>
+                      <div class="mt-1 text-xs text-blue-700 truncate"><%= entry[:website_url] %></div>
+                    <% end %>
+                  </td>
+                  <td class="px-4 py-3 align-top">
+                    <div class="text-sm font-medium text-slate-900"><%= entry[:title].presence || "(No title)" %></div>
+                    <% if entry[:status] == "malformed" && entry[:error].present? %>
+                      <div class="mt-1 text-xs text-amber-800">Parse error: <%= entry[:error] %></div>
+                    <% end %>
+                  </td>
+                  <td class="px-4 py-3 align-top">
+                    <% if entry[:duplicate] %>
+                      <span class="inline-flex items-center rounded-full bg-amber-100 px-2 py-1 text-xs font-semibold text-amber-800">Already Imported</span>
+                    <% elsif entry[:status] == "malformed" %>
+                      <span class="inline-flex items-center rounded-full bg-red-100 px-2 py-1 text-xs font-semibold text-red-800">Malformed</span>
+                    <% else %>
+                      <span class="inline-flex items-center rounded-full bg-green-100 px-2 py-1 text-xs font-semibold text-green-800">New</span>
+                    <% end %>
+                  </td>
+                </tr>
+              <% end %>
+              <% if @paginated_entries.blank? %>
+                <tr>
+                  <td colspan="4" class="px-4 py-6 text-sm text-slate-600">No entries match this filter.</td>
+                </tr>
+              <% end %>
+            </tbody>
+          </table>
+        </div>
+
+        <div class="flex flex-col items-center gap-3 border-t border-slate-200 px-4 py-4 sm:flex-row sm:justify-between">
+          <div class="text-xs text-slate-500">Page <%= @page %></div>
+          <div class="flex gap-2">
+            <% prev_params = { page: @page - 1, filter: @filter } %>
+            <% next_params = { page: @page + 1, filter: @filter } %>
+
+            <% if @has_previous_page %>
+              <%= link_to "Previous",
+                    source_monitor.step_import_session_path(import_session, step: "preview", **prev_params),
+                    class: "inline-flex items-center rounded-md border border-slate-300 px-3 py-2 text-sm font-medium text-slate-700 hover:bg-slate-50",
+                    data: { turbo_frame: "import_session_step" } %>
+            <% else %>
+              <span class="inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-medium text-slate-300">Previous</span>
+            <% end %>
+
+            <% if @has_next_page %>
+              <%= link_to "Next",
+                    source_monitor.step_import_session_path(import_session, step: "preview", **next_params),
+                    class: "inline-flex items-center rounded-md border border-slate-300 px-3 py-2 text-sm font-medium text-slate-700 hover:bg-slate-50",
+                    data: { turbo_frame: "import_session_step" } %>
+            <% else %>
+              <span class="inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-medium text-slate-300">Next</span>
+            <% end %>
+          </div>
+        </div>
+
+        <div class="flex items-center justify-between gap-3 border-t border-slate-200 px-4 py-4">
+          <div>
+            <%= button_tag "Back",
+                  type: :submit,
+                  name: "import_session[next_step]",
+                  value: "upload",
+                  class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+          </div>
+          <div class="flex items-center gap-3">
+            <%= link_to "Cancel", source_monitor.import_session_path(import_session),
+                  data: { turbo_method: :delete, action: "confirm-navigation#disable" },
+                  class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+            <%= button_tag "Continue",
+                  type: :submit,
+                  name: "import_session[next_step]",
+                  value: "health_check",
+                  class: "inline-flex items-center rounded-md bg-blue-600 px-4 py-2 text-sm font-semibold text-white shadow hover:bg-blue-500" %>
+          </div>
+        </div>
+      <% end %>
+    <% end %>
+  </div>
+</div>

data/app/views/source_monitor/import_sessions/steps/_upload.html.erb

@@ -0,0 +1,42 @@
+<div class="space-y-6">
+  <div class="rounded-lg border border-slate-200 bg-white shadow-sm">
+    <div class="border-b border-slate-100 px-4 py-3">
+      <h2 class="text-lg font-semibold text-slate-900">Upload OPML file</h2>
+      <p class="mt-1 text-sm text-slate-600">Select an OPML file to begin. Parsing runs immediately so you can review results in Preview.</p>
+    </div>
+    <div class="px-4 py-5">
+      <%= form_with model: import_session,
+            url: source_monitor.step_import_session_path(import_session, step: "upload"),
+            method: :patch,
+            data: { turbo: false, action: "confirm-navigation#disable" },
+            html: { enctype: "multipart/form-data", class: "space-y-4" } do |form| %>
+        <% if defined?(@upload_errors) && @upload_errors.present? %>
+          <div class="rounded-md border border-red-200 bg-red-50 px-4 py-3 text-sm text-red-800">
+            <ul class="list-disc space-y-1 pl-5">
+              <% @upload_errors.each do |message| %>
+                <li><%= message %></li>
+              <% end %>
+            </ul>
+          </div>
+        <% end %>
+        <div>
+          <%= form.label :opml_file, "OPML file", class: "block text-sm font-semibold text-slate-800" %>
+          <p class="text-xs text-slate-500">Max size and validation are handled later in the flow.</p>
+          <div class="mt-2 flex items-center gap-3">
+            <%= form.file_field :opml_file, name: :opml_file, accept: ".opml, text/xml, application/xml", class: "block w-full text-sm text-slate-700" %>
+          </div>
+          <% if import_session.opml_file_metadata.present? %>
+            <p class="mt-2 text-xs text-slate-500">Current upload: <%= import_session.opml_file_metadata["filename"] %></p>
+          <% end %>
+        </div>
+        <div class="flex items-center justify-end gap-3">
+          <%= link_to "Cancel", source_monitor.import_session_path(import_session),
+                data: { turbo_method: :delete, action: "confirm-navigation#disable" },
+                class: "inline-flex items-center rounded-md border border-slate-200 px-3 py-2 text-sm font-semibold text-slate-700 hover:bg-slate-50" %>
+          <%= hidden_field_tag :next_step, "preview", name: "import_session[next_step]" %>
+          <%= form.submit "Continue", class: "inline-flex items-center rounded-md bg-blue-600 px-4 py-2 text-sm font-semibold text-white shadow hover:bg-blue-500" %>
+        </div>
+      <% end %>
+    </div>
+  </div>
+</div>

data/app/views/source_monitor/sources/_form.html.erb

@@ -1,143 +1,13 @@
-<%= form_with model: source, class: "space-y-6" do |form| %>
-  <div class="space-y-4 rounded-lg border border-slate-200 bg-white px-5 py-5 shadow-sm">
-    <div>
-      <%= form.label :name, class: "block text-sm font-medium text-slate-800" %>
-      <%= form.text_field :name, class: "mt-1 block w-full rounded border border-slate-300 px-3 py-2" %>
-    </div>
-
-    <div>
-      <%= form.label :feed_url, class: "block text-sm font-medium text-slate-800" %>
-      <%= form.url_field :feed_url, class: "mt-1 block w-full rounded border border-slate-300 px-3 py-2" %>
-    </div>
-
-    <div>
-      <%= form.label :website_url, class: "block text-sm font-medium text-slate-800" %>
-      <%= form.url_field :website_url, class: "mt-1 block w-full rounded border border-slate-300 px-3 py-2" %>
-    </div>
+<% submit_label = local_assigns.fetch(:submit_label, source.persisted? ? "Update Source" : "Create Source") %>
+<% show_submit = local_assigns.fetch(:show_submit, true) %>
+<% form_options = { class: "space-y-6" }.merge(local_assigns.fetch(:form_options, {})) %>
 
-    <label class="inline-flex items-center space-x-2">
-      <%= form.check_box :active %>
-      <span>Active</span>
-    </label>
-  </div>
+<%= form_with model: source, url: local_assigns[:form_url], **form_options do |form| %>
+  <%= render "source_monitor/sources/form_fields", form: form %>
 
-  <div class="space-y-4 rounded-lg border border-slate-200 bg-white px-5 py-5 shadow-sm">
+  <% if show_submit %>
     <div>
-      <%= form.label :fetch_interval_minutes, "Fetch interval (minutes)", class: "block text-sm font-medium text-slate-800" %>
-      <%= form.number_field :fetch_interval_minutes, min: 1, step: 1, class: "mt-1 block w-full rounded border border-slate-300 px-3 py-2" %>
-      <p class="mt-1 text-xs text-slate-500">Minimum 5 minutes recommended.</p>
+      <%= form.submit submit_label, class: "inline-flex items-center rounded bg-blue-600 px-4 py-2 text-white shadow hover:bg-blue-500" %>
     </div>
-
-    <label class="flex items-start space-x-3 rounded-lg border border-slate-200 bg-slate-50 px-4 py-3">
-      <%= form.check_box :adaptive_fetching_enabled, class: "mt-1" %>
-      <span>
-        <span class="block text-sm font-medium text-slate-800">Automatically adjust fetch interval</span>
-        <span class="mt-1 block text-xs text-slate-500">
-          When enabled, SourceMonitor lengthens or shortens the interval based on feed activity and failures. Disable to keep the interval fixed at the value above.
-        </span>
-      </span>
-    </label>
-
-    <div>
-      <%= form.label :health_auto_pause_threshold, "Auto-pause threshold", class: "block text-sm font-medium text-slate-800" %>
-      <%= form.number_field :health_auto_pause_threshold, min: 0, max: 1, step: 0.05, class: "mt-1 block w-full rounded border border-slate-300 px-3 py-2" %>
-      <p class="mt-1 text-xs text-slate-500">
-        Optional override for this source. When the rolling success rate falls below this fraction (defaults to <%= SourceMonitor.config.health.auto_pause_threshold %>), the source is temporarily auto-paused.
-      </p>
-    </div>
-  </div>
-
-  <div class="rounded-lg border border-slate-200 bg-white shadow-sm">
-    <div class="border-b border-slate-200 px-5 py-4">
-      <h2 class="text-lg font-medium">Retention</h2>
-      <p class="mt-1 text-xs text-slate-500">
-        Choose how long SourceMonitor keeps items for this source. Pruning runs after every fetch so the UI always reflects active retention rules.
-      </p>
-    </div>
-    <div class="grid gap-4 px-5 py-5 sm:grid-cols-2">
-      <div>
-        <%= form.label :items_retention_days, "Retention window (days)", class: "block text-sm font-medium text-slate-800" %>
-        <%= form.number_field :items_retention_days, min: 0, class: "mt-1 block w-full rounded border border-slate-300 px-3 py-2" %>
-        <p class="mt-1 text-xs text-slate-500">
-          Leave blank to keep historical items indefinitely. Items older than the configured window are removed automatically.
-        </p>
-      </div>
-
-      <div>
-        <%= form.label :max_items, "Maximum stored items", class: "block text-sm font-medium text-slate-800" %>
-        <%= form.number_field :max_items, min: 0, class: "mt-1 block w-full rounded border border-slate-300 px-3 py-2" %>
-        <p class="mt-1 text-xs text-slate-500">
-          Optional hard cap. SourceMonitor keeps the newest items and prunes the rest after each fetch.
-        </p>
-      </div>
-    </div>
-  </div>
-
-  <div class="rounded-lg border border-slate-200 bg-white shadow-sm">
-    <div class="border-b border-slate-200 px-5 py-4">
-      <h2 class="text-lg font-medium">Feed Content Processing</h2>
-      <p class="mt-1 text-xs text-slate-500">Control whether feed-supplied content is cleaned with Readability before storing items.</p>
-    </div>
-    <div class="space-y-4 px-5 py-5">
-      <label class="inline-flex items-center space-x-2">
-        <%= form.check_box :feed_content_readability_enabled %>
-        <span>Process feed content with Readability</span>
-      </label>
-      <p class="text-xs text-slate-500">
-        When enabled, the raw feed HTML is passed through Readability. Scraping configuration below remains independent, so you can mix-and-match feed content and scraping strategies.
-      </p>
-    </div>
-  </div>
-
-  <% scrape_settings = (source.scrape_settings || {}).with_indifferent_access %>
-  <% selectors = scrape_settings[:selectors] || {} %>
-
-  <div class="rounded-lg border border-slate-200 bg-white shadow-sm">
-    <div class="border-b border-slate-200 px-5 py-4">
-      <h2 class="text-lg font-medium">Scraping Configuration</h2>
-      <p class="mt-1 text-xs text-slate-500">Control how content extraction runs for this source.</p>
-    </div>
-    <div class="space-y-5 px-5 py-5">
-      <div class="grid gap-3 sm:grid-cols-2">
-        <label class="inline-flex items-center space-x-2">
-          <%= form.check_box :scraping_enabled %>
-          <span>Scraping enabled</span>
-        </label>
-
-        <label class="inline-flex items-center space-x-2">
-          <%= form.check_box :auto_scrape %>
-          <span>Auto scrape after fetch</span>
-        </label>
-
-        <label class="inline-flex items-center space-x-2">
-          <%= form.check_box :requires_javascript %>
-          <span>Requires JavaScript</span>
-        </label>
-      </div>
-
-      <div>
-        <%= form.label :scraper_adapter, "Scraper adapter", class: "block text-sm font-medium text-slate-800" %>
-        <%= form.select :scraper_adapter, [["Readability", "readability"]], {}, class: "mt-1 block w-full rounded border border-slate-300 px-3 py-2" %>
-        <p class="mt-1 text-xs text-slate-500">Additional adapters can be configured in future phases.</p>
-      </div>
-
-      <div class="grid gap-4 sm:grid-cols-2">
-        <div>
-          <label for="source_scrape_settings_selectors_content" class="block text-sm font-medium text-slate-800">Content CSS selector</label>
-          <%= text_field_tag "source[scrape_settings][selectors][content]", selectors[:content], id: "source_scrape_settings_selectors_content", class: "mt-1 block w-full rounded border border-slate-300 px-3 py-2", placeholder: ".article-body" %>
-          <p class="mt-1 text-xs text-slate-500">Optional. Overrides Readability extraction target.</p>
-        </div>
-
-        <div>
-          <label for="source_scrape_settings_selectors_title" class="block text-sm font-medium text-slate-800">Title CSS selector</label>
-          <%= text_field_tag "source[scrape_settings][selectors][title]", selectors[:title], id: "source_scrape_settings_selectors_title", class: "mt-1 block w-full rounded border border-slate-300 px-3 py-2", placeholder: "h1.article-title" %>
-          <p class="mt-1 text-xs text-slate-500">Optional. Leave blank to use feed-provided title.</p>
-        </div>
-      </div>
-    </div>
-  </div>
-
-  <div>
-    <%= form.submit (source.persisted? ? "Update Source" : "Create Source"), class: "inline-flex items-center rounded bg-blue-600 px-4 py-2 text-white shadow hover:bg-blue-500" %>
-  </div>
+  <% end %>
 <% end %>