@talkpilot/core-db 1.3.80 → 1.3.82

package/src/municipal/muniIssues/__tests__/muniIssues.setters.spec.ts

@@ -1,4 +1,8 @@
-import { createMuniIssue } from "../muniIssues.setters";
+import {
+  createMuniIssue,
+  setMuniIssueResponse,
+  updateMuniIssueStatus,
+} from "../muniIssues.setters";
 import {
   generateMuniIssueId,
   getMuniIssueById,
@@ -25,6 +29,18 @@ describe("muniIssues setters", () => {
       expect(saved?.createdAt).toBeInstanceOf(Date);
     });
 
+    it("should start the issue as in_review with updatedAt and no response", async () => {
+      const insertedId = await createMuniIssue(
+        createMuniIssueInput(),
+        generateMuniIssueId(),
+      );
+
+      const saved = await getMuniIssueById(insertedId.toString());
+      expect(saved?.status).toBe("in_review");
+      expect(saved?.updatedAt).toBeInstanceOf(Date);
+      expect(saved?.response).toBeUndefined();
+    });
+
     it("should default jira.syncStatus to created when not provided", async () => {
       const id = generateMuniIssueId();
 
@@ -91,4 +107,93 @@ describe("muniIssues setters", () => {
       expect(await getMuniIssueByJiraKey("CP-nope")).toBeNull();
     });
   });
+
+  describe("updateMuniIssueStatus", () => {
+    it("should change the status of an unanswered issue", async () => {
+      const input = createMuniIssueInput();
+      const id = (await createMuniIssue(input, generateMuniIssueId())).toString();
+
+      expect(await updateMuniIssueStatus(id, input.clientId, "in_progress")).toBe(true);
+      expect((await getMuniIssueById(id))?.status).toBe("in_progress");
+    });
+
+    it("should not touch another client's issue or a malformed id", async () => {
+      const id = (
+        await createMuniIssue(createMuniIssueInput(), generateMuniIssueId())
+      ).toString();
+
+      expect(await updateMuniIssueStatus(id, "other-client", "resolved")).toBe(false);
+      expect(await updateMuniIssueStatus("not-a-valid-id", "any", "resolved")).toBe(false);
+      expect((await getMuniIssueById(id))?.status).toBe("in_review");
+    });
+  });
+
+  describe("setMuniIssueResponse", () => {
+    it("should save the response and status", async () => {
+      const input = createMuniIssueInput();
+      const id = (await createMuniIssue(input, generateMuniIssueId())).toString();
+
+      const saved = await setMuniIssueResponse(id, input.clientId, {
+        status: "resolved",
+        text: "Fixed",
+        imageUrls: ["https://example.com/a.png", "https://example.com/b.png"],
+      });
+
+      expect(saved).toBe(true);
+      const issue = await getMuniIssueById(id);
+      expect(issue?.status).toBe("resolved");
+      expect(issue?.response).toMatchObject({
+        text: "Fixed",
+        imageUrls: ["https://example.com/a.png", "https://example.com/b.png"],
+      });
+      expect(issue?.response?.createdAt).toBeInstanceOf(Date);
+      expect(issue?.response?.updatedAt).toBeUndefined();
+    });
+
+    it("should let an answered issue be edited, keeping the first-sent date", async () => {
+      const input = createMuniIssueInput();
+      const id = (await createMuniIssue(input, generateMuniIssueId())).toString();
+      await setMuniIssueResponse(id, input.clientId, {
+        status: "in_review",
+        text: "First",
+        imageUrls: ["https://example.com/a.png"],
+      });
+      const firstSentAt = (await getMuniIssueById(id))?.response?.createdAt;
+
+      expect(
+        await setMuniIssueResponse(id, input.clientId, { status: "resolved", text: "Second" }),
+      ).toBe(true);
+      expect(await updateMuniIssueStatus(id, input.clientId, "no_issue_found")).toBe(true);
+
+      const issue = await getMuniIssueById(id);
+      expect(issue?.status).toBe("no_issue_found");
+      expect(issue?.response?.text).toBe("Second");
+      expect(issue?.response?.imageUrls).toEqual([]);
+      expect(issue?.response?.createdAt).toEqual(firstSentAt);
+      expect(issue?.response?.updatedAt).toBeInstanceOf(Date);
+    });
+
+    it("should store text starting with $ as plain text", async () => {
+      const input = createMuniIssueInput();
+      const id = (await createMuniIssue(input, generateMuniIssueId())).toString();
+
+      await setMuniIssueResponse(id, input.clientId, { status: "resolved", text: "$status" });
+
+      expect((await getMuniIssueById(id))?.response?.text).toBe("$status");
+    });
+
+    it("should not touch another client's issue or a malformed id", async () => {
+      const id = (
+        await createMuniIssue(createMuniIssueInput(), generateMuniIssueId())
+      ).toString();
+
+      expect(
+        await setMuniIssueResponse(id, "other-client", { status: "resolved", text: "X" }),
+      ).toBe(false);
+      expect(
+        await setMuniIssueResponse("not-a-valid-id", "any", { status: "resolved", text: "X" }),
+      ).toBe(false);
+      expect((await getMuniIssueById(id))?.response).toBeUndefined();
+    });
+  });
 });

package/src/municipal/muniIssues/__tests__/muniIssues.setup.spec.ts

@@ -28,6 +28,7 @@ describe("muniIssues setup", () => {
         expect.arrayContaining([
           "callSid_1",
           "createdAt_-1",
+          "clientId_1_createdAt_-1",
           "jira_key_unique",
         ]),
       );

package/src/municipal/muniIssues/index.ts

@@ -3,6 +3,8 @@ export * from "./muniIssues.setters";
 export * from "./muniIssues.setup";
 export {
   MUNI_ISSUES_COLLECTION,
+  MUNI_ISSUE_MAX_RESPONSE_IMAGES,
+  MUNI_ISSUE_STATUSES,
   MUNI_ISSUE_SYNC_STATUSES,
   MUNI_ISSUE_TEXT_MAX_LENGTH,
   MUNI_ISSUE_NAME_MAX_LENGTH,
@@ -13,5 +15,9 @@ export type {
   MuniIssueContent,
   MuniIssueJira,
   MuniIssueSyncStatus,
+  MuniIssueStatus,
+  MuniIssueResponse,
   CreateMuniIssueInput,
+  SetMuniIssueResponseInput,
+  FindMuniIssuesOptions,
 } from "./muniIssues.types";

package/src/municipal/muniIssues/muniIssues.constants.ts

@@ -4,6 +4,17 @@ export const MUNI_ISSUES_COLLECTION = "muniIssues";
 /** Sync state of the issue's Jira ticket at write time. */
 export const MUNI_ISSUE_SYNC_STATUSES = ["created", "failed"] as const;
 
+/** Customer-facing complaint status, set manually by the TalkPilot team. */
+export const MUNI_ISSUE_STATUSES = [
+  "in_review",
+  "in_progress",
+  "resolved",
+  "no_issue_found",
+] as const;
+
+/** Max images on a single TalkPilot response. */
+export const MUNI_ISSUE_MAX_RESPONSE_IMAGES = 5;
+
 /** Max length for the long free-text fields — well above the ~2000-char target. */
 export const MUNI_ISSUE_TEXT_MAX_LENGTH = 5000;
 

package/src/municipal/muniIssues/muniIssues.getters.ts

@@ -1,6 +1,6 @@
-import { Collection, ObjectId } from "mongodb";
+import { Collection, Filter, ObjectId } from "mongodb";
 import { getDb } from "../index";
-import { MuniIssue } from "./muniIssues.types";
+import { FindMuniIssuesOptions, MuniIssue } from "./muniIssues.types";
 import { MUNI_ISSUES_COLLECTION } from "./muniIssues.constants";
 
 export const getMuniIssuesCollection = (): Collection<MuniIssue> => {
@@ -22,3 +22,47 @@ export const getMuniIssueByJiraKey = async (
 ): Promise<MuniIssue | null> => {
   return getMuniIssuesCollection().findOne({ "jira.key": jiraKey });
 };
+
+// Tracked issues of one client. Issues without a status predate complaint tracking and are excluded.
+const clientIssuesFilter = (
+  clientId: string,
+  answered?: boolean,
+): Filter<MuniIssue> => ({
+  clientId,
+  status: { $exists: true },
+  ...(answered !== undefined && { response: { $exists: answered } }),
+});
+
+/** A client's tracked issues, newest first. `limit` 0 means no limit (Mongo). */
+export const findMuniIssuesByClient = async (
+  clientId: string,
+  { answered, skip = 0, limit = 0 }: FindMuniIssuesOptions = {},
+): Promise<MuniIssue[]> => {
+  return getMuniIssuesCollection()
+    .find(clientIssuesFilter(clientId, answered))
+    .sort({ createdAt: -1 })
+    .skip(skip)
+    .limit(limit)
+    .toArray();
+};
+
+export const countMuniIssuesByClient = async (
+  clientId: string,
+  { answered }: Pick<FindMuniIssuesOptions, "answered"> = {},
+): Promise<number> => {
+  return getMuniIssuesCollection().countDocuments(
+    clientIssuesFilter(clientId, answered),
+  );
+};
+
+/** One tracked issue, only if it belongs to the client. */
+export const getMuniIssueByIdForClient = async (
+  id: string,
+  clientId: string,
+): Promise<MuniIssue | null> => {
+  if (!ObjectId.isValid(id)) return null;
+  return getMuniIssuesCollection().findOne({
+    ...clientIssuesFilter(clientId),
+    _id: new ObjectId(id),
+  });
+};

package/src/municipal/muniIssues/muniIssues.schema.ts

@@ -1,5 +1,7 @@
 import type { Document, IndexDescription } from "mongodb";
 import {
+  MUNI_ISSUE_MAX_RESPONSE_IMAGES,
+  MUNI_ISSUE_STATUSES,
   MUNI_ISSUE_SYNC_STATUSES,
   MUNI_ISSUE_TEXT_MAX_LENGTH,
   MUNI_ISSUE_NAME_MAX_LENGTH,
@@ -55,12 +57,30 @@ export const muniIssuesJsonSchema: Document = {
       },
     },
     createdAt: { bsonType: "date" },
+    status: { enum: [...MUNI_ISSUE_STATUSES] },
+    response: {
+      bsonType: "object",
+      required: ["text", "createdAt"],
+      properties: {
+        text: { bsonType: "string", maxLength: MUNI_ISSUE_TEXT_MAX_LENGTH },
+        imageUrls: {
+          bsonType: "array",
+          maxItems: MUNI_ISSUE_MAX_RESPONSE_IMAGES,
+          items: { bsonType: "string" },
+        },
+        createdAt: { bsonType: "date" },
+        updatedAt: { bsonType: "date" },
+      },
+    },
+    updatedAt: { bsonType: "date" },
   },
 };
 
 export const muniIssuesIndexes: IndexDescription[] = [
   { key: { callSid: 1 }, name: "callSid_1" },
   { key: { createdAt: -1 }, name: "createdAt_-1" },
+  // Complaint list: one municipality's issues, newest first.
+  { key: { clientId: 1, createdAt: -1 }, name: "clientId_1_createdAt_-1" },
   // Unique: one stored document per Jira ticket, guarding against duplicate writes.
   { key: { "jira.key": 1 }, name: "jira_key_unique", unique: true },
 ];

package/src/municipal/muniIssues/muniIssues.setters.ts

@@ -1,5 +1,10 @@
-import { ObjectId } from "mongodb";
-import { CreateMuniIssueInput, MuniIssue } from "./muniIssues.types";
+import { Filter, ObjectId } from "mongodb";
+import {
+  CreateMuniIssueInput,
+  MuniIssue,
+  MuniIssueStatus,
+  SetMuniIssueResponseInput,
+} from "./muniIssues.types";
 import { getMuniIssuesCollection } from "./muniIssues.getters";
 import { ensureMuniIssuesCollectionOnce } from "./muniIssues.setup";
 
@@ -10,6 +15,7 @@ export const createMuniIssue = async (
 ): Promise<ObjectId> => {
   await ensureMuniIssuesCollectionOnce();
 
+  const now = new Date();
   const issue: MuniIssue = {
     _id: id,
     ...issueData,
@@ -17,8 +23,71 @@ export const createMuniIssue = async (
       ...issueData.jira,
       syncStatus: issueData.jira.syncStatus ?? "created",
     },
-    createdAt: new Date(),
+    status: "in_review",
+    createdAt: now,
+    updatedAt: now,
   };
   const { insertedId } = await getMuniIssuesCollection().insertOne(issue);
   return insertedId;
 };
+
+// An issue of this client that is tracked (has a status); answered issues stay editable.
+const trackedIssueFilter = (id: string, clientId: string): Filter<MuniIssue> => ({
+  _id: new ObjectId(id),
+  clientId,
+  status: { $exists: true },
+});
+
+/** Changes the status of an issue, answered or not. Returns false if none matched. */
+export const updateMuniIssueStatus = async (
+  id: string,
+  clientId: string,
+  status: MuniIssueStatus,
+): Promise<boolean> => {
+  if (!ObjectId.isValid(id)) return false;
+  await ensureMuniIssuesCollectionOnce();
+
+  const { matchedCount } = await getMuniIssuesCollection().updateOne(
+    trackedIssueFilter(id, clientId),
+    { $set: { status, updatedAt: new Date() } },
+  );
+  return matchedCount > 0;
+};
+
+/**
+ * Saves the TalkPilot response, or replaces an existing one (text and images).
+ * Keeps the original `response.createdAt` and sets `response.updatedAt` on an edit.
+ * Returns false if none matched.
+ */
+export const setMuniIssueResponse = async (
+  id: string,
+  clientId: string,
+  { status, text, imageUrls }: SetMuniIssueResponseInput,
+): Promise<boolean> => {
+  if (!ObjectId.isValid(id)) return false;
+  await ensureMuniIssuesCollectionOnce();
+
+  const now = new Date();
+  // Pipeline update, so the first-sent date can be read from the stored response.
+  // $literal: user text starting with "$" must not be read as a field path.
+  const { matchedCount } = await getMuniIssuesCollection().updateOne(
+    trackedIssueFilter(id, clientId),
+    [
+      {
+        $set: {
+          status: { $literal: status },
+          updatedAt: now,
+          response: {
+            text: { $literal: text },
+            imageUrls: { $literal: imageUrls ?? [] },
+            createdAt: { $ifNull: ["$response.createdAt", now] },
+            updatedAt: {
+              $cond: [{ $ifNull: ["$response.createdAt", false] }, now, "$$REMOVE"],
+            },
+          },
+        },
+      },
+    ],
+  );
+  return matchedCount > 0;
+};

package/src/municipal/muniIssues/muniIssues.types.ts

@@ -1,10 +1,23 @@
 import { ObjectId } from "mongodb";
 
 import { CityName } from "../utils/types";
-import { MUNI_ISSUE_SYNC_STATUSES } from "./muniIssues.constants";
+import {
+  MUNI_ISSUE_STATUSES,
+  MUNI_ISSUE_SYNC_STATUSES,
+} from "./muniIssues.constants";
 
 export type MuniIssueSyncStatus = (typeof MUNI_ISSUE_SYNC_STATUSES)[number];
 
+export type MuniIssueStatus = (typeof MUNI_ISSUE_STATUSES)[number];
+
+/** The TalkPilot response. Editable: `createdAt` is when it was first sent, `updatedAt` the last edit. */
+export type MuniIssueResponse = {
+  text: string;
+  imageUrls?: string[];
+  createdAt: Date;
+  updatedAt?: Date;
+};
+
 export type MuniIssueContent = {
   issuerName: string;
   issuerRole: string;
@@ -25,13 +38,32 @@ export type MuniIssue = {
   issueContent: MuniIssueContent;
   jira: MuniIssueJira;
   createdAt: Date;
+  // Complaint tracking (CP-1888). Optional: issues created before it have none
+  // of these, which is how they are kept out of tracking.
+  status?: MuniIssueStatus;
+  response?: MuniIssueResponse;
+  updatedAt?: Date;
 };
 
-// _id/createdAt are set on insert; jira.syncStatus defaults to "created"
-// because the issue is only stored after its Jira ticket exists (CP-1115).
+// _id/createdAt/status/updatedAt are set on insert, response only later (setMuniIssueResponse);
+// jira.syncStatus defaults to "created" because the issue is only stored after its Jira
+// ticket exists (CP-1115).
 export type CreateMuniIssueInput = Omit<
   MuniIssue,
-  "_id" | "createdAt" | "jira"
+  "_id" | "createdAt" | "jira" | "status" | "response" | "updatedAt"
 > & {
   jira: { key: string; syncStatus?: MuniIssueSyncStatus };
 };
+
+export type SetMuniIssueResponseInput = {
+  status: MuniIssueStatus;
+  text: string;
+  imageUrls?: string[];
+};
+
+/** `answered` filters by whether a TalkPilot response exists; omit it for all issues. */
+export type FindMuniIssuesOptions = {
+  answered?: boolean;
+  skip?: number;
+  limit?: number;
+};

package/src/municipal/residents/__tests__/residents.spec.ts

@@ -0,0 +1,92 @@
+import { ObjectId } from "mongodb";
+import {
+  findResidentsByCityAndPhone,
+  getResidentByCityAndIdNumber,
+  getResidentByCityPhoneAndIdNumber,
+  getResidentById,
+  getResidentsCollection,
+} from "../residents.getters";
+import { createResident } from "../../../test-utils/factories/municipal/residents";
+
+const CITY = "tests" as const;
+const OTHER_CITY = "eilat" as const;
+
+describe("residents getters", () => {
+  it("getResidentById returns the document, or null for a missing / invalid id", async () => {
+    const resident = createResident({ cityName: CITY });
+    await getResidentsCollection().insertOne(resident);
+
+    const found = await getResidentById(resident._id.toHexString());
+    expect(found?._id.equals(resident._id)).toBe(true);
+    expect(found?.idNumber).toBe(resident.idNumber);
+
+    expect(await getResidentById("not-an-id")).toBeNull();
+    expect(await getResidentById(new ObjectId().toHexString())).toBeNull();
+  });
+
+  it("findResidentsByCityAndPhone returns only matching city + phone", async () => {
+    const phone = "+972501111111";
+    const match = createResident({ cityName: CITY, phoneNumber: phone });
+    const samePhoneOtherCity = createResident({
+      cityName: OTHER_CITY,
+      phoneNumber: phone,
+    });
+    const sameCityOtherPhone = createResident({
+      cityName: CITY,
+      phoneNumber: "+972502222222",
+    });
+    await getResidentsCollection().insertMany([
+      match,
+      samePhoneOtherCity,
+      sameCityOtherPhone,
+    ]);
+
+    const found = await findResidentsByCityAndPhone(CITY, phone);
+    expect(found).toHaveLength(1);
+    expect(found[0]._id.equals(match._id)).toBe(true);
+  });
+
+  it("allows a document with only _id", async () => {
+    const resident = { _id: new ObjectId() };
+    await getResidentsCollection().insertOne(resident);
+
+    const found = await getResidentById(resident._id.toHexString());
+    expect(found?._id.equals(resident._id)).toBe(true);
+    expect(found?.firstName).toBeUndefined();
+  });
+
+  it("getResidentByCityAndIdNumber is scoped to the city", async () => {
+    const idNumber = "123456789";
+    const match = createResident({ cityName: CITY, idNumber });
+    const otherCity = createResident({ cityName: OTHER_CITY, idNumber: "987654321" });
+    await getResidentsCollection().insertMany([match, otherCity]);
+
+    const found = await getResidentByCityAndIdNumber(CITY, idNumber);
+    expect(found?._id.equals(match._id)).toBe(true);
+
+    expect(await getResidentByCityAndIdNumber(OTHER_CITY, idNumber)).toBeNull();
+    expect(await getResidentByCityAndIdNumber(CITY, "000000000")).toBeNull();
+  });
+
+  it("getResidentByCityPhoneAndIdNumber requires both phone and id in that city", async () => {
+    const phone = "+972501111111";
+    const idNumber = "201111111";
+    const match = createResident({ cityName: CITY, phoneNumber: phone, idNumber });
+    const samePhoneWrongId = createResident({
+      cityName: CITY,
+      phoneNumber: phone,
+      idNumber: "209999999",
+    });
+    await getResidentsCollection().insertMany([match, samePhoneWrongId]);
+
+    const found = await getResidentByCityPhoneAndIdNumber(CITY, phone, idNumber);
+    expect(found?._id.equals(match._id)).toBe(true);
+
+    expect(
+      await getResidentByCityPhoneAndIdNumber(CITY, phone, "000000000"),
+    ).toBeNull();
+    expect(
+      await getResidentByCityPhoneAndIdNumber(OTHER_CITY, phone, idNumber),
+    ).toBeNull();
+  });
+});

package/src/municipal/residents/index.ts

@@ -0,0 +1,4 @@
+export * from "./residents.getters";
+export { RESIDENTS_COLLECTION } from "./residents.constants";
+export { residentsJsonSchema, residentsIndexes } from "./residents.schema";
+export type { Resident, ResidentDoc } from "./residents.types";

package/src/municipal/residents/residents.constants.ts

@@ -0,0 +1,2 @@
+/** Mongo collection name for municipal resident records. */
+export const RESIDENTS_COLLECTION = "residents";

package/src/municipal/residents/residents.getters.ts

@@ -0,0 +1,42 @@
+import { Collection, Filter } from "mongodb";
+import { CityName, getDb, ObjectId, Resident } from "../index";
+import { RESIDENTS_COLLECTION } from "./residents.constants";
+
+export const getResidentsCollection = (): Collection<Resident> => {
+  return getDb().collection<Resident>(RESIDENTS_COLLECTION);
+};
+
+export const findResidents = async (
+  filter: Filter<Resident> = {},
+): Promise<Resident[]> => {
+  return getResidentsCollection().find(filter).toArray();
+};
+
+export const getResidentById = async (
+  residentId: string,
+): Promise<Resident | null> => {
+  if (!ObjectId.isValid(residentId)) return null;
+  return getResidentsCollection().findOne({ _id: new ObjectId(residentId) });
+};
+
+export const findResidentsByCityAndPhone = async (
+  cityName: CityName,
+  phoneNumber: string,
+): Promise<Resident[]> => {
+  return getResidentsCollection().find({ cityName, phoneNumber }).toArray();
+};
+
+export const getResidentByCityAndIdNumber = async (
+  cityName: CityName,
+  idNumber: string,
+): Promise<Resident | null> => {
+  return getResidentsCollection().findOne({ cityName, idNumber });
+};
+
+export const getResidentByCityPhoneAndIdNumber = async (
+  cityName: CityName,
+  phoneNumber: string,
+  idNumber: string,
+): Promise<Resident | null> => {
+  return getResidentsCollection().findOne({ cityName, phoneNumber, idNumber });
+};

package/src/municipal/residents/residents.schema.ts

@@ -0,0 +1,28 @@
+import type { Document, IndexDescription } from "mongodb";
+
+export const residentsJsonSchema: Document = {
+  bsonType: "object",
+  properties: {
+    cityName: { bsonType: "string" },
+    firstName: { bsonType: "string" },
+    lastName: { bsonType: "string" },
+    idNumber: { bsonType: "string" },
+    phoneNumber: { bsonType: "string" },
+    address: { bsonType: "string" },
+    createdAt: { bsonType: "date" },
+    updatedAt: { bsonType: "date" },
+  },
+};
+
+export const residentsIndexes: IndexDescription[] = [
+  { key: { cityName: 1, phoneNumber: 1 }, name: "cityName_1_phoneNumber_1" },
+  {
+    key: { cityName: 1, idNumber: 1 },
+    name: "cityName_1_idNumber_unique",
+    unique: true,
+    partialFilterExpression: {
+      cityName: { $type: "string" },
+      idNumber: { $type: "string" },
+    },
+  },
+];

package/src/municipal/residents/residents.types.ts

@@ -0,0 +1,17 @@
+import { ObjectId, WithId } from "mongodb";
+
+import { CityName } from "../utils/types";
+
+export type Resident = {
+  _id: ObjectId;
+  createdAt?: Date;
+  updatedAt?: Date;
+  cityName?: CityName;
+  firstName?: string;
+  lastName?: string;
+  idNumber?: string;
+  phoneNumber?: string;
+  address?: string;
+};
+
+export type ResidentDoc = WithId<Resident>;

package/src/talkpilot/calls/__tests__/calls.spec.ts

@@ -11,7 +11,6 @@ import {
   pushToolExecution,
   getToolExecutionsByCallSid,
   updateCallStatusByCallSid,
-  setCallEndedBy,
 } from "../calls.getters";
 import type { ToolExecution, ToolExecutionStep } from "../calls.types";
 import { ObjectId } from "mongodb";
@@ -75,53 +74,6 @@ describe("db.calls", () => {
     });
   });
 
-  describe("setCallEndedBy()", () => {
-    it("should set endedBy when it is not set yet", async () => {
-      const call = createOutGoingCallDoc();
-      await createCallDoc(call);
-
-      const written = await setCallEndedBy(call.callSid, "user");
-
-      expect(written).toBe(true);
-      expect(await getCallByCallSid(call.callSid)).toMatchObject({
-        endedBy: "user",
-      });
-    });
-
-    it("should keep the first value when called again", async () => {
-      const call = createOutGoingCallDoc();
-      await createCallDoc(call);
-
-      await setCallEndedBy(call.callSid, "user");
-      const written = await setCallEndedBy(call.callSid, "agent");
-
-      expect(written).toBe(false);
-      expect(await getCallByCallSid(call.callSid)).toMatchObject({
-        endedBy: "user",
-      });
-    });
-
-    it("should let only one of two concurrent writers win", async () => {
-      const call = createOutGoingCallDoc();
-      await createCallDoc(call);
-
-      const results = await Promise.all([
-        setCallEndedBy(call.callSid, "user"),
-        setCallEndedBy(call.callSid, "humanAgent"),
-      ]);
-
-      expect(results.filter(Boolean)).toHaveLength(1);
-      const stored = await getCallByCallSid(call.callSid);
-      expect(stored?.endedBy).toBe(results[0] ? "user" : "humanAgent");
-    });
-
-    it("should return false when the call does not exist", async () => {
-      await expect(setCallEndedBy("nonexistent-sid", "system")).resolves.toBe(
-        false,
-      );
-    });
-  });
-
   describe("findCallsByQuery()", () => {
     it("should find calls by status with limit and sort", async () => {
       const clientId = "client123";