Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to confirm that I understand this correctly - and that we've thought it through 🙂.

We need to expose this to support cases where we want to implement a new with_<NAME>_static_parser method on AdaptivePlaywrightCrawler - to allow custom parsers other than BeautifulSoup and Parsel.

The generic StatisticsState is already exposed, but for the AdaptivePlaywrightCrawler, the method needs to use AdaptivePlaywrightCrawlerStatisticState instead. That makes sense.

In that case LGTM.