From dc94f2060c456087aa71239aeb7edfb26eb86989 Mon Sep 17 00:00:00 2001 From: 2ndbeam <2ndbeam@disroot.org> Date: Tue, 2 Dec 2025 16:12:42 +0300 Subject: [PATCH 1/7] feat: Added interaction with accounts in CLI - Account creation - Account deletion - Account balance management - Account quest completion - Added account CRUD test in tests/io.rs --- src/bin/cli.rs | 199 +++++++++++++++++++++++++++++++-- tests/io.rs | 33 ++++-- tests/io/accounts/.placeholder | 1 + 3 files changed, 215 insertions(+), 18 deletions(-) create mode 100644 tests/io/accounts/.placeholder diff --git a/src/bin/cli.rs b/src/bin/cli.rs index 4d06a2a..05e2c24 100644 --- a/src/bin/cli.rs +++ b/src/bin/cli.rs @@ -2,13 +2,13 @@ use std::path::PathBuf; use clap::{Parser,Subcommand,Args,ValueEnum}; use serde::Deserialize; -use squad_quest::{SquadObject, config::Config, quest::{Quest,QuestDifficulty as LibQuestDifficulty}}; +use squad_quest::{SquadObject, account::Account, config::Config, quest::{Quest,QuestDifficulty as LibQuestDifficulty}}; use toml::value::Date; use chrono::{Datelike, NaiveDate, Utc}; #[derive(Deserialize)] struct DateWrapper { - date: Date + date: Date, } fn parse_date(arg: &str) -> Result { @@ -33,7 +33,10 @@ struct Cli { enum Objects { /// Operations on the quests #[command(subcommand)] - Quest(QuestCommands) + Quest(QuestCommands), + /// Operations on the accounts + #[command(subcommand)] + Account(AccountCommands), } #[derive(Copy, Clone, PartialEq, Eq, PartialOrd, Ord, ValueEnum)] @@ -45,7 +48,7 @@ enum QuestDifficulty { /// Hard quest Hard, /// Special case of hard quests. - Secret + Secret, } impl From for LibQuestDifficulty { @@ -75,11 +78,12 @@ enum QuestCommands { Publish(QuestPublishArgs), } + #[derive(Args)] struct QuestListArgs { /// Only list id and name of the quest #[arg(short, long)] - short: bool + short: bool, } #[derive(Args)] @@ -137,7 +141,7 @@ struct QuestUpdateArgs { #[derive(Args)] struct QuestDeleteArgs { /// Id of the quest to delete - id: u16 + id: u16, } #[derive(Args)] @@ -149,6 +153,61 @@ struct QuestPublishArgs { reverse: bool, } +#[derive(Subcommand)] +enum AccountCommands { + /// List accounts + List, + /// Create empty account + Create(AccountCreateArgs), + /// Update balance value + Balance(AccountBalanceArgs), + /// Approve account answer for quest + Complete(AccountCompleteArgs), + /// Delete account + Delete(AccountDeleteArgs), +} + +#[derive(Args)] +struct AccountCreateArgs { + /// Account will be created with this id + id: String, +} + +#[derive(Clone, Copy, PartialEq, Eq, ValueEnum)] +enum AccountBalanceActions { + Set, + Add, + Remove, +} + +#[derive(Args)] +struct AccountBalanceArgs { + /// Account id + id: String, + /// What to do with the balance + #[arg(value_enum)] + action: AccountBalanceActions, + /// Amount of doing + value: u32, + /// If action is remove, set balance to 0 if the result is negative instead of returning error + #[arg(short,long)] + negative_ok: bool, +} + +#[derive(Args)] +struct AccountCompleteArgs { + /// Id of the account + account: String, + /// Id of the quest + quest: u16, +} + +#[derive(Args)] +struct AccountDeleteArgs { + /// Id of the account to delete + id: String, +} + fn print_quest_short(quest: &Quest) { println!("Quest #{}: {}", quest.id, quest.name); } @@ -296,7 +355,133 @@ fn main() { eprintln!("Error: couldn't find quest with id {}.", args.id); } } - } + }, + } + }, + Objects::Account(args) => { + match args { + AccountCommands::List => { + let accounts = config.load_accounts(); + + for account in accounts { + println!("\"{}\": Balance {}", account.id, account.balance); + } + }, + AccountCommands::Create(args) => { + let account = Account { + id: args.id.clone(), + ..Default::default() + }; + + let accounts = config.load_accounts(); + + if let Some(_) = accounts.iter().find(|a| a.id == account.id) { + eprintln!("Error: account {} exists.", account.id); + return; + } + + let accounts_path = config.full_accounts_path(); + + match account.save(accounts_path) { + Ok(_) => { + println!("Successfully created account \"{}\"", account.id); + }, + Err(error) => { + eprintln!("Error while saving account: {error}"); + } + } + }, + AccountCommands::Balance(args) => { + let mut accounts = config.load_accounts(); + + let account = match accounts.iter_mut().find(|a| a.id == args.id) { + Some(acc) => acc, + None => { + eprintln!("Could not find account \"{}\"", args.id); + return; + } + }; + + match args.action { + AccountBalanceActions::Set => { + account.balance = args.value; + }, + AccountBalanceActions::Add => { + account.balance += args.value; + }, + AccountBalanceActions::Remove => { + if args.value > account.balance { + if args.negative_ok { + account.balance = 0u32; + } else { + eprintln!("Error: balance ({}) is less than {}.", account.balance, args.value); + return; + } + } else { + account.balance -= args.value; + } + } + } + + let accounts_path = config.full_accounts_path(); + + match account.save(accounts_path) { + Ok(_) => { + println!("Successfully updated account \"{}\" balance.", account.id); + }, + Err(error) => { + eprintln!("Error while saving account: {error}"); + } + }; + }, + AccountCommands::Complete(args) => { + let mut accounts = config.load_accounts(); + + let account = match accounts.iter_mut().find(|a| a.id == args.account) { + Some(acc) => acc, + None => { + eprintln!("Could not find account \"{}\"", args.account); + return; + } + }; + + let quests = config.load_quests(); + + if let None = quests.iter().find(|q| q.id == args.quest) { + eprintln!("Could not find quest #{}", args.quest); + return; + } + + match account.quests_completed.iter().find(|qid| **qid == args.quest) { + Some(_) => { + println!("Quest #{} is already completed on account \"{}\"", args.quest, args.account); + }, + None => { + account.quests_completed.push(args.quest); + let accounts_path = config.full_accounts_path(); + match account.save(accounts_path) { + Ok(_) => { + println!("Account \"{}\" completed quest #{}.", args.account, args.quest); + }, + Err(error) => { + eprintln!("Error while saving account: {error}"); + } + } + } + } + }, + AccountCommands::Delete(args) => { + let mut accounts_path = config.full_accounts_path(); + accounts_path.push(format!("{}.toml", args.id)); + match Account::delete(accounts_path) { + Ok(_) => { + println!("Successfully deleted account \"{}\".", args.id); + }, + Err(error) => { + eprintln!("Error deleting account: {error}"); + } + } + }, } } } diff --git a/tests/io.rs b/tests/io.rs index 983e4cd..356f972 100644 --- a/tests/io.rs +++ b/tests/io.rs @@ -1,5 +1,4 @@ -use squad_quest::{SquadObject, config::Config, error::Error, quest::Quest}; -use std::path::PathBuf; +use squad_quest::{SquadObject, account::Account, config::Config, quest::Quest}; const CONFIG_PATH: &str = "tests/io/config.toml"; @@ -7,23 +6,35 @@ const CONFIG_PATH: &str = "tests/io/config.toml"; // and Quest::save can override files, // so this test covers full quest CRUD #[test] -fn quest_crud() -> Result<(), Error> { +fn quest_crud() { let config = Config::load(CONFIG_PATH.into()); - let mut quests_path = PathBuf::from(CONFIG_PATH).parent().unwrap().to_owned(); - quests_path.push(config.quests_path); + let mut quests_path = config.full_quests_path(); let quest = Quest::default(); - println!("{:?}", quests_path.clone()); - - quest.save(quests_path.clone())?; + quest.save(quests_path.clone()).unwrap(); let filename = format!("{}.toml", quest.id); quests_path.push(filename); - Quest::delete(quests_path)?; - - Ok(()) + Quest::delete(quests_path).unwrap(); +} + +#[test] +fn account_crud() { + let config = Config::load(CONFIG_PATH.into()); + + let mut accounts_path = config.full_accounts_path(); + + let account = Account::default(); + + account.save(accounts_path.clone()).unwrap(); + + let filename = format!("{}.toml", account.id); + + accounts_path.push(filename); + + Account::delete(accounts_path).unwrap(); } diff --git a/tests/io/accounts/.placeholder b/tests/io/accounts/.placeholder new file mode 100644 index 0000000..16680ce --- /dev/null +++ b/tests/io/accounts/.placeholder @@ -0,0 +1 @@ +Placeholder file for git From b9f75e426c7c37140622541f32b00bb9f43cac3e Mon Sep 17 00:00:00 2001 From: 2ndbeam <2ndbeam@disroot.org> Date: Wed, 3 Dec 2025 17:01:40 +0300 Subject: [PATCH 2/7] feat: Added Map - Implemented Map - Partially implemented CLI interaction with map - Added load_map test --- src/bin/cli.rs | 185 +++++++++++++++++++++++++++++++++++++++----- src/config/mod.rs | 27 ++++++- src/map/mod.rs | 108 +++++++++++++++++++++++++- tests/io/map.toml | 1 + tests/main.rs | 49 +++++++++++- tests/main/map.toml | 26 +++++++ 6 files changed, 371 insertions(+), 25 deletions(-) create mode 100644 tests/io/map.toml create mode 100644 tests/main/map.toml diff --git a/src/bin/cli.rs b/src/bin/cli.rs index 05e2c24..19bbf4e 100644 --- a/src/bin/cli.rs +++ b/src/bin/cli.rs @@ -1,8 +1,8 @@ -use std::path::PathBuf; +use std::path::{Path, PathBuf}; use clap::{Parser,Subcommand,Args,ValueEnum}; use serde::Deserialize; -use squad_quest::{SquadObject, account::Account, config::Config, quest::{Quest,QuestDifficulty as LibQuestDifficulty}}; +use squad_quest::{SquadObject, account::Account, config::Config, error::Error, map::{Map, Room}, quest::{Quest,QuestDifficulty as LibQuestDifficulty}}; use toml::value::Date; use chrono::{Datelike, NaiveDate, Utc}; @@ -37,6 +37,9 @@ enum Objects { /// Operations on the accounts #[command(subcommand)] Account(AccountCommands), + /// Operations on the map rooms + #[command(subcommand)] + Map(MapCommands), } #[derive(Copy, Clone, PartialEq, Eq, PartialOrd, Ord, ValueEnum)] @@ -208,6 +211,62 @@ struct AccountDeleteArgs { id: String, } +#[derive(Subcommand)] +enum MapCommands { + /// List all rooms with connections + List, + /// Add new room to map + Add(MapAddArgs), + /// Connect two rooms + Connect(MapConnectArgs), + /// Disconnect two rooms if they're connected + Disconnect(MapConnectArgs), + /// Remove all connections with the room + Delete(MapDeleteArgs), + /// Update room data + Update(MapUpdateArgs), +} + +#[derive(Args)] +struct MapAddArgs { + /// Name of the room + name: String, + /// Price of the room + value: u32, + /// Optional description for the room + #[arg(long,short)] + description: Option, +} + +#[derive(Args)] +struct MapConnectArgs { + /// First room ID + first: u16, + /// Second room ID + second: u16, +} + +#[derive(Args)] +struct MapDeleteArgs { + /// ID of the room to delete + id: u16, +} + +#[derive(Args)] +struct MapUpdateArgs { + /// ID of the room to update + id: u16, + /// Room name + #[arg(short,long)] + name: Option, + /// Room description + #[arg(short,long)] + description: Option, + /// Room price + #[arg(short,long)] + value: Option, +} + fn print_quest_short(quest: &Quest) { println!("Quest #{}: {}", quest.id, quest.name); } @@ -219,7 +278,7 @@ fn print_quest_long(quest: &Quest) { println!("Answer:\n{}", quest.answer); } -fn main() { +fn main() -> Result<(), Error> { let cli = Cli::parse(); let config = Config::load(cli.config.clone()); @@ -241,9 +300,6 @@ fn main() { let mut quests = config.load_quests(); quests.sort_by(|a,b| a.id.cmp(&b.id)); let next_id = match quests.last() { - Some(quest) if quest.id == u16::MAX => { - panic!("Error: quest list contains quest with u16::MAX id."); - }, Some(quest) => quest.id + 1u16, None => 0u16 }; @@ -342,8 +398,7 @@ fn main() { let not_str = if args.reverse {" not "} else {" "}; if quest.public != args.reverse { - println!("Quest #{} is already{}public", quest.id, not_str); - return; + panic!("Quest #{} is already{}public", quest.id, not_str); } quest.public = !args.reverse; @@ -358,8 +413,8 @@ fn main() { }, } }, - Objects::Account(args) => { - match args { + Objects::Account(commands) => { + match commands { AccountCommands::List => { let accounts = config.load_accounts(); @@ -376,8 +431,7 @@ fn main() { let accounts = config.load_accounts(); if let Some(_) = accounts.iter().find(|a| a.id == account.id) { - eprintln!("Error: account {} exists.", account.id); - return; + panic!("Error: account {} exists.", account.id); } let accounts_path = config.full_accounts_path(); @@ -397,8 +451,7 @@ fn main() { let account = match accounts.iter_mut().find(|a| a.id == args.id) { Some(acc) => acc, None => { - eprintln!("Could not find account \"{}\"", args.id); - return; + panic!("Could not find account \"{}\"", args.id); } }; @@ -414,8 +467,7 @@ fn main() { if args.negative_ok { account.balance = 0u32; } else { - eprintln!("Error: balance ({}) is less than {}.", account.balance, args.value); - return; + panic!("Error: balance ({}) is less than {}.", account.balance, args.value); } } else { account.balance -= args.value; @@ -440,16 +492,14 @@ fn main() { let account = match accounts.iter_mut().find(|a| a.id == args.account) { Some(acc) => acc, None => { - eprintln!("Could not find account \"{}\"", args.account); - return; + panic!("Could not find account \"{}\"", args.account); } }; let quests = config.load_quests(); if let None = quests.iter().find(|q| q.id == args.quest) { - eprintln!("Could not find quest #{}", args.quest); - return; + panic!("Could not find quest #{}", args.quest); } match account.quests_completed.iter().find(|qid| **qid == args.quest) { @@ -483,6 +533,101 @@ fn main() { } }, } + }, + Objects::Map(commands) => { + let map_path = config.full_map_path(); + let mut map = Map::load(map_path.clone())?; + map.room.sort_by(|a,b| a.id.cmp(&b.id)); + match commands { + MapCommands::List => { + for room in map.room { + println!("Room #{}: {}; Connections: {:?}", room.id, room.name, room.children); + } + }, + MapCommands::Add(args) => { + let last_id = match map.room.last() { + Some(r) => r.id + 1u16, + None => 0u16 + }; + let room = Room { + id: last_id, + name: args.name.clone(), + description: args.description.clone(), + ..Default::default() + }; + let r_id = room.id; + map.room.push(room); + match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { + Ok(_) => { + println!("Created room #{}.", r_id); + println!("Successfully saved map."); + }, + Err(error) => { + eprintln!("Error while saving map: {error}"); + } + } + }, + MapCommands::Delete(args) => { + let Some(room) = map.room.iter().find(|r| r.id == args.id) else { + panic!("Error: Room #{} not found", args.id); + }; + + let r_id = room.id; + let index = map.room.iter().position(|r| r.eq(room)).unwrap(); + map.room.remove(index); + + for room in map.room.iter_mut().filter(|r| r.children.contains(&r_id)) { + let idx = room.children.iter() + .position(|id| *id == r_id) + .unwrap(); + room.children.remove(idx); + } + + match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { + Ok(_) => { + println!("Removed room #{}.", r_id); + println!("Successfully saved map."); + }, + Err(error) => { + eprintln!("Error while saving map: {error}"); + } + } + }, + MapCommands::Update(args) => { + let Some(room) = map.room.iter_mut().find(|r| r.id == args.id) else { + panic!("Error: Room #{} not found", args.id); + }; + + if let Some(name) = &args.name { + room.name = name.to_string(); + } + + if args.description.is_some() { + room.description = args.description.clone(); + } + + if let Some(value) = args.value { + room.value = value; + } + + match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { + Ok(_) => { + println!("Updated room #{}.", args.id); + println!("Successfully saved map."); + }, + Err(error) => { + eprintln!("Error while saving map: {error}"); + } + } + }, + MapCommands::Connect(_) => { + todo!(); + }, + MapCommands::Disconnect(_) => { + todo!(); + } + } } } + Ok(()) } diff --git a/src/config/mod.rs b/src/config/mod.rs index d5a86da..b47b565 100644 --- a/src/config/mod.rs +++ b/src/config/mod.rs @@ -171,7 +171,7 @@ impl Config { out_vec } - /// Returns full path to quests folder + /// Returns full path to accounts folder /// This path will be relative to $PWD, not to config. /// /// # Examples @@ -237,4 +237,29 @@ impl Config { out_vec } + + /// Returns full path to map.toml + /// This path will be relative to $PWD, not to config. + /// + /// # Examples + /// ```rust + /// use squad_quest::{config::Config,error::Error,map::Map,SquadObject}; + /// # fn main() { + /// # let _ = wrapper(); + /// # } + /// # fn wrapper() -> Result<(),Error> { + /// + /// let path = "cfg/config.toml".into(); + /// let config = Config::load(path); + /// + /// let map_path = config.full_map_path(); + /// let map = Map::load(map_path)?; + /// # Ok(()) + /// # } + /// ``` + pub fn full_map_path(&self) -> PathBuf { + let mut path = self.path.clone(); + path.push(self.map.clone()); + path + } } diff --git a/src/map/mod.rs b/src/map/mod.rs index d4eb205..17e42ce 100644 --- a/src/map/mod.rs +++ b/src/map/mod.rs @@ -1,5 +1,107 @@ //! Map, a.k.a. a graph of rooms -#![allow(dead_code)] -/// Graph for room nodes -pub struct Map; +use std::{fs, io::Write, path::PathBuf}; + +use serde::{Deserialize, Serialize}; + +use crate::{SquadObject, error::Error}; + +/// THE Graph. Actually, this is a Vec. +#[derive(Serialize, Deserialize)] +#[serde(default)] +pub struct Map { + /// Rooms go here + pub room: Vec +} + +impl Default for Map { + fn default() -> Self { + Map { room: Vec::new() } + } +} + +impl SquadObject for Map { + fn load(path: PathBuf) -> Result { + match std::fs::read_to_string(path) { + Ok(string) => { + match toml::from_str::(&string) { + Ok(object) => Ok(object), + Err(error) => Err(Error::TomlDeserializeError(error)) + } + }, + Err(error) => Err(Error::IoError(error)) + } + } + + fn delete(path: PathBuf) -> Result<(), Error> { + match Self::load(path.clone()) { + Ok(_) => { + if let Err(error) = fs::remove_file(path) { + return Err(Error::IoError(error)); + } + + Ok(()) + }, + Err(error) => Err(error) + } + } + + fn save(&self, path: PathBuf) -> Result<(), Error> { + let filename = "map.toml".to_string(); + let mut full_path = path; + full_path.push(filename); + + let str = match toml::to_string_pretty(&self) { + Ok(string) => string, + Err(error) => { + return Err(Error::TomlSerializeError(error)); + } + }; + + let mut file = match fs::File::create(full_path) { + Ok(f) => f, + Err(error) => { + return Err(Error::IoError(error)); + } + }; + + if let Err(error) = file.write_all(str.as_bytes()) { + return Err(Error::IoError(error)); + } + + Ok(()) + } + +} + +/// Component of the map +#[derive(Serialize, Deserialize, PartialEq, Debug)] +#[serde(default)] +pub struct Room { + /// Room id + pub id: u16, + /// Rooms that are connected with this + pub children: Vec, + /// Price of the room + pub value: u32, + /// Room name + pub name: String, + /// Room description + pub description: Option, +} + +fn default_name() -> String { + "Hall".to_string() +} + +impl Default for Room { + fn default() -> Self { + Room { + id: u16::default(), + children: Vec::new(), + value: u32::default(), + name: default_name(), + description: None, + } + } +} diff --git a/tests/io/map.toml b/tests/io/map.toml new file mode 100644 index 0000000..8b13789 --- /dev/null +++ b/tests/io/map.toml @@ -0,0 +1 @@ + diff --git a/tests/main.rs b/tests/main.rs index b582e5c..b2d054d 100644 --- a/tests/main.rs +++ b/tests/main.rs @@ -1,4 +1,4 @@ -use squad_quest::{account::Account, config::Config, quest::Quest}; +use squad_quest::{SquadObject, account::Account, config::Config, map::{Map, Room}, quest::Quest}; static CONFIG_PATH: &str = "./tests/main/config.toml"; @@ -81,3 +81,50 @@ fn account_test() { assert_eq!(*account, expected); } + +#[test] +fn load_map() { + let config = Config::load(CONFIG_PATH.into()); + + let room0 = Room { + id: 0, + children: vec![1, 2], + value: 0, + name: "Entrance".to_string(), + description: Some("Enter the dungeon".to_string()), + }; + + let room1 = Room { + id: 1, + children: vec![0, 3], + value: 100, + name: "Kitchen hall".to_string(), + description: None, + }; + + let room2 = Room { + id: 2, + children: vec![0], + value: 250, + name: "Room".to_string(), + description: Some("Simple room with no furniture".to_string()), + }; + + let room3 = Room { + id: 3, + children: vec![1], + value: 175, + name: "Kitchen".to_string(), + description: Some("Knives are stored here".to_string()), + }; + + let expected = Map { + room: vec![room0, room1, room2, room3], + }; + + let map_path = config.full_map_path(); + + let map = Map::load(map_path).unwrap(); + + assert_eq!(map.room, expected.room); +} diff --git a/tests/main/map.toml b/tests/main/map.toml new file mode 100644 index 0000000..018fb2e --- /dev/null +++ b/tests/main/map.toml @@ -0,0 +1,26 @@ +[[room]] +id = 0 +children = [ 1, 2 ] +value = 0 +name = "Entrance" +description = "Enter the dungeon" + +[[room]] +id = 1 +children = [ 0, 3 ] +value = 100 +name = "Kitchen hall" + +[[room]] +id = 2 +children = [ 0 ] +value = 250 +name = "Room" +description = "Simple room with no furniture" + +[[room]] +id = 3 +children = [ 1 ] +value = 175 +name = "Kitchen" +description = "Knives are stored here" From 5d6aa0422dd7a1a8ad3602fd30d0e81cdd47196d Mon Sep 17 00:00:00 2001 From: 2ndbeam <2ndbeam@disroot.org> Date: Thu, 4 Dec 2025 12:54:22 +0300 Subject: [PATCH 3/7] feat: Added CLI rooms (dis-)connect functionality --- src/bin/cli.rs | 56 +++++++++++++++++++++++++++++++++++++++++++---- tests/io/map.toml | 2 +- 2 files changed, 53 insertions(+), 5 deletions(-) diff --git a/src/bin/cli.rs b/src/bin/cli.rs index 19bbf4e..ff2bac7 100644 --- a/src/bin/cli.rs +++ b/src/bin/cli.rs @@ -620,11 +620,59 @@ fn main() -> Result<(), Error> { } } }, - MapCommands::Connect(_) => { - todo!(); + MapCommands::Connect(args) => { + // We iterate twice to make references first->second and second->first + for (first, second) in [(args.first, args.second),(args.second, args.first)] { + let Some(room) = map.room.iter_mut().find(|r| r.id == first) else { + panic!("Error: Room #{} not found", first); + }; + + match room.children.iter().find(|id| **id == second) { + Some(_) => { + println!("Room #{} already has reference to #{}", first, second); + }, + None => { + room.children.push(second); + } + } + } + + match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { + Ok(_) => { + println!("Connected rooms #{} <-> #{}.", args.first, args.second); + println!("Successfully saved map."); + }, + Err(error) => { + eprintln!("Error while saving map: {error}"); + } + } }, - MapCommands::Disconnect(_) => { - todo!(); + MapCommands::Disconnect(args) => { + // We iterate twice to make references first->second and second->first + for (first, second) in [(args.first, args.second),(args.second, args.first)] { + let Some(room) = map.room.iter_mut().find(|r| r.id == first) else { + panic!("Error: Room #{} not found", first); + }; + + match room.children.iter().position(|id| *id == second) { + Some(id) => { + room.children.remove(id as usize); + }, + None => { + println!("Room #{} has no reference to #{}", first, second); + } + } + } + + match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { + Ok(_) => { + println!("Disconnected rooms #{} #{}.", args.first, args.second); + println!("Successfully saved map."); + }, + Err(error) => { + eprintln!("Error while saving map: {error}"); + } + } } } } diff --git a/tests/io/map.toml b/tests/io/map.toml index 8b13789..38dffa5 100644 --- a/tests/io/map.toml +++ b/tests/io/map.toml @@ -1 +1 @@ - +room = [] From 47f55105ddae1d3df2f373e25aabd2b6ba7e84a7 Mon Sep 17 00:00:00 2001 From: 2ndbeam <2ndbeam@disroot.org> Date: Thu, 4 Dec 2025 13:56:53 +0300 Subject: [PATCH 4/7] refactor!: Moved CLI to inner package - Increased version to 0.3.0 - Repository now is a Cargo workspace - Added squad-quest-cli package - Removed CLI-specific dependencies from main crate - Removed bin target from main package --- .gitignore | 1 + Cargo.lock | 12 ++++++++++-- Cargo.toml | 18 +++++++++++++----- cli/Cargo.toml | 13 +++++++++++++ src/bin/cli.rs => cli/src/main.rs | 0 5 files changed, 37 insertions(+), 7 deletions(-) create mode 100644 cli/Cargo.toml rename src/bin/cli.rs => cli/src/main.rs (100%) diff --git a/.gitignore b/.gitignore index ea8c4bf..f20fcf9 100644 --- a/.gitignore +++ b/.gitignore @@ -1 +1,2 @@ /target +/cli/target diff --git a/Cargo.lock b/Cargo.lock index 1c88b30..d263382 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -332,12 +332,20 @@ checksum = "0fda2ff0d084019ba4d7c6f371c95d8fd75ce3524c3cb8fb653a3023f6323e64" [[package]] name = "squad-quest" -version = "0.2.0" +version = "0.3.0" +dependencies = [ + "serde", + "toml", +] + +[[package]] +name = "squad-quest-cli" +version = "0.3.0" dependencies = [ "chrono", "clap", - "clap_derive", "serde", + "squad-quest", "toml", ] diff --git a/Cargo.toml b/Cargo.toml index f912a91..669dcea 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -1,11 +1,19 @@ +[workspace] +members = ["cli"] + +[workspace.package] +version = "0.3.0" +edition = "2024" +repository = "https://2ndbeam.ru/git/2ndbeam/squad-quest" +license = "MIT" + [package] name = "squad-quest" -version = "0.2.0" -edition = "2024" +edition.workspace = true +version.workspace = true +repository.workspace = true +license.workspace = true [dependencies] -chrono = "0.4.42" -clap = { version = "4.5.53", features = ["derive"] } -clap_derive = "4.5.49" serde = { version = "1.0.228", features = ["derive"] } toml = "0.9.8" diff --git a/cli/Cargo.toml b/cli/Cargo.toml new file mode 100644 index 0000000..2612f50 --- /dev/null +++ b/cli/Cargo.toml @@ -0,0 +1,13 @@ +[package] +name = "squad-quest-cli" +version.workspace = true +edition.workspace = true +repository.workspace = true +license.workspace = true + +[dependencies] +chrono = "0.4.42" +clap = { version = "4.5.53", features = ["derive"] } +serde = { version = "1.0.228", features = ["derive"] } +squad-quest = { version = "0.3.0", path = ".." } +toml = "0.9.8" diff --git a/src/bin/cli.rs b/cli/src/main.rs similarity index 100% rename from src/bin/cli.rs rename to cli/src/main.rs From 790fa88fe3b034c7823ce62f71768bfccc2a3491 Mon Sep 17 00:00:00 2001 From: 2ndbeam <2ndbeam@disroot.org> Date: Thu, 4 Dec 2025 17:37:01 +0300 Subject: [PATCH 5/7] refactor(cli)!: Moved CLI stuff to crate::cli - Bump version to 0.4.0 - Added Config::try_load - Added Config.verbose field - Made Config.path public - Added -q/--quiet flag to CLI BREAKING CHANGE: Moved CLI-related objects to squad-quest-cli::cli --- Cargo.lock | 4 +- Cargo.toml | 2 +- cli/Cargo.toml | 2 +- cli/src/cli/account.rs | 57 ++++ cli/src/cli/map.rs | 57 ++++ cli/src/cli/mod.rs | 36 +++ cli/src/cli/quest.rs | 130 +++++++++ cli/src/lib.rs | 1 + cli/src/main.rs | 582 ++++++++++++----------------------------- src/config/mod.rs | 111 ++++++-- 10 files changed, 529 insertions(+), 453 deletions(-) create mode 100644 cli/src/cli/account.rs create mode 100644 cli/src/cli/map.rs create mode 100644 cli/src/cli/mod.rs create mode 100644 cli/src/cli/quest.rs create mode 100644 cli/src/lib.rs diff --git a/Cargo.lock b/Cargo.lock index d263382..3d80605 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -332,7 +332,7 @@ checksum = "0fda2ff0d084019ba4d7c6f371c95d8fd75ce3524c3cb8fb653a3023f6323e64" [[package]] name = "squad-quest" -version = "0.3.0" +version = "0.4.0" dependencies = [ "serde", "toml", @@ -340,7 +340,7 @@ dependencies = [ [[package]] name = "squad-quest-cli" -version = "0.3.0" +version = "0.4.0" dependencies = [ "chrono", "clap", diff --git a/Cargo.toml b/Cargo.toml index 669dcea..e66a59f 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -2,7 +2,7 @@ members = ["cli"] [workspace.package] -version = "0.3.0" +version = "0.4.0" edition = "2024" repository = "https://2ndbeam.ru/git/2ndbeam/squad-quest" license = "MIT" diff --git a/cli/Cargo.toml b/cli/Cargo.toml index 2612f50..ca88ec5 100644 --- a/cli/Cargo.toml +++ b/cli/Cargo.toml @@ -9,5 +9,5 @@ license.workspace = true chrono = "0.4.42" clap = { version = "4.5.53", features = ["derive"] } serde = { version = "1.0.228", features = ["derive"] } -squad-quest = { version = "0.3.0", path = ".." } +squad-quest = { version = "0.4.0", path = ".." } toml = "0.9.8" diff --git a/cli/src/cli/account.rs b/cli/src/cli/account.rs new file mode 100644 index 0000000..c6da1ac --- /dev/null +++ b/cli/src/cli/account.rs @@ -0,0 +1,57 @@ +use clap::{Args,Subcommand,ValueEnum}; + +#[derive(Subcommand)] +pub enum AccountCommands { + /// List accounts + List, + /// Create empty account + Create(AccountCreateArgs), + /// Update balance value + Balance(AccountBalanceArgs), + /// Approve account answer for quest + Complete(AccountCompleteArgs), + /// Delete account + Delete(AccountDeleteArgs), +} + +#[derive(Args)] +pub struct AccountCreateArgs { + /// Account will be created with this id + pub id: String, +} + +#[derive(Clone, Copy, PartialEq, Eq, ValueEnum)] +pub enum AccountBalanceActions { + Set, + Add, + Remove, +} + +#[derive(Args)] +pub struct AccountBalanceArgs { + /// Account id + pub id: String, + /// What to do with the balance + #[arg(value_enum)] + pub action: AccountBalanceActions, + /// Amount of doing + pub value: u32, + /// If action is remove, set balance to 0 if the result is negative instead of returning error + #[arg(short,long)] + pub negative_ok: bool, +} + +#[derive(Args)] +pub struct AccountCompleteArgs { + /// Id of the account + pub account: String, + /// Id of the quest + pub quest: u16, +} + +#[derive(Args)] +pub struct AccountDeleteArgs { + /// Id of the account to delete + pub id: String, +} + diff --git a/cli/src/cli/map.rs b/cli/src/cli/map.rs new file mode 100644 index 0000000..c98a158 --- /dev/null +++ b/cli/src/cli/map.rs @@ -0,0 +1,57 @@ +use clap::{Args,Subcommand}; + +#[derive(Subcommand)] +pub enum MapCommands { + /// List all rooms with connections + List, + /// Add new room to map + Add(MapAddArgs), + /// Connect two rooms + Connect(MapConnectArgs), + /// Disconnect two rooms if they're connected + Disconnect(MapConnectArgs), + /// Remove all connections with the room + Delete(MapDeleteArgs), + /// Update room data + Update(MapUpdateArgs), +} + +#[derive(Args)] +pub struct MapAddArgs { + /// Name of the room + pub name: String, + /// Price of the room + pub value: u32, + /// Optional description for the room + #[arg(long,short)] + pub description: Option, +} + +#[derive(Args)] +pub struct MapConnectArgs { + /// First room ID + pub first: u16, + /// Second room ID + pub second: u16, +} + +#[derive(Args)] +pub struct MapDeleteArgs { + /// ID of the room to delete + pub id: u16, +} + +#[derive(Args)] +pub struct MapUpdateArgs { + /// ID of the room to update + pub id: u16, + /// Room name + #[arg(short,long)] + pub name: Option, + /// Room description + #[arg(short,long)] + pub description: Option, + /// Room price + #[arg(short,long)] + pub value: Option, +} diff --git a/cli/src/cli/mod.rs b/cli/src/cli/mod.rs new file mode 100644 index 0000000..e44c361 --- /dev/null +++ b/cli/src/cli/mod.rs @@ -0,0 +1,36 @@ +use std::path::PathBuf; + +use clap::{Parser,Subcommand}; + +pub mod account; +pub mod map; +pub mod quest; + +#[derive(Parser)] +#[command(version, about, long_about = None)] +#[command(propagate_version = true)] +pub struct Cli { + /// Path to config + #[arg(short, long)] + pub config: PathBuf, + /// Object to make operation on + #[command(subcommand)] + pub command: Objects, + /// Suppress most output + #[arg(short, long)] + pub quiet: bool, +} + +#[derive(Subcommand)] +pub enum Objects { + /// Operations on the quests + #[command(subcommand)] + Quest(quest::QuestCommands), + /// Operations on the accounts + #[command(subcommand)] + Account(account::AccountCommands), + /// Operations on the map rooms + #[command(subcommand)] + Map(map::MapCommands), +} + diff --git a/cli/src/cli/quest.rs b/cli/src/cli/quest.rs new file mode 100644 index 0000000..41a7820 --- /dev/null +++ b/cli/src/cli/quest.rs @@ -0,0 +1,130 @@ +use squad_quest::quest::QuestDifficulty as LibQuestDifficulty; +use toml::value::Date; +use serde::Deserialize; +use clap::{Args,Subcommand,ValueEnum}; + +#[derive(Deserialize)] +struct DateWrapper { + date: Date, +} + +fn parse_date(arg: &str) -> Result { + let toml_str = format!("date = {arg}"); + let wrapper: DateWrapper = toml::from_str(&toml_str)?; + Ok(wrapper.date) +} + +#[derive(Copy, Clone, PartialEq, Eq, PartialOrd, Ord, ValueEnum)] +pub enum QuestDifficulty { + /// Easy quest + Easy, + /// Normal quest + Normal, + /// Hard quest + Hard, + /// Special case of hard quests. + Secret, +} + +impl From for LibQuestDifficulty { + fn from(value: QuestDifficulty) -> Self { + match value { + QuestDifficulty::Easy => LibQuestDifficulty::Easy, + QuestDifficulty::Normal => LibQuestDifficulty::Normal, + QuestDifficulty::Hard => LibQuestDifficulty::Hard, + QuestDifficulty::Secret => LibQuestDifficulty::Secret, + } + } +} + +#[derive(Subcommand)] +pub enum QuestCommands { + /// List available quests + List(QuestListArgs), + /// Create new quest and automatically assign it id + Create(QuestCreateArgs), + /// Update existing quest + Update(QuestUpdateArgs), + /// Delete quest + Delete(QuestDeleteArgs), + /// Make certain quests public + Daily, + /// Publish quest with specified id + Publish(QuestPublishArgs), +} + + +#[derive(Args)] +pub struct QuestListArgs { + /// Only list id and name of the quest + #[arg(short, long)] + pub short: bool, +} + +#[derive(Args)] +pub struct QuestCreateArgs { + /// Difficulty of the quest #[arg(value_enum)] + pub difficulty: QuestDifficulty, + /// Reward for the quest + pub reward: u32, + /// Name of the quest + pub name: String, + /// Visible description of the quest + pub description: String, + /// Answer for the quest for admins + pub answer: String, + /// Create quest and make it public immediately + #[arg(short,long)] + pub public: bool, + /// Make quest available on date (format = YYYY-MM-DD, ex. 2025-12-24) + #[arg(short,long,value_parser = parse_date)] + pub available: Option, + /// Quest expiration date (format = YYYY-MM-DD, ex. 2025-12-24) + #[arg(short,long,value_parser = parse_date)] + pub deadline: Option, +} + +#[derive(Args)] +pub struct QuestUpdateArgs { + /// Id of the quest to update + pub id: u16, + /// Difficulty of the quest + #[arg(value_enum,long)] + pub difficulty: Option, + /// Reward for the quest + #[arg(long)] + pub reward: Option, + /// Name of the quest + #[arg(long)] + pub name: Option, + /// Visible description of the quest + #[arg(long)] + pub description: Option, + /// Answer for the quest for admins + #[arg(long)] + pub answer: Option, + /// Create quest and make it public immediately + #[arg(long)] + pub public: Option, + /// Make quest available on date (format = YYYY-MM-DD, ex. 2025-12-24) + #[arg(long,value_parser = parse_date)] + pub available: Option, + /// Quest expiration date (format = YYYY-MM-DD, ex. 2025-12-24) + #[arg(long,value_parser = parse_date)] + pub deadline: Option, +} + +#[derive(Args)] +pub struct QuestDeleteArgs { + /// Id of the quest to delete + pub id: u16, +} + +#[derive(Args)] +pub struct QuestPublishArgs { + /// Id of the quest to publish + pub id: u16, + /// Make it non-public instead + #[arg(long,short)] + pub reverse: bool, +} diff --git a/cli/src/lib.rs b/cli/src/lib.rs new file mode 100644 index 0000000..4f77372 --- /dev/null +++ b/cli/src/lib.rs @@ -0,0 +1 @@ +pub mod cli; diff --git a/cli/src/main.rs b/cli/src/main.rs index ff2bac7..cd2e5f6 100644 --- a/cli/src/main.rs +++ b/cli/src/main.rs @@ -1,272 +1,11 @@ -use std::path::{Path, PathBuf}; +use std::path::Path; -use clap::{Parser,Subcommand,Args,ValueEnum}; -use serde::Deserialize; -use squad_quest::{SquadObject, account::Account, config::Config, error::Error, map::{Map, Room}, quest::{Quest,QuestDifficulty as LibQuestDifficulty}}; +use clap::Parser; +use squad_quest_cli::cli::{Cli,Objects,account::*,map::*,quest::*}; +use squad_quest::{SquadObject, account::Account, config::Config, map::{Map, Room}, quest::Quest}; use toml::value::Date; use chrono::{Datelike, NaiveDate, Utc}; -#[derive(Deserialize)] -struct DateWrapper { - date: Date, -} - -fn parse_date(arg: &str) -> Result { - let toml_str = format!("date = {arg}"); - let wrapper: DateWrapper = toml::from_str(&toml_str)?; - Ok(wrapper.date) -} - -#[derive(Parser)] -#[command(version, about, long_about = None)] -#[command(propagate_version = true)] -struct Cli { - /// Path to config - #[arg(short, long)] - config: PathBuf, - /// Object to make operation on - #[command(subcommand)] - command: Objects, -} - -#[derive(Subcommand)] -enum Objects { - /// Operations on the quests - #[command(subcommand)] - Quest(QuestCommands), - /// Operations on the accounts - #[command(subcommand)] - Account(AccountCommands), - /// Operations on the map rooms - #[command(subcommand)] - Map(MapCommands), -} - -#[derive(Copy, Clone, PartialEq, Eq, PartialOrd, Ord, ValueEnum)] -enum QuestDifficulty { - /// Easy quest - Easy, - /// Normal quest - Normal, - /// Hard quest - Hard, - /// Special case of hard quests. - Secret, -} - -impl From for LibQuestDifficulty { - fn from(value: QuestDifficulty) -> Self { - match value { - QuestDifficulty::Easy => LibQuestDifficulty::Easy, - QuestDifficulty::Normal => LibQuestDifficulty::Normal, - QuestDifficulty::Hard => LibQuestDifficulty::Hard, - QuestDifficulty::Secret => LibQuestDifficulty::Secret, - } - } -} - -#[derive(Subcommand)] -enum QuestCommands { - /// List available quests - List(QuestListArgs), - /// Create new quest and automatically assign it id - Create(QuestCreateArgs), - /// Update existing quest - Update(QuestUpdateArgs), - /// Delete quest - Delete(QuestDeleteArgs), - /// Make certain quests public - Daily, - /// Publish quest with specified id - Publish(QuestPublishArgs), -} - - -#[derive(Args)] -struct QuestListArgs { - /// Only list id and name of the quest - #[arg(short, long)] - short: bool, -} - -#[derive(Args)] -struct QuestCreateArgs { /// Difficulty of the quest #[arg(value_enum)] - difficulty: QuestDifficulty, - /// Reward for the quest - reward: u32, - /// Name of the quest - name: String, - /// Visible description of the quest - description: String, - /// Answer for the quest for admins - answer: String, - /// Create quest and make it public immediately - #[arg(short,long)] - public: bool, - /// Make quest available on date (format = YYYY-MM-DD, ex. 2025-12-24) - #[arg(short,long,value_parser = parse_date)] - available: Option, - /// Quest expiration date (format = YYYY-MM-DD, ex. 2025-12-24) - #[arg(short,long,value_parser = parse_date)] - deadline: Option, -} - -#[derive(Args)] -struct QuestUpdateArgs { - /// Id of the quest to update - id: u16, - /// Difficulty of the quest - #[arg(value_enum,long)] - difficulty: Option, - /// Reward for the quest - #[arg(long)] - reward: Option, - /// Name of the quest - #[arg(long)] - name: Option, - /// Visible description of the quest - #[arg(long)] - description: Option, - /// Answer for the quest for admins - #[arg(long)] - answer: Option, - /// Create quest and make it public immediately - #[arg(long)] - public: Option, - /// Make quest available on date (format = YYYY-MM-DD, ex. 2025-12-24) - #[arg(long,value_parser = parse_date)] - available: Option, - /// Quest expiration date (format = YYYY-MM-DD, ex. 2025-12-24) - #[arg(long,value_parser = parse_date)] - deadline: Option, -} - -#[derive(Args)] -struct QuestDeleteArgs { - /// Id of the quest to delete - id: u16, -} - -#[derive(Args)] -struct QuestPublishArgs { - /// Id of the quest to publish - id: u16, - /// Make it non-public instead - #[arg(long,short)] - reverse: bool, -} - -#[derive(Subcommand)] -enum AccountCommands { - /// List accounts - List, - /// Create empty account - Create(AccountCreateArgs), - /// Update balance value - Balance(AccountBalanceArgs), - /// Approve account answer for quest - Complete(AccountCompleteArgs), - /// Delete account - Delete(AccountDeleteArgs), -} - -#[derive(Args)] -struct AccountCreateArgs { - /// Account will be created with this id - id: String, -} - -#[derive(Clone, Copy, PartialEq, Eq, ValueEnum)] -enum AccountBalanceActions { - Set, - Add, - Remove, -} - -#[derive(Args)] -struct AccountBalanceArgs { - /// Account id - id: String, - /// What to do with the balance - #[arg(value_enum)] - action: AccountBalanceActions, - /// Amount of doing - value: u32, - /// If action is remove, set balance to 0 if the result is negative instead of returning error - #[arg(short,long)] - negative_ok: bool, -} - -#[derive(Args)] -struct AccountCompleteArgs { - /// Id of the account - account: String, - /// Id of the quest - quest: u16, -} - -#[derive(Args)] -struct AccountDeleteArgs { - /// Id of the account to delete - id: String, -} - -#[derive(Subcommand)] -enum MapCommands { - /// List all rooms with connections - List, - /// Add new room to map - Add(MapAddArgs), - /// Connect two rooms - Connect(MapConnectArgs), - /// Disconnect two rooms if they're connected - Disconnect(MapConnectArgs), - /// Remove all connections with the room - Delete(MapDeleteArgs), - /// Update room data - Update(MapUpdateArgs), -} - -#[derive(Args)] -struct MapAddArgs { - /// Name of the room - name: String, - /// Price of the room - value: u32, - /// Optional description for the room - #[arg(long,short)] - description: Option, -} - -#[derive(Args)] -struct MapConnectArgs { - /// First room ID - first: u16, - /// Second room ID - second: u16, -} - -#[derive(Args)] -struct MapDeleteArgs { - /// ID of the room to delete - id: u16, -} - -#[derive(Args)] -struct MapUpdateArgs { - /// ID of the room to update - id: u16, - /// Room name - #[arg(short,long)] - name: Option, - /// Room description - #[arg(short,long)] - description: Option, - /// Room price - #[arg(short,long)] - value: Option, -} - fn print_quest_short(quest: &Quest) { println!("Quest #{}: {}", quest.id, quest.name); } @@ -278,16 +17,36 @@ fn print_quest_long(quest: &Quest) { println!("Answer:\n{}", quest.answer); } -fn main() -> Result<(), Error> { +fn main() { let cli = Cli::parse(); - let config = Config::load(cli.config.clone()); + let config = match cli.quiet { + false => Config::load(cli.config.clone()), + true => { + match Config::try_load(cli.config.clone()) { + Ok(mut config) => { + config.verbose = false; + config + }, + Err(_) => { + let path = cli.config.clone().parent().unwrap_or(&Path::new(".")).to_owned(); + Config { + verbose: false, + path, + ..Default::default() + } + } + } + }, + }; match &cli.command { Objects::Quest(commands) => { + let mut quests = config.load_quests(); + let mut path = config.full_quests_path(); + match commands { QuestCommands::List(args) => { - let quests = config.load_quests(); for quest in quests { if args.short { print_quest_short(&quest); @@ -297,24 +56,27 @@ fn main() -> Result<(), Error> { } }, QuestCommands::Create(args) => { - let mut quests = config.load_quests(); quests.sort_by(|a,b| a.id.cmp(&b.id)); let next_id = match quests.last() { Some(quest) => quest.id + 1u16, None => 0u16 }; - let path = config.full_quests_path(); - let mut quest_path = path.clone(); - quest_path.push(format!("{next_id}.toml")); - match std::fs::exists(&quest_path) { + path.push(format!("{next_id}.toml")); + match std::fs::exists(&path) { Ok(exists) => { if exists { - panic!("Error: {:?} is not empty.", quest_path); + if !cli.quiet { + eprintln!("Error: {:?} is not empty.", path); + } + return; } }, Err(error) => { - panic!("Error while retrieving {:?}: {}.", quest_path, error); + if !cli.quiet { + eprintln!("Error while retrieving {:?}: {}.", path, error); + } + return; } } @@ -329,16 +91,19 @@ fn main() -> Result<(), Error> { available_on: args.available.clone(), deadline: args.deadline.clone() }; - if let Err(error) = quest.save(path) { - eprintln!("Error while saving quest: {error}."); - } else { - println!("Successfully saved quest #{}.", quest.id); + + match quest.save(path) { + Ok(_) if !cli.quiet => println!("Successfully saved quest #{}", quest.id), + Err(error) if !cli.quiet => eprintln!("Error while saving quest: {error}"), + _ => {}, } }, QuestCommands::Update(args) => { - let quests = config.load_quests(); let Some(quest) = quests.iter().find(|q| q.id == args.id) else { - panic!("Error: Quest #{} not found.", args.id); + if !cli.quiet { + eprintln!("Error: Quest #{} not found.", args.id); + } + return; }; let quest = Quest { id: args.id, @@ -354,22 +119,20 @@ fn main() -> Result<(), Error> { available_on: args.available.clone().or(quest.available_on.clone()), deadline: args.deadline.clone().or(quest.deadline.clone()) }; - let path = config.full_quests_path(); match quest.save(path) { - Ok(_) => println!("Updated quest #{}", quest.id), - Err(error) => eprintln!("Error while updating quest: {error}") + Ok(_) if !cli.quiet => println!("Updated quest #{}", quest.id), + Err(error) if !cli.quiet => eprintln!("Error while updating quest: {error}"), + _ => {}, } }, QuestCommands::Delete(args) => { - let mut path = config.full_quests_path(); - path.push(format!("{}.toml", args.id)); match Quest::delete(path) { - Ok(_) => println!("Successfully deleted quest #{}", args.id), - Err(error) => eprintln!("Error deleting quest #{}: {}", args.id, error), + Ok(_) if !cli.quiet => println!("Successfully deleted quest #{}", args.id), + Err(error) if !cli.quiet => eprintln!("Error deleting quest #{}: {}", args.id, error), + _ => {}, } }, QuestCommands::Daily => { - let mut quests = config.load_quests(); let today: NaiveDate = Utc::now().date_naive(); let toml_today = Date { year: today.year() as u16, @@ -377,46 +140,52 @@ fn main() -> Result<(), Error> { day: today.day() as u8 }; - let path = config.full_quests_path(); - for quest in quests.iter_mut().filter(|q| !q.public && q.available_on.is_some_and(|date| date.le(&toml_today))) { - println!("Quest #{} will be published.", quest.id); quest.public = true; - if let Err(error) = quest.save(path.clone()) { - eprintln!("Error while saving quest: {error}."); + + match quest.save(path.clone()) { + Ok(_) if !cli.quiet => println!("Published quest #{}", quest.id), + Err(error) if !cli.quiet => eprintln!("Error while publishing quest: {error}"), + _ => {}, } } }, QuestCommands::Publish(args) => { - let mut quests = config.load_quests(); let quest = quests.iter_mut().find(|q| q.id == args.id); - let path = config.full_quests_path(); - match quest { Some(quest) => { let not_str = if args.reverse {" not "} else {" "}; if quest.public != args.reverse { - panic!("Quest #{} is already{}public", quest.id, not_str); + if !cli.quiet { + eprintln!("Quest #{} is already{}public", quest.id, not_str); + } + return; } quest.public = !args.reverse; - if let Err(error) = quest.save(path) { - eprintln!("Error while saving quest: {error}."); - }; + + match quest.save(path.clone()) { + Ok(_) if !cli.quiet => println!("Published quest #{}", quest.id), + Err(error) if !cli.quiet => eprintln!("Error while publishing quest: {error}"), + _ => {}, + } }, - None => { + None if !cli.quiet => { eprintln!("Error: couldn't find quest with id {}.", args.id); - } + }, + _ => {}, } }, } }, Objects::Account(commands) => { + let mut accounts = config.load_accounts(); + let mut path = config.full_accounts_path(); + match commands { AccountCommands::List => { - let accounts = config.load_accounts(); for account in accounts { println!("\"{}\": Balance {}", account.id, account.balance); @@ -428,30 +197,27 @@ fn main() -> Result<(), Error> { ..Default::default() }; - let accounts = config.load_accounts(); - if let Some(_) = accounts.iter().find(|a| a.id == account.id) { - panic!("Error: account {} exists.", account.id); + if !cli.quiet { + eprintln!("Error: account {} exists.", account.id); + } + return; } - let accounts_path = config.full_accounts_path(); - - match account.save(accounts_path) { - Ok(_) => { - println!("Successfully created account \"{}\"", account.id); - }, - Err(error) => { - eprintln!("Error while saving account: {error}"); - } + match account.save(path) { + Ok(_) if !cli.quiet => println!("Successfully created account \"{}\"", account.id), + Err(error) if !cli.quiet => eprintln!("Error while saving account: {error}"), + _ => {}, } }, AccountCommands::Balance(args) => { - let mut accounts = config.load_accounts(); - let account = match accounts.iter_mut().find(|a| a.id == args.id) { Some(acc) => acc, None => { - panic!("Could not find account \"{}\"", args.id); + if !cli.quiet { + eprintln!("Could not find account \"{}\"", args.id); + } + return; } }; @@ -467,7 +233,10 @@ fn main() -> Result<(), Error> { if args.negative_ok { account.balance = 0u32; } else { - panic!("Error: balance ({}) is less than {}.", account.balance, args.value); + if !cli.quiet { + eprintln!("Error: balance ({}) is less than {}.", account.balance, args.value); + } + return; } } else { account.balance -= args.value; @@ -475,69 +244,68 @@ fn main() -> Result<(), Error> { } } - let accounts_path = config.full_accounts_path(); - - match account.save(accounts_path) { - Ok(_) => { - println!("Successfully updated account \"{}\" balance.", account.id); - }, - Err(error) => { - eprintln!("Error while saving account: {error}"); - } - }; + match account.save(path) { + Ok(_) if !cli.quiet => println!("Successfully updated account \"{}\" balance", account.id), + Err(error) if !cli.quiet => eprintln!("Error while saving account: {error}"), + _ => {}, + } }, AccountCommands::Complete(args) => { - let mut accounts = config.load_accounts(); - let account = match accounts.iter_mut().find(|a| a.id == args.account) { Some(acc) => acc, None => { - panic!("Could not find account \"{}\"", args.account); + if !cli.quiet { + eprintln!("Could not find account \"{}\"", args.account); + } + return; } }; let quests = config.load_quests(); if let None = quests.iter().find(|q| q.id == args.quest) { - panic!("Could not find quest #{}", args.quest); + if !cli.quiet { + eprintln!("Could not find quest #{}", args.quest); + } + return; } match account.quests_completed.iter().find(|qid| **qid == args.quest) { - Some(_) => { + Some(_) if !cli.quiet => { println!("Quest #{} is already completed on account \"{}\"", args.quest, args.account); }, None => { account.quests_completed.push(args.quest); - let accounts_path = config.full_accounts_path(); - match account.save(accounts_path) { - Ok(_) => { - println!("Account \"{}\" completed quest #{}.", args.account, args.quest); - }, - Err(error) => { - eprintln!("Error while saving account: {error}"); - } + match account.save(path) { + Ok(_) if !cli.quiet => println!("Successfully completed quest #{} on account \"{}\".", args.quest, account.id), + Err(error) if !cli.quiet => eprintln!("Error while saving account: {error}"), + _ => {}, } - } + }, + _ => {}, } }, AccountCommands::Delete(args) => { - let mut accounts_path = config.full_accounts_path(); - accounts_path.push(format!("{}.toml", args.id)); - match Account::delete(accounts_path) { - Ok(_) => { - println!("Successfully deleted account \"{}\".", args.id); - }, - Err(error) => { - eprintln!("Error deleting account: {error}"); - } + path.push(format!("{}.toml", args.id)); + match Account::delete(path) { + Ok(_) if !cli.quiet => println!("Successfully deleted account \"{}\".", args.id), + Err(error) if !cli.quiet => eprintln!("Error while deleting account: {error}"), + _ => {}, } }, } }, Objects::Map(commands) => { let map_path = config.full_map_path(); - let mut map = Map::load(map_path.clone())?; - map.room.sort_by(|a,b| a.id.cmp(&b.id)); + let mut map = match Map::load(map_path.clone()) { + Ok(map) => map, + Err(error) => { + if !cli.quiet { + eprintln!("Error while loading map: {error}"); + } + return; + } + }; match commands { MapCommands::List => { for room in map.room { @@ -545,6 +313,7 @@ fn main() -> Result<(), Error> { } }, MapCommands::Add(args) => { + map.room.sort_by(|a,b| a.id.cmp(&b.id)); let last_id = match map.room.last() { Some(r) => r.id + 1u16, None => 0u16 @@ -558,18 +327,17 @@ fn main() -> Result<(), Error> { let r_id = room.id; map.room.push(room); match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { - Ok(_) => { - println!("Created room #{}.", r_id); - println!("Successfully saved map."); - }, - Err(error) => { - eprintln!("Error while saving map: {error}"); - } + Ok(_) if !cli.quiet => println!("Created room #{}.", r_id), + Err(error) if !cli.quiet => eprintln!("Error while saving map: {error}"), + _ => {}, } }, MapCommands::Delete(args) => { let Some(room) = map.room.iter().find(|r| r.id == args.id) else { - panic!("Error: Room #{} not found", args.id); + if !cli.quiet { + eprintln!("Error: Room #{} not found", args.id); + } + return; }; let r_id = room.id; @@ -584,18 +352,17 @@ fn main() -> Result<(), Error> { } match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { - Ok(_) => { - println!("Removed room #{}.", r_id); - println!("Successfully saved map."); - }, - Err(error) => { - eprintln!("Error while saving map: {error}"); - } + Ok(_) if !cli.quiet => println!("Deleted room #{}.", r_id), + Err(error) if !cli.quiet => eprintln!("Error while saving map: {error}"), + _ => {}, } }, MapCommands::Update(args) => { let Some(room) = map.room.iter_mut().find(|r| r.id == args.id) else { - panic!("Error: Room #{} not found", args.id); + if !cli.quiet { + eprintln!("Error: Room #{} not found", args.id); + } + return; }; if let Some(name) = &args.name { @@ -611,71 +378,44 @@ fn main() -> Result<(), Error> { } match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { - Ok(_) => { - println!("Updated room #{}.", args.id); - println!("Successfully saved map."); - }, - Err(error) => { - eprintln!("Error while saving map: {error}"); - } + Ok(_) if !cli.quiet => println!("Updated room #{}.", args.id), + Err(error) if !cli.quiet => eprintln!("Error while saving map: {error}"), + _ => {}, } }, - MapCommands::Connect(args) => { + MapCommands::Connect(args) | MapCommands::Disconnect(args) => { + let connect = match commands { + MapCommands::Connect(_) => true, + _ => false, + }; + // We iterate twice to make references first->second and second->first for (first, second) in [(args.first, args.second),(args.second, args.first)] { let Some(room) = map.room.iter_mut().find(|r| r.id == first) else { - panic!("Error: Room #{} not found", first); - }; - - match room.children.iter().find(|id| **id == second) { - Some(_) => { - println!("Room #{} already has reference to #{}", first, second); - }, - None => { - room.children.push(second); + if !cli.quiet { + eprintln!("Error: Room #{} not found", first); } - } - } - - match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { - Ok(_) => { - println!("Connected rooms #{} <-> #{}.", args.first, args.second); - println!("Successfully saved map."); - }, - Err(error) => { - eprintln!("Error while saving map: {error}"); - } - } - }, - MapCommands::Disconnect(args) => { - // We iterate twice to make references first->second and second->first - for (first, second) in [(args.first, args.second),(args.second, args.first)] { - let Some(room) = map.room.iter_mut().find(|r| r.id == first) else { - panic!("Error: Room #{} not found", first); + return; }; match room.children.iter().position(|id| *id == second) { - Some(id) => { - room.children.remove(id as usize); - }, - None => { - println!("Room #{} has no reference to #{}", first, second); - } + Some(_) if connect && !cli.quiet => println!("Room #{} already has reference to #{}", first, second), + None if connect => room.children.push(second), + Some(id) if !connect => {room.children.remove(id as usize);}, + None if !connect && !cli.quiet => println!("Room #{} has no reference to #{}", first, second), + _ => {}, } } + let connected = if connect { "Connected" } else { "Disconnected" }; + match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { - Ok(_) => { - println!("Disconnected rooms #{} #{}.", args.first, args.second); - println!("Successfully saved map."); - }, - Err(error) => { - eprintln!("Error while saving map: {error}"); - } + Ok(_) if !cli.quiet => println!("{connected} rooms #{} <-> #{}.", args.first, args.second), + Err(error) if !cli.quiet => eprintln!("Error while saving map: {error}"), + _ => {}, } - } + }, } } } - Ok(()) } diff --git a/src/config/mod.rs b/src/config/mod.rs index b47b565..c805002 100644 --- a/src/config/mod.rs +++ b/src/config/mod.rs @@ -11,7 +11,7 @@ use crate::{SquadObject, account::Account, error::Error, quest::Quest}; pub struct Config { /// Path to config directory #[serde(skip)] - path: PathBuf, + pub path: PathBuf, /// Path to serialized [quests][`crate::quest::Quest`] folder pub quests_path: PathBuf, @@ -20,7 +20,10 @@ pub struct Config { pub accounts_path: PathBuf, /// Path to serialized [map][`crate::map::Map`] file - pub map: PathBuf + pub map: PathBuf, + + /// If true, print to std{out/err} + pub verbose: bool, } impl Default for Config { @@ -29,7 +32,8 @@ impl Default for Config { path: ".".into(), quests_path: "quests".into(), accounts_path: "accounts".into(), - map: "map.toml".into() + map: "map.toml".into(), + verbose: true, } } } @@ -67,8 +71,12 @@ fn handle_account_entry(account_entry: DirEntry) -> Result{ } impl Config { - /// Deserialize config from TOML - /// Logs all errors and returns default config if that happens + /// Deserialize config from TOML. + /// + /// This function wraps [try_load][Config::try_load]. + /// + /// Logs all errors if `config.verbose == true`. + /// Returns default config on error. /// /// # Examples /// ```rust @@ -81,25 +89,62 @@ impl Config { let dir = path.parent() .unwrap_or(Path::new(".")) .to_owned(); + + match Self::try_load(path) { + Ok(conf) => { + if conf.verbose { + println!("Successfully loaded config"); + } + conf + }, + Err(error) => { + let conf = Config { + path: dir, + ..Default::default() + }; + + if conf.verbose { + println!("Error while loading config: {error}"); + } + + conf + } + } + } + + /// Deserialize config from TOML + /// + /// # Examples + /// ```rust + /// use squad_quest::{config::Config,error::Error}; + /// # fn main() { + /// # let _ = wrapper(); + /// # } + /// # fn wrapper() -> Result<(), Error> { + /// let path = "cfg/config.toml".into(); + /// let config = Config::try_load(path)?; + /// # Ok(()) + /// # } + /// ``` + pub fn try_load(path: PathBuf) -> Result { + let dir = path.parent() + .unwrap_or(Path::new(".")) + .to_owned(); + match fs::read_to_string(path) { Ok(string) => { match toml::from_str::(&string) { Ok(mut conf) => { - println!("Successfully loaded config"); conf.path = dir; - conf + Ok(conf) }, Err(error) => { - eprintln!("Error on parsing config: {error}"); - let mut cfg = Config::default(); - cfg.path = dir; - cfg + Err(Error::TomlDeserializeError(error)) } } }, Err(error) => { - eprintln!("Error on reading config path: {error}"); - Config::default() + Err(Error::IoError(error)) } } } @@ -150,23 +195,28 @@ impl Config { Ok(quest_entry) => { match handle_quest_entry(quest_entry) { Ok(quest) => out_vec.push(quest), - Err(error) => { + Err(error) if self.verbose => { eprintln!("Error on loading single quest: {error}"); - } + }, + _ => {}, } }, - Err(error) => { + Err(error) if self.verbose => { eprintln!("Error on loading single quest: {error}"); - } + }, + _ => {}, } } }, - Err(error) => { + Err(error) if self.verbose => { eprintln!("Error on loading quests: {error}"); - } + }, + _ => {}, + } + + if self.verbose { + println!("Loaded {} quests successfully", out_vec.len()); } - - println!("Loaded {} quests successfully", out_vec.len()); out_vec } @@ -217,23 +267,28 @@ impl Config { Ok(acc_entry) => { match handle_account_entry(acc_entry) { Ok(quest) => out_vec.push(quest), - Err(error) => { + Err(error) if self.verbose => { eprintln!("Error on loading single account: {error}"); - } + }, + _ => {}, } }, - Err(error) => { + Err(error) if self.verbose => { eprintln!("Error on loading single account: {error}"); - } + }, + _ => {}, } } }, - Err(error) => { + Err(error) if self.verbose => { eprintln!("Error on loading accounts: {error}"); - } + }, + _ => {}, } - println!("Loaded {} accounts successfully", out_vec.len()); + if self.verbose { + println!("Loaded {} accounts successfully", out_vec.len()); + } out_vec } From f88e010b4f0bf9758f802bf852eb8d4148026e1f Mon Sep 17 00:00:00 2001 From: 2ndbeam <2ndbeam@disroot.org> Date: Fri, 5 Dec 2025 15:38:14 +0300 Subject: [PATCH 6/7] feat: Added interactions with accounts/quests/map - Bump version to 0.5.0 - Added Quest::complete_for_account public function - Added Map::unlock_room_for_account public function - cli: Added "account unlock" command - cli: "account complete" now uses Quest::complete_for_account - cli: refactored logging --- Cargo.lock | 4 +- Cargo.toml | 2 +- cli/Cargo.toml | 2 +- cli/src/cli/account.rs | 9 ++ cli/src/cli/quest.rs | 3 +- cli/src/main.rs | 242 +++++++++++++++++++---------------------- src/error.rs | 38 +++++++ src/map/mod.rs | 38 ++++++- src/quest/mod.rs | 31 +++++- 9 files changed, 231 insertions(+), 138 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index 3d80605..a87f7bb 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -332,7 +332,7 @@ checksum = "0fda2ff0d084019ba4d7c6f371c95d8fd75ce3524c3cb8fb653a3023f6323e64" [[package]] name = "squad-quest" -version = "0.4.0" +version = "0.5.0" dependencies = [ "serde", "toml", @@ -340,7 +340,7 @@ dependencies = [ [[package]] name = "squad-quest-cli" -version = "0.4.0" +version = "0.5.0" dependencies = [ "chrono", "clap", diff --git a/Cargo.toml b/Cargo.toml index e66a59f..d4e8eca 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -2,7 +2,7 @@ members = ["cli"] [workspace.package] -version = "0.4.0" +version = "0.5.0" edition = "2024" repository = "https://2ndbeam.ru/git/2ndbeam/squad-quest" license = "MIT" diff --git a/cli/Cargo.toml b/cli/Cargo.toml index ca88ec5..56fed8f 100644 --- a/cli/Cargo.toml +++ b/cli/Cargo.toml @@ -9,5 +9,5 @@ license.workspace = true chrono = "0.4.42" clap = { version = "4.5.53", features = ["derive"] } serde = { version = "1.0.228", features = ["derive"] } -squad-quest = { version = "0.4.0", path = ".." } +squad-quest = { version = "0.5.0", path = ".." } toml = "0.9.8" diff --git a/cli/src/cli/account.rs b/cli/src/cli/account.rs index c6da1ac..bde7393 100644 --- a/cli/src/cli/account.rs +++ b/cli/src/cli/account.rs @@ -12,6 +12,8 @@ pub enum AccountCommands { Complete(AccountCompleteArgs), /// Delete account Delete(AccountDeleteArgs), + /// Unlock room for account if it has enough balance + Unlock(AccountUnlockArgs), } #[derive(Args)] @@ -55,3 +57,10 @@ pub struct AccountDeleteArgs { pub id: String, } +#[derive(Args)] +pub struct AccountUnlockArgs { + /// Id of the account + pub account: String, + /// Id of the room to unlock + pub room: u16, +} diff --git a/cli/src/cli/quest.rs b/cli/src/cli/quest.rs index 41a7820..1ed541c 100644 --- a/cli/src/cli/quest.rs +++ b/cli/src/cli/quest.rs @@ -63,7 +63,8 @@ pub struct QuestListArgs { #[derive(Args)] pub struct QuestCreateArgs { - /// Difficulty of the quest #[arg(value_enum)] + /// Difficulty of the quest + #[arg(value_enum)] pub difficulty: QuestDifficulty, /// Reward for the quest pub reward: u32, diff --git a/cli/src/main.rs b/cli/src/main.rs index cd2e5f6..31b5b41 100644 --- a/cli/src/main.rs +++ b/cli/src/main.rs @@ -1,8 +1,8 @@ -use std::path::Path; +use std::path::{Path, PathBuf}; use clap::Parser; use squad_quest_cli::cli::{Cli,Objects,account::*,map::*,quest::*}; -use squad_quest::{SquadObject, account::Account, config::Config, map::{Map, Room}, quest::Quest}; +use squad_quest::{SquadObject, account::Account, config::Config, error::Error, map::{Map, Room}, quest::Quest}; use toml::value::Date; use chrono::{Datelike, NaiveDate, Utc}; @@ -17,6 +17,14 @@ fn print_quest_long(quest: &Quest) { println!("Answer:\n{}", quest.answer); } +fn do_and_log(result: Result<(),Error>, log: bool, ok_text: String) { + match result { + Ok(_) if log => println!("{ok_text}"), + Err(error) if log => eprintln!("Error: {error}"), + _ => {}, + } +} + fn main() { let cli = Cli::parse(); @@ -61,20 +69,19 @@ fn main() { Some(quest) => quest.id + 1u16, None => 0u16 }; - - path.push(format!("{next_id}.toml")); - match std::fs::exists(&path) { + + let mut check_path = path.clone(); + check_path.push(format!("{next_id}.toml")); + match std::fs::exists(&check_path) { Ok(exists) => { if exists { - if !cli.quiet { - eprintln!("Error: {:?} is not empty.", path); - } + if !cli.quiet { eprintln!("Error: {:?} is not empty.", path); } return; } }, Err(error) => { if !cli.quiet { - eprintln!("Error while retrieving {:?}: {}.", path, error); + eprintln!("Error: {error}"); } return; } @@ -92,17 +99,11 @@ fn main() { deadline: args.deadline.clone() }; - match quest.save(path) { - Ok(_) if !cli.quiet => println!("Successfully saved quest #{}", quest.id), - Err(error) if !cli.quiet => eprintln!("Error while saving quest: {error}"), - _ => {}, - } + do_and_log(quest.save(path), !cli.quiet, format!("Created quest #{}.", quest.id)); }, QuestCommands::Update(args) => { let Some(quest) = quests.iter().find(|q| q.id == args.id) else { - if !cli.quiet { - eprintln!("Error: Quest #{} not found.", args.id); - } + if !cli.quiet { eprintln!("Error: Quest #{} not found.", args.id); } return; }; let quest = Quest { @@ -119,17 +120,28 @@ fn main() { available_on: args.available.clone().or(quest.available_on.clone()), deadline: args.deadline.clone().or(quest.deadline.clone()) }; - match quest.save(path) { - Ok(_) if !cli.quiet => println!("Updated quest #{}", quest.id), - Err(error) if !cli.quiet => eprintln!("Error while updating quest: {error}"), - _ => {}, - } + + do_and_log(quest.save(path), !cli.quiet, format!("Updated quest #{}.", quest.id)); }, QuestCommands::Delete(args) => { + path.push(format!("{}.toml", args.id)); match Quest::delete(path) { - Ok(_) if !cli.quiet => println!("Successfully deleted quest #{}", args.id), - Err(error) if !cli.quiet => eprintln!("Error deleting quest #{}: {}", args.id, error), - _ => {}, + Ok(_) => { + if !cli.quiet { println!("Deleted quest #{}.", args.id); } + + let mut accounts = config.load_accounts(); + let accounts_path = config.full_accounts_path(); + for account in accounts.iter_mut() { + if let Some(index) = account.quests_completed.iter().position(|qid| *qid == args.id) { + account.quests_completed.remove(index); + do_and_log(account.save(accounts_path.clone()), !cli.quiet, format!("Removed quest #{} from account \"{}\" completed quests", args.id, account.id)); + } + } + }, + Err(error) if !cli.quiet => { + eprintln!("Error: {error}"); + }, + _ => {}, } }, QuestCommands::Daily => { @@ -142,12 +154,7 @@ fn main() { for quest in quests.iter_mut().filter(|q| !q.public && q.available_on.is_some_and(|date| date.le(&toml_today))) { quest.public = true; - - match quest.save(path.clone()) { - Ok(_) if !cli.quiet => println!("Published quest #{}", quest.id), - Err(error) if !cli.quiet => eprintln!("Error while publishing quest: {error}"), - _ => {}, - } + do_and_log(quest.save(path.clone()), !cli.quiet, format!("Published quest #{}.", quest.id)); } }, QuestCommands::Publish(args) => { @@ -158,23 +165,14 @@ fn main() { let not_str = if args.reverse {" not "} else {" "}; if quest.public != args.reverse { - if !cli.quiet { - eprintln!("Quest #{} is already{}public", quest.id, not_str); - } + if !cli.quiet { eprintln!("Error: quest #{} is already{}public.", quest.id, not_str); } return; } quest.public = !args.reverse; - - match quest.save(path.clone()) { - Ok(_) if !cli.quiet => println!("Published quest #{}", quest.id), - Err(error) if !cli.quiet => eprintln!("Error while publishing quest: {error}"), - _ => {}, - } - }, - None if !cli.quiet => { - eprintln!("Error: couldn't find quest with id {}.", args.id); + do_and_log(quest.save(path), !cli.quiet, format!("Published quest #{}.", quest.id)); }, + None if !cli.quiet => eprintln!("Error: quest #{} not found.", args.id), _ => {}, } }, @@ -186,7 +184,6 @@ fn main() { match commands { AccountCommands::List => { - for account in accounts { println!("\"{}\": Balance {}", account.id, account.balance); } @@ -198,27 +195,16 @@ fn main() { }; if let Some(_) = accounts.iter().find(|a| a.id == account.id) { - if !cli.quiet { - eprintln!("Error: account {} exists.", account.id); - } + if !cli.quiet { eprintln!("Error: account \"{}\" exists.", account.id); } return; } - match account.save(path) { - Ok(_) if !cli.quiet => println!("Successfully created account \"{}\"", account.id), - Err(error) if !cli.quiet => eprintln!("Error while saving account: {error}"), - _ => {}, - } + do_and_log(account.save(path), !cli.quiet, format!("Created account \"{}\".", account.id)); }, AccountCommands::Balance(args) => { - let account = match accounts.iter_mut().find(|a| a.id == args.id) { - Some(acc) => acc, - None => { - if !cli.quiet { - eprintln!("Could not find account \"{}\"", args.id); - } - return; - } + let Some(account) = accounts.iter_mut().find(|a| a.id == args.id) else { + if !cli.quiet { eprintln!("Error: account \"{}\" not found.", args.id); } + return; }; match args.action { @@ -233,9 +219,7 @@ fn main() { if args.negative_ok { account.balance = 0u32; } else { - if !cli.quiet { - eprintln!("Error: balance ({}) is less than {}.", account.balance, args.value); - } + if !cli.quiet { eprintln!("Error: account \"{}\" balance is less than {}.", account.id, args.value); } return; } } else { @@ -244,54 +228,54 @@ fn main() { } } - match account.save(path) { - Ok(_) if !cli.quiet => println!("Successfully updated account \"{}\" balance", account.id), - Err(error) if !cli.quiet => eprintln!("Error while saving account: {error}"), - _ => {}, - } + do_and_log(account.save(path), !cli.quiet, format!("Updated balance of account \"{}\".", account.id)); }, AccountCommands::Complete(args) => { - let account = match accounts.iter_mut().find(|a| a.id == args.account) { - Some(acc) => acc, - None => { - if !cli.quiet { - eprintln!("Could not find account \"{}\"", args.account); - } - return; - } + let Some(account) = accounts.iter_mut().find(|a| a.id == args.account) else { + if !cli.quiet { eprintln!("Error: account \"{}\" not found.", args.account); } + return; }; let quests = config.load_quests(); - - if let None = quests.iter().find(|q| q.id == args.quest) { - if !cli.quiet { - eprintln!("Could not find quest #{}", args.quest); - } - return; - } - - match account.quests_completed.iter().find(|qid| **qid == args.quest) { - Some(_) if !cli.quiet => { - println!("Quest #{} is already completed on account \"{}\"", args.quest, args.account); - }, + + let quest = match quests.iter().find(|q| q.id == args.quest) { + Some(quest) => quest, None => { - account.quests_completed.push(args.quest); - match account.save(path) { - Ok(_) if !cli.quiet => println!("Successfully completed quest #{} on account \"{}\".", args.quest, account.id), - Err(error) if !cli.quiet => eprintln!("Error while saving account: {error}"), - _ => {}, - } + if !cli.quiet { eprintln!("Error: quest #{} not found.", args.quest); } + return; }, + }; + + match quest.complete_for_account(account) { + Err(error) if !cli.quiet => println!("Error: {error}"), + Ok(_) => do_and_log(account.save(path), !cli.quiet, format!("Completed quest #{} on account \"{}\".", args.quest, account.id)), _ => {}, } }, AccountCommands::Delete(args) => { path.push(format!("{}.toml", args.id)); - match Account::delete(path) { - Ok(_) if !cli.quiet => println!("Successfully deleted account \"{}\".", args.id), - Err(error) if !cli.quiet => eprintln!("Error while deleting account: {error}"), - _ => {}, + do_and_log(Account::delete(path), !cli.quiet, format!("Deleted account \"{}\".", args.id)) + }, + AccountCommands::Unlock(args) => { + let Some(account) = accounts.iter_mut().find(|a| a.id == args.account) else { + if !cli.quiet { eprintln!("Error: account \"{}\" not found.", args.account) }; + return; + }; + + let map = match Map::load(config.full_map_path()) { + Ok(map) => map, + Err(error) => { + if !cli.quiet { eprintln!("Error: {error}"); } + return; + } + }; + + if let Err(error) = map.unlock_room_for_account(args.room, account) { + eprintln!("Error: {error}"); + return; } + + do_and_log(account.save(path), !cli.quiet, format!("Unlocked room #{} for account \"{}\"", args.room, args.account)); }, } }, @@ -300,12 +284,12 @@ fn main() { let mut map = match Map::load(map_path.clone()) { Ok(map) => map, Err(error) => { - if !cli.quiet { - eprintln!("Error while loading map: {error}"); - } + if !cli.quiet { eprintln!("Error: {error}"); } return; } }; + + let map_save = |map: Map, map_path: PathBuf| { map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) }; match commands { MapCommands::List => { for room in map.room { @@ -321,22 +305,17 @@ fn main() { let room = Room { id: last_id, name: args.name.clone(), + value: args.value, description: args.description.clone(), ..Default::default() }; let r_id = room.id; map.room.push(room); - match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { - Ok(_) if !cli.quiet => println!("Created room #{}.", r_id), - Err(error) if !cli.quiet => eprintln!("Error while saving map: {error}"), - _ => {}, - } + do_and_log(map_save(map, map_path), !cli.quiet, format!("Created room #{r_id}.")) }, MapCommands::Delete(args) => { let Some(room) = map.room.iter().find(|r| r.id == args.id) else { - if !cli.quiet { - eprintln!("Error: Room #{} not found", args.id); - } + if !cli.quiet { eprintln!("Error: room #{} not found.", args.id); } return; }; @@ -351,17 +330,29 @@ fn main() { room.children.remove(idx); } - match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { - Ok(_) if !cli.quiet => println!("Deleted room #{}.", r_id), - Err(error) if !cli.quiet => eprintln!("Error while saving map: {error}"), + match map_save(map, map_path) { + Ok(_) => { + if !cli.quiet { println!("Deleted room #{r_id}."); } + + let mut accounts = config.load_accounts(); + let accounts_path = config.full_accounts_path(); + + for account in accounts.iter_mut() { + if let Some(index) = account.rooms_unlocked.iter().position(|rid| *rid == r_id) { + account.rooms_unlocked.remove(index); + do_and_log(account.save(accounts_path.clone()), !cli.quiet, format!("Removed room #{r_id} from account \"{}\" unlocked rooms.", account.id)); + } + } + }, + Err(error) if !cli.quiet => { + eprintln!("Error: {error}"); + }, _ => {}, } }, MapCommands::Update(args) => { let Some(room) = map.room.iter_mut().find(|r| r.id == args.id) else { - if !cli.quiet { - eprintln!("Error: Room #{} not found", args.id); - } + if !cli.quiet { eprintln!("Error: room #{} not found", args.id); } return; }; @@ -377,11 +368,7 @@ fn main() { room.value = value; } - match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { - Ok(_) if !cli.quiet => println!("Updated room #{}.", args.id), - Err(error) if !cli.quiet => eprintln!("Error while saving map: {error}"), - _ => {}, - } + do_and_log(map_save(map, map_path), !cli.quiet, format!("Updated room #{}.", args.id)) }, MapCommands::Connect(args) | MapCommands::Disconnect(args) => { let connect = match commands { @@ -392,28 +379,21 @@ fn main() { // We iterate twice to make references first->second and second->first for (first, second) in [(args.first, args.second),(args.second, args.first)] { let Some(room) = map.room.iter_mut().find(|r| r.id == first) else { - if !cli.quiet { - eprintln!("Error: Room #{} not found", first); - } + if !cli.quiet { eprintln!("Error: room #{} not found.", first); } return; }; match room.children.iter().position(|id| *id == second) { - Some(_) if connect && !cli.quiet => println!("Room #{} already has reference to #{}", first, second), + Some(_) if connect && !cli.quiet => println!("Room #{} already has reference to #{}.", first, second), None if connect => room.children.push(second), Some(id) if !connect => {room.children.remove(id as usize);}, - None if !connect && !cli.quiet => println!("Room #{} has no reference to #{}", first, second), + None if !connect && !cli.quiet => println!("Room #{} has no reference to #{}.", first, second), _ => {}, } } let connected = if connect { "Connected" } else { "Disconnected" }; - - match map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) { - Ok(_) if !cli.quiet => println!("{connected} rooms #{} <-> #{}.", args.first, args.second), - Err(error) if !cli.quiet => eprintln!("Error while saving map: {error}"), - _ => {}, - } + do_and_log(map_save(map, map_path), !cli.quiet, format!("{connected} rooms #{} <-> #{}.", args.first, args.second)); }, } } diff --git a/src/error.rs b/src/error.rs index 501d6c4..0804b58 100644 --- a/src/error.rs +++ b/src/error.rs @@ -26,3 +26,41 @@ impl fmt::Display for Error { } } } + +/// Error related to quest logic +#[derive(Debug)] +#[non_exhaustive] +pub enum QuestError { + /// Quest (self.0) is already completed for given account (self.1) + AlreadyCompleted(u16, String), +} + +impl fmt::Display for QuestError { + fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result { + match self { + Self::AlreadyCompleted(quest_id, account_id) => write!(f, "quest #{quest_id} is already completed for account \"{account_id}\""), + } + } +} + +/// Error related to map logic +#[derive(Debug)] +#[non_exhaustive] +pub enum MapError { + /// Room not found in map file + RoomNotFound(u16), + /// Room (self.0) is already unlocked on account (self.1) + RoomAlreadyUnlocked(u16, String), + /// Account (self.1) does not have much money (self.0) + InsufficientFunds(u16, String), +} + +impl fmt::Display for MapError { + fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result { + match self { + Self::RoomNotFound(id) => write!(f, "could not find room #{id}"), + Self::RoomAlreadyUnlocked(room_id, account_id) => write!(f, "room #{room_id} is already unlocked on account \"{account_id}\""), + Self::InsufficientFunds(room_id, account_id) => write!(f, "account \"{account_id}\" does not have enough money to unlock room #{room_id}"), + } + } +} diff --git a/src/map/mod.rs b/src/map/mod.rs index 17e42ce..69ccb68 100644 --- a/src/map/mod.rs +++ b/src/map/mod.rs @@ -4,7 +4,7 @@ use std::{fs, io::Write, path::PathBuf}; use serde::{Deserialize, Serialize}; -use crate::{SquadObject, error::Error}; +use crate::{SquadObject, account::Account, error::{Error, MapError}}; /// THE Graph. Actually, this is a Vec. #[derive(Serialize, Deserialize)] @@ -71,7 +71,43 @@ impl SquadObject for Map { Ok(()) } +} +impl Map { + /// Try to unlock room for account, or return [MapError] + /// + /// # Examples + /// ```rust + /// use squad_quest::{account::Account,map::{Map,Room},error::MapError}; + /// + /// let map = Map { + /// room: vec![Room { id: 0, value: 100, ..Default::default() }], + /// }; + /// + /// let mut account = Account { balance: 100, ..Default::default() }; + /// + /// if let Err(error) = map.unlock_room_for_account(0, &mut account) { + /// // handle error + /// } + /// ``` + pub fn unlock_room_for_account(&self, room_id: u16, account: &mut Account) -> Result<(), MapError> { + let Some(room) = self.room.iter().find(|r| r.id == room_id) else { + return Err(MapError::RoomNotFound(room_id)); + }; + + if let Some(_) = account.rooms_unlocked.iter().find(|rid| **rid == room_id) { + return Err(MapError::RoomAlreadyUnlocked(room_id, account.id.clone())); + } + + if account.balance < room.value { + return Err(MapError::InsufficientFunds(room_id, account.id.clone())); + } + + account.balance -= room.value; + account.rooms_unlocked.push(room_id); + + Ok(()) + } } /// Component of the map diff --git a/src/quest/mod.rs b/src/quest/mod.rs index b8d7886..669d061 100644 --- a/src/quest/mod.rs +++ b/src/quest/mod.rs @@ -3,7 +3,7 @@ use std::{fs, io::Write, path::PathBuf}; use serde::{ Serialize, Deserialize }; -use crate::{SquadObject, error::Error}; +use crate::{SquadObject, account::Account, error::{Error, QuestError}}; use toml::value::Date; /// Difficulty of the quest @@ -137,3 +137,32 @@ impl SquadObject for Quest { Ok(()) } } + +impl Quest { + /// Complete quest for account and add reward to it's balance. + /// Does nothing and returns [QuestError::AlreadyCompleted] + /// if it is already completed. + /// + /// # Examples + /// + /// ```rust + /// use squad_quest::{account::Account,quest::Quest}; + /// + /// let quest = Quest::default(); + /// let mut account = Account::default(); + /// + /// if let Err(error) = quest.complete_for_account(&mut account) { + /// // handle error + /// } + /// ``` + pub fn complete_for_account(&self, account: &mut Account) -> Result<(),QuestError> { + match account.quests_completed.iter().find(|qid| **qid == self.id) { + Some(_) => Err(QuestError::AlreadyCompleted(self.id, account.id.clone())), + None => { + account.quests_completed.push(self.id); + account.balance += self.reward; + Ok(()) + }, + } + } +} From 2960b6dfc4ff5647f012869583326c2820bc864a Mon Sep 17 00:00:00 2001 From: 2ndbeam <2ndbeam@disroot.org> Date: Fri, 5 Dec 2025 17:16:40 +0300 Subject: [PATCH 7/7] feat: Implemented file hierarchy initialization - Bump version to 0.5.1 - Added Config::save method - cli: Added init command --- Cargo.toml | 2 +- cfg/config.toml | 14 +++------- cli/src/cli/mod.rs | 9 +++++- cli/src/main.rs | 69 +++++++++++++++++++++++++++++++++++++++------- src/config/mod.rs | 42 ++++++++++++++++++++++++++-- 5 files changed, 111 insertions(+), 25 deletions(-) diff --git a/Cargo.toml b/Cargo.toml index d4e8eca..ec4ada1 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -2,7 +2,7 @@ members = ["cli"] [workspace.package] -version = "0.5.0" +version = "0.5.1" edition = "2024" repository = "https://2ndbeam.ru/git/2ndbeam/squad-quest" license = "MIT" diff --git a/cfg/config.toml b/cfg/config.toml index 1d81f41..0a9e166 100644 --- a/cfg/config.toml +++ b/cfg/config.toml @@ -1,10 +1,4 @@ -# Default config - -# Path to quests folder relative to config -quests_path = "./quests" - -# Path to accounts folder relative to config -accounts_path = "./accounts" - -# Path to map .toml file relative to config -map = "./map.toml" +quests_path = "quests" +accounts_path = "accounts" +map = "map.toml" +verbose = true diff --git a/cli/src/cli/mod.rs b/cli/src/cli/mod.rs index e44c361..db78d60 100644 --- a/cli/src/cli/mod.rs +++ b/cli/src/cli/mod.rs @@ -1,6 +1,6 @@ use std::path::PathBuf; -use clap::{Parser,Subcommand}; +use clap::{Args,Parser,Subcommand}; pub mod account; pub mod map; @@ -23,6 +23,8 @@ pub struct Cli { #[derive(Subcommand)] pub enum Objects { + /// Initialize new SquadQuest in current working directory + Init(InitArgs), /// Operations on the quests #[command(subcommand)] Quest(quest::QuestCommands), @@ -34,3 +36,8 @@ pub enum Objects { Map(map::MapCommands), } +#[derive(Args)] +pub struct InitArgs { + #[arg(long,short)] + pub path: Option, +} diff --git a/cli/src/main.rs b/cli/src/main.rs index 31b5b41..d120a9d 100644 --- a/cli/src/main.rs +++ b/cli/src/main.rs @@ -1,4 +1,4 @@ -use std::path::{Path, PathBuf}; +use std::{fs::DirBuilder, path::{Path, PathBuf}}; use clap::Parser; use squad_quest_cli::cli::{Cli,Objects,account::*,map::*,quest::*}; @@ -25,19 +25,17 @@ fn do_and_log(result: Result<(),Error>, log: bool, ok_text: String) { } } -fn main() { - let cli = Cli::parse(); - - let config = match cli.quiet { - false => Config::load(cli.config.clone()), +fn load_config_silent(quiet: bool, path: PathBuf) -> Config { + match quiet { + false => Config::load(path.clone()), true => { - match Config::try_load(cli.config.clone()) { + match Config::try_load(path.clone()) { Ok(mut config) => { config.verbose = false; config }, Err(_) => { - let path = cli.config.clone().parent().unwrap_or(&Path::new(".")).to_owned(); + let path = path.clone().parent().unwrap_or(&Path::new(".")).to_owned(); Config { verbose: false, path, @@ -46,9 +44,61 @@ fn main() { } } }, - }; + } +} + +fn main() { + let cli = Cli::parse(); + + let config = load_config_silent(cli.quiet, cli.config.clone()); + let map_save = |map: Map, map_path: PathBuf| { map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) }; match &cli.command { + Objects::Init(args) => { + let path = match args.path.clone() { + Some(path) => path, + None => PathBuf::new(), + }; + + match DirBuilder::new().recursive(true).create(path.clone()) { + Ok(_) if !cli.quiet => println!("Created directory {:?}", path), + Err(error) => { + if !cli.quiet { eprintln!("Error: {error}"); } + return; + }, + _ => {}, + } + + let config = Config { + path: path.clone(), + ..Default::default() + }; + + do_and_log(config.save(path.clone()), !cli.quiet, format!("Created file {:?}/config.toml", path)); + let mut config_path = path.clone(); + config_path.push("config.toml"); + let mut config = load_config_silent(true, config_path); + config.verbose = Config::default().verbose; + + let map = Map::default(); + let map_path = config.full_map_path(); + + do_and_log(map_save(map, map_path.clone()), !cli.quiet, format!("Created file {:?}/map.toml", map_path)); + + let quests_path = config.full_quests_path(); + let accounts_path = config.full_accounts_path(); + + for path in [quests_path, accounts_path] { + match DirBuilder::new().recursive(true).create(path.clone()) { + Ok(_) if !cli.quiet => println!("Created directory {:?}", path), + Err(error) => { + if !cli.quiet { eprintln!("Error: {error}"); } + return; + }, + _ => {}, + } + } + }, Objects::Quest(commands) => { let mut quests = config.load_quests(); let mut path = config.full_quests_path(); @@ -289,7 +339,6 @@ fn main() { } }; - let map_save = |map: Map, map_path: PathBuf| { map.save(map_path.parent().unwrap_or(Path::new("")).to_owned()) }; match commands { MapCommands::List => { for room in map.room { diff --git a/src/config/mod.rs b/src/config/mod.rs index c805002..f0b2f94 100644 --- a/src/config/mod.rs +++ b/src/config/mod.rs @@ -1,12 +1,12 @@ //! Configuration file that handles (de-)serializing other components -use std::{fs::{self, DirEntry},path::{Path, PathBuf}}; -use serde::Deserialize; +use std::{fs::{self, DirEntry}, io::Write, path::{Path, PathBuf}}; +use serde::{Deserialize, Serialize}; use crate::{SquadObject, account::Account, error::Error, quest::Quest}; /// Struct for containing paths to other (de-)serializable things -#[derive(Deserialize)] +#[derive(Serialize, Deserialize)] #[serde(default)] pub struct Config { /// Path to config directory @@ -112,6 +112,42 @@ impl Config { } } + /// Serialize config into TOML. + /// Config will be saved as `path/config.toml` + /// + /// # Examples + /// ```rust + /// use squad_quest::config::Config; + /// + /// let path = "cfg".into(); + /// + /// let config = Config::default(); + /// + /// if let Err(error) = config.save(path) { + /// // handle error + /// } + /// ``` + pub fn save(&self, path: PathBuf) -> Result<(), Error> { + let mut path = path; + path.push("config.toml"); + + let str = match toml::to_string_pretty(&self) { + Ok(string) => string, + Err(error) => return Err(Error::TomlSerializeError(error)), + }; + + let mut file = match fs::File::create(path) { + Ok(f) => f, + Err(error) => return Err(Error::IoError(error)), + }; + + if let Err(error) = file.write_all(str.as_bytes()) { + return Err(Error::IoError(error)); + } + + Ok(()) + } + /// Deserialize config from TOML /// /// # Examples