@escapenavigator/utils 2.0.35 → 2.0.36

package/dist/classify-api-error.d.ts

@@ -23,7 +23,9 @@
  *
  *   skip     → не репортим
  *              304 (etag-success), отменённые axios-запросы (ERR_CANCELED),
- *              401/403 на app/auth (ожидаемый session/role flow).
+ *              401/403 на app/auth (ожидаемый session/role flow), а также
+ *              обрывы связи на стороне клиента: `Network Error` и
+ *              «timeout of 0ms exceeded» (запрос умер до ответа сервера).
  *
  * Группировка (`fingerprint`) делается так, чтобы все события с одинаковым
  * `apiKey + errorCode` сливались в один issue. Иначе Sentry плодит отдельный

package/dist/classify-api-error.js

@@ -24,7 +24,9 @@
  *
  *   skip     → не репортим
  *              304 (etag-success), отменённые axios-запросы (ERR_CANCELED),
- *              401/403 на app/auth (ожидаемый session/role flow).
+ *              401/403 на app/auth (ожидаемый session/role flow), а также
+ *              обрывы связи на стороне клиента: `Network Error` и
+ *              «timeout of 0ms exceeded» (запрос умер до ответа сервера).
  *
  * Группировка (`fingerprint`) делается так, чтобы все события с одинаковым
  * `apiKey + errorCode` сливались в один issue. Иначе Sentry плодит отдельный
@@ -84,14 +86,14 @@ function classifyApiError(ctx) {
         return { severity: 'skip', level: 'info', fingerprint: null, shouldReport: false };
     }
     // «timeout of 0ms exceeded» — Safari/iOS обрывает XHR при смене сети или
-    // уходе вкладки в фон. Не баг, но полезно для картины мобильной связности.
+    // уходе вкладки в фон (timeout у запроса не выставлен, обрыв не наш).
+    //
+    // Раньше репортили как business/info «для картины мобильной связности»:
+    // за месяц это дало десяток issue на 50+ человек
+    // (ESCAPE-NAVIGATOR-APP-3J/3K/3M/3H/2P/3G/33), в которых ни одного бага.
+    // Картину связности всё равно лучше видно по метрикам, а не по issue.
     if (code === 'ECONNABORTED' && typeof message === 'string' && /timeout of 0ms/i.test(message)) {
-        return {
-            severity: 'business',
-            level: 'info',
-            fingerprint: ['api', 'mobile-abort', apiKey],
-            shouldReport: true,
-        };
+        return { severity: 'skip', level: 'info', fingerprint: null, shouldReport: false };
     }
     if (typeof status === 'number' && SKIP_HTTP_STATUSES.has(status)) {
         return { severity: 'skip', level: 'info', fingerprint: null, shouldReport: false };
@@ -155,14 +157,15 @@ function classifyApiError(ctx) {
     // code === 'ERR_NETWORK' (axios>=0.27). На проде это почти всегда
     // клиентская связность (мобильный обрыв, смена соты, троттлинг/RKN).
     // На виджете GET'ы уже ретраятся — сюда долетает только финальный фейл.
+    //
+    // Не репортим: за месяц это ~1000 событий в app/widget/auth, разбитых на
+    // десятки issue по apiKey, и ни одно не было багом. Настоящая
+    // недоступность бэка видна иначе и этим фильтром не задета: 5xx и
+    // сработавший таймаут (25s у виджета) выше классифицируются как critical,
+    // а падение домена/сертификата ловится мониторингом со стороны бэкенда.
     const isNetworkError = !status && (code === 'ERR_NETWORK' || (!code && isNetworkErrorMessage(message)));
     if (isNetworkError) {
-        return {
-            severity: 'business',
-            level: 'warning',
-            fingerprint: ['api', 'network', apiKey],
-            shouldReport: true,
-        };
+        return { severity: 'skip', level: 'info', fingerprint: null, shouldReport: false };
     }
     // Сюда падают: какие-то странные кейсы без status/кода/сетевого message.
     // Считаем critical, но без агрессивного группирования — пусть Sentry

package/dist/classify-api-error.spec.js

@@ -43,6 +43,28 @@ describe('classifyApiError', () => {
             const r = (0, classify_api_error_1.classifyApiError)({ ...base, status: 404, app: 'auth' });
             expect(r.shouldReport).toBe(true);
         });
+        it('чистый network error → skip (обрыв связи у клиента)', () => {
+            expect((0, classify_api_error_1.classifyApiError)({ ...base, code: 'ERR_NETWORK' }).shouldReport).toBe(false);
+            expect((0, classify_api_error_1.classifyApiError)({ ...base, message: 'Network Error' }).shouldReport).toBe(false);
+        });
+        it('мобильный обрыв (timeout of 0ms) → skip', () => {
+            const r = (0, classify_api_error_1.classifyApiError)({
+                ...base,
+                code: 'ECONNABORTED',
+                message: 'timeout of 0ms exceeded',
+            });
+            expect(r.severity).toBe('skip');
+            expect(r.shouldReport).toBe(false);
+        });
+        it('сработавший таймаут виджета (25s) → НЕ skip, это медленный бэк', () => {
+            const r = (0, classify_api_error_1.classifyApiError)({
+                ...base,
+                code: 'ECONNABORTED',
+                message: 'timeout of 25000ms exceeded',
+            });
+            expect(r.severity).toBe('critical');
+            expect(r.shouldReport).toBe(true);
+        });
         it('5xx на login → НЕ skip (это уже настоящий сбой входа)', () => {
             const r = (0, classify_api_error_1.classifyApiError)({
                 apiKey: 'login',
@@ -99,20 +121,6 @@ describe('classifyApiError', () => {
             expect(r.severity).toBe('business');
             expect(r.fingerprint).toContain('bookingWindowClosed');
         });
-        it('мобильный обрыв (timeout of 0ms) → business/info', () => {
-            const r = (0, classify_api_error_1.classifyApiError)({
-                ...base,
-                code: 'ECONNABORTED',
-                message: 'timeout of 0ms exceeded',
-            });
-            expect(r.severity).toBe('business');
-            expect(r.level).toBe('info');
-            expect(r.fingerprint).toContain('mobile-abort');
-        });
-        it('чистый network error → business', () => {
-            expect((0, classify_api_error_1.classifyApiError)({ ...base, code: 'ERR_NETWORK' }).severity).toBe('business');
-            expect((0, classify_api_error_1.classifyApiError)({ ...base, message: 'Network Error' }).severity).toBe('business');
-        });
     });
     describe('debug (generic 4xx — шум, но репортим info-level)', () => {
         it.each([

package/dist/get-order-cancelation-date/index.d.ts

@@ -3,7 +3,8 @@ type Props = {
     utcDate: string;
     minHoursForFreeCanceling: number;
     cancelationRule: QuestroomCancelationTypeEnum;
-    timeZone: string;
+    /** Часовой пояс локации; при отсутствии считаем в UTC (см. ниже). */
+    timeZone?: string;
 };
 export declare function getOrderCancelationDate({ utcDate, minHoursForFreeCanceling, cancelationRule, timeZone, }: Props): {
     orderCancelationDate: Date;

package/dist/get-order-cancelation-date/index.js

@@ -5,7 +5,12 @@ const questroom_cancelation_type_enum_1 = require("@escapenavigator/types/dist/q
 const date_1 = require("../date");
 function getOrderCancelationDate({ utcDate, minHoursForFreeCanceling, cancelationRule, timeZone, }) {
     const minHours = cancelationRule === questroom_cancelation_type_enum_1.QuestroomCancelationTypeEnum.NO ? 0 : minHoursForFreeCanceling;
-    const date = (0, date_1.addHours)((0, date_1.utcToZonedTime)(utcDate, timeZone), minHours * -1);
+    // Локация заказа может быть удалена (`questroom.location === null`), и
+    // тогда часового пояса просто нет. `utcToZonedTime` на undefined даёт
+    // Invalid Date, а `formatDate` на нём кидает — падала вся клиентская
+    // страница заказа (ESCAPE-NAVIGATOR-ORDERS-2D). Дата в UTC хуже
+    // локальной, но лучше пустого экрана.
+    const date = (0, date_1.addHours)((0, date_1.utcToZonedTime)(utcDate, timeZone || 'UTC'), minHours * -1);
     return {
         orderCancelationDate: date,
         orderCancelationHumantDate: (0, date_1.formatDate)(date, 'dd.MM.yyyy, HH:mm'),

package/dist/tz-date.d.ts

@@ -6,7 +6,7 @@ type ToUTC = {
     offset?: number;
 };
 type ToZoned = {
-    timeZone: Timezone;
+    timeZone?: Timezone;
     date: string;
     offset?: number;
     format?: string;

package/dist/tz-date.js

@@ -14,9 +14,14 @@ exports.convertDateToZonedDate = convertDateToZonedDate;
 const convertUTCDateToZonedDate = ({ date, timeZone, offset = 0, format = 'dd.MM.yyyy, HH:mm', showTZ = false, }) => {
     if (!date)
         return '';
-    const d = (0, date_1.utcToZonedTime)(date, timeZone);
+    // Часового пояса может не быть (удалённая локация заказа — см.
+    // getOrderCancelationDate): на undefined `utcToZonedTime` даёт Invalid
+    // Date, а `formatDate` на нём кидает и роняет весь экран. Показываем
+    // время в UTC и честно подписываем его в `showTZ`.
+    const tz = timeZone || 'UTC';
+    const d = (0, date_1.utcToZonedTime)(date, tz);
     const dt = (0, date_1.addHours)(d, offset);
     const res = (0, date_1.formatDate)(dt, format);
-    return showTZ ? `${res} (${timeZone})` : res;
+    return showTZ ? `${res} (${tz})` : res;
 };
 exports.convertUTCDateToZonedDate = convertUTCDateToZonedDate;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@escapenavigator/utils",
-  "version": "2.0.35",
+  "version": "2.0.36",
   "publishConfig": {
     "access": "public"
   },
@@ -14,7 +14,7 @@
     "test": "jest"
   },
   "dependencies": {
-    "@escapenavigator/types": "^2.0.33",
+    "@escapenavigator/types": "^2.0.34",
     "axios": "^0.21.4",
     "class-transformer": "^0.5.1",
     "class-validator": "^0.13.2",
@@ -29,5 +29,5 @@
     "ts-jest": "^29.1.1",
     "typescript": "^5.6"
   },
-  "gitHead": "2c47385d8e152226b70a9b11213eb511b671ad0f"
+  "gitHead": "3112d9d954979e406e77a787d459e16c49da3d5a"
 }