Fix/xsup 76577/rasterize extract lightweight freeze

[MosheEichler]

Status

• In Progress
• Ready
• In Hold - (Reason for hold)

Related Issues

Description

In lightweight mode the memory guard froze the tab after every page load, which disables JavaScript execution and purges the V8 heap. rasterize-extract is the only rasterize type that runs JS in the page (Runtime.evaluate), so the freeze destroyed the execution context it was about to use — the command failed with Cannot find default execution context and returned no content.

Change: added a freeze_on_load flag (default True) to wait_for_page_load_with_memory_guard and navigate_to_path, and pass False from extract_text_content.

• Screenshot/PDF captures are unchanged — they only need the last painted frame, which survives the freeze.
• The memory-pressure freeze is unaffected: under real pressure the tab is still frozen, since avoiding the OOM takes precedence.
• Non-lightweight tenants are unaffected (that path never freezes).

Must have

• Tests
• Documentation

[content-bot]

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

• @marketplace-ai-reviewer start review - Initiate a full AI code review
• @marketplace-ai-reviewer re-review - Incremental review for new commits

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
Packs/rasterize/Integrations/rasterize
rasterize.py	830	203	75%	109–111, 118–120, 126–128, 143–145, 149–151, 155–157, 161–163, 165, 289–290, 294, 594, 609–611, 614–616, 619, 622–624, 628–630, 636–637, 640, 653–654, 658–659, 663–664, 689–691, 744–749, 771–773, 788–790, 798–799, 803, 840, 853, 856, 865, 868, 1081–1083, 1097–1099, 1127–1128, 1143–1145, 1153–1154, 1159, 1178, 1197–1198, 1222, 1238, 1265–1266, 1325–1326, 1329–1332, 1334, 1463–1465, 1486–1492, 1499–1506, 1509–1512, 1902–1904, 1924–1926, 1937–1939, 1942, 1947–1953, 1955–1960, 1962–1965, 1967, 1989–1990, 2097, 2109–2110, 2150, 2154–2155, 2172–2175, 2179–2180, 2183–2184, 2188–2194, 2196–2199, 2201–2204, 2206–2207, 2210–2211, 2213–2214, 2217–2218, 2220–2226, 2228–2231, 2420–2423
TOTAL	830	203	75%

Tests	Skipped	Failures	Errors	Time
116	0 💤	0 ❌	0 🔥	59.263s ⏱️

[marketplace-ai-reviewer]

🤖 Analysis started. Please wait for results...

[marketplace-ai-reviewer]

🤖 AI Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

[marketplace-ai-reviewer]

Hi there! Thanks for your contribution to the rasterize pack. The changes look great, but there is just a minor formatting tweak needed in the release notes to align with our entity styling guidelines. Great work on fixing the content extraction issue!

@MosheEichler please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

[content-bot]

Validate summary
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

Verdict: PR can be force merged from validate perspective? ✅

[content-bot]

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: 7d5de5da537f7929

📋 Triage Report
💡 Resolutions are available in the full report.

⚠️ AI-generated triage. Validate before acting.