chore: add coverage for safe copy

[ovflowd]

Adds unit tests for the recently introduced safeCopy method from #480

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Updated (UTC)
api-docs-tooling	Ready	Preview	Nov 3, 2025 10:04pm

[codecov]

Codecov Report

❌ Patch coverage is 99.20635% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 74.40%. Comparing base (179ca04) to head (64767b0).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...tors/legacy-html/utils/__tests__/safeCopy.test.mjs	99.20%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #481      +/-   ##
==========================================
+ Coverage   73.86%   74.40%   +0.54%     
==========================================
  Files         109      110       +1     
  Lines       10356    10482     +126     
  Branches      686      704      +18     
==========================================
+ Hits         7649     7799     +150     
+ Misses       2704     2680      -24     
  Partials        3        3              

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

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull Request Overview

This PR adds comprehensive test coverage for the safeCopy utility function, which copies files from a source to a target directory while intelligently skipping files that haven't changed based on file size and modification time.

• Implements 7 test cases covering various scenarios including new file copying, multiple file handling, and conditional copying based on file stats
• Uses Node.js native test runner and filesystem promises for test setup/teardown
• Tests edge cases like empty directories and files with identical sizes but different content

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[avivkeller]

[avivkeller]

Note that there is a non-zero chance that we revert the change in favor of a different approach, per nodejs/node#57343 (comment), so let's hold off on merging this for the time being

(Not saying we should, just that it's on the table)