fix: align base64 io error handling in message components

Author: zouyonghe

astrbot/core/message/components.py

@@ -203,7 +203,7 @@ async def convert_to_base64(self) -> str:
         else:
             try:
                 bs64_data = await file_to_base64(self.file)
-            except (FileNotFoundError, IsADirectoryError) as exc:
+            except OSError as exc:
                 raise Exception(f"not a valid file: {self.file}") from exc
         bs64_data = bs64_data.removeprefix("base64://")
         return bs64_data
@@ -500,7 +500,7 @@ async def convert_to_base64(self) -> str:
         else:
             try:
                 bs64_data = await file_to_base64(url)
-            except (FileNotFoundError, IsADirectoryError) as exc:
+            except OSError as exc:
                 raise Exception(f"not a valid file: {url}") from exc
         bs64_data = bs64_data.removeprefix("base64://")
         return bs64_data

tests/unit/test_message_components_paths.py

@@ -3,6 +3,7 @@
 
 import pytest
 
+from astrbot.core.message import components as components_module
 from astrbot.core.message.components import File, Image, Record
 
 
@@ -77,3 +78,27 @@ async def test_record_convert_to_base64_reads_existing_local_file(tmp_path):
     encoded = await record.convert_to_base64()
 
     assert base64.b64decode(encoded) == raw
+
+
+@pytest.mark.asyncio
+async def test_image_convert_to_base64_maps_permission_error(monkeypatch):
+    async def _raise_permission_error(_path: str) -> str:
+        raise PermissionError("permission denied")
+
+    monkeypatch.setattr(components_module, "file_to_base64", _raise_permission_error)
+
+    image = Image(file="/tmp/forbidden-image")
+    with pytest.raises(Exception, match="not a valid file"):
+        await image.convert_to_base64()
+
+
+@pytest.mark.asyncio
+async def test_record_convert_to_base64_maps_permission_error(monkeypatch):
+    async def _raise_permission_error(_path: str) -> str:
+        raise PermissionError("permission denied")
+
+    monkeypatch.setattr(components_module, "file_to_base64", _raise_permission_error)
+
+    record = Record(file="/tmp/forbidden-record")
+    with pytest.raises(Exception, match="not a valid file"):
+        await record.convert_to_base64()