Skip to content

fix: allow webp as transformation option#1011

Merged
itslenny merged 3 commits intomasterfrom
fix/allow-webp-in-transformation-options
Apr 16, 2026
Merged

fix: allow webp as transformation option#1011
itslenny merged 3 commits intomasterfrom
fix/allow-webp-in-transformation-options

Conversation

@fenos
Copy link
Copy Markdown
Contributor

@fenos fenos commented Apr 12, 2026

What kind of change does this PR introduce?

Bug fix,

What is the current behavior?

  • strict validation doesn't allow WebP

What is the new behavior?

  • Allow WebP in transformation options

Closes #1010

Copilot AI review requested due to automatic review settings April 12, 2026 16:41
@fenos fenos requested a review from a team as a code owner April 12, 2026 16:41
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the HTTP transformation options schema to allow webp as a valid format value, addressing strict validation that previously rejected it.

Changes:

  • Extend format enum in transformationOptionsSchema to include webp.

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

Comment thread src/http/schemas/transformations.ts
Comment thread src/http/schemas/transformations.ts
@itslenny itslenny force-pushed the fix/allow-webp-in-transformation-options branch from 6718618 to 188cb39 Compare April 16, 2026 14:22
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 16, 2026

Coverage Report for CI Build 24516935782

Coverage increased (+0.01%) to 82.575%

Details

  • Coverage increased (+0.01%) from the base build.
  • Patch coverage: 2 of 2 lines across 2 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 32671
Covered Lines: 29395
Line Coverage: 89.97%
Relevant Branches: 7450
Covered Branches: 3735
Branch Coverage: 50.13%
Branches in Coverage %: Yes
Coverage Strength: 356.5 hits per line

💛 - Coveralls

@itslenny itslenny force-pushed the fix/allow-webp-in-transformation-options branch from 097b8e3 to 1f16f85 Compare April 16, 2026 14:48
@itslenny itslenny merged commit 473aa5c into master Apr 16, 2026
7 checks passed
@itslenny itslenny deleted the fix/allow-webp-in-transformation-options branch April 16, 2026 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

format=webp parameter rejected with 400 error after PR #921

4 participants