refactor(sidebar): derive recent entry id from Entry.id, cover clearAll

Remove private entryId helper — push now constructs Entry first and
deduplicates via Entry.id, keeping a single identity source.

Add ClearAll test covering two keys (nil and string database).

Author: J2TeamNNL

TablePro/Core/Storage/RecentTablesStore.swift

@@ -28,17 +28,10 @@ final class RecentTablesStore {
 
     func push(connectionID: UUID, database: String?, table: TableInfo) {
         let key = Key(connectionID: connectionID, database: database)
+        let entry = Entry(name: table.name, schema: table.schema, type: table.type)
         var list = entriesByKey[key] ?? []
-        let newEntryId = entryId(name: table.name, schema: table.schema)
-        list.removeAll { $0.id == newEntryId }
-        list.insert(
-            Entry(
-                name: table.name,
-                schema: table.schema,
-                type: table.type
-            ),
-            at: 0
-        )
+        list.removeAll { $0.id == entry.id }
+        list.insert(entry, at: 0)
         if list.count > cap {
             list = Array(list.prefix(cap))
         }
@@ -61,8 +54,4 @@ final class RecentTablesStore {
     }
 
     var cappedSize: Int { cap }
-
-    private func entryId(name: String, schema: String?) -> String {
-        schema.map { "\($0).\(name)" } ?? name
-    }
 }

TableProTests/Storage/RecentTablesStoreTests.swift

@@ -92,4 +92,16 @@ struct RecentTablesStoreTests {
         #expect(store.entries(connectionID: conn, database: nil).map(\.name) == ["sqlite_table"])
         #expect(store.entries(connectionID: conn, database: "postgres").map(\.name) == ["pg_table"])
     }
+
+    @Test("ClearAll empties every key")
+    func clearAllEmptiesEveryKey() {
+        let store = makeStore()
+        let connA = UUID()
+        let connB = UUID()
+        store.push(connectionID: connA, database: "db", table: makeTable("a"))
+        store.push(connectionID: connB, database: nil, table: makeTable("b"))
+        store.clearAll()
+        #expect(store.entries(connectionID: connA, database: "db").isEmpty)
+        #expect(store.entries(connectionID: connB, database: nil).isEmpty)
+    }
 }