Skip to content

Conversation

@seer-by-sentry
Copy link
Contributor

Fixes WORKER-Q80. The issue was that: Celery task chain initialized with list [] instead of expected dictionary, causing previous_result.get() to fail.

  • Added defensive handling to check if previous_result is a dictionary.
  • If previous_result is not a dictionary, log a warning with relevant context and initialize processing_results to an empty list to prevent errors.

This fix was generated by Seer in Sentry, triggered by Max Weng. 👁️ Run ID: 1649849

Not quite right? Click here to continue debugging with Seer.

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@sentry
Copy link

sentry bot commented Sep 29, 2025

Codecov Report

❌ Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.81%. Comparing base (7ef1718) to head (2739fcb).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
apps/worker/tasks/bundle_analysis_processor.py 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #516      +/-   ##
==========================================
- Coverage   93.82%   93.81%   -0.01%     
==========================================
  Files        1279     1279              
  Lines       46084    46087       +3     
  Branches     1511     1511              
==========================================
+ Hits        43237    43238       +1     
- Misses       2539     2541       +2     
  Partials      308      308              
Flag Coverage Δ
workerintegration 58.80% <0.00%> (-0.02%) ⬇️
workerunit 91.13% <50.00%> (-0.02%) ⬇️

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

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

@codecov-notifications
Copy link

codecov-notifications bot commented Sep 29, 2025

Codecov Report

❌ Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
apps/worker/tasks/bundle_analysis_processor.py 50.00% 2 Missing ⚠️

❌ Your patch check has failed because the patch coverage (50.00%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

📢 Thoughts on this report? Let us know!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant