@terreno/api 0.18.0 → 0.19.0

package/src/populate.test.ts

@@ -265,6 +265,31 @@ describe("getOpenApiSpecForModel edge cases", () => {
   });
 });
 
+describe("getOpenApiSpecForModel populate with existing properties", () => {
+  it("merges populated properties into a path that already has properties", () => {
+    const result = getOpenApiSpecForModel(FoodModel, {
+      populatePaths: [{path: "likesIds.userId"}],
+    });
+    // likesIds is an array subschema with its own properties already;
+    // populating userId should merge the user properties into the existing structure.
+    expect(result.properties.likesIds).toBeDefined();
+    const likesIds = result.properties.likesIds as Record<string, unknown>;
+    const items = likesIds.items as Record<string, Record<string, unknown>>;
+    expect(items.properties.userId).toBeDefined();
+  });
+
+  it("creates intermediate path structure when navigating to nested populate", () => {
+    // eatenBy is defined as [{ ref: "User", type: ObjectId }] - an array of refs.
+    // When we populate eatenBy, the openApiPath resolves through items.
+    const result = getOpenApiSpecForModel(FoodModel, {
+      populatePaths: [{path: "eatenBy"}],
+    });
+    expect(result.properties.eatenBy).toBeDefined();
+    const eatenBy = result.properties.eatenBy as Record<string, unknown>;
+    expect(eatenBy.items).toBeDefined();
+  });
+});
+
 describe("filterKeys (via getOpenApiSpecForModel populatePaths)", () => {
   it("filters populated fields using dot-notation keys", () => {
     const result = getOpenApiSpecForModel(FoodModel, {

package/src/realtime/queryMatcher.ts

@@ -6,7 +6,6 @@
  * Supports: equality, $eq, $ne, $gt, $gte, $lt, $lte, $in, $nin, $exists, $and, $or, $not.
  */
 
-// biome-ignore lint/suspicious/noExplicitAny: traversing arbitrary nested document fields by user-supplied dotted path
 const getNestedValue = (doc: any, path: string): any => {
   const parts = path.split(".");
   let current = doc;
@@ -19,7 +18,6 @@ const getNestedValue = (doc: any, path: string): any => {
   return current;
 };
 
-// biome-ignore lint/suspicious/noExplicitAny: value may be any document field type (string, number, ObjectId, etc.)
 const normalize = (value: any): any => {
   if (value === null || value === undefined) {
     return value;
@@ -36,7 +34,6 @@ const normalize = (value: any): any => {
   return value;
 };
 
-// biome-ignore lint/suspicious/noExplicitAny: rawValue is an arbitrary document field, condition is an arbitrary user query operand
 const matchesCondition = (rawValue: any, condition: any): boolean => {
   const value = normalize(rawValue);
 
@@ -91,7 +88,6 @@ const matchesCondition = (rawValue: any, condition: any): boolean => {
           return false;
         }
         const inValues = operand.map(normalize);
-        // biome-ignore lint/suspicious/noExplicitAny: normalized value of arbitrary document field
         if (!inValues.some((v: any) => v === value || String(v) === String(value))) {
           return false;
         }
@@ -102,7 +98,6 @@ const matchesCondition = (rawValue: any, condition: any): boolean => {
           return false;
         }
         const ninValues = operand.map(normalize);
-        // biome-ignore lint/suspicious/noExplicitAny: normalized value of arbitrary document field
         if (ninValues.some((v: any) => v === value || String(v) === String(value))) {
           return false;
         }
@@ -137,7 +132,6 @@ const matchesCondition = (rawValue: any, condition: any): boolean => {
  * @param query - MongoDB-style query object
  * @returns true if the document matches all query conditions
  */
-// biome-ignore lint/suspicious/noExplicitAny: doc is arbitrary; query values are arbitrary user-supplied JSON
 export const matchesQuery = (doc: any, query: Record<string, any>): boolean => {
   for (const [key, condition] of Object.entries(query)) {
     if (key === "$and") {

package/src/realtime/queryStore.ts

@@ -9,7 +9,6 @@
 
 interface QuerySubscription {
   collection: string;
-  // biome-ignore lint/suspicious/noExplicitAny: MongoDB query filter values are arbitrary user-supplied JSON
   query: Record<string, any>;
   queryId: string;
 }
@@ -18,13 +17,8 @@ interface QuerySubscription {
  * Compute a deterministic queryId from collection and query on the server side.
  * This prevents clients from hijacking other subscriptions by providing a colliding queryId.
  */
-export const computeQueryId = (
-  collection: string,
-  // biome-ignore lint/suspicious/noExplicitAny: MongoDB query filter values are arbitrary user-supplied JSON
-  query: Record<string, any>
-): string => {
+export const computeQueryId = (collection: string, query: Record<string, any>): string => {
   const sortedKeys = Object.keys(query).sort();
-  // biome-ignore lint/suspicious/noExplicitAny: mirrors the input query value shape
   const normalized: Record<string, any> = {};
   for (const key of sortedKeys) {
     normalized[key] = query[key];
@@ -45,7 +39,6 @@ const socketQueries = new Map<string, Set<string>>();
 export const addQuerySubscription = (
   socketId: string,
   collection: string,
-  // biome-ignore lint/suspicious/noExplicitAny: MongoDB query filter values are arbitrary user-supplied JSON
   query: Record<string, any>,
   queryId: string
 ): void => {
@@ -109,9 +102,7 @@ export const removeAllSocketQueries = (socketId: string): void => {
  */
 export const getQuerySubscriptionsForCollection = (
   collection: string
-  // biome-ignore lint/suspicious/noExplicitAny: MongoDB query filter values are arbitrary user-supplied JSON
 ): {queryId: string; query: Record<string, any>}[] => {
-  // biome-ignore lint/suspicious/noExplicitAny: MongoDB query filter values are arbitrary user-supplied JSON
   const result: {queryId: string; query: Record<string, any>}[] = [];
 
   for (const [queryId, sub] of querySubscriptions) {

package/src/realtime/realtime.test.ts

@@ -1776,6 +1776,17 @@ describe("startChangeStreamWatcher", () => {
     };
   };
 
+  const invokeRegisteredChangeHandler = async (
+    mockStream: ReturnType<typeof createMockChangeStream>,
+    event: Record<string, unknown>
+  ): Promise<void> => {
+    const changeHandler = mockStream.listeners.get("change");
+    if (!changeHandler) {
+      throw new Error("expected change handler");
+    }
+    await changeHandler(event);
+  };
+
   const createMockIo = () => {
     const rooms = new Map<string, Set<string>>();
     const sockets = new Map<string, any>();
@@ -1864,9 +1875,7 @@ describe("startChangeStreamWatcher", () => {
     startChangeStreamWatcher(io, {}, true);
 
     // Trigger an insert change event
-    const changeHandler = mockStream.listeners.get("change");
-    expect(changeHandler).toBeDefined();
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       documentKey: {_id: "doc-1"},
       fullDocument: {_id: "doc-1", name: "Test Todo"},
       ns: {coll: "todos"},
@@ -1886,9 +1895,8 @@ describe("startChangeStreamWatcher", () => {
 
     startChangeStreamWatcher(io, {}, true);
 
-    const changeHandler = mockStream.listeners.get("change");
     // Trigger for an unregistered collection — should not throw
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       documentKey: {_id: "doc-1"},
       fullDocument: {_id: "doc-1"},
       ns: {coll: "unknown_collection"},
@@ -1927,9 +1935,8 @@ describe("startChangeStreamWatcher", () => {
 
     startChangeStreamWatcher(io, {}, true);
 
-    const changeHandler = mockStream.listeners.get("change");
     // Update event should be skipped because "update" not in methods
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       documentKey: {_id: "doc-1"},
       fullDocument: {_id: "doc-1", name: "Updated"},
       ns: {coll: "todos"},
@@ -1969,9 +1976,8 @@ describe("startChangeStreamWatcher", () => {
 
     startChangeStreamWatcher(io, {}, true);
 
-    const changeHandler = mockStream.listeners.get("change");
     // Hard delete (no fullDocument)
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       documentKey: {_id: "doc-1"},
       ns: {coll: "todos"},
       operationType: "delete",
@@ -2009,9 +2015,8 @@ describe("startChangeStreamWatcher", () => {
 
     startChangeStreamWatcher(io, {}, true);
 
-    const changeHandler = mockStream.listeners.get("change");
     // Hard delete for broadcast strategy
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       documentKey: {_id: "doc-1"},
       ns: {coll: "broadcasts"},
       operationType: "delete",
@@ -2049,8 +2054,7 @@ describe("startChangeStreamWatcher", () => {
 
     startChangeStreamWatcher(io, {}, true);
 
-    const changeHandler = mockStream.listeners.get("change");
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       documentKey: {_id: "doc-1"},
       fullDocument: {_id: "doc-1", name: "Updated", status: "done"},
       ns: {coll: "todos"},
@@ -2110,9 +2114,8 @@ describe("startChangeStreamWatcher", () => {
 
     startChangeStreamWatcher(io, {ignoredOperations: ["insert"]}, true);
 
-    const changeHandler = mockStream.listeners.get("change");
     // This insert should be skipped because "insert" is ignored
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       documentKey: {_id: "doc-1"},
       fullDocument: {_id: "doc-1"},
       ns: {coll: "todos"},
@@ -2132,15 +2135,14 @@ describe("startChangeStreamWatcher", () => {
 
     startChangeStreamWatcher(io, {}, true);
 
-    const changeHandler = mockStream.listeners.get("change");
     // Missing ns.coll
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       documentKey: {_id: "doc-1"},
       ns: {},
       operationType: "insert",
     });
     // Missing documentKey
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       documentKey: {},
       ns: {coll: "todos"},
       operationType: "insert",
@@ -2159,9 +2161,8 @@ describe("startChangeStreamWatcher", () => {
 
     startChangeStreamWatcher(io, {}, true);
 
-    const changeHandler = mockStream.listeners.get("change");
     // "drop" is not in our pipeline filter, should be skipped
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       operationType: "drop",
     });
   });
@@ -2257,9 +2258,8 @@ describe("startChangeStreamWatcher", () => {
 
     startChangeStreamWatcher(io, {}, true);
 
-    const changeHandler = mockStream.listeners.get("change");
     // Should not throw even though permission check throws
-    await changeHandler!({
+    await invokeRegisteredChangeHandler(mockStream, {
       documentKey: {_id: "doc-1"},
       fullDocument: {_id: "doc-1", name: "Test"},
       ns: {coll: "todos"},
@@ -2317,7 +2317,7 @@ describe("RealtimeApp.onServerCreated", () => {
   });
 
   const makeServer = (): import("http").Server => {
-    const http = require("http");
+    const http = require("node:http");
     const server = http.createServer();
     servers.push(server);
     return server;
@@ -2366,7 +2366,7 @@ describe("RealtimeApp.setupAdapter (private, via onServerCreated config)", () =>
   });
 
   const makeServer = (): import("http").Server => {
-    const http = require("http");
+    const http = require("node:http");
     const server = http.createServer();
     servers.push(server);
     return server;

package/src/realtime/realtimeApp.ts

@@ -60,7 +60,6 @@ export interface RealtimeSocketLike extends SocketWithDecodedToken {
   join: (room: string) => Promise<void> | void;
   leave: (room: string) => Promise<void> | void;
   emit: (event: string, payload: unknown) => void;
-  // biome-ignore lint/suspicious/noExplicitAny: Socket.io event handlers accept arbitrary argument shapes per event name
   on: (event: string, handler: (...args: any[]) => any) => void;
 }
 

package/src/realtime/registry.ts

@@ -17,7 +17,6 @@ export interface RealtimeRegistryEntry {
   /**
    * Full modelRouter options (for responseHandler, permissions, etc.).
    */
-  // biome-ignore lint/suspicious/noExplicitAny: registry stores heterogeneous models — narrowing the generic is not useful at the registry level
   options: ModelRouterOptions<any>;
 }
 

package/src/realtime/types.ts

@@ -19,10 +19,8 @@ export interface RealtimeConfig {
     | "owner"
     | "model"
     | "broadcast"
-    // biome-ignore lint/suspicious/noExplicitAny: doc is an arbitrary Mongoose document; consumers cast to their model type
     | ((doc: any, method: string, req: express.Request) => string[]);
   /** Custom serializer for real-time events. Falls back to the modelRouter responseHandler. */
-  // biome-ignore lint/suspicious/noExplicitAny: doc shape and return value are consumer-defined per model
   realtimeResponseHandler?: (doc: any, method: string) => any;
 }
 
@@ -39,7 +37,6 @@ export interface RealtimeEvent {
   /** Document ID */
   id: string;
   /** Serialized document data (omitted for hard deletes) */
-  // biome-ignore lint/suspicious/noExplicitAny: emitted document shape varies by model and serializer
   data?: any;
   /** Fields that were updated (for update events from change streams) */
   updatedFields?: string[];
@@ -105,7 +102,6 @@ export interface QuerySubscription {
   /** Collection tag (e.g. "todos") */
   collection: string;
   /** MongoDB-style query filter (e.g. {completed: false}) */
-  // biome-ignore lint/suspicious/noExplicitAny: MongoDB query filter values are arbitrary user-supplied JSON
   query: Record<string, any>;
   /** Client-provided queryId (ignored — server computes a canonical ID) */
   queryId?: string;