fix: correct composite key migration check

- Fix _check_composite_key_migration_needed to properly check if internalTransactionId is the primary key
- Use PRAGMA table_info pk flag instead of just checking column existence
- This ensures migration only runs when internalTransactionId is actually the primary key

leggend/services/database_service.py

@@ -551,10 +551,15 @@ class DatabaseService:
             # Check if transactions table has the old primary key structure
             cursor.execute("PRAGMA table_info(transactions)")
             columns = cursor.fetchall()
-            column_names = [col[1] for col in columns]
 
-            # If we have internalTransactionId as primary key, migration is needed
+            # Check if internalTransactionId is the primary key (old structure)
-            if "internalTransactionId" in column_names:
+            internal_transaction_id_is_pk = any(
+                col[1] == "internalTransactionId" and col[5] == 1  # col[5] is pk flag
+                for col in columns
+            )
+
+            # If internalTransactionId is still the primary key, migration is needed
+            if internal_transaction_id_is_pk:
                 # Check if there are duplicate (accountId, transactionId) pairs
                 cursor.execute("""
                     SELECT COUNT(*) as duplicates
@@ -570,6 +575,7 @@ class DatabaseService:
                 conn.close()
                 return duplicates > 0
             else:
+                # Migration already completed
                 conn.close()
                 return False