From ecfbad67b99bc7071869f048cd3651b853be6784 Mon Sep 17 00:00:00 2001
From: Antoine Stevan <44101798+amtoine@users.noreply.github.com>
Date: Mon, 22 Jul 2024 21:55:32 +0200
Subject: [PATCH] work on the `nu-themes` make script (#905)

i was mostly having fun there :yum:

cc/ @NotTheDr01ds

## changes
- using some colors for the logging
- slight refactor of the main, i.e. i build a `table<name: string,
source: string>` of themes and then iterate over it and call `make-theme
$t.name $t.source`, thus getting rid of the twin pipelines
- i also build a `table<name: string, source: string>` of failing
themes, to show them at the end

## reviewing
i think the best to review this is
- go to `themes/`
- remove any copy of the `lemnos/` directory
- run `./src/make.nu` twice and see the output :wink:
---
 themes/src/make.nu | 74 ++++++++++++++++++++++++++++++----------------
 1 file changed, 48 insertions(+), 26 deletions(-)

diff --git a/themes/src/make.nu b/themes/src/make.nu
index 852b84a..20692df 100755
--- a/themes/src/make.nu
+++ b/themes/src/make.nu
@@ -15,6 +15,18 @@ let CUSTOM_SOURCE = {
 
 let themes_dir = ($current_dir | path join "../nu-themes")
 
+def info [msg: string, --no-newline (-n)] {
+    print --no-newline=$no_newline $"(ansi green)[INFO](ansi reset) ($msg)"
+}
+
+def err [msg: string, --no-newline (-n)] {
+    print --no-newline=$no_newline $"(ansi red_bold)[ERR](ansi reset) ($msg)"
+}
+
+def warn [msg: string, --no-newline (-n)] {
+    print --no-newline=$no_newline $"(ansi yellow)[WARNING](ansi reset) ($msg)"
+}
+
 # For lemnos themes, create the color_config from the lemnos theme
 # definition.
 # Custom Nushell themes should be defined in themes/src/custom-nu-themes
@@ -205,36 +217,46 @@ def make_theme [ name: string, origin: string = "lemnos" ] {
 def main [] {
     mkdir $themes_dir
 
-    try { git clone $LEMNOS_SOURCE.remote_repo $LEMNOS_SOURCE.local_repo }
-
-    ls $LEMNOS_SOURCE.dir
-    | get name
-    | path parse
-    | get stem
-    | each {|theme|
-        print $"Converting ($theme)"
+    if ($LEMNOS_SOURCE.local_repo | path exists) {
+        warn "local copy of Lemnos' themes already exists"
+        info "updating local copy"
         try {
-            make_theme $theme
-        } catch {|e|
-            print -e $"Error converting ($theme)"
-            print -e $e.debug 
+            git -C $LEMNOS_SOURCE.local_repo pull origin master
+        } catch {
+            err "failed updating local copy"
         }
-    }
-    | ignore
-
-    ls $CUSTOM_SOURCE.dir
-    | get name
-    | path parse
-    | get stem
-    | each {|theme|
-        print $"Converting ($theme)"
+    } else {
+        info "cloning Lemnos' themes"
         try {
-            make_theme $theme "custom"
-        } catch {|e|
-            print -e $"Error converting ($theme)"
-            print -e $e.debug 
+            git clone $LEMNOS_SOURCE.remote_repo $LEMNOS_SOURCE.local_repo
+        } catch {
+            error make --unspanned { msg: "failed cloning local copy" }
         }
     }
 
-    print "all done"
+    let themes: table<name: string, source: string> = (
+        ls $LEMNOS_SOURCE.dir
+            | insert source "lemnos"
+            | append (ls $CUSTOM_SOURCE.dir | insert source "custom")
+            | update name { path parse | get stem }
+            | select name source
+    )
+
+    let failed = $themes | each { |t|
+        info -n $"Converting ($t.name)                                       \r"
+        try {
+            make_theme $t.name $t.source
+        } catch { |e|
+            err $"Converting ($t.name) failed: ($e.msg)"
+            $t
+        }
+    }
+
+    print ''
+    if not ($failed | is-empty) {
+        warn "The following themes have failed:"
+        print $failed
+    } else {
+        print "all done"
+    }
 }