Skip to content

STREAM-6638 Update MediaTransformationResult methods to return Promises#6096

Merged
danlapid merged 2 commits intocloudflare:mainfrom
austin-mc:achristiansen/stream-6638
Feb 22, 2026
Merged

STREAM-6638 Update MediaTransformationResult methods to return Promises#6096
danlapid merged 2 commits intocloudflare:mainfrom
austin-mc:achristiansen/stream-6638

Conversation

@austin-mc
Copy link
Copy Markdown
Contributor

Change media(), response(), and contentType() methods in MediaTransformationResult interface to return Promise types instead of synchronous values. This update aligns the type definitions with the actual existing return types of these methods.

@austin-mc austin-mc requested review from a team as code owners February 17, 2026 20:55
@austin-mc
Copy link
Copy Markdown
Contributor Author

@danlapid would you mind approving/merging this for me?

@austin-mc austin-mc force-pushed the achristiansen/stream-6638 branch from 24a5ba0 to e44e394 Compare February 19, 2026 23:08
Change media(), response(), and contentType() methods in MediaTransformationResult
interface to return Promise types instead of synchronous values. This update
aligns the type definitions with the actual existing return types of these methods.
@danlapid danlapid force-pushed the achristiansen/stream-6638 branch from e44e394 to a14670a Compare February 20, 2026 14:11
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Feb 20, 2026

Merging this PR will improve performance by 30.64%

⚡ 1 improved benchmark
✅ 69 untouched benchmarks
⏩ 129 skipped benchmarks1

Performance Changes

Benchmark BASE HEAD Efficiency
bodyWithHeaders[Response] 39.5 µs 30.3 µs +30.64%

Comparing austin-mc:achristiansen/stream-6638 (da66331) with main (c4d649b)

Open in CodSpeed

Footnotes

  1. 129 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@danlapid danlapid merged commit 31c07ba into cloudflare:main Feb 22, 2026
17 of 18 checks passed
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.

3 participants