oas-toolkit 0.5.0 → 0.5.1

package/cli/commands/remove-unused-components.js

@@ -10,14 +10,8 @@ module.exports = async function ({ argv }) {
 
     const components = require("../../components");
     let oas = yaml.load(fs.readFileSync(oasFiles[0]));
-
-    const used = components.getReferencedComponents(oas);
-    const defined = components.getDefinedComponents(oas);
-    const unused = components.getUnusedComponents(defined, used);
-
-    oas = components.removeComponents(oas, unused);
-
-    fs.writeFileSync(oasFiles[0], oas);
+    oas = components.removeUnusedComponents(oas);
+    console.log(yaml.dump(oas));
   } catch (e) {
     console.error(`ERROR: ${e.message}`);
     process.exit(1);

package/components.js

@@ -2,7 +2,15 @@ const traverse = require("traverse");
 const get = require("lodash.get");
 const difference = require("lodash.difference");
 
-function removeComponents(oas, unused) {
+function removeUnusedComponents(oas) {
+  const used = getReferencedComponents(oas);
+  const defined = getDefinedComponents(oas);
+  const unused = getUnusedComponents(defined, used);
+
+  return removeSpecifiedComponents(oas, unused);
+}
+
+function removeSpecifiedComponents(oas, unused) {
   oas = traverse(oas).clone();
   return traverse(oas).forEach(function (x) {
     const path = this.path.join(".");
@@ -47,5 +55,6 @@ module.exports = {
   getReferencedComponents,
   getDefinedComponents,
   getUnusedComponents,
-  removeComponents,
+  removeSpecifiedComponents,
+  removeUnusedComponents,
 };

package/components.test.js

@@ -93,7 +93,9 @@ describe("#components", () => {
 
   it("removed unused components including parent", () => {
     expect(
-      c.removeComponents({ components }, ["components.requestBodies.CreateFoo"])
+      c.removeSpecifiedComponents({ components }, [
+        "components.requestBodies.CreateFoo",
+      ])
     ).toEqual({
       components: {
         schemas: {
@@ -117,7 +119,7 @@ describe("#components", () => {
 
   it("removed unused components but leaves the parent", () => {
     expect(
-      c.removeComponents({ components }, ["components.schemas.Foo"])
+      c.removeSpecifiedComponents({ components }, ["components.schemas.Foo"])
     ).toEqual({
       components: {
         schemas: {
@@ -145,4 +147,55 @@ describe("#components", () => {
       },
     });
   });
+
+  it("autodiscovers and removes unused components", () => {
+    expect(c.removeUnusedComponents(oas)).toEqual({
+      info: { title: "One" },
+      paths: {
+        "/foo": {
+          post: {
+            requestBody: {
+              $ref: "#/components/requestBodies/CreateFoo",
+            },
+            responses: {
+              201: {
+                content: {
+                  "application/json": {
+                    schema: {
+                      $ref: "#/components/schemas/Foo",
+                    },
+                  },
+                },
+              },
+            },
+          },
+        },
+      },
+      components: {
+        schemas: {
+          Foo: {
+            type: "object",
+            properties: {
+              bar: { type: "string" },
+              created_at: { type: "string" },
+            },
+          },
+        },
+        requestBodies: {
+          CreateFoo: {
+            content: {
+              "application/json": {
+                schema: {
+                  type: "object",
+                  properties: {
+                    bar: { type: "string" },
+                  },
+                },
+              },
+            },
+          },
+        },
+      },
+    });
+  });
 });

package/index.js

@@ -0,0 +1,4 @@
+module.exports = {
+  components: require("./components"),
+  merger: require("./merger"),
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oas-toolkit",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "",
   "main": "index.js",
   "bin": {