@talkpilot/core-db 1.3.4 → 1.3.5

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

@@ -0,0 +1,24 @@
+import { Collection, ObjectId } from "mongodb";
+import { getDb } from "../index";
+import { MuniIssue } from "./muniIssues.types";
+import { MUNI_ISSUES_COLLECTION } from "./muniIssues.constants";
+
+export const getMuniIssuesCollection = (): Collection<MuniIssue> => {
+  return getDb().collection<MuniIssue>(MUNI_ISSUES_COLLECTION);
+};
+
+export const generateMuniIssueId = (): ObjectId => new ObjectId();
+
+export const getMuniIssueById = async (
+  id: string,
+): Promise<MuniIssue | null> => {
+  // A malformed id is treated as not-found rather than throwing on `new ObjectId`.
+  if (!ObjectId.isValid(id)) return null;
+  return getMuniIssuesCollection().findOne({ _id: new ObjectId(id) });
+};
+
+export const getMuniIssueByJiraKey = async (
+  jiraKey: string,
+): Promise<MuniIssue | null> => {
+  return getMuniIssuesCollection().findOne({ "jira.key": jiraKey });
+};

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

@@ -0,0 +1,66 @@
+import type { Document, IndexDescription } from "mongodb";
+import {
+  MUNI_ISSUE_SYNC_STATUSES,
+  MUNI_ISSUE_TEXT_MAX_LENGTH,
+  MUNI_ISSUE_NAME_MAX_LENGTH,
+} from "./muniIssues.constants";
+
+export const muniIssuesJsonSchema: Document = {
+  bsonType: "object",
+  required: [
+    "callSid",
+    "clientId",
+    "cityName",
+    "issueContent",
+    "jira",
+    "createdAt",
+  ],
+  properties: {
+    callSid: { bsonType: "string" },
+    clientId: { bsonType: "string" },
+    cityName: { bsonType: "string" },
+    issueContent: {
+      bsonType: "object",
+      required: [
+        "issuerName",
+        "issuerRole",
+        "issueDescription",
+        "desiredBehavior",
+      ],
+      properties: {
+        issuerName: {
+          bsonType: "string",
+          maxLength: MUNI_ISSUE_NAME_MAX_LENGTH,
+        },
+        issuerRole: {
+          bsonType: "string",
+          maxLength: MUNI_ISSUE_NAME_MAX_LENGTH,
+        },
+        issueDescription: {
+          bsonType: "string",
+          maxLength: MUNI_ISSUE_TEXT_MAX_LENGTH,
+        },
+        desiredBehavior: {
+          bsonType: "string",
+          maxLength: MUNI_ISSUE_TEXT_MAX_LENGTH,
+        },
+      },
+    },
+    jira: {
+      bsonType: "object",
+      required: ["key", "syncStatus"],
+      properties: {
+        key: { bsonType: "string" },
+        syncStatus: { enum: [...MUNI_ISSUE_SYNC_STATUSES] },
+      },
+    },
+    createdAt: { bsonType: "date" },
+  },
+};
+
+export const muniIssuesIndexes: IndexDescription[] = [
+  { key: { callSid: 1 }, name: "callSid_1" },
+  { key: { createdAt: -1 }, name: "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

@@ -0,0 +1,24 @@
+import { ObjectId } from "mongodb";
+import { CreateMuniIssueInput, MuniIssue } from "./muniIssues.types";
+import { getMuniIssuesCollection } from "./muniIssues.getters";
+import { ensureMuniIssuesCollectionOnce } from "./muniIssues.setup";
+
+// Required `id`: MIS pre-generates it (generateMuniIssueId) and embeds it in the Jira ticket — the two-way link.
+export const createMuniIssue = async (
+  issueData: CreateMuniIssueInput,
+  id: ObjectId,
+): Promise<ObjectId> => {
+  await ensureMuniIssuesCollectionOnce();
+
+  const issue: MuniIssue = {
+    _id: id,
+    ...issueData,
+    jira: {
+      ...issueData.jira,
+      syncStatus: issueData.jira.syncStatus ?? "created",
+    },
+    createdAt: new Date(),
+  };
+  const { insertedId } = await getMuniIssuesCollection().insertOne(issue);
+  return insertedId;
+};

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

@@ -0,0 +1,65 @@
+import { Db, Document, MongoServerError } from "mongodb";
+import { getDb } from "../index";
+import {
+  MUNI_ISSUES_COLLECTION,
+  NAMESPACE_EXISTS,
+  VALIDATION_LEVEL,
+  VALIDATION_ACTION,
+} from "./muniIssues.constants";
+import { muniIssuesIndexes, muniIssuesJsonSchema } from "./muniIssues.schema";
+
+const applyValidator = (db: Db, validator: Document): Promise<unknown> =>
+  db.command({
+    collMod: MUNI_ISSUES_COLLECTION,
+    validator,
+    validationLevel: VALIDATION_LEVEL,
+    validationAction: VALIDATION_ACTION,
+  });
+
+// Provisions validator and indexes: createCollection first, collMod if already exists.
+export const ensureMuniIssuesCollection = async (): Promise<void> => {
+  const db = getDb();
+  const validator = { $jsonSchema: muniIssuesJsonSchema };
+
+  const existing = await db
+    .listCollections({ name: MUNI_ISSUES_COLLECTION })
+    .toArray();
+
+  if (existing.length === 0) {
+    try {
+      await db.createCollection(MUNI_ISSUES_COLLECTION, {
+        validator,
+        validationLevel: VALIDATION_LEVEL,
+        validationAction: VALIDATION_ACTION,
+      });
+    } catch (err) {
+      // NamespaceExists race: collection exists — apply validator via collMod.
+      if (err instanceof MongoServerError && err.code === NAMESPACE_EXISTS) {
+        await applyValidator(db, validator);
+      } else {
+        throw err;
+      }
+    }
+  } else {
+    await applyValidator(db, validator);
+  }
+
+  await db.collection(MUNI_ISSUES_COLLECTION).createIndexes(muniIssuesIndexes);
+};
+
+// Per-Db cache so each test's in-memory Db re-provisions.
+const provisioningByDb = new WeakMap<Db, Promise<void>>();
+
+// Once per Db; evicts failed run so the next call retries.
+export const ensureMuniIssuesCollectionOnce = (): Promise<void> => {
+  const db = getDb();
+  let provisioning = provisioningByDb.get(db);
+  if (!provisioning) {
+    provisioning = ensureMuniIssuesCollection().catch((err) => {
+      provisioningByDb.delete(db);
+      throw err;
+    });
+    provisioningByDb.set(db, provisioning);
+  }
+  return provisioning;
+};

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

@@ -0,0 +1,37 @@
+import { ObjectId } from "mongodb";
+
+import { CityName } from "../utils/types";
+import { MUNI_ISSUE_SYNC_STATUSES } from "./muniIssues.constants";
+
+export type MuniIssueSyncStatus = (typeof MUNI_ISSUE_SYNC_STATUSES)[number];
+
+export type MuniIssueContent = {
+  issuerName: string;
+  issuerRole: string;
+  issueDescription: string;
+  desiredBehavior: string;
+};
+
+export type MuniIssueJira = {
+  key: string;
+  syncStatus: MuniIssueSyncStatus;
+};
+
+export type MuniIssue = {
+  _id: ObjectId;
+  callSid: string;
+  clientId: string;
+  cityName: CityName;
+  issueContent: MuniIssueContent;
+  jira: MuniIssueJira;
+  createdAt: 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).
+export type CreateMuniIssueInput = Omit<
+  MuniIssue,
+  "_id" | "createdAt" | "jira"
+> & {
+  jira: { key: string; syncStatus?: MuniIssueSyncStatus };
+};

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

@@ -25,8 +25,8 @@ describe("getDashboardStats", () => {
 
     const params = {
       clientId,
-      startDate: "2026-05-01",
-      endDate: "2026-05-31",
+      startDate: "2026-05-01T00:00:00.000Z",
+      endDate: "2026-05-31T23:59:59.999Z",
     };
 
     const result = await getDashboardStats(params);
@@ -35,12 +35,115 @@ describe("getDashboardStats", () => {
     expect(result.kpis.totalCalls).toBe(0);
     expect(result.kpis.completedCount).toBe(0);
     expect(result.kpis.avgDurationSeconds).toBe(0);
-    expect(result.charts.volumeData).toEqual([]);
-    expect(result.charts.heatmap).toEqual({});
-    expect(result.charts.callLengthBuckets).toEqual({
-      short: 0,
-      medium: 0,
-      long: 0,
+    expect(result.kpis.busyCount).toBe(0);
+    expect(result.kpis.timeSavedMinutes).toBe(0);
+    // ~30 day range -> daily buckets, gap-filled with zeros across the whole range
+    expect(result.charts.volumeGranularity).toBe("day");
+    expect(result.charts.volumeData.length).toBeGreaterThan(0);
+    expect(result.charts.volumeData.every((d) => d.completed === 0)).toBe(true);
+    expect(result.charts.volumeData[0].date).toBe("2026-05-01");
+    expect(
+      result.charts.volumeData[result.charts.volumeData.length - 1].date,
+    ).toBe("2026-05-31");
+  });
+
+  it("should pick a bucket size that keeps the volume chart readable across range lengths", async () => {
+    const clientId = "client-dash-456";
+
+    await getClientsConfigCollection().insertOne({
+      clientId,
+      timezone: "UTC",
+      products: {},
+    } as any);
+
+    const oneDay = await getDashboardStats({
+      clientId,
+      startDate: "2026-05-01T00:00:00.000Z",
+      endDate: "2026-05-01T23:59:59.999Z",
+    });
+    expect(oneDay.charts.volumeGranularity).toBe("hour");
+
+    const threeWeeks = await getDashboardStats({
+      clientId,
+      startDate: "2026-05-01T00:00:00.000Z",
+      endDate: "2026-05-21T23:59:59.999Z",
+    });
+    expect(threeWeeks.charts.volumeGranularity).toBe("day");
+
+    const fourMonths = await getDashboardStats({
+      clientId,
+      startDate: "2026-01-01T00:00:00.000Z",
+      endDate: "2026-04-30T23:59:59.999Z",
+    });
+    expect(fourMonths.charts.volumeGranularity).toBe("week");
+
+    const eighteenMonths = await getDashboardStats({
+      clientId,
+      startDate: "2025-01-01T00:00:00.000Z",
+      endDate: "2026-06-30T23:59:59.999Z",
+    });
+    expect(eighteenMonths.charts.volumeGranularity).toBe("month");
+
+    const threeYears = await getDashboardStats({
+      clientId,
+      startDate: "2023-01-01T00:00:00.000Z",
+      endDate: "2026-01-01T23:59:59.999Z",
+    });
+    expect(threeYears.charts.volumeGranularity).toBe("quarter");
+
+    const fiveYears = await getDashboardStats({
+      clientId,
+      startDate: "2021-01-01T00:00:00.000Z",
+      endDate: "2026-01-01T23:59:59.999Z",
+    });
+    expect(fiveYears.charts.volumeGranularity).toBe("year");
+  });
+
+  it("should fill gaps with zero-count buckets so the chart has no missing points", async () => {
+    const clientId = "client-dash-789";
+
+    await getClientsConfigCollection().insertOne({
+      clientId,
+      timezone: "UTC",
+      products: {},
+    } as any);
+
+    // Sparse data: only two of the day's hours have calls.
+    await getCallsCollection().insertMany([
+      {
+        clientId,
+        status: "completed",
+        callLength: 60,
+        createdAt: new Date("2026-05-01T09:15:00Z"),
+      },
+      {
+        clientId,
+        status: "completed",
+        callLength: 90,
+        createdAt: new Date("2026-05-01T18:45:00Z"),
+      },
+    ] as any);
+
+    const result = await getDashboardStats({
+      clientId,
+      startDate: "2026-05-01T00:00:00.000Z",
+      endDate: "2026-05-01T23:59:59.999Z",
     });
+
+    expect(result.charts.volumeGranularity).toBe("hour");
+    // A full day of hourly buckets, with no gaps, sorted chronologically.
+    expect(result.charts.volumeData).toHaveLength(24);
+    expect(result.charts.volumeData[0].date).toBe("2026-05-01T00:00");
+    expect(result.charts.volumeData[23].date).toBe("2026-05-01T23:00");
+
+    const populated = result.charts.volumeData.filter((d) => d.completed > 0);
+    expect(populated).toHaveLength(2);
+    expect(populated.map((d) => d.date)).toEqual([
+      "2026-05-01T09:00",
+      "2026-05-01T18:00",
+    ]);
+
+    const empty = result.charts.volumeData.filter((d) => d.completed === 0);
+    expect(empty).toHaveLength(22);
   });
 });