From 5595a29352daee3647e646ac13986f9e7378be19 Mon Sep 17 00:00:00 2001
From: Martin Dimitrov <martin3dimitrov@gmail.com>
Date: Sun, 8 Jun 2025 19:19:30 -0700
Subject: [PATCH] change err to msg

---
 packages/doorman-api/src/common/DoormanHandler.ts    | 10 +++++-----
 packages/doorman-api/src/functions/api/door/auth.ts  |  4 ++--
 packages/doorman-api/src/functions/api/door/edit.ts  |  6 +++---
 packages/doorman-api/src/functions/api/door/info.ts  |  4 ++--
 .../doorman-api/src/functions/api/door/notify.ts     |  4 ++--
 .../doorman-api/src/functions/api/door/onboard.ts    | 12 ++++++------
 packages/doorman-homeassistant/src/index.ts          |  4 ++--
 packages/doorman-ui/src/pages/DoorPage.tsx           |  8 ++++----
 packages/doorman-ui/src/pages/EditPage.tsx           |  4 ++--
 packages/doorman-ui/src/pages/RedirectPage.tsx       |  2 +-
 10 files changed, 29 insertions(+), 29 deletions(-)

diff --git a/packages/doorman-api/src/common/DoormanHandler.ts b/packages/doorman-api/src/common/DoormanHandler.ts
index f51a8e6..4fc23ba 100644
--- a/packages/doorman-api/src/common/DoormanHandler.ts
+++ b/packages/doorman-api/src/common/DoormanHandler.ts
@@ -12,6 +12,7 @@ import LokiTransport from "winston-loki";
 import pTimeout, { TimeoutError } from "promise.timeout";
 import { ZodError } from "zod";
 import { fromError } from "zod-validation-error";
+import { setResponseJson } from "../utils/responseUtils";
 
 export type BaseEvent = { request: { cookies: {}; headers: {}; }; }
 
@@ -248,13 +249,12 @@ export function withMetrics<T extends DoormanLambdaContext, U extends BaseEvent>
           } else if (e instanceof ZodError) {
             // global catch for validation errors
             const response = new Twilio.Response();
-            response.setStatusCode(400);
-            response.setHeaders({
-              "Content-Type": "application/json"
-            });
 
             // return nice to read error message from ZOD
-            response.setBody({ err: fromError(e).toString() });
+            setResponseJson(response, 400, {
+              msg: fromError(e).toString(),
+            });
+            
             callbackResult = [null, response];
           }
         }
diff --git a/packages/doorman-api/src/functions/api/door/auth.ts b/packages/doorman-api/src/functions/api/door/auth.ts
index 308dc6f..a286b0e 100644
--- a/packages/doorman-api/src/functions/api/door/auth.ts
+++ b/packages/doorman-api/src/functions/api/door/auth.ts
@@ -36,7 +36,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, AuthRequestTwil
   if (!config) {
     getMetricFromRegistry<Counter>(metricsRegistry, AuthMetrics.DOOR_CONFIG_NOT_FOUND).inc({ door }, 1);
     setResponseJson(response, 404, {
-      err: `Door ${door} not found`,
+      msg: `Door ${door} not found`,
     });
 
     return callback(null, response);
@@ -57,7 +57,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, AuthRequestTwil
 
   if (!method) {
     setResponseJson(response, 401, {
-      err: "Invalid PIN",
+      msg: "Invalid PIN",
     });
     return callback(null, response);
   }
diff --git a/packages/doorman-api/src/functions/api/door/edit.ts b/packages/doorman-api/src/functions/api/door/edit.ts
index 3270c43..b73da65 100644
--- a/packages/doorman-api/src/functions/api/door/edit.ts
+++ b/packages/doorman-api/src/functions/api/door/edit.ts
@@ -71,7 +71,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, EditRequestTwil
 
   if (!newConfigRaw) {
     setResponseJson(response, 400, {
-      err: "Missing new config",
+      msg: "Missing new config",
     });
     return callback(null, response);
   }
@@ -82,7 +82,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, EditRequestTwil
 
   if (!config) {
     setResponseJson(response, 404, {
-      err: `Door not found ${door}`,
+      msg: `Door not found ${door}`,
     });
     return callback(null, response);
   }
@@ -95,7 +95,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, EditRequestTwil
   // if nothing changed, we should throw since this is a pointless change
   if (isDeepSubset(config, newConfig)) {
     setResponseJson(response, 400, {
-      err: "Nothing changed in the new config",
+      msg: "Nothing changed in the new config",
     });
 
     return callback(null, response);
diff --git a/packages/doorman-api/src/functions/api/door/info.ts b/packages/doorman-api/src/functions/api/door/info.ts
index 40324f0..ec0a025 100644
--- a/packages/doorman-api/src/functions/api/door/info.ts
+++ b/packages/doorman-api/src/functions/api/door/info.ts
@@ -48,7 +48,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, InfoRequestTwil
       .then(async (alias) => {
         if (!alias) {
           setResponseJson(response, 404, {
-            err: "This buzzer is not registered",
+            msg: "This buzzer is not registered",
           });
           return undefined;
         }
@@ -61,7 +61,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, InfoRequestTwil
 
     if (!config) {
       setResponseJson(response, 404, {
-        err: "This buzzer is not registered",
+        msg: "This buzzer is not registered",
       });
     } else {
       if (buzzer) {
diff --git a/packages/doorman-api/src/functions/api/door/notify.ts b/packages/doorman-api/src/functions/api/door/notify.ts
index aae7aec..cae7830 100644
--- a/packages/doorman-api/src/functions/api/door/notify.ts
+++ b/packages/doorman-api/src/functions/api/door/notify.ts
@@ -33,7 +33,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, NotifyRequestTw
   if (req.key !== context.NOTIFY_SECRET_KEY) {
     getMetricFromRegistry<Counter>(metricsRegistry, NotifyMetrics.UNAUTHENTICATED_CALL).inc(1);
     setResponseJson(response, 401, {
-      err: "Unauthenticated call", event
+      msg: "Unauthenticated call", event
     });
     return callback(null, response);
   }
@@ -63,7 +63,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, NotifyRequestTw
     );
   } catch (e) {
     console.error(e);
-    setResponseJson(response, 500, { err: e, event });
+    setResponseJson(response, 500, { msg: e, event });
     return callback(null, response);
   }
 
diff --git a/packages/doorman-api/src/functions/api/door/onboard.ts b/packages/doorman-api/src/functions/api/door/onboard.ts
index 9edcea5..0ae4a54 100644
--- a/packages/doorman-api/src/functions/api/door/onboard.ts
+++ b/packages/doorman-api/src/functions/api/door/onboard.ts
@@ -55,7 +55,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, OnboardRequestT
 
     const config = await db.entities.onboardDoorConfig.findById(getOnboardDoorId(discordState.name));
     if (!config || config.nonce !== discordState.id) {
-      setResponseJson(response, 404, { err: "approval not found" });
+      setResponseJson(response, 404, { msg: "approval not found" });
       return callback(null, response);
     }
 
@@ -81,7 +81,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, OnboardRequestT
     console.log(profileId);
 
     if (!profileId) {
-      setResponseJson(response, 404, { err: "profile not found" });
+      setResponseJson(response, 404, { msg: "profile not found" });
       return callback(null, response);
     }
 
@@ -144,7 +144,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, OnboardRequestT
       getMetricFromRegistry<Counter>(metricsRegistry, OnboardMetrics.TRANSACTION_CONFLICT).inc(1);
       console.error(e);
 
-      setResponseJson(response, 409, { err: "something went wrong during onboarding" });
+      setResponseJson(response, 409, { msg: "something went wrong during onboarding" });
     }
 
     return callback(null, response);
@@ -153,7 +153,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, OnboardRequestT
   let newConfig = req.newConfig;
 
   if (!newConfig) {
-    setResponseJson(response, 400, { err: "missing newConfig" });
+    setResponseJson(response, 400, { msg: "missing newConfig" });
     return callback(null, response);
   }
 
@@ -166,7 +166,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, OnboardRequestT
 
   if (existingAlias) {
     getMetricFromRegistry<Counter>(metricsRegistry, OnboardMetrics.BUZZER_CONFLICT).inc({ buzzer: newConfigObj.buzzer }, 1);
-    setResponseJson(response, 409, { err: `A buzzer is already registered with the number ${newConfigObj.buzzer}` });
+    setResponseJson(response, 409, { msg: `A buzzer is already registered with the number ${newConfigObj.buzzer}` });
     return callback(null, response);
   }
 
@@ -174,7 +174,7 @@ export const handler: ServerlessFunctionSignature<TwilioContext, OnboardRequestT
 
   if (existingConfig) {
     getMetricFromRegistry<Counter>(metricsRegistry, OnboardMetrics.NAME_CONFLICT).inc({ name: newConfigObj.name }, 1);
-    setResponseJson(response, 409, { err: `A buzzer is already registered with the name ${newConfigObj.name}` });
+    setResponseJson(response, 409, { msg: `A buzzer is already registered with the name ${newConfigObj.name}` });
     return callback(null, response);
   }
 
diff --git a/packages/doorman-homeassistant/src/index.ts b/packages/doorman-homeassistant/src/index.ts
index 4c3c3fc..18b600b 100644
--- a/packages/doorman-homeassistant/src/index.ts
+++ b/packages/doorman-homeassistant/src/index.ts
@@ -14,7 +14,7 @@ app.get("/api/door/info", async (c) => {
   const door = c.req.query().door;
   if (!door) {
     return c.json({
-      err: "Must specify a door",
+      msg: "Must specify a door",
     }, 400);
   }
   
@@ -24,7 +24,7 @@ app.get("/api/door/info", async (c) => {
 
   if (!config) {
     return c.json({
-      err: "This buzzer is not registered properly",
+      msg: "This buzzer is not registered properly",
     }, 404);
   }
 
diff --git a/packages/doorman-ui/src/pages/DoorPage.tsx b/packages/doorman-ui/src/pages/DoorPage.tsx
index a6fd23c..d561e49 100644
--- a/packages/doorman-ui/src/pages/DoorPage.tsx
+++ b/packages/doorman-ui/src/pages/DoorPage.tsx
@@ -14,17 +14,17 @@ export async function loader({ params, request }: any) {
     return {};
   }
 
-  const response = await fetchUrlEncoded('/api/door/info', {
+  const res = await fetchUrlEncoded('/api/door/info', {
     door,
   }).then(res => res.json());
 
-  console.log(response);
+  console.log(res);
 
-  if (response.err) {
+  if (res.msg) {
     throw new Error("Not a valid door");
   }
 
-  return response as DoorResponse;
+  return res as DoorResponse;
 }
 
 export function DoorPage() {
diff --git a/packages/doorman-ui/src/pages/EditPage.tsx b/packages/doorman-ui/src/pages/EditPage.tsx
index c64873a..d9f734f 100644
--- a/packages/doorman-ui/src/pages/EditPage.tsx
+++ b/packages/doorman-ui/src/pages/EditPage.tsx
@@ -116,8 +116,8 @@ export const EditPage = ({ isOnboarding }: EditPageProps) => {
                   fetchUrlEncoded(apiRoute, body)
                     .then(res => res.json())
                     .then(res => {
-                      if (res.err) {
-                        addAlert("error", res.err);
+                      if (res.msg) {
+                        addAlert("error", res.msg);
                         return;
                       }
                       if (!isOnboarding) {
diff --git a/packages/doorman-ui/src/pages/RedirectPage.tsx b/packages/doorman-ui/src/pages/RedirectPage.tsx
index ae68a10..f34d136 100644
--- a/packages/doorman-ui/src/pages/RedirectPage.tsx
+++ b/packages/doorman-ui/src/pages/RedirectPage.tsx
@@ -18,7 +18,7 @@ export const RedirectPage = () => {
     fetch(state.apiRedirect + '?' + params.toString())
       .then(res => res.json())
       .then(res => {
-        if (res.err) {
+        if (res.msg) {
           console.error(res);
           window.location.href = '/';
           return;