firebase-tools 15.0.0 → 15.2.0

package/lib/mcp/tools/crashlytics/events.js

@@ -1,13 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.batch_get_events = exports.list_events = void 0;
-const zod_1 = require("zod");
-const tool_1 = require("../../tool");
 const js_yaml_1 = require("js-yaml");
+const zod_1 = require("zod");
 const events_1 = require("../../../crashlytics/events");
-const types_1 = require("../../../crashlytics/types");
 const filters_1 = require("../../../crashlytics/filters");
-const util_1 = require("../../util");
+const types_1 = require("../../../crashlytics/types");
+const app_id_1 = require("../../resources/guides/app_id");
+const tool_1 = require("../../tool");
 const DUMP_OPTIONS = { lineWidth: 200 };
 function formatFrames(origFrames, maxFrames = 20) {
     const frames = origFrames || [];
@@ -33,7 +33,9 @@ function formatFrames(origFrames, maxFrames = 20) {
     return formatted;
 }
 function toText(event) {
-    var _a, _b;
+    if (!event) {
+        return {};
+    }
     const result = {};
     for (const [key, value] of Object.entries(event)) {
         if (key === "logs") {
@@ -46,7 +48,7 @@ function toText(event) {
             const breadcrumbs = value || [];
             const slicedBreadcrumbs = breadcrumbs.length > 10 ? breadcrumbs.slice(-10) : breadcrumbs;
             const breadcrumbLines = slicedBreadcrumbs.map((b) => {
-                const paramString = Object.entries(b.params)
+                const paramString = Object.entries(b?.params || {})
                     .map(([k, v]) => `${k}: ${v}`)
                     .join(", ");
                 const params = paramString ? ` { ${paramString} }` : "";
@@ -56,7 +58,7 @@ function toText(event) {
         }
         else if (key === "threads") {
             let threads = value || [];
-            if (((_a = event.issue) === null || _a === void 0 ? void 0 : _a.errorType) === types_1.ErrorType.FATAL || ((_b = event.issue) === null || _b === void 0 ? void 0 : _b.errorType) === types_1.ErrorType.ANR) {
+            if (event.issue?.errorType === types_1.ErrorType.FATAL || event.issue?.errorType === types_1.ErrorType.ANR) {
                 threads = threads.filter((t) => t.crashed || t.blamed);
             }
             const threadStrings = threads.map((thread) => {
@@ -109,13 +111,24 @@ exports.list_events = (0, tool_1.tool)("crashlytics", {
         requiresAuth: true,
     },
 }, async ({ appId, filter, pageSize }) => {
-    var _a;
-    if (!appId)
-        return (0, util_1.mcpError)(`Must specify 'appId' parameter.`);
-    if (!filter || (!filter.issueId && !filter.issueVariantId))
-        return (0, util_1.mcpError)(`Must specify 'filter.issueId' or 'filter.issueVariantId' parameters.`);
+    const result = { content: [] };
+    if (!appId) {
+        result.isError = true;
+        result.content.push({ type: "text", text: "Must specify 'appId' parameter" });
+        result.content.push({ type: "text", text: app_id_1.RESOURCE_CONTENT });
+    }
+    if (!filter || (!filter.issueId && !filter.issueVariantId)) {
+        result.isError = true;
+        result.content.push({
+            type: "text",
+            text: `Must specify 'filter.issueId' or 'filter.issueVariantId' parameters.`,
+        });
+    }
+    if (result.content.length > 0) {
+        return result;
+    }
     const response = await (0, events_1.listEvents)(appId, filter, pageSize);
-    const eventsContent = ((_a = response.events) === null || _a === void 0 ? void 0 : _a.map((e) => toText(e))) || [];
+    const eventsContent = response.events?.map((e) => toText(e)) || [];
     return {
         content: [{ type: "text", text: (0, js_yaml_1.dump)(eventsContent, DUMP_OPTIONS) }],
     };
@@ -139,13 +152,24 @@ exports.batch_get_events = (0, tool_1.tool)("crashlytics", {
         requiresAuth: true,
     },
 }, async ({ appId, names }) => {
-    var _a;
-    if (!appId)
-        return (0, util_1.mcpError)(`Must specify 'appId' parameter.`);
-    if (!names || names.length === 0)
-        return (0, util_1.mcpError)(`Must provide event resource names in name parameter.`);
+    const result = { content: [] };
+    if (!appId) {
+        result.isError = true;
+        result.content.push({ type: "text", text: "Must specify 'appId' parameter." });
+        result.content.push({ type: "text", text: app_id_1.RESOURCE_CONTENT });
+    }
+    if (!names || names.length === 0) {
+        result.isError = true;
+        result.content.push({
+            type: "text",
+            text: "Must provide event resource names in name parameter.",
+        });
+    }
+    if (result.content.length > 0) {
+        return result;
+    }
     const response = await (0, events_1.batchGetEvents)(appId, names);
-    const eventsContent = ((_a = response.events) === null || _a === void 0 ? void 0 : _a.map((e) => toText(e))) || [];
+    const eventsContent = response.events?.map((e) => toText(e)) || [];
     return {
         content: [{ type: "text", text: (0, js_yaml_1.dump)(eventsContent, DUMP_OPTIONS) }],
     };

package/lib/mcp/tools/crashlytics/index.js

@@ -12,11 +12,6 @@ exports.crashlyticsTools = [
     events_1.list_events,
     events_1.batch_get_events,
     notes_1.list_notes,
-    reports_1.get_top_issues,
-    reports_1.get_top_variants,
-    reports_1.get_top_versions,
-    reports_1.get_top_apple_devices,
-    reports_1.get_top_android_devices,
-    reports_1.get_top_operating_systems,
+    reports_1.get_report,
     issues_1.update_issue,
 ];

package/lib/mcp/tools/crashlytics/issues.js

@@ -2,11 +2,12 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.update_issue = exports.get_issue = void 0;
 const zod_1 = require("zod");
-const tool_1 = require("../../tool");
+const filters_1 = require("../../../crashlytics/filters");
 const issues_1 = require("../../../crashlytics/issues");
 const types_1 = require("../../../crashlytics/types");
-const filters_1 = require("../../../crashlytics/filters");
+const tool_1 = require("../../tool");
 const util_1 = require("../../util");
+const app_id_1 = require("../../resources/guides/app_id");
 exports.get_issue = (0, tool_1.tool)("crashlytics", {
     name: "get_issue",
     description: `Gets data for a Crashlytics issue, which can be used as a starting point for debugging.`,
@@ -22,10 +23,19 @@ exports.get_issue = (0, tool_1.tool)("crashlytics", {
         requiresAuth: true,
     },
 }, async ({ appId, issueId }) => {
-    if (!appId)
-        return (0, util_1.mcpError)(`Must specify 'appId' parameter.`);
-    if (!issueId)
-        return (0, util_1.mcpError)(`Must specify 'issueId' parameter.`);
+    const result = { content: [] };
+    if (!appId) {
+        result.isError = true;
+        result.content.push({ type: "text", text: "Must specify 'appId' parameter" });
+        result.content.push({ type: "text", text: app_id_1.RESOURCE_CONTENT });
+    }
+    if (!issueId) {
+        result.isError = true;
+        result.content.push({ type: "text", text: "Must specify 'issueId' parameter." });
+    }
+    if (result.content.length > 0) {
+        return result;
+    }
     return (0, util_1.toContent)(await (0, issues_1.getIssue)(appId, issueId));
 });
 exports.update_issue = (0, tool_1.tool)("crashlytics", {
@@ -46,11 +56,22 @@ exports.update_issue = (0, tool_1.tool)("crashlytics", {
         requiresAuth: true,
     },
 }, async ({ appId, issueId, state }) => {
-    if (!appId)
-        return (0, util_1.mcpError)(`Must specify 'app_id' parameter.`);
-    if (!issueId)
-        return (0, util_1.mcpError)(`Must specify 'issue_id' parameter.`);
-    if (!state)
-        return (0, util_1.mcpError)(`Must specify 'state' parameter.`);
+    const result = { content: [] };
+    if (!appId) {
+        result.isError = true;
+        result.content.push({ type: "text", text: "Must specify 'appId' parameter" });
+        result.content.push({ type: "text", text: app_id_1.RESOURCE_CONTENT });
+    }
+    if (!issueId) {
+        result.isError = true;
+        result.content.push({ type: "text", text: "Must specify 'issueId' parameter." });
+    }
+    if (!state) {
+        result.isError = true;
+        result.content.push({ type: "text", text: "Must specify 'state' parameter" });
+    }
+    if (result.content.length > 0) {
+        return result;
+    }
     return (0, util_1.toContent)(await (0, issues_1.updateIssue)(appId, issueId, state));
 });

package/lib/mcp/tools/crashlytics/reports.js

@@ -1,129 +1,78 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.get_top_operating_systems = exports.get_top_android_devices = exports.get_top_apple_devices = exports.get_top_versions = exports.get_top_variants = exports.get_top_issues = void 0;
-const tool_1 = require("../../tool");
-const util_1 = require("../../util");
-const reports_1 = require("../../../crashlytics/reports");
+exports.get_report = void 0;
+const js_yaml_1 = require("js-yaml");
 const filters_1 = require("../../../crashlytics/filters");
-function getReportContent(report, additionalPrompt) {
-    return async ({ appId, filter, pageSize }) => {
-        var _a;
-        if (!appId)
-            return (0, util_1.mcpError)(`Must specify 'appId' parameter.`);
-        filter !== null && filter !== void 0 ? filter : (filter = {});
-        if (!!filter.intervalStartTime && !filter.intervalEndTime) {
-            filter.intervalEndTime = new Date().toISOString();
-        }
-        if (report === reports_1.CrashlyticsReport.TopIssues && !!filter.issueId) {
-            delete filter.issueId;
-        }
-        (0, filters_1.validateEventFilters)(filter);
-        const reportResponse = (0, reports_1.simplifyReport)(await (0, reports_1.getReport)(report, appId, filter, pageSize));
-        if (!((_a = reportResponse.groups) === null || _a === void 0 ? void 0 : _a.length)) {
-            additionalPrompt = "This report response contains no results.";
-        }
-        if (additionalPrompt) {
-            reportResponse.usage = (reportResponse.usage || "").concat("\n", additionalPrompt);
-        }
-        return (0, util_1.toContent)(reportResponse);
+const reports_1 = require("../../../crashlytics/reports");
+const tool_1 = require("../../tool");
+const app_id_1 = require("../../resources/guides/app_id");
+const DUMP_OPTIONS = { lineWidth: 200 };
+const REPORT_ERROR_CONTENT = `
+Must specify the desired report:
+  * TOP_ISSUES - metrics grouped by *issue*.
+  * TOP_VARIANTS - metrics grouped by issue *variant*
+  * TOP_VERSIONS - metrics grouped by *version*
+  * TOP_OPERATING_SYSTEMS - metrics grouped by *operating system*
+  * TOP_ANDROID_DEVICES - metrics grouped by *device*
+  * TOP_APPLE_DEVICES - metrics grouped by *device*
+`.trim();
+function toText(response, filters) {
+    const result = {
+        name: response.name || "",
+        filters: (0, js_yaml_1.dump)(filters, DUMP_OPTIONS),
     };
+    for (const [key, value] of Object.entries(response)) {
+        if (key === "name") {
+            continue;
+        }
+        result[key] = (0, js_yaml_1.dump)(value, DUMP_OPTIONS);
+    }
+    return result;
 }
-exports.get_top_issues = (0, tool_1.tool)("crashlytics", {
-    name: "get_top_issues",
-    description: `Use this to count events and distinct impacted users, grouped by *issue*.
-      Groups are sorted by event count, in descending order.
-      Only counts events matching the given filters.`,
-    inputSchema: reports_1.ReportInputSchema,
-    annotations: {
-        title: "Get Crashlytics Top Issues Report",
-        readOnlyHint: true,
-    },
-    _meta: {
-        requiresAuth: true,
-    },
-}, getReportContent(reports_1.CrashlyticsReport.TopIssues, `The crashlytics_batch_get_event tool can retrieve the sample events in this response.
-    Pass the sampleEvent in the names field.
-    The crashlytics_list_events tool can retrieve a list of events for an issue in this response.
-    Pass the issue.id in the filter.issueId field.`));
-exports.get_top_variants = (0, tool_1.tool)("crashlytics", {
-    name: "get_top_variants",
-    description: `Counts events and distinct impacted users, grouped by issue *variant*.
-      Groups are sorted by event count, in descending order.
-      Only counts events matching the given filters.`,
-    inputSchema: reports_1.ReportInputSchema,
-    annotations: {
-        title: "Get Crashlytics Top Variants Report",
-        readOnlyHint: true,
-    },
-    _meta: {
-        requiresAuth: true,
-    },
-}, getReportContent(reports_1.CrashlyticsReport.TopVariants, `The crashlytics_get_top_issues tool can report the top issues for the variants in this response.
-    Pass the variant.displayName in the filter.variantDisplayNames field. 
-    The crashlytics_list_events tool can retrieve a list of events for a variant in this response.`));
-exports.get_top_versions = (0, tool_1.tool)("crashlytics", {
-    name: "get_top_versions",
-    description: `Counts events and distinct impacted users, grouped by *version*.
-      Groups are sorted by event count, in descending order.
-      Only counts events matching the given filters.`,
+exports.get_report = (0, tool_1.tool)("crashlytics", {
+    name: "get_report",
+    description: `Use this to request numerical reports from Crashlytics. The result aggregates the sum of events and impacted users, grouped by a dimension appropriate for that report. Agents must read the [Firebase Crashlytics Reports Guide](firebase://guides/crashlytics/reports) using the \`firebase_read_resources\` tool before calling to understand criticial prerequisites for requesting reports and how to interpret the results.
+    `.trim(),
     inputSchema: reports_1.ReportInputSchema,
     annotations: {
-        title: "Get Crashlytics Top Versions Report",
+        title: "Get Crashlytics Report",
         readOnlyHint: true,
     },
     _meta: {
         requiresAuth: true,
     },
-}, getReportContent(reports_1.CrashlyticsReport.TopVersions, `The crashlytics_get_top_issues tool can report the top issues for the versions in this response.
-    Pass the version.displayName in the filter.versionDisplayNames field. 
-    The crashlytics_list_events tool can retrieve a list of events for a version in this response.`));
-exports.get_top_apple_devices = (0, tool_1.tool)("crashlytics", {
-    name: "get_top_apple_devices",
-    description: `Counts events and distinct impacted users, grouped by apple *device*.
-      Groups are sorted by event count, in descending order.
-      Only counts events matching the given filters.
-      Only relevant for iOS, iPadOS and MacOS applications.`,
-    inputSchema: reports_1.ReportInputSchema,
-    annotations: {
-        title: "Get Crashlytics Top Apple Devices Report",
-        readOnlyHint: true,
-    },
-    _meta: {
-        requiresAuth: true,
-    },
-}, getReportContent(reports_1.CrashlyticsReport.TopAppleDevices, `The crashlytics_get_top_issues tool can report the top issues for the devices in this response.
-    Pass the device.displayName in the filter.deviceDisplayNames field. 
-    The crashlytics_list_events tool can retrieve a list of events for a device in this response.`));
-exports.get_top_android_devices = (0, tool_1.tool)("crashlytics", {
-    name: "get_top_android_devices",
-    description: `Counts events and distinct impacted users, grouped by android *device*.
-      Groups are sorted by event count, in descending order.
-      Only counts events matching the given filters.
-      Only relevant for Android applications.`,
-    inputSchema: reports_1.ReportInputSchema,
-    annotations: {
-        title: "Get Crashlytics Top Android Devices Report",
-        readOnlyHint: true,
-    },
-    _meta: {
-        requiresAuth: true,
-    },
-}, getReportContent(reports_1.CrashlyticsReport.TopAndroidDevices, `The crashlytics_get_top_issues tool can report the top issues for the devices in this response.
-    Pass the device.displayName in the filter.deviceDisplayNames field. 
-    The crashlytics_list_events tool can retrieve a list of events for a device in this response.`));
-exports.get_top_operating_systems = (0, tool_1.tool)("crashlytics", {
-    name: "get_top_operating_systems",
-    description: `Counts events and distinct impacted users, grouped by *operating system*.
-      Groups are sorted by event count, in descending order.
-      Only counts events matching the given filters.`,
-    inputSchema: reports_1.ReportInputSchema,
-    annotations: {
-        title: "Get Crashlytics Top Operating Systems Report",
-        readOnlyHint: true,
-    },
-    _meta: {
-        requiresAuth: true,
-    },
-}, getReportContent(reports_1.CrashlyticsReport.TopOperatingSystems, `The crashlytics_get_top_issues tool can report the top issues for the operating systems in this response.
-    Pass the operatingSystem.displayName in the filter.operatingSystemDisplayNames field. 
-    The crashlytics_list_events tool can retrieve a list of events for an operating system in this response.`));
+}, async ({ appId, report, pageSize, filter }) => {
+    const result = { content: [] };
+    if (!report) {
+        result.isError = true;
+        result.content.push({ type: "text", text: `Error: ${REPORT_ERROR_CONTENT}` });
+    }
+    if (!appId) {
+        result.isError = true;
+        result.content.push({ type: "text", text: "Must specify 'appId' parameter" });
+        result.content.push({ type: "text", text: app_id_1.RESOURCE_CONTENT });
+    }
+    try {
+        filter = (0, filters_1.validateEventFilters)(filter || {});
+    }
+    catch (error) {
+        result.isError = true;
+        result.content.push({ type: "text", text: `Error: ${error.message}` });
+    }
+    if (result.content.length > 0) {
+        return result;
+    }
+    const reportResponse = (0, reports_1.simplifyReport)(await (0, reports_1.getReport)(report, appId, filter, pageSize));
+    reportResponse.usage =
+        reportResponse.groups && reportResponse.groups.length
+            ? reportResponse.usage || ""
+            : "This report response contains no results.";
+    return {
+        content: [
+            {
+                type: "text",
+                text: (0, js_yaml_1.dump)(toText(reportResponse, filter), DUMP_OPTIONS),
+            },
+        ],
+    };
+});

package/lib/mcp/tools/dataconnect/list_services.js

@@ -68,15 +68,14 @@ exports.list_services = (0, tool_1.tool)("dataconnect", {
     const remoteOnlyServices = Array.from(serviceInfos.values()).filter((s) => !s.local);
     const output = [];
     function includeDeployedServiceInfo(deployed) {
-        var _a, _b;
-        if ((_a = deployed.schemas) === null || _a === void 0 ? void 0 : _a.length) {
+        if (deployed.schemas?.length) {
             output.push(`### Schemas`);
             for (const s of deployed.schemas) {
                 clearCCFEFields(s);
                 output.push((0, js_yaml_1.dump)(s));
             }
         }
-        if ((_b = deployed.connectors) === null || _b === void 0 ? void 0 : _b.length) {
+        if (deployed.connectors?.length) {
             output.push(`### Connectors`);
             for (const c of deployed.connectors) {
                 clearCCFEFields(c);

package/lib/mcp/tools/firestore/converter.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.firestoreDocumentToJson = exports.convertInputToValue = void 0;
+exports.convertInputToValue = convertInputToValue;
+exports.firestoreDocumentToJson = firestoreDocumentToJson;
 const logger_1 = require("../../../logger");
 function convertInputToValue(inputValue) {
     if (inputValue === null) {
@@ -47,9 +48,7 @@ function convertInputToValue(inputValue) {
     }
     return { nullValue: null };
 }
-exports.convertInputToValue = convertInputToValue;
 function firestoreValueToJson(firestoreValue) {
-    var _a, _b;
     if ("nullValue" in firestoreValue)
         return null;
     if ("booleanValue" in firestoreValue)
@@ -77,7 +76,7 @@ function firestoreValueToJson(firestoreValue) {
             value: [firestoreValue.geoPointValue.latitude, firestoreValue.geoPointValue.longitude],
         };
     if ("arrayValue" in firestoreValue)
-        return (_b = (_a = firestoreValue.arrayValue.values) === null || _a === void 0 ? void 0 : _a.map((v) => firestoreValueToJson(v))) !== null && _b !== void 0 ? _b : [];
+        return firestoreValue.arrayValue.values?.map((v) => firestoreValueToJson(v)) ?? [];
     if ("mapValue" in firestoreValue) {
         const map = firestoreValue.mapValue.fields || {};
         const obj = {};
@@ -100,4 +99,3 @@ function firestoreDocumentToJson(firestoreDoc) {
     }
     return result;
 }
-exports.firestoreDocumentToJson = firestoreDocumentToJson;

package/lib/mcp/tools/firestore/delete_document.js

@@ -38,7 +38,7 @@ exports.delete_document = (0, tool_1.tool)("firestore", {
         return (0, util_1.mcpError)(`None of the specified documents were found in project '${projectId}'`);
     }
     const firestoreDelete = new delete_1.FirestoreDelete(projectId, path, {
-        databaseId: database !== null && database !== void 0 ? database : "(default)",
+        databaseId: database ?? "(default)",
         urlPrefix: emulatorUrl,
     });
     await firestoreDelete.execute();

package/lib/mcp/tools/functions/get_logs.js

@@ -86,8 +86,8 @@ exports.get_logs = (0, tool_1.tool)("functions", {
         requiresProject: true,
     },
 }, async ({ function_names, page_size, order, page_token, min_severity, start_time, end_time, filter }, { projectId }) => {
-    const resolvedOrder = (order === null || order === void 0 ? void 0 : order.toLowerCase()) === "asc" ? "asc" : "desc";
-    const resolvedPageSize = page_size !== null && page_size !== void 0 ? page_size : 50;
+    const resolvedOrder = order?.toLowerCase() === "asc" ? "asc" : "desc";
+    const resolvedPageSize = page_size ?? 50;
     const normalizedSelectors = normalizeFunctionSelectors(function_names);
     const filterParts = [(0, functionslog_1.getApiFilter)(normalizedSelectors)];
     if (min_severity) {
@@ -115,18 +115,20 @@ exports.get_logs = (0, tool_1.tool)("functions", {
     try {
         const { entries, nextPageToken } = await (0, cloudlogging_1.listEntries)(projectId, combinedFilter, resolvedPageSize, resolvedOrder, page_token);
         const formattedEntries = entries.map((entry) => {
-            var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t;
-            const functionName = (_f = (_c = (_b = (_a = entry.resource) === null || _a === void 0 ? void 0 : _a.labels) === null || _b === void 0 ? void 0 : _b.function_name) !== null && _c !== void 0 ? _c : (_e = (_d = entry.resource) === null || _d === void 0 ? void 0 : _d.labels) === null || _e === void 0 ? void 0 : _e.service_name) !== null && _f !== void 0 ? _f : null;
-            const payload = (_k = (_j = (_h = (_g = entry.textPayload) !== null && _g !== void 0 ? _g : entry.jsonPayload) !== null && _h !== void 0 ? _h : entry.protoPayload) !== null && _j !== void 0 ? _j : entry.labels) !== null && _k !== void 0 ? _k : null;
+            const functionName = entry.resource?.labels?.function_name ?? entry.resource?.labels?.service_name ?? null;
+            const payload = entry.textPayload ?? entry.jsonPayload ?? entry.protoPayload ?? entry.labels ?? null;
             return {
-                timestamp: (_m = (_l = entry.timestamp) !== null && _l !== void 0 ? _l : entry.receiveTimestamp) !== null && _m !== void 0 ? _m : null,
-                severity: (_o = entry.severity) !== null && _o !== void 0 ? _o : "DEFAULT",
+                timestamp: entry.timestamp ?? entry.receiveTimestamp ?? null,
+                severity: entry.severity ?? "DEFAULT",
                 function: functionName,
-                message: (_r = (_q = (_p = entry.textPayload) !== null && _p !== void 0 ? _p : (entry.jsonPayload ? JSON.stringify(entry.jsonPayload) : undefined)) !== null && _q !== void 0 ? _q : (entry.protoPayload ? JSON.stringify(entry.protoPayload) : undefined)) !== null && _r !== void 0 ? _r : "",
+                message: entry.textPayload ??
+                    (entry.jsonPayload ? JSON.stringify(entry.jsonPayload) : undefined) ??
+                    (entry.protoPayload ? JSON.stringify(entry.protoPayload) : undefined) ??
+                    "",
                 payload,
                 log_name: entry.logName,
-                trace: (_s = entry.trace) !== null && _s !== void 0 ? _s : null,
-                span_id: (_t = entry.spanId) !== null && _t !== void 0 ? _t : null,
+                trace: entry.trace ?? null,
+                span_id: entry.spanId ?? null,
             };
         });
         const response = {
@@ -134,7 +136,7 @@ exports.get_logs = (0, tool_1.tool)("functions", {
             order: resolvedOrder,
             page_size: resolvedPageSize,
             entries: resolvedOrder === "asc" ? formattedEntries : formattedEntries.reverse(),
-            next_page_token: nextPageToken !== null && nextPageToken !== void 0 ? nextPageToken : null,
+            next_page_token: nextPageToken ?? null,
             has_more: Boolean(nextPageToken),
         };
         if (!entries.length) {
@@ -145,7 +147,7 @@ exports.get_logs = (0, tool_1.tool)("functions", {
         return (0, util_1.toContent)(response);
     }
     catch (err) {
-        const errMsg = (0, error_1.getErrMsg)((err === null || err === void 0 ? void 0 : err.original) || err, "Failed to retrieve Cloud Logging entries.");
+        const errMsg = (0, error_1.getErrMsg)(err?.original || err, "Failed to retrieve Cloud Logging entries.");
         return (0, util_1.mcpError)(errMsg);
     }
 });

package/lib/mcp/tools/functions/list_functions.js

@@ -42,7 +42,7 @@ exports.list_functions = (0, tool_1.tool)("functions", {
         return (0, util_1.toContent)(formattedList);
     }
     catch (err) {
-        const errMsg = (0, error_1.getErrMsg)((err === null || err === void 0 ? void 0 : err.original) || err, "Failed to list functions.");
+        const errMsg = (0, error_1.getErrMsg)(err?.original || err, "Failed to list functions.");
         return (0, util_1.mcpError)(errMsg);
     }
 });