From 55d35e150ce00d8f117e09c04b560864c1af1041 Mon Sep 17 00:00:00 2001
From: Avraham Sakal <avraham@torahanytime.com>
Date: Mon, 28 Jul 2025 22:15:40 -0400
Subject: [PATCH] slight refactor of delete-by-id implementation

---
 pages/chat/facts.ts | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/pages/chat/facts.ts b/pages/chat/facts.ts
index f11b45e..1637de1 100644
--- a/pages/chat/facts.ts
+++ b/pages/chat/facts.ts
@@ -25,12 +25,11 @@ export const facts = router({
         },
     )
     .mutation(async ({ input: { factId } }) => {
-      const deletedFact = db.data.facts.find((fact) => fact.id === factId);
-      if (!deletedFact) throw new Error("Fact not found");
-      db.data.facts.splice(
-        db.data.facts.findIndex((fact) => fact.id === factId),
-        1,
+      const deletedFactId = db.data.facts.findIndex(
+        (fact) => fact.id === factId,
       );
+      if (deletedFactId === -1) throw new Error("Fact not found");
+      db.data.facts.splice(deletedFactId, 1);
       db.write();
       return { ok: true };
     }),