imrpovement in player autocomplete

src/cmds/setting.rs

@@ -25,11 +25,11 @@ pub async fn link(
     #[description = "your game server & ign"]
     player: String, // the String is a Serialized PartialPlayer struct
 ) -> Result<(), Error> {
+    let api = WowsApi::new(&ctx);
     let partial_player = {
         let (region, ign) = parse_region_ign(&player)?;
-        cache_methods::player(WowsApi::new(&ctx), &region, &ign).await?
+        cache_methods::player(&api, &region, &ign).await?
     };
-    let api = WowsApi::new(&ctx);
     let player = partial_player.full_player(&api).await?;
     {
         let mut guard = ctx.data().link.write().await;

src/dc_utils/auto_complete.rs

@@ -42,7 +42,7 @@ pub async fn player(ctx: Context<'_>, input: &str) -> Vec<AutocompleteChoice> {
     candidates
         .into_iter()
         .map(|vortex_p| {
-            let output = format!("[{}] {}", region, vortex_p.name);
+            let output = format!("{}  ({})", vortex_p.name, region);
             AutocompleteChoice::new(output.clone(), output)
         })
         .collect()

src/utils/parse.rs

@@ -11,7 +11,7 @@ use crate::{
 /// # Error
 /// [`IsacInfo::GeneralError`] if received a malformed input
 pub fn parse_region_ign(input: &str) -> Result<(Region, String), IsacError> {
-    static RE: Lazy<Regex> = Lazy::new(|| Regex::new(r"\[?(\w+)\]?\s+(\w+)").unwrap());
+    static RE: Lazy<Regex> = Lazy::new(|| Regex::new(r"(\w+)\s+\((\w+)\)").unwrap());
     let (_, [region_str, ign]) = RE
         .captures(input)
         .ok_or(IsacInfo::GeneralError {