@smartbear/mcp 0.12.0 → 0.12.1

package/dist/bugsnag/client/api/Error.js

@@ -1,5 +1,6 @@
+import { toUrlSearchParams } from "../filters.js";
 import { ErrorsApiFetchParamCreator, } from "./api.js";
-import { BaseAPI, getQueryParams } from "./base.js";
+import { BaseAPI } from "./base.js";
 export class ErrorAPI extends BaseAPI {
     static filterFields = ["url", "project_url", "events_url"];
     /**
@@ -14,9 +15,27 @@ export class ErrorAPI extends BaseAPI {
      * Get the latest Event in a Project, with optional filters
      * GET /projects/{project_id}/events
      */
-    async listEventsOnProject(projectId, base, sort, direction, perPage, filters, fullReports) {
-        const localVarFetchArgs = ErrorsApiFetchParamCreator(this.configuration).listEventsOnProject(projectId, base ?? undefined, sort, direction, perPage, undefined, fullReports, { query: filters });
-        return await this.requestArray(localVarFetchArgs.url, localVarFetchArgs.options, false);
+    async listEventsOnProject(projectId, base, sort, direction, perPage, filters, fullReports, nextUrl) {
+        if (nextUrl) {
+            // Don't allow override of these params when using nextUrl
+            direction = undefined;
+            sort = undefined;
+            base = undefined;
+        }
+        const localVarFetchArgs = ErrorsApiFetchParamCreator(this.configuration).listEventsOnProject(projectId, base ?? undefined, sort, direction, perPage, undefined, // Filters are encoded separately below
+        fullReports, undefined);
+        const url = new URL(nextUrl ?? localVarFetchArgs.url, this.configuration.basePath);
+        if (perPage) {
+            // Allow override of per page, even with nextUrl
+            url.searchParams.set("per_page", perPage.toString());
+        }
+        if (!nextUrl && filters) {
+            // Apply our own encoding of filters
+            toUrlSearchParams(filters).forEach((value, key) => {
+                url.searchParams.append(key, value);
+            });
+        }
+        return await this.requestArray(url.toString(), localVarFetchArgs.options, false);
     }
     /**
      * View an Event by ID
@@ -31,19 +50,26 @@ export class ErrorAPI extends BaseAPI {
      * GET /projects/{project_id}/errors
      */
     async listProjectErrors(projectId, base, sort, direction, perPage, filters, nextUrl) {
-        const options = getQueryParams(nextUrl, nextUrl ? undefined : filters);
         if (nextUrl) {
-            if (perPage) {
-                // Next links need to be used as-is to ensure results are consistent, so only the page size can be modified
-                // the others will get overridden
-                options.query.per_page = perPage.toString();
-            }
+            // Don't allow override of these params when using nextUrl
             direction = undefined;
             sort = undefined;
             base = undefined;
         }
-        const localVarFetchArgs = ErrorsApiFetchParamCreator(this.configuration).listProjectErrors(projectId, base ?? undefined, sort, direction, perPage, undefined, undefined, options);
-        return await this.requestArray(localVarFetchArgs.url, localVarFetchArgs.options, false);
+        const localVarFetchArgs = ErrorsApiFetchParamCreator(this.configuration).listProjectErrors(projectId, base ?? undefined, sort, direction, undefined, undefined, // Filters are encoded separately below
+        undefined, undefined);
+        const url = new URL(nextUrl ?? localVarFetchArgs.url, this.configuration.basePath);
+        if (perPage) {
+            // Allow override of per page, even with nextUrl
+            url.searchParams.set("per_page", perPage.toString());
+        }
+        if (!nextUrl && filters) {
+            // Apply our own encoding of filters
+            toUrlSearchParams(filters).forEach((value, key) => {
+                url.searchParams.append(key, value);
+            });
+        }
+        return await this.requestArray(url.toString(), localVarFetchArgs.options, false);
     }
     /**
      * Update an Error on a Project
@@ -58,7 +84,19 @@ export class ErrorAPI extends BaseAPI {
      * GET /projects/{project_id}/errors/{error_id}/pivots
      */
     async getPivotValuesOnAnError(projectId, errorId, filters, summarySize, pivots, perPage) {
-        const localVarFetchArgs = ErrorsApiFetchParamCreator(this.configuration).listPivotsOnAnError(projectId, errorId, undefined, summarySize, pivots, perPage, { query: filters });
-        return await this.requestArray(localVarFetchArgs.url, localVarFetchArgs.options, false);
+        const localVarFetchArgs = ErrorsApiFetchParamCreator(this.configuration).listPivotsOnAnError(projectId, errorId, undefined, // filters are encoded separately below
+        summarySize, pivots, undefined);
+        const url = new URL(localVarFetchArgs.url, this.configuration.basePath);
+        if (perPage) {
+            // Allow override of per page, even with nextUrl
+            url.searchParams.set("per_page", perPage.toString());
+        }
+        if (filters) {
+            // Apply our own encoding of filters
+            toUrlSearchParams(filters).forEach((value, key) => {
+                url.searchParams.append(key, value);
+            });
+        }
+        return await this.requestArray(url.toString(), localVarFetchArgs.options, false);
     }
 }

package/dist/bugsnag/client/api/Project.js

@@ -68,21 +68,25 @@ export class ProjectAPI extends BaseAPI {
      * Lists releases for a specific project.
      * GET /projects/{project_id}/release_groups
      * @param projectId The ID of the project.
-     * @param opts Options for listing releases, including filtering by release stage and visibility.
+     * @param releaseStageName The name of the release stage to filter by.
+     * @param topOnly Whether to only include top-level releases.
+     * @param visibleOnly Whether to only include visible releases.
+     * @param perPage The number of results per page.
+     * @param nextUrl Optional URL for next page (overrides other pagination params).
      * @returns A promise that resolves to an array of `ReleaseSummaryResponse` objects.
      */
     async listProjectReleaseGroups(projectId, releaseStageName, topOnly, visibleOnly, perPage, nextUrl) {
-        const options = getQueryParams(nextUrl);
-        // Next links need to be used as-is to ensure results are consistent, so only the page size can be modified
-        // the others will get overridden
         if (nextUrl) {
-            options.query.per_page = perPage ? perPage.toString() : undefined;
             topOnly = undefined;
             visibleOnly = undefined;
         }
-        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).listProjectReleaseGroups(projectId, releaseStageName, topOnly, visibleOnly, perPage, undefined, // pageToken passed in through options
-        options);
-        return await this.requestArray(localVarFetchArgs.url, localVarFetchArgs.options, false, // Paginate results
+        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).listProjectReleaseGroups(projectId, releaseStageName, topOnly, visibleOnly, perPage, undefined, undefined);
+        const url = new URL(nextUrl ?? localVarFetchArgs.url, this.configuration.basePath);
+        if (perPage) {
+            // Allow override of per page, even with nextUrl
+            url.searchParams.set("per_page", perPage.toString());
+        }
+        return await this.requestArray(url.toString(), localVarFetchArgs.options, false, // Paginate results
         ProjectAPI.releaseFields);
     }
     /**
@@ -122,17 +126,25 @@ export class ProjectAPI extends BaseAPI {
      * @returns A promise that resolves to an array of span groups.
      */
     async listProjectSpanGroups(projectId, sort, direction, perPage, offset, filters, starredOnly, nextUrl) {
-        const options = getQueryParams(nextUrl);
         if (nextUrl) {
-            options.query.per_page = perPage ? perPage.toString() : undefined;
+            sort = undefined;
+            direction = undefined;
+            offset = undefined;
         }
-        // Serialize filters if provided
-        if (filters) {
-            Object.assign(options.query, toUrlSearchParams(filters));
+        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).listProjectSpanGroups(projectId, sort, direction, perPage, offset, undefined, // Filters are encoded separately below
+        starredOnly, undefined);
+        const url = new URL(nextUrl ?? localVarFetchArgs.url, this.configuration.basePath);
+        if (perPage) {
+            // Allow override of per page, even with nextUrl
+            url.searchParams.set("per_page", perPage.toString());
         }
-        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).listProjectSpanGroups(projectId, sort, direction, perPage, offset, undefined, // Don't pass filters to API - already serialized in options
-        starredOnly, options);
-        return await this.requestArray(localVarFetchArgs.url, localVarFetchArgs.options, false);
+        if (!nextUrl && filters) {
+            // Apply our own encoding of filters
+            toUrlSearchParams(filters).forEach((value, key) => {
+                url.searchParams.append(key, value);
+            });
+        }
+        return await this.requestArray(url.toString(), localVarFetchArgs.options, false);
     }
     /**
      * Get detailed information about a specific span group.
@@ -143,12 +155,15 @@ export class ProjectAPI extends BaseAPI {
      * @returns A promise that resolves to the span group.
      */
     async getProjectSpanGroup(projectId, id, filters) {
-        const options = {};
+        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).getProjectSpanGroup(projectId, id, undefined, undefined);
+        const url = new URL(localVarFetchArgs.url, this.configuration.basePath);
         if (filters) {
-            options.query = toUrlSearchParams(filters);
+            // Apply our own encoding of filters
+            toUrlSearchParams(filters).forEach((value, key) => {
+                url.searchParams.append(key, value);
+            });
         }
-        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).getProjectSpanGroup(projectId, id, undefined, options);
-        return await this.requestObject(localVarFetchArgs.url, localVarFetchArgs.options);
+        return await this.requestObject(url.toString(), localVarFetchArgs.options);
     }
     /**
      * Get time-series performance metrics for a span group.
@@ -159,12 +174,15 @@ export class ProjectAPI extends BaseAPI {
      * @returns A promise that resolves to the timeline data.
      */
     async getProjectSpanGroupTimeline(projectId, id, filters) {
-        const options = {};
+        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).getProjectSpanGroupTimeline(projectId, id, undefined, undefined);
+        const url = new URL(localVarFetchArgs.url, this.configuration.basePath);
         if (filters) {
-            options.query = toUrlSearchParams(filters);
+            // Apply our own encoding of filters
+            toUrlSearchParams(filters).forEach((value, key) => {
+                url.searchParams.append(key, value);
+            });
         }
-        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).getProjectSpanGroupTimeline(projectId, id, undefined, options);
-        return await this.requestObject(localVarFetchArgs.url, localVarFetchArgs.options);
+        return await this.requestObject(url.toString(), localVarFetchArgs.options);
     }
     /**
      * Get distribution histogram of span durations for a span group.
@@ -179,8 +197,15 @@ export class ProjectAPI extends BaseAPI {
         if (filters) {
             options.query = toUrlSearchParams(filters);
         }
-        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).getProjectSpanGroupDistribution(projectId, id, undefined, options);
-        return await this.requestObject(localVarFetchArgs.url, localVarFetchArgs.options);
+        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).getProjectSpanGroupDistribution(projectId, id, undefined, undefined);
+        const url = new URL(localVarFetchArgs.url, this.configuration.basePath);
+        if (filters) {
+            // Apply our own encoding of filters
+            toUrlSearchParams(filters).forEach((value, key) => {
+                url.searchParams.append(key, value);
+            });
+        }
+        return await this.requestObject(url.toString(), localVarFetchArgs.options);
     }
     /**
      * List individual spans for a specific span group with filtering and sorting.
@@ -195,15 +220,23 @@ export class ProjectAPI extends BaseAPI {
      * @returns A promise that resolves to an array of spans.
      */
     async listSpansBySpanGroupId(projectId, id, filters, sort, direction, perPage, nextUrl) {
-        const options = getQueryParams(nextUrl);
         if (nextUrl) {
-            options.query.per_page = perPage ? perPage.toString() : undefined;
+            sort = undefined;
+            direction = undefined;
         }
-        if (filters) {
-            Object.assign(options.query, toUrlSearchParams(filters));
+        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).listSpansBySpanGroupId(projectId, id, undefined, sort, direction, perPage, undefined);
+        const url = new URL(nextUrl ?? localVarFetchArgs.url, this.configuration.basePath);
+        if (perPage) {
+            // Allow override of per page, even with nextUrl
+            url.searchParams.set("per_page", perPage.toString());
         }
-        const localVarFetchArgs = ProjectsApiFetchParamCreator(this.configuration).listSpansBySpanGroupId(projectId, id, undefined, sort, direction, perPage, options);
-        return await this.requestArray(localVarFetchArgs.url, localVarFetchArgs.options, false);
+        if (!nextUrl && filters) {
+            // Apply our own encoding of filters
+            toUrlSearchParams(filters).forEach((value, key) => {
+                url.searchParams.append(key, value);
+            });
+        }
+        return await this.requestArray(url.toString(), localVarFetchArgs.options, false);
     }
     /**
      * List all spans that belong to a specific trace.

package/dist/bugsnag/client.js

@@ -234,6 +234,17 @@ export class BugsnagClient {
         const projectEvents = await Promise.all(projectIds.map((projectId) => this.errorsApi.viewEventById(projectId, eventId).catch((_e) => null)));
         return projectEvents.find((event) => event && !!event.body)?.body || null;
     }
+    async validateEventFields(project, fields) {
+        if (fields) {
+            const eventFields = await this.getProjectEventFields(project);
+            const validKeys = new Set(eventFields.map((f) => f.displayId));
+            for (const key of Object.keys(fields)) {
+                if (!validKeys.has(key)) {
+                    throw new ToolError(`Invalid filter key: ${key}`);
+                }
+            }
+        }
+    }
     async getInputProject(projectId) {
         if (typeof projectId === "string") {
             const maybeProject = await this.getProject(projectId);
@@ -389,6 +400,7 @@ export class BugsnagClient {
                 error: [{ type: "eq", value: params.errorId }],
                 ...args.filters,
             };
+            await this.validateEventFields(project, filters);
             // Get the latest event for this error using the events endpoint with filters
             let latestEvent = null;
             try {
@@ -554,21 +566,13 @@ export class BugsnagClient {
         }, async (args, _extra) => {
             const params = listProjectErrorsInputSchema.parse(args);
             const project = await this.getInputProject(params.projectId);
-            // Validate filter keys against cached event fields
-            if (params.filters) {
-                const eventFields = await this.getProjectEventFields(project);
-                const validKeys = new Set(eventFields.map((f) => f.displayId));
-                for (const key of Object.keys(params.filters)) {
-                    if (!validKeys.has(key)) {
-                        throw new ToolError(`Invalid filter key: ${key}`);
-                    }
-                }
-            }
             const filters = {
                 "event.since": [{ type: "eq", value: "30d" }],
                 "error.status": [{ type: "eq", value: "open" }],
                 ...params.filters,
             };
+            // Validate filter keys against cached event fields
+            await this.validateEventFields(project, filters);
             const response = await this.errorsApi.listProjectErrors(project.id, null, params.sort, params.direction, params.perPage, filters, params.nextUrl);
             const result = {
                 data: response.body,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smartbear/mcp",
-  "version": "0.12.0",
+  "version": "0.12.1",
   "description": "MCP server for interacting SmartBear Products",
   "keywords": [
     "smartbear",