use struct in serialize impl

Cargo.lock

@@ -28,7 +28,7 @@ checksum = "c4512299f36f043ab09a583e57bceb5a5aab7a73db1805848e8fef3c9e8c78b3"
 
 [[package]]
 name = "card_game"
-version = "0.4.1"
+version = "0.4.0"
 dependencies = [
  "arrayvec",
  "serde",
@@ -52,17 +52,6 @@ dependencies = [
  "rand_core",
 ]
 
-[[package]]
-name = "console"
-version = "0.16.3"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d64e8af5551369d19cf50138de61f1c42074ab970f74e99be916646777f8fc87"
-dependencies = [
- "encode_unicode",
- "libc",
- "windows-sys",
-]
-
 [[package]]
 name = "cpufeatures"
 version = "0.3.0"
@@ -72,34 +61,12 @@ dependencies = [
  "libc",
 ]
 
-[[package]]
-name = "encode_unicode"
-version = "1.0.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "34aa73646ffb006b8f5147f3dc182bd4bcb190227ce861fc4a4844bf8e3cb2c0"
-
 [[package]]
 name = "equivalent"
 version = "1.0.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "877a4ace8713b0bcf2a4e7eec82529c029f1d0619886d18145fea96c3ffe5c0f"
 
-[[package]]
-name = "errno"
-version = "0.3.14"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "39cab71617ae0d63f51a36d69f866391735b51691dbda63cf6f96d042b63efeb"
-dependencies = [
- "libc",
- "windows-sys",
-]
-
-[[package]]
-name = "fastrand"
-version = "2.4.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9f1f227452a390804cdb637b74a86990f2a7d7ba4b7d5693aac9b4dd6defd8d6"
-
 [[package]]
 name = "foldhash"
 version = "0.1.5"
@@ -159,18 +126,6 @@ dependencies = [
  "serde_core",
 ]
 
-[[package]]
-name = "insta"
-version = "1.47.2"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "7b4a6248eb93a4401ed2f37dfe8ea592d3cf05b7cf4f8efa867b6895af7e094e"
-dependencies = [
- "console",
- "once_cell",
- "similar",
- "tempfile",
-]
-
 [[package]]
 name = "itoa"
 version = "1.0.18"
@@ -179,15 +134,12 @@ checksum = "8f42a60cbdf9a97f5d2305f08a87dc4e09308d1276d28c869c684d7777685682"
 
 [[package]]
 name = "klondike"
-version = "0.4.0"
+version = "0.3.0"
 dependencies = [
  "card_game",
- "insta",
  "rand",
- "rmp-serde",
  "serde",
  "serde_derive",
- "serde_json",
 ]
 
 [[package]]
@@ -206,6 +158,9 @@ dependencies = [
  "card_game",
  "klondike",
  "rand",
+ "rmp-serde",
+ "serde",
+ "serde_json",
 ]
 
 [[package]]
@@ -220,12 +175,6 @@ version = "0.2.186"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "68ab91017fe16c622486840e4c83c9a37afeff978bd239b5293d61ece587de66"
 
-[[package]]
-name = "linux-raw-sys"
-version = "0.12.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "32a66949e030da00e8c7d4434b251670a91556f4144941d37452769c25d58a53"
-
 [[package]]
 name = "log"
 version = "0.4.29"
@@ -247,12 +196,6 @@ dependencies = [
  "autocfg",
 ]
 
-[[package]]
-name = "once_cell"
-version = "1.21.4"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9f7c3e4beb33f85d45ae3e3a1792185706c8e16d043238c593331cc7cd313b50"
-
 [[package]]
 name = "prettyplease"
 version = "0.2.37"
@@ -323,19 +266,6 @@ dependencies = [
  "serde",
 ]
 
-[[package]]
-name = "rustix"
-version = "1.1.4"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "b6fe4565b9518b83ef4f91bb47ce29620ca828bd32cb7e408f0062e9930ba190"
-dependencies = [
- "bitflags",
- "errno",
- "libc",
- "linux-raw-sys",
- "windows-sys",
-]
-
 [[package]]
 name = "semver"
 version = "1.0.28"
@@ -384,12 +314,6 @@ dependencies = [
  "zmij",
 ]
 
-[[package]]
-name = "similar"
-version = "2.7.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "bbbb5d9659141646ae647b42fe094daf6c6192d1620870b449d9557f748b2daa"
-
 [[package]]
 name = "syn"
 version = "2.0.117"
@@ -401,19 +325,6 @@ dependencies = [
  "unicode-ident",
 ]
 
-[[package]]
-name = "tempfile"
-version = "3.27.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "32497e9a4c7b38532efcdebeef879707aa9f794296a4f0244f6f69e9bc8574bd"
-dependencies = [
- "fastrand",
- "getrandom",
- "once_cell",
- "rustix",
- "windows-sys",
-]
-
 [[package]]
 name = "unicode-ident"
 version = "1.0.24"
@@ -478,21 +389,6 @@ dependencies = [
  "semver",
 ]
 
-[[package]]
-name = "windows-link"
-version = "0.2.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f0805222e57f7521d6a62e36fa9163bc891acd422f971defe97d64e70d0a4fe5"
-
-[[package]]
-name = "windows-sys"
-version = "0.61.2"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ae137229bcbd6cdf0f7b80a31df61766145077ddf49416a728b02cb3921ff3fc"
-dependencies = [
- "windows-link",
-]
-
 [[package]]
 name = "wit-bindgen"
 version = "0.51.0"

Cargo.toml

@@ -8,8 +8,8 @@ members = [
 resolver = "3"
 
 [workspace.dependencies]
-card_game = { version = "0.4.1", path = "card_game", registry = "Quaternions" }
-klondike = { version = "0.4.0", path = "klondike", registry = "Quaternions" }
+card_game = { version = "0.4.0", path = "card_game", registry = "Quaternions" }
+klondike = { version = "0.3.0", path = "klondike", registry = "Quaternions" }
 
 [workspace.lints.rust]
 # unsafe_code = "forbid"

card_game/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "card_game"
-version = "0.4.1"
+version = "0.4.0"
 edition = "2024"
 repository = "https://git.aleshym.co/Quaternions/card_game"
 license = "MIT OR Apache-2.0"

card_game/src/lib.rs

@@ -234,7 +234,6 @@ impl<const CAP: usize> IntoIterator for Stack<CAP> {
 		self.0.into_iter()
 	}
 }
-
 #[cfg(feature = "serde")]
 impl<'de, const CAP: usize> serde::Deserialize<'de> for Stack<CAP> {
 	fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
@@ -251,10 +250,10 @@ impl<'de, const CAP: usize> serde::Deserialize<'de> for Stack<CAP> {
 			where
 				A: serde::de::SeqAccess<'de>,
 			{
-				use serde::de::Error;
 				let mut stack = Stack::new();
 				while let Some(card) = seq.next_element()? {
-					stack.try_push(card).map_err(A::Error::custom)?;
+					// TODO: Error
+					stack.try_push(card).unwrap();
 				}
 				Ok(stack)
 			}
@@ -675,12 +674,12 @@ where
 		"))]
 		struct SerializedSession<G: Game> {
 			config: SessionConfig<G::Config>,
-			initial_state: G,
-			instructions: Vec<G::Instruction>,
+			state: G,
+			history: Vec<G::Instruction>,
 		}
 		let serialized = SerializedSession::deserialize(deserializer)?;
-		let mut session = Session::new(serialized.initial_state, serialized.config);
-		for instruction in serialized.instructions {
+		let mut session = Session::new(serialized.state, serialized.config);
+		for instruction in serialized.history {
 			session.process_instruction(instruction);
 		}
 		Ok(session)
@@ -725,22 +724,22 @@ where
 		"))]
 		struct SerializedSession<'a, G: Game> {
 			config: &'a SessionConfig<G::Config>,
-			initial_state: &'a G,
-			instructions: History<'a, G>,
+			state: &'a G,
+			history: History<'a, G>,
 		}
 
 		// serialize the initial state of the game.
 		// if there is history, it is the first snapshot's state,
 		// otherwise it is the current game state since there are no moves.
-		let initial_state = if let Some(snapshot) = self.state.history.first() {
+		let state = if let Some(snapshot) = self.state.history.first() {
 			snapshot.state()
 		} else {
 			&self.state.state
 		};
 		let session = SerializedSession {
 			config: &self.config,
-			initial_state,
-			instructions: History(&self.state.history),
+			state,
+			history: History(&self.state.history),
 		};
 		session.serialize(serializer)
 	}

klondike-cli/Cargo.toml

@@ -8,5 +8,10 @@ card_game.workspace = true
 klondike.workspace = true
 rand = { version = "0.10.1", default-features = false, features = ["thread_rng"] }
 
+[dev-dependencies]
+rmp-serde = "1.3.1"
+serde = { version = "1.0.228", default-features = false }
+serde_json = "1.0.149"
+
 [lints]
 workspace = true

klondike-cli/src/main.rs

@@ -4,6 +4,9 @@ use klondike::{
 	KlondikePile, KlondikePileStack, SkipCards, Tableau, TableauStack,
 };
 
+#[cfg(test)]
+mod test;
+
 use std::fmt::Display;
 struct Displayed<T>(T);
 

klondike-cli/src/test.rs

@@ -1,6 +1,5 @@
-use crate::Klondike;
 use card_game::Session;
-
+use klondike::Klondike;
 #[test]
 fn test_is_winnable() {
 	// is winnable
@@ -16,7 +15,6 @@ fn test_is_winnable() {
 	}
 }
 
-#[cfg(feature = "serde")]
 #[test]
 fn test_json() {
 	let mut session = Session::new_default(Klondike::with_seed(124));
@@ -31,11 +29,8 @@ fn test_json() {
 	let round_trip_session: Session<Klondike> = serde_json::from_str(&serialized).unwrap();
 	let serialized2 = serde_json::to_string(&round_trip_session).unwrap();
 	assert_eq!(serialized, serialized2);
-
-	insta::assert_snapshot!(serialized);
 }
 
-#[cfg(feature = "serde")]
 #[test]
 fn test_rmp() {
 	let mut session = Session::new_default(Klondike::with_seed(124));
@@ -50,6 +45,4 @@ fn test_rmp() {
 	let round_trip_session: Session<Klondike> = rmp_serde::from_slice(&serialized).unwrap();
 	let serialized2 = rmp_serde::to_vec(&round_trip_session).unwrap();
 	assert_eq!(serialized, serialized2);
-
-	insta::assert_binary_snapshot!("save_rmp.bin", serialized);
 }

klondike/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "klondike"
-version = "0.4.0"
+version = "0.3.0"
 edition = "2024"
 
 [dependencies]
@@ -9,15 +9,9 @@ rand = { version = "0.10.1", default-features = false, features = ["std_rng"] }
 serde = { version = "1.0.228", default-features = false, optional = true }
 serde_derive = { version = "1.0.228", default-features = false, optional = true }
 
-[dev-dependencies]
-insta = "1.47.2"
-rmp-serde = "1.3.1"
-serde = { version = "1.0.228", default-features = false }
-serde_json = "1.0.149"
-
 [lints]
 workspace = true
 
 [features]
 default = ["serde"]
-serde = ["dep:serde", "dep:serde_derive", "card_game/serde"]
+serde = ["dep:serde", "dep:serde_derive"]

klondike/src/lib.rs

@@ -2,9 +2,6 @@ pub type Rng = rand::rngs::StdRng;
 
 use card_game::{Card, Game, Pile, Rank, Stack};
 
-#[cfg(test)]
-mod test;
-
 // test readme
 #[doc = include_str!("../README.md")]
 #[cfg(doctest)]
@@ -164,38 +161,6 @@ impl Tableau {
 		})
 	}
 }
-pub struct TableauIter {
-	tableau: Option<Tableau>,
-}
-impl TableauIter {
-	pub const fn new() -> Self {
-		Self {
-			tableau: Some(Tableau::ITER_BEGIN),
-		}
-	}
-}
-impl Default for TableauIter {
-	fn default() -> Self {
-		Self::new()
-	}
-}
-impl Iterator for TableauIter {
-	type Item = Tableau;
-	fn next(&mut self) -> Option<Self::Item> {
-		let tableau = self.tableau?;
-		self.tableau = tableau.next();
-		Some(tableau)
-	}
-}
-impl IntoIterator for Tableau {
-	type Item = Self;
-	type IntoIter = TableauIter;
-	fn into_iter(self) -> Self::IntoIter {
-		TableauIter {
-			tableau: Some(self),
-		}
-	}
-}
 
 #[derive(Clone, Copy, Debug, Eq, Hash, PartialEq)]
 pub enum Foundation {
@@ -216,38 +181,6 @@ impl Foundation {
 		})
 	}
 }
-pub struct FoundationIter {
-	foundation: Option<Foundation>,
-}
-impl FoundationIter {
-	pub const fn new() -> Self {
-		Self {
-			foundation: Some(Foundation::ITER_BEGIN),
-		}
-	}
-}
-impl Default for FoundationIter {
-	fn default() -> Self {
-		Self::new()
-	}
-}
-impl Iterator for FoundationIter {
-	type Item = Foundation;
-	fn next(&mut self) -> Option<Self::Item> {
-		let foundation = self.foundation?;
-		self.foundation = foundation.next();
-		Some(foundation)
-	}
-}
-impl IntoIterator for Foundation {
-	type Item = Self;
-	type IntoIter = FoundationIter;
-	fn into_iter(self) -> Self::IntoIter {
-		FoundationIter {
-			foundation: Some(self),
-		}
-	}
-}
 
 #[derive(Clone, Copy, Debug, Eq, Hash, PartialEq)]
 pub enum KlondikePile {

klondike/src/snapshots/klondike__test__json.snap

@@ -1,5 +0,0 @@
----
-source: klondike/src/test.rs
-expression: serialized
----
-{"config":{"inner":{"draw_stock":"DrawOne","move_from_foundation":"Allowed","scoring":{"move_to_foundation":10,"flip_up_bonus":5,"move_to_tableau":5,"move_from_foundation":-15,"recycle":0}},"undo_penalty":-15,"solve_moves_budget":100000,"solve_states_budget":100000},"initial_state":{"stock":{"face_down":[52,51,21,59,36,45,49,50,22,3,58,39,41,61,13,57,34,56,28,1,19,12,17,5],"face_up":[]},"foundations":[[],[],[],[]],"tableau1":{"face_down":[],"face_up":[40]},"tableau2":{"face_down":[9],"face_up":[4]},"tableau3":{"face_down":[35,8],"face_up":[60]},"tableau4":{"face_down":[11,27,44],"face_up":[25]},"tableau5":{"face_down":[6,7,23,18],"face_up":[29]},"tableau6":{"face_down":[2,55,10,42,24],"face_up":[54]},"tableau7":{"face_down":[26,43,38,53,37,33],"face_up":[20]}},"instructions":[336,100,720,619,606,6,720,19,16,388,497,580,388,720,139,720,19,720,17,31,720,720,720,7,720,720,523,458,720,720,720,720,720,720,720,720,43,43,720,720,720,139,720,19,720,43,720,43,720,720,720,720,720,720,720,720,720,623,619,720,720,720,720,720,720,720,720,720,720,693,281,333,253,209,113,87,720,720,720,720,720,720,720,720,75,223,619,533,74,2,720,116,331,279,279,305,305,29,25,24,12,37,25,222,24,42,126,471,720,720,720,362,292,331,253,720,19,720,183,151,12,67,336,61,93,235,720,7,2,0,7,38,26,14,27,15,26,36,0,7,3,6,18,43,36,24,12,0,1,19,38,26,39,27,15,38,26,36]}

klondike/src/snapshots/klondike__test__save_rmp.snap

@@ -1,6 +0,0 @@
----
-source: klondike/src/test.rs
-expression: serialized
-extension: bin
-snapshot_kind: binary
----