feat: improve error messages (#471)

ref #467

Author: amrbashir

.changes/unsupported-template-error.md

@@ -0,0 +1,7 @@
+---
+"create-tauri-app": "patch"
+"create-tauri-app-js": "patch"
+---
+
+Improve the error messages for  unsupported package manager, unsupported template or when a supported template is used with a package manager that is not intended to be used with.
+

packages/cli/src/lib.rs

@@ -259,9 +259,9 @@ where
     // is valid, otherwise, we error and exit
     if !pkg_manager.templates().contains(&template) {
         eprintln!(
-            "{BOLD}{RED}error{RESET}: the {GREEN}{}{RESET} template is not suppported for the {GREEN}{pkg_manager}{RESET} package manager\n       possible templates for {GREEN}{pkg_manager}{RESET} are: [{}]",
-            template,
-            templates_no_flavors.iter().map(|e|format!("{GREEN}{e}{RESET}")).collect::<Vec<_>>().join(", ")
+            "{BOLD}{RED}error{RESET}: the {GREEN}{template}{RESET} template is not suppported for the {GREEN}{pkg_manager}{RESET} package manager\n       possible templates for {GREEN}{pkg_manager}{RESET} are: [{}]\n       or maybe you meant to use another package manager\n       possible package managers for {GREEN}{template}{RESET} are: [{}]" ,
+            templates_no_flavors.iter().map(|e|format!("{GREEN}{e}{RESET}")).collect::<Vec<_>>().join(", "),
+            template.possible_package_managers().iter().map(|e|format!("{GREEN}{e}{RESET}")).collect::<Vec<_>>().join(", "),
         );
         exit(1);
     }

packages/cli/src/package_manager.rs

@@ -4,7 +4,7 @@
 
 use std::{fmt::Display, str::FromStr};
 
-use crate::template::Template;
+use crate::{colors::*, template::Template};
 
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 #[non_exhaustive]
@@ -28,6 +28,13 @@ impl<'a> PackageManager {
         PackageManager::Yarn,
         PackageManager::Npm,
     ];
+
+    /// Node.js managers
+    pub const NODE: &'a [PackageManager] = &[
+        PackageManager::Pnpm,
+        PackageManager::Yarn,
+        PackageManager::Npm,
+    ];
 }
 impl PackageManager {
     /// Returns templates without flavors
@@ -122,7 +129,14 @@ impl FromStr for PackageManager {
             "pnpm" => Ok(PackageManager::Pnpm),
             "yarn" => Ok(PackageManager::Yarn),
             "npm" => Ok(PackageManager::Npm),
-            _ => Err("Invalid package manager".to_string()),
+            _ => Err(format!(
+                "{YELLOW}{s}{RESET} is not a valid package manager. Valid package mangers are [{}]",
+                PackageManager::ALL
+                    .iter()
+                    .map(|e| format!("{GREEN}{e}{RESET}"))
+                    .collect::<Vec<_>>()
+                    .join(", ")
+            )),
         }
     }
 }

packages/cli/src/template.rs

@@ -7,7 +7,7 @@ use std::{fmt::Display, fs, io::Write, path, str::FromStr};
 use anyhow::Context;
 use rust_embed::RustEmbed;
 
-use crate::{manifest::Manifest, package_manager::PackageManager};
+use crate::{colors::*, manifest::Manifest, package_manager::PackageManager};
 
 #[derive(RustEmbed)]
 #[folder = "fragments"]
@@ -102,7 +102,14 @@ impl FromStr for Template {
             "angular" => Ok(Template::Angular),
             "preact" => Ok(Template::Preact),
             "preact-ts" => Ok(Template::PreactTs),
-            _ => Err("Invalid template".to_string()),
+            _ => Err(format!(
+                "{YELLOW}{s}{RESET} is not a valid template. Valid templates are [{}]",
+                Template::ALL
+                    .iter()
+                    .map(|e| format!("{GREEN}{e}{RESET}"))
+                    .collect::<Vec<_>>()
+                    .join(", ")
+            )),
         }
     }
 }
@@ -169,15 +176,43 @@ impl<'a> Template {
         }
     }
 
+    pub const fn possible_package_managers(&self) -> &[PackageManager] {
+        match self {
+            Template::Vanilla => &[
+                PackageManager::Cargo,
+                PackageManager::Pnpm,
+                PackageManager::Yarn,
+                PackageManager::Npm,
+            ],
+            Template::VanillaTs => PackageManager::NODE,
+            Template::Vue => PackageManager::NODE,
+            Template::VueTs => PackageManager::NODE,
+            Template::Svelte => PackageManager::NODE,
+            Template::SvelteTs => PackageManager::NODE,
+            Template::React => PackageManager::NODE,
+            Template::ReactTs => PackageManager::NODE,
+            Template::Solid => PackageManager::NODE,
+            Template::SolidTs => PackageManager::NODE,
+            Template::Yew => &[PackageManager::Cargo],
+            Template::Leptos => &[PackageManager::Cargo],
+            Template::Sycamore => &[PackageManager::Cargo],
+            Template::Angular => PackageManager::NODE,
+            Template::Preact => PackageManager::NODE,
+            Template::PreactTs => PackageManager::NODE,
+        }
+    }
+
     pub const fn needs_trunk(&self) -> bool {
         matches!(self, Template::Sycamore | Template::Yew | Template::Leptos)
     }
+
     pub const fn needs_tauri_cli(&self) -> bool {
         matches!(
             self,
             Template::Sycamore | Template::Yew | Template::Leptos | Template::Vanilla
         )
     }
+
     pub const fn needs_wasm32_target(&self) -> bool {
         matches!(self, Template::Sycamore | Template::Yew | Template::Leptos)
     }