fix(transcription): accept every filename extension a decodable type declares (#868)

[yzxcj797]

Closes #868.

Root cause

supportedFileExtensions kept only each UTType's preferredFilenameExtension. The Ogg audio type (org.xiph.ogg-audio) declares the extensions ["ogg", "oga", "opus"] with ogg preferred — so .opus (WhatsApp voice notes arrive as Ogg Opus) and .oga were rejected by both the drop zone and the file picker, even though macOS decodes them natively (AVAudioFile/AVAsset read them fine; renaming the same file to .ogg made it work).

Fix

The set is now built from all of a type's filename-extension tags (utType.tags.filter { $0.isFilenameExtension }), plus the preferred one as a safety net for types where it is occasionally absent from tags. The defining property is preserved: everything accepted is something AVURLAsset.audiovisualTypes() says AVFoundation can decode — the set just no longer loses decodable extensions to the preferred-only projection.

Tests

Four in Tests/FluidDictationIntegrationTests/SupportedFileExtensionsTests.swift (XCTest, matching the existing integration-test target):

• the Ogg family (ogg, opus, oga) is accepted;
• well-known formats (wav, mp3, m4a, mp4, mov) stay accepted;
• the service's set exactly equals the tag-derived set recomputed the same way — pins the construction so it cannot silently drift back to preferred-only;
• non-media extensions (txt, pdf) stay rejected.

(No Swift toolchain on this Windows machine, so the compile/test gate is CI; the change uses only UTType.tags/isFilenameExtension, both public UniformTypeIdentifiers API on macOS 11+.)

[github-actions]

The PR Policy check is blocking this PR because required template information is missing.

Please update the PR description with:

• Description
• Type of Change
• Related Issue or Discussion
• Testing
• Screenshots / Video

Screenshots or video are required for UI, UX, settings, onboarding, overlay, menu bar, or visual behavior changes. If this PR has no visual changes, check the no-visual-change box in the template.

If this remains incomplete for 48 hours after opening, the PR may be closed.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d61092c438

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[chatgpt-codex-connector]

Add the test file to the Xcode test target

This directory is represented by an explicit PBXGroup, not a filesystem-synchronized group, and Fluid.xcodeproj/project.pbxproj contains no file reference, build-file entry, or sources-phase entry for SupportedFileExtensionsTests.swift. Consequently, the xcodebuild test workflow will never compile or execute this new suite, so the regression coverage added here is ineffective until the file is included in FluidDictationIntegrationTests target membership.

Useful? React with 👍 / 👎.

[yzxcj797]

Fixed and pushed: the directory is an explicit PBXGroup, so the file now has all four entries in project.pbxproj (file reference, build-file entry, group child, sources-phase entry), mirroring the AudioBufferConverterTests pattern. The xcodebuild test workflow will compile and run it.

[greptile-apps]

Greptile Summary

The PR expands meeting-transcription format validation to include every filename extension declared by each AVFoundation-decodable audio or movie type, fixing .opus and .oga selection.

• Collects all filename-extension tags while retaining the preferred extension as a fallback.
• Adds integration coverage for Ogg-family and common media extensions, exact set construction, and rejection of non-media extensions.
• Registers the new test file with the Xcode integration-test target.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

_{Reviews (2): Last reviewed commit: "Address review: register the new test fi..." | Re-trigger Greptile}