feat: Preserve OpenAPI extensions on Operation

Author: thced

src/main/java/com/retailsvc/http/spec/Operation.java

@@ -10,4 +10,5 @@ public record Operation(
     PathTemplate path,
     Optional<RequestBody> requestBody,
     List<Parameter> parameters,
-    Map<String, Response> responses) {}
+    Map<String, Response> responses,
+    Map<String, Object> extensions) {}

src/main/java/com/retailsvc/http/spec/Spec.java

@@ -179,7 +179,7 @@ private static Operation parseOperation(
             .orElse(List.of());
     Map<String, Response> responses =
         parseResponses((Map<String, Object>) raw.getOrDefault("responses", Map.of()));
-    return new Operation(opId, method, path, body, params, responses);
+    return new Operation(opId, method, path, body, params, responses, extractExtensions(raw));
   }
 
   private static Parameter resolveParameterOrParse(

src/test/java/com/retailsvc/http/internal/RequestPreparationFilterTest.java

@@ -71,6 +71,7 @@ void successPathBindsRequestContextDuringChain() throws Exception {
             PathTemplate.compile("/users/{id}"),
             Optional.empty(),
             List.of(),
+            Map.of(),
             Map.of());
     Spec spec = specWith(op);
     Filter f = newFilter(spec);
@@ -106,6 +107,7 @@ void unknownPathThrowsNotFound() {
                 PathTemplate.compile("/x"),
                 Optional.empty(),
                 List.of(),
+                Map.of(),
                 Map.of()));
     Filter f = newFilter(spec);
 
@@ -124,6 +126,7 @@ void wrongMethodThrowsMethodNotAllowed() {
                 PathTemplate.compile("/x"),
                 Optional.empty(),
                 List.of(),
+                Map.of(),
                 Map.of()));
     Filter f = newFilter(spec);
 
@@ -142,6 +145,7 @@ void invalidQueryParamThrowsValidation() {
             PathTemplate.compile("/x"),
             Optional.empty(),
             List.of(new Parameter("q", Parameter.Location.QUERY, true, stringSchema)),
+            Map.of(),
             Map.of());
     Spec spec = specWith(op);
     Filter f = newFilter(spec);

src/test/java/com/retailsvc/http/internal/RouterTest.java

@@ -12,7 +12,8 @@
 
 class RouterTest {
   private Operation op(String id, HttpMethod m, String path) {
-    return new Operation(id, m, PathTemplate.compile(path), Optional.empty(), List.of(), Map.of());
+    return new Operation(
+        id, m, PathTemplate.compile(path), Optional.empty(), List.of(), Map.of(), Map.of());
   }
 
   @Test

src/test/java/com/retailsvc/http/spec/ExtensionsTest.java

@@ -57,4 +57,31 @@ void infoExtensionsExposeXKeys() {
     Spec spec = Spec.from(raw);
     assertThat(spec.info().extensions()).containsEntry("x-contact-team", "platform");
   }
+
+  @Test
+  void operationExtensionsExposeXPermissions() {
+    Map<String, Object> raw =
+        Map.of(
+            "openapi",
+            "3.1.0",
+            "info",
+            Map.of("title", "t", "version", "1"),
+            "servers",
+            List.of(Map.of("url", "https://example.com")),
+            "paths",
+            Map.of(
+                "/promotions",
+                Map.of(
+                    "post",
+                    Map.of(
+                        "operationId",
+                        "createPromotion",
+                        "x-permissions",
+                        List.of("pro.promotion.create"),
+                        "responses",
+                        Map.of()))));
+    Spec spec = Spec.from(raw);
+    Operation op = spec.operations().getFirst();
+    assertThat(op.extensions()).containsEntry("x-permissions", List.of("pro.promotion.create"));
+  }
 }

src/test/java/com/retailsvc/http/spec/OperationTest.java

@@ -18,7 +18,8 @@ void operationCarriesAllFields() {
         new Parameter(
             "id", Parameter.Location.PATH, true, new BooleanSchema(Set.of(TypeName.BOOLEAN)));
     Operation op =
-        new Operation("get-user", HttpMethod.GET, path, Optional.empty(), List.of(param), Map.of());
+        new Operation(
+            "get-user", HttpMethod.GET, path, Optional.empty(), List.of(param), Map.of(), Map.of());
     assertThat(op.operationId()).isEqualTo("get-user");
     assertThat(op.method()).isEqualTo(HttpMethod.GET);
     assertThat(op.parameters()).hasSize(1);