@robotical/webapp-types 3.21.9 → 3.21.10

package/dist-types/package.json

@@ -1,6 +1,6 @@
 {
     "name": "marty-web-app",
-    "version": "3.21.9",
+    "version": "3.21.10",
     "private": true,
     "dependencies": {
         "@radix-ui/react-dropdown-menu": "^2.1.2",

package/dist-types/src/application/ApplicationManager/ApplicationManager.js

@@ -58,7 +58,6 @@ import { AppSentMessage } from "../../types/communication-between-apps/wrapper-c
 import VerificationModalPhoneApp from "../../components/modals/VerificationModalPhoneApp";
 import { RaftConnEvent, } from "@robdobsn/raftjs";
 import Logger from "../../services/logger/Logger";
-import { getSafeErrorMetadata } from "../../services/observability/sanitizeTelemetry";
 import isPhoneApp from "../../utils/phone-app-communication/is-phone-app";
 import DisconnectConfirmationModal from "../../components/modals/DisconnectConfirmation";
 import Toaster from "../../utils/Toaster";
@@ -83,7 +82,7 @@ var UNSUPPORTED_WEB_BLUETOOTH_BROWSER_MESSAGE = "This browser can't connect to M
 var publishMissedConnectionEvent = function (raftId) {
     window.wrapperCommunicator
         .sendMessageAndWait(AppSentMessage.GET_MISSED_CONN_EVENT, { raftId: raftId })
-        .catch(function (error) { return Logger.warn(SHOW_LOGS, TAG, "Failed to get missed connection event", { raftId: raftId }, getSafeErrorMetadata(error)); });
+        .catch(function (error) { return Logger.warn(SHOW_LOGS, TAG, "Failed to get missed connection event", { raftId: raftId }, error); });
 };
 var canUseWebBluetooth = function () {
     var bluetooth = navigator.bluetooth;
@@ -211,7 +210,7 @@ var ApplicationManager = /** @class */ (function () {
                         return [3 /*break*/, 4];
                     case 3:
                         error_1 = _e.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to determine phone app version for Wi-Fi", getSafeErrorMetadata(error_1));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to determine phone app version for Wi-Fi", error_1);
                         return [3 /*break*/, 4];
                     case 4:
                         if (isPhoneWrapperWifiSupported((_a = this.deviceInfo) === null || _a === void 0 ? void 0 : _a.appVersion)) {
@@ -285,7 +284,7 @@ var ApplicationManager = /** @class */ (function () {
                     case 2: return [2 /*return*/, (_a.sent()) || ""];
                     case 3:
                         error_2 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to read the RAFT name", { raftId: raft.id }, getSafeErrorMetadata(error_2));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to read the RAFT name", { raftId: raft.id }, error_2);
                         return [2 /*return*/, ""];
                     case 4: return [2 /*return*/];
                 }
@@ -350,7 +349,7 @@ var ApplicationManager = /** @class */ (function () {
                         return [3 /*break*/, 5];
                     case 4:
                         error_3 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to publish the QR verified event", { raftId: raft.id }, getSafeErrorMetadata(error_3));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to publish the QR verified event", { raftId: raft.id }, error_3);
                         this.publishLocalVerifiedConnection(raft);
                         return [3 /*break*/, 5];
                     case 5: return [2 /*return*/];
@@ -879,7 +878,7 @@ var ApplicationManager = /** @class */ (function () {
                     case 7: return [3 /*break*/, 9];
                     case 8:
                         e_1 = _a.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to start discovery", getSafeErrorMetadata(e_1));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to start discovery", e_1);
                         return [3 /*break*/, 9];
                     case 9: return [3 /*break*/, 18];
                     case 10:
@@ -908,7 +907,7 @@ var ApplicationManager = /** @class */ (function () {
                     case 15: return [3 /*break*/, 17];
                     case 16:
                         e_2 = _a.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to connect to new robot", getSafeErrorMetadata(e_2));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to connect to new robot", e_2);
                         if (isWifiConnection) {
                             this.toaster.error(i18n.t("connection.wifi_connect_failed", {
                                 defaultValue: "Couldn't connect over Wi-Fi. Check the address, local-network permission, and that the device is online.",
@@ -959,7 +958,7 @@ var ApplicationManager = /** @class */ (function () {
                         return [3 /*break*/, 6];
                     case 5:
                         e_3 = _a.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to disconnect from robot", getSafeErrorMetadata(e_3));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to disconnect from robot", e_3);
                         return [3 /*break*/, 6];
                     case 6: return [2 /*return*/];
                 }

package/dist-types/src/application/RAFTs/Marty/CalibrationManager.js

@@ -36,7 +36,6 @@ var __generator = (this && this.__generator) || function (thisArg, body) {
 };
 import { RaftOKFail } from "@robdobsn/raftjs";
 import Logger from "../../../services/logger/Logger";
-import { getSafeErrorMetadata } from "../../../services/observability/sanitizeTelemetry";
 var SHOW_LOGS = true;
 var TAG = "CalibrationManager";
 var CalibrationManager = /** @class */ (function () {
@@ -110,7 +109,7 @@ var CalibrationManager = /** @class */ (function () {
                         return [3 /*break*/, 8];
                     case 7:
                         error_1 = _e.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Joint calibration failed", { joint: jnt }, error_1 instanceof Error ? error_1 : getSafeErrorMetadata(error_1));
+                        Logger.error(SHOW_LOGS, TAG, "Joint calibration failed", { joint: jnt }, error_1);
                         return [3 /*break*/, 8];
                     case 8:
                         _i++;
@@ -142,7 +141,7 @@ var CalibrationManager = /** @class */ (function () {
                         return [3 /*break*/, 15];
                     case 14:
                         error_2 = _e.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to enable joint", { joint: jnt }, error_2 instanceof Error ? error_2 : getSafeErrorMetadata(error_2));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to enable joint", { joint: jnt }, error_2);
                         return [3 /*break*/, 15];
                     case 15:
                         _d++;

package/dist-types/src/application/RAFTs/Marty/Marty.js

@@ -57,7 +57,6 @@ import { MartyWifiManager } from "./MartyWifiManager";
 import MartyAddonsManager from "./MartyAddonsManager";
 import CalibrationManager from "./CalibrationManager";
 import Logger from "../../../services/logger/Logger";
-import { getSafeErrorMetadata } from "../../../services/observability/sanitizeTelemetry";
 import { AppSentMessage } from "../../../types/communication-between-apps/wrapper-communication";
 import { RICNotificationsManager } from "./notifications-manager/RICNotificationsManager";
 import PublishedDataAnalyser from "./PublishedDataAnalyser";
@@ -190,7 +189,7 @@ var Marty = /** @class */ (function (_super) {
                         return [2 /*return*/, !!response];
                     case 2:
                         error_1 = _a.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Atomic read operation failed", error_1 instanceof Error ? error_1 : getSafeErrorMetadata(error_1));
+                        Logger.error(SHOW_LOGS, TAG, "Atomic read operation failed", error_1);
                         return [2 /*return*/, false];
                     case 3: return [2 /*return*/];
                 }
@@ -305,7 +304,7 @@ var Marty = /** @class */ (function (_super) {
                         return [3 /*break*/, 3];
                     case 2:
                         error_2 = _a.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to process Marty notification", error_2 instanceof Error ? error_2 : getSafeErrorMetadata(error_2));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to process Marty notification", error_2);
                         return [3 /*break*/, 3];
                     case 3: return [2 /*return*/];
                 }

package/dist-types/src/application/RAFTs/Marty/MartyAddonsManager.js

@@ -36,7 +36,6 @@ var __generator = (this && this.__generator) || function (thisArg, body) {
 };
 import { RICConfiguredAddOns, RICHWElemList_Str } from "@robotical/roboticaljs";
 import Logger from "../../../services/logger/Logger";
-import { getSafeErrorMetadata } from "../../../services/observability/sanitizeTelemetry";
 import { RIC_WHOAMI_TYPE_CODE_ADDON_COLOUR } from "@robotical/ricjs-robotical-addons/dist/RICRoboticalAddOns";
 var SHOW_LOGS = true;
 var TAG = "MartyAddonsManager";
@@ -138,7 +137,7 @@ var MartyAddonsManager = /** @class */ (function () {
                         return [2 /*return*/, addOnList];
                     case 2:
                         error_2 = _a.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to get add-on configurations", error_2 instanceof Error ? error_2 : getSafeErrorMetadata(error_2));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to get add-on configurations", error_2);
                         return [2 /*return*/, new RICConfiguredAddOns()];
                     case 3: return [2 /*return*/];
                 }
@@ -158,7 +157,7 @@ var MartyAddonsManager = /** @class */ (function () {
                         return [2 /*return*/, true];
                     case 2:
                         error_3 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to identify add-on", getSafeErrorMetadata(error_3));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to identify add-on", error_3);
                         return [3 /*break*/, 3];
                     case 3: return [2 /*return*/, false];
                 }
@@ -178,7 +177,7 @@ var MartyAddonsManager = /** @class */ (function () {
                         return [2 /*return*/, true];
                     case 2:
                         error_4 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to set add-on configuration", getSafeErrorMetadata(error_4));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to set add-on configuration", error_4);
                         return [3 /*break*/, 3];
                     case 3: return [2 /*return*/, false];
                 }
@@ -198,7 +197,7 @@ var MartyAddonsManager = /** @class */ (function () {
                         return [2 /*return*/, true];
                     case 2:
                         error_5 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to delete add-on", getSafeErrorMetadata(error_5));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to delete add-on", error_5);
                         return [3 /*break*/, 3];
                     case 3: return [2 /*return*/, false];
                 }

package/dist-types/src/application/RAFTs/Marty/notifications-manager/RICNotificationsManager.js

@@ -43,7 +43,6 @@ import { ReportedWarningsState } from "./ReportedWarningsState";
 import { createTicket, isSerialNumberRegistered } from "../../../../utils/warranty-service/warranty-service-utils";
 import { RICHWElemList_Str } from "@robotical/roboticaljs";
 import Logger from "../../../../services/logger/Logger";
-import { getSafeErrorMetadata } from "../../../../services/observability/sanitizeTelemetry";
 var SHOW_LOGS = true;
 var TAG = "RICNotificationsManager";
 var USER_WARNING_MESSAGE = {
@@ -232,7 +231,7 @@ var RICNotificationsManager = /** @class */ (function () {
                     case 6: return [3 /*break*/, 10];
                     case 7:
                         err_1 = _a.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to get hardware status; retrying", err_1 instanceof Error ? err_1 : getSafeErrorMetadata(err_1));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to get hardware status; retrying", err_1);
                         if (!(i < maxRetries - 1)) return [3 /*break*/, 9];
                         return [4 /*yield*/, new Promise(function (resolve) { return setTimeout(resolve, delayMs); })];
                     case 8:
@@ -289,7 +288,7 @@ var RICNotificationsManager = /** @class */ (function () {
                         return [3 /*break*/, 4];
                     case 3:
                         err_2 = _a.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to get hardware version status", err_2 instanceof Error ? err_2 : getSafeErrorMetadata(err_2));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to get hardware version status", err_2);
                         return [2 /*return*/, "Couldn't get versions info"];
                     case 4: return [2 /*return*/];
                 }
@@ -393,7 +392,7 @@ var RICNotificationsManager = /** @class */ (function () {
                         return [3 /*break*/, 8];
                     case 7:
                         err_3 = _d.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to process servo fault report", err_3 instanceof Error ? err_3 : getSafeErrorMetadata(err_3));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to process servo fault report", err_3);
                         return [3 /*break*/, 8];
                     case 8:
                         _i++;
@@ -456,7 +455,7 @@ var RICNotificationsManager = /** @class */ (function () {
                         return [2 /*return*/];
                 }
             });
-        }); }).catch(function (err) { return Logger.error(SHOW_LOGS, TAG, "Failed to process element communications report", err instanceof Error ? err : getSafeErrorMetadata(err)); });
+        }); }).catch(function (err) { return Logger.error(SHOW_LOGS, TAG, "Failed to process element communications report", err); });
     };
     RICNotificationsManager.prototype.handleBusFailDet = function (report) {
         var _this = this;
@@ -507,7 +506,7 @@ var RICNotificationsManager = /** @class */ (function () {
                         return [2 /*return*/];
                 }
             });
-        }); }).catch(function (err) { return Logger.error(SHOW_LOGS, TAG, "Failed to process bus communications report", err instanceof Error ? err : getSafeErrorMetadata(err)); });
+        }); }).catch(function (err) { return Logger.error(SHOW_LOGS, TAG, "Failed to process bus communications report", err); });
     };
     return RICNotificationsManager;
 }());

package/dist-types/src/application/RAFTs/RaftWifiManager.js

@@ -36,7 +36,6 @@ var __generator = (this && this.__generator) || function (thisArg, body) {
 };
 import { RaftWifiConnState, RaftWifiConnStatus, } from "@robdobsn/raftjs";
 import Logger from "../../services/logger/Logger";
-import { getSafeErrorMetadata } from "../../services/observability/sanitizeTelemetry";
 var SHOW_LOGS = true;
 var TAG = "RaftWifiManager";
 var flagIsSet = function (value) {
@@ -99,7 +98,7 @@ var RaftWifiManager = /** @class */ (function () {
                         return [3 /*break*/, 4];
                     case 3:
                         error_1 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to send Wi-Fi credentials", getSafeErrorMetadata(error_1));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to send Wi-Fi credentials", error_1);
                         return [2 /*return*/, false];
                     case 4:
                         timeoutCount = 0;
@@ -151,7 +150,7 @@ var RaftWifiManager = /** @class */ (function () {
                         return [2 /*return*/, true];
                     case 5:
                         error_2 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to clear Wi-Fi credentials", getSafeErrorMetadata(error_2));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to clear Wi-Fi credentials", error_2);
                         return [2 /*return*/, false];
                     case 6: return [2 /*return*/];
                 }
@@ -222,7 +221,7 @@ var RaftWifiManager = /** @class */ (function () {
                                 this._wifiConnStatus.isPaused === pause];
                     case 4:
                         error_4 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, pause ? "Failed to pause Wi-Fi" : "Failed to resume Wi-Fi", getSafeErrorMetadata(error_4));
+                        Logger.warn(SHOW_LOGS, TAG, pause ? "Failed to pause Wi-Fi" : "Failed to resume Wi-Fi", error_4);
                         return [2 /*return*/, false];
                     case 5: return [2 /*return*/];
                 }
@@ -242,7 +241,7 @@ var RaftWifiManager = /** @class */ (function () {
                         return [2 /*return*/, response];
                     case 2:
                         error_5 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to get Wi-Fi scan results", getSafeErrorMetadata(error_5));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to get Wi-Fi scan results", error_5);
                         return [2 /*return*/, false];
                     case 3: return [2 /*return*/];
                 }
@@ -262,7 +261,7 @@ var RaftWifiManager = /** @class */ (function () {
                         return [2 /*return*/, responseSucceeded(response)];
                     case 2:
                         error_6 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to start Wi-Fi scan", getSafeErrorMetadata(error_6));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to start Wi-Fi scan", error_6);
                         return [2 /*return*/, false];
                     case 3: return [2 /*return*/];
                 }

package/dist-types/src/services/observability/Observability.js

@@ -12,7 +12,6 @@ var __assign = (this && this.__assign) || function () {
 import * as Sentry from "@sentry/react";
 import packageInfo from "../../../package.json";
 import isPhoneApp from "../../utils/phone-app-communication/is-phone-app";
-import { getSafeErrorType, redactTelemetryText, sanitizeTelemetryValue, } from "./sanitizeTelemetry";
 var appVersion = packageInfo.version;
 var getRuntime = function () {
     try {
@@ -32,7 +31,7 @@ var getSafeAppSnapshot = function () {
         .filter(function (type) { return type !== undefined; })
         .map(function (type) { return String(type); })));
     var deviceInfo = applicationManager === null || applicationManager === void 0 ? void 0 : applicationManager.deviceInfo;
-    return sanitizeTelemetryValue({
+    return {
         appVersion: appVersion,
         runtime: getRuntime(),
         route: window.location.pathname,
@@ -53,22 +52,7 @@ var getSafeAppSnapshot = function () {
                 systemVersion: deviceInfo.systemVersion,
             }
             : undefined,
-    });
-};
-var sanitizeEvent = function (event) {
-    var _a, _b;
-    var eventWithPrivateData = event;
-    delete eventWithPrivateData.user;
-    delete eventWithPrivateData.request;
-    (_b = (_a = eventWithPrivateData.exception) === null || _a === void 0 ? void 0 : _a.values) === null || _b === void 0 ? void 0 : _b.forEach(function (exception) {
-        if (exception.type) {
-            exception.type = getSafeErrorType(exception.type);
-        }
-        if (exception.value) {
-            exception.value = "[REDACTED]";
-        }
-    });
-    return sanitizeTelemetryValue(eventWithPrivateData);
+    };
 };
 export function initializeObservability() {
     var _a;
@@ -93,13 +77,10 @@ export function initializeObservability() {
                 xhr: false,
             }),
         ],
-        beforeBreadcrumb: function (breadcrumb) {
-            return sanitizeTelemetryValue(breadcrumb);
-        },
         beforeSend: function (event) {
             event.tags = __assign(__assign({}, event.tags), { app_runtime: getRuntime(), app_version: appVersion });
             event.contexts = __assign(__assign({}, event.contexts), { app_snapshot: getSafeAppSnapshot() });
-            return sanitizeEvent(event);
+            return event;
         },
     });
     Sentry.setTags({
@@ -120,47 +101,44 @@ export function isObservabilityTransportUrl(url) {
         return false;
     }
 }
-var serializeLogArguments = function (messages) {
-    return messages.map(function (message) { return sanitizeTelemetryValue(message); });
-};
 var getLogMessage = function (messages) {
     var message = messages
         .map(function (item) {
+        var _a;
         if (item instanceof Error)
-            return getSafeErrorType(item);
+            return "".concat(item.name, ": ").concat(item.message);
         if (typeof item === "string")
             return item;
         try {
-            return JSON.stringify(sanitizeTelemetryValue(item));
+            return (_a = JSON.stringify(item)) !== null && _a !== void 0 ? _a : String(item);
         }
-        catch (_a) {
+        catch (_b) {
             return String(item);
         }
     })
         .join(" ")
         .trim();
-    return redactTelemetryText(message || "Unknown error");
+    return message || "Unknown error";
 };
 export function captureLoggerError(tag, messages) {
     if (!Sentry.isInitialized())
         return;
     try {
         var error_1 = messages.find(function (message) { return message instanceof Error; });
-        var safeTag_1 = redactTelemetryText(tag).slice(0, 100);
-        var safeMessage_1 = getLogMessage(messages);
+        var message_1 = getLogMessage(messages);
         Sentry.withScope(function (scope) {
             scope.setLevel("error");
-            scope.setTag("logger_tag", safeTag_1);
+            scope.setTag("logger_tag", tag);
             scope.setContext("logger", {
-                message: safeMessage_1,
-                arguments: serializeLogArguments(messages),
+                message: message_1,
+                arguments: messages,
             });
             scope.setContext("app_snapshot", getSafeAppSnapshot());
             if (error_1) {
                 Sentry.captureException(error_1);
             }
             else {
-                Sentry.captureMessage("".concat(safeTag_1, ": ").concat(safeMessage_1), "error");
+                Sentry.captureMessage("".concat(tag, ": ").concat(message_1), "error");
             }
         });
     }
@@ -172,13 +150,12 @@ export function addLoggerWarningBreadcrumb(tag, messages) {
     if (!Sentry.isInitialized())
         return;
     try {
-        var safeTag = redactTelemetryText(tag).slice(0, 100);
         Sentry.addBreadcrumb({
             category: "app.warning",
             level: "warning",
-            message: "".concat(safeTag, ": ").concat(getLogMessage(messages)),
+            message: "".concat(tag, ": ").concat(getLogMessage(messages)),
             data: {
-                loggerTag: safeTag,
+                loggerTag: tag,
                 route: window.location.pathname,
                 runtime: getRuntime(),
             },

package/dist-types/src/utils/feedback/submitFeedback.js

@@ -46,7 +46,6 @@ var __generator = (this && this.__generator) || function (thisArg, body) {
     }
 };
 import Logger from "../../services/logger/Logger";
-import { getSafeErrorMetadata } from "../../services/observability/sanitizeTelemetry";
 import { FEEDBACK_REQUIRES_ROBOT_CONNECTION_MESSAGE, getActiveFeedbackRobotConnection, isRobotConnectionRequiredForFeedbackSource, } from "./feedbackConnectionEligibility";
 import randomHashGenerator from "../helpers/randomHashGenerator";
 import isPhoneApp from "../phone-app-communication/is-phone-app";
@@ -298,7 +297,7 @@ export function submitFeedback(input) {
                 case 4:
                     error_1 = _e.sent();
                     slackDelivered = false;
-                    Logger.error(SHOW_LOGS, TAG, "Failed to submit feedback to Slack", getSafeErrorMetadata(error_1));
+                    Logger.error(SHOW_LOGS, TAG, "Failed to submit feedback to Slack", error_1);
                     return [3 /*break*/, 5];
                 case 5: return [2 /*return*/, {
                         record: record,

package/dist-types/src/utils/warranty-service/warranty-service-utils.js

@@ -35,7 +35,6 @@ var __generator = (this && this.__generator) || function (thisArg, body) {
     }
 };
 import Logger from "../../services/logger/Logger";
-import { getSafeErrorMetadata } from "../../services/observability/sanitizeTelemetry";
 import { getOnboardingCookie, isOnboardingCookieExpired } from "./cookies";
 import WarranyFacade from "@robotical/appv2-warranty-service-lib";
 var SHOW_LOGS = true;
@@ -63,7 +62,7 @@ export function shouldShowOnboardingModal(serialNo) {
                 case 3: return [3 /*break*/, 5];
                 case 4:
                     error_1 = _a.sent();
-                    Logger.error(SHOW_LOGS, TAG, "Error checking if onboarding should be shown", getSafeErrorMetadata(error_1));
+                    Logger.error(SHOW_LOGS, TAG, "Error checking if onboarding should be shown", error_1);
                     // Don't show the modal if there was an error (probably no internet connection)
                     return [2 /*return*/, false];
                 case 5: return [2 /*return*/, shouldShowModal];
@@ -109,7 +108,7 @@ export function registerUser(email, establishment, serialNumbers) {
                 return [2 /*return*/, WarranyFacade.registerUser(email, establishment, serialNumbers)];
             }
             catch (error) {
-                Logger.error(SHOW_LOGS, TAG, "Error registering user", getSafeErrorMetadata(error));
+                Logger.error(SHOW_LOGS, TAG, "Error registering user", error);
                 return [2 /*return*/, false];
             }
             return [2 /*return*/];
@@ -130,7 +129,7 @@ export function isUserRegistered(email) {
                 case 2: return [2 /*return*/, !!(_a.sent())];
                 case 3:
                     error_2 = _a.sent();
-                    Logger.error(SHOW_LOGS, TAG, "Error checking if user is registered", getSafeErrorMetadata(error_2));
+                    Logger.error(SHOW_LOGS, TAG, "Error checking if user is registered", error_2);
                     return [2 /*return*/, false];
                 case 4: return [2 /*return*/];
             }
@@ -153,7 +152,7 @@ export function getUserGivenEmail(email) {
                     return [2 /*return*/, user];
                 case 3:
                     error_3 = _a.sent();
-                    Logger.error(SHOW_LOGS, TAG, "Error getting user", getSafeErrorMetadata(error_3));
+                    Logger.error(SHOW_LOGS, TAG, "Error getting user", error_3);
                     return [2 /*return*/, null];
                 case 4: return [2 /*return*/];
             }
@@ -167,7 +166,7 @@ export var isSerialNumberRegistered = function (serialNo) { return __awaiter(voi
             return [2 /*return*/, WarranyFacade.isSerialNumberRegistered(serialNo)];
         }
         catch (error) {
-            Logger.error(SHOW_LOGS, TAG, "Error checking if serial number is registered", getSafeErrorMetadata(error));
+            Logger.error(SHOW_LOGS, TAG, "Error checking if serial number is registered", error);
             return [2 /*return*/, false];
         }
         return [2 /*return*/];
@@ -191,7 +190,7 @@ export var createTicket = function (serialNo, title, description, robotName, rob
             case 4: return [2 /*return*/, false];
             case 5:
                 error_4 = _a.sent();
-                Logger.error(SHOW_LOGS, TAG, "Error creating ticket", getSafeErrorMetadata(error_4));
+                Logger.error(SHOW_LOGS, TAG, "Error creating ticket", error_4);
                 return [2 /*return*/, false];
             case 6: return [2 /*return*/];
         }
@@ -212,7 +211,7 @@ export var getEmailGivenSerialNumber = function (serialNo) { return __awaiter(vo
                 return [2 /*return*/, serialNumber === null || serialNumber === void 0 ? void 0 : serialNumber.email];
             case 3:
                 error_5 = _a.sent();
-                Logger.error(SHOW_LOGS, TAG, "Error getting email for serial number", getSafeErrorMetadata(error_5));
+                Logger.error(SHOW_LOGS, TAG, "Error getting email for serial number", error_5);
                 return [2 /*return*/, undefined];
             case 4: return [2 /*return*/];
         }
@@ -225,7 +224,7 @@ export var addRobotNameToSerialNumber = function (serialNo, name) { return __awa
             return [2 /*return*/, WarranyFacade.addRobotNameToSerialNumber(serialNo, name)];
         }
         catch (error) {
-            Logger.error(SHOW_LOGS, TAG, "Error adding robot name to serial number", getSafeErrorMetadata(error));
+            Logger.error(SHOW_LOGS, TAG, "Error adding robot name to serial number", error);
             return [2 /*return*/, false];
         }
         return [2 /*return*/];

package/dist-types/src/wrapper-app/WrapperAppManager.js

@@ -46,7 +46,6 @@ var __spreadArray = (this && this.__spreadArray) || function (to, from, pack) {
 import { WrapperSentMessage } from "../types/communication-between-apps/wrapper-communication";
 import { RaftConnectionMethod, } from "../types/raft";
 import Logger from "../services/logger/Logger";
-import { getSafeErrorMetadata } from "../services/observability/sanitizeTelemetry";
 import { RaftConnEvent } from "@robdobsn/raftjs";
 import { ConnManager } from "@robotical/roboticaljs";
 import ConnectorFactory from "./connectors/ConnectorFactory";
@@ -166,13 +165,13 @@ var WrapperAppManager = /** @class */ (function () {
                                             return [3 /*break*/, 4];
                                         case 3:
                                             error_2 = _a.sent();
-                                            Logger.warn(SHOW_LOGS, TAG, "Failed late WebSocket cleanup", getSafeErrorMetadata(error_2));
+                                            Logger.warn(SHOW_LOGS, TAG, "Failed late WebSocket cleanup", error_2);
                                             return [3 /*break*/, 4];
                                         case 4: return [2 /*return*/];
                                     }
                                 });
                             }); }).catch(function (error) {
-                                Logger.warn(SHOW_LOGS, TAG, "WebSocket connection settled after timeout", getSafeErrorMetadata(error));
+                                Logger.warn(SHOW_LOGS, TAG, "WebSocket connection settled after timeout", error);
                             });
                             Logger.warn(SHOW_LOGS, TAG, "WebSocket connection timed out after ".concat(WEBSOCKET_CONNECT_TIMEOUT_MS, "ms"));
                             return [2 /*return*/, { success: false, reason: "failed" }];
@@ -230,7 +229,7 @@ var WrapperAppManager = /** @class */ (function () {
                     case 12:
                         if (!setupResult) {
                             void connectorConnectionPromise.catch(function (error) {
-                                Logger.warn(SHOW_LOGS, TAG, "WebSocket setup settled after timeout", getSafeErrorMetadata(error));
+                                Logger.warn(SHOW_LOGS, TAG, "WebSocket setup settled after timeout", error);
                             });
                             Logger.warn(SHOW_LOGS, TAG, "WebSocket setup timed out after ".concat(WEBSOCKET_CONNECT_TIMEOUT_MS, "ms"));
                             return [2 /*return*/, { success: false, reason: "failed" }];
@@ -537,7 +536,7 @@ var WrapperAppManager = /** @class */ (function () {
             connManager.getConnector().disconnectForPageUnload();
         }
         catch (error) {
-            Logger.warn(SHOW_LOGS, TAG, "Failed pending page-unload cleanup", getSafeErrorMetadata(error));
+            Logger.warn(SHOW_LOGS, TAG, "Failed pending page-unload cleanup", error);
         }
     };
     WrapperAppManager.prototype.isConnected = function (raftId) {
@@ -649,7 +648,7 @@ var WrapperAppManager = /** @class */ (function () {
                     case 6: return [3 /*break*/, 14];
                     case 7:
                         error_3 = _a.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to discover warranty device", getSafeErrorMetadata(error_3));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to discover warranty device", error_3);
                         return [2 /*return*/, existingDevices];
                     case 8:
                         if (!ownsTemporaryConnection) return [3 /*break*/, 13];
@@ -662,7 +661,7 @@ var WrapperAppManager = /** @class */ (function () {
                         return [3 /*break*/, 13];
                     case 11:
                         error_4 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed warranty connection cleanup", getSafeErrorMetadata(error_4));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed warranty connection cleanup", error_4);
                         return [3 /*break*/, 13];
                     case 12:
                         this._pendingConnManagers.delete(warrantyConnManager);
@@ -695,7 +694,7 @@ var WrapperAppManager = /** @class */ (function () {
                         return [2 /*return*/, true];
                     case 3:
                         error_5 = _a.sent();
-                        Logger.error(SHOW_LOGS, TAG, "Failed to send atomic read operation to Marty", getSafeErrorMetadata(error_5));
+                        Logger.error(SHOW_LOGS, TAG, "Failed to send atomic read operation to Marty", error_5);
                         return [3 /*break*/, 4];
                     case 4: return [3 /*break*/, 6];
                     case 5:

package/dist-types/src/wrapper-app/connectors/ConnectorFactory.js

@@ -39,7 +39,6 @@ import randomHashGenerator from "../../utils/helpers/randomHashGenerator";
 import CogConnector from "./CogConnector/CogConnector";
 import MartyConnector from "./MartyConnector/MartyConnector";
 import Logger from "../../services/logger/Logger";
-import { getSafeErrorMetadata } from "../../services/observability/sanitizeTelemetry";
 var SHOW_LOGS = true;
 var TAG = "ConnectorFactory";
 var ConnectorFactory = /** @class */ (function () {
@@ -103,7 +102,7 @@ var ConnectorFactory = /** @class */ (function () {
                         return [3 /*break*/, 7];
                     case 6:
                         error_1 = _a.sent();
-                        Logger.warn(SHOW_LOGS, TAG, "Failed to clean up uncommitted RAFT connection", getSafeErrorMetadata(error_1));
+                        Logger.warn(SHOW_LOGS, TAG, "Failed to clean up uncommitted RAFT connection", error_1);
                         return [3 /*break*/, 7];
                     case 7: return [7 /*endfinally*/];
                     case 8: return [2 /*return*/];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@robotical/webapp-types",
-  "version": "3.21.9",
+  "version": "3.21.10",
   "description": "Type definitions for the Application Manager",
   "main": "dist/application-manager.d.ts",
   "types": "dist/application-manager.d.ts",

package/dist-types/src/services/observability/sanitizeTelemetry.d.ts

@@ -1,7 +0,0 @@
-export declare const isSensitiveTelemetryKey: (key: string) => boolean;
-export declare const getSafeErrorType: (value: unknown) => string;
-export declare const getSafeErrorMetadata: (error: unknown) => {
-    errorType: string;
-};
-export declare function redactTelemetryText(value: string): string;
-export declare function sanitizeTelemetryValue(value: unknown, depth?: number, seen?: WeakSet<object>): unknown;

package/dist-types/src/services/observability/sanitizeTelemetry.js

@@ -1,136 +0,0 @@
-var REDACTED = "[REDACTED]";
-var MAX_DEPTH = 12;
-var MAX_ARRAY_ITEMS = 100;
-var MAX_OBJECT_KEYS = 100;
-var MAX_STRING_LENGTH = 4000;
-var SAFE_ERROR_TYPES = new Set([
-    "AbortError",
-    "AggregateError",
-    "DOMException",
-    "Error",
-    "EvalError",
-    "NetworkError",
-    "NotAllowedError",
-    "RangeError",
-    "ReferenceError",
-    "SecurityError",
-    "SyntaxError",
-    "TimeoutError",
-    "TypeError",
-    "URIError",
-]);
-var SENSITIVE_KEYS = new Set([
-    "authorization",
-    "authtoken",
-    "base64",
-    "bleaddress",
-    "bssid",
-    "cookie",
-    "cookies",
-    "credential",
-    "credentials",
-    "connectionlocator",
-    "dataurl",
-    "deviceid",
-    "devicename",
-    "email",
-    "filecontents",
-    "friendlyname",
-    "ipaddress",
-    "locator",
-    "macaddress",
-    "password",
-    "passphrase",
-    "raftid",
-    "robotid",
-    "secret",
-    "serial",
-    "serialno",
-    "serialnumber",
-    "ssid",
-    "streamcontents",
-    "token",
-    "username",
-    "wifipassword",
-    "wifimac",
-    "wifiusername",
-]);
-var normalizeKey = function (key) {
-    return key.replace(/[^a-z0-9]/gi, "").toLowerCase();
-};
-export var isSensitiveTelemetryKey = function (key) {
-    return SENSITIVE_KEYS.has(normalizeKey(key));
-};
-export var getSafeErrorType = function (value) {
-    var errorType = value instanceof Error ? value.name : value;
-    return typeof errorType === "string" && SAFE_ERROR_TYPES.has(errorType)
-        ? errorType
-        : "Error";
-};
-export var getSafeErrorMetadata = function (error) { return ({
-    errorType: error instanceof Error ? getSafeErrorType(error) : typeof error,
-}); };
-export function redactTelemetryText(value) {
-    var redacted = value
-        .replace(/\b[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,}\b/gi, REDACTED)
-        .replace(/\b(password|passphrase|token|authorization|cookie|secret|ssid|serial(?:\s*(?:number|no))?|device\s*id|device\s*name|robot\s*id|robot\s*name|raft(?:\s+with)?\s*id|mac\s*address|ble\s*address)\s*(?::|=|\s)\s*[^\s,;]+/gi, function (_match, label) { return "".concat(label, ": ").concat(REDACTED); })
-        .replace(/\bWi-?Fi\s+network\s+[^\s,;]+/gi, "Wi-Fi network ".concat(REDACTED))
-        .replace(/https:\/\/hooks\.slack\.com\/services\/[^\s"']+/gi, REDACTED)
-        .replace(/([?&][^=&#\s]+)=([^&#\s]*)/g, "$1=".concat(REDACTED))
-        .replace(/\b[0-9a-f]{8}-[0-9a-f]{4}-[1-5][0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}\b/gi, REDACTED)
-        .replace(/\b(?:[0-9a-f]{2}:){5}[0-9a-f]{2}\b/gi, REDACTED);
-    return redacted.length > MAX_STRING_LENGTH
-        ? "".concat(redacted.slice(0, MAX_STRING_LENGTH), "\u2026")
-        : redacted;
-}
-export function sanitizeTelemetryValue(value, depth, seen) {
-    if (depth === void 0) { depth = 0; }
-    if (seen === void 0) { seen = new WeakSet(); }
-    if (typeof value === "string") {
-        return redactTelemetryText(value);
-    }
-    if (value === null ||
-        value === undefined ||
-        typeof value === "boolean" ||
-        typeof value === "number") {
-        return value;
-    }
-    if (typeof value === "bigint" || typeof value === "symbol") {
-        return String(value);
-    }
-    if (typeof value === "function") {
-        return "[Function]";
-    }
-    if (depth >= MAX_DEPTH) {
-        return "[Truncated]";
-    }
-    if (value instanceof Error) {
-        return {
-            name: getSafeErrorType(value),
-            message: REDACTED,
-        };
-    }
-    if (value instanceof Date) {
-        return value.toISOString();
-    }
-    if (typeof value !== "object") {
-        return redactTelemetryText(String(value));
-    }
-    if (seen.has(value)) {
-        return "[Circular]";
-    }
-    seen.add(value);
-    if (Array.isArray(value)) {
-        return value
-            .slice(0, MAX_ARRAY_ITEMS)
-            .map(function (item) { return sanitizeTelemetryValue(item, depth + 1, seen); });
-    }
-    var sanitized = {};
-    for (var _i = 0, _a = Object.entries(value).slice(0, MAX_OBJECT_KEYS); _i < _a.length; _i++) {
-        var _b = _a[_i], key = _b[0], item = _b[1];
-        sanitized[key] = isSensitiveTelemetryKey(key)
-            ? REDACTED
-            : sanitizeTelemetryValue(item, depth + 1, seen);
-    }
-    return sanitized;
-}