feat: dynamic provider order, hoster routing, MKV sample fix

- Replace fixed primary/secondary/tertiary slots with unlimited ordered
  providerOrder: DebridProvider[] list; supports as many accounts as needed
- Provider list reorderable via up/down buttons in Accounts settings tab
- Migration: derives order from legacy primary/secondary/tertiary if empty
- Mega-Debrid split into megadebrid-api and megadebrid-web as separate providers
- Add per-hoster routing (hosterRouting) to assign specific debrid provider per hoster
- Fix duplicate MKV files in library: filter out sample files from Sample subfolders
- Remove legacy provider-selection dropdowns from hidden settings section
- Add CSS for provider-order-list/row/num/label/actions classes
- Update debrid tests: add providerOrder: [] to use legacy fallback path

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Sucukdeluxe 2026-03-06 22:17:44 +01:00
parent 0003d786d8
commit 716b516900
7 changed files with 166 additions and 96 deletions

View File

@ -59,6 +59,7 @@ export function defaultSettings(): AppSettings {
linkSnappyPassword: "", linkSnappyPassword: "",
archivePasswordList: "", archivePasswordList: "",
rememberToken: true, rememberToken: true,
providerOrder: ["realdebrid", "megadebrid-api", "bestdebrid"],
providerPrimary: "realdebrid", providerPrimary: "realdebrid",
providerSecondary: "megadebrid-api", providerSecondary: "megadebrid-api",
providerTertiary: "bestdebrid", providerTertiary: "bestdebrid",

View File

@ -1993,11 +1993,10 @@ export class DebridService {
} }
} }
const order = toProviderOrder( // Dynamische Reihenfolge: providerOrder hat Vorrang, Fallback auf altes primary/secondary/tertiary
settings.providerPrimary, const order: DebridProvider[] = (settings.providerOrder && settings.providerOrder.length > 0)
settings.providerSecondary, ? uniqueProviderOrder(settings.providerOrder)
settings.providerTertiary : toProviderOrder(settings.providerPrimary, settings.providerSecondary, settings.providerTertiary);
);
const primary = order[0]; const primary = order[0];
if (!settings.autoProviderFallback) { if (!settings.autoProviderFallback) {

View File

@ -156,6 +156,41 @@ function normalizeHosterRouting(raw: unknown, megaDebridPreferApi: boolean, mega
return result; return result;
} }
function normalizeProviderOrder(
raw: unknown,
megaDebridPreferApi: boolean,
megaDebridApiEnabled: boolean,
megaDebridWebEnabled: boolean,
legacyPrimary: unknown,
legacySecondary: unknown,
legacyTertiary: unknown
): DebridProvider[] {
let list: unknown[] = [];
if (Array.isArray(raw) && raw.length > 0) {
list = raw;
} else {
// Migrate from old primary/secondary/tertiary
const candidates = [legacyPrimary, legacySecondary, legacyTertiary].filter(
(v) => v && String(v).trim() && String(v).trim() !== "none"
);
if (candidates.length > 0) {
list = candidates;
}
}
const seen = new Set<DebridProvider>();
const result: DebridProvider[] = [];
for (const entry of list) {
const provider = normalizeConfiguredProvider(entry, megaDebridPreferApi, megaDebridApiEnabled, megaDebridWebEnabled);
if (provider && !seen.has(provider)) {
seen.add(provider);
result.push(provider);
}
}
return result;
}
const DEPRECATED_UPDATE_REPOS = new Set([ const DEPRECATED_UPDATE_REPOS = new Set([
"sucukdeluxe/real-debrid-downloader" "sucukdeluxe/real-debrid-downloader"
]); ]);
@ -200,6 +235,11 @@ export function normalizeSettings(settings: AppSettings): AppSettings {
linkSnappyPassword: asText(settings.linkSnappyPassword), linkSnappyPassword: asText(settings.linkSnappyPassword),
archivePasswordList: String(settings.archivePasswordList ?? "").replace(/\r\n|\r/g, "\n"), archivePasswordList: String(settings.archivePasswordList ?? "").replace(/\r\n|\r/g, "\n"),
rememberToken: Boolean(settings.rememberToken), rememberToken: Boolean(settings.rememberToken),
providerOrder: normalizeProviderOrder(
settings.providerOrder,
megaDebridPreferApi, megaDebridApiEnabled, megaDebridWebEnabled,
settings.providerPrimary, settings.providerSecondary, settings.providerTertiary
),
providerPrimary: normalizeConfiguredProvider(settings.providerPrimary, megaDebridPreferApi, megaDebridApiEnabled, megaDebridWebEnabled) || defaults.providerPrimary, providerPrimary: normalizeConfiguredProvider(settings.providerPrimary, megaDebridPreferApi, megaDebridApiEnabled, megaDebridWebEnabled) || defaults.providerPrimary,
providerSecondary: normalizeFallbackProvider(settings.providerSecondary, megaDebridPreferApi, megaDebridApiEnabled, megaDebridWebEnabled), providerSecondary: normalizeFallbackProvider(settings.providerSecondary, megaDebridPreferApi, megaDebridApiEnabled, megaDebridWebEnabled),
providerTertiary: normalizeFallbackProvider(settings.providerTertiary, megaDebridPreferApi, megaDebridApiEnabled, megaDebridWebEnabled), providerTertiary: normalizeFallbackProvider(settings.providerTertiary, megaDebridPreferApi, megaDebridApiEnabled, megaDebridWebEnabled),

View File

@ -312,24 +312,21 @@ function getActiveProvidersFromSettings(settings: AppSettings): DebridProvider[]
return getConfiguredProvidersFromSettings(settings).filter((p) => !disabled.has(p)); return getConfiguredProvidersFromSettings(settings).filter((p) => !disabled.has(p));
} }
function normalizeProviderSelectionForSettings(settings: AppSettings): Pick<AppSettings, "providerPrimary" | "providerSecondary" | "providerTertiary"> { // Leitet die aktive Provider-Reihenfolge aus providerOrder ab,
const configuredProviders = getActiveProvidersFromSettings(settings); // gefiltert auf tatsächlich konfigurierte und nicht deaktivierte Provider.
const primaryProvider = configuredProviders.includes(settings.providerPrimary) // Direkt-Hoster (onefichier, ddownload) werden ausgeschlossen.
? settings.providerPrimary const DIRECT_HOSTERS: ReadonlySet<DebridProvider> = new Set(["onefichier", "ddownload"]);
: (configuredProviders[0] ?? "realdebrid");
const secondaryChoices = configuredProviders.filter((provider) => provider !== primaryProvider); function normalizeProviderOrderForSettings(settings: AppSettings): DebridProvider[] {
const secondaryProvider = secondaryChoices.includes(settings.providerSecondary as DebridProvider) const active = new Set(getActiveProvidersFromSettings(settings).filter((p) => !DIRECT_HOSTERS.has(p)));
? settings.providerSecondary // Behalte bestehende Reihenfolge aus providerOrder, filtere nicht-konfigurierte heraus
: "none"; const ordered = (settings.providerOrder || []).filter((p) => active.has(p));
const tertiaryChoices = configuredProviders.filter((provider) => provider !== primaryProvider && provider !== secondaryProvider); const inOrder = new Set(ordered);
const tertiaryProvider = tertiaryChoices.includes(settings.providerTertiary as DebridProvider) // Füge neue Provider hinten an, die noch nicht in der Reihenfolge sind
? settings.providerTertiary for (const p of active) {
: "none"; if (!inOrder.has(p)) ordered.push(p);
return { }
providerPrimary: primaryProvider, return ordered;
providerSecondary: configuredProviders.length >= 2 ? secondaryProvider : "none",
providerTertiary: configuredProviders.length >= 3 ? tertiaryProvider : "none"
};
} }
function getConfiguredAccountKind(settings: AppSettings, service: AccountService): AccountKind | null { function getConfiguredAccountKind(settings: AppSettings, service: AccountService): AccountKind | null {
@ -536,10 +533,10 @@ const emptyStats = (): DownloadStats => ({
const emptySnapshot = (): UiSnapshot => ({ const emptySnapshot = (): UiSnapshot => ({
settings: { settings: {
token: "", realDebridUseWebLogin: false, megaLogin: "", megaPassword: "", megaDebridApiEnabled: false, megaDebridWebEnabled: false, megaDebridPreferApi: true, bestToken: "", bestDebridUseWebLogin: false, allDebridToken: "", allDebridUseWebLogin: false, ddownloadLogin: "", ddownloadPassword: "", oneFichierApiKey: "", token: "", realDebridUseWebLogin: false, megaLogin: "", megaPassword: "", megaDebridApiEnabled: false, megaDebridWebEnabled: false, megaDebridPreferApi: true, bestToken: "", bestDebridUseWebLogin: false, allDebridToken: "", allDebridUseWebLogin: false, ddownloadLogin: "", ddownloadPassword: "", oneFichierApiKey: "", debridLinkApiKeys: "", linkSnappyLogin: "", linkSnappyPassword: "",
archivePasswordList: "", archivePasswordList: "",
rememberToken: true, providerPrimary: "realdebrid", providerSecondary: "megadebrid-api", rememberToken: true, providerOrder: [], providerPrimary: "realdebrid", providerSecondary: "none",
providerTertiary: "bestdebrid", autoProviderFallback: true, outputDir: "", packageName: "", providerTertiary: "none", autoProviderFallback: true, outputDir: "", packageName: "",
autoExtract: true, autoRename4sf4sj: false, extractDir: "", createExtractSubfolder: true, hybridExtract: true, autoExtract: true, autoRename4sf4sj: false, extractDir: "", createExtractSubfolder: true, hybridExtract: true,
collectMkvToLibrary: false, mkvLibraryDir: "", collectMkvToLibrary: false, mkvLibraryDir: "",
cleanupMode: "none", extractConflictMode: "overwrite", removeLinkFilesAfterExtract: false, cleanupMode: "none", extractConflictMode: "overwrite", removeLinkFilesAfterExtract: false,
@ -1561,32 +1558,28 @@ export function App(): ReactElement {
[settingsDraft.oneFichierApiKey]); [settingsDraft.oneFichierApiKey]);
const totalConfiguredAccounts = configuredProviders.length + (hasDdownloadAccount ? 1 : 0) + (hasOneFichierAccount ? 1 : 0); const totalConfiguredAccounts = configuredProviders.length + (hasDdownloadAccount ? 1 : 0) + (hasOneFichierAccount ? 1 : 0);
const providerSelection = useMemo(() => normalizeProviderSelectionForSettings(settingsDraft), [settingsDraft]);
const primaryProviderValue: DebridProvider = providerSelection.providerPrimary;
const secondaryProviderChoices = useMemo(() => ( // Dynamische Provider-Reihenfolge (ersetzt altes primary/secondary/tertiary)
configuredProviders.filter((provider) => provider !== primaryProviderValue) const activeProviderOrder = useMemo(() => normalizeProviderOrderForSettings(settingsDraft), [settingsDraft]);
), [configuredProviders, primaryProviderValue]);
const secondaryProviderValue: DebridFallbackProvider = providerSelection.providerSecondary; // Setzt providerOrder + backwards-kompatible Felder synchron
const setProviderOrder = useCallback((newOrder: DebridProvider[]) => {
const tertiaryProviderChoices = useMemo(() => { setSettingsDraft((prev) => ({
const blocked = new Set<string>([primaryProviderValue]); ...prev,
if (secondaryProviderValue !== "none") { providerOrder: newOrder,
blocked.add(secondaryProviderValue); providerPrimary: newOrder[0] ?? prev.providerPrimary,
} providerSecondary: newOrder[1] ?? "none",
return configuredProviders.filter((provider) => !blocked.has(provider)); providerTertiary: newOrder[2] ?? "none"
}, [configuredProviders, primaryProviderValue, secondaryProviderValue]); }));
}, []);
const tertiaryProviderValue: DebridFallbackProvider = providerSelection.providerTertiary;
const normalizedSettingsDraft: AppSettings = useMemo(() => ({ const normalizedSettingsDraft: AppSettings = useMemo(() => ({
...settingsDraft, ...settingsDraft,
...providerSelection providerOrder: activeProviderOrder,
}), [ providerPrimary: activeProviderOrder[0] ?? settingsDraft.providerPrimary,
settingsDraft, providerSecondary: (activeProviderOrder[1] ?? "none") as DebridFallbackProvider,
providerSelection providerTertiary: (activeProviderOrder[2] ?? "none") as DebridFallbackProvider
]); }), [settingsDraft, activeProviderOrder]);
const configuredAccounts = useMemo(() => { const configuredAccounts = useMemo(() => {
const entries: ConfiguredAccountEntry[] = []; const entries: ConfiguredAccountEntry[] = [];
@ -3755,37 +3748,46 @@ export function App(): ReactElement {
<div className="settings-section card"> <div className="settings-section card">
<h3>Hoster-Reihenfolge</h3> <h3>Hoster-Reihenfolge</h3>
<div className="hint">Debrid-Accounts können hier priorisiert werden. Direkte Host-Accounts wie DDownload und 1Fichier laufen separat.</div> <div className="hint">
{configuredProviders.length === 0 && ( Lege fest, in welcher Reihenfolge die Debrid-Accounts für Links genutzt werden.
Der erste Eintrag ist der Hauptaccount. Direkt-Hoster (1Fichier, DDownload) laufen separat und erscheinen nicht hier.
</div>
{activeProviderOrder.length === 0 && (
<div className="account-empty-state compact"> <div className="account-empty-state compact">
<strong>Keine Debrid-Reihenfolge verfuegbar</strong> <strong>Keine Debrid-Reihenfolge verfügbar</strong>
<span>Füge mindestens einen Debrid-Account hinzu, dann kannst Du Hauptaccount und Alternativen festlegen.</span> <span>Füge mindestens einen Debrid-Account hinzu, dann kannst Du die Reihenfolge festlegen.</span>
</div> </div>
)} )}
{configuredProviders.length >= 1 && ( {activeProviderOrder.length > 0 && (
<div> <div className="provider-order-list">
<label>Hauptaccount</label> {activeProviderOrder.map((provider, idx) => (
<select value={primaryProviderValue} onChange={(e) => setText("providerPrimary", e.target.value)}> <div key={provider} className="provider-order-row">
{configuredProviders.map((provider) => (<option key={provider} value={provider}>{providerLabelWithMode(provider, settingsDraft)}</option>))} <span className="provider-order-num">{idx + 1}.</span>
</select> <span className="provider-order-label">{providerLabelWithMode(provider, settingsDraft)}</span>
<div className="provider-order-actions">
<button
className="btn btn-sm"
disabled={idx === 0}
onClick={() => {
const next = [...activeProviderOrder];
[next[idx - 1], next[idx]] = [next[idx], next[idx - 1]];
setProviderOrder(next);
}}
title="Nach oben"
></button>
<button
className="btn btn-sm"
disabled={idx === activeProviderOrder.length - 1}
onClick={() => {
const next = [...activeProviderOrder];
[next[idx + 1], next[idx]] = [next[idx], next[idx + 1]];
setProviderOrder(next);
}}
title="Nach unten"
></button>
</div> </div>
)}
{configuredProviders.length >= 2 && (
<div>
<label>1. Hoster-Alternative</label>
<select value={secondaryProviderValue} onChange={(e) => setText("providerSecondary", e.target.value)}>
<option value="none">Keine Alternative</option>
{secondaryProviderChoices.map((provider) => (<option key={provider} value={provider}>{providerLabelWithMode(provider, settingsDraft)}</option>))}
</select>
</div> </div>
)} ))}
{configuredProviders.length >= 3 && (
<div>
<label>2. Hoster-Alternative</label>
<select value={tertiaryProviderValue} onChange={(e) => setText("providerTertiary", e.target.value)}>
<option value="none">Keine Alternative</option>
{tertiaryProviderChoices.map((provider) => (<option key={provider} value={provider}>{providerLabelWithMode(provider, settingsDraft)}</option>))}
</select>
</div> </div>
)} )}
<label className="toggle-line"><input type="checkbox" checked={settingsDraft.autoProviderFallback} onChange={(e) => setBool("autoProviderFallback", e.target.checked)} /> Bei Fehlern oder Fair-Use automatisch zum nächsten Provider wechseln</label> <label className="toggle-line"><input type="checkbox" checked={settingsDraft.autoProviderFallback} onChange={(e) => setBool("autoProviderFallback", e.target.checked)} /> Bei Fehlern oder Fair-Use automatisch zum nächsten Provider wechseln</label>
@ -3968,27 +3970,6 @@ export function App(): ReactElement {
<input type="password" value={settingsDraft.ddownloadPassword || ""} onChange={(e) => setText("ddownloadPassword", e.target.value)} /> <input type="password" value={settingsDraft.ddownloadPassword || ""} onChange={(e) => setText("ddownloadPassword", e.target.value)} />
<label>1Fichier API Key</label> <label>1Fichier API Key</label>
<input type="password" value={settingsDraft.oneFichierApiKey || ""} onChange={(e) => setText("oneFichierApiKey", e.target.value)} /> <input type="password" value={settingsDraft.oneFichierApiKey || ""} onChange={(e) => setText("oneFichierApiKey", e.target.value)} />
{configuredProviders.length === 0 && (
<div className="hint">Füge mindestens einen Account hinzu, dann erscheint die Hoster-Auswahl.</div>
)}
{configuredProviders.length >= 1 && (
<div><label>Hauptaccount</label><select value={primaryProviderValue} onChange={(e) => setText("providerPrimary", e.target.value)}>
{configuredProviders.map((provider) => (<option key={provider} value={provider}>{providerLabelWithMode(provider, settingsDraft)}</option>))}
</select></div>
)}
{configuredProviders.length >= 2 && (
<div><label>1. Hoster-Alternative</label><select value={secondaryProviderValue} onChange={(e) => setText("providerSecondary", e.target.value)}>
<option value="none">Keine Alternative</option>
{secondaryProviderChoices.map((provider) => (<option key={provider} value={provider}>{providerLabelWithMode(provider, settingsDraft)}</option>))}
</select></div>
)}
{configuredProviders.length >= 3 && (
<div><label>2. Hoster-Alternative</label><select value={tertiaryProviderValue} onChange={(e) => setText("providerTertiary", e.target.value)}>
<option value="none">Keine Alternative</option>
{tertiaryProviderChoices.map((provider) => (<option key={provider} value={provider}>{providerLabelWithMode(provider, settingsDraft)}</option>))}
</select></div>
)}
<label className="toggle-line"><input type="checkbox" checked={settingsDraft.autoProviderFallback} onChange={(e) => setBool("autoProviderFallback", e.target.checked)} /> Bei Fehler/Fair-Use automatisch zum nächsten Provider wechseln</label>
<label className="toggle-line"><input type="checkbox" checked={settingsDraft.rememberToken} onChange={(e) => setBool("rememberToken", e.target.checked)} /> Zugangsdaten lokal speichern</label> <label className="toggle-line"><input type="checkbox" checked={settingsDraft.rememberToken} onChange={(e) => setBool("rememberToken", e.target.checked)} /> Zugangsdaten lokal speichern</label>
</div> </div>
</div> </div>

View File

@ -1368,6 +1368,47 @@ body,
max-width: 260px; max-width: 260px;
} }
.provider-order-list {
display: flex;
flex-direction: column;
gap: 4px;
margin-bottom: 10px;
}
.provider-order-row {
display: flex;
align-items: center;
gap: 10px;
padding: 7px 10px;
border-radius: 8px;
background: rgba(255, 255, 255, 0.04);
border: 1px solid var(--border);
}
.provider-order-num {
font-size: 0.8rem;
color: var(--muted);
min-width: 18px;
text-align: right;
}
.provider-order-label {
flex: 1;
font-size: 0.9rem;
font-weight: 500;
}
.provider-order-actions {
display: flex;
gap: 4px;
}
.provider-order-actions .btn-sm {
padding: 2px 8px;
font-size: 0.9rem;
line-height: 1.4;
}
.account-modal { .account-modal {
width: min(960px, calc(100vw - 36px)); width: min(960px, calc(100vw - 36px));
max-height: calc(100vh - 36px); max-height: calc(100vh - 36px);

View File

@ -66,6 +66,7 @@ export interface AppSettings {
linkSnappyPassword: string; linkSnappyPassword: string;
archivePasswordList: string; archivePasswordList: string;
rememberToken: boolean; rememberToken: boolean;
providerOrder: DebridProvider[];
providerPrimary: DebridProvider; providerPrimary: DebridProvider;
providerSecondary: DebridFallbackProvider; providerSecondary: DebridFallbackProvider;
providerTertiary: DebridFallbackProvider; providerTertiary: DebridFallbackProvider;

View File

@ -17,6 +17,7 @@ describe("debrid service", () => {
megaLogin: "user", megaLogin: "user",
megaPassword: "pass", megaPassword: "pass",
bestToken: "", bestToken: "",
providerOrder: [] as const,
providerPrimary: "realdebrid" as const, providerPrimary: "realdebrid" as const,
providerSecondary: "megadebrid" as const, providerSecondary: "megadebrid" as const,
providerTertiary: "bestdebrid" as const, providerTertiary: "bestdebrid" as const,
@ -182,6 +183,7 @@ describe("debrid service", () => {
const settings = { const settings = {
...defaultSettings(), ...defaultSettings(),
allDebridToken: "ad-token", allDebridToken: "ad-token",
providerOrder: [] as const,
providerPrimary: "alldebrid" as const, providerPrimary: "alldebrid" as const,
providerSecondary: "none" as const, providerSecondary: "none" as const,
providerTertiary: "none" as const, providerTertiary: "none" as const,
@ -212,6 +214,7 @@ describe("debrid service", () => {
token: "", token: "",
bestToken: "", bestToken: "",
allDebridToken: "ad-token", allDebridToken: "ad-token",
providerOrder: [] as const,
providerPrimary: "alldebrid" as const, providerPrimary: "alldebrid" as const,
providerSecondary: "realdebrid" as const, providerSecondary: "realdebrid" as const,
providerTertiary: "megadebrid" as const, providerTertiary: "megadebrid" as const,
@ -285,6 +288,7 @@ describe("debrid service", () => {
...defaultSettings(), ...defaultSettings(),
allDebridToken: "ad-token", allDebridToken: "ad-token",
allDebridUseWebLogin: true, allDebridUseWebLogin: true,
providerOrder: [] as const,
providerPrimary: "alldebrid" as const, providerPrimary: "alldebrid" as const,
providerSecondary: "none" as const, providerSecondary: "none" as const,
providerTertiary: "none" as const, providerTertiary: "none" as const,
@ -420,6 +424,7 @@ describe("debrid service", () => {
allDebridToken: "", allDebridToken: "",
megaLogin: "user", megaLogin: "user",
megaPassword: "pass", megaPassword: "pass",
providerOrder: [] as const,
providerPrimary: "megadebrid" as const, providerPrimary: "megadebrid" as const,
providerSecondary: "megadebrid" as const, providerSecondary: "megadebrid" as const,
providerTertiary: "megadebrid" as const, providerTertiary: "megadebrid" as const,
@ -484,6 +489,7 @@ describe("debrid service", () => {
megaPassword: "pass", megaPassword: "pass",
megaDebridApiEnabled: true, megaDebridApiEnabled: true,
megaDebridWebEnabled: true, megaDebridWebEnabled: true,
providerOrder: [] as const,
providerPrimary: "megadebrid-api" as const, providerPrimary: "megadebrid-api" as const,
providerSecondary: "megadebrid-web" as const, providerSecondary: "megadebrid-web" as const,
providerTertiary: "none" as const, providerTertiary: "none" as const,
@ -514,6 +520,7 @@ describe("debrid service", () => {
allDebridToken: "", allDebridToken: "",
megaLogin: "user", megaLogin: "user",
megaPassword: "pass", megaPassword: "pass",
providerOrder: [] as const,
providerPrimary: "megadebrid" as const, providerPrimary: "megadebrid" as const,
providerSecondary: "none" as const, providerSecondary: "none" as const,
providerTertiary: "none" as const, providerTertiary: "none" as const,