Fix: Accurate login status detection in popup

Problem:
- Showed 'Syncing to SteamLedger' even when logged out
- Logic only checked old transaction history, not current login state

Solution:
- Track lastSyncSuccess timestamp when transactions sync
- Only show 'logged in' state if:
  • Successful sync within last 5 minutes AND
  • No pending transactions
- Otherwise show 'Create Account' section

This prevents false positives when:
- User logs out after syncing
- User never logged in but has no transactions
- Sync failed due to auth errors

More conservative = better conversion prompts

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

Author: rainerstudios

background.js

@@ -1290,6 +1290,10 @@ async function syncSingleTransaction(tx, transactions) {
       // Success - mark as synced
       tx.status = 'synced';
       tx.syncedAt = Date.now();
+
+      // Track last successful sync for login status detection
+      await chrome.storage.local.set({ lastSyncSuccess: Date.now() });
+
       console.log(`[Background] Successfully synced: ${tx.itemName}`);
       return;
 

popup.js

@@ -345,28 +345,28 @@ function initialize() {
 async function loadAccountStatus() {
   try {
     // Get pending transactions from storage
-    const result = await chrome.storage.local.get(['pendingTransactions']);
+    const result = await chrome.storage.local.get(['pendingTransactions', 'lastSyncSuccess']);
     const transactions = result.pendingTransactions || [];
+    const lastSyncSuccess = result.lastSyncSuccess || 0;
 
     const pendingCount = transactions.filter(tx =>
       tx.status === 'pending' || tx.status === 'failed'
     ).length;
 
-    const syncedCount = transactions.filter(tx =>
-      tx.status === 'synced'
-    ).length;
-
     const notLoggedInDiv = document.getElementById('accountNotLoggedIn');
     const loggedInDiv = document.getElementById('accountLoggedIn');
     const pendingCountSpan = document.getElementById('pendingCount');
 
-    // Show appropriate section based on sync status
-    if (syncedCount > 0 || pendingCount === 0) {
-      // User is likely logged in (has synced transactions) or no transactions yet
+    // Check if user successfully synced recently (within last 5 minutes)
+    const recentlySync = (Date.now() - lastSyncSuccess) < 5 * 60 * 1000;
+
+    // Conservative logic: Only show "logged in" if recently synced successfully
+    if (recentlySync && pendingCount === 0) {
+      // User is logged in and all transactions are synced
       notLoggedInDiv.style.display = 'none';
       loggedInDiv.style.display = 'block';
     } else {
-      // User has pending transactions (not logged in)
+      // Show "Create Account" if any pending OR no recent sync
       notLoggedInDiv.style.display = 'block';
       loggedInDiv.style.display = 'none';
       pendingCountSpan.textContent = pendingCount;