@boostercloud/framework-provider-azure 3.4.4-debug.1 → 3.4.4

package/dist/helpers/query-helper.js

@@ -16,6 +16,7 @@ async function replaceOrDeleteItem(cosmosDb, container, config, id, partitionKey
         await cosmosDb.database(config.resourceNames.applicationStack).container(container).item(id, partitionKey).delete();
     }
 }
+const DEFAULT_PAGE_SIZE = 100;
 async function search(cosmosDb, config, containerName, filters, limit, afterCursor, paginatedVersion = false, order, projections = '*') {
     const logger = (0, framework_common_helpers_1.getLogger)(config, 'query-helper#search');
     const filterExpression = buildFilterExpression(filters);
@@ -36,85 +37,50 @@ async function search(cosmosDb, config, containerName, filters, limit, afterCurs
         // - Legacy cursors: Numeric cursor.id values must use OFFSET/LIMIT for backward compatibility
         const canUseContinuationToken = !isDistinctQuery;
         const hasLegacyCursor = typeof (afterCursor === null || afterCursor === void 0 ? void 0 : afterCursor.id) === 'string' && /^\d+$/.test(afterCursor.id);
-        logger.debug('🔍 PAGINATION DEBUG - Initial analysis:', {
-            isDistinctQuery,
-            canUseContinuationToken,
-            hasLegacyCursor,
-            afterCursor,
-            limit,
-            finalQuery,
-        });
         // Use Cosmos DB's continuation token pagination
         const feedOptions = {};
-        if (limit) {
-            feedOptions.maxItemCount = limit;
-        }
         // Extract continuation token from the cursor (backward compatibility)
         if (afterCursor === null || afterCursor === void 0 ? void 0 : afterCursor.continuationToken) {
             feedOptions.continuationToken = afterCursor.continuationToken;
-            logger.debug('🔍 PAGINATION DEBUG - Using provided continuation token:', {
-                continuationToken: afterCursor.continuationToken,
-                feedOptions,
-            });
         }
-        else if (!canUseContinuationToken || hasLegacyCursor) {
+        // Azure Cosmos DB requires maxItemCount when using continuation tokens
+        // Always set maxItemCount in the continuation token path to ensure consistent page sizes
+        if (limit || (afterCursor === null || afterCursor === void 0 ? void 0 : afterCursor.continuationToken) || canUseContinuationToken) {
+            feedOptions.maxItemCount = limit !== null && limit !== void 0 ? limit : DEFAULT_PAGE_SIZE;
+        }
+        if (!(afterCursor === null || afterCursor === void 0 ? void 0 : afterCursor.continuationToken) && (!canUseContinuationToken || hasLegacyCursor)) {
             // Legacy cursor format - fallback to OFFSET for backward compatibility
-            const offset = (afterCursor === null || afterCursor === void 0 ? void 0 : afterCursor.id) ? parseInt(afterCursor.id) : 0;
-            let legacyQuery = `${finalQuery} OFFSET ${offset}`;
-            if (limit) {
-                legacyQuery += ` LIMIT ${limit} `;
-            }
+            const parsedLegacyId = (afterCursor === null || afterCursor === void 0 ? void 0 : afterCursor.id) ? parseInt(afterCursor.id, 10) : NaN;
+            const offset = Number.isFinite(parsedLegacyId) ? parsedLegacyId : 0;
+            // Azure Cosmos DB requires LIMIT when using OFFSET
+            const effectiveLimit = limit !== null && limit !== void 0 ? limit : DEFAULT_PAGE_SIZE;
+            const legacyQuery = `${finalQuery} OFFSET ${offset} LIMIT ${effectiveLimit} `;
             const legacyQuerySpec = { ...querySpec, query: legacyQuery };
-            logger.debug('🔍 PAGINATION DEBUG - Using LEGACY OFFSET/LIMIT approach:', {
-                reason: !canUseContinuationToken ? 'DISTINCT query detected' : 'Legacy numeric cursor detected',
-                offset,
-                legacyQuery,
-                legacyQuerySpec,
-            });
             const { resources } = await container.items.query(legacyQuerySpec).fetchAll();
             const processedResources = processResources(resources);
-            logger.debug('🔍 PAGINATION DEBUG - Legacy query results:', {
-                resourcesCount: (resources === null || resources === void 0 ? void 0 : resources.length) || 0,
-                processedResourcesCount: processedResources.length,
-                nextCursor: { id: (offset + processedResources.length).toString() },
-            });
             return {
                 items: processedResources !== null && processedResources !== void 0 ? processedResources : [],
                 count: processedResources.length,
                 cursor: {
-                    id: (offset + processedResources.length).toString(),
+                    id: (offset + effectiveLimit).toString(),
                 },
             };
         }
-        logger.debug('🔍 PAGINATION DEBUG - About to execute continuation token query with feedOptions:', feedOptions);
         const queryIterator = container.items.query(querySpec, feedOptions);
         const { resources, continuationToken } = await queryIterator.fetchNext();
-        logger.debug('🔍 PAGINATION DEBUG - Cosmos SDK response:', {
-            resourcesCount: (resources === null || resources === void 0 ? void 0 : resources.length) || 0,
-            continuationTokenReceived: !!continuationToken,
-            continuationTokenValue: continuationToken,
-            continuationTokenType: typeof continuationToken,
-            continuationTokenLength: continuationToken === null || continuationToken === void 0 ? void 0 : continuationToken.length,
-        });
         const finalResources = processResources(resources || []);
+        // cursor.id advances by the page size (limit) to maintain consistent page-based offsets
+        // that frontends rely on (e.g., limit=5 produces cursors 5, 10 ,15, ...)
+        const parsedId = (afterCursor === null || afterCursor === void 0 ? void 0 : afterCursor.id) ? parseInt(afterCursor.id, 10) : NaN;
+        const previousOffset = Number.isFinite(parsedId) ? parsedId : 0;
+        const effectiveLimit = limit !== null && limit !== void 0 ? limit : DEFAULT_PAGE_SIZE;
         let cursor;
         if (continuationToken) {
-            cursor = { continuationToken };
-            logger.debug('🔍 PAGINATION DEBUG - Setting cursor with continuation token:', cursor);
+            cursor = { continuationToken, id: (previousOffset + effectiveLimit).toString() };
         }
         else if (finalResources.length > 0) {
-            const currentOffset = (afterCursor === null || afterCursor === void 0 ? void 0 : afterCursor.id) && !isNaN(parseInt(afterCursor.id)) ? parseInt(afterCursor.id) : 0;
-            cursor = { id: (currentOffset + finalResources.length).toString() };
-            logger.debug('🔍 PAGINATION DEBUG - No continuation token, setting fallback cursor:', cursor);
-        }
-        else {
-            logger.debug('🔍 PAGINATION DEBUG - No continuation token and no resources, no cursor set');
+            cursor = { id: (previousOffset + effectiveLimit).toString() };
         }
-        logger.debug('🔍 PAGINATION DEBUG - Final response:', {
-            itemsCount: finalResources.length,
-            cursor,
-            hasMoreResults: !!continuationToken,
-        });
         return {
             items: finalResources,
             count: finalResources.length,

package/dist/types/azure-func-types.d.ts

@@ -0,0 +1,116 @@
+/**
+ * Azure Functions v4 Programming Model Types
+ *
+ * These types provide a wrapper for Azure Functions vs inputs to standardize
+ * the interface between function handlers and Booster adapters
+ */
+import { HttpRequest, InvocationContext } from '@azure/functions';
+/**
+ * Standard wrapper for HTTP-triggered Azure Function v4 inputs.
+ * This replaces the v3 Context object for HTTP triggers.
+ */
+export interface AzureHttpFunctionInput {
+    request: HttpRequest;
+    context: InvocationContext;
+}
+/**
+ * Standard wrapper for CosmosDB-triggered Azure Function v4 inputs.
+ */
+export interface AzureCosmosDBFunctionInput {
+    documents: unknown[];
+    context: InvocationContext;
+}
+/**
+ * Standard wrapper for EventHub-triggered Azure Function v4 inputs.
+ */
+export interface AzureEventHubFunctionInput {
+    messages: unknown[];
+    context: InvocationContext;
+}
+/**
+ * Standard wrapper for Timer-triggered Azure Function v4 inputs.
+ */
+export interface AzureTimerFunctionInput {
+    timer: {
+        isPastDue: boolean;
+        schedule: {
+            adjustForDST: boolean;
+        };
+        scheduleStatus?: {
+            last: string;
+            next: string;
+            lastUpdated: string;
+        };
+    };
+    context: InvocationContext;
+}
+/**
+ * Web PubSub connection context structure
+ */
+export interface WebPubSubConnectionContext {
+    connectionId: string;
+    userId?: string;
+    hub: string;
+    eventType: string;
+    eventName: string;
+}
+/**
+ * Standard wrapper for Web PubSub-triggered Azure Function v4 inputs.
+ * Note: connectionContext can be found in either request.connectionContext
+ * or context.triggerMetadata.connectionContext depending on Azure Functions version.
+ */
+export interface AzureWebPubSubFunctionInput {
+    request: {
+        connectionContext?: WebPubSubConnectionContext;
+        data?: unknown;
+        dataType?: string;
+    };
+    context: InvocationContext & {
+        triggerMetadata?: {
+            connectionContext?: WebPubSubConnectionContext;
+        };
+    };
+}
+/**
+ * Type alias for the raw request that Booster adapters receive.
+ * This can be any of the v4 input types.
+ */
+export type AzureFunctionRawRequest = AzureHttpFunctionInput | AzureCosmosDBFunctionInput | AzureEventHubFunctionInput | AzureTimerFunctionInput | AzureWebPubSubFunctionInput;
+/**
+ * Type guard to check if the input is an HTTP function input
+ * @param input - The input to check
+ * @returns True if the input is an HTTP function input, false otherwise
+ */
+export declare function isHttpFunctionInput(input: unknown): input is AzureHttpFunctionInput;
+/**
+ * Type guard to check if the input is a CosmosDB function input
+ * @param input - The input to check
+ * @returns True if the input is a CosmosDB function input, false otherwise
+ */
+export declare function isCosmosDBFunctionInput(input: unknown): input is AzureCosmosDBFunctionInput;
+/**
+ * Type guard to check if the input is an EventHub function input
+ * @param input - The input to check
+ * @returns True if the input is an EventHub function input, false otherwise
+ */
+export declare function isEventHubFunctionInput(input: unknown): input is AzureEventHubFunctionInput;
+/**
+ * Type guard to check if the input is a Timer function input
+ * @param input - The input to check
+ * @returns True if the input is a Timer function input, false otherwise
+ */
+export declare function isTimerFunctionInput(input: unknown): input is AzureTimerFunctionInput;
+/**
+ * Type guard to check if the input is a Web PubSub function input.
+ * Checks for connectionContext in both request.connectionContext and context.triggerMetadata.connectionContext
+ * as Azure Functions v4 may provide it in different locations.
+ * @param input - The input to check
+ * @returns True if the input is a Web PubSub function input, false otherwise
+ */
+export declare function isWebPubSubFunctionInput(input: unknown): input is AzureWebPubSubFunctionInput;
+/**
+ * Helper to extract connectionContext from either location in Web PubSub input
+ * @param input - The Azure Web PubSub function input
+ * @returns The connection context if present, undefined otherwise
+ */
+export declare function getWebPubSubConnectionContext(input: AzureWebPubSubFunctionInput): WebPubSubConnectionContext | undefined;

package/dist/types/azure-func-types.js

@@ -0,0 +1,87 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.isHttpFunctionInput = isHttpFunctionInput;
+exports.isCosmosDBFunctionInput = isCosmosDBFunctionInput;
+exports.isEventHubFunctionInput = isEventHubFunctionInput;
+exports.isTimerFunctionInput = isTimerFunctionInput;
+exports.isWebPubSubFunctionInput = isWebPubSubFunctionInput;
+exports.getWebPubSubConnectionContext = getWebPubSubConnectionContext;
+/**
+ * Type guard to check if the input is an HTTP function input
+ * @param input - The input to check
+ * @returns True if the input is an HTTP function input, false otherwise
+ */
+function isHttpFunctionInput(input) {
+    var _a;
+    return (typeof input === 'object' &&
+        input !== null &&
+        'request' in input &&
+        'context' in input &&
+        typeof ((_a = input.request) === null || _a === void 0 ? void 0 : _a.method) === 'string');
+}
+/**
+ * Type guard to check if the input is a CosmosDB function input
+ * @param input - The input to check
+ * @returns True if the input is a CosmosDB function input, false otherwise
+ */
+function isCosmosDBFunctionInput(input) {
+    return (typeof input === 'object' &&
+        input !== null &&
+        'documents' in input &&
+        'context' in input &&
+        Array.isArray(input.documents));
+}
+/**
+ * Type guard to check if the input is an EventHub function input
+ * @param input - The input to check
+ * @returns True if the input is an EventHub function input, false otherwise
+ */
+function isEventHubFunctionInput(input) {
+    return (typeof input === 'object' &&
+        input !== null &&
+        'messages' in input &&
+        'context' in input &&
+        Array.isArray(input.messages));
+}
+/**
+ * Type guard to check if the input is a Timer function input
+ * @param input - The input to check
+ * @returns True if the input is a Timer function input, false otherwise
+ */
+function isTimerFunctionInput(input) {
+    var _a;
+    return (typeof input === 'object' &&
+        input !== null &&
+        'timer' in input &&
+        'context' in input &&
+        typeof ((_a = input.timer) === null || _a === void 0 ? void 0 : _a.isPastDue) === 'boolean');
+}
+/**
+ * Type guard to check if the input is a Web PubSub function input.
+ * Checks for connectionContext in both request.connectionContext and context.triggerMetadata.connectionContext
+ * as Azure Functions v4 may provide it in different locations.
+ * @param input - The input to check
+ * @returns True if the input is a Web PubSub function input, false otherwise
+ */
+function isWebPubSubFunctionInput(input) {
+    var _a, _b, _c, _d, _e;
+    if (typeof input !== 'object' || input === null || !('request' in input) || !('context' in input)) {
+        return false;
+    }
+    const typedInput = input;
+    // Check connectionContext in request.connectionContext
+    if (typeof ((_b = (_a = typedInput.request) === null || _a === void 0 ? void 0 : _a.connectionContext) === null || _b === void 0 ? void 0 : _b.connectionId) === 'string') {
+        return true;
+    }
+    // Check connectionContext in context.triggerMetadata.connectionContext
+    return typeof ((_e = (_d = (_c = typedInput.context) === null || _c === void 0 ? void 0 : _c.triggerMetadata) === null || _d === void 0 ? void 0 : _d.connectionContext) === null || _e === void 0 ? void 0 : _e.connectionId) === 'string';
+}
+/**
+ * Helper to extract connectionContext from either location in Web PubSub input
+ * @param input - The Azure Web PubSub function input
+ * @returns The connection context if present, undefined otherwise
+ */
+function getWebPubSubConnectionContext(input) {
+    var _a, _b, _c, _d;
+    return (_b = (_a = input.request) === null || _a === void 0 ? void 0 : _a.connectionContext) !== null && _b !== void 0 ? _b : (_d = (_c = input.context) === null || _c === void 0 ? void 0 : _c.triggerMetadata) === null || _d === void 0 ? void 0 : _d.connectionContext;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boostercloud/framework-provider-azure",
-  "version": "3.4.4-debug.1",
+  "version": "3.4.4",
   "description": "Handle Booster's integration with Azure",
   "keywords": [
     "framework-provider-azure"
@@ -27,14 +27,14 @@
     "@azure/functions": "^1.2.2",
     "@azure/identity": "~4.7.0",
     "@azure/event-hubs": "5.11.1",
-    "@boostercloud/framework-common-helpers": "workspace:^3.4.3",
-    "@boostercloud/framework-types": "workspace:^3.4.3",
+    "@boostercloud/framework-common-helpers": "^3.4.4",
+    "@boostercloud/framework-types": "^3.4.4",
     "tslib": "^2.4.0",
     "@effect-ts/core": "^0.60.4",
     "@azure/web-pubsub": "~1.1.0"
   },
   "devDependencies": {
-    "@boostercloud/eslint-config": "workspace:^3.4.3",
+    "@boostercloud/eslint-config": "^3.4.4",
     "@types/chai": "4.2.18",
     "@types/chai-as-promised": "7.1.4",
     "@types/faker": "5.1.5",
@@ -61,27 +61,16 @@
     "typescript": "5.7.3",
     "eslint-plugin-unicorn": "~44.0.2"
   },
+  "bugs": {
+    "url": "https://github.com/boostercloud/booster/issues"
+  },
   "scripts": {
     "format": "prettier --write --ext '.js,.ts' **/*.ts **/*/*.ts",
     "lint:check": "eslint --ext '.js,.ts' **/*.ts",
     "lint:fix": "eslint --quiet --fix --ext '.js,.ts' **/*.ts",
     "build": "tsc -b tsconfig.json",
     "clean": "rimraf ./dist tsconfig.tsbuildinfo",
-    "prepack": "tsc -b tsconfig.json",
     "test:provider-azure": "npm run test",
     "test": "nyc --extension .ts mocha --forbid-only \"test/**/*.test.ts\""
-  },
-  "bugs": {
-    "url": "https://github.com/boostercloud/booster/issues"
-  },
-  "pnpm": {
-    "overrides": {
-      "pac-resolver@<5.0.0": ">=5.0.0",
-      "underscore@>=1.3.2 <1.12.1": ">=1.13.6",
-      "node-fetch@<2.6.7": ">=2.6.7",
-      "ws@>=7.0.0 <7.4.6": ">=7.4.6",
-      "nanoid@>=3.0.0 <3.1.31": ">=3.1.31",
-      "node-fetch@<2.6.1": ">=2.6.1"
-    }
   }
-}
+}