Media service to support the mobile apps

CHANGELOG.md

@@ -17,6 +17,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - [Global Styles](https://developer.wordpress.org/rest-api/reference/wp_global_styles/) endpoint
 - [Pattern Directory Items](https://developer.wordpress.org/rest-api/reference/pattern-directory-items/) endpoint
 - [Rendered Blocks](https://developer.wordpress.org/rest-api/reference/rendered-blocks/) endpoint
+- `MediaService` on `WpService` (sync, fetch, state tracking, `delete_media_permanently`) and `MediaMetadataCollectionWithEditContext`, mirroring the existing `PostService` / `PostMetadataCollectionWithEditContext` pattern for a cached, paginated, observable media list
+- `MediaListFilter`, the subset of `MediaListParams` that backs `MediaService.create_media_metadata_collection_with_edit_context` (excludes pagination, include/exclude, and date ranges)
+- `wp_mobile_cache` storage for media: `media_edit_context` table (migration 0014), `DbTable::MediaEditContext`, `EntityType::MediaEditContext`, and a `MediaRepository<EditContext>` mirroring `PostRepository` minus term relationships
+- `MetadataService::remove_entity_from_lists_with_key_prefix` so service-level deletes can scrub a deleted entity from every cached list for a site without waiting for a refresh
+
+### Changed
+
+- `MediaDetails` now derives `Eq + Hash` (raw-JSON-string comparison, FIXME left in place) and is exported via `#[uniffi::export(Eq, Hash)]`; `SparseMedia`'s `#[WpContextualDontDerivePartialEq]` opt-out is removed so `MediaWithEditContext` and the generated `FullEntityMediaWithEditContext` Swift wrapper synthesize `Equatable + Hashable`
 
 ### Removed
 

scripts/swift-bindings.sh

@@ -80,6 +80,12 @@ import SQLite3\
 extension AnyPostWithEditContext: Hashable {}
 extension AnyPostWithEmbedContext: Hashable {}
 extension AnyPostWithViewContext: Hashable {}
+
+// MediaWith*Context types contain `MediaDetails` (a reference type) which
+// prevents automatic Hashable synthesis. Add the conformance manually.
+extension MediaWithEditContext: Hashable {}
+extension MediaWithEmbedContext: Hashable {}
+extension MediaWithViewContext: Hashable {}
 PATCH
 }
 

wp_api/src/media.rs

@@ -359,7 +359,6 @@ impl From<MediaCreateParams> for HashMap<String, String> {
 }
 
 #[derive(Debug, Serialize, Deserialize, uniffi::Record, WpContextual)]
-#[WpContextualDontDerivePartialEq]
 pub struct SparseMedia {
     #[WpContext(edit, embed, view)]
     pub id: Option<MediaId>,
@@ -428,6 +427,7 @@ pub struct SparseMedia {
 
 #[derive(Debug, Serialize, Deserialize, uniffi::Object)]
 #[serde(transparent)]
+#[uniffi::export(Eq, Hash)]
 pub struct MediaDetails {
     pub payload: Box<RawValue>,
 }
@@ -457,6 +457,20 @@ impl MediaDetails {
     }
 }
 
+impl PartialEq for MediaDetails {
+    fn eq(&self, other: &Self) -> bool {
+        self.payload.get() == other.payload.get()
+    }
+}
+
+impl Eq for MediaDetails {}
+
+impl std::hash::Hash for MediaDetails {
+    fn hash<H: std::hash::Hasher>(&self, state: &mut H) {
+        self.payload.get().hash(state);
+    }
+}
+
 #[derive(Debug, uniffi::Enum)]
 pub enum MediaDetailsPayload {
     Audio(AudioMediaDetails),

wp_mobile/src/cache_key.rs

@@ -6,11 +6,11 @@
 
 use url::Url;
 use wp_api::{
-    posts::PostListParamsField, request::endpoint::posts_endpoint::PostEndpointType,
-    url_query::QueryPairsExtension,
+    media::MediaListParamsField, posts::PostListParamsField,
+    request::endpoint::posts_endpoint::PostEndpointType, url_query::QueryPairsExtension,
 };
 
-use crate::filters::PostListFilter;
+use crate::filters::{MediaListFilter, PostListFilter};
 
 /// Generates a cache key segment from a `PostEndpointType`.
 ///
@@ -103,6 +103,38 @@ pub fn post_list_filter_cache_key(filter: &PostListFilter) -> String {
     url.query().unwrap_or("").to_string()
 }
 
+/// Generates a deterministic cache key from `MediaListFilter`.
+///
+/// All fields in `MediaListFilter` are included in the cache key since it only
+/// contains filter-relevant fields (pagination, instance-specific, and date
+/// range fields are excluded by design in `MediaListFilter`).
+pub fn media_list_filter_cache_key(filter: &MediaListFilter) -> String {
+    let mut url = Url::parse("https://cache-key-generator.local").expect("valid base URL");
+
+    {
+        let mut q = url.query_pairs_mut();
+
+        // Alphabetically ordered for determinism.
+        q.append_vec_query_value_pair(MediaListParamsField::Author, &filter.author);
+        q.append_vec_query_value_pair(MediaListParamsField::AuthorExclude, &filter.author_exclude);
+        q.append_option_query_value_pair(
+            MediaListParamsField::MediaType,
+            filter.media_type.as_ref(),
+        );
+        q.append_option_query_value_pair(MediaListParamsField::MimeType, filter.mime_type.as_ref());
+        q.append_option_query_value_pair(MediaListParamsField::Order, filter.order.as_ref());
+        q.append_option_query_value_pair(MediaListParamsField::Orderby, filter.orderby.as_ref());
+        q.append_vec_query_value_pair(MediaListParamsField::Parent, &filter.parent);
+        q.append_vec_query_value_pair(MediaListParamsField::ParentExclude, &filter.parent_exclude);
+        q.append_option_query_value_pair(MediaListParamsField::Search, filter.search.as_ref());
+        q.append_vec_query_value_pair(MediaListParamsField::SearchColumns, &filter.search_columns);
+        q.append_vec_query_value_pair(MediaListParamsField::Slug, &filter.slug);
+        q.append_vec_query_value_pair(MediaListParamsField::Status, &filter.status);
+    }
+
+    url.query().unwrap_or("").to_string()
+}
+
 #[cfg(test)]
 mod tests {
     use super::*;
@@ -167,4 +199,37 @@ mod tests {
         let key = endpoint_type_cache_key(&PostEndpointType::Custom("products".to_string()));
         assert_eq!(key, "post_type_custom_products");
     }
+
+    #[test]
+    fn media_empty_filter_produces_empty_key() {
+        let filter = MediaListFilter::default();
+        let key = media_list_filter_cache_key(&filter);
+        assert_eq!(key, "");
+    }
+
+    #[test]
+    fn media_status_filter() {
+        use wp_api::media::MediaStatus;
+        let filter = MediaListFilter {
+            status: vec![MediaStatus::Inherit],
+            ..Default::default()
+        };
+        let key = media_list_filter_cache_key(&filter);
+        assert_eq!(key, "status=inherit");
+    }
+
+    #[test]
+    fn media_multi_field_sorted() {
+        use wp_api::media::{MediaStatus, MediaTypeParam};
+        use wp_api::users::UserId;
+        let filter = MediaListFilter {
+            status: vec![MediaStatus::Inherit],
+            author: vec![UserId(5)],
+            media_type: Some(MediaTypeParam::Image),
+            ..Default::default()
+        };
+        let key = media_list_filter_cache_key(&filter);
+        // Fields in alphabetical order: author, media_type, status
+        assert_eq!(key, "author=5&media_type=image&status=inherit");
+    }
 }