Removed the optional value feature

Author: ericallam

src/@types.ts

@@ -2,39 +2,39 @@ import { JSONStringFormat, JSONObjectFormat } from "./formats";
 
 export type JSONNullType = {
   name: "null";
-  value?: null;
+  value: null;
 };
 
 export type JSONBoolType = {
   name: "bool";
-  value?: boolean;
+  value: boolean;
 };
 
 export type JSONFloatType = {
   name: "float";
-  value?: number;
+  value: number;
 };
 
 export type JSONIntType = {
   name: "int";
-  value?: number;
+  value: number;
 };
 
 export type JSONStringType = {
   name: "string";
   format?: JSONStringFormat;
-  value?: string;
+  value: string;
 };
 
 export type JSONObjectType = {
   name: "object";
   format?: JSONObjectFormat;
-  value?: object;
+  value: object;
 };
 
 export type JSONArrayType = {
   name: "array";
-  value?: Array<unknown>;
+  value: Array<unknown>;
 };
 
 export type JSONValueType =

src/index.ts

@@ -34,35 +34,27 @@ export {
   JSONArrayType,
 } from "./@types";
 
-export type InferOptions = {
-  returnValue?: "yes" | "no";
-};
-
-export function inferType(value: unknown, options?: InferOptions): JSONValueType {
-  const opts: Required<InferOptions> = Object.assign({}, { returnValue: "yes" }, options);
-
-  const shouldReturnValue = opts.returnValue === "yes";
-
+export function inferType(value: unknown): JSONValueType {
   if (value === null) {
-    return { name: "null", value: shouldReturnValue ? null : undefined };
+    return { name: "null", value };
   }
 
   if (typeof value === "boolean") {
-    return { name: "bool", value: shouldReturnValue ? value : undefined };
+    return { name: "bool", value };
   }
 
   if (typeof value === "number") {
     if (Number.isInteger(value)) {
-      return { name: "int", value: shouldReturnValue ? value : undefined };
+      return { name: "int", value };
     } else {
-      return { name: "float", value: shouldReturnValue ? value : undefined };
+      return { name: "float", value };
     }
   }
 
   if (typeof value === "string") {
     return {
       name: "string",
-      value: shouldReturnValue ? value : undefined,
+      value,
       format: inferFormat(value),
     };
   }
@@ -71,16 +63,16 @@ export function inferType(value: unknown, options?: InferOptions): JSONValueType
     if (Array.isArray(value)) {
       return {
         name: "array",
-        value: shouldReturnValue ? value : undefined,
+        value,
       };
     }
 
     return {
       name: "object",
       format: inferObjectFormat(value),
-      value: shouldReturnValue ? value : undefined,
+      value,
     };
   }
 
-  return { name: "null", value: shouldReturnValue ? null : undefined };
+  return { name: "null", value: null };
 }

tests/basicTypes.test.ts

@@ -15,10 +15,6 @@ test("It should infer basic types", () => {
   expect(inferType([{ foo: "bar" }])).toEqual({ name: "array", value: [{ foo: "bar" }] });
 });
 
-test("it should allow an option to not include the values in the result", () => {
-  expect(inferType("", { returnValue: "no" })).toEqual({ name: "string" });
-});
-
 test("It should narrow the type of the value", () => {
   const unknownInfer = (value: unknown): number | undefined => {
     const result = inferType(value);