From 9886d4f1f51a6f0671d68ac3ba115e02b27ed474 Mon Sep 17 00:00:00 2001
From: Adam Soltys <adam@coinos.io>
Date: Fri, 6 Nov 2026 21:59:36 -0700
Subject: The wallet paints its balance without deep-copying the whole ark
 state five to seven times on every launch. "Its so slow to load … I'm using
 the apk. Compared to Minibits or something its way slower" (a user,
 2026-11-05): profiled on Adam's Pixel 9 over DevTools, a launch kept the main
 thread pegged for ~1.7 s, and 1.4 s of it was loadArkState — every
 arkStateNow() during the first render (balance, history, notices, the
 Spending figures) structured-cloned the stored state, ~1 MB on a wallet with
 deep coins, ~220 ms per copy, before the manager connected and its live state
 took over. ArkStore.peek() hands back the stored object itself (before the
 store opens, or on localStorage, the parse is kept until the key is next
 written); wallet.peekArkState() is loadArkState through it, with the
 misfiled-state repair still going through loadArkState and each object's
 owner decoded once; arkStateNow and arkWanted read through it — read-only, as
 they already were with the live ark.state. Same boot on the phone, live tab
 with the local build: main-thread busy over the first 4 s 3.7 s → 1.8 s, no
 pegged stretch. tools/ark-store-test.js: peek returns the same object, sees
 writes at once, and drops a kept localStorage parse on write
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 src/features/ark.js | 23 +++++++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

(limited to 'src/features')

diff --git a/src/features/ark.js b/src/features/ark.js
index 5d3b293..f5ba9d8 100644
--- a/src/features/ark.js
+++ b/src/features/ark.js
@@ -77,11 +77,15 @@ if (typeof document !== 'undefined') {
   document.addEventListener('visibilitychange', () => { if (document.visibilityState === 'hidden') flushAll(); });
 }
 
+// A read-only look at the stored state (stub wallets in tests have only loadArkState).
+const peekArk = (wallet, url) => (wallet.peekArkState ? wallet.peekArkState(url) : wallet.loadArkState(url));
+
 // Wallet storage/key helpers for this feature, installed onto the core
 // wallet instance so a build without the feature ships none of it.
 export function installArkWallet(wallet) {
   if (wallet.loadArkState) return; // already installed
   const store = arkStore(); // IndexedDB-backed, memory-fronted; opens now
+  const peekOwnerOk = new WeakSet(); // stored states whose owner check passed
   Object.assign(wallet, {
     _arkStore: store,
     // ---- Ark support -------------------------------------------------------
@@ -133,6 +137,21 @@ export function installArkWallet(wallet) {
         return st;
       } catch { return null; }
     },
+    // loadArkState for readers that only look (see ArkStore.peek): the stored
+    // object, no copy — never mutate it. Misfiled state goes through
+    // loadArkState's repair; each object's owner is decoded once.
+    peekArkState(url) {
+      try {
+        const st = store.peek(this._arkKey(url));
+        if (!st) return null;
+        if (!peekOwnerOk.has(st)) {
+          const owner = this._arkVtxoOwner(st);
+          if (owner && st.serverPubkey && owner !== st.serverPubkey) return this.loadArkState(url);
+          peekOwnerOk.add(st);
+        }
+        return st;
+      } catch { return null; }
+    },
     // Offer up pre-namespacing state, but only to the ASP that actually
     // cosigned it. Its owner is read from the state's own stamp, or failing
     // that decoded straight out of a vtxo — the vtxo wire format carries the
@@ -554,7 +573,7 @@ export function arkFeature(ctx) {
 
   function arkWanted() {
     const cfg = getArkConfig();
-    const st = cfg && wallet.loadArkState(cfg.ark);
+    const st = cfg && peekArk(wallet, cfg.ark);
     return !!(st && ((st.vtxos || []).length || (st.movements || []).length
       || (st.actions || []).some((a) => !['done', 'failed'].includes(a.step))));
   }
@@ -797,7 +816,7 @@ export function arkFeature(ctx) {
     // (send/exit) still requires the seed and is gated separately.
     const cfg = getArkConfig();
     if (!cfg) return null;
-    return wallet.loadArkState(cfg.ark);
+    return peekArk(wallet, cfg.ark);
   }
 
   // Which receives actually arrived over Lightning? A payment to the user's
-- 
cgit v1.2.3