Merge pull request 'fix: notify when clipboard utility is missing' (#2) from fix/copy-fail into main

Reviewed-on: #2

README.md

@@ -84,20 +84,22 @@ You can define your preferred defaults in `~/.config/color-tool/config.toml`. Th
 
 ```toml
 [defaults]
-output = "hex"  # default output format(s)
+# Set any option to true to enable it by default when using the terminal
+output = "hex"  # default output format(s): hex, rgb, hsl, rgba, hsla, hexa, all
 json   = false  # output in JSON format
 swatch = false  # show color swatch in terminal
-name   = false  # fetch color name
+name   = false  # fetch color name from thecolorapi.com
 copy   = false  # copy result to clipboard
-pick   = false  # auto-launch picker
+pick   = false  # auto-launch color picker when invoked with no arguments
 notify = false  # show desktop notification
 
 [desktop]
-output = "hex"
-json   = false
-name   = false
-copy   = true
-notify = true
+# Defaults for --desktop mode (launched from the app menu; copy is always enabled by default)
+output = "hex"  # format to copy
+json   = false  # copy JSON format instead of plain text
+name   = false  # fetch color name (requires network)
+copy   = true   # copy result to clipboard
+notify = true   # show desktop notification with the copied value
 ```
 
 ## 🤝 Credits

color-tool

@@ -76,7 +76,8 @@ load_config() {
     if [[ "$line" =~ ^[[:space:]]*([A-Za-z_]+)[[:space:]]*=[[:space:]]*([^[:space:]]+) ]]; then
       key="${BASH_REMATCH[1]}"
       val="${BASH_REMATCH[2],,}"
-      val="${val#\"}" ; val="${val%\"}"
+      val="${val#\"}"
+      val="${val%\"}"
       case "$section:$key" in
       defaults:json) [[ "$val" == "true" ]] && json_mode=1 || json_mode=0 ;;
       defaults:alpha) [[ "$val" == "true" ]] && alpha_mode=1 || alpha_mode=0 ;;
@@ -159,7 +160,7 @@ copy_to_clipboard() {
   elif command -v xclip &>/dev/null; then
     printf '%s' "$text" | xclip -selection clipboard
   else
-    echo "Warning: no clipboard utility found (install wl-copy or xclip)" >&2
+    echo "Warning: Missing clipboard utility. Please install wl-clipboard (preferred) or xclip." >&2
     echo "  Value: $text" >&2
     return 1
   fi
@@ -171,6 +172,12 @@ notify_result() {
   notify-send -i "color-picker" "color-tool" "$value" || true
 }
 
+notify_error() {
+  local value="$1"
+  command -v notify-send &>/dev/null || return 0
+  notify-send -u normal -i "dialog-warning" "color-tool" "$value" || true
+}
+
 # ── Color picker ──────────────────────────────────────────────────────────────
 
 # Generate the internal Python helper for KDE Plasma color picking
@@ -281,7 +288,6 @@ do_install() {
 # Set any option to true to enable it by default when using the terminal
 output = "hex"  # default output format(s): hex, rgb, hsl, rgba, hsla, hexa, all
 json   = false  # output in JSON format
-alpha  = false  # include alpha channel (8-digit hex)
 swatch = false  # show color swatch in terminal
 name   = false  # fetch color name from thecolorapi.com
 copy   = false  # copy result to clipboard
@@ -292,7 +298,6 @@ notify = false  # show desktop notification
 # Defaults for --desktop mode (launched from the app menu; copy is always enabled by default)
 output = "hex"  # format to copy
 json   = false  # copy JSON format instead of plain text
-alpha  = false  # include alpha channel
 name   = false  # fetch color name (requires network)
 copy   = true   # copy result to clipboard
 notify = true   # show desktop notification with the copied value
@@ -398,7 +403,10 @@ validate_output_formats() {
 process_color() {
   local input="$1"
   local formats_json
-  formats_json=$(get_all_formats "$input") || { echo "Error: Invalid color: $input" >&2; return 1; }
+  formats_json=$(get_all_formats "$input") || {
+    echo "Error: Invalid color: $input" >&2
+    return 1
+  }
 
   local name=""
   if [[ $name_mode -eq 1 ]]; then
@@ -436,7 +444,10 @@ process_color() {
   if [[ $json_mode -eq 1 ]]; then
     output_text="$json_obj"
   else
-    output_text=$(IFS=' ' ; echo "${display_parts[*]}")
+    output_text=$(
+      IFS=' '
+      echo "${display_parts[*]}"
+    )
     [[ -n "$name" ]] && output_text="$output_text ($name)"
   fi
 
@@ -446,14 +457,29 @@ process_color() {
       r=$(echo "$formats_json" | jq -r '._raw.r')
       g=$(echo "$formats_json" | jq -r '._raw.g')
       b=$(echo "$formats_json" | jq -r '._raw.b')
-      if [[ $json_mode -eq 1 ]]; then printf "\033[48;2;${r};${g};${b}m  \033[0m\n"
+      if [[ $json_mode -eq 1 ]]; then
+        printf "\033[48;2;${r};${g};${b}m  \033[0m\n"
       else printf "\033[48;2;${r};${g};${b}m  \033[0m "; fi
     fi
     echo -e "$output_text"
   fi
 
-  [[ $copy_mode -eq 1 ]] && copy_to_clipboard "$output_text" || true
-  [[ $notify_mode -eq 1 ]] && notify_result "$output_text" || true
+  local copy_failed=0
+  if [[ $copy_mode -eq 1 ]]; then
+    if ! copy_to_clipboard "$output_text"; then
+      copy_failed=1
+    fi
+  fi
+
+  if [[ $notify_mode -eq 1 ]]; then
+    if [[ $copy_failed -eq 1 ]]; then
+      notify_error "Missing clipboard utility. Please install wl-clipboard (preferred) or xclip.
+
+Value: $output_text"
+    else
+      notify_result "$output_text"
+    fi
+  fi
 }
 
 # ── Argument parsing ──────────────────────────────────────────────────────────
@@ -465,11 +491,20 @@ do_pick=0
 # First pass: collect CLI overrides
 while [[ $# -gt 0 ]]; do
   case "$1" in
-    --help|-h) show_help; exit 0 ;;
-    --output)  cli_output="$2"; shift ;;
+  --help | -h)
+    show_help
+    exit 0
+    ;;
+  --output)
+    cli_output="$2"
+    shift
+    ;;
   --json) cli_json=1 ;;
   --no-json) cli_json=0 ;;
-    --alpha)   cli_alpha=1; cli_output="hexa" ;;
+  --alpha)
+    cli_alpha=1
+    cli_output="hexa"
+    ;;
   --no-alpha) cli_alpha=0 ;;
   --name) cli_name=1 ;;
   --no-name) cli_name=0 ;;
@@ -482,8 +517,14 @@ while [[ $# -gt 0 ]]; do
   --pick) cli_pick=1 ;;
   --no-pick) cli_pick=0 ;;
   --desktop) desktop_mode=1 ;;
-    --install) do_install; exit 0 ;;
-    -*) echo "Unknown option: $1" >&2; exit 1 ;;
+  --install)
+    do_install
+    exit 0
+    ;;
+  -*)
+    echo "Unknown option: $1" >&2
+    exit 1
+    ;;
   *) args+=("$1") ;;
   esac
   shift
@@ -525,7 +566,8 @@ elif [[ ${#args[@]} -eq 0 ]]; then
     picked="$(run_color_picker)" || exit 1
     [[ -n "$picked" ]] && args+=("$picked")
   elif [[ -t 0 ]]; then
-    show_help; exit 0
+    show_help
+    exit 0
   fi
 fi