Merge pull request #64 from NikiforovAll/feature/63-marketplace-ux

feat: Marketplace UX improvements and automated quality gates

Author: NikiforovAll

.claude/hooks/quality-gates.sh

@@ -0,0 +1,41 @@
+#!/bin/bash
+# Hook: Run quality gates after Claude finishes responding
+# Triggered by: Stop hook event
+# Exit 2 + stderr = blocks stop, shows output to Claude
+# Filters checks based on changed file types
+
+# Read input from stdin
+input=$(cat)
+
+# Check if stop hook is already active (prevent infinite loops)
+if echo "$input" | grep -q '"stop_hook_active":true'; then
+  exit 0
+fi
+
+# Check for changed Python files (staged or unstaged)
+python_changed=$(git diff --name-only HEAD 2>/dev/null | grep -E '\.py$' || true)
+python_staged=$(git diff --cached --name-only 2>/dev/null | grep -E '\.py$' || true)
+
+if [ -z "$python_changed" ] && [ -z "$python_staged" ]; then
+  # No Python files changed, skip quality gates
+  exit 0
+fi
+
+# Run the quality gates script with passed arguments
+if [ -f ".claude/skills/quality-gates/scripts/check.sh" ]; then
+  output=$(bash ".claude/skills/quality-gates/scripts/check.sh" "$@" 2>&1)
+  exit_code=$?
+
+  if [ $exit_code -eq 0 ]; then
+    # Silent on success
+    exit 0
+  else
+    # Pass the full output back to Claude via stderr
+    echo "Quality gates failed. Please fix the issues:" >&2
+    echo "" >&2
+    echo "$output" >&2
+    exit 2
+  fi
+fi
+
+exit 0

.claude/settings.json

@@ -10,5 +10,18 @@
     ],
     "deny": [],
     "ask": []
+  },
+  "hooks": {
+    "Stop": [
+      {
+        "hooks": [
+          {
+            "type": "command",
+            "command": "bash .claude/hooks/quality-gates.sh --lint --format --mypy",
+            "timeout": 120
+          }
+        ]
+      }
+    ]
   }
 }

.claude/skills/quality-gates/scripts/check.sh

@@ -1,29 +1,81 @@
 #!/usr/bin/env bash
-# Quality Gates - Run all code quality checks for lazyclaude
+# Quality Gates - Run code quality checks for lazyclaude
+# Usage: check.sh [--lint] [--format] [--mypy] [--test] [--all]
+#   --lint    Run ruff lint
+#   --format  Run ruff format
+#   --mypy    Run mypy type check
+#   --test    Run pytest
+#   --all     Run all checks (default if no flags)
 set -e
 
 cd "$(git rev-parse --show-toplevel)"
 
+RUN_LINT=false
+RUN_FORMAT=false
+RUN_MYPY=false
+RUN_TEST=false
+
+# Parse arguments
+if [[ $# -eq 0 ]]; then
+  # No args = run all checks
+  RUN_LINT=true
+  RUN_FORMAT=true
+  RUN_MYPY=true
+  RUN_TEST=true
+fi
+
+for arg in "$@"; do
+  case $arg in
+    --lint)   RUN_LINT=true ;;
+    --format) RUN_FORMAT=true ;;
+    --mypy)   RUN_MYPY=true ;;
+    --test)   RUN_TEST=true ;;
+    --all)
+      RUN_LINT=true
+      RUN_FORMAT=true
+      RUN_MYPY=true
+      RUN_TEST=true
+      ;;
+  esac
+done
+
 echo "=== Running Quality Gates ==="
 echo ""
 
-echo "[1/4] Ruff Lint (with auto-fix)..."
-uv run ruff check src tests --fix
-echo "OK"
-echo ""
+step=1
 
-echo "[2/4] Ruff Format..."
-uv run ruff format src tests
-echo "OK"
-echo ""
+if $RUN_LINT; then
+  echo "[$step] Ruff Lint (with auto-fix)..."
+  uv run ruff check src tests --fix
+  echo "OK"
+  echo ""
+  ((step++))
+fi
 
-echo "[3/4] Mypy Type Check..."
-uv run mypy src
-echo "OK"
-echo ""
+if $RUN_FORMAT; then
+  echo "[$step] Ruff Format..."
+  uv run ruff format src tests
+  echo "OK"
+  echo ""
+  ((step++))
+fi
 
-echo "[4/4] Pytest..."
-uv run pytest tests/ -q
-echo ""
+if $RUN_MYPY; then
+  echo "[$step] Mypy Type Check..."
+  uv run mypy src
+  echo "OK"
+  echo ""
+  ((step++))
+fi
+
+if $RUN_TEST; then
+  echo -n "[$step] Pytest... "
+  uv run pytest tests/ -q --tb=no --no-header 2>&1 | tail -1
+  pytest_status=${PIPESTATUS[0]}
+  echo ""
+  if [[ $pytest_status -ne 0 ]]; then
+    exit "$pytest_status"
+  fi
+fi
 
 echo "=== All Quality Gates Passed ==="

src/lazyclaude/app.py

@@ -9,7 +9,6 @@
 from textual.app import App, ComposeResult
 from textual.containers import Container
 from textual.theme import Theme
-from textual.widgets import Footer
 
 from lazyclaude import __version__
 from lazyclaude.bindings import APP_BINDINGS
@@ -34,12 +33,15 @@
 from lazyclaude.services.marketplace_loader import MarketplaceLoader
 from lazyclaude.services.settings import SettingsService
 from lazyclaude.themes import CUSTOM_THEMES
+from lazyclaude.widgets.app_footer import AppFooter
 from lazyclaude.widgets.combined_panel import CombinedPanel
 from lazyclaude.widgets.delete_confirm import DeleteConfirm
 from lazyclaude.widgets.detail_pane import MainPane
 from lazyclaude.widgets.filter_input import FilterInput
 from lazyclaude.widgets.level_selector import LevelSelector
+from lazyclaude.widgets.marketplace_confirm import MarketplaceConfirm
 from lazyclaude.widgets.marketplace_modal import MarketplaceModal
+from lazyclaude.widgets.marketplace_source_input import MarketplaceSourceInput
 from lazyclaude.widgets.plugin_confirm import PluginConfirm
 from lazyclaude.widgets.status_panel import StatusPanel
 from lazyclaude.widgets.type_panel import TypePanel
@@ -100,7 +102,10 @@ def __init__(
         self._plugin_confirm: PluginConfirm | None = None
         self._delete_confirm: DeleteConfirm | None = None
         self._marketplace_modal: MarketplaceModal | None = None
+        self._marketplace_confirm: MarketplaceConfirm | None = None
+        self._marketplace_source_input: MarketplaceSourceInput | None = None
         self._marketplace_loader: MarketplaceLoader | None = None
+        self._app_footer: AppFooter | None = None
         self._help_visible = False
         self._last_focused_panel: TypePanel | None = None
         self._last_focused_combined: bool = False
@@ -158,7 +163,16 @@ def compose(self) -> ComposeResult:
         self._marketplace_modal = MarketplaceModal(id="marketplace-modal")
         yield self._marketplace_modal
 
-        yield Footer()
+        self._marketplace_confirm = MarketplaceConfirm(id="marketplace-confirm")
+        yield self._marketplace_confirm
+
+        self._marketplace_source_input = MarketplaceSourceInput(
+            id="marketplace-source-input"
+        )
+        yield self._marketplace_source_input
+
+        self._app_footer = AppFooter(id="app-footer")
+        yield self._app_footer
 
     def on_mount(self) -> None:
         """Handle mount event - load customizations."""
@@ -199,6 +213,20 @@ def check_action(
         if action == "exit_preview":
             return self._plugin_preview_mode
 
+        marketplace_blocked_actions = {
+            "filter_all",
+            "filter_user",
+            "filter_project",
+            "filter_plugin",
+            "toggle_plugin_enabled_filter",
+        }
+        if (
+            self._marketplace_modal
+            and self._marketplace_modal.is_visible
+            and action in marketplace_blocked_actions
+        ):
+            return False
+
         if self._plugin_preview_mode:
             preview_allowed_actions = {
                 "quit",
@@ -414,6 +442,11 @@ def on_filter_input_filter_changed(
         self._last_focused_panel = None
         if self._main_pane:
             self._main_pane.customization = None
+        search_active = bool(message.query)
+        if self._status_panel:
+            self._status_panel.search_active = search_active
+        if self._app_footer:
+            self._app_footer.search_active = search_active
         self._update_panels()
         self._update_subtitle()
 
@@ -426,6 +459,10 @@ def on_filter_input_filter_cancelled(
         self._last_focused_panel = None
         if self._main_pane:
             self._main_pane.customization = None
+        if self._status_panel:
+            self._status_panel.search_active = False
+        if self._app_footer:
+            self._app_footer.search_active = False
         self._update_panels()
         self._update_subtitle()
         self.refresh_bindings()

src/lazyclaude/mixins/filtering.py

@@ -6,6 +6,7 @@
 
 if TYPE_CHECKING:
     from lazyclaude.services.discovery import ConfigDiscoveryService
+    from lazyclaude.widgets.app_footer import AppFooter
     from lazyclaude.widgets.detail_pane import MainPane
     from lazyclaude.widgets.filter_input import FilterInput
     from lazyclaude.widgets.status_panel import StatusPanel
@@ -21,6 +22,7 @@ class FilterMixin:
     _main_pane: "MainPane | None"
     _filter_input: "FilterInput | None"
     _status_panel: "StatusPanel | None"
+    _app_footer: "AppFooter | None"
     _discovery_service: "ConfigDiscoveryService"
 
     def action_filter_all(self) -> None:
@@ -73,6 +75,11 @@ def action_toggle_plugin_enabled_filter(self) -> None:
         self._last_focused_panel = None
         if self._main_pane:
             self._main_pane.customization = None
+        disabled_active = self._plugin_enabled_filter is None
+        if self._status_panel:
+            self._status_panel.disabled_filter_active = disabled_active
+        if self._app_footer:
+            self._app_footer.disabled_filter_active = disabled_active
         self._update_panels()  # type: ignore[attr-defined]
         self._update_subtitle()  # type: ignore[attr-defined]
 
@@ -82,7 +89,7 @@ def action_search(self) -> None:
             self._filter_input.show()
 
     def _update_status_filter(self, level: str) -> None:
-        """Update status panel filter level and path display."""
+        """Update status panel and footer filter level and path display."""
         if self._status_panel:
             self._status_panel.filter_level = level
             if level == "User":
@@ -96,3 +103,5 @@ def _update_status_filter(self, level: str) -> None:
             else:
                 project_name = self._discovery_service.project_root.name
                 self._status_panel.config_path = project_name
+        if self._app_footer:
+            self._app_footer.filter_level = level