chore: add pre-commit hook and fix release workflow

  - Add pre-commit hook for ruff linting
  - Add 'make install-hooks' target
  - Fix release.yml: use ref_name, add error handling, fix git refs

Author: d7omdev

.githooks/pre-commit

@@ -0,0 +1,38 @@
+#!/bin/bash
+# Pre-commit hook to run linting on staged Python files
+
+echo "Running pre-commit lint checks..."
+
+# Get list of staged Python files
+STAGED_PY_FILES=$(git diff --cached --name-only --diff-filter=ACM | grep '\.py$' || true)
+
+if [ -z "$STAGED_PY_FILES" ]; then
+    echo "No Python files staged, skipping lint check."
+    exit 0
+fi
+
+echo "Linting staged Python files:"
+echo "$STAGED_PY_FILES"
+echo ""
+
+# Check if ruff is available
+if command -v ruff &> /dev/null; then
+    RUFF_CMD="ruff"
+elif [ -d "venv" ] && [ -f "venv/bin/ruff" ]; then
+    RUFF_CMD="venv/bin/ruff"
+else
+    echo "Error: ruff not found. Please install it or ensure venv is set up."
+    exit 1
+fi
+
+# Run ruff check on staged files
+if ! echo "$STAGED_PY_FILES" | xargs "$RUFF_CMD" check; then
+    echo ""
+    echo "❌ Linting failed! Please fix the errors above before committing."
+    echo "   Run 'make lint' for full details."
+    exit 1
+fi
+
+echo ""
+echo "✅ Linting passed!"
+exit 0

.github/workflows/release.yml

@@ -27,11 +27,12 @@ jobs:
       - name: Generate changelog
         id: changelog
         run: |
-          PREV_TAG=$(git describe --tags --abbrev=0 ${{ github.ref }}^ 2>/dev/null || echo "")
+          set -e
+          PREV_TAG=$(git describe --tags --abbrev=0 HEAD^ 2>/dev/null || echo "")
           if [ -z "$PREV_TAG" ]; then
-            CHANGELOG=$(git log --pretty=format:"- %s" ${{ github.ref }})
+            CHANGELOG=$(git log --pretty=format:"- %s" ${{ github.ref_name }})
           else
-            CHANGELOG=$(git log --pretty=format:"- %s" $PREV_TAG..${{ github.ref }})
+            CHANGELOG=$(git log --pretty=format:"- %s" $PREV_TAG..${{ github.ref_name }})
           fi
           CHANGELOG=$(echo "$CHANGELOG" | grep -vE "^(chore|ci|docs):" || echo "- Initial release")
           echo "changelog<<EOF" >> $GITHUB_OUTPUT
@@ -71,6 +72,7 @@ jobs:
 
       - name: Build
         run: |
+          set -e
           python3 -m venv --system-site-packages venv
           . venv/bin/activate
           pip install -U pip Nuitka==2.6.9 ordered-set==4.1.0 zstandard==0.23.0
@@ -100,6 +102,7 @@ jobs:
 
       - name: Install AppImage tools
         run: |
+          set -e
           sudo apt-get update
           sudo apt-get install -y \
             libfuse2 wget patchelf desktop-file-utils
@@ -111,6 +114,7 @@ jobs:
 
       - name: Prepare AppDir
         run: |
+          set -e
           mkdir -p AppDir/usr/bin
           mv dist/clipse-gui-v${{ needs.generate-changelog.outputs.version }}-linux-x86_64 \
              AppDir/usr/bin/clipse-gui
@@ -136,6 +140,7 @@ jobs:
 
       - name: Build AppImage
         run: |
+          set -e
           export ARCH=x86_64
           ./linuxdeploy-x86_64.AppImage \
             --appdir AppDir \
@@ -163,9 +168,11 @@ jobs:
 
       - name: Build in ARM64 container
         run: |
+          set -e
           docker run --rm --platform linux/arm64 \
             -v ${{ github.workspace }}:/workspace \
             -w /workspace arm64v8/ubuntu:22.04 bash -c "
+              set -e
               apt-get update &&
               apt-get install -y python3 python3-pip python3-venv python3-dev \
                 python3-gi python3-gi-cairo gir1.2-gtk-3.0 \
@@ -207,8 +214,9 @@ jobs:
 
       - name: Build
         run: |
-          pip install pyinstaller pillow
-          pyinstaller --onefile \
+          set -e
+          python -m pip install pyinstaller pillow
+          python -m PyInstaller --onefile \
             --name clipse-gui-v${{ needs.generate-changelog.outputs.version }}-macos-${{ matrix.arch }} \
             --hidden-import=gi \
             --collect-all gi \
@@ -236,7 +244,7 @@ jobs:
 
       - uses: softprops/action-gh-release@v2
         with:
-          tag_name: ${{ github.ref }}
+          tag_name: ${{ github.ref_name }}
           name: "Release v${{ needs.generate-changelog.outputs.version }}"
           body: ${{ needs.generate-changelog.outputs.release_body }}
           files: |

Makefile

@@ -39,13 +39,14 @@ NUITKA_OPTS := \
 
 help:
 	@echo "Available targets:"
-	@echo "  run        - Run the Clipse GUI from source"
-	@echo "  lint       - Run linting and type checking"
-	@echo "  nuitka     - Build a standalone binary using Nuitka"
-	@echo "  install    - Install built binary and assets system-wide"
-	@echo "  uninstall  - Uninstall the application"
-	@echo "  clean      - Clean build and temp files"
-	@echo "  bump       - Interactively bump version (major/minor/patch)"
+	@echo "  run           - Run the Clipse GUI from source"
+	@echo "  lint          - Run linting and type checking"
+	@echo "  install-hooks - Install git pre-commit hook for linting"
+	@echo "  nuitka        - Build a standalone binary using Nuitka"
+	@echo "  install       - Install built binary and assets system-wide"
+	@echo "  uninstall     - Uninstall the application"
+	@echo "  clean         - Clean build and temp files"
+	@echo "  bump          - Interactively bump version (major/minor/patch)"
 
 run:
 	@echo "Running Clipse GUI..."
@@ -134,3 +135,9 @@ bump:
 	@echo "Bumping version..."
 	@$(PYTHON) bump_version.py
 
+install-hooks:
+	@echo "Installing git hooks..."
+	@cp .githooks/pre-commit .git/hooks/pre-commit
+	@chmod +x .git/hooks/pre-commit
+	@echo "Pre-commit hook installed successfully!"
+

clipse_gui/controller.py

@@ -32,7 +32,6 @@
     show_help_window,
     show_preview_window,
     show_settings_window,
-    create_pin_icon,
     animate_pin_shake,
 )
 from .ui_builder import build_main_window_content

clipse_gui/tray_manager.py

@@ -74,7 +74,7 @@ def _setup_status_icon(self):
             self.status_icon.connect("activate", self._on_tray_activate)
             self.status_icon.connect("popup-menu", self._on_tray_popup_menu)
             self.status_icon.set_visible(False)
-        except Exception as e:
+        except Exception:
             self.status_icon = None
 
     def _create_basic_menu(self):
@@ -109,7 +109,7 @@ def _update_menu_with_items(self):
                 and hasattr(self.application.controller, "data_manager")
             ):
                 items = self.application.controller.data_manager.load_history()
-        except:
+        except Exception:
             pass
 
         recent_items = items[:5] if items else []
@@ -171,7 +171,7 @@ def _add_item_to_menu(self, item, index):
             )
             menu_item.show()
             self.menu.append(menu_item)
-        except:
+        except Exception:
             pass
 
     def _copy_item_to_clipboard(self, item):
@@ -187,7 +187,7 @@ def _copy_item_to_clipboard(self, item):
                     controller.copy_image_to_clipboard(file_path)
                 else:
                     controller.copy_text_to_clipboard(value)
-        except:
+        except Exception:
             pass
 
     def _on_tray_activate(self, status_icon):

clipse_gui/ui_components.py

@@ -1,10 +1,27 @@
 import os
 import json
 import logging
-from io import BytesIO
 from gi.repository import Gtk, Gdk, Pango, GdkPixbuf, GLib
 
 from .utils import format_date
+from .constants import (
+    LIST_ITEM_IMAGE_WIDTH,
+    LIST_ITEM_IMAGE_HEIGHT,
+    DEFAULT_PREVIEW_TEXT_WIDTH,
+    DEFAULT_PREVIEW_TEXT_HEIGHT,
+    DEFAULT_PREVIEW_IMG_WIDTH,
+    DEFAULT_PREVIEW_IMG_HEIGHT,
+    DEFAULT_HELP_WIDTH,
+    DEFAULT_HELP_HEIGHT,
+    PROTECT_PINNED_ITEMS,
+    COMPACT_MODE,
+    HOVER_TO_SELECT,
+    ENTER_TO_PASTE,
+    MINIMIZE_TO_TRAY,
+    config,
+)
+
+log = logging.getLogger(__name__)
 
 # SVG icon data for pushpin (rotated 25 degrees to the right for a natural look)
 PIN_SVG_BASE = """<?xml version="1.0" encoding="UTF-8"?>
@@ -77,24 +94,6 @@ def apply_wiggle(index):
 
     apply_wiggle(0)
 
-log = logging.getLogger(__name__)
-from .constants import (
-    LIST_ITEM_IMAGE_WIDTH,
-    LIST_ITEM_IMAGE_HEIGHT,
-    DEFAULT_PREVIEW_TEXT_WIDTH,
-    DEFAULT_PREVIEW_TEXT_HEIGHT,
-    DEFAULT_PREVIEW_IMG_WIDTH,
-    DEFAULT_PREVIEW_IMG_HEIGHT,
-    DEFAULT_HELP_WIDTH,
-    DEFAULT_HELP_HEIGHT,
-    PROTECT_PINNED_ITEMS,
-    COMPACT_MODE,
-    HOVER_TO_SELECT,
-    ENTER_TO_PASTE,
-    MINIMIZE_TO_TRAY,
-    config,
-)
-
 
 def create_list_row_widget(
     item_info,