[mvolfik]

fix for breaking change from apify/apify-core#32513

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.76%. Comparing base (eb8ded4) to head (b2df23f).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1220   +/-   ##
=======================================
  Coverage   95.76%   95.76%           
=======================================
  Files          63       63           
  Lines        6676     6676           
=======================================
  Hits         6393     6393           
  Misses        283      283           

Flag	Coverage Δ
integration	91.05% <ø> (+0.38%)	⬆️
unit	91.07% <ø> (ø)

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.

[apify-factory]

Looks good, nothing blocking.

🏭 Factory review — ℹ️ not auto-approved

This change was reviewed but cannot be auto-approved: a gate failed, so the review cannot end in an approval.

check	result
Static gates	❌ 15/16 — no-risky-content
Review findings	✅ none
Reviewer escalation	✅ none
Juror 1 — claude-sonnet-5-5, adversarial	⏭️ skipped — a gate failed

Why it was not auto-approved

Failed gates:

• no-risky-content: tests/_utils.py: environment variable access (env-access); tests/integration/test_run.py: environment variable access (env-access)

The comments above are the review itself; this section only explains the approval decision. Push a change and request @apify-factory again to re-run it, or ask a human to review.

_{Reviewed b2df23f71 · workflow run}

[vdusek]

Thanks @mvolfik. Did you open the PR manually or via agent?

I renamed it to the "test" type, since the change touches only tests, not the source code (we don't want this in changelog).

I'm asking because our CLAUDE.md links to the commit conventions, so I'm curios whether your agent ignores it for some reason or not, because you opened it manually (if you wrote the title yourself, then it's ok).

[mvolfik]

nope, it was done manually :) fair point, in apify-core, the commit conventions are a bit of a wild west (mostly because we don't have to care about semver that much), thanks for fixing it