fix: use indexer reported langague in evaluator in front of release title parsed value, specify source in parsed badge

Author: santiagosayshey

docs/api/v1/schemas/entity-testing.yaml

@@ -12,6 +12,7 @@ ParsedInfo:
     - resolution
     - modifier
     - languages
+    - languageSource
     - year
   properties:
     source:
@@ -28,6 +29,12 @@ ParsedInfo:
       items:
         type: string
       description: Detected languages
+    languageSource:
+      type: string
+      enum:
+        - Indexer
+        - Title
+      description: Where the language information came from (Indexer = indexer-provided, Title = parsed from release title)
     releaseGroup:
       type: string
       nullable: true
@@ -59,6 +66,11 @@ ReleaseInput:
       description: Release title to parse and evaluate
     type:
       $ref: '#/MediaType'
+    languages:
+      type: array
+      items:
+        type: string
+      description: Indexer-provided languages. When present and non-empty, these override languages parsed from the title.
 
 ReleaseEvaluation:
   type: object

src/lib/api/v1.d.ts

@@ -287,6 +287,11 @@ export interface components {
             modifier: string;
             /** @description Detected languages */
             languages: string[];
+            /**
+             * @description Where the language information came from (Indexer = indexer-provided, Title = parsed from release title)
+             * @enum {string}
+             */
+            languageSource: "Indexer" | "Title";
             /** @description Detected release group */
             releaseGroup?: string | null;
             /** @description Detected year */
@@ -302,6 +307,8 @@ export interface components {
             /** @description Release title to parse and evaluate */
             title: string;
             type: components["schemas"]["MediaType"];
+            /** @description Indexer-provided languages. When present and non-empty, these override languages parsed from the title. */
+            languages?: string[];
         };
         ReleaseEvaluation: {
             /** @description Release ID */

src/lib/server/pcd/entities/customFormats/evaluator.ts

@@ -63,6 +63,20 @@ const sourceNames: Record<QualitySource, string> = {
 	[QualitySource.Bluray]: 'bluray'
 };
 
+// Display-friendly names for parsed info shown in the UI
+const sourceDisplayNames: Record<QualitySource, string> = {
+	[QualitySource.Unknown]: 'Unknown',
+	[QualitySource.Cam]: 'CAM',
+	[QualitySource.Telesync]: 'Telesync',
+	[QualitySource.Telecine]: 'Telecine',
+	[QualitySource.Workprint]: 'Workprint',
+	[QualitySource.DVD]: 'DVD',
+	[QualitySource.TV]: 'Television',
+	[QualitySource.WebDL]: 'WEB-DL',
+	[QualitySource.WebRip]: 'WEBRip',
+	[QualitySource.Bluray]: 'Blu-ray'
+};
+
 const resolutionNames: Record<Resolution, string> = {
 	[Resolution.Unknown]: 'unknown',
 	[Resolution.R360p]: '360p',
@@ -83,6 +97,15 @@ const modifierNames: Record<QualityModifier, string> = {
 	[QualityModifier.Remux]: 'remux'
 };
 
+const modifierDisplayNames: Record<QualityModifier, string> = {
+	[QualityModifier.None]: 'None',
+	[QualityModifier.Regional]: 'Regional',
+	[QualityModifier.Screener]: 'Screener',
+	[QualityModifier.RawHD]: 'Raw-HD',
+	[QualityModifier.BRDisk]: 'BR-DISK',
+	[QualityModifier.Remux]: 'Remux'
+};
+
 const releaseTypeNames: Record<ReleaseType, string> = {
 	[ReleaseType.Unknown]: 'unknown',
 	[ReleaseType.SingleEpisode]: 'single_episode',
@@ -152,15 +175,36 @@ const languageNames: Record<Language, string> = {
 	[Language.Original]: 'Original'
 };
 
+/** Reverse lookup: display name → Language enum value */
+const languageNameToEnum = new Map<string, Language>(
+	Object.entries(languageNames).map(([enumVal, name]) => [name, Number(enumVal) as Language])
+);
+
+/** Convert indexer language display names to Language enum values. Unrecognized names are skipped. */
+function indexerLanguagesToEnum(names: string[]): Language[] {
+	const result: Language[] = [];
+	for (const name of names) {
+		const enumVal = languageNameToEnum.get(name);
+		if (enumVal !== undefined) {
+			result.push(enumVal);
+		}
+	}
+	return result;
+}
+
 /**
  * Get serializable parsed info for frontend display
  */
-export function getParsedInfo(parsed: ParseResult): ParsedInfo {
+export function getParsedInfo(parsed: ParseResult, indexerLangs?: string[]): ParsedInfo {
+	const useIndexer = indexerLangs && indexerLangs.length > 0;
+	const effectiveLangs = useIndexer ? indexerLanguagesToEnum(indexerLangs) : parsed.languages;
+
 	return {
-		source: sourceNames[parsed.source] || 'Unknown',
+		source: sourceDisplayNames[parsed.source] || 'Unknown',
 		resolution: resolutionNames[parsed.resolution] || 'Unknown',
-		modifier: modifierNames[parsed.modifier] || 'None',
-		languages: parsed.languages.map((l) => languageNames[l] || 'Unknown'),
+		modifier: modifierDisplayNames[parsed.modifier] || 'None',
+		languages: effectiveLangs.map((l) => languageNames[l] || 'Unknown'),
+		languageSource: useIndexer ? 'Indexer' : 'Title',
 		releaseGroup: parsed.releaseGroup,
 		year: parsed.year,
 		edition: parsed.edition,
@@ -181,14 +225,15 @@ function evaluateCondition(
 	condition: ConditionData,
 	parsed: ParseResult,
 	title: string,
-	patternMatches?: Map<string, boolean>
+	patternMatches?: Map<string, boolean>,
+	indexerLangs?: Language[]
 ): ConditionEvalResult {
 	switch (condition.type) {
 		case 'release_title':
 			return evaluatePattern(condition, title, patternMatches);
 
 		case 'language':
-			return evaluateLanguage(condition, parsed);
+			return evaluateLanguage(condition, parsed, indexerLangs);
 
 		case 'source':
 			return evaluateSource(condition, parsed);
@@ -262,13 +307,21 @@ function evaluatePattern(
 /**
  * Evaluate language condition
  */
-function evaluateLanguage(condition: ConditionData, parsed: ParseResult): ConditionEvalResult {
+function evaluateLanguage(
+	condition: ConditionData,
+	parsed: ParseResult,
+	indexerLangs?: Language[]
+): ConditionEvalResult {
 	if (!condition.languages || condition.languages.length === 0) {
 		return { matched: false, expected: 'No languages defined', actual: 'N/A' };
 	}
 
-	const parsedLangNames = parsed.languages.map((l) => languageNames[l] || 'Unknown');
-	const actual = parsedLangNames.length > 0 ? parsedLangNames.join(', ') : 'None detected';
+	// Prefer indexer languages when available
+	const effectiveLangs =
+		indexerLangs && indexerLangs.length > 0 ? indexerLangs : parsed.languages;
+
+	const langNames = effectiveLangs.map((l) => languageNames[l] || 'Unknown');
+	const actual = langNames.length > 0 ? langNames.join(', ') : 'None detected';
 
 	const expectedParts: string[] = [];
 	for (const lang of condition.languages) {
@@ -284,7 +337,7 @@ function evaluateLanguage(condition: ConditionData, parsed: ParseResult): Condit
 		const langEnum = Language[lang.name as keyof typeof Language];
 		if (langEnum === undefined) continue;
 
-		const hasLanguage = parsed.languages.includes(langEnum);
+		const hasLanguage = effectiveLangs.includes(langEnum);
 
 		if (lang.except) {
 			if (hasLanguage) return { matched: false, expected, actual };
@@ -477,12 +530,19 @@ export function evaluateCustomFormat(
 	conditions: ConditionData[],
 	parsed: ParseResult,
 	title: string,
-	patternMatches?: Map<string, boolean>
+	patternMatches?: Map<string, boolean>,
+	indexerLangNames?: string[]
 ): EvaluationResult {
+	// Convert indexer language names to enum values once
+	const indexerLangs =
+		indexerLangNames && indexerLangNames.length > 0
+			? indexerLanguagesToEnum(indexerLangNames)
+			: undefined;
+
 	const results: ConditionResult[] = [];
 
 	for (const condition of conditions) {
-		const evalResult = evaluateCondition(condition, parsed, title, patternMatches);
+		const evalResult = evaluateCondition(condition, parsed, title, patternMatches, indexerLangs);
 		const passes = condition.negate ? !evalResult.matched : evalResult.matched;
 
 		results.push({

src/lib/shared/pcd/display.ts

@@ -193,6 +193,7 @@ export interface ParsedInfo {
 	resolution: string;
 	modifier: string;
 	languages: string[];
+	languageSource: 'Indexer' | 'Title';
 	releaseGroup: string | null;
 	year: number;
 	edition: string | null;

src/routes/api/v1/entity-testing/evaluate/+server.ts

@@ -58,7 +58,7 @@ export const POST: RequestHandler = async ({ request }) => {
 			return {
 				releaseId: release.id,
 				title: release.title,
-				parsed: parsed ? getParsedInfo(parsed) : undefined,
+				parsed: parsed ? getParsedInfo(parsed, release.languages) : undefined,
 				cfMatches: {}
 			};
 		});
@@ -108,14 +108,14 @@ export const POST: RequestHandler = async ({ request }) => {
 				continue;
 			}
 
-			const result = evaluateCustomFormat(cf.conditions, parsed, release.title, patternMatches);
+			const result = evaluateCustomFormat(cf.conditions, parsed, release.title, patternMatches, release.languages);
 			cfMatches[cf.name] = result.matches;
 		}
 
 		return {
 			releaseId: release.id,
 			title: release.title,
-			parsed: getParsedInfo(parsed),
+			parsed: getParsedInfo(parsed, release.languages),
 			cfMatches
 		};
 	});

src/routes/quality-profiles/entity-testing/[databaseId]/+page.svelte

@@ -100,7 +100,8 @@
 					releases: entity.releases.map((r) => ({
 						id: r.id,
 						title: r.title,
-						type: entity.type
+						type: entity.type,
+						languages: r.languages.length > 0 ? r.languages : undefined
 					}))
 				})
 			});
@@ -359,15 +360,15 @@
 		<ActionButton
 			icon={Sliders}
 			hasDropdown={true}
-			dropdownPosition="middle"
+			dropdownPosition="right"
 			square={!selectedProfile}
 		>
 			{#if selectedProfile}
 				<span class="ml-2 text-sm text-neutral-700 dark:text-neutral-300"
 					>{selectedProfile.name}</span
 				>
 			{/if}
-			<Dropdown slot="dropdown" position="middle">
+			<Dropdown slot="dropdown" position="right">
 				<DropdownItem
 					label="No Profile"
 					selected={selectedProfileId === null}
@@ -382,8 +383,8 @@
 				{/each}
 			</Dropdown>
 		</ActionButton>
-		<ActionButton icon={Clapperboard} hasDropdown={true} dropdownPosition="middle">
-			<Dropdown slot="dropdown" position="middle">
+		<ActionButton icon={Clapperboard} hasDropdown={true} dropdownPosition="right">
+			<Dropdown slot="dropdown" position="right">
 				<DropdownItem
 					icon={Film}
 					label="Movies"