Merge pull request 'feat(config): enable desktop swatch and improve comment alignment' (#11) from feat/desktop-swatch-and-config-alignment into main

Reviewed-on: #11

.gitea/workflows/label-pr.yaml

@@ -0,0 +1,42 @@
+name: Auto Label PRs
+
+on:
+  pull_request:
+    types: [opened, edited]
+
+jobs:
+  label:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Apply labels based on PR title
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          PR_TITLE: ${{ github.event.pull_request.title }}
+          PR_NUMBER: ${{ github.event.pull_request.number }}
+          REPO_OWNER: ${{ github.repository_owner }}
+          REPO_NAME: ${{ github.event.repository.name }}
+          API_URL: ${{ github.api_url }}
+        run: |
+          shopt -s nocasematch
+          
+          labels="["
+          
+          if [[ "$PR_TITLE" =~ ^(ci|test)(/|:) ]]; then
+            labels="${labels}13," # 13 is Kind/Testing
+          fi
+          
+          if [[ "$PR_TITLE" =~ ^docs(/|:) ]]; then
+            labels="${labels}14," # 14 is Kind/Documentation
+          fi
+          
+          labels="${labels%,}]"
+          
+          if [ "$labels" != "[]" ]; then
+            echo "Adding labels to PR #${PR_NUMBER}: ${labels}"
+            curl -X POST "${API_URL}/repos/${REPO_OWNER}/${REPO_NAME}/issues/${PR_NUMBER}/labels" \
+              -H "Authorization: token ${GITHUB_TOKEN}" \
+              -H "Content-Type: application/json" \
+              -d "{\"labels\": ${labels}}"
+          else
+            echo "No matching prefixes found in title: ${PR_TITLE}"
+          fi

.gitea/workflows/release.yaml

@@ -8,7 +8,10 @@ on:
 
 jobs:
   release:
-    if: github.event.pull_request.merged == true
+    if: |
+      github.event.pull_request.merged == true &&
+      !contains(github.event.pull_request.labels.*.name, 'Kind/Testing') &&
+      !contains(github.event.pull_request.labels.*.name, 'Kind/Documentation')
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code

.gitea/workflows/test.yaml

@@ -0,0 +1,22 @@
+name: Test PR
+
+on:
+  pull_request:
+    branches:
+      - main
+
+jobs:
+  test:
+    if: "!contains(github.event.pull_request.labels.*.name, 'Kind/Documentation')"
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Make scripts executable
+        run: |
+          chmod +x color-tool
+          chmod +x tests/run_tests.sh
+
+      - name: Run test suite
+        run: ./tests/run_tests.sh

README.md

@@ -10,7 +10,7 @@ A feature-rich, portable CLI color utility for Linux, specializing in color pick
 - **Smart JSON Output:** Generate machine-readable JSON tables for easy integration with other tools.
 - **Color Naming:** Resolve hex values to human-readable names via thecolorapi.com.
 - **Visual Previews:** Render color swatches directly in your terminal using truecolor ANSI escapes.
-- **Desktop Integration:** Built-in support for clipboard copying (`wl-copy`/`xclip`) and desktop notifications.
+- **Desktop Integration:** Built-in support for clipboard copying (`wl-copy`/`xclip`) and rich desktop notifications with visual color swatches.
 - **Highly Configurable:** Separate configuration profiles for standard terminal usage and desktop/launcher mode.
 - **Portable Design:** The entire tool is contained within a single Bash script, with embedded Python helpers.
 
@@ -22,12 +22,14 @@ To utilize all features, ensure the following are installed:
 - **curl + jq:** Required for color naming support.
 - **wl-clipboard** (Wayland) or **xclip** (X11): Required for clipboard support.
 - **libnotify:** Required for desktop notifications (`notify-send`).
+- **ImageMagick (`magick` or `convert`):** Required for generating visual color swatches in desktop notifications.
 
 ## 🚀 Installation
 
-Simply download the `color-tool` script and run the install command:
+You can download the script directly from the [latest release](https://git.rootiest.dev/rootiest/color-tool/releases/latest) and install it:
 
 ```bash
+curl -sSLO https://git.rootiest.dev/rootiest/color-tool/releases/download/latest/color-tool
 chmod +x color-tool
 ./color-tool --install
 ```
@@ -37,6 +39,7 @@ This will:
 2. Symlink the binary to `~/.local/bin/color-tool`.
 3. Generate a sample configuration at `~/.config/color-tool/config.toml`.
 4. Create a `.desktop` entry so you can launch the picker from your application menu.
+5. Verify and warn about any missing optional or required system dependencies.
 
 ## 🛠 Usage
 
@@ -52,6 +55,9 @@ Options:
   --[no-]copy    Copy result to clipboard
   --[no-]notify  Show desktop notification
   --desktop      GUI mode: pick → copy → notify
+  --get-config   Print the current configuration
+  --set-config   Update the configuration (e.g. --set-config desktop --copy)
+  --reset-config Restore the configuration file to its default values
   --install      Install to ~/.local/share/ and symlink to ~/.local/bin/
   --help, -h     Show this help message
 ```
@@ -84,22 +90,53 @@ 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
+swatch = true   # show color swatch in notification
 ```
 
+You can also use the CLI to manage your configuration:
+
+```bash
+# View current config
+color-tool --get-config
+
+# Update specific keys (e.g., enable swatch by default)
+color-tool --set-config defaults --swatch
+
+# Set output format for desktop mode
+color-tool --set-config desktop output=rgba
+
+# Reset to factory defaults
+color-tool --reset-config
+```
+
+## 🧪 Testing
+
+The project includes a comprehensive test suite that verifies color conversion, configuration loading, and system integrations (clipboard/notifications) using mocks.
+
+To run the tests:
+
+```bash
+./tests/run_tests.sh
+```
+
+The test script creates a temporary isolated environment, so it won't affect your system configuration or active clipboard.
+
 ## 🤝 Credits
 
 The `wl-colorpicker-plasma` integration is based on the original work by [SASUPERNOVA](https://github.com/SASUPERNOVA/wl-colorpicker-plasma).

color-tool

@@ -23,6 +23,7 @@ set -euo pipefail
 # Resolve the real directory of this script so we can find bundled helpers
 SCRIPT_DIR="$(dirname "$(readlink -f "$0")")"
 CONFIG_FILE="$HOME/.config/color-tool/config.toml"
+REPO_URL="https://git.rootiest.dev/rootiest/color-tool"
 
 # ── Initial Defaults ──────────────────────────────────────────────────────────
 
@@ -44,6 +45,7 @@ desktop_name=0
 desktop_notify=1
 desktop_copy=1
 desktop_output="hex"
+desktop_swatch=1
 
 # Track which settings were explicitly set via CLI flags to ensure they override config
 cli_json=""
@@ -57,6 +59,182 @@ cli_output=""
 
 # ── Config Loader ─────────────────────────────────────────────────────────────
 
+get_config() {
+  if [[ ! -f "$CONFIG_FILE" ]]; then
+    echo "Error: Config file not found at $CONFIG_FILE" >&2
+    exit 1
+  fi
+  
+  local first_section=1
+  while IFS= read -r line || [[ -n "$line" ]]; do
+    local stripped="${line%%#*}"
+    stripped="${stripped%"${stripped##*[![:space:]]}"}"
+    if [[ -z "$stripped" ]]; then
+      continue
+    fi
+    
+    if [[ "$stripped" =~ ^\[([A-Za-z_-]+)\]$ ]]; then
+      if [[ $first_section -eq 0 ]]; then
+        echo ""
+      fi
+      first_section=0
+      echo "$stripped"
+    elif [[ "$stripped" =~ ^[[:space:]]*([A-Za-z_]+)[[:space:]]*=[[:space:]]*(.*)$ ]]; then
+      local key="${BASH_REMATCH[1]}"
+      local val="${BASH_REMATCH[2]}"
+      printf "%-6s = %s\n" "$key" "$val"
+    fi
+  done < "$CONFIG_FILE"
+}
+
+set_config() {
+  local target_section="defaults"
+  if [[ $# -gt 0 ]]; then
+    if [[ "$1" == "default" || "$1" == "defaults" ]]; then
+      target_section="defaults"
+      shift
+    elif [[ "$1" == "desktop" ]]; then
+      target_section="desktop"
+      shift
+    fi
+  fi
+  
+  local new_output=""
+  local new_json=""
+  local new_swatch=""
+  local new_name=""
+  local new_copy=""
+  local new_pick=""
+  local new_notify=""
+  local new_alpha=""
+
+  while [[ $# -gt 0 ]]; do
+    case "$1" in
+      --output=*) new_output="${1#*=}" ;;
+      output=*) new_output="${1#*=}" ;;
+      --json) new_json="true" ;;
+      --no-json) new_json="false" ;;
+      --alpha) new_alpha="true" ;;
+      --no-alpha) new_alpha="false" ;;
+      --name) new_name="true" ;;
+      --no-name) new_name="false" ;;
+      --swatch) new_swatch="true" ;;
+      --no-swatch) new_swatch="false" ;;
+      --copy) new_copy="true" ;;
+      --no-copy) new_copy="false" ;;
+      --notify) new_notify="true" ;;
+      --no-notify) new_notify="false" ;;
+      --pick) new_pick="true" ;;
+      --no-pick) new_pick="false" ;;
+      -*) 
+        echo "Error: Unknown option for --set-config: $1" >&2
+        exit 1
+        ;;
+      *)
+        echo "Error: Unknown argument for --set-config: $1" >&2
+        exit 1
+        ;;
+    esac
+    shift
+  done
+  
+  if [[ -n "$new_output" ]]; then
+    new_output="${new_output#\"}"
+    new_output="${new_output%\"}"
+  fi
+
+  if [[ ! -f "$CONFIG_FILE" ]]; then
+    echo "Error: Config file not found at $CONFIG_FILE. Run --install first." >&2
+    exit 1
+  fi
+  
+  local temp_file
+  temp_file="$(mktemp)"
+  local current_section=""
+  
+  while IFS= read -r line || [[ -n "$line" ]]; do
+    local original_line="$line"
+    local parsed_line="${line%%#*}"
+    
+    if [[ "$parsed_line" =~ ^\[([A-Za-z_-]+)\]$ ]]; then
+      current_section="${BASH_REMATCH[1]}"
+      echo "$original_line" >> "$temp_file"
+      continue
+    fi
+    
+    if [[ "$current_section" == "$target_section" && "$parsed_line" =~ ^[[:space:]]*([A-Za-z_]+)[[:space:]]*= ]]; then
+      local key="${BASH_REMATCH[1]}"
+      local new_val=""
+      case "$key" in
+        output)
+          if [[ -n "$new_output" ]]; then new_val="\"$new_output\""; fi
+          ;;
+        json)
+          if [[ -n "$new_json" ]]; then new_val="$new_json"; fi
+          ;;
+        alpha)
+          if [[ -n "$new_alpha" ]]; then new_val="$new_alpha"; fi
+          ;;
+        swatch)
+          if [[ -n "$new_swatch" ]]; then new_val="$new_swatch"; fi
+          ;;
+        name)
+          if [[ -n "$new_name" ]]; then new_val="$new_name"; fi
+          ;;
+        copy)
+          if [[ -n "$new_copy" ]]; then new_val="$new_copy"; fi
+          ;;
+        pick)
+          if [[ -n "$new_pick" ]]; then new_val="$new_pick"; fi
+          ;;
+        notify)
+          if [[ -n "$new_notify" ]]; then new_val="$new_notify"; fi
+          ;;
+      esac
+      
+      if [[ -n "$new_val" ]]; then
+        if [[ "$original_line" =~ ^([[:space:]]*[A-Za-z_]+[[:space:]]*=[[:space:]]*)([^[:space:]#]+)(.*)$ ]]; then
+           local prefix="${BASH_REMATCH[1]}"
+           local old_val="${BASH_REMATCH[2]}"
+           local suffix="${BASH_REMATCH[3]}"
+           
+           # Adjust whitespace to keep comments aligned if the value length changed
+           if [[ "$key" != "output" ]]; then
+             local old_len=${#old_val}
+             local new_len=${#new_val}
+             local diff=$((old_len - new_len))
+             
+             if [[ $diff -gt 0 ]]; then
+               # new value is shorter, add spaces to suffix
+               local spaces=""
+               for ((i=0; i<diff; i++)); do spaces+=" "; done
+               suffix="$spaces$suffix"
+             elif [[ $diff -lt 0 ]]; then
+               # new value is longer, remove spaces from suffix
+               local remove=$((-diff))
+               for ((i=0; i<remove; i++)); do
+                 if [[ "$suffix" == " "* ]]; then
+                   suffix="${suffix# }"
+                 fi
+               done
+             fi
+           fi
+           
+           echo "$prefix$new_val$suffix" >> "$temp_file"
+           continue
+        elif [[ "$original_line" =~ ^([[:space:]]*[A-Za-z_]+[[:space:]]*=[[:space:]]*)(.*)$ ]]; then
+           echo "${BASH_REMATCH[1]}$new_val" >> "$temp_file"
+           continue
+        fi
+      fi
+    fi
+    
+    echo "$original_line" >> "$temp_file"
+  done < "$CONFIG_FILE"
+  
+  mv "$temp_file" "$CONFIG_FILE"
+}
+
 # Parse ~/.config/color-tool/config.toml and apply [defaults] and [desktop] values.
 load_config() {
   [[ ! -f "$CONFIG_FILE" ]] && return 0
@@ -76,7 +254,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 ;;
@@ -90,11 +269,44 @@ load_config() {
       desktop:alpha) [[ "$val" == "true" ]] && desktop_alpha=1 || desktop_alpha=0 ;;
       desktop:name) [[ "$val" == "true" ]] && desktop_name=1 || desktop_name=0 ;;
       desktop:notify) [[ "$val" == "true" ]] && desktop_notify=1 || desktop_notify=0 ;;
+      desktop:swatch) [[ "$val" == "true" ]] && desktop_swatch=1 || desktop_swatch=0 ;;
       desktop:copy) [[ "$val" == "true" ]] && desktop_copy=1 || desktop_copy=0 ;;
       desktop:output) desktop_output="$val" ;;
       esac
     fi
-  done < "$CONFIG_FILE"
+  done <"$CONFIG_FILE"
+}
+
+write_default_config() {
+  mkdir -p "$(dirname "$CONFIG_FILE")"
+  cat >"$CONFIG_FILE" <<EOF
+# color-tool configuration
+# Source: $REPO_URL
+
+[defaults]
+# 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 from thecolorapi.com
+copy   = false  # copy result to clipboard
+pick   = false  # auto-launch color picker when invoked with no arguments
+notify = false  # show desktop notification
+
+[desktop]
+# 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
+swatch = true   # show color swatch in notification
+EOF
+}
+
+reset_config() {
+  write_default_config
+  echo "Configuration reset to defaults: $CONFIG_FILE"
 }
 
 # ── Help ──────────────────────────────────────────────────────────────────────
@@ -119,10 +331,13 @@ show_help() {
   printf "  ${bold}${cyan}--output${reset} ${dim}FMT${reset}    Format(s) to output: ${cyan}hex, rgb, hsl, rgba, hsla, hexa, all${reset} ${dim}(comma-separated)${reset}\n"
   printf "  ${bold}${cyan}--[no-]json${reset}    Output as a JSON table of selected formats\n"
   printf "  ${bold}${cyan}--[no-]name${reset}    Fetch nearest color name from thecolorapi.com ${dim}(requires curl, jq)${reset}\n"
-  printf "  ${bold}${cyan}--[no-]swatch${reset}  Include a color swatch in the terminal output\n"
-  printf "  ${bold}${cyan}--[no-]copy${reset}    Copy result to clipboard ${dim}(wl-copy preferred, xclip as fallback)${reset}\n"
+  printf "  ${bold}${cyan}--[no-]swatch${reset}  Include a color swatch in the terminal output and desktop notification\n"
+  printf "  ${bold}${cyan}--[no-]copy${reset}    Copy result to clipboard ${dim}(wl-clipboard preferred, xclip as fallback)${reset}\n"
   printf "  ${bold}${cyan}--[no-]notify${reset}  Show desktop notification ${dim}(on by default in --desktop)${reset}\n"
   printf "  ${bold}${cyan}--desktop${reset}      GUI mode: pick → copy → notify ${dim}(for app menu / .desktop launcher)${reset}\n"
+  printf "  ${bold}${cyan}--get-config${reset}   Print the current configuration\n"
+  printf "  ${bold}${cyan}--set-config${reset}   Update the configuration ${dim}(e.g. --set-config desktop --copy)${reset}\n"
+  printf "  ${bold}${cyan}--reset-config${reset} Restore the configuration file to its default values\n"
   printf "  ${bold}${cyan}--install${reset}      Install to ~/.local/share/color-tool/ and symlink into ~/.local/bin/\n"
   printf "  ${bold}${cyan}--help${reset}, ${bold}${cyan}-h${reset}     Show this help message\n\n"
 
@@ -133,7 +348,7 @@ show_help() {
 
   printf "  ${bold}${yellow}Config${reset}  ${dim}~/.config/color-tool/config.toml${reset}\n"
   printf "  ${dim}[defaults]${reset} keys: ${cyan}output json swatch name copy pick notify${reset}\n"
-  printf "  ${dim}[desktop]${reset}  keys: ${cyan}output json name notify copy${reset}  ${dim}(pick always on)${reset}\n\n"
+  printf "  ${dim}[desktop]${reset}  keys: ${cyan}output json name notify copy swatch${reset}  ${dim}(pick always on)${reset}\n\n"
 }
 
 # ── Environment detection ─────────────────────────────────────────────────────
@@ -159,7 +374,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
@@ -167,8 +382,30 @@ copy_to_clipboard() {
 
 notify_result() {
   local value="$1"
+  local hex_color="${2:-}"
   command -v notify-send &>/dev/null || return 0
-  notify-send -i "color-picker" "color-tool" "$value" || true
+
+  local icon_name="color-picker"
+  if [[ $swatch_mode -eq 1 && -n "$hex_color" ]]; then
+    local swatch_path="/tmp/color_swatch_${hex_color//\#/}.png"
+    if command -v magick >/dev/null 2>&1; then
+      if magick -size 64x64 xc:"$hex_color" "$swatch_path" 2>/dev/null; then
+        icon_name="$swatch_path"
+      fi
+    elif command -v convert >/dev/null 2>&1; then
+      if convert -size 64x64 xc:"$hex_color" "$swatch_path" 2>/dev/null; then
+        icon_name="$swatch_path"
+      fi
+    fi
+  fi
+
+  notify-send -i "$icon_name" "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 ──────────────────────────────────────────────────────────────
@@ -176,7 +413,7 @@ notify_result() {
 # Generate the internal Python helper for KDE Plasma color picking
 generate_picker_script() {
   local target="$1"
-  cat > "$target" <<'EOF'
+  cat >"$target" <<'EOF'
 #!/usr/bin/python3
 #
 # Original work Copyright (C) 2024 SASUPERNOVA
@@ -256,6 +493,60 @@ suggest_path_add() {
   fi
 }
 
+check_dependencies() {
+  local missing=()
+  
+  if ! command -v magick >/dev/null 2>&1 && ! command -v convert >/dev/null 2>&1; then
+    missing+=("ImageMagick (magick or convert) — required for notification color swatches")
+  fi
+  
+  if ! command -v wl-copy >/dev/null 2>&1 && ! command -v xclip >/dev/null 2>&1; then
+    missing+=("wl-clipboard (wl-copy) or xclip — required for clipboard copy support")
+  fi
+  
+  if ! command -v python3 >/dev/null 2>&1; then
+    missing+=("python3 — required for color conversions and picker GUI")
+  elif ! python3 -c "import PyQt6.QtDBus" >/dev/null 2>&1; then
+    missing+=("python3-pyqt6 — required for the KDE color picker DBus interface")
+  fi
+  
+  if ! command -v jq >/dev/null 2>&1; then
+    missing+=("jq — required for JSON formatting and color names")
+  fi
+  
+  if ! command -v curl >/dev/null 2>&1; then
+    missing+=("curl — required for fetching color names from the API")
+  fi
+  
+  if ! command -v notify-send >/dev/null 2>&1; then
+    missing+=("libnotify (notify-send) — required for desktop notifications")
+  fi
+  
+  local env_ok=1
+  if [[ -z "${WAYLAND_DISPLAY:-}" ]]; then
+    env_ok=0
+  fi
+  local desktop="${XDG_CURRENT_DESKTOP:-}"
+  if [[ "$desktop" != *KDE* ]] && [[ -z "${KDE_FULL_SESSION:-}" ]]; then
+    env_ok=0
+  fi
+  
+  if [[ $env_ok -eq 0 ]]; then
+    missing+=("KDE Plasma on Wayland — required for the screen color picker feature")
+  fi
+  
+  if [[ ${#missing[@]} -gt 0 ]]; then
+    local yellow='\033[33m'
+    local bold='\033[1m'
+    local reset='\033[0m'
+    printf "\n  ${bold}${yellow}Warnings — Missing optional dependencies:${reset}\n"
+    for item in "${missing[@]}"; do
+      printf "    - %s\n" "$item"
+    done
+    printf "  ${yellow}Functionality will be limited without these installed.${reset}\n"
+  fi
+}
+
 do_install() {
   local data_dir="$HOME/.local/share/color-tool"
   local bin_dir="$HOME/.local/bin"
@@ -271,32 +562,8 @@ do_install() {
 
   local config_dir="$HOME/.config/color-tool"
   local config_file="$config_dir/config.toml"
-  mkdir -p "$config_dir"
   if [[ ! -f "$config_file" ]]; then
-    cat > "$config_file" <<'EOF'
-# color-tool configuration
-# https://github.com/rootiest/color-tool
-
-[defaults]
-# 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
-pick   = false  # auto-launch color picker when invoked with no arguments
-notify = false  # show desktop notification
-
-[desktop]
-# 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
-EOF
+    write_default_config
     printf "  config  %s  (sample created)\n" "$config_file"
   else
     printf "  config  %s\n" "$config_file"
@@ -306,7 +573,7 @@ EOF
   local desktop_file="$app_dir/color-tool.desktop"
   local bin_path="$HOME/.local/bin/color-tool"
   mkdir -p "$app_dir"
-  cat > "$desktop_file" <<EOF
+  cat >"$desktop_file" <<EOF
 [Desktop Entry]
 Version=1.1
 Type=Application
@@ -329,6 +596,8 @@ EOF
     printf "\nNote: %s is not in your PATH.\n" "$bin_dir"
     suggest_path_add "$bin_dir"
   fi
+
+  check_dependencies
 }
 
 # ── Color conversion ──────────────────────────────────────────────────────────
@@ -381,7 +650,7 @@ print(json.dumps(formats))
 validate_output_formats() {
   local formats="$1"
   local valid_fmts=("hex" "hexa" "rgb" "rgba" "hsl" "hsla")
-  IFS=',' read -ra ADDR <<< "$formats"
+  IFS=',' read -ra ADDR <<<"$formats"
   for fmt in "${ADDR[@]}"; do
     [[ "$fmt" == "all" ]] && continue
     local is_valid=0
@@ -398,18 +667,23 @@ 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 hex_color
+  hex_color=$(echo "$formats_json" | jq -r '.hex // empty')
 
   local name=""
   if [[ $name_mode -eq 1 ]]; then
-    local hex_for_api
-    hex_for_api=$(echo "$formats_json" | jq -r '.hex' | tr -d '#')
+    local hex_for_api="${hex_color//#/}"
     name=$(curl -s "https://www.thecolorapi.com/id?hex=$hex_for_api" | jq -r '.name.value // empty')
   fi
 
   local selected_fmts=()
   local valid_fmts=("hex" "hexa" "rgb" "rgba" "hsl" "hsla")
-  IFS=',' read -ra ADDR <<< "$output_formats"
+  IFS=',' read -ra ADDR <<<"$output_formats"
   for fmt in "${ADDR[@]}"; do
     if [[ "$fmt" == "all" ]]; then
       selected_fmts=("${valid_fmts[@]}")
@@ -436,7 +710,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 +723,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" "$hex_color"
+    fi
+  fi
 }
 
 # ── Argument parsing ──────────────────────────────────────────────────────────
@@ -465,31 +757,58 @@ 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" ;;
-    --no-alpha)cli_alpha=0   ;;
+  --alpha)
+    cli_alpha=1
+    cli_output="hexa"
+    ;;
+  --no-alpha) cli_alpha=0 ;;
   --name) cli_name=1 ;;
   --no-name) cli_name=0 ;;
   --swatch) cli_swatch=1 ;;
-    --no-swatch)cli_swatch=0 ;;
+  --no-swatch) cli_swatch=0 ;;
   --copy) cli_copy=1 ;;
   --no-copy) cli_copy=0 ;;
   --notify) cli_notify=1 ;;
-    --no-notify)cli_notify=0 ;;
+  --no-notify) cli_notify=0 ;;
   --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
+    ;;
+  --get-config)
+    get_config
+    exit 0
+    ;;
+  --set-config)
+    shift
+    set_config "$@"
+    exit 0
+    ;;
+  --reset-config)
+    reset_config
+    exit 0
+    ;;
+  -*)
+    echo "Unknown option: $1" >&2
+    exit 1
+    ;;
   *) args+=("$1") ;;
   esac
   shift
 done
 
-# Apply final hierarchy: CLI Flags > Desktop Config (if --desktop) > Default Config
 if [[ $desktop_mode -eq 1 ]]; then
   json_mode=${cli_json:-$desktop_json}
   alpha_mode=${cli_alpha:-$desktop_alpha}
@@ -497,8 +816,8 @@ if [[ $desktop_mode -eq 1 ]]; then
   notify_mode=${cli_notify:-$desktop_notify}
   output_formats=${cli_output:-$desktop_output}
   copy_mode=${cli_copy:-$desktop_copy}
+  swatch_mode=${cli_swatch:-$desktop_swatch}
   do_pick=${cli_pick:-1}
-  swatch_mode=${cli_swatch:-0} # swatch usually off in desktop mode
 else
   json_mode=${cli_json:-$json_mode}
   alpha_mode=${cli_alpha:-$alpha_mode}
@@ -525,7 +844,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
 

tests/run_tests.sh

@@ -0,0 +1,242 @@
+#!/bin/bash
+set -euo pipefail
+
+# ── Setup ─────────────────────────────────────────────────────────────────────
+# Using a static-ish path since we can't use command substitution in cat
+TEST_DIR="/tmp/color-tool-tests-repro"
+BIN_DIR="$TEST_DIR/bin"
+rm -rf "$TEST_DIR"
+mkdir -p "$BIN_DIR"
+
+# Mock notify-send
+cat <<'MOCK' > "$BIN_DIR/notify-send"
+#!/bin/bash
+echo "NOTIFY: $*" >> "/tmp/color-tool-tests-repro/notify.log"
+MOCK
+
+# Mock wl-copy
+cat <<'MOCK' > "$BIN_DIR/wl-copy"
+#!/bin/bash
+cat > "/tmp/color-tool-tests-repro/clipboard.txt"
+MOCK
+
+# Mock python3
+cat <<'MOCK' > "$BIN_DIR/python3"
+#!/bin/bash
+if [[ "$*" == *wl-colorpicker*.py ]]; then
+  echo "#aabbcc"
+  exit 0
+fi
+exec /usr/bin/python3 "$@"
+MOCK
+
+chmod +x "$BIN_DIR"/*
+
+# Symlink essential tools
+ln -s /usr/bin/bash "$BIN_DIR/bash"
+ln -s /usr/bin/cat "$BIN_DIR/cat"
+ln -s /usr/bin/printf "$BIN_DIR/printf"
+ln -s /usr/bin/echo "$BIN_DIR/echo"
+ln -s /usr/bin/sed "$BIN_DIR/sed"
+ln -s /usr/bin/grep "$BIN_DIR/grep"
+ln -s /usr/bin/awk "$BIN_DIR/awk"
+ln -s /usr/bin/jq "$BIN_DIR/jq"
+ln -s /usr/bin/readlink "$BIN_DIR/readlink"
+ln -s /usr/bin/dirname "$BIN_DIR/dirname"
+ln -s /usr/bin/mkdir "$BIN_DIR/mkdir"
+ln -s /usr/bin/mktemp "$BIN_DIR/mktemp"
+ln -s /usr/bin/chmod "$BIN_DIR/chmod"
+ln -s /usr/bin/rm "$BIN_DIR/rm"
+# Mock curl for thecolorapi.com
+cat <<'MOCK' > "$BIN_DIR/curl"
+#!/bin/bash
+if [[ "$*" == *"thecolorapi.com"* ]]; then
+  echo '{"name": {"value": "Mocked Color Name"}}'
+  exit 0
+fi
+exec /usr/bin/curl "$@"
+MOCK
+chmod +x "$BIN_DIR/curl"
+ln -s /usr/bin/mv "$BIN_DIR/mv"
+ln -s /usr/bin/tr "$BIN_DIR/tr"
+
+export PATH="$BIN_DIR"
+export HOME="$TEST_DIR"
+export WAYLAND_DISPLAY=wayland-0
+export XDG_CURRENT_DESKTOP=KDE
+
+# Resolve the absolute path of color-tool
+COLOR_TOOL="$(/usr/bin/readlink -f ./color-tool)"
+
+# ── Test Helpers ──────────────────────────────────────────────────────────────
+total=0
+passed=0
+
+it() {
+  local label="$1"
+  total=$((total + 1))
+  printf "Test: %s... " "$label"
+}
+
+assert_contains() {
+  local haystack="$1"
+  local needle="$2"
+  if [[ "$haystack" == *"$needle"* ]]; then
+    echo "PASS"
+    passed=$((passed + 1))
+  else
+    echo "FAIL"
+    echo "  Expected to find: $needle"
+    echo "  Actual output: $haystack"
+    return 1
+  fi
+}
+
+# ── Tests ─────────────────────────────────────────────────────────────────────
+
+# 1. Basic Conversion
+it "converts hex to rgb"
+output=$("$COLOR_TOOL" "#ffffff" --output rgb --no-copy --no-notify)
+assert_contains "$output" "rgb(255, 255, 255)"
+
+# 2. Multiple Formats
+it "handles multiple formats"
+output=$("$COLOR_TOOL" "#000000" --output hex,rgba --no-copy --no-notify)
+assert_contains "$output" "#000000 rgba(0, 0, 0, 1.0)"
+
+# 3. Clipboard Integration
+it "copies to clipboard (mocked)"
+rm -f "$TEST_DIR/clipboard.txt"
+"$COLOR_TOOL" "#ff0000" --copy --no-notify >/dev/null
+if [[ -f "$TEST_DIR/clipboard.txt" ]]; then
+  assert_contains "$(cat "$TEST_DIR/clipboard.txt")" "#ff0000"
+else
+  echo "FAIL (clipboard.txt not created)"
+fi
+
+# 4. Notification Integration
+it "sends notifications (mocked)"
+rm -f "$TEST_DIR/notify.log"
+"$COLOR_TOOL" "#00ff00" --notify --no-copy >/dev/null
+if [[ -f "$TEST_DIR/notify.log" ]]; then
+  assert_contains "$(cat "$TEST_DIR/notify.log")" "NOTIFY: -i color-picker color-tool #00ff00"
+else
+  echo "FAIL (notify.log not created)"
+fi
+
+# 5. Color Picker (Mocked)
+it "launches color picker and processes result"
+output=$("$COLOR_TOOL" --pick --no-copy --no-notify)
+assert_contains "$output" "#aabbcc"
+
+# 6. Config Loading
+it "loads defaults from config.toml"
+mkdir -p "$HOME/.config/color-tool"
+cat <<'CONF' > "$HOME/.config/color-tool/config.toml"
+[defaults]
+output = "rgba"
+CONF
+output=$("$COLOR_TOOL" "#ffffff" --no-copy --no-notify)
+assert_contains "$output" "rgba(255, 255, 255, 1.0)"
+
+# 7. Error Notification
+it "notifies on missing clipboard utility"
+rm -f "$BIN_DIR/wl-copy"
+rm -f "$TEST_DIR/notify.log"
+"$COLOR_TOOL" "#123456" --copy --notify 2>/dev/null >/dev/null
+if [[ -f "$TEST_DIR/notify.log" ]]; then
+  assert_contains "$(cat "$TEST_DIR/notify.log")" "NOTIFY: -u normal -i dialog-warning color-tool Missing clipboard utility"
+else
+  echo "FAIL (notify.log not created)"
+fi
+
+# 8. Config Management
+it "manages configuration via CLI"
+mkdir -p "$HOME/.config/color-tool"
+"$COLOR_TOOL" --reset-config >/dev/null
+
+# Test --set-config
+"$COLOR_TOOL" --set-config defaults output=rgba
+output=$("$COLOR_TOOL" --get-config)
+if [[ "$output" == *"output = \"rgba\""* ]]; then
+  # Test --reset-config
+  "$COLOR_TOOL" --reset-config >/dev/null
+  output=$("$COLOR_TOOL" --get-config)
+  assert_contains "$output" "output = \"hex\""
+else
+  echo "FAIL"
+  echo "  Expected to find: output = \"rgba\""
+  echo "  Actual output: $output"
+fi
+
+# 9. JSON Output
+it "outputs selected formats as JSON"
+output=$("$COLOR_TOOL" "#00ff00" --output hex,rgb --json --no-copy --no-notify)
+if [[ "$output" == *'"hex": "#00ff00"'* ]] && [[ "$output" == *'"rgb": "rgb(0, 255, 0)"'* ]]; then
+  echo "PASS"
+  passed=$((passed + 1))
+else
+  echo "FAIL"
+  echo "  Actual output: $output"
+fi
+
+# 10. Swatch Output
+it "outputs visual swatch"
+output=$("$COLOR_TOOL" "#112233" --output hex --swatch --no-copy --no-notify)
+if [[ "$output" == *"$(printf "\033[48;2;17;34;51m  \033[0m")"* ]] && [[ "$output" == *"#112233"* ]]; then
+  echo "PASS"
+  passed=$((passed + 1))
+else
+  echo "FAIL"
+  echo "  Actual output: $output"
+fi
+
+# 11. Color Name Fetching
+it "fetches color name from thecolorapi.com"
+output=$("$COLOR_TOOL" "#ffffff" --output hex --name --no-copy --no-notify)
+assert_contains "$output" "(Mocked Color Name)"
+
+# 12. Invalid Color Input
+it "rejects invalid colors"
+output=$("$COLOR_TOOL" "invalid-color" --no-copy --no-notify 2>&1 || true)
+assert_contains "$output" "Error: Invalid color: invalid-color"
+
+# 13. Invalid Format Input
+it "rejects invalid formats"
+output=$("$COLOR_TOOL" "#000000" --output badfmt --no-copy --no-notify 2>&1 || true)
+assert_contains "$output" "Error: Invalid output format: badfmt"
+
+# 14. Installation Dependency Warnings
+it "warns about missing dependencies during install"
+mkdir -p "$TEST_DIR/install_bin"
+for cmd in bash cat echo printf mkdir mktemp chmod rm cp ln dirname readlink ps; do
+  if [[ -e "/usr/bin/$cmd" || -L "/usr/bin/$cmd" ]]; then
+    /usr/bin/ln -s "/usr/bin/$cmd" "$TEST_DIR/install_bin/$cmd"
+  elif [[ -e "/bin/$cmd" || -L "/bin/$cmd" ]]; then
+    /usr/bin/ln -s "/bin/$cmd" "$TEST_DIR/install_bin/$cmd"
+  fi
+done
+output=$(PATH="$TEST_DIR/install_bin" HOME="$TEST_DIR" WAYLAND_DISPLAY="" XDG_CURRENT_DESKTOP="" KDE_FULL_SESSION="" "$COLOR_TOOL" --install 2>&1 || true)
+
+if [[ "$output" == *"python3 — required"* ]] &&
+   [[ "$output" == *"jq — required"* ]] &&
+   [[ "$output" == *"curl — required"* ]] &&
+   [[ "$output" == *"libnotify (notify-send)"* ]] &&
+   [[ "$output" == *"KDE Plasma on Wayland"* ]] &&
+   [[ "$output" == *"ImageMagick"* ]] &&
+   [[ "$output" == *"wl-clipboard"* ]]; then
+  echo "PASS"
+  passed=$((passed + 1))
+else
+  echo "FAIL"
+  echo "  Actual output: $output"
+fi
+
+# ── Cleanup ───────────────────────────────────────────────────────────────────
+echo "---------------------------------------"
+echo "Result: $passed/$total tests passed."
+rm -rf "$TEST_DIR"
+
+if [[ $passed -ne $total ]]; then
+  exit 1
+fi