Add more unknown variants

src/modules/civilization/models/crime.rs

@@ -96,6 +96,11 @@ pub enum Crime {
 
     #[serde(rename = "onFoot_propertyTheft")]
     OnFootPropertyTheft,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }
 
 #[cfg(test)]

src/modules/civilization/models/engineer.rs

@@ -48,6 +48,11 @@ pub enum Engineer {
     /// This is a special engineer that does not actually exist, but seems to be used for
     /// pre-engineered modules.
     System,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }
 
 impl Engineer {
@@ -92,6 +97,9 @@ impl Engineer {
             Engineer::RosaDayette => 400012,
             Engineer::YiShen => 400013,
             Engineer::System => 399999,
+
+            #[cfg(feature = "allow-unknown")]
+            Engineer::Unknown(_) => 0,
         }
     }
 
@@ -136,6 +144,9 @@ impl Engineer {
             Engineer::RosaDayette => 59166629864010,
             Engineer::YiShen => 13736779007129,
             Engineer::System => 0,
+
+            #[cfg(feature = "allow-unknown")]
+            Engineer::Unknown(_) => 0,
         }
     }
 
@@ -180,6 +191,9 @@ impl Engineer {
             Engineer::RosaDayette => 128986587,
             Engineer::YiShen => 128987355,
             Engineer::System => 0,
+
+            #[cfg(feature = "allow-unknown")]
+            Engineer::Unknown(_) => 0,
         }
     }
 }
@@ -229,6 +243,9 @@ impl Display for Engineer {
                 Engineer::RosaDayette => "Rosa Dayette",
                 Engineer::YiShen => "Yi Shen",
                 Engineer::System => "System",
+
+                #[cfg(feature = "allow-unknown")]
+                Engineer::Unknown(unknown) => return write!(f, "Unknown engineer: {}", unknown),
             }
         )
     }

src/modules/exploration/models/codex_anomaly_entry.rs

@@ -6,6 +6,7 @@ use thiserror::Error;
 
 /// Codex entries related to anomalies.
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq, Eq, Hash)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum CodexAnomalyEntry {
     LagrangeClouds,
 

src/modules/exploration/models/codex_geological_entry.rs

@@ -7,6 +7,7 @@ use thiserror::Error;
 
 /// Codex entries related to geological points-of-interest.
 #[derive(Debug, Serialize, Clone, PartialEq, Eq, Hash)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum CodexGeologicalEntry {
     Fumarole,
     FumaroleAmmoniaGeysers,

src/modules/exploration/models/codex_guardian_entry.rs

@@ -6,6 +6,7 @@ use std::str::FromStr;
 use thiserror::Error;
 
 #[derive(Debug, Serialize, Clone, PartialEq, Eq, Hash)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum CodexGuardianEntry {
     AncientCasket,
     AncientKey,

src/modules/exploration/models/codex_organic_structure_entry.rs

@@ -7,6 +7,7 @@ use thiserror::Error;
 /// Codex entries related to organic structures other than the ones already covered by
 /// [crate::exobiology::Genus], [crate::exobiology::Species] and [crate::exobiology::Variant].
 #[derive(Debug, Serialize, Clone, PartialEq, Eq, Hash)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum CodexOrganicStructureEntry {
     StolonTree,
 

src/modules/exploration/models/codex_thargoid_entry.rs

@@ -7,6 +7,7 @@ use thiserror::Error;
 
 /// Codex entries related to Thargoids.
 #[derive(Debug, Serialize, Clone, PartialEq, Eq, Hash)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum CodexThargoidEntry {
     LargeSpike,
     Tower,

src/modules/galaxy/models/atmosphere_type.rs

@@ -66,6 +66,11 @@ pub enum AtmosphereType {
 
     #[serde(alias = "metallic vapour")]
     MetallicVapour,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }
 
 impl FromStr for AtmosphereType {

src/modules/galaxy/models/body_type.rs

@@ -11,8 +11,4 @@ pub enum BodyType {
 
     // TODO add description on when this is used
     Null,
-
-    #[cfg(feature = "allow-unknown")]
-    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
-    Unknown(String),
 }

src/modules/galaxy/models/star_luminosity.rs

@@ -99,6 +99,6 @@
         assert!(StarLuminosity::Vb > StarLuminosity::Vz);
         assert!(StarLuminosity::Vz > StarLuminosity::VI);
         assert!(StarLuminosity::VI > StarLuminosity::VII);
         assert!(StarLuminosity::VII > StarLuminosity::Zero);
     }
-}
+}

src/modules/odyssey/models/citizen.rs

@@ -47,6 +47,11 @@ pub enum Citizen {
         alias = "rangedsuitai_class3"
     )]
     Sniper,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }
 
 impl Display for Citizen {
@@ -62,6 +67,9 @@ impl Display for Citizen {
                 Citizen::Scout => "Scout",
                 Citizen::Striker => "Striker",
                 Citizen::Sniper => "Sniper",
+
+                #[cfg(feature = "allow-unknown")]
+                Citizen::Unknown(unknown) => return write!(f, "Unknown citizen: {}", unknown),
             }
         )
     }

src/modules/odyssey/models/suit_mod.rs

@@ -3,6 +3,7 @@ use serde::{Deserialize, Serialize};
 /// A mod applied to a suit.
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
 #[serde(rename_all = "PascalCase")]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum SuitMod {
     /// Reduces the battery consumption of the suit's utility tool.
     #[serde(rename = "suit_reducedtoolbatteryconsumption")]
@@ -59,4 +60,9 @@ pub enum SuitMod {
     /// Decreases noise and in turn improves stealth.
     #[serde(rename = "suit_quieterfootsteps")]
     QuieterFootsteps,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }

src/modules/odyssey/models/weapon.rs

@@ -3,6 +3,7 @@ use std::fmt::{Display, Formatter};
 use serde::{Deserialize, Serialize};
 
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum Weapon {
     #[serde(
         alias = "Wpn_M_AssaultRifle_Kinetic_FAuto",
@@ -66,6 +67,11 @@ pub enum Weapon {
 
     #[serde(alias = "Wpn_S_Pistol_Laser_SAuto", alias = "wpn_s_pistol_laser_sauto")]
     TKZenith,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }
 
 impl Display for Weapon {
@@ -87,6 +93,9 @@ impl Display for Weapon {
                 Weapon::TKAphelion => "TK Aphelion",
                 Weapon::TKEclipse => "TK Eclipse",
                 Weapon::TKZenith => "TK Zenith ",
+
+                #[cfg(feature = "allow-unknown")]
+                Weapon::Unknown(unknown) => return write!(f, "Unknown weapon {}", unknown),
             }
         )
     }

src/modules/odyssey/models/weapon_mod.rs

@@ -3,6 +3,7 @@ use serde::{Deserialize, Serialize};
 /// A mod applied to an Odyssey weapon.
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
 #[serde(rename_all = "PascalCase")]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum WeaponMod {
     #[serde(rename = "weapon_handling")]
     FasterHandling,
@@ -33,4 +34,9 @@ pub enum WeaponMod {
 
     #[serde(rename = "weapon_stability")]
     Stability,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }

src/modules/ship/models/blueprint.rs

@@ -3,6 +3,7 @@ use std::fmt::{Display, Formatter};
 
 /// Engineering blueprint that can be applied to certain ship modules.
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum Blueprint {
     // Armor
     #[serde(rename = "Armour_Explosive")]

src/modules/ship/models/blueprint_modifier.rs

@@ -3,6 +3,7 @@ use serde::{Deserialize, Serialize};
 /// When applying blueprint to modules, modifiers are applied to the modules which are the things
 /// that actually change the stats of the module.
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum BlueprintModifier {
     Size,
     Class,

src/modules/ship/models/fighter_loadout.rs

@@ -1,6 +1,7 @@
 use serde::{Deserialize, Serialize};
 
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum FighterLoadout {
     #[serde(rename = "zero")]
     Zero,
@@ -22,4 +23,9 @@ pub enum FighterLoadout {
 
     #[serde(rename = "starter")]
     Starter,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }

src/modules/ship/models/fighter_type.rs

@@ -1,6 +1,7 @@
 use serde::{Deserialize, Serialize};
 
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum FighterType {
     #[serde(rename = "independent_fighter")]
     Taipan,
@@ -10,4 +11,9 @@ pub enum FighterType {
 
     #[serde(rename = "federation_fighter")]
     F63Condor,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }

src/modules/ship/models/ship_module.rs

@@ -35,6 +35,7 @@ pub mod ship_weapon_color;
 ///
 /// The same is true for core internals and optional internals which both use [ShipInternalModule].
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum ShipModule {
     /// Special case for the cargo bay door.
     #[serde(

src/modules/ship/models/ship_slot.rs

@@ -21,6 +21,7 @@
 
 // TODO kinda want to refactor this to use untagged variants
 #[derive(Debug, Serialize, Clone, PartialEq)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum ShipSlotKind {
     ShipCockpit,
     CargoHatch,
@@ -44,9 +45,12 @@
     EngineColor,
     WeaponColor,
     ShipKitSpoiler,
     ShipKitWings,
     ShipKitTail,
     ShipKitBumper,
+
+    // There is currently no Unknown variant here as it wouldn't make sense to have an unknown kind
+    // and then return a ShipSlot after parsing.
 }
 
 #[derive(Debug, Error)]

src/modules/ship/models/srv_type.rs

@@ -3,13 +3,19 @@ use std::fmt::{Display, Formatter};
 use serde::{Deserialize, Serialize};
 
 #[derive(Debug, Default, Serialize, Deserialize, Clone, PartialEq)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum SRVType {
     #[default]
     #[serde(rename = "testbuggy")]
     Scarab,
 
     #[serde(rename = "combat_multicrew_srv_01")]
     Scorpion,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }
 
 impl Display for SRVType {
@@ -20,6 +26,9 @@ impl Display for SRVType {
             match self {
                 SRVType::Scarab => "Scarab",
                 SRVType::Scorpion => "Scorpion",
+
+                #[cfg(feature = "allow-unknown")]
+                SRVType::Unknown(unknown) => return write!(f, "Unknown SRV type: {}", unknown),
             }
         )
     }

src/modules/status/models/gui_focus.rs

@@ -46,6 +46,7 @@ impl TryFrom<u8> for GuiFocus {
             9 => GuiFocus::FSSMode,
             10 => GuiFocus::SAAMode,
             11 => GuiFocus::Codex,
+
             _ => return Err(GuiFocusError::UnknownValue(value)),
         })
     }
@@ -68,6 +69,7 @@ impl FromStr for GuiFocus {
             "FSSMode" => GuiFocus::FSSMode,
             "SAAMode" => GuiFocus::SAAMode,
             "Codex" => GuiFocus::Codex,
+
             _ => return Err(GuiFocusError::UnknownStringValue(s.to_string())),
         })
     }

src/modules/thargoid/models/thargoid_ship.rs

@@ -1,10 +1,16 @@
 use serde::{Deserialize, Serialize};
 
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
+#[cfg_attr(not(feature = "allow-unknown"), non_exhaustive)]
 pub enum ThargoidShip {
     #[serde(rename = "scout_hq")]
     Scout,
 
     #[serde(rename = "glaive")]
     Glaive,
+
+    #[cfg(feature = "allow-unknown")]
+    #[cfg_attr(docsrs, doc(cfg(feature = "allow-unknown")))]
+    #[serde(untagged)]
+    Unknown(String),
 }