Review comments

Author: Pijukatel

src/crawlee/crawlers/_basic/_basic_crawler.py

@@ -355,8 +355,8 @@ def __init__(
             status_message_logging_interval: Interval for logging the crawler status messages.
             status_message_callback: Allows overriding the default status message. The default status message is
                 provided in the parameters. Returning `None` suppresses the status message.
-            id: Id of the crawler used for state tracking. You can use same explicit id to share state and between two
-                crawlers. By default, each crawler will use own state.
+            id: Identifier used for crawler state tracking. Use the same id across multiple crawlers to share state
+                between them.
             _context_pipeline: Enables extending the request lifecycle and modifying the crawling context.
                 Intended for use by subclasses rather than direct instantiation of `BasicCrawler`.
             _additional_context_managers: Additional context managers used throughout the crawler lifecycle.

tests/unit/crawlers/_basic/test_basic_crawler.py

@@ -1722,7 +1722,7 @@ async def test_add_requests_with_rq_param(queue_name: str | None, queue_alias: s
     crawler = BasicCrawler()
     rq = await RequestQueue.open(name=queue_name, alias=queue_alias)
     if by_id:
-        queue_id = rq._id
+        queue_id = rq.id
         queue_name = None
     else:
         queue_id = None
@@ -1816,6 +1816,7 @@ async def request_handler(context: BasicCrawlingContext) -> None:
         context.log.info(f'State {state}')
 
     crawler = BasicCrawler(
+        id=crawler_id,
         request_handler=request_handler,
         concurrency_settings=ConcurrencySettings(max_concurrency=1, desired_concurrency=1),
         configuration=Configuration(