fix: refactor image handling to use `@netlify/images`
๐ Thanks for submitting a pull request! ๐
Summary
Refactors logic of the initializeImageProxy function to use @netlify/images instead of express function. Removed unit tests for the old logic as well.
Fixes #EX-536
For us to review and ship your PR efficiently, please perform the following steps:
- [ ] Open a bug/issue before writing your code ๐งโ๐ป. This ensures we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a typo or something that`s on fire ๐ฅ (e.g. incident related), you can skip this step.
- [ ] Read the contribution guidelines ๐. This ensures your code follows our style guide and passes our tests.
- [ ] Update or add tests (if any source code was changed or added) ๐งช
- [ ] Update or add documentation (if features were changed or added) ๐
- [ ] Make sure the status checks below are successful โ
A picture of a cute animal (not mandatory, but encouraged)
๐ Benchmark results
Comparing with 3539b0dc68802af88a032fb3925af24174c82a09
- Dependency count: 1,074 โฌ๏ธ 0.09% increase vs. 3539b0dc68802af88a032fb3925af24174c82a09
- Package size: 298 MB โฌ๏ธ 0.01% increase vs. 3539b0dc68802af88a032fb3925af24174c82a09
- Number of ts-expect-error directives: 381 (no change)
@copilot fix the merge conflicts and address the comment. also please let me know the extent of coverage we have in place for the image behaviour.
@serhalp I've opened a new pull request, #7802, to work on those changes. Once the pull request is ready, I'll request review from you.
@copilot merge main into the branch and resolve the conflicts
@serhalp I've opened a new pull request, #7803, to work on those changes. Once the pull request is ready, I'll request review from you.