simplify imports (#98)

remove broken uint64 converter - upstream std/json also includes a
broken uint64 converter

json_rpc.nimble

@@ -6,7 +6,7 @@ license       = "Apache License 2.0"
 skipDirs      = @["tests"]
 
 ### Dependencies
-requires "nim >= 0.17.3",
+requires "nim >= 1.2.0",
          "nimcrypto",
          "stint",
          "chronos",

json_rpc/client.nim

@@ -1,12 +1,12 @@
 import
-  std/[tables, json, macros],
+  std/[tables, macros],
   chronos,
   ./jsonmarshal
 
 from strutils import toLowerAscii, replace
 
 export
-  chronos, json, tables
+  chronos, jsonmarshal, tables
 
 type
   ClientId* = int64

json_rpc/clients/httpclient.nim

@@ -1,9 +1,11 @@
 import
-  std/[json, strutils, tables, uri],
+  std/[strutils, tables, uri],
   stew/byteutils,
-  chronicles, httputils, chronos, json_serialization/std/net,
+  chronicles, httputils, json_serialization/std/net,
   ../client
 
+export client
+
 logScope:
   topics = "JSONRPC-HTTP-CLIENT"
 

json_rpc/clients/socketclient.nim

@@ -1,6 +1,9 @@
 import
-  std/[json, tables],
-  ../client, chronos
+  std/tables,
+  chronos,
+  ../client
+
+export client
 
 type
   RpcSocketClient* = ref object of RpcClient

json_rpc/clients/websocketclient.nim

@@ -1,6 +1,9 @@
 import
-  std/[json, strtabs, tables],
-  ../client, chronos
+  std/[strtabs, tables],
+  chronos,
+  ../client
+
+export client
 
 const newsUseChronos = true
 include news

json_rpc/jsonmarshal.nim

@@ -2,16 +2,12 @@ import
   std/[macros, json, options, typetraits],
   stew/byteutils
 
-export json
+export json, options
 
 proc expect*(actual, expected: JsonNodeKind, argName: string) =
-  if actual != expected: raise newException(ValueError, "Parameter [" & argName & "] expected " & $expected & " but got " & $actual)
-
-proc `%`*(n: byte{not lit}): JsonNode =
-  newJInt(int(n))
-
-proc `%`*(n: uint64{not lit}): JsonNode =
-  newJInt(int(n))
+  if actual != expected:
+    raise newException(
+      ValueError, "Parameter [" & argName & "] expected " & $expected & " but got " & $actual)
 
 proc `%`*(n: ref SomeInteger): JsonNode =
   if n.isNil:
@@ -19,13 +15,6 @@ proc `%`*(n: ref SomeInteger): JsonNode =
   else:
     newJInt(n[])
 
-when (NimMajor, NimMinor, NimPatch) < (0, 19, 9):
-  proc `%`*[T](option: Option[T]): JsonNode =
-    if option.isSome:
-      `%`(option.get)
-    else:
-      newJNull()
-
 # Compiler requires forward decl when processing out of module
 proc fromJson*(n: JsonNode, argName: string, result: var bool)
 proc fromJson*(n: JsonNode, argName: string, result: var int)

json_rpc/router.nim

@@ -1,10 +1,10 @@
 import
-  std/[json, macros, options, strutils, tables],
+  std/[macros, options, strutils, tables],
   chronicles, chronos, json_serialization/writer,
   ./jsonmarshal
 
 export
-  chronos, json, jsonmarshal
+  chronos, jsonmarshal
 
 type
   StringOfJson* = JsonString

json_rpc/server.nim

@@ -1,10 +1,9 @@
 import
-  std/[json, macros],
-  chronos, chronicles,
+  chronos,
   ./router,
   ./jsonmarshal
 
-export chronos, json, jsonmarshal, router, chronicles
+export chronos, jsonmarshal, router
 
 type
   RpcServer* = ref object of RootRef

json_rpc/servers/httpserver.nim

@@ -1,7 +1,9 @@
 import
-  std/[json, strutils],
+  std/[strutils],
   chronicles, httputils, chronos,
-  ../server, ../errors
+  ".."/[errors, server]
+
+export server
 
 logScope:
   topics = "JSONRPC-HTTP-SERVER"

json_rpc/servers/socketserver.nim

@@ -1,10 +1,9 @@
 import
-  std/json,
   chronicles,
   json_serialization/std/net,
-  ../server, ../errors
+  ".."/[errors, server]
 
-export server
+export errors, server
 
 type
   RpcSocketServer* = ref object of RpcServer