website-scraper / node-website-scraper / 28006450831

96%

push

github

Drop usage of fs-extra and switch to native fs (#608)

* Drop usage of fs-extra and switch to native fs

* Drop fs-extra usage in tests and remove dependency

Replace fs-extra with native fs in all test files:
- fs.removeSync(dir) -> fs.rmSync(dir, {recursive: true, force: true})
- fs.mkdirpSync(dir) -> fs.mkdirSync(dir, {recursive: true})
- fs.emptyDirSync(dir) -> fs.rmSync + fs.mkdirSync

Remove fs-extra from devDependencies as it is no longer used.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* Use async fs.promises.rm for error cleanup instead of sync rmSync

Keeps the error-action cleanup non-blocking, matching the original
awaited fs.remove behavior and the async outputFile write path. The
error action runs at most once (single catch in Scraper.scrape), so
the async call carries no concurrency risk.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

359 of 363 branches covered (98.9%)

17 of 17 new or added lines in 2 files covered. (100.0%)

1484 of 1539 relevant lines covered (96.43%)

88.66 hits per line