Make %(ApiResponse) a proc as conversion of wrapped types can have side-effects.

2025-08-02 12:40:14 -05:00
2 changed files with 2 additions and 2 deletions
--- a/buffoonery.nimble
+++ b/buffoonery.nimble
@@ -1,6 +1,6 @@
 # Package

-version       = "0.4.9"
+version       = "0.4.10"
 author        = "Jonathan Bernard"
 description   = "Jonathan's opinionated extensions and auth layer for Jester."
 license       = "MIT"
--- a/src/buffoonery/apiutils.nim
+++ b/src/buffoonery/apiutils.nim
@@ -31,7 +31,7 @@ func initApiResponse*[T](
    totalItems: totalItems, nextLink: nextLink, prevLink: prevLink)


-func `%`*(r: ApiResponse): JsonNode =
+proc `%`*(r: ApiResponse): JsonNode =
  result = newJObject()
  if r.details.isSome: result["details"] = %r.details
  if r.data.isSome: result["data"] = %r.data