refactor: change factionType filter to accept multiple values

Rename to factionTypes (plural) and accept a list so roster builders
can filter by multiple faction classifications at once, e.g.
factionTypes: [GREAT_HOUSE, PERIPHERY]. Uses IN() instead of = in
the availability subquery.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: graylikeme

crates/api/src/db/units.rs

@@ -49,7 +49,7 @@ pub struct UnitFilter<'a> {
     pub bv_min: Option<i32>,
     pub bv_max: Option<i32>,
     pub faction_slug: Option<&'a str>,
-    pub faction_type: Option<&'a str>,
+    pub faction_types: &'a [&'a str],
     pub era_slug: Option<&'a str>,
     pub is_omnimech: Option<bool>,
     pub config: Option<&'a str>,
@@ -131,13 +131,16 @@ pub async fn search(
         builder.push_bind(faction);
         builder.push(")");
     }
-    if let Some(ft) = filter.faction_type {
+    if !filter.faction_types.is_empty() {
         builder.push(r#" AND EXISTS (
             SELECT 1 FROM unit_availability ua
             JOIN factions f2 ON f2.id = ua.faction_id
-            WHERE ua.unit_id = u.id AND f2.faction_type = "#);
-        builder.push_bind(ft);
-        builder.push(")");
+            WHERE ua.unit_id = u.id AND f2.faction_type IN ("#);
+        let mut sep = builder.separated(", ");
+        for ft in filter.faction_types {
+            sep.push_bind(*ft);
+        }
+        builder.push("))");
     }
     if let Some(era) = filter.era_slug {
         builder.push(r#" AND EXISTS (
@@ -373,7 +376,7 @@ mod tests {
             bv_min: None,
             bv_max: None,
             faction_slug: None,
-            faction_type: None,
+            faction_types: &[],
             era_slug: None,
             is_omnimech: None,
             config: None,

crates/api/src/graphql/query.rs

@@ -148,7 +148,7 @@ impl QueryRoot {
         #[graphql(desc = "Minimum Battle Value filter (inclusive).")] bv_min: Option<i32>,
         #[graphql(desc = "Maximum Battle Value filter (inclusive).")] bv_max: Option<i32>,
         #[graphql(desc = "Filter to units available to this faction. Lowercase, hyphen-separated slug (e.g. \"clan-wolf\").")] faction_slug: Option<String>,
-        #[graphql(desc = "Filter to units available to any faction of this type.")] faction_type: Option<FactionTypeFilter>,
+        #[graphql(desc = "Filter to units available to any faction of these types. Accepts multiple values.")] faction_types: Option<Vec<FactionTypeFilter>>,
         #[graphql(desc = "Filter to units available in this era.")] era_slug: Option<EraFilter>,
         #[graphql(desc = "Filter to OmniMechs only (true) or non-OmniMechs (false).")] is_omnimech: Option<bool>,
         #[graphql(desc = "Filter by chassis config. One of: Biped, Quad, Tripod, LAM.")] config: Option<String>,
@@ -161,6 +161,12 @@ impl QueryRoot {
         let first = first.unwrap_or(20).clamp(1, 100) as i64;
         let after_cursor = after.as_deref().and_then(decode_cursor);
 
+        let faction_type_strs: Vec<&str> = faction_types
+            .unwrap_or_default()
+            .iter()
+            .map(|f| f.as_db_str())
+            .collect();
+
         let filter = units::UnitFilter {
             name_search: name_search.as_deref(),
             tech_base: tech_base.map(|t| t.as_db_str()),
@@ -170,7 +176,7 @@ impl QueryRoot {
             bv_min,
             bv_max,
             faction_slug: faction_slug.as_deref(),
-            faction_type: faction_type.map(|f| f.as_db_str()),
+            faction_types: &faction_type_strs,
             era_slug: era_slug.map(|e| e.as_db_str()),
             is_omnimech,
             config: config.as_deref(),