fix(types): Optional JSON params where undefined is not valid (#134)

legobeat · Mrtenz · web-flow · commit e8cad2fe3cd5 · 2023-08-31T18:05:07.000+09:00
* fix(types): Optional JSON params where undefined is not valid
* Reduce type duplication by excluding `undefined`

---------

Co-authored-by: Maarten Zuidhoorn &lt;maarten@zuidhoorn.com&gt;
diff --git a/src/json.ts b/src/json.ts
@@ -13,7 +13,6 @@ import {
   nullable,
   number,
   object,
-  omit,
   optional,
   record,
   string,
@@ -178,36 +177,35 @@ export type JsonRpcError = OptionalField<
 >;
 
 export const JsonRpcParamsStruct: Struct<Json[] | Record<string, Json>, null> =
-  optional(union([record(string(), JsonStruct), array(JsonStruct)])) as any;
+  union([record(string(), JsonStruct), array(JsonStruct)]);
 
 export type JsonRpcParams = Json[] | Record<string, Json>;
 
 export const JsonRpcRequestStruct = object({
   id: JsonRpcIdStruct,
   jsonrpc: JsonRpcVersionStruct,
   method: string(),
-  params: JsonRpcParamsStruct,
+  params: optional(JsonRpcParamsStruct),
 });
 
 export type InferWithParams<
   Type extends Struct<any>,
   Params extends JsonRpcParams,
-> = Omit<Infer<Type>, 'params'> &
-  (keyof Params extends undefined
-    ? {
-        params?: Params;
-      }
-    : {
-        params: Params;
-      });
+> = Omit<Infer<Type>, 'params'> & {
+  params?: Exclude<Params, undefined>;
+};
 
 /**
  * A JSON-RPC request object.
  */
 export type JsonRpcRequest<Params extends JsonRpcParams = JsonRpcParams> =
   InferWithParams<typeof JsonRpcRequestStruct, Params>;
 
-export const JsonRpcNotificationStruct = omit(JsonRpcRequestStruct, ['id']);
+export const JsonRpcNotificationStruct = object({
+  jsonrpc: JsonRpcVersionStruct,
+  method: string(),
+  params: optional(JsonRpcParamsStruct),
+});
 
 /**
  * A JSON-RPC notification object.
