Add warning for usage of FileSystemStorageClient in Actor context

Author: Pijukatel

src/apify/_actor.py

@@ -25,6 +25,7 @@
     EventPersistStateData,
     EventSystemInfoData,
 )
+from crawlee.storage_clients import FileSystemStorageClient
 
 from apify._charging import ChargeResult, ChargingManager, ChargingManagerImplementation
 from apify._configuration import Configuration
@@ -244,7 +245,13 @@ def _finalize_implicit_local_storage_client(self) -> StorageClient:
                 'Storage client in service locator was set explicitly before Actor.init was called.'
                 'Using the existing storage client as implicit storage client for the Actor.'
             )
+
         self._local_storage_client = service_locator.get_storage_client()
+        if type(self._local_storage_client) is FileSystemStorageClient:
+            self.log.warning(
+                'Using `FileSystemStorageClient` in Actor context is not recommended and can lead to '
+                'problems with reading the input file. Use `ApifyFileSystemStorageClient` instead.'
+            )
         return self._local_storage_client
 
     @property

tests/unit/actor/test_configuration.py

@@ -7,6 +7,7 @@
 from crawlee.configuration import Configuration as CrawleeConfiguration
 from crawlee.crawlers import BasicCrawler
 from crawlee.errors import ServiceConflictError
+from crawlee.storage_clients import FileSystemStorageClient
 
 from apify import Actor
 from apify import Configuration as ApifyConfiguration
@@ -215,3 +216,15 @@ async def test_storage_retrieved_is_same_with_same_config() -> None:
         crawler_kvs = await crawler.get_key_value_store()
 
     assert actor_kvs is crawler_kvs
+
+
+async def test_file_system_storage_client_warning(caplog: pytest.LogCaptureFixture) -> None:
+    service_locator.set_storage_client(FileSystemStorageClient())
+    caplog.set_level('WARNING')
+    async with Actor():
+        ...
+
+    assert (
+        'Using `FileSystemStorageClient` in Actor context is not recommended and can lead to problems with reading '
+        'the input file. Use `ApifyFileSystemStorageClient` instead.'
+    ) in caplog.messages