janpeter/Media-Organizer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

PBI-29: FLV, RM en RMVB afspelen in de bestaande viewer #94

Merged
janpeter merged 5 commits from codex/flv-realmedia into main 2026-09-27 17:22:32 +02:00
Conversation 1 Commits 5 Files changed 18 +890 -13
janpeter commented 2026-09-27 17:09:11 +02:00
Owner
Copy link

Resultaat

Nieuw geïndexeerde FLV-, RM- en RMVB-video’s spelen via de bestaande viewer en begrensde FFmpeg/NVENC-afspeelkopieën. Originelen blijven behouden.

  • Voeg extensies en MIME-types toe aan scanner en media-endpoint.
  • Laat specifieke legacy container/codec-combinaties toe; deel het begrensde SAR-/fps-uitvoerprofiel tussen converter en validator.
  • Toon onbruikbare probegegevens als terminale melding zonder ineffectieve voorbereidknop.
  • Behoud bestaande profielversies, cachegrenzen en proces-/publicatiecontroles.

Verificatie

  • Node 22.23.2: volledige suite geserialiseerd, 994 geslaagd / 0 gefaald / 15 overgeslagen; typecheck en build groen.
  • Vijf echte codeccombinaties doorlopen scan, conversie, volledige uitvoerdecode en viewerbediening. Bronhashes blijven gelijk; heropenen gebruikt dezelfde cachekopieën.
  • Een echte onbruikbare RM-bron bevestigt de terminale melding.
  • Onafhankelijke Claude-review op de Mac: GO na reparatie en RED/GREEN-verificatie van één MAJOR. JP heeft geluid, lipsynchroniteit en beeldverhouding van alle vijf geaccepteerd.

De parallelle suite had twee botsingen tussen bestaande auth-tests met gedeelde pairing-data. De volledige selectie slaagt met --test-concurrency=1. De 15 skips betreffen aparte empty-trash-CI en Linux-tests. Automatische taak-planvergelijking heeft bij deze interactieve uitvoering geen frozen jobbaseline; handmatige plancontrole en onafhankelijke review zijn vastgelegd.

Bewijs en uitrol

Scrum4Me: PBI-29 / ST-056, T-204 t/m T-206 afgerond. Proef uitsluitend in een geïsoleerde omgeving. Latere uitrol vereist nieuwe webapp én worker vóór de gerichte herscan; deze PR voert geen productie-uitrol uit.

## Resultaat Nieuw geïndexeerde FLV-, RM- en RMVB-video’s spelen via de bestaande viewer en begrensde FFmpeg/NVENC-afspeelkopieën. Originelen blijven behouden. - Voeg extensies en MIME-types toe aan scanner en media-endpoint. - Laat specifieke legacy container/codec-combinaties toe; deel het begrensde SAR-/fps-uitvoerprofiel tussen converter en validator. - Toon onbruikbare probegegevens als terminale melding zonder ineffectieve voorbereidknop. - Behoud bestaande profielversies, cachegrenzen en proces-/publicatiecontroles. ## Verificatie - Node 22.23.2: volledige suite geserialiseerd, **994 geslaagd / 0 gefaald / 15 overgeslagen**; typecheck en build groen. - Vijf echte codeccombinaties doorlopen scan, conversie, volledige uitvoerdecode en viewerbediening. Bronhashes blijven gelijk; heropenen gebruikt dezelfde cachekopieën. - Een echte onbruikbare RM-bron bevestigt de terminale melding. - Onafhankelijke Claude-review op de Mac: **GO** na reparatie en RED/GREEN-verificatie van één MAJOR. JP heeft geluid, lipsynchroniteit en beeldverhouding van alle vijf geaccepteerd. De parallelle suite had twee botsingen tussen bestaande auth-tests met gedeelde pairing-data. De volledige selectie slaagt met `--test-concurrency=1`. De 15 skips betreffen aparte empty-trash-CI en Linux-tests. Automatische taak-planvergelijking heeft bij deze interactieve uitvoering geen frozen jobbaseline; handmatige plancontrole en onafhankelijke review zijn vastgelegd. ## Bewijs en uitrol - [Productverificatie](docs/research/2026-09-27-flv-rm-productverificatie.md) - [Reviewverslag](docs/reviews/2026-09-27-flv-rm-implementatiereview.md) - [Conversie- en tijdlijnbewijs](docs/research/2026-09-27-flv-rm-conversie-verificatie.md) Scrum4Me: PBI-29 / ST-056, T-204 t/m T-206 afgerond. Proef uitsluitend in een geïsoleerde omgeving. Latere uitrol vereist nieuwe webapp én worker vóór de gerichte herscan; deze PR voert geen productie-uitrol uit.
docs: record accepted FLV and RealMedia product verification
Some checks failed
CI / test (historical-bootstrap) (pull_request) Successful in 1m9s
Details
CI / test (empty-trash) (pull_request) Successful in 1m49s
Details
CI / test (video-migration) (pull_request) Successful in 1m17s
Details
CI / test (suite) (pull_request) Failing after 2m46s
Details
CI / docker-build (pull_request) Successful in 2m30s
Details
1489a05ab5
s4m-codex-reviewer approved these changes 2026-09-27 17:11:29 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

Findings

  • Geen bevindingen met blocker/error-severity.

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

De wijziging blijft binnen de bestaande viewer-, indexeer- en conversiearchitectuur. De FLV/RM/RMVB-toelating is container- en codecgebonden, de uitvoerframerate wordt centraal door converter en validator gedeeld, en de terminale probefouten onderdrukken nu ook in de gemonteerde viewer de ineffectieve voorbereidingsactie. Gerichte regressietests dekken MIME/range, scanning, profiel/validator en viewerstatus; de toegevoegde verificatiedocumentatie beschrijft de kandidaat- en runtimecontroles.

# Verdict: APPROVED ## Findings - Geen bevindingen met blocker/error-severity. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. De wijziging blijft binnen de bestaande viewer-, indexeer- en conversiearchitectuur. De FLV/RM/RMVB-toelating is container- en codecgebonden, de uitvoerframerate wordt centraal door converter en validator gedeeld, en de terminale probefouten onderdrukken nu ook in de gemonteerde viewer de ineffectieve voorbereidingsactie. Gerichte regressietests dekken MIME/range, scanning, profiel/validator en viewerstatus; de toegevoegde verificatiedocumentatie beschrijft de kandidaat- en runtimecontroles.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/Media-Organizer!94
No description provided.