ref(init): prompt user for site configuration

This behaviour can be reverted by using `--no-prompt` which will automatically set sane default values as before

Author: NTBBloodbath

src/cli.rs

@@ -36,6 +36,12 @@ struct Cli {
 enum Commands {
     /// Initialize a new Norgolith site
     Init {
+        #[arg(long, default_value_t = true, overrides_with = "_no_prompt", help = "Whether to prompt for site info")]
+        prompt: bool,
+
+        #[arg(long = "no-prompt")]
+        _no_prompt: bool,
+
         /// Site name
         name: Option<String>,
     },
@@ -114,7 +120,7 @@ pub async fn start() -> Result<()> {
     }
 
     match &cli.command {
-        Commands::Init { name } => init_site(name.as_ref()).await?,
+        Commands::Init { name, prompt: _, _no_prompt } => init_site(name.as_ref(), !_no_prompt).await?,
         Commands::Theme { subcommand } => theme_handle(subcommand).await?,
         Commands::Serve { port, drafts: _, _no_drafts, open } => check_and_serve(*port, !_no_drafts, *open).await?,
         Commands::Build { minify } => build_site(*minify).await?,
@@ -134,9 +140,9 @@ pub async fn start() -> Result<()> {
 ///
 /// # Returns:
 ///   A `Result<()>` indicating success or error. On error, the context message will provide information on why the site could not be initialized.
-async fn init_site(name: Option<&String>) -> Result<()> {
+async fn init_site(name: Option<&String>, prompt: bool) -> Result<()> {
     if let Some(name) = name {
-        cmd::init(name).await?;
+        cmd::init(name, prompt).await?;
     } else {
         bail!("Missing name for the site: could not initialize the new Norgolith site");
     }
@@ -229,7 +235,7 @@ mod tests {
     #[tokio::test]
     async fn test_init_site_with_name() {
         let test_name = String::from("my-site");
-        let result = init_site(Some(&test_name)).await;
+        let result = init_site(Some(&test_name), false).await;
         assert!(result.is_ok());
 
         // Cleanup
@@ -238,7 +244,7 @@ mod tests {
 
     #[tokio::test]
     async fn test_init_site_without_name() {
-        let result = init_site(None).await;
+        let result = init_site(None, false).await;
         assert!(result.is_err());
         assert!(result
             .unwrap_err()
@@ -274,7 +280,7 @@ mod tests {
 
         // Create temporal site
         let test_site_dir = String::from("my-unavailable-site");
-        init_site(Some(&test_site_dir)).await?;
+        init_site(Some(&test_site_dir), false).await?;
 
         // Save current directory as the previous directory to restore it later
         let previous_dir = std::env::current_dir()?;

src/cmd/init.rs

@@ -6,10 +6,11 @@ use comfy_table::presets::UTF8_FULL;
 use comfy_table::{Cell, ContentArrangement, Table};
 use eyre::{bail, Result};
 use indoc::formatdoc;
+use inquire::Text;
 use tokio::fs;
 
 /// Create basic site configuration TOML
-async fn create_config(root: &str) -> Result<()> {
+async fn create_config(root: &str, root_url: &str, language: &str, title: &str) -> Result<()> {
     let site_config = formatdoc!(
         r#"
         rootUrl = '{}'
@@ -21,9 +22,9 @@ async fn create_config(root: &str) -> Result<()> {
         [highlighter]
         enable = false
         # engine = 'prism' # Can be 'prism' or 'hljs'. Defaults to 'prism'"#,
-        "http://localhost:3030", // this is the default port
-        "en-us",
-        root.to_owned(),
+        root_url, // this is the default port
+        language,
+        title,
         whoami::username()
     );
     // TBD: add Windows separator support
@@ -92,7 +93,7 @@ async fn create_directories(path: &str) -> Result<()> {
     Ok(())
 }
 
-pub async fn init(name: &str) -> Result<()> {
+pub async fn init(name: &str, prompt: bool) -> Result<()> {
     let path_exists = fs::try_exists(name).await?;
 
     if path_exists {
@@ -103,12 +104,38 @@ pub async fn init(name: &str) -> Result<()> {
             path.display()
         );
     } else {
+        // Prompt site configuration if wanted, otherwise fallback to sane default values
+        let root_url = if prompt {
+            Text::new("Site URL:")
+                .with_default("http://localhost:3030")
+                .with_help_message("URL to your production site")
+                .prompt()?
+        } else {
+            String::from("http://localhost:3030")
+        };
+        let language = if prompt {
+            Text::new("Site language:")
+                .with_default("en-US")
+                .with_help_message("Your site language")
+                .prompt()?
+        } else {
+            String::from("en-US")
+        };
+        let title = if prompt {
+            Text::new("Site title:")
+                .with_default(name)
+                .with_help_message("Site title")
+                .prompt()?
+        } else {
+            String::from(name)
+        };
+
         // Create site directories
         create_directories(name).await?;
 
         // Create initial files
         // TBD: Basic HTML templates
-        create_config(name).await?;
+        create_config(name, &root_url, &language, &title).await?;
         create_index_norg(name).await?;
         create_html_templates(name).await?;
         create_assets(name).await?;
@@ -131,7 +158,7 @@ pub async fn init(name: &str) -> Result<()> {
             .add_row(vec![Cell::new("templates"), Cell::new("HTML templates")])
             .add_row(vec![
                 Cell::new("assets"),
-                Cell::new("Site assets (JS, CSS, favicon, etc)"),
+                Cell::new("Site assets (JS, CSS, images, etc)"),
             ])
             .add_row(vec![Cell::new("theme"), Cell::new("Site theme files")])
             .add_row(vec![Cell::new(".build"), Cell::new("Dev server artifacts")]);