Image processing improvements (vercel#4649)

### Description

* refactor image processing
* fallback guess format for extension
* expose pathname_to_url for wrapped source usage

see also vercel/next.js#48622

---------

Co-authored-by: Alex Kirszenberg <alex.kirszenberg@vercel.com>

Author: sokra

crates/turbo-tasks/src/task_input.rs

@@ -330,6 +330,8 @@ pub enum TaskInput {
     String(String),
     Bool(bool),
     Usize(usize),
+    I8(i8),
+    U8(u8),
     I16(i16),
     U16(u16),
     I32(i32),
@@ -515,6 +517,8 @@ impl Display for TaskInput {
             TaskInput::String(s) => write!(f, "string {:?}", s),
             TaskInput::Bool(b) => write!(f, "bool {:?}", b),
             TaskInput::Usize(v) => write!(f, "usize {}", v),
+            TaskInput::I8(v) => write!(f, "i8 {}", v),
+            TaskInput::U8(v) => write!(f, "u8 {}", v),
             TaskInput::I16(v) => write!(f, "i16 {}", v),
             TaskInput::U16(v) => write!(f, "u16 {}", v),
             TaskInput::I32(v) => write!(f, "i32 {}", v),
@@ -548,6 +552,18 @@ impl From<bool> for TaskInput {
     }
 }
 
+impl From<i8> for TaskInput {
+    fn from(v: i8) -> Self {
+        TaskInput::I8(v)
+    }
+}
+
+impl From<u8> for TaskInput {
+    fn from(v: u8) -> Self {
+        TaskInput::U8(v)
+    }
+}
+
 impl From<i16> for TaskInput {
     fn from(v: i16) -> Self {
         TaskInput::I16(v)
@@ -694,6 +710,28 @@ impl<'a, T: FromTaskInput<'a, Error = anyhow::Error>> FromTaskInput<'a> for Vec<
     }
 }
 
+impl FromTaskInput<'_> for u8 {
+    type Error = anyhow::Error;
+
+    fn try_from(value: &TaskInput) -> Result<Self, Self::Error> {
+        match value {
+            TaskInput::U8(value) => Ok(*value),
+            _ => Err(anyhow!("invalid task input type, expected u8")),
+        }
+    }
+}
+
+impl FromTaskInput<'_> for i8 {
+    type Error = anyhow::Error;
+
+    fn try_from(value: &TaskInput) -> Result<Self, Self::Error> {
+        match value {
+            TaskInput::I8(value) => Ok(*value),
+            _ => Err(anyhow!("invalid task input type, expected i8")),
+        }
+    }
+}
+
 impl FromTaskInput<'_> for u16 {
     type Error = anyhow::Error;
 

crates/turbopack-dev-server/src/source/source_maps.rs

@@ -1,5 +1,3 @@
-use std::{borrow::Cow, iter::once};
-
 use anyhow::Result;
 use mime::APPLICATION_JSON;
 use turbo_tasks::{primitives::StringVc, Value};
@@ -12,7 +10,10 @@ use turbopack_core::{
 
 use super::{
     query::QueryValue,
-    wrapping_source::{ContentSourceProcessor, ContentSourceProcessorVc, WrappedContentSourceVc},
+    wrapping_source::{
+        encode_pathname_to_url, ContentSourceProcessor, ContentSourceProcessorVc,
+        WrappedContentSourceVc,
+    },
     ContentSource, ContentSourceContent, ContentSourceContentVc, ContentSourceData,
     ContentSourceDataFilter, ContentSourceDataVary, ContentSourceResultVc, ContentSourceVc,
     NeededData, RewriteBuilder,
@@ -42,17 +43,6 @@ impl SourceMapContentSourceVc {
     }
 }
 
-fn encode_pathname_to_url(pathname: &str) -> String {
-    once(Cow::Borrowed("/"))
-        .chain(
-            pathname
-                .split('/')
-                .map(urlencoding::encode)
-                .intersperse(Cow::Borrowed("/")),
-        )
-        .collect()
-}
-
 #[turbo_tasks::value_impl]
 impl ContentSource for SourceMapContentSource {
     #[turbo_tasks::function]

crates/turbopack-dev-server/src/source/wrapping_source.rs

@@ -1,3 +1,5 @@
+use std::{borrow::Cow, iter::once};
+
 use anyhow::Result;
 use turbo_tasks::Value;
 
@@ -20,6 +22,17 @@ pub trait ContentSourceProcessor {
     fn process(&self, content: ContentSourceContentVc) -> ContentSourceContentVc;
 }
 
+pub fn encode_pathname_to_url(pathname: &str) -> String {
+    once(Cow::Borrowed("/"))
+        .chain(
+            pathname
+                .split('/')
+                .map(urlencoding::encode)
+                .intersperse(Cow::Borrowed("/")),
+        )
+        .collect()
+}
+
 /// A ContentSourceProcessor allows a [ContentSource] implementation to easily
 /// register a final process step over some inner ContentSource's fully resolved
 /// [ContentSourceResult] and [ContentSourceContent] without having to manually