Skip to content

feat: forward crawler status messages to the run status message - #1192

Open
vdusek wants to merge 3 commits into
masterfrom
feat/forward-crawler-status
Open

vdusek wants to merge 3 commits into
masterfrom
feat/forward-crawler-status

Conversation

@vdusek

@vdusek vdusek commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

On the Apify platform, the Actor forwards Crawlee crawler status messages (Event.CRAWLER_STATUS) to the run status message, so crawler progress shows up in Console, as it does with the JS SDK. The listener is removed before the terminal status message is set on exit, and messages that arrive during exit are dropped, so they can't overwrite it. A failed update logs a warning.

Closes: #1188

✍️ Drafted by Claude Code

@vdusek vdusek added the t-tooling Issues with this label are in the ownership of the tooling team. label Oct 16, 2026
@vdusek vdusek self-assigned this Oct 16, 2026
@codecov

codecov Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.08%. Comparing base (290af90) to head (f16a0c6).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1192      +/-   ##
==========================================
+ Coverage   93.00%   93.08%   +0.08%     
==========================================
  Files          54       54              
  Lines        3672     3689      +17     
==========================================
+ Hits         3415     3434      +19     
+ Misses        257      255       -2     
Flag Coverage Δ
e2e 31.71% <31.57%> (-0.02%) ⬇️
integration 51.66% <47.36%> (-0.03%) ⬇️
unit 89.83% <100.00%> (+0.10%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vdusek
vdusek marked this pull request as ready for review October 16, 2026 11:03
@apify-service-account apify-service-account added the tested Temporary label used only programatically for some analytics. label Oct 16, 2026
@vdusek
vdusek requested review from Mantisus and janbuchar October 16, 2026 11:06
@vdusek

vdusek commented Oct 16, 2026

Copy link
Copy Markdown
Contributor Author

@janbuchar Feature parity with the JS SDK. According to Claude it should be fine, even though you guys have reworked this in Crawlee JS v4. Once that rework is ported to Crawlee for Python, we can update this the same way.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Forward crawler status messages to the run status message

2 participants