Rename last_updated_at to last_fetched_at in list metadata

Improves naming consistency with `last_first_page_fetched_at`. The field
tracks when any page was last fetched, so `last_fetched_at` is clearer
than the ambiguous `last_updated_at`.

Changes:
- Rename column in SQL migration
- Update `DbListMetadata` struct field
- Update `ListMetadataColumn` enum variant
- Update repository SQL and tests

Author: oguzkocer

wp_mobile_cache/migrations/0007-create-list-metadata-tables.sql

@@ -8,7 +8,7 @@ CREATE TABLE `list_metadata` (
   `current_page` INTEGER NOT NULL DEFAULT 0,
   `per_page` INTEGER NOT NULL DEFAULT 20,
   `last_first_page_fetched_at` TEXT,
-  `last_updated_at` TEXT,
+  `last_fetched_at` TEXT,
   `version` INTEGER NOT NULL DEFAULT 0,
 
   FOREIGN KEY (db_site_id) REFERENCES db_sites(id) ON DELETE CASCADE

wp_mobile_cache/src/db_types/db_list_metadata.rs

@@ -18,7 +18,7 @@ pub enum ListMetadataColumn {
     CurrentPage = 5,
     PerPage = 6,
     LastFirstPageFetchedAt = 7,
-    LastUpdatedAt = 8,
+    LastFetchedAt = 8,
     Version = 9,
 }
 
@@ -42,7 +42,7 @@ impl DbListMetadata {
             current_page: row.get_column(Col::CurrentPage)?,
             per_page: row.get_column(Col::PerPage)?,
             last_first_page_fetched_at: row.get_column(Col::LastFirstPageFetchedAt)?,
-            last_updated_at: row.get_column(Col::LastUpdatedAt)?,
+            last_fetched_at: row.get_column(Col::LastFetchedAt)?,
             version: row.get_column(Col::Version)?,
         })
     }

wp_mobile_cache/src/list_metadata.rs

@@ -21,8 +21,8 @@ pub struct DbListMetadata {
     pub per_page: i64,
     /// ISO 8601 timestamp of when page 1 was last fetched
     pub last_first_page_fetched_at: Option<String>,
-    /// ISO 8601 timestamp of last update
-    pub last_updated_at: Option<String>,
+    /// ISO 8601 timestamp of when any page was last fetched
+    pub last_fetched_at: Option<String>,
     /// Version number, incremented on page 1 refresh for concurrency control
     pub version: i64,
 }

wp_mobile_cache/src/repository/list_metadata.rs

@@ -277,7 +277,7 @@ impl ListMetadataRepository {
         self.get_or_create(executor, site, key)?;
 
         let sql = format!(
-            "UPDATE {} SET total_pages = ?, total_items = ?, current_page = ?, per_page = ?, last_updated_at = strftime('%Y-%m-%dT%H:%M:%fZ', 'now') WHERE db_site_id = ? AND key = ?",
+            "UPDATE {} SET total_pages = ?, total_items = ?, current_page = ?, per_page = ?, last_fetched_at = strftime('%Y-%m-%dT%H:%M:%fZ', 'now') WHERE db_site_id = ? AND key = ?",
             Self::header_table().table_name()
         );
 
@@ -747,7 +747,7 @@ mod tests {
     fn test_list_metadata_column_enum_matches_schema(test_ctx: TestContext) {
         // Verify column order by selecting specific columns and checking positions
         let sql = format!(
-            "SELECT rowid, db_site_id, key, total_pages, total_items, current_page, per_page, last_first_page_fetched_at, last_updated_at, version FROM {}",
+            "SELECT rowid, db_site_id, key, total_pages, total_items, current_page, per_page, last_first_page_fetched_at, last_fetched_at, version FROM {}",
             ListMetadataRepository::header_table().table_name()
         );
         let stmt = test_ctx.conn.prepare(&sql);
@@ -954,7 +954,7 @@ mod tests {
         assert_eq!(header.total_items, Some(100));
         assert_eq!(header.current_page, 1);
         assert_eq!(header.per_page, 20);
-        assert!(header.last_updated_at.is_some());
+        assert!(header.last_fetched_at.is_some());
     }
 
     #[rstest]