@talkpilot/core-db 1.3.84 → 1.3.86

package/src/municipal/departmentsSubjects/departmentsSubjects.types.ts

@@ -1,73 +1,73 @@
-import { ObjectId, WithId } from "mongodb";
-import { Request } from "express";
-import { ClientConfigDoc, Products } from "../../talkpilot";
-import { CityName } from "../utils/types";
-
-export type DepartmentSubject = {
-  _id: ObjectId;
-  createdAt: Date;
-  updatedAt: Date;
-  subjectName: string; // e.g. "אחזקה- איטום"
-  subject_id: string; // e.g. "6121"
-  sub_subject_name: string; // e.g. "איטום"
-  sub_subject_id: string; // e.g. "6123"
-  sub_subject_name2?: string; // e.g. "ניקיון באגם" (level 3 - grandchild)
-  sub_subject_id2?: string; // e.g. "2898" (level 3 - grandchild)
-  descriptions: string[]; // array of guideline / example descriptions
-  embedding_text?: string; // the text used to create the embedding
-  embedding?: number[]; // length should match embedding_dims (e.g. 1536)
-  embedding_model?: string; // e.g. "text-embedding-3-small"
-  embedding_dims?: number; // e.g. 1536
-  cityName: CityName;
-  guidelines?: string | null;
-  instructions?: Instruction[];
-  communications?: Communication[];
-  sla?: number;
-  isActive?: boolean;
-};
-
-export interface Instruction {
-  id: number;
-  instruction: string;
-}
-
-export interface Communication {
-  id: number;
-  type: CommunicationType;
-  message_text?: string;
-  template_id?: string;
-  to?: string;
-  is_deleted?: boolean;
-}
-
-export type DepartmentSubjectDoc = WithId<DepartmentSubject>;
-
-/**
- * Vector search result from MongoDB
- */
-export type VectorSearchResult = {
-  subjectName: string;
-  subject_id: number;
-  sub_subject_id: string;
-  sub_subject_id2?: string | null;
-  sub_subject_name: string;
-  sub_subject_name2?: string | null;
-  descriptions?: string[];
-  guidelines?: string | null;
-  instructions?: string | null;
-  score?: number;
-};
-
-export type CommunicationType = "free_text_sms" | "upload_url_sms";
-
-/**
- * Extended Express Request type with client configuration
- */
-export type RequestWithClientConfig = Request & {
-  uid: string;
-  clientConfig: ClientConfigDoc<Products>;
-  isGuest: boolean;
-  claims: Record<string, unknown>;
-  crmName?: string;
-  callSid?: string;
-};
+import { ObjectId, WithId } from "mongodb";
+import { Request } from "express";
+import { ClientConfigDoc, Products } from "../../talkpilot";
+import { CityName } from "../utils/types";
+
+export type DepartmentSubject = {
+  _id: ObjectId;
+  createdAt: Date;
+  updatedAt: Date;
+  subjectName: string; // e.g. "אחזקה- איטום"
+  subject_id: string; // e.g. "6121"
+  sub_subject_name: string; // e.g. "איטום"
+  sub_subject_id: string; // e.g. "6123"
+  sub_subject_name2?: string; // e.g. "ניקיון באגם" (level 3 - grandchild)
+  sub_subject_id2?: string; // e.g. "2898" (level 3 - grandchild)
+  descriptions: string[]; // array of guideline / example descriptions
+  embedding_text?: string; // the text used to create the embedding
+  embedding?: number[]; // length should match embedding_dims (e.g. 1536)
+  embedding_model?: string; // e.g. "text-embedding-3-small"
+  embedding_dims?: number; // e.g. 1536
+  cityName: CityName;
+  guidelines?: string | null;
+  instructions?: Instruction[];
+  communications?: Communication[];
+  sla?: number;
+  isActive?: boolean;
+};
+
+export interface Instruction {
+  id: number;
+  instruction: string;
+}
+
+export interface Communication {
+  id: number;
+  type: CommunicationType;
+  message_text?: string;
+  template_id?: string;
+  to?: string;
+  is_deleted?: boolean;
+}
+
+export type DepartmentSubjectDoc = WithId<DepartmentSubject>;
+
+/**
+ * Vector search result from MongoDB
+ */
+export type VectorSearchResult = {
+  subjectName: string;
+  subject_id: number;
+  sub_subject_id: string;
+  sub_subject_id2?: string | null;
+  sub_subject_name: string;
+  sub_subject_name2?: string | null;
+  descriptions?: string[];
+  guidelines?: string | null;
+  instructions?: string | null;
+  score?: number;
+};
+
+export type CommunicationType = "free_text_sms" | "upload_url_sms";
+
+/**
+ * Extended Express Request type with client configuration
+ */
+export type RequestWithClientConfig = Request & {
+  uid: string;
+  clientConfig: ClientConfigDoc<Products>;
+  isGuest: boolean;
+  claims: Record<string, unknown>;
+  crmName?: string;
+  callSid?: string;
+};

package/src/municipal/departmentsSubjects/index.ts

@@ -1,9 +1,9 @@
-export * from "./departmentsSubjects.getters";
-export type {
-  DepartmentSubject,
-  DepartmentSubjectDoc,
-  VectorSearchResult,
-  RequestWithClientConfig,
-  Communication,
-  Instruction,
-} from "./departmentsSubjects.types";
+export * from "./departmentsSubjects.getters";
+export type {
+  DepartmentSubject,
+  DepartmentSubjectDoc,
+  VectorSearchResult,
+  RequestWithClientConfig,
+  Communication,
+  Instruction,
+} from "./departmentsSubjects.types";

package/src/municipal/index.ts

@@ -1,23 +1,23 @@
-import { Db, ObjectId as MongoObjectId } from "mongodb";
-
-export * from "./cities";
-export * from "./streets";
-export * from "./departmentsSubjects";
-export * from "./tickets";
-export * from "./systemInstructions";
-export * from "./muniIssues";
-export * from "./residents";
-export * from "./utils/types";
-export type { CityName } from "./utils/types";
-export { municipalDataMongodbClient } from "./mongodb-client";
-
-let db: Db;
-export const setDb = (d: Db) => {
-  db = d;
-};
-
-export const getDb = (): Db => {
-  if (!db) throw new Error("Municipal Data DB not initialised");
-  return db;
-};
-export const ObjectId = MongoObjectId;
+import { Db, ObjectId as MongoObjectId } from "mongodb";
+
+export * from "./cities";
+export * from "./streets";
+export * from "./departmentsSubjects";
+export * from "./tickets";
+export * from "./systemInstructions";
+export * from "./muniIssues";
+export * from "./residents";
+export * from "./utils/types";
+export type { CityName } from "./utils/types";
+export { municipalDataMongodbClient } from "./mongodb-client";
+
+let db: Db;
+export const setDb = (d: Db) => {
+  db = d;
+};
+
+export const getDb = (): Db => {
+  if (!db) throw new Error("Municipal Data DB not initialised");
+  return db;
+};
+export const ObjectId = MongoObjectId;

package/src/municipal/mongodb-client.ts

@@ -1,61 +1,61 @@
-import { MongoClient, Db } from "mongodb";
-import { setDb } from "./index";
-import { validateConfig, validateMongoUri } from "../utils/validation";
-
-class MunicipalDataMongoDBClient {
-  private client: MongoClient | null = null;
-  private db: Db | null = null;
-  private readonly defaultDbName = "municipal-data";
-
-  async connect(uri?: string, dbName?: string): Promise<void> {
-    if (this.client) {
-      return;
-    }
-
-    const mongodbUri = uri || process.env.MONGO_URI || process.env.MONGODB_URI;
-    validateConfig("MONGO_URI", mongodbUri);
-    validateMongoUri(mongodbUri!);
-
-    try {
-      this.client = new MongoClient(mongodbUri!);
-      await this.client.connect();
-      const targetDbName =
-        dbName || process.env.MUNICIPAL_DB_NAME || this.defaultDbName;
-      this.db = this.client.db(targetDbName);
-      setDb(this.db);
-      console.info(`[core-db] Municipal MongoDB connected: ${targetDbName}`);
-    } catch (error) {
-      console.error("[core-db] Municipal connection failed", error);
-      throw error;
-    }
-  }
-
-  async disconnect(): Promise<void> {
-    if (this.client) {
-      try {
-        await this.client.close();
-        this.client = null;
-        this.db = null;
-        console.info("Municipal Data MongoDB disconnected successfully");
-      } catch (error) {
-        console.error("[core-db] Municipal disconnection failed", error);
-        throw error;
-      }
-    }
-  }
-
-  getDb(): Db {
-    if (!this.db) {
-      throw new Error(
-        "Municipal Data database not initialized. Call connect() first.",
-      );
-    }
-    return this.db;
-  }
-
-  isConnected(): boolean {
-    return this.client !== null && this.client !== undefined;
-  }
-}
-
-export const municipalDataMongodbClient = new MunicipalDataMongoDBClient();
+import { MongoClient, Db } from "mongodb";
+import { setDb } from "./index";
+import { validateConfig, validateMongoUri } from "../utils/validation";
+
+class MunicipalDataMongoDBClient {
+  private client: MongoClient | null = null;
+  private db: Db | null = null;
+  private readonly defaultDbName = "municipal-data";
+
+  async connect(uri?: string, dbName?: string): Promise<void> {
+    if (this.client) {
+      return;
+    }
+
+    const mongodbUri = uri || process.env.MONGO_URI || process.env.MONGODB_URI;
+    validateConfig("MONGO_URI", mongodbUri);
+    validateMongoUri(mongodbUri!);
+
+    try {
+      this.client = new MongoClient(mongodbUri!);
+      await this.client.connect();
+      const targetDbName =
+        dbName || process.env.MUNICIPAL_DB_NAME || this.defaultDbName;
+      this.db = this.client.db(targetDbName);
+      setDb(this.db);
+      console.info(`[core-db] Municipal MongoDB connected: ${targetDbName}`);
+    } catch (error) {
+      console.error("[core-db] Municipal connection failed", error);
+      throw error;
+    }
+  }
+
+  async disconnect(): Promise<void> {
+    if (this.client) {
+      try {
+        await this.client.close();
+        this.client = null;
+        this.db = null;
+        console.info("Municipal Data MongoDB disconnected successfully");
+      } catch (error) {
+        console.error("[core-db] Municipal disconnection failed", error);
+        throw error;
+      }
+    }
+  }
+
+  getDb(): Db {
+    if (!this.db) {
+      throw new Error(
+        "Municipal Data database not initialized. Call connect() first.",
+      );
+    }
+    return this.db;
+  }
+
+  isConnected(): boolean {
+    return this.client !== null && this.client !== undefined;
+  }
+}
+
+export const municipalDataMongodbClient = new MunicipalDataMongoDBClient();

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

@@ -1,87 +1,87 @@
-import { ObjectId } from "mongodb";
-import { createMuniIssue, setMuniIssueResponse } from "../muniIssues.setters";
-import {
-  countMuniIssuesByClient,
-  findMuniIssuesByClient,
-  generateMuniIssueId,
-  getMuniIssueByIdForClient,
-  getMuniIssuesCollection,
-} from "../muniIssues.getters";
-import { createMuniIssueInput } from "../../../test-utils/factories";
-
-const CLIENT = "client-a";
-
-const createIssue = async (clientId = CLIENT) =>
-  (await createMuniIssue(createMuniIssueInput({ clientId }), generateMuniIssueId())).toString();
-
-describe("muniIssues getters (complaint tracking)", () => {
-  describe("findMuniIssuesByClient / countMuniIssuesByClient", () => {
-    it("should return only the client's tracked issues, newest first", async () => {
-      const older = await createIssue();
-      const newer = await createIssue();
-      // Both can land in the same millisecond; make the order explicit.
-      await getMuniIssuesCollection().updateOne(
-        { _id: new ObjectId(older) },
-        { $set: { createdAt: new Date(Date.now() - 60_000) } },
-      );
-      await createIssue("client-b");
-      // An issue created before complaint tracking has no status.
-      await getMuniIssuesCollection().insertOne({
-        ...createMuniIssueInput({ clientId: CLIENT }),
-        _id: generateMuniIssueId(),
-        jira: { key: "CP-legacy", syncStatus: "created" },
-        createdAt: new Date(),
-      });
-
-      const issues = await findMuniIssuesByClient(CLIENT);
-
-      expect(issues.map((i) => i._id.toString())).toEqual([newer, older]);
-      expect(await countMuniIssuesByClient(CLIENT)).toBe(2);
-    });
-
-    it("should filter by answered and apply skip/limit", async () => {
-      const answered = await createIssue();
-      await createIssue();
-      await createIssue();
-      await setMuniIssueResponse(answered, CLIENT, { status: "resolved", text: "Done" });
-
-      expect(await countMuniIssuesByClient(CLIENT, { answered: true })).toBe(1);
-      expect(await countMuniIssuesByClient(CLIENT, { answered: false })).toBe(2);
-      expect(
-        (await findMuniIssuesByClient(CLIENT, { answered: true }))[0]._id.toString(),
-      ).toBe(answered);
-      expect(await findMuniIssuesByClient(CLIENT, { skip: 1, limit: 1 })).toHaveLength(1);
-    });
-
-    it("should treat a response with only images as answered and a cleared response as waiting", async () => {
-      const imagesOnly = await createIssue();
-      const cleared = await createIssue();
-      const textOnly = await createIssue();
-      await setMuniIssueResponse(imagesOnly, CLIENT, {
-        status: "resolved",
-        text: "",
-        imageUrls: ["https://example.com/a.png"],
-      });
-      await setMuniIssueResponse(cleared, CLIENT, { status: "resolved", text: "Done" });
-      await setMuniIssueResponse(cleared, CLIENT, { status: "resolved", text: "", imageUrls: [] });
-      await setMuniIssueResponse(textOnly, CLIENT, { status: "resolved", text: "Done" });
-
-      const answered = await findMuniIssuesByClient(CLIENT, { answered: true });
-      const waiting = await findMuniIssuesByClient(CLIENT, { answered: false });
-
-      expect(answered.map((i) => i._id.toString()).sort()).toEqual([imagesOnly, textOnly].sort());
-      expect(waiting.map((i) => i._id.toString())).toEqual([cleared]);
-      expect(await countMuniIssuesByClient(CLIENT, { answered: false })).toBe(1);
-    });
-  });
-
-  describe("getMuniIssueByIdForClient", () => {
-    it("should return the issue only for its own client", async () => {
-      const id = await createIssue();
-
-      expect((await getMuniIssueByIdForClient(id, CLIENT))?._id.toString()).toBe(id);
-      expect(await getMuniIssueByIdForClient(id, "client-b")).toBeNull();
-      expect(await getMuniIssueByIdForClient("not-a-valid-id", CLIENT)).toBeNull();
-    });
-  });
-});
+import { ObjectId } from "mongodb";
+import { createMuniIssue, setMuniIssueResponse } from "../muniIssues.setters";
+import {
+  countMuniIssuesByClient,
+  findMuniIssuesByClient,
+  generateMuniIssueId,
+  getMuniIssueByIdForClient,
+  getMuniIssuesCollection,
+} from "../muniIssues.getters";
+import { createMuniIssueInput } from "../../../test-utils/factories";
+
+const CLIENT = "client-a";
+
+const createIssue = async (clientId = CLIENT) =>
+  (await createMuniIssue(createMuniIssueInput({ clientId }), generateMuniIssueId())).toString();
+
+describe("muniIssues getters (complaint tracking)", () => {
+  describe("findMuniIssuesByClient / countMuniIssuesByClient", () => {
+    it("should return only the client's tracked issues, newest first", async () => {
+      const older = await createIssue();
+      const newer = await createIssue();
+      // Both can land in the same millisecond; make the order explicit.
+      await getMuniIssuesCollection().updateOne(
+        { _id: new ObjectId(older) },
+        { $set: { createdAt: new Date(Date.now() - 60_000) } },
+      );
+      await createIssue("client-b");
+      // An issue created before complaint tracking has no status.
+      await getMuniIssuesCollection().insertOne({
+        ...createMuniIssueInput({ clientId: CLIENT }),
+        _id: generateMuniIssueId(),
+        jira: { key: "CP-legacy", syncStatus: "created" },
+        createdAt: new Date(),
+      });
+
+      const issues = await findMuniIssuesByClient(CLIENT);
+
+      expect(issues.map((i) => i._id.toString())).toEqual([newer, older]);
+      expect(await countMuniIssuesByClient(CLIENT)).toBe(2);
+    });
+
+    it("should filter by answered and apply skip/limit", async () => {
+      const answered = await createIssue();
+      await createIssue();
+      await createIssue();
+      await setMuniIssueResponse(answered, CLIENT, { status: "resolved", text: "Done" });
+
+      expect(await countMuniIssuesByClient(CLIENT, { answered: true })).toBe(1);
+      expect(await countMuniIssuesByClient(CLIENT, { answered: false })).toBe(2);
+      expect(
+        (await findMuniIssuesByClient(CLIENT, { answered: true }))[0]._id.toString(),
+      ).toBe(answered);
+      expect(await findMuniIssuesByClient(CLIENT, { skip: 1, limit: 1 })).toHaveLength(1);
+    });
+
+    it("should treat a response with only images as answered and a cleared response as waiting", async () => {
+      const imagesOnly = await createIssue();
+      const cleared = await createIssue();
+      const textOnly = await createIssue();
+      await setMuniIssueResponse(imagesOnly, CLIENT, {
+        status: "resolved",
+        text: "",
+        imageUrls: ["https://example.com/a.png"],
+      });
+      await setMuniIssueResponse(cleared, CLIENT, { status: "resolved", text: "Done" });
+      await setMuniIssueResponse(cleared, CLIENT, { status: "resolved", text: "", imageUrls: [] });
+      await setMuniIssueResponse(textOnly, CLIENT, { status: "resolved", text: "Done" });
+
+      const answered = await findMuniIssuesByClient(CLIENT, { answered: true });
+      const waiting = await findMuniIssuesByClient(CLIENT, { answered: false });
+
+      expect(answered.map((i) => i._id.toString()).sort()).toEqual([imagesOnly, textOnly].sort());
+      expect(waiting.map((i) => i._id.toString())).toEqual([cleared]);
+      expect(await countMuniIssuesByClient(CLIENT, { answered: false })).toBe(1);
+    });
+  });
+
+  describe("getMuniIssueByIdForClient", () => {
+    it("should return the issue only for its own client", async () => {
+      const id = await createIssue();
+
+      expect((await getMuniIssueByIdForClient(id, CLIENT))?._id.toString()).toBe(id);
+      expect(await getMuniIssueByIdForClient(id, "client-b")).toBeNull();
+      expect(await getMuniIssueByIdForClient("not-a-valid-id", CLIENT)).toBeNull();
+    });
+  });
+});