fix(v5): Der Kern verwarf 'titel-lesen' STILL — Auswahl-Dialog hing ewig
Ampel / ampel (push) Successful in 1m48s

Commander: "das funktioniert nicht, du kannst gerne am lebenen objekt
pruefen". Der Dialog blieb bei "Rippy liest die Titel der Disc ..."
stehen, ohne Fehler, ohne Ende.

Ursache, am lebenden Objekt gemessen: NICHT der Info-Lauf.

  makemkvcon -r --noscan info dev:G:  ->  0,8 s, TCOUNT:0
  MSG:5042 "konnte keine verwendbaren optischen Laufwerke finden"

Der Lauf antwortet also sofort. Der Fehler lag in MEINEM Code von 5.2.0:
istFensterNachricht() kannte 'titel-lesen' und 'rip-ueberspringen' nicht
— beide standen in der FensterNachricht-Union, aber nicht im Pruefer.
Der Kern verwarf sie mit `if (!istFensterNachricht(frage)) return`.
STILL. Genau die R4-Wunde, vor der die Hausordnung warnt.

Auf drei Ebenen behoben:

1. Der Pruefer kennt beide Nachrichten.
2. Eine verworfene Nachricht ist jetzt LAUT: console.error plus
   kern-fehler ins Fenster mit der Art, die verworfen wurde.
3. Neuer Waechter-Test (waechter.test.ts): Er liest das Schema und
   vergleicht JEDE `art: '...'` der Unions mit dem jeweiligen Pruefer;
   dazu prueft er, dass der Kern nicht mehr still verwirft.
   GEGENPROBE gemacht: Mit zurueckgenommener Reparatur schlaegt er fehl
   ("expected [ 'titel-lesen' ] to deeply equal []"), mit Reparatur ist
   er gruen. Kein Deko-Test.

Dazu: Der Info-Lauf reicht die URSACHE durch (DiscInfo.ursachen aus den
kritischen MSG-Nummern). Kam nichts heraus und MakeMKV hat einen Grund
genannt, steht im Dialog der Grund statt "keine Titel gefunden" — beim
5042-Zustand also die Abhilfe (Disc neu einlegen, Laufwerk ab- und
anstecken, notfalls neu starten) statt eines Fingerzeigs auf die Disc.

Neu test/messung.titelwahl.test.ts — faehrt am echten Laufwerk genau den
Weg des Kerns nach (Info-Lauf, Vorauswahl, Dialog-Fehlertext).

Zweimal in dieselbe Heredoc-Falle getappt (AGENTS.md warnt davor):
Escape-Sequenzen wurden halbiert, ein rohes CR landete in makemkv.ts.
Beides korrigiert; der Hygiene-Waechter haette das CR auch gefangen.

260 Tests gruen (vorher 257), Typpruefung sauber. Version 5.3.1.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Hitonabi
2026-09-01 20:12:44 +02:00
co-authored by Claude Opus 5
parent 306b090ccf
commit 9622f4e6ef
8 changed files with 564 additions and 411 deletions
+17 -19
View File
@@ -1,24 +1,22 @@
Version 5.3.0 Version 5.3.1
Serien landen jetzt dort, wo Jellyfin sie sucht. Die Titel-Auswahl funktioniert jetzt wirklich.
Bisher kamen auch Folgen unter "Filme" — Jellyfin sah lauter Einzelfilme In 5.2.0 und 5.3.0 blieb der Dialog ewig bei "Rippy liest die Titel der
statt einer Staffel. Jetzt legt Rippy sie unter Serien/<Titel>/Season 01 Disc …" stehen. Der Grund war ein Fehler in Rippy selbst: Die Anfrage
ab. Die Episoden-Nummern (S01E02) setzt es NUR, wenn die Laufzeiten "lies die Titel" war dem Kern unbekannt und wurde weggeworfen — still.
eindeutig zur Staffel passen; sonst behalten die Dateien ihre Namen und Behoben, und zwar auf drei Ebenen: Die Anfrage ist jetzt bekannt, eine
Rippy sagt das. Lieber gar nicht als falsch. verworfene Nachricht sagt künftig laut warum, und ein neuer Prüf-Test
vergleicht bei jedem Bau die Nachrichten-Liste mit dem Prüfer. Derselbe
Fehler kann so nicht wiederkommen.
Bei Filmen wählst du jetzt auch, was mit soll. Rippy sagt jetzt, WARUM keine Titel kamen.
Nur das Hauptfeature, oder Hauptfeature plus Extras. In der Auswahl legst Findet MakeMKV das Laufwerk nicht (das passiert nach abgebrochenen
du je Titel fest, was er ist: Hauptfilm, Extra oder Folge. Extras landen Rips), stand dort bisher nur "keine Titel gefunden" — als läge es an der
im Unterordner "extras" neben dem Film — so lesen Jellyfin, Emby und Kodi Disc. Jetzt steht die echte Ursache da, samt Abhilfe: Disc auswerfen und
sie als Zugaben und nicht als zweiten Film. neu einlegen, Laufwerk ab- und anstecken, notfalls neu starten.
Neuer Knopf in der Auswahl: "Hauptinhalt + Extras". Aus 5.3.0: Serien landen unter Serien/<Titel>/Season NN, Filme bekommen
Hakt alles Sehenswerte an, aber weder Sammeltitel noch Logos und ihre Extras im Unterordner "extras".
Alterskennzeichen. Aus 5.2.0: Auswahl, was gerippt wird — plus der Defekt-Waechter.
Aus 5.2.0: Titel-Auswahl vor dem Rip und der Defekt-Waechter.
Aus 5.1.5: Windows-Rand weg, Erst-Einrichtung ueberarbeitet.
Aus 5.1.4: Das Update installiert sich wirklich.
Unveraendert: Updates warten immer, bis kein Rip mehr laeuft. Unveraendert: Updates warten immer, bis kein Rip mehr laeuft.
+2 -2
View File
@@ -1,12 +1,12 @@
{ {
"name": "rippy", "name": "rippy",
"version": "5.3.0", "version": "5.3.1",
"lockfileVersion": 3, "lockfileVersion": 3,
"requires": true, "requires": true,
"packages": { "packages": {
"": { "": {
"name": "rippy", "name": "rippy",
"version": "5.3.0", "version": "5.3.1",
"license": "UNLICENSED", "license": "UNLICENSED",
"dependencies": { "dependencies": {
"electron-updater": "^6.8.9", "electron-updater": "^6.8.9",
+1 -1
View File
@@ -1,7 +1,7 @@
{ {
"name": "rippy", "name": "rippy",
"productName": "Rippy", "productName": "Rippy",
"version": "5.3.0", "version": "5.3.1",
"description": "Rippy v5 — Disc-Ripping als eigenständiges Windows-Programm (KONZEPT-WINDOWS.md)", "description": "Rippy v5 — Disc-Ripping als eigenständiges Windows-Programm (KONZEPT-WINDOWS.md)",
"main": "out/haupt/index.js", "main": "out/haupt/index.js",
"author": "KrBrZ", "author": "KrBrZ",
+7 -1
View File
@@ -376,7 +376,13 @@ export function istDbStatus(wert: unknown): wert is DbStatus {
export function istFensterNachricht(wert: unknown): wert is FensterNachricht { export function istFensterNachricht(wert: unknown): wert is FensterNachricht {
if (!istObjekt(wert)) return false if (!istObjekt(wert)) return false
if (wert.art === 'ping') return typeof wert.zeit === 'number' if (wert.art === 'ping') return typeof wert.zeit === 'number'
if (wert.art === 'auswerfen' || wert.art === 'rip-start' || wert.art === 'rip-abbruch') { if (
wert.art === 'auswerfen' ||
wert.art === 'titel-lesen' ||
wert.art === 'rip-start' ||
wert.art === 'rip-abbruch' ||
wert.art === 'rip-ueberspringen'
) {
return typeof wert.id === 'string' && wert.id.length > 0 return typeof wert.id === 'string' && wert.id.length > 0
} }
if (wert.art === 'einstellung-setzen') { if (wert.art === 'einstellung-setzen') {
+27 -2
View File
@@ -313,9 +313,22 @@ async function titelLesen(id: string): Promise<void> {
istSerie: merkmale?.istSerie ?? false, istSerie: merkmale?.istSerie ?? false,
folgen: merkmale?.folgen ?? 0, folgen: merkmale?.folgen ?? 0,
}) })
// Kam nichts heraus UND MakeMKV hat einen Grund genannt, dann ist der
// Grund die Nachricht — nicht „keine Titel gefunden". Am 01.09.2026
// gemessen: Beim 5042-Zustand antwortet der Info-Lauf nach 0,7 s mit
// TCOUNT:0, und ohne diesen Zweig stünde im Fenster, die DISC gäbe
// nichts her — dabei sieht MakeMKV das LAUFWERK nicht.
const ohneErgebnis = info.titel.length === 0 && info.ursachen.length > 0
anFenster({ anFenster({
art: 'disc-titel', art: 'disc-titel',
stand: { id, laeuft: false, titel: wahl.titel, einordnung: wahl.einordnung, unsicher: wahl.unsicher, fehler: '' }, stand: {
id,
laeuft: false,
titel: wahl.titel,
einordnung: wahl.einordnung,
unsicher: wahl.unsicher,
fehler: ohneErgebnis ? info.ursachen.join(' · ') : '',
},
}) })
} catch (fehler) { } catch (fehler) {
// R4: laut scheitern. „Konnte nicht nachsehen" ist etwas anderes als // R4: laut scheitern. „Konnte nicht nachsehen" ist etwas anderes als
@@ -640,7 +653,19 @@ process.parentPort.on('message', (ereignis) => {
if (fensterPort === null) return if (fensterPort === null) return
fensterPort.on('message', (portEreignis) => { fensterPort.on('message', (portEreignis) => {
const frage: unknown = portEreignis.data const frage: unknown = portEreignis.data
if (!istFensterNachricht(frage)) return if (!istFensterNachricht(frage)) {
// R4: Eine verworfene Nachricht verschwindet NICHT still. Genau
// das kostete am 01.09.2026 eine Runde — 'titel-lesen' fehlte im
// Prüfer, der Kern warf sie weg, und im Fenster stand ewig
// „Rippy liest die Titel …", ohne dass irgendwo etwas stand.
const art =
typeof frage === 'object' && frage !== null
? String((frage as Record<string, unknown>).art)
: typeof frage
console.error(`[kern] Fenster-Nachricht verworfen (unbekannt): ${art}`)
anFenster({ art: 'kern-fehler', text: `Nachricht „${art}" wurde verworfen — sie ist dem Kern unbekannt.` })
return
}
if (frage.art === 'ping') { if (frage.art === 'ping') {
anFenster({ art: 'pong', zeit: frage.zeit }) anFenster({ art: 'pong', zeit: frage.zeit })
} else if (frage.art === 'auswerfen') { } else if (frage.art === 'auswerfen') {
+19 -1
View File
@@ -133,6 +133,15 @@ export class ZeilenLeser {
} }
export interface DiscInfo { export interface DiscInfo {
/** Was MakeMKV als URSACHE genannt hat, wenn nichts herauskam — die
* kritischen MSG-Nummern im Klartext. Leer, wenn alles glatt lief.
*
* Warum (01.09.2026 am Laufwerk gemessen): Beim Zustand 5042 („Das
* Programm konnte keine verwendbaren optischen Laufwerke finden")
* antwortet der Info-Lauf nach 0,7 s mit TCOUNT:0. Ohne diese Zeile
* stünde im Fenster nur „keine Titel gefunden" — und niemand wüsste,
* dass das Laufwerk gemeint ist und nicht die Disc. */
ursachen: string[]
titel: TitelInfo[] titel: TitelInfo[]
streams: Map<number, TitelStreams> streams: Map<number, TitelStreams>
sprachen: { audio: SprachEintrag[]; untertitel: SprachEintrag[] } sprachen: { audio: SprachEintrag[]; untertitel: SprachEintrag[] }
@@ -166,7 +175,16 @@ export function titelInfoLesen(
const ausgabe = textVon(Buffer.concat(stuecke)) const ausgabe = textVon(Buffer.concat(stuecke))
const titel = parseTitelInfo(ausgabe) const titel = parseTitelInfo(ausgabe)
const streams = parseStreamInfo(ausgabe) const streams = parseStreamInfo(ausgabe)
fertig({ titel, streams, sprachen: sprachenZusammenfassen(streams) }) // Die Ursache steht in den MSG-Nummern, nicht in der letzten Zeile.
const ursachen: string[] = []
for (const zeile of ausgabe.split('\n')) {
const meldung = parseMsg(zeile)
if (meldung === null) continue
if (meldung.code in KRITISCHE_CODES && !ursachen.includes(KRITISCHE_CODES[meldung.code])) {
ursachen.push(KRITISCHE_CODES[meldung.code])
}
}
fertig({ ursachen, titel, streams, sprachen: sprachenZusammenfassen(streams) })
}) })
}) })
} }
@@ -0,0 +1,66 @@
// MESSUNG am ECHTEN Laufwerk: Was bekäme der Auswahl-Dialog zu sehen?
//
// RIPPY_MESSUNG_DISC=G npx vitest run test/messung.titelwahl.test.ts --reporter=verbose
//
// Der Anlass (01.09.2026): Der Dialog blieb ewig bei „Rippy liest die
// Titel der Disc …". Ursache war NICHT der Info-Lauf, sondern der
// Nachrichten-Prüfer: `istFensterNachricht` kannte 'titel-lesen' nicht,
// der Kern verwarf die Anfrage — still. Gemessen wurde danach:
//
// makemkvcon -r --noscan info dev:G: -> 0,7 s, TCOUNT:0
// MSG:5042 „konnte keine verwendbaren optischen Laufwerke finden"
// MSG:2024 „Unknown device - 'G:'" · MSG:5010 „Öffnen der Disk schlug fehl"
//
// Der Info-Lauf HING also nie. Diese Messung fährt genau den Weg des
// Kerns nach — Info-Lauf, Vorauswahl, und was als Fehlertext im Dialog
// stünde. Sie läuft nur auf Verlangen und nur unter Windows.
import { describe, expect, it } from 'vitest'
const buchstabe = (process.env.RIPPY_MESSUNG_DISC ?? '').replace(/:$/, '').toUpperCase()
const messen = /^[A-Z]$/.test(buchstabe) && process.platform === 'win32'
describe.skipIf(!messen)('Titelwahl am echten Laufwerk', () => {
it('liest die Titel und baut die Vorauswahl', { timeout: 320_000 }, async () => {
const { finden } = await import('../src/kern/werkzeuge/katalog')
const { titelInfoLesen } = await import('../src/kern/rip/makemkv')
const { titelVorauswahl } = await import('../src/kern/rip/titelwahl')
const { discTitel } = await import('../src/kern/metadaten/discmerkmale')
const programm = finden('makemkv', '')
expect(programm.length, 'makemkvcon nicht gefunden').toBeGreaterThan(0)
const geraetepfad = `\\\\.\\${buchstabe}:`
const merkmale = discTitel(geraetepfad)
console.log(`MESSUNG Disc sagt: "${merkmale.titel}" · Folgen ${merkmale.folgen} · Serie ${merkmale.istSerie}`)
const start = Date.now()
const info = await titelInfoLesen(programm, geraetepfad)
console.log(`MESSUNG Info-Lauf: ${((Date.now() - start) / 1000).toFixed(1)} s, ${info.titel.length} Titel`)
if (info.ursachen.length > 0) console.log(`MESSUNG Ursachen: ${info.ursachen.join(' · ')}`)
const wahl = titelVorauswahl(info.titel, {
istSerie: merkmale.istSerie,
folgen: merkmale.folgen,
})
console.log(`MESSUNG Einordnung: ${wahl.einordnung} (unsicher: ${wahl.unsicher})`)
for (const t of wahl.titel) {
const min = Math.floor(t.dauerS / 60)
console.log(
`MESSUNG ${t.gewaehlt ? '[x]' : '[ ]'} Titel ${t.nr} | ${min}:${String(t.dauerS % 60).padStart(2, '0')} min` +
` | ${(t.groesseBytes / 1e9).toFixed(2)} GB | ${t.kapitel} Kap. | ${t.rolle} | ${t.grund}`,
)
}
// Was im Dialog als Fehler stünde — genau die Regel aus kern/index.ts.
const dialogFehler = info.titel.length === 0 && info.ursachen.length > 0 ? info.ursachen.join(' · ') : ''
console.log(`MESSUNG Dialog-Fehlertext: ${dialogFehler.length > 0 ? dialogFehler : '(keiner)'}`)
// Der Lauf muss ANTWORTEN — leer sein darf er, hängen nicht.
expect(Date.now() - start).toBeLessThan(300_000)
})
})
describe.skipIf(messen)('Titelwahl-Messung übersprungen', () => {
it('läuft nur mit RIPPY_MESSUNG_DISC=<Laufwerksbuchstabe> unter Windows', () => {
expect(true).toBe(true)
})
})
+40
View File
@@ -105,6 +105,46 @@ describe('Quelltext-Hygiene — keine rohen Steuerbytes', () => {
}) })
}) })
describe('Nachrichten-Schema — der Prüfer kennt JEDE Nachrichtenart', () => {
// Der teure Fund vom 01.09.2026: 'titel-lesen' und 'rip-ueberspringen'
// standen in der FensterNachricht-Union, aber NICHT in
// istFensterNachricht(). Der Kern verwarf sie — still. Im Fenster stand
// ewig „Rippy liest die Titel …", und nirgends stand ein Grund.
//
// Dieser Test liest den Quelltext: Jede `art: '…'` der Union muss im
// Prüfer wörtlich vorkommen. Eine neue Nachricht ohne Prüfung fällt
// damit sofort auf, statt erst am lebenden Objekt.
const schema = readFileSync(join(WURZEL, 'src', 'gemeinsam', 'nachrichten.ts'), 'utf8')
function artenAus(unionName: string): string[] {
const start = schema.indexOf(`export type ${unionName} =`)
expect(start, `${unionName} nicht gefunden`).toBeGreaterThan(-1)
// Der Union-Block reicht bis zur nächsten Deklaration auf oberster Ebene.
const rest = schema.slice(start + `export type ${unionName} =`.length)
const ende = rest.indexOf('\nexport ')
const block = ende > 0 ? rest.slice(0, ende) : rest
return [...new Set([...block.matchAll(/art:\s*'([a-z0-9-]+)'/g)].map((m) => m[1]))]
}
it('istFensterNachricht kennt jede FensterNachricht', () => {
const pruefer = schema.slice(schema.indexOf('export function istFensterNachricht'))
const fehlend = artenAus('FensterNachricht').filter((art) => !pruefer.includes(`'${art}'`))
expect(fehlend).toEqual([])
})
it('istKernNachricht kennt jede KernNachricht', () => {
const pruefer = schema.slice(schema.indexOf('export function istKernNachricht'))
const fehlend = artenAus('KernNachricht').filter((art) => !pruefer.includes(`'${art}'`))
expect(fehlend).toEqual([])
})
it('der Kern verwirft nichts still', () => {
// Wer eine Nachricht wegwirft, sagt WARUM (R4).
const kern = readFileSync(join(WURZEL, 'src', 'kern', 'index.ts'), 'utf8')
expect(kern).not.toMatch(/if\s*\(!istFensterNachricht\([^)]*\)\)\s*return/)
})
})
describe('§ 6.7 — EINE Quelle für Einstellungen', () => { describe('§ 6.7 — EINE Quelle für Einstellungen', () => {
it('node:sqlite wird nur in kern/speicher/db.ts angefasst', () => { it('node:sqlite wird nur in kern/speicher/db.ts angefasst', () => {
// Der rc11-Fund: Oberfläche schrieb in die Datenbank, der Betrieb las // Der rc11-Fund: Oberfläche schrieb in die Datenbank, der Betrieb las