Refactor path params and optimize JSON handling

.gitignore

@@ -7,3 +7,5 @@
 assets/myadam.jpg
 .github/copilot-instructions.md
 docs/UPDATE_SUMMARIES.md
+assets/cb7d0daf60d7675081996d81393e2ae5.jpg
+assets/b9c93c1cd427d8f50e68dbd11ed2b000.jpg

crates/rustapi-core/src/app.rs

@@ -266,17 +266,19 @@ impl RustApi {
             entry.insert_boxed_with_operation(method_enum, route.handler, route.operation);
         }
 
-        let route_count = by_path
+        #[cfg(feature = "tracing")]
+        let route_count: usize = by_path
             .values()
             .map(|mr| mr.allowed_methods().len())
-            .sum::<usize>();
+            .sum();
+        #[cfg(feature = "tracing")]
         let path_count = by_path.len();
 
         for (path, method_router) in by_path {
             self = self.route(&path, method_router);
         }
 
-        tracing::info!(
+        crate::trace_info!(
             paths = path_count,
             routes = route_count,
             "Auto-registered routes"
@@ -887,12 +889,12 @@ impl Default for RustApi {
 mod tests {
     use super::RustApi;
     use crate::extract::{FromRequestParts, State};
+    use crate::path_params::PathParams;
     use crate::request::Request;
     use crate::router::{get, post, Router};
     use bytes::Bytes;
     use http::Method;
     use proptest::prelude::*;
-    use std::collections::HashMap;
 
     #[test]
     fn state_is_available_via_extractor() {
@@ -906,7 +908,7 @@ mod tests {
             .unwrap();
         let (parts, _) = req.into_parts();
 
-        let request = Request::new(parts, Bytes::new(), router.state_ref(), HashMap::new());
+        let request = Request::new(parts, Bytes::new(), router.state_ref(), PathParams::new());
         let State(value) = State::<u32>::from_request_parts(&request).unwrap();
         assert_eq!(value, 123u32);
     }

crates/rustapi-core/src/extract.rs

@@ -55,6 +55,7 @@
 //! in any order.
 
 use crate::error::{ApiError, Result};
+use crate::json;
 use crate::request::Request;
 use crate::response::IntoResponse;
 use bytes::Bytes;
@@ -116,7 +117,8 @@ impl<T: DeserializeOwned + Send> FromRequest for Json<T> {
             .take_body()
             .ok_or_else(|| ApiError::internal("Body already consumed"))?;
 
-        let value: T = serde_json::from_slice(&body)?;
+        // Use simd-json accelerated parsing when available (2-4x faster)
+        let value: T = json::from_slice(&body)?;
         Ok(Json(value))
     }
 }
@@ -141,10 +143,15 @@ impl<T> From<T> for Json<T> {
     }
 }
 
+/// Default pre-allocation size for JSON response buffers (256 bytes)
+/// This covers most small to medium JSON responses without reallocation.
+const JSON_RESPONSE_INITIAL_CAPACITY: usize = 256;
+
 // IntoResponse for Json - allows using Json<T> as a return type
 impl<T: Serialize> IntoResponse for Json<T> {
     fn into_response(self) -> crate::response::Response {
-        match serde_json::to_vec(&self.0) {
+        // Use pre-allocated buffer to reduce allocations
+        match json::to_vec_with_capacity(&self.0, JSON_RESPONSE_INITIAL_CAPACITY) {
             Ok(body) => http::Response::builder()
                 .status(StatusCode::OK)
                 .header(header::CONTENT_TYPE, "application/json")
@@ -199,12 +206,12 @@ impl<T> ValidatedJson<T> {
 
 impl<T: DeserializeOwned + rustapi_validate::Validate + Send> FromRequest for ValidatedJson<T> {
     async fn from_request(req: &mut Request) -> Result<Self> {
-        // First, deserialize the JSON body
+        // First, deserialize the JSON body using simd-json when available
         let body = req
             .take_body()
             .ok_or_else(|| ApiError::internal("Body already consumed"))?;
 
-        let value: T = serde_json::from_slice(&body)?;
+        let value: T = json::from_slice(&body)?;
 
         // Then, validate it
         if let Err(validation_error) = rustapi_validate::Validate::validate(&value) {
@@ -778,10 +785,17 @@ impl<T: for<'a> Schema<'a>> OperationModifier for Json<T> {
     }
 }
 
-// Path - Placeholder for path params
+// Path - Path parameters are automatically extracted from route patterns
+// The add_path_params_to_operation function in app.rs handles OpenAPI documentation
+// based on the {param} syntax in route paths (e.g., "/users/{id}")
 impl<T> OperationModifier for Path<T> {
     fn update_operation(_op: &mut Operation) {
-        // TODO: Implement path param extraction
+        // Path parameters are automatically documented by add_path_params_to_operation
+        // in app.rs based on the route pattern. No additional implementation needed here.
+        // 
+        // For typed path params, the schema type defaults to "string" but will be
+        // inferred from the actual type T when more sophisticated type introspection
+        // is implemented.
     }
 }
 
@@ -885,6 +899,7 @@ impl<T: for<'a> Schema<'a>> ResponseModifier for Json<T> {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use crate::path_params::PathParams;
     use bytes::Bytes;
     use http::{Extensions, Method};
     use proptest::prelude::*;
@@ -912,7 +927,7 @@ mod tests {
             parts,
             Bytes::new(),
             Arc::new(Extensions::new()),
-            HashMap::new(),
+            PathParams::new(),
         )
     }
 
@@ -933,7 +948,7 @@ mod tests {
             parts,
             Bytes::new(),
             Arc::new(Extensions::new()),
-            HashMap::new(),
+            PathParams::new(),
         )
     }
 
@@ -1109,7 +1124,7 @@ mod tests {
                     parts,
                     Bytes::new(),
                     Arc::new(Extensions::new()),
-                    HashMap::new(),
+                    PathParams::new(),
                 );
 
                 let extracted = ClientIp::extract_with_config(&request, trust_proxy)
@@ -1171,7 +1186,7 @@ mod tests {
                     parts,
                     Bytes::new(),
                     Arc::new(Extensions::new()),
-                    HashMap::new(),
+                    PathParams::new(),
                 );
 
                 let result = Extension::<TestExtension>::from_request_parts(&request);
@@ -1271,7 +1286,7 @@ mod tests {
             parts,
             Bytes::new(),
             Arc::new(Extensions::new()),
-            HashMap::new(),
+            PathParams::new(),
         );
 
         let ip = ClientIp::extract_with_config(&request, false).unwrap();

crates/rustapi-core/src/middleware/body_limit.rs

@@ -121,11 +121,11 @@ impl MiddlewareLayer for BodyLimitLayer {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use crate::path_params::PathParams;
     use crate::request::Request;
     use bytes::Bytes;
     use http::{Extensions, Method};
     use proptest::prelude::*;
-    use std::collections::HashMap;
     use std::sync::Arc;
 
     /// Create a test request with the given body
@@ -139,7 +139,7 @@ mod tests {
         let req = builder.body(()).unwrap();
         let (parts, _) = req.into_parts();
 
-        Request::new(parts, body, Arc::new(Extensions::new()), HashMap::new())
+        Request::new(parts, body, Arc::new(Extensions::new()), PathParams::new())
     }
 
     /// Create a test request without Content-Length header
@@ -150,7 +150,7 @@ mod tests {
         let req = builder.body(()).unwrap();
         let (parts, _) = req.into_parts();
 
-        Request::new(parts, body, Arc::new(Extensions::new()), HashMap::new())
+        Request::new(parts, body, Arc::new(Extensions::new()), PathParams::new())
     }
 
     /// Create a simple handler that returns 200 OK

crates/rustapi-core/src/middleware/layer.rs

@@ -192,13 +192,13 @@ impl Service<Request> for NextService {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use crate::path_params::PathParams;
     use crate::request::Request;
     use crate::response::Response;
     use bytes::Bytes;
     use http::{Extensions, Method, StatusCode};
     use proptest::prelude::*;
     use proptest::test_runner::TestCaseError;
-    use std::collections::HashMap;
 
     /// Create a test request with the given method and path
     fn create_test_request(method: Method, path: &str) -> Request {
@@ -212,7 +212,7 @@ mod tests {
             parts,
             Bytes::new(),
             Arc::new(Extensions::new()),
-            HashMap::new(),
+            PathParams::new(),
         )
     }
 

crates/rustapi-core/src/middleware/request_id.rs

@@ -167,11 +167,12 @@ fn generate_uuid() -> String {
 mod tests {
     use super::*;
     use crate::middleware::layer::{BoxedNext, LayerStack};
+    use crate::path_params::PathParams;
     use bytes::Bytes;
     use http::{Extensions, Method, StatusCode};
     use proptest::prelude::*;
     use proptest::test_runner::TestCaseError;
-    use std::collections::{HashMap, HashSet};
+    use std::collections::HashSet;
     use std::sync::Arc;
 
     /// Create a test request with the given method and path
@@ -186,7 +187,7 @@ mod tests {
             parts,
             Bytes::new(),
             Arc::new(Extensions::new()),
-            HashMap::new(),
+            PathParams::new(),
         )
     }
 

crates/rustapi-core/src/middleware/tracing_layer.rs

@@ -204,6 +204,7 @@ mod tests {
     use super::*;
     use crate::middleware::layer::{BoxedNext, LayerStack};
     use crate::middleware::request_id::RequestIdLayer;
+    use crate::path_params::PathParams;
     use bytes::Bytes;
     use http::{Extensions, Method, StatusCode};
     use proptest::prelude::*;
@@ -224,7 +225,7 @@ mod tests {
             parts,
             Bytes::new(),
             Arc::new(Extensions::new()),
-            HashMap::new(),
+            PathParams::new(),
         )
     }