Refactor indexer management and improve search logic

Refactors admin settings to use a new IndexersTab and card-based indexer management UI, supporting category selection and improved configuration. Updates backend and API routes to handle indexer categories, propagate ASIN for better search scoring, and group indexers by categories to optimize Prowlarr searches. Enhances documentation to clarify non-terminal request matching and auto-completion behavior. Adds new reusable components for indexer management and category selection.

src/app/setup/steps/ProwlarrStep.tsx

@@ -5,9 +5,10 @@
 
 'use client';
 
-import { useState } from 'react';
+import { useState, useEffect } from 'react';
 import { Button } from '@/components/ui/Button';
 import { Input } from '@/components/ui/Input';
+import { IndexerManagement } from '@/components/admin/indexers/IndexerManagement';
 
 interface ProwlarrStepProps {
   prowlarrUrl: string;
@@ -17,19 +18,13 @@ interface ProwlarrStepProps {
   onBack: () => void;
 }
 
-interface IndexerInfo {
-  id: number;
-  name: string;
-  protocol: string;
-  supportsRss: boolean;
-}
-
 interface SelectedIndexer {
   id: number;
   name: string;
   priority: number;
   seedingTimeMinutes: number;
   rssEnabled: boolean;
+  categories: number[];
 }
 
 export function ProwlarrStep({
@@ -39,141 +34,24 @@ export function ProwlarrStep({
   onNext,
   onBack,
 }: ProwlarrStepProps) {
-  const [testing, setTesting] = useState(false);
-  const [testResult, setTestResult] = useState<{
-    success: boolean;
-    message: string;
-    indexerCount?: number;
-  } | null>(null);
-  const [availableIndexers, setAvailableIndexers] = useState<IndexerInfo[]>([]);
-  const [selectedIndexers, setSelectedIndexers] = useState<Record<number, SelectedIndexer>>({});
+  const [configuredIndexers, setConfiguredIndexers] = useState<SelectedIndexer[]>([]);
+  const [errorMessage, setErrorMessage] = useState<string | null>(null);
 
-  const testConnection = async () => {
-    setTesting(true);
-    setTestResult(null);
-
-    try {
-      const response = await fetch('/api/setup/test-prowlarr', {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ url: prowlarrUrl, apiKey: prowlarrApiKey }),
-      });
-
-      const data = await response.json();
-
-      if (response.ok && data.success) {
-        setTestResult({
-          success: true,
-          message: `Connected successfully! Found ${data.indexerCount || 0} configured indexers.`,
-          indexerCount: data.indexerCount,
-        });
-        setAvailableIndexers(data.indexers || []);
-
-        // Auto-select all indexers with default priority of 10, seeding time of 0 (unlimited), and RSS enabled if supported
-        const autoSelected: Record<number, SelectedIndexer> = {};
-        data.indexers.forEach((indexer: IndexerInfo) => {
-          autoSelected[indexer.id] = {
-            id: indexer.id,
-            name: indexer.name,
-            priority: 10,
-            seedingTimeMinutes: 0,
-            rssEnabled: indexer.supportsRss, // Enable RSS by default if supported
-          };
-        });
-        setSelectedIndexers(autoSelected);
-        onUpdate('prowlarrIndexers', Object.values(autoSelected));
-      } else {
-        setTestResult({
-          success: false,
-          message: data.error || 'Connection failed',
-        });
-      }
-    } catch (error) {
-      setTestResult({
-        success: false,
-        message: error instanceof Error ? error.message : 'Connection test failed',
-      });
-    } finally {
-      setTesting(false);
-    }
-  };
-
-  const toggleIndexer = (indexer: IndexerInfo) => {
-    setSelectedIndexers((prev) => {
-      const newSelected = { ...prev };
-      if (newSelected[indexer.id]) {
-        delete newSelected[indexer.id];
-      } else {
-        newSelected[indexer.id] = {
-          id: indexer.id,
-          name: indexer.name,
-          priority: 10, // Default priority
-          seedingTimeMinutes: 0, // Default: unlimited seeding
-          rssEnabled: indexer.supportsRss, // Enable RSS by default if supported
-        };
-      }
-      onUpdate('prowlarrIndexers', Object.values(newSelected));
-      return newSelected;
-    });
-  };
-
-  const updatePriority = (indexerId: number, priority: number) => {
-    setSelectedIndexers((prev) => {
-      const newSelected = { ...prev };
-      if (newSelected[indexerId]) {
-        newSelected[indexerId] = {
-          ...newSelected[indexerId],
-          priority: Math.max(1, Math.min(25, priority)), // Clamp between 1-25
-        };
-      }
-      onUpdate('prowlarrIndexers', Object.values(newSelected));
-      return newSelected;
-    });
-  };
-
-  const updateSeedingTime = (indexerId: number, value: string) => {
-    setSelectedIndexers((prev) => {
-      const newSelected = { ...prev };
-      if (newSelected[indexerId]) {
-        const seedingTimeMinutes = value === '' ? 0 : parseInt(value);
-        newSelected[indexerId] = {
-          ...newSelected[indexerId],
-          seedingTimeMinutes: isNaN(seedingTimeMinutes) ? 0 : Math.max(0, seedingTimeMinutes),
-        };
-      }
-      onUpdate('prowlarrIndexers', Object.values(newSelected));
-      return newSelected;
-    });
-  };
-
-  const toggleRss = (indexerId: number) => {
-    setSelectedIndexers((prev) => {
-      const newSelected = { ...prev };
-      if (newSelected[indexerId]) {
-        newSelected[indexerId] = {
-          ...newSelected[indexerId],
-          rssEnabled: !newSelected[indexerId].rssEnabled,
-        };
-      }
-      onUpdate('prowlarrIndexers', Object.values(newSelected));
-      return newSelected;
-    });
-  };
+  // Sync configured indexers with parent
+  useEffect(() => {
+    onUpdate('prowlarrIndexers', configuredIndexers);
+  }, [configuredIndexers, onUpdate]);
 
   const handleNext = () => {
-    if (!testResult?.success) {
-      setTestResult({
-        success: false,
-        message: 'Please test the connection before proceeding',
-      });
+    setErrorMessage(null);
+
+    if (!prowlarrUrl || !prowlarrApiKey) {
+      setErrorMessage('Please enter Prowlarr URL and API key');
       return;
     }
 
-    if (Object.keys(selectedIndexers).length === 0) {
-      setTestResult({
-        success: false,
-        message: 'Please select at least one indexer',
-      });
+    if (configuredIndexers.length === 0) {
+      setErrorMessage('Please add at least one indexer');
       return;
     }
 
@@ -222,207 +100,52 @@ export function ProwlarrStep({
           </p>
         </div>
 
-        <Button
-          onClick={testConnection}
-          loading={testing}
-          disabled={!prowlarrUrl || !prowlarrApiKey}
-          variant="outline"
-          className="w-full"
-        >
-          Test Connection
-        </Button>
-
-        {testResult && (
-          <div
-            className={`rounded-lg p-4 ${
-              testResult.success
-                ? 'bg-green-50 dark:bg-green-900/20 border border-green-200 dark:border-green-800'
-                : 'bg-red-50 dark:bg-red-900/20 border border-red-200 dark:border-red-800'
-            }`}
-          >
+        {errorMessage && (
+          <div className="rounded-lg p-4 bg-red-50 dark:bg-red-900/20 border border-red-200 dark:border-red-800">
             <div className="flex gap-3">
               <svg
-                className={`w-6 h-6 flex-shrink-0 ${
-                  testResult.success
-                    ? 'text-green-600 dark:text-green-400'
-                    : 'text-red-600 dark:text-red-400'
-                }`}
+                className="w-6 h-6 flex-shrink-0 text-red-600 dark:text-red-400"
                 fill="currentColor"
                 viewBox="0 0 20 20"
               >
-                {testResult.success ? (
-                  <path
-                    fillRule="evenodd"
-                    d="M10 18a8 8 0 100-16 8 8 0 000 16zm3.707-9.293a1 1 0 00-1.414-1.414L9 10.586 7.707 9.293a1 1 0 00-1.414 1.414l2 2a1 1 0 001.414 0l4-4z"
-                    clipRule="evenodd"
-                  />
-                ) : (
-                  <path
-                    fillRule="evenodd"
-                    d="M10 18a8 8 0 100-16 8 8 0 000 16zM8.707 7.293a1 1 0 00-1.414 1.414L8.586 10l-1.293 1.293a1 1 0 101.414 1.414L10 11.414l1.293 1.293a1 1 0 001.414-1.414L11.414 10l1.293-1.293a1 1 0 00-1.414-1.414L10 8.586 8.707 7.293z"
-                    clipRule="evenodd"
-                  />
-                )}
+                <path
+                  fillRule="evenodd"
+                  d="M10 18a8 8 0 100-16 8 8 0 000 16zM8.707 7.293a1 1 0 00-1.414 1.414L8.586 10l-1.293 1.293a1 1 0 101.414 1.414L10 11.414l1.293 1.293a1 1 0 001.414-1.414L11.414 10l1.293-1.293a1 1 0 00-1.414-1.414L10 8.586 8.707 7.293z"
+                  clipRule="evenodd"
+                />
               </svg>
               <div>
-                <h3
-                  className={`text-sm font-medium ${
-                    testResult.success
-                      ? 'text-green-800 dark:text-green-200'
-                      : 'text-red-800 dark:text-red-200'
-                  }`}
-                >
-                  {testResult.success ? 'Success' : 'Error'}
+                <h3 className="text-sm font-medium text-red-800 dark:text-red-200">
+                  Error
                 </h3>
-                <p
-                  className={`text-sm mt-1 ${
-                    testResult.success
-                      ? 'text-green-700 dark:text-green-300'
-                      : 'text-red-700 dark:text-red-300'
-                  }`}
-                >
-                  {testResult.message}
+                <p className="text-sm mt-1 text-red-700 dark:text-red-300">
+                  {errorMessage}
                 </p>
               </div>
             </div>
           </div>
         )}
 
-        {/* Indexer Selection */}
-        {availableIndexers.length > 0 && (
-          <div className="space-y-3">
-            <div className="border-t border-gray-200 dark:border-gray-700 pt-4">
-              <h3 className="text-sm font-medium text-gray-900 dark:text-gray-100 mb-3">
-                Select Indexers & Configure (Priority: 1-25, Seeding Time, RSS)
-              </h3>
-              <p className="text-sm text-gray-600 dark:text-gray-400 mb-4">
-                Higher priority indexers (closer to 25) will be preferred when ranking search results.
-                Seeding time is in minutes (0 = unlimited). Files will be kept until the seeding requirement is met.
-                Enable RSS to automatically monitor indexer feeds for new releases matching your missing list (default: every 15 minutes, configurable in scheduled jobs settings).
-              </p>
-              <div className="space-y-2 max-h-64 overflow-y-auto">
-                {availableIndexers.map((indexer) => (
-                  <div
-                    key={indexer.id}
-                    className="flex items-center gap-3 p-3 bg-gray-50 dark:bg-gray-800 rounded-lg"
-                  >
-                    <input
-                      type="checkbox"
-                      id={`indexer-${indexer.id}`}
-                      checked={!!selectedIndexers[indexer.id]}
-                      onChange={() => toggleIndexer(indexer)}
-                      className="w-4 h-4 text-blue-600 bg-gray-100 border-gray-300 rounded focus:ring-blue-500"
-                    />
-                    <label
-                      htmlFor={`indexer-${indexer.id}`}
-                      className="flex-1 text-sm font-medium text-gray-900 dark:text-gray-100 cursor-pointer"
-                    >
-                      {indexer.name}
-                      <span className="text-xs text-gray-500 dark:text-gray-400 ml-2">
-                        ({indexer.protocol})
-                      </span>
-                    </label>
-                    {selectedIndexers[indexer.id] && (
-                      <div className="flex items-center gap-4">
-                        <div className="flex items-center gap-2">
-                          <label
-                            htmlFor={`priority-${indexer.id}`}
-                            className="text-xs text-gray-600 dark:text-gray-400"
-                          >
-                            Priority:
-                          </label>
-                          <input
-                            id={`priority-${indexer.id}`}
-                            type="number"
-                            min="1"
-                            max="25"
-                            value={selectedIndexers[indexer.id].priority}
-                            onChange={(e) =>
-                              updatePriority(indexer.id, parseInt(e.target.value) || 10)
-                            }
-                            className="w-16 px-2 py-1 text-sm border border-gray-300 dark:border-gray-600 rounded bg-white dark:bg-gray-700 text-gray-900 dark:text-gray-100"
-                          />
-                        </div>
-                        <div className="flex items-center gap-2">
-                          <label
-                            htmlFor={`seeding-${indexer.id}`}
-                            className="text-xs text-gray-600 dark:text-gray-400"
-                          >
-                            Seeding (min):
-                          </label>
-                          <input
-                            id={`seeding-${indexer.id}`}
-                            type="number"
-                            min="0"
-                            step="1"
-                            value={selectedIndexers[indexer.id].seedingTimeMinutes}
-                            onChange={(e) =>
-                              updateSeedingTime(indexer.id, e.target.value)
-                            }
-                            className="w-20 px-2 py-1 text-sm border border-gray-300 dark:border-gray-600 rounded bg-white dark:bg-gray-700 text-gray-900 dark:text-gray-100"
-                            placeholder="0 = ∞"
-                          />
-                        </div>
-                        {indexer.supportsRss && (
-                          <div className="flex items-center gap-2">
-                            <label
-                              htmlFor={`rss-${indexer.id}`}
-                              className="text-xs text-gray-600 dark:text-gray-400"
-                            >
-                              RSS:
-                            </label>
-                            <input
-                              id={`rss-${indexer.id}`}
-                              type="checkbox"
-                              checked={selectedIndexers[indexer.id].rssEnabled}
-                              onChange={() => toggleRss(indexer.id)}
-                              className="w-4 h-4 text-blue-600 bg-gray-100 border-gray-300 rounded focus:ring-blue-500"
-                            />
-                          </div>
-                        )}
-                      </div>
-                    )}
-                  </div>
-                ))}
-              </div>
-              <p className="text-xs text-gray-500 dark:text-gray-400 mt-2">
-                Selected: {Object.keys(selectedIndexers).length} of {availableIndexers.length} indexers
-              </p>
-            </div>
-          </div>
-        )}
-      </div>
-
-      <div className="bg-blue-50 dark:bg-blue-900/20 rounded-lg p-4">
-        <div className="flex gap-3">
-          <svg
-            className="w-6 h-6 text-blue-600 dark:text-blue-400 flex-shrink-0"
-            fill="currentColor"
-            viewBox="0 0 20 20"
-          >
-            <path
-              fillRule="evenodd"
-              d="M18 10a8 8 0 11-16 0 8 8 0 0116 0zm-7-4a1 1 0 11-2 0 1 1 0 012 0zM9 9a1 1 0 000 2v3a1 1 0 001 1h1a1 1 0 100-2v-3a1 1 0 00-1-1H9z"
-              clipRule="evenodd"
-            />
-          </svg>
-          <div>
-            <p className="text-sm font-medium text-blue-900 dark:text-blue-100">
-              About Prowlarr Indexers
-            </p>
-            <p className="text-sm text-blue-700 dark:text-blue-300 mt-1">
-              Prowlarr searches across multiple torrent indexers. Select which indexers to use and assign priorities to control
-              how search results are ranked. Make sure you have at least one indexer configured in Prowlarr before proceeding.
-            </p>
-          </div>
+        {/* Indexer Management Component */}
+        <div className="border-t border-gray-200 dark:border-gray-700 pt-6">
+          <IndexerManagement
+            prowlarrUrl={prowlarrUrl}
+            prowlarrApiKey={prowlarrApiKey}
+            mode="wizard"
+            initialIndexers={configuredIndexers}
+            onIndexersChange={setConfiguredIndexers}
+          />
         </div>
       </div>
 
-      <div className="flex justify-between pt-4">
+      {/* Navigation Buttons */}
+      <div className="flex justify-between pt-6 border-t border-gray-200 dark:border-gray-700">
         <Button onClick={onBack} variant="outline">
           Back
         </Button>
-        <Button onClick={handleNext}>Next</Button>
+        <Button onClick={handleNext} variant="primary">
+          Next
+        </Button>
       </div>
     </div>
   );