@talkpilot/core-db 1.1.19 → 1.2.0

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

@@ -1,62 +1,62 @@
-import { municipalDataMongodbClient } from "../mongodb-client";
-import { MongoClient } from "mongodb";
-
-describe("MunicipalDataMongoDBClient Validation", () => {
-  const originalEnv = process.env;
-
-  beforeEach(() => {
-    process.env = { ...originalEnv };
-    delete process.env.MONGO_URI;
-    delete process.env.MONGODB_URI;
-
-    // Force reset singleton state
-    (municipalDataMongodbClient as any).client = null;
-    (municipalDataMongodbClient as any).db = null;
-  });
-
-  afterAll(async () => {
-    process.env = originalEnv;
-    await municipalDataMongodbClient.disconnect();
-  });
-
-  it("should succeed with default database name if only URI is provided", async () => {
-    const mockDb = { databaseName: "municipal-data" };
-    const connectSpy = jest
-      .spyOn(MongoClient.prototype, "connect")
-      .mockResolvedValue({} as any);
-    const dbSpy = jest
-      .spyOn(MongoClient.prototype, "db")
-      .mockReturnValue(mockDb as any);
-
-    const validUri = "mongodb://127.0.0.1:27017";
-    await municipalDataMongodbClient.connect(validUri);
-
-    expect(municipalDataMongodbClient.getDb()).toBeDefined();
-    expect(municipalDataMongodbClient.getDb().databaseName).toBe(
-      "municipal-data",
-    );
-
-    connectSpy.mockRestore();
-    dbSpy.mockRestore();
-  });
-
-  it("should allow overriding the default database name", async () => {
-    const mockDb = { databaseName: "custom-municipal-db" };
-    const connectSpy = jest
-      .spyOn(MongoClient.prototype, "connect")
-      .mockResolvedValue({} as any);
-    const dbSpy = jest
-      .spyOn(MongoClient.prototype, "db")
-      .mockReturnValue(mockDb as any);
-
-    const validUri = "mongodb://127.0.0.1:27017";
-    await municipalDataMongodbClient.connect(validUri, "custom-municipal-db");
-
-    expect(municipalDataMongodbClient.getDb().databaseName).toBe(
-      "custom-municipal-db",
-    );
-
-    connectSpy.mockRestore();
-    dbSpy.mockRestore();
-  });
-});
+import { municipalDataMongodbClient } from "../mongodb-client";
+import { MongoClient } from "mongodb";
+
+describe("MunicipalDataMongoDBClient Validation", () => {
+  const originalEnv = process.env;
+
+  beforeEach(() => {
+    process.env = { ...originalEnv };
+    delete process.env.MONGO_URI;
+    delete process.env.MONGODB_URI;
+
+    // Force reset singleton state
+    (municipalDataMongodbClient as any).client = null;
+    (municipalDataMongodbClient as any).db = null;
+  });
+
+  afterAll(async () => {
+    process.env = originalEnv;
+    await municipalDataMongodbClient.disconnect();
+  });
+
+  it("should succeed with default database name if only URI is provided", async () => {
+    const mockDb = { databaseName: "municipal-data" };
+    const connectSpy = jest
+      .spyOn(MongoClient.prototype, "connect")
+      .mockResolvedValue({} as any);
+    const dbSpy = jest
+      .spyOn(MongoClient.prototype, "db")
+      .mockReturnValue(mockDb as any);
+
+    const validUri = "mongodb://127.0.0.1:27017";
+    await municipalDataMongodbClient.connect(validUri);
+
+    expect(municipalDataMongodbClient.getDb()).toBeDefined();
+    expect(municipalDataMongodbClient.getDb().databaseName).toBe(
+      "municipal-data",
+    );
+
+    connectSpy.mockRestore();
+    dbSpy.mockRestore();
+  });
+
+  it("should allow overriding the default database name", async () => {
+    const mockDb = { databaseName: "custom-municipal-db" };
+    const connectSpy = jest
+      .spyOn(MongoClient.prototype, "connect")
+      .mockResolvedValue({} as any);
+    const dbSpy = jest
+      .spyOn(MongoClient.prototype, "db")
+      .mockReturnValue(mockDb as any);
+
+    const validUri = "mongodb://127.0.0.1:27017";
+    await municipalDataMongodbClient.connect(validUri, "custom-municipal-db");
+
+    expect(municipalDataMongodbClient.getDb().databaseName).toBe(
+      "custom-municipal-db",
+    );
+
+    connectSpy.mockRestore();
+    dbSpy.mockRestore();
+  });
+});

package/src/municipal/cities/cities.getters.ts

@@ -1,50 +1,50 @@
-import { getDb, ObjectId, City } from "../index";
-import { Collection, Filter, ObjectId as MongoObjectId } from "mongodb";
-
-export const getCitiesCollection = (): Collection<City> => {
-  return getDb().collection<City>("cities");
-};
-
-export const findCities = async (
-  filter: Filter<City> = {},
-): Promise<City[]> => {
-  return await getCitiesCollection().find(filter).toArray();
-};
-
-export const getCityById = async (cityId: string): Promise<City | null> => {
-  const city = await getCitiesCollection().findOne({
-    _id: new ObjectId(cityId),
-  });
-  return city ? city : null;
-};
-
-export const createCity = async (
-  cityData: Omit<City, "_id" | "createdAt" | "updatedAt">,
-): Promise<MongoObjectId> => {
-  const city: Omit<City, "_id"> = {
-    ...cityData,
-    createdAt: new Date(),
-    updatedAt: new Date(),
-  };
-  const { insertedId } = await getCitiesCollection().insertOne(city as City);
-  return insertedId;
-};
-
-export const updateCity = async (
-  cityId: string,
-  data: Partial<Omit<City, "_id" | "createdAt" | "updatedAt">>,
-): Promise<City | null> => {
-  const result = await getCitiesCollection().findOneAndUpdate(
-    { _id: new ObjectId(cityId) },
-    { $set: { ...data, updatedAt: new Date() } },
-    { returnDocument: "after" },
-  );
-  return result || null;
-};
-
-export const deleteCity = async (cityId: string): Promise<boolean> => {
-  const result = await getCitiesCollection().deleteOne({
-    _id: new ObjectId(cityId),
-  });
-  return result.deletedCount > 0;
-};
+import { getDb, ObjectId, City } from "../index";
+import { Collection, Filter, ObjectId as MongoObjectId } from "mongodb";
+
+export const getCitiesCollection = (): Collection<City> => {
+  return getDb().collection<City>("cities");
+};
+
+export const findCities = async (
+  filter: Filter<City> = {},
+): Promise<City[]> => {
+  return await getCitiesCollection().find(filter).toArray();
+};
+
+export const getCityById = async (cityId: string): Promise<City | null> => {
+  const city = await getCitiesCollection().findOne({
+    _id: new ObjectId(cityId),
+  });
+  return city ? city : null;
+};
+
+export const createCity = async (
+  cityData: Omit<City, "_id" | "createdAt" | "updatedAt">,
+): Promise<MongoObjectId> => {
+  const city: Omit<City, "_id"> = {
+    ...cityData,
+    createdAt: new Date(),
+    updatedAt: new Date(),
+  };
+  const { insertedId } = await getCitiesCollection().insertOne(city as City);
+  return insertedId;
+};
+
+export const updateCity = async (
+  cityId: string,
+  data: Partial<Omit<City, "_id" | "createdAt" | "updatedAt">>,
+): Promise<City | null> => {
+  const result = await getCitiesCollection().findOneAndUpdate(
+    { _id: new ObjectId(cityId) },
+    { $set: { ...data, updatedAt: new Date() } },
+    { returnDocument: "after" },
+  );
+  return result || null;
+};
+
+export const deleteCity = async (cityId: string): Promise<boolean> => {
+  const result = await getCitiesCollection().deleteOne({
+    _id: new ObjectId(cityId),
+  });
+  return result.deletedCount > 0;
+};

package/src/municipal/cities/cities.types.ts

@@ -1,11 +1,11 @@
-import { ObjectId, WithId } from "mongodb";
-
-export type City = {
-  _id: ObjectId;
-  createdAt: Date;
-  updatedAt: Date;
-  name: string;
-  displayName: string;
-};
-
-export type CityDoc = WithId<City>;
+import { ObjectId, WithId } from "mongodb";
+
+export type City = {
+  _id: ObjectId;
+  createdAt: Date;
+  updatedAt: Date;
+  name: string;
+  displayName: string;
+};
+
+export type CityDoc = WithId<City>;

package/src/municipal/cities/index.ts

@@ -1,2 +1,2 @@
-export * from "./cities.getters";
-export type { City, CityDoc } from "./cities.types";
+export * from "./cities.getters";
+export type { City, CityDoc } from "./cities.types";