refactor: Drop fromYaml(InputStream, Function) BYO-parser overload

SnakeYAML is the only YAML parser in common Java use, so the parser-
injecting variant carries API surface for no real callers. Users who
need a different YAML library can decode the stream themselves and
call Spec.from(Map).

Author: thced

README.md

@@ -134,7 +134,7 @@ try (InputStream in = YourServerLauncher.class.getResourceAsStream("/openapi.jso
 }
 ```
 
-The matching `Spec.fromYaml(InputStream)` uses SnakeYAML. Both close the stream before returning. If you can't (or don't want to) depend on Gson/SnakeYAML, supply your own parser:
+The matching `Spec.fromYaml(InputStream)` uses SnakeYAML. Both close the stream before returning. If you can't (or don't want to) depend on Gson, supply your own JSON parser:
 
 ``` java
 ObjectMapper jackson = new ObjectMapper();
@@ -144,7 +144,7 @@ try (InputStream in = YourServerLauncher.class.getResourceAsStream("/openapi.jso
 }
 ```
 
-`Spec.fromYaml(InputStream, Function<byte[], Map<String, Object>>)` is the equivalent escape hatch for YAML.
+YAML always parses through SnakeYAML — there's no parser-injecting overload. If you want a different YAML library, decode the stream yourself and call `Spec.from(Map<String, Object>)`.
 
 ### JSON mapping
 

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

@@ -142,26 +142,12 @@ public static Spec fromJson(InputStream in, Function<byte[], Map<String, Object>
    * classpath; otherwise throws {@link IllegalStateException}. The stream is fully consumed and
    * closed before this method returns.
    *
-   * <p>To avoid the SnakeYAML dependency, use {@link #fromYaml(InputStream, Function)} instead.
-   *
    * @throws NullPointerException if {@code in} is {@code null}
    * @throws UncheckedIOException if the stream cannot be read
    * @throws IllegalStateException if SnakeYAML is not on the classpath
    */
   public static Spec fromYaml(InputStream in) {
-    return fromYaml(in, Spec::parseYamlWithSnakeYaml);
-  }
-
-  /**
-   * Reads a YAML OpenAPI specification from {@code in} using the supplied {@code parser}. The
-   * stream is fully consumed and closed before this method returns.
-   *
-   * @throws NullPointerException if {@code in} or {@code parser} is {@code null}
-   * @throws UncheckedIOException if the stream cannot be read
-   */
-  public static Spec fromYaml(InputStream in, Function<byte[], Map<String, Object>> parser) {
-    Objects.requireNonNull(parser, "parser");
-    return from(parser.apply(readAll(in)));
+    return from(parseYamlWithSnakeYaml(readAll(in)));
   }
 
   private static byte[] readAll(InputStream in) {

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

@@ -13,7 +13,6 @@
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.function.Function;
 import org.junit.jupiter.api.Test;
-import org.yaml.snakeyaml.Yaml;
 
 class SpecFromInputStreamTest {
 
@@ -51,19 +50,6 @@ void fromJsonWithCustomParserDoesNotRequireGson() throws Exception {
     }
   }
 
-  @Test
-  void fromYamlWithCustomParserDoesNotRequireSnakeYaml() throws Exception {
-    Yaml yaml = new Yaml();
-    Function<byte[], Map<String, Object>> parser =
-        bytes -> yaml.load(new String(bytes, StandardCharsets.UTF_8));
-
-    try (InputStream in = getClass().getResourceAsStream("/openapi.yaml")) {
-      Spec spec = Spec.fromYaml(in, parser);
-
-      assertThat(spec.openapi()).startsWith("3.1");
-    }
-  }
-
   @Test
   void fromJsonClosesStream() throws Exception {
     AtomicBoolean closed = new AtomicBoolean(false);
@@ -126,15 +112,6 @@ void fromJsonWithParserRejectsNullArgs() {
     assertThatThrownBy(() -> Spec.fromJson(in, null)).isInstanceOf(NullPointerException.class);
   }
 
-  @Test
-  void fromYamlWithParserRejectsNullArgs() {
-    InputStream in = new ByteArrayInputStream(new byte[0]);
-    Function<byte[], Map<String, Object>> parser = bytes -> Map.of();
-
-    assertThatThrownBy(() -> Spec.fromYaml(null, parser)).isInstanceOf(NullPointerException.class);
-    assertThatThrownBy(() -> Spec.fromYaml(in, null)).isInstanceOf(NullPointerException.class);
-  }
-
   @Test
   void fromJsonPropagatesIoFailure() {
     InputStream broken = brokenStream();