rework domain field

Cargo.lock

@@ -743,6 +743,7 @@ checksum = "cf8baf1c55e62ffcace7a9f06f4bd9cd3f0c4beb022d3b367256b91b87513d98"
 [[package]]
 name = "meta-feeder-sdk"
 version = "0.1.0"
+source = "git+https://github.com/worph/meta-feeder-sdk?tag=v1.1.0#4baf81e2bf87140175b1f51f6be51a84015e361a"
 dependencies = [
  "anyhow",
  "async-trait",

feeder-plugin/musicbrainz-feeder/Cargo.toml

@@ -18,9 +18,17 @@ name = "musicbrainz-feeder"
 path = "src/main.rs"
 
 [dependencies]
-# TEMPORARY: path dep while the SDK's new rev is unpushed. Flip to:
-#   meta-feeder-sdk = { git = "https://github.com/worph/meta-feeder-sdk", rev = "<sha>" }
-meta-feeder-sdk = { path = "../../../meta-feeder-sdk" }
+# Pinned by TAG, not rev or path. The tag is the SDK's release identity, and it
+# is the one form every feeder in the fleet can be lined up on.
+#
+# ⚠ It must never go back to `path = "../../../meta-feeder-sdk"`. That resolves
+# OUTSIDE this repo, and this repo is the Docker build context — so the image
+# build died at manifest resolution with a bare "No such file or directory",
+# and, worse, this crate became unconsumable as a git dependency by anyone
+# ("no matching package named `meta-feeder-sdk`"), because a consumer's checkout
+# has no such sibling directory either. A path dep here is not a local
+# convenience; it is a defect visible to every downstream repo.
+meta-feeder-sdk = { git = "https://github.com/worph/meta-feeder-sdk", tag = "v1.1.0" }
 
 tokio = { version = "1.40", features = ["full"] }
 axum = { version = "0.7", features = ["json"] }

feeder-plugin/musicbrainz-feeder/src/card.rs

@@ -262,6 +262,13 @@ impl Card {
         // is a well-formed query and routing needs no special case.
         fields.insert("fileType".to_string(), "card".to_string());
         fields.insert("contentKind".to_string(), self.kind.content_kind().to_string());
+        // Third axis (METADATA_KEYS.md §1): MusicBrainz is the identity graph
+        // that *defines* the music domain, so both card kinds land there.
+        // Literal rather than via `meta_feeder_sdk::domain` — this feeder pins
+        // the SDK by git tag. An album and an artist are both closed works, not
+        // instalments of a run, so the fourth axis is `standalone`.
+        fields.insert("domain".to_string(), "music".to_string());
+        fields.insert("workForm".to_string(), "standalone".to_string());
         fields.insert("title".to_string(), title.to_string());
 
         // The id bag. Which field carries the card's own MBID depends on what

feeder-plugin/musicbrainz-feeder/src/consts.rs

@@ -11,9 +11,9 @@
 /// application and carry contact information. A 403 here looks exactly like a
 /// rate-limit and is easy to misdiagnose, so keep this descriptive.
 pub const USER_AGENT: &str = concat!(
-    "meta-music/",
+    "musicbrainz-feeder/",
     env!("CARGO_PKG_VERSION"),
-    " ( https://github.com/worph/meta-feeder-music )"
+    " ( https://github.com/worph/meta-feeder-musicbrainz )"
 );
 
 // ---------------------------------------------------------------------------