chore: minor bug fixes

apps/api/src/controllers/leo/bureau-of-firearms-controller.ts

@@ -1,4 +1,4 @@
-import { WhitelistStatus } from "@prisma/client";
+import { Prisma, WhitelistStatus } from "@prisma/client";
 import { BodyParams, PathParams, QueryParams, UseBeforeEach } from "@tsed/common";
 import { Controller } from "@tsed/di";
 import { BadRequest, NotFound } from "@tsed/exceptions";
@@ -14,6 +14,11 @@ import type * as APITypes from "@snailycad/types/api";
 import { IsFeatureEnabled, Feature } from "middlewares/is-enabled";
 import { citizenInclude } from "controllers/citizen/CitizenController";
 
+const includes = Prisma.validator<Prisma.WeaponSelect>()({
+  ...citizenInclude.weapons.include,
+  citizen: true,
+});
+
 @Controller("/leo/bureau-of-firearms")
 @UseBeforeEach(IsAuth)
 @ContentType("application/json")
@@ -32,10 +37,7 @@ export class BureauOfFirearmsController {
       prisma.weapon.count({ where: { bofStatus: { not: null } } }),
       prisma.weapon.findMany({
         where: { bofStatus: { not: null } },
-        include: {
-          ...citizenInclude.weapons.include,
-          citizen: true,
-        },
+        include: includes,
         orderBy: { createdAt: "desc" },
         take: includeAll ? undefined : 35,
         skip: includeAll ? undefined : skip,
@@ -66,10 +68,7 @@ export class BureauOfFirearmsController {
     const updated = await prisma.weapon.update({
       where: { id: weapon.id },
       data: { bofStatus },
-      include: {
-        ...citizenInclude.weapons.include,
-        citizen: true,
-      },
+      include: includes,
     });
 
     return updated;

apps/api/src/lib/leo/records/upsert-record.ts

@@ -174,6 +174,12 @@ export async function upsertRecord(options: UpsertRecordOptions) {
   );
 
   if (Object.keys(errors).length >= 1) {
+    captureException({
+      name: "InvalidViolations",
+      message: "Invalid violations appended to record",
+      violations: JSON.stringify(errors),
+    });
+
     await prisma.record.delete({ where: { id: ticket.id } });
     throw new ExtendedBadRequest(errors);
   }

apps/client/sentry.client.config.ts

@@ -16,6 +16,9 @@ init({
     "Hydration failed because the initial UI does not match what was rendered on the server.",
     "Cannot have two HTML5 backends at the same time.",
     /Cannot resolve a DOM point from Slate point:/gi,
+    /Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node./gi,
+    "Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.",
+    "The object can not be found here.",
   ],
   integrations(integrations) {
     return [...integrations, new BrowserTracing()];

apps/client/sentry.edge.config.ts

@@ -15,5 +15,8 @@ init({
     "Hydration failed because the initial UI does not match what was rendered on the server.",
     "Cannot have two HTML5 backends at the same time.",
     /Cannot resolve a DOM point from Slate point:/gi,
+    /Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node./gi,
+    "Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.",
+    "The object can not be found here.",
   ],
 });

apps/client/sentry.server.config.ts

@@ -16,5 +16,8 @@ init({
     "Hydration failed because the initial UI does not match what was rendered on the server.",
     "Cannot have two HTML5 backends at the same time.",
     /Cannot resolve a DOM point from Slate point:/gi,
+    /Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node./gi,
+    "Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.",
+    "The object can not be found here.",
   ],
 });