Skip to content

Lowercase the extension in IsMedia - #490

Open
KillerX wants to merge 2 commits into
fix/07-isdirempty-eoffrom
fix/08-media-extension-case
Open

Lowercase the extension in IsMedia#490
KillerX wants to merge 2 commits into
fix/07-isdirempty-eoffrom
fix/08-media-extension-case

Conversation

@KillerX

@KillerX KillerX commented Aug 21, 2026

Copy link
Copy Markdown
Member

ValidRawFilename lowercases but IsMedia did not, so .MOV/.MXF files were ingested yet skipped media analysis.

Part of the stacked bugfix series fix/00 → fix/20; based on fix/07-isdirempty-eof.

🤖 Generated with Claude Code

@KillerX
KillerX force-pushed the fix/08-media-extension-case branch from 611053c to 60e3a53 Compare August 21, 2026 10:59
KillerX and others added 2 commits August 21, 2026 13:08
ValidRawFilename compares extensions case-insensitively but IsMedia did
not, so files like .MOV/.MXF passed raw-filename validation yet skipped
the media analysis branch in the raw material ingest workflow.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@KillerX
KillerX force-pushed the fix/08-media-extension-case branch from 60e3a53 to 7e74bd4 Compare August 21, 2026 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant