@talkpilot/core-db 1.3.9 → 1.3.12

package/jest.config.js

@@ -1,20 +1,20 @@
-/** @type {import('ts-jest').JestConfigWithTsJest} */
-module.exports = {
-  preset: 'ts-jest',
-  testEnvironment: 'node',
-  roots: ['<rootDir>/src'],
-  testPathIgnorePatterns: ['<rootDir>/src/__tests__/legacy/'],
-  testMatch: ['**/__tests__/**/*.spec.ts', '**/*.spec.ts'],
-  transform: {
-    '^.+\\.tsx?$': ['ts-jest', {
-      tsconfig: 'tsconfig.json',
-    }],
-  },
-  setupFilesAfterEnv: ['<rootDir>/src/__tests__/setup.ts'],
-  maxWorkers: 1,
-  verbose: true,
-  forceExit: true,
-  clearMocks: true,
-  resetMocks: true,
-  restoreMocks: true,
-};
+/** @type {import('ts-jest').JestConfigWithTsJest} */
+module.exports = {
+  preset: 'ts-jest',
+  testEnvironment: 'node',
+  roots: ['<rootDir>/src'],
+  testPathIgnorePatterns: ['<rootDir>/src/__tests__/legacy/'],
+  testMatch: ['**/__tests__/**/*.spec.ts', '**/*.spec.ts'],
+  transform: {
+    '^.+\\.tsx?$': ['ts-jest', {
+      tsconfig: 'tsconfig.json',
+    }],
+  },
+  setupFilesAfterEnv: ['<rootDir>/src/__tests__/setup.ts'],
+  maxWorkers: 1,
+  verbose: true,
+  forceExit: true,
+  clearMocks: true,
+  resetMocks: true,
+  restoreMocks: true,
+};

package/package.json

@@ -1,46 +1,46 @@
-{
-  "name": "@talkpilot/core-db",
-  "version": "1.3.9",
-  "description": "Core database package for centralized connections and ORM integration.",
-  "main": "dist/index.js",
-  "types": "dist/index.d.ts",
-  "scripts": {
-    "build": "tsc",
-    "dev": "tsc --watch",
-    "test": "jest",
-    "lint": "eslint 'src/**/*.{ts,tsx}'",
-    "format": "prettier --write \"src/**/*.{ts,tsx}\"",
-    "prepare": "npm run build"
-  },
-  "keywords": [
-    "database",
-    "orm",
-    "typescript",
-    "postgres",
-    "db-client"
-  ],
-  "author": "",
-  "license": "MIT",
-  "dependencies": {
-    "dayjs": "^1.11.21",
-    "express": "^4.18.0",
-    "google-libphonenumber": "^3.2.0",
-    "mongodb": "^6.11.0"
-  },
-  "devDependencies": {
-    "@faker-js/faker": "^8.0.0",
-    "@types/express": "^4.17.0",
-    "@types/google-libphonenumber": "^7.4.0",
-    "@types/jest": "^29.0.0",
-    "@types/node": "^20.0.0",
-    "fishery": "^2.4.0",
-    "jest": "^29.0.0",
-    "mongodb-memory-server": "^10.0.0",
-    "prettier": "^3.8.2",
-    "ts-jest": "^29.0.0",
-    "typescript": "^5.0.0"
-  },
-  "publishConfig": {
-    "access": "public"
-  }
-}
+{
+  "name": "@talkpilot/core-db",
+  "version": "1.3.12",
+  "description": "Core database package for centralized connections and ORM integration.",
+  "main": "dist/index.js",
+  "types": "dist/index.d.ts",
+  "scripts": {
+    "build": "tsc",
+    "dev": "tsc --watch",
+    "test": "jest",
+    "lint": "eslint 'src/**/*.{ts,tsx}'",
+    "format": "prettier --write \"src/**/*.{ts,tsx}\"",
+    "prepare": "npm run build"
+  },
+  "keywords": [
+    "database",
+    "orm",
+    "typescript",
+    "postgres",
+    "db-client"
+  ],
+  "author": "",
+  "license": "MIT",
+  "dependencies": {
+    "dayjs": "^1.11.21",
+    "express": "^4.18.0",
+    "google-libphonenumber": "^3.2.0",
+    "mongodb": "^6.11.0"
+  },
+  "devDependencies": {
+    "@faker-js/faker": "^8.0.0",
+    "@types/express": "^4.17.0",
+    "@types/google-libphonenumber": "^7.4.0",
+    "@types/jest": "^29.0.0",
+    "@types/node": "^20.0.0",
+    "fishery": "^2.4.0",
+    "jest": "^29.0.0",
+    "mongodb-memory-server": "^10.0.0",
+    "prettier": "^3.8.2",
+    "ts-jest": "^29.0.0",
+    "typescript": "^5.0.0"
+  },
+  "publishConfig": {
+    "access": "public"
+  }
+}

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

@@ -1,92 +1,94 @@
-import { createMuniIssue } from "../muniIssues.setters";
-import {
-  generateMuniIssueId,
-  getMuniIssueById,
-  getMuniIssueByJiraKey,
-} from "../muniIssues.getters";
-import { createMuniIssueInput } from "../../../test-utils/factories";
-
-describe("muniIssues setters", () => {
-  describe("createMuniIssue", () => {
-    it("should insert a well-formed document with createdAt and nested content/jira", async () => {
-      const input = createMuniIssueInput();
-      const id = generateMuniIssueId();
-
-      const insertedId = await createMuniIssue(input, id);
-
-      const saved = await getMuniIssueById(insertedId.toString());
-      expect(saved).toMatchObject({
-        callSid: input.callSid,
-        clientId: input.clientId,
-        cityName: input.cityName,
-        issueContent: input.issueContent,
-        jira: input.jira,
-      });
-      expect(saved?.createdAt).toBeInstanceOf(Date);
-    });
-
-    it("should default jira.syncStatus to created when not provided", async () => {
-      const id = generateMuniIssueId();
-
-      const insertedId = await createMuniIssue(
-        createMuniIssueInput({ jira: { key: "CP-9" } }),
-        id,
-      );
-
-      const saved = await getMuniIssueById(insertedId.toString());
-      expect(saved?.jira.syncStatus).toBe("created");
-    });
-
-    it("should store long issueDescription and desiredBehavior in full", async () => {
-      const issueDescription = "a".repeat(2000);
-      const desiredBehavior = "b".repeat(2000);
-      const id = generateMuniIssueId();
-
-      const insertedId = await createMuniIssue(
-        createMuniIssueInput({
-          issueContent: {
-            issuerName: "Dana",
-            issuerRole: "Agent",
-            issueDescription,
-            desiredBehavior,
-          },
-        }),
-        id,
-      );
-
-      const saved = await getMuniIssueById(insertedId.toString());
-      expect(saved?.issueContent.issueDescription).toHaveLength(2000);
-      expect(saved?.issueContent.desiredBehavior).toBe(desiredBehavior);
-    });
-
-    it("should persist the caller-supplied _id so the Jira ticket can link back", async () => {
-      const id = generateMuniIssueId();
-
-      const insertedId = await createMuniIssue(createMuniIssueInput(), id);
-
-      expect(insertedId.toString()).toBe(id.toString());
-      expect(await getMuniIssueById(id.toString())).not.toBeNull();
-    });
-  });
-
-  describe("getMuniIssueById", () => {
-    it("should return null for an unknown or malformed id instead of throwing", async () => {
-      expect(await getMuniIssueById(generateMuniIssueId().toString())).toBeNull();
-      expect(await getMuniIssueById("not-a-valid-id")).toBeNull();
-    });
-  });
-
-  describe("getMuniIssueByJiraKey", () => {
-    it("should find the issue with the given Jira key", async () => {
-      await createMuniIssue(
-        createMuniIssueInput({ jira: { key: "CP-555" } }),
-        generateMuniIssueId(),
-      );
-
-      const found = await getMuniIssueByJiraKey("CP-555");
-
-      expect(found?.jira.key).toBe("CP-555");
-      expect(await getMuniIssueByJiraKey("CP-nope")).toBeNull();
-    });
-  });
-});
+import { createMuniIssue } from "../muniIssues.setters";
+import {
+  generateMuniIssueId,
+  getMuniIssueById,
+  getMuniIssueByJiraKey,
+} from "../muniIssues.getters";
+import { createMuniIssueInput } from "../../../test-utils/factories";
+
+describe("muniIssues setters", () => {
+  describe("createMuniIssue", () => {
+    it("should insert a well-formed document with createdAt and nested content/jira", async () => {
+      const input = createMuniIssueInput();
+      const id = generateMuniIssueId();
+
+      const insertedId = await createMuniIssue(input, id);
+
+      const saved = await getMuniIssueById(insertedId.toString());
+      expect(saved).toMatchObject({
+        callSid: input.callSid,
+        clientId: input.clientId,
+        cityName: input.cityName,
+        issueContent: input.issueContent,
+        jira: input.jira,
+      });
+      expect(saved?.createdAt).toBeInstanceOf(Date);
+    });
+
+    it("should default jira.syncStatus to created when not provided", async () => {
+      const id = generateMuniIssueId();
+
+      const insertedId = await createMuniIssue(
+        createMuniIssueInput({ jira: { key: "CP-9" } }),
+        id,
+      );
+
+      const saved = await getMuniIssueById(insertedId.toString());
+      expect(saved?.jira.syncStatus).toBe("created");
+    });
+
+    it("should store long issueDescription and desiredBehavior in full", async () => {
+      const issueDescription = "a".repeat(2000);
+      const desiredBehavior = "b".repeat(2000);
+      const id = generateMuniIssueId();
+
+      const insertedId = await createMuniIssue(
+        createMuniIssueInput({
+          issueContent: {
+            issuerName: "Dana",
+            issuerRole: "Agent",
+            issueDescription,
+            desiredBehavior,
+          },
+        }),
+        id,
+      );
+
+      const saved = await getMuniIssueById(insertedId.toString());
+      expect(saved?.issueContent.issueDescription).toHaveLength(2000);
+      expect(saved?.issueContent.desiredBehavior).toBe(desiredBehavior);
+    });
+
+    it("should persist the caller-supplied _id so the Jira ticket can link back", async () => {
+      const id = generateMuniIssueId();
+
+      const insertedId = await createMuniIssue(createMuniIssueInput(), id);
+
+      expect(insertedId.toString()).toBe(id.toString());
+      expect(await getMuniIssueById(id.toString())).not.toBeNull();
+    });
+  });
+
+  describe("getMuniIssueById", () => {
+    it("should return null for an unknown or malformed id instead of throwing", async () => {
+      expect(
+        await getMuniIssueById(generateMuniIssueId().toString()),
+      ).toBeNull();
+      expect(await getMuniIssueById("not-a-valid-id")).toBeNull();
+    });
+  });
+
+  describe("getMuniIssueByJiraKey", () => {
+    it("should find the issue with the given Jira key", async () => {
+      await createMuniIssue(
+        createMuniIssueInput({ jira: { key: "CP-555" } }),
+        generateMuniIssueId(),
+      );
+
+      const found = await getMuniIssueByJiraKey("CP-555");
+
+      expect(found?.jira.key).toBe("CP-555");
+      expect(await getMuniIssueByJiraKey("CP-nope")).toBeNull();
+    });
+  });
+});

package/src/municipal/tickets/__tests__/tickets.statistics.spec.ts

@@ -1,68 +1,99 @@
-import {
-  findCallSidDraftTicketCountsByCity,
-  findCallSidTicketCountsByCity,
-  getTicketsCollection,
-  ticketStatsDateScopeFromYmd,
-} from "../../../index";
-import { createTicket } from "../../../test-utils/factories";
-
-const CITY = "tests" as const;
-const TZ = "Asia/Jerusalem";
-const mayScope = () => ticketStatsDateScopeFromYmd("2026-05-01", "2026-05-31", TZ);
-
-describe("tickets statistics getters", () => {
-  it("should scope callSid lookups to the provided createdAt range", async () => {
-    await getTicketsCollection().insertMany([
-      createTicket({
-        cityName: CITY,
-        callSid: "CA-in-range",
-        createdAt: new Date("2026-05-15T10:00:00.000Z"),
-        updatedAt: new Date("2026-05-15T10:00:00.000Z"),
-      }),
-      createTicket({
-        cityName: CITY,
-        callSid: "CA-out-of-range",
-        createdAt: new Date("2024-01-01T10:00:00.000Z"),
-        updatedAt: new Date("2024-01-01T10:00:00.000Z"),
-      }),
-    ]);
-    expect(await findCallSidTicketCountsByCity(CITY, mayScope())).toEqual(new Map([["CA-in-range", 1]]));
-  });
-
-  it("should treat whitespace-only event_subject_id as draft", async () => {
-    await getTicketsCollection().insertMany([
-      createTicket({
-        cityName: CITY,
-        callSid: "CA-draft-whitespace",
-        createdAt: new Date("2026-05-15T10:00:00.000Z"),
-        updatedAt: new Date("2026-05-15T10:00:00.000Z"),
-        externalCallFields: { event_subject_id: "  \t  " },
-      }),
-      createTicket({
-        cityName: CITY,
-        callSid: "CA-not-draft",
-        createdAt: new Date("2026-05-16T10:00:00.000Z"),
-        updatedAt: new Date("2026-05-16T10:00:00.000Z"),
-        externalCallFields: { event_subject_id: "100" },
-      }),
-    ]);
-    expect(await findCallSidDraftTicketCountsByCity(CITY, mayScope())).toEqual(new Map([["CA-draft-whitespace", 1]]));
-  });
-    it("should count multiple tickets on the same callSid", async () => {
-    await getTicketsCollection().insertMany([
-      createTicket({
-        cityName: CITY,
-        callSid: "CA-multi",
-        createdAt: new Date("2026-05-15T10:00:00.000Z"),
-        updatedAt: new Date("2026-05-15T10:00:00.000Z"),
-      }),
-      createTicket({
-        cityName: CITY,
-        callSid: "CA-multi",
-        createdAt: new Date("2026-05-15T11:00:00.000Z"),
-        updatedAt: new Date("2026-05-15T11:00:00.000Z"),
-      }),
-    ]);
-    expect(await findCallSidTicketCountsByCity(CITY, mayScope())).toEqual(new Map([["CA-multi", 2]]));
-  });
-});
+import {
+  findCallSidDraftTicketCountsByCity,
+  findCallSidTicketCountsByCity,
+  getTicketsCollection,
+  ticketStatsDateScopeFromYmd,
+} from "../../../index";
+import { createTicket } from "../../../test-utils/factories";
+
+const CITY = "tests" as const;
+const TZ = "Asia/Jerusalem";
+const mayScope = () =>
+  ticketStatsDateScopeFromYmd("2026-05-01", "2026-05-31", TZ);
+
+describe("tickets statistics getters", () => {
+  it("should scope callSid lookups to the provided createdAt range", async () => {
+    await getTicketsCollection().insertMany([
+      createTicket({
+        cityName: CITY,
+        callSid: "CA-in-range",
+        createdAt: new Date("2026-05-15T10:00:00.000Z"),
+        updatedAt: new Date("2026-05-15T10:00:00.000Z"),
+      }),
+      createTicket({
+        cityName: CITY,
+        callSid: "CA-out-of-range",
+        createdAt: new Date("2024-01-01T10:00:00.000Z"),
+        updatedAt: new Date("2024-01-01T10:00:00.000Z"),
+      }),
+    ]);
+    expect(await findCallSidTicketCountsByCity(CITY, mayScope())).toEqual(
+      new Map([["CA-in-range", 1]]),
+    );
+  });
+
+  it("should treat whitespace-only event_subject_id as draft for legacy tickets", async () => {
+    await getTicketsCollection().insertMany([
+      createTicket({
+        cityName: CITY,
+        callSid: "CA-draft-whitespace",
+        createdAt: new Date("2026-05-15T10:00:00.000Z"),
+        updatedAt: new Date("2026-05-15T10:00:00.000Z"),
+        externalCallFields: { event_subject_id: "  \t  " },
+      }),
+      createTicket({
+        cityName: CITY,
+        callSid: "CA-not-draft",
+        createdAt: new Date("2026-05-16T10:00:00.000Z"),
+        updatedAt: new Date("2026-05-16T10:00:00.000Z"),
+        externalCallFields: { event_subject_id: "100" },
+      }),
+    ]);
+    expect(await findCallSidDraftTicketCountsByCity(CITY, mayScope())).toEqual(
+      new Map([["CA-draft-whitespace", 1]]),
+    );
+  });
+
+  it("should use isDraft when set, overriding legacy heuristics", async () => {
+    await getTicketsCollection().insertMany([
+      createTicket({
+        cityName: CITY,
+        callSid: "CA-explicit-draft",
+        isDraft: true,
+        createdAt: new Date("2026-05-15T10:00:00.000Z"),
+        updatedAt: new Date("2026-05-15T10:00:00.000Z"),
+        externalCallFields: { event_subject_id: "100" },
+      }),
+      createTicket({
+        cityName: CITY,
+        callSid: "CA-explicit-not-draft",
+        isDraft: false,
+        createdAt: new Date("2026-05-16T10:00:00.000Z"),
+        updatedAt: new Date("2026-05-16T10:00:00.000Z"),
+        externalCallFields: {},
+      }),
+    ]);
+    expect(await findCallSidDraftTicketCountsByCity(CITY, mayScope())).toEqual(
+      new Map([["CA-explicit-draft", 1]]),
+    );
+  });
+  it("should count multiple tickets on the same callSid", async () => {
+    await getTicketsCollection().insertMany([
+      createTicket({
+        cityName: CITY,
+        callSid: "CA-multi",
+        createdAt: new Date("2026-05-15T10:00:00.000Z"),
+        updatedAt: new Date("2026-05-15T10:00:00.000Z"),
+      }),
+      createTicket({
+        cityName: CITY,
+        callSid: "CA-multi",
+        createdAt: new Date("2026-05-15T11:00:00.000Z"),
+        updatedAt: new Date("2026-05-15T11:00:00.000Z"),
+      }),
+    ]);
+    expect(await findCallSidTicketCountsByCity(CITY, mayScope())).toEqual(
+      new Map([["CA-multi", 2]]),
+    );
+  });
+});

package/src/municipal/tickets/index.ts

@@ -1,3 +1,9 @@
-export * from "./tickets.getters";
-export * from "./tickets.statistics.getters";
-export type { Ticket, TicketDoc, SubjectItem, TicketStatsDateScope, TicketSubjectRow } from "./tickets.types";
+export * from "./tickets.getters";
+export * from "./tickets.statistics.getters";
+export type {
+  Ticket,
+  TicketDoc,
+  SubjectItem,
+  TicketStatsDateScope,
+  TicketSubjectRow,
+} from "./tickets.types";