fix: improve diagnostic logging for probe and all heal paths

- Probe start: logInfo when tab becomes visible and probe begins
- Probe healthy: logInfo confirming connection is healthy
- Probe stale: logAlert with error details when stale connection detected
- Heal attempts/success/exhaustion/non-recoverable: same as Expensify#780
- Updated test assertions to match new log messages and levels

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: leshniak

lib/storage/providers/IDBKeyValProvider/createStore.ts

@@ -140,13 +140,19 @@ function createStore(dbName: string, storeName: string): UseStore {
                 return;
             }
 
+            Logger.logInfo('IDB visibilitychange probe: tab became visible, checking connection health', {dbName, storeName});
+
             const probePromise = dbp;
 
             const dropCacheIfStale = (error: unknown) => {
                 if (dbp !== probePromise || !isStaleConnectionError(error)) {
                     return;
                 }
-                Logger.logInfo('IDB visibilitychange probe: connection lost, dropping cached connection', {dbName, storeName});
+                Logger.logAlert('IDB visibilitychange probe: stale connection detected, dropping cached connection', {
+                    dbName,
+                    storeName,
+                    errorMessage: error instanceof Error ? error.message : String(error),
+                });
                 dbp = undefined;
             };
 
@@ -161,6 +167,7 @@ function createStore(dbName: string, storeName: string): UseStore {
                     req.onerror = () => {
                         dropCacheIfStale(req.error);
                     };
+                    Logger.logInfo('IDB visibilitychange probe: connection is healthy', {dbName, storeName});
                 } catch (error) {
                     dropCacheIfStale(error);
                 }

tests/unit/storage/providers/createStoreTest.ts

@@ -594,7 +594,7 @@ describe('createStore', () => {
 
             const result = await store('readonly', (s) => IDB.promisifyRequest(s.get('key1')));
             expect(result).toBe('value');
-            expect(logInfoSpy).toHaveBeenCalledWith('IDB visibilitychange probe: connection lost, dropping cached connection', expect.objectContaining({dbName: expect.any(String)}));
+            expect(logAlertSpy).toHaveBeenCalledWith(expect.stringContaining('IDB visibilitychange probe: stale connection detected'), expect.objectContaining({dbName: expect.any(String)}));
         });
 
         it('should not probe when no connection exists yet', async () => {
@@ -629,7 +629,9 @@ describe('createStore', () => {
 
             const result = await store('readonly', (s) => IDB.promisifyRequest(s.get('key1')));
             expect(result).toBe('value');
-            expect(logInfoSpy).not.toHaveBeenCalledWith(expect.stringContaining('visibilitychange probe'), expect.anything());
+            // Probe ran but found healthy connection — no stale connection alert
+            expect(logAlertSpy).not.toHaveBeenCalledWith(expect.stringContaining('stale connection detected'), expect.anything());
+            expect(logInfoSpy).toHaveBeenCalledWith(expect.stringContaining('connection is healthy'), expect.anything());
         });
 
         it('should drop dbp when probe throws InvalidStateError', async () => {
@@ -662,7 +664,7 @@ describe('createStore', () => {
 
             const result = await store('readonly', (s) => IDB.promisifyRequest(s.get('key1')));
             expect(result).toBe('value');
-            expect(logInfoSpy).toHaveBeenCalledWith('IDB visibilitychange probe: connection lost, dropping cached connection', expect.objectContaining({dbName: expect.any(String)}));
+            expect(logAlertSpy).toHaveBeenCalledWith(expect.stringContaining('IDB visibilitychange probe: stale connection detected'), expect.objectContaining({dbName: expect.any(String)}));
         });
     });
 });