refactor: drop unread SourceProductMirror fields

[alukach]

What

SourceProductMirror carried storage_type and is_primary fields that are deserialized from the API but never read anywhere — which is why the struct needed an #[allow(dead_code)]. Remove both fields and the attribute.

#[derive(Debug, Clone, Deserialize)]
pub struct SourceProductMirror {
    pub connection_id: String,
    pub prefix: String,
}

Why

serde ignores unknown JSON fields by default, so a struct only needs the fields it actually uses. Dropping the dead fields also lets the #[allow(dead_code)] go, so the lint stays honest for the rest of the struct.

Verification

cargo clippy --target wasm32-unknown-unknown is clean with the allow removed (confirming connection_id and prefix are both still used). Full pre-push suite green.

Found via ponytail over-engineering audit.

🤖 Generated with Claude Code

[claude]

Claude finished @alukach's task in 18s —— View job

---

✅ No blocking issues — safe to merge.

• src/registry.rs:259 — Clean removal. serde correctly ignores unknown fields by default, so dropping storage_type and is_primary won't break deserialization if the API still returns them. The #[allow(dead_code)] suppression going away is the right outcome, not a side effect.
• No other references to storage_type, is_primary, or the removed fields found anywhere in src/.

[github-actions]

🚀 Latest commit deployed to https://source-data-proxy-pr-159.source-coop.workers.dev

• Date: 2026-06-18T23:37:28Z
• Commit: ee02bbc