@oxyhq/core 10.1.2 → 10.1.4

package/dist/cjs/session/accountDialogController.js

@@ -30,6 +30,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.AccountDialogController = void 0;
 exports.createAccountDialogController = createAccountDialogController;
 const loggerUtils_1 = require("../utils/loggerUtils");
+const errorUtils_1 = require("../utils/errorUtils");
 const authWebUrl_1 = require("../utils/authWebUrl");
 const oauthPkce_1 = require("../utils/oauthPkce");
 const accountProjection_1 = require("./accountProjection");
@@ -245,10 +246,20 @@ class AccountDialogController {
             graph = await this.oxyServices.listAccounts();
         }
         catch (error) {
-            // A graph-load failure is non-fatal: device rows still render. Surface the
-            // message but keep going with whatever graph we already had.
-            this.error = errorMessage(error);
-            loggerUtils_1.logger.warn('[AccountDialogController] listAccounts failed', { component: 'AccountDialogController' }, error);
+            // A 401 here is the EXPECTED signed-out edge, not a failure: the bearer was
+            // stale/revoked, so `HttpService` already cleared it and emitted
+            // `onTokensChanged(null)`, which drops the graph via `reconcileAuth`. Log at
+            // debug and leave the dialog error-free — a signed-out device with zero
+            // accounts is a normal state, not a warning. Any other error (network, 5xx,
+            // malformed) IS unexpected: surface it and warn while keeping the prior graph
+            // so device rows still render.
+            if ((0, errorUtils_1.extractErrorStatus)(error) === 401) {
+                loggerUtils_1.logger.debug('[AccountDialogController] listAccounts unauthorized (signed out)', { component: 'AccountDialogController' }, error);
+            }
+            else {
+                this.error = errorMessage(error);
+                loggerUtils_1.logger.warn('[AccountDialogController] listAccounts failed', { component: 'AccountDialogController' }, error);
+            }
         }
         if (seq !== this.refreshSeq)
             return; // superseded by a newer refresh
@@ -288,9 +299,16 @@ class AccountDialogController {
             profiles = await this.oxyServices.getUsersByIds(ids);
         }
         catch (error) {
-            // `getUsersByIds` already swallows per-chunk failures and returns `[]`;
-            // this guards the unexpected total failure. Non-fatal — keep prior map.
-            loggerUtils_1.logger.warn('[AccountDialogController] getUsersByIds failed', { component: 'AccountDialogController' }, error);
+            // A 401 is the EXPECTED signed-out edge (stale/cleared bearer) — log at debug.
+            // `getUsersByIds` already swallows per-chunk failures and returns `[]`, so any
+            // OTHER error here is an unexpected total failure worth a warn. Either way keep
+            // the prior profile map.
+            if ((0, errorUtils_1.extractErrorStatus)(error) === 401) {
+                loggerUtils_1.logger.debug('[AccountDialogController] getUsersByIds unauthorized (signed out)', { component: 'AccountDialogController' }, error);
+            }
+            else {
+                loggerUtils_1.logger.warn('[AccountDialogController] getUsersByIds failed', { component: 'AccountDialogController' }, error);
+            }
             return;
         }
         if (seq !== this.refreshSeq)

package/dist/cjs/utils/textNormalization.js

@@ -46,28 +46,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.normalizeInlineText = normalizeInlineText;
 exports.normalizeMultilineText = normalizeMultilineText;
-/**
- * Characters that make a value ineligible for the zero-work fast path, and the
- * whitespace shapes that a normalized INLINE value can never contain.
- *
- * A value that matches nothing here is, by construction, already normalized:
- * it holds only printable ASCII (which is NFC-stable, so `normalize('NFC')`
- * would be a no-op), its only whitespace is the plain space, it has no leading
- * or trailing space, and no run of two spaces. Returning it untouched skips
- * three string allocations — worth it because the common case in the feed
- * hydration hot path is text that is already clean.
- *
- * Non-global (safe for repeated `.test()`; a global regex is stateful).
- */
-const INLINE_NEEDS_NORMALIZATION = /[^\x20-\x7E]|^ | $| {2}/;
-/**
- * Same idea as {@link INLINE_NEEDS_NORMALIZATION}, for MULTILINE values: `\n`
- * joins the printable-ASCII fast-path alphabet, and the additional shapes a
- * normalized body can never contain are a space adjacent to a line break — on
- * either side, since every line is trimmed — and a run of three line breaks
- * (more than one blank line).
- */
-const MULTILINE_NEEDS_NORMALIZATION = /[^\x20-\x7E\n]|^[ \n]|[ \n]$| {2}| \n|\n |\n{3}/;
 /** Any run of whitespace, including tabs, line breaks and Unicode spaces. */
 const ANY_WHITESPACE_RUN = /\s+/g;
 /**
@@ -129,9 +107,6 @@ const EXCESS_BLANK_LINES = /\n{3,}/g;
  * Idempotent: `f(f(x)) === f(x)`.
  */
 function normalizeInlineText(value) {
-    if (!INLINE_NEEDS_NORMALIZATION.test(value)) {
-        return value;
-    }
     return value.normalize('NFC').replace(ANY_WHITESPACE_RUN, ' ').trim();
 }
 /**
@@ -167,9 +142,6 @@ function normalizeInlineText(value) {
  * Idempotent: `f(f(x)) === f(x)`.
  */
 function normalizeMultilineText(value) {
-    if (!MULTILINE_NEEDS_NORMALIZATION.test(value)) {
-        return value;
-    }
     return value
         .normalize('NFC')
         .replace(LINE_BREAK_FORMS, '\n')