@hey-api/json-schema-ref-parser 1.0.7 → 1.0.8

package/dist/lib/__tests__/pointer.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/lib/__tests__/pointer.test.js

@@ -0,0 +1,27 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const vitest_1 = require("vitest");
+const __1 = require("..");
+const path_1 = __importDefault(require("path"));
+(0, vitest_1.describe)("pointer", () => {
+    (0, vitest_1.it)("inlines internal JSON Pointer refs under #/paths/ for OpenAPI bundling", async () => {
+        const refParser = new __1.$RefParser();
+        const pathOrUrlOrSchema = path_1.default.resolve("lib", "__tests__", "spec", "openapi-paths-ref.json");
+        const schema = (await refParser.bundle({ pathOrUrlOrSchema }));
+        // The GET endpoint should have its schema defined inline
+        const getSchema = schema.paths["/foo"].get.responses["200"].content["application/json"].schema;
+        (0, vitest_1.expect)(getSchema.$ref).toBeUndefined();
+        (0, vitest_1.expect)(getSchema.type).toBe("object");
+        (0, vitest_1.expect)(getSchema.properties.bar.type).toBe("string");
+        // The POST endpoint should have its schema inlined (copied) instead of a $ref
+        const postSchema = schema.paths["/foo"].post.responses["200"].content["application/json"].schema;
+        (0, vitest_1.expect)(postSchema.$ref).toBeUndefined();
+        (0, vitest_1.expect)(postSchema.type).toBe("object");
+        (0, vitest_1.expect)(postSchema.properties.bar.type).toBe("string");
+        // Both schemas should be identical objects
+        (0, vitest_1.expect)(postSchema).toEqual(getSchema);
+    });
+});

package/dist/lib/bundle.js

@@ -249,7 +249,7 @@ function remap(inventory) {
     let file, hash, pathFromRoot;
     for (const entry of inventory) {
         // console.log('Re-mapping $ref pointer "%s" at %s', entry.$ref.$ref, entry.pathFromRoot);
-        if (!entry.external) {
+        if (!entry.external && !entry.hash?.startsWith("#/paths/")) {
             // This $ref already resolves to the main JSON Schema file
             entry.$ref.$ref = entry.hash;
         }

package/lib/__tests__/pointer.test.ts

@@ -0,0 +1,26 @@
+import { describe, expect, it } from "vitest";
+import { $RefParser } from "..";
+import path from "path";
+
+describe("pointer", () => {
+  it("inlines internal JSON Pointer refs under #/paths/ for OpenAPI bundling", async () => {
+    const refParser = new $RefParser();
+    const pathOrUrlOrSchema = path.resolve("lib", "__tests__", "spec", "openapi-paths-ref.json");
+    const schema = (await refParser.bundle({ pathOrUrlOrSchema })) as any;
+
+    // The GET endpoint should have its schema defined inline
+    const getSchema = schema.paths["/foo"].get.responses["200"].content["application/json"].schema;
+    expect(getSchema.$ref).toBeUndefined();
+    expect(getSchema.type).toBe("object");
+    expect(getSchema.properties.bar.type).toBe("string");
+
+    // The POST endpoint should have its schema inlined (copied) instead of a $ref
+    const postSchema = schema.paths["/foo"].post.responses["200"].content["application/json"].schema;
+    expect(postSchema.$ref).toBeUndefined();
+    expect(postSchema.type).toBe("object");
+    expect(postSchema.properties.bar.type).toBe("string");
+
+    // Both schemas should be identical objects
+    expect(postSchema).toEqual(getSchema);
+  });
+});

package/lib/__tests__/spec/openapi-paths-ref.json

@@ -0,0 +1,46 @@
+{
+  "openapi": "3.1.0",
+  "info": {
+    "title": "Sample API",
+    "version": "1.0.0"
+  },
+  "paths": {
+    "/foo": {
+      "get": {
+        "summary": "Get foo",
+        "responses": {
+          "200": {
+            "description": "OK",
+            "content": {
+              "application/json": {
+                "schema": {
+                  "type": "object",
+                  "properties": {
+                    "bar": {
+                      "type": "string"
+                    }
+                  }
+                }
+              }
+            }
+          }
+        }
+      },
+      "post": {
+        "summary": "Create foo",
+        "responses": {
+          "200": {
+            "description": "OK",
+            "content": {
+              "application/json": {
+                "schema": {
+                  "$ref": "#/paths/~1foo/get/responses/200/content/application~1json/schema"
+                }
+              }
+            }
+          }
+        }
+      }
+    }
+  }
+}

package/lib/bundle.ts

@@ -300,7 +300,7 @@ function remap(inventory: InventoryEntry[]) {
   for (const entry of inventory) {
     // console.log('Re-mapping $ref pointer "%s" at %s', entry.$ref.$ref, entry.pathFromRoot);
 
-    if (!entry.external) {
+    if (!entry.external && !entry.hash?.startsWith("#/paths/")) {
       // This $ref already resolves to the main JSON Schema file
       entry.$ref.$ref = entry.hash;
     } else if (entry.file === file && entry.hash === hash) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hey-api/json-schema-ref-parser",
-  "version": "1.0.7",
+  "version": "1.0.8",
   "description": "Parse, Resolve, and Dereference JSON Schema $ref pointers",
   "homepage": "https://heyapi.dev/",
   "repository": {
@@ -42,6 +42,7 @@
   ],
   "scripts": {
     "build": "rimraf dist && tsc",
+    "dev": "rimraf dist && tsc --watch",
     "lint": "eslint lib",
     "prepublishOnly": "yarn build",
     "prettier": "prettier --write \"**/*.+(js|jsx|ts|tsx|har||json|css|md)\"",