@dbsp/nql 1.0.1 → 1.0.3

package/dist/index.d.ts

@@ -438,6 +438,8 @@ interface NqlCompilerOptions {
     readonly recursiveKeywords?: readonly string[];
     /** BATCH-001: Named parameters for ANY(:param) expressions */
     readonly params?: Readonly<Record<string, unknown>>;
+    /** Maximum number of items allowed in an ANY(:param) array. Defaults to MAX_ANY_ITEMS (10000). */
+    readonly maxAnyItems?: number;
 }
 
 /**

package/dist/index.js

@@ -1,3 +1,4 @@
+import { isRankingWindowFunction } from '@dbsp/types';
 import { createToken, Lexer, CstParser } from 'chevrotain';
 
 // src/errors/types.ts
@@ -787,6 +788,7 @@ function getISOWeekCount(year) {
 }
 
 // src/compiler/compile-expression.ts
+var MAX_ANY_ITEMS = 1e4;
 function compileLogical(expr, ctx, fns, aliasContext, outerAliases) {
   if (expr.type === "binary") {
     const binary = expr;
@@ -961,7 +963,19 @@ function compileMembership(expr, ctx, fns, aliasContext, outerAliases) {
     }
     validateWhereField(ctx, field2, aliasContext, anyExpr.column);
     const rawValues = ctx.params[anyExpr.paramName];
-    const values2 = Array.isArray(rawValues) ? rawValues : [];
+    if (!Array.isArray(rawValues)) {
+      throw new NqlSemanticException(
+        NqlErrorCodes.SEM_INVALID_SYNTAX,
+        `ANY(:${anyExpr.paramName}) requires an array argument but received ${rawValues === void 0 ? "undefined (parameter not bound)" : typeof rawValues}`
+      );
+    }
+    if (rawValues.length > ctx.maxAnyItems) {
+      throw new NqlSemanticException(
+        NqlErrorCodes.SEM_INVALID_SYNTAX,
+        `ANY(:${anyExpr.paramName}) array length ${rawValues.length} exceeds maximum of ${ctx.maxAnyItems}`
+      );
+    }
+    const values2 = rawValues;
     return { kind: "any", field: field2, values: values2 };
   }
   const inExpr = expr;
@@ -995,7 +1009,6 @@ function compileMembership(expr, ctx, fns, aliasContext, outerAliases) {
     const result2 = {
       kind: "in",
       field,
-      values: [],
       subquery
     };
     if (inExpr.negated) {
@@ -1438,8 +1451,9 @@ function resolveBindingsInWhere(where, bindings) {
   if (!bindings || bindings.size === 0) return where;
   if (where.kind === "in") {
     const inWhere = where;
-    if (inWhere.values && inWhere.values.length === 1) {
-      const val = inWhere.values[0];
+    const inValues = inWhere.subquery ? void 0 : inWhere.values;
+    if (inValues && inValues.length === 1) {
+      const val = inValues[0];
       if (val && typeof val === "object" && "$ref" in val) {
         const ref = val.$ref;
         if (bindings.has(ref)) {
@@ -1459,7 +1473,6 @@ function resolveBindingsInWhere(where, bindings) {
           return {
             kind: "in",
             field: inWhere.field,
-            values: [],
             subquery: cteRef
           };
         }
@@ -1494,8 +1507,6 @@ function extractBindName(stmt) {
   }
   return void 0;
 }
-
-// src/compiler/compile-select.ts
 function compileSelectClause(clause, ctx, fns) {
   if (clause.items.length === 1 && clause.items[0]?.type === "star") {
     return { type: "all" };
@@ -1620,17 +1631,36 @@ function compileSelectExpression(item, ctx, fns) {
       }
       return { field: f, direction: o.direction };
     }) : void 0;
+    const over = {
+      ...partitionBy && { partitionBy },
+      ...orderBy && { orderBy }
+    };
+    const alias = exprItem.alias ?? fn;
+    if (isRankingWindowFunction(fn)) {
+      return {
+        kind: "window",
+        function: fn,
+        alias,
+        over
+      };
+    }
+    if (fn === "lag" || fn === "lead") {
+      return {
+        kind: "window",
+        function: fn,
+        field: field ?? "",
+        alias,
+        ...offset !== void 0 && { offset },
+        ...defaultValue !== void 0 && { defaultValue },
+        over
+      };
+    }
     return {
       kind: "window",
       function: fn,
       ...field !== void 0 && { field },
-      alias: exprItem.alias ?? fn,
-      ...offset !== void 0 && { offset },
-      ...defaultValue !== void 0 && { defaultValue },
-      over: {
-        ...partitionBy && { partitionBy },
-        ...orderBy && { orderBy }
-      }
+      alias,
+      over
     };
   }
   if (expr.type === "subquery") {
@@ -1900,13 +1930,22 @@ var NqlCompiler = class {
     const recursive = options?.recursiveKeywords ?? DEFAULT_RECURSIVE_KEYWORDS;
     const recursiveKeywords = new Set(recursive.map((k) => k.toLowerCase()));
     const validator = schema ? new ColumnValidator(schema) : null;
+    const maxAnyItemsRaw = options?.maxAnyItems;
+    if (maxAnyItemsRaw !== void 0) {
+      if (!Number.isSafeInteger(maxAnyItemsRaw) || maxAnyItemsRaw <= 0) {
+        throw new Error(
+          `Invalid maxAnyItems: ${maxAnyItemsRaw}. Must be a positive integer.`
+        );
+      }
+    }
     this.ctx = {
       currentFromTable: void 0,
       currentRelationTarget: void 0,
       pseudoColumnKeywords,
       recursiveKeywords,
       validator,
-      params: options?.params ?? {}
+      params: options?.params ?? {},
+      maxAnyItems: maxAnyItemsRaw ?? MAX_ANY_ITEMS
     };
     this.fns = {
       compileQuery: (query, ctx) => compileQuery(query, ctx, this.fns),