fix(cli): honor --render-js for .html extract output

[dyk1454683243-sudo]

Fixes #188

Problem

pyscrappy extract URL out.html --render-js accepted the flag, exited 0, and still fetched via GenericScraper.http.get_html() — the un-rendered HTTP response. Other extensions already pass render_js into scrape().

Fix

In the ext == "html" branch of run_extract:

• render_js=True uses the scraper's browser-aware fetch_html(url, render_js=True)
• render_js=False still uses http.get_html (existing CLI test pin)
• --css-selector with .html now emits a UserWarning instead of being silently ignored

Tests

• run_extract(..., out.html, render_js=True) calls fetch_html, not http.get_html
• run_extract(..., out.html, render_js=False) still calls http.get_html
• --css-selector with .html warns

Locally:

pytest tests/test_cli/test_extract.py -v
# 8 passed
ruff check src/pyscrappy/cli.py tests/test_cli/test_extract.py
ruff format --check src/pyscrappy/cli.py tests/test_cli/test_extract.py

[Copilot]

Copilot review overview

🟢 Approval recommended

The implementation matches the stated behavior and has focused regression coverage.

Review effort: Lite
Findings: None

What changed in this PR

Fixes .html extraction so --render-js uses browser-rendered markup and warns when --css-selector is ignored.

Changes:

• Routes rendered HTML through fetch_html(..., render_js=True).
• Preserves plain HTTP fetching when rendering is disabled.
• Adds a warning and tests for unsupported CSS selectors on HTML output.

File	Description
src/​pyscrappy/​cli.py	Updates HTML extraction behavior and warning handling.
tests/​test_cli/​test_extract.py	Covers rendered, non-rendered, and selector-warning cases.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[vedaant00]

Nice work @dyk1454683243-sudo, this is exactly the fix. Verified locally: render_js=True takes the browser path, render_js=False is unchanged, and the css-selector warning fires. Both new tests fail on the pre-fix code and pass with it, with the existing tests untouched. 600 passed, ruff clean. Good call strengthening the existing test with fetch_html.assert_not_called() rather than loosening it. Merging, thanks!