Harden audio file validation

Author: saptob-oai-7777

examples/audio/speaker_aware_meeting_intelligence/meeting_intelligence.py

@@ -281,8 +281,10 @@ def parse_known_speakers(entries: list[str]) -> list[tuple[str, Path]]:
         if not raw_path:
             raise ValueError(f"Known speaker reference path cannot be empty: {entry}")
         path = Path(raw_path).expanduser()
-        if not path.exists():
-            raise FileNotFoundError(f"Known speaker reference does not exist: {path}")
+        if not path.is_file():
+            raise FileNotFoundError(
+                f"Known speaker reference does not exist or is not a regular file: {path}"
+            )
         speakers.append((name, path))
     return speakers
 
@@ -301,8 +303,8 @@ def transcribe_with_diarization(
     known_speakers: list[tuple[str, Path]],
     model: str = DEFAULT_TRANSCRIPTION_MODEL,
 ) -> Any:
-    if not audio_file.exists():
-        raise FileNotFoundError(f"Audio file does not exist: {audio_file}")
+    if not audio_file.is_file():
+        raise FileNotFoundError(f"Audio file does not exist or is not a regular file: {audio_file}")
 
     from openai import OpenAI
 

examples/audio/speaker_aware_meeting_intelligence/test_meeting_intelligence.py

@@ -108,6 +108,20 @@ def test_parse_known_speakers_trims_path_whitespace(self) -> None:
 
             self.assertEqual(speakers, [("Agent", reference)])
 
+    def test_parse_known_speakers_rejects_directory_reference(self) -> None:
+        with tempfile.TemporaryDirectory() as tmpdir:
+            with self.assertRaises(FileNotFoundError) as error:
+                meeting_intelligence.parse_known_speakers([f"Agent={tmpdir}"])
+
+            self.assertIn("regular file", str(error.exception))
+
+    def test_transcribe_with_diarization_rejects_directory_audio(self) -> None:
+        with tempfile.TemporaryDirectory() as tmpdir:
+            with self.assertRaises(FileNotFoundError) as error:
+                meeting_intelligence.transcribe_with_diarization(Path(tmpdir), [])
+
+            self.assertIn("regular file", str(error.exception))
+
     def test_to_data_url_accepts_path_or_string(self) -> None:
         with tempfile.TemporaryDirectory() as tmpdir:
             reference = Path(tmpdir) / "speaker.wav"