wup 0.2.48__tar.gz → 0.2.61__tar.gz

{wup-0.2.48/wup.egg-info → wup-0.2.61}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: wup
-Version: 0.2.48
+Version: 0.2.61
 Summary: WUP (What's Up) - Intelligent file watcher for regression testing in large projects
 Author-email: Tom Sapletta <tom@sapletta.com>
 License-Expression: Apache-2.0
@@ -31,17 +31,17 @@ Dynamic: license-file
 
 ## AI Cost Tracking
 
-![PyPI](https://img.shields.io/badge/pypi-costs-blue) ![Version](https://img.shields.io/badge/version-0.2.48-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
-![AI Cost](https://img.shields.io/badge/AI%20Cost-$3.35-orange) ![Human Time](https://img.shields.io/badge/Human%20Time-25.7h-blue) ![Model](https://img.shields.io/badge/Model-openrouter%2Fqwen%2Fqwen3--coder--next-lightgrey)
+![PyPI](https://img.shields.io/badge/pypi-costs-blue) ![Version](https://img.shields.io/badge/version-0.2.61-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
+![AI Cost](https://img.shields.io/badge/AI%20Cost-$3.49-orange) ![Human Time](https://img.shields.io/badge/Human%20Time-30.7h-blue) ![Model](https://img.shields.io/badge/Model-openrouter%2Fqwen%2Fqwen3--coder--next-lightgrey)
 
-- 🤖 **LLM usage:** $3.3508 (60 commits)
-- 👤 **Human dev:** ~$2573 (25.7h @ $100/h, 30min dedup)
+- 🤖 **LLM usage:** $3.4944 (74 commits)
+- 👤 **Human dev:** ~$3068 (30.7h @ $100/h, 30min dedup)
 
-Generated on 2026-05-24 using [openrouter/qwen/qwen3-coder-next](https://openrouter.ai/qwen/qwen3-coder-next)
+Generated on 2026-05-27 using [openrouter/qwen/qwen3-coder-next](https://openrouter.ai/qwen/qwen3-coder-next)
 
 ---
 
-![PyPI](https://img.shields.io/badge/pypi-wup-blue) ![Version](https://img.shields.io/badge/version-0.2.48-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
+![PyPI](https://img.shields.io/badge/pypi-wup-blue) ![Version](https://img.shields.io/badge/version-0.2.61-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
 
 **WUP (What's Up)** - Intelligent file watcher for regression testing in large projects.
 
@@ -512,6 +512,26 @@ python3 -m pytest tests/test_testql_watcher.py -v
 python3 -m pytest tests/ --cov=wup
 ```
 
+### Goal wrapper (local `.venv`)
+
+When `goal` is installed globally, it may inherit another project's `VIRTUAL_ENV`.
+Use the local wrapper to force `wup/.venv` before running `goal` commands:
+
+```bash
+# Default: runs goal -a
+bash scripts/goal-local
+
+# Explicit arguments
+bash scripts/goal-local -a
+bash scripts/goal-local --dry-run
+```
+
+If needed, point to a specific `goal` binary:
+
+```bash
+GOAL_BIN=/home/tom/.local/bin/goal bash scripts/goal-local -a
+```
+
 ### Examples
 
 ```bash

{wup-0.2.48 → wup-0.2.61}/README.md

@@ -3,17 +3,17 @@
 
 ## AI Cost Tracking
 
-![PyPI](https://img.shields.io/badge/pypi-costs-blue) ![Version](https://img.shields.io/badge/version-0.2.48-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
-![AI Cost](https://img.shields.io/badge/AI%20Cost-$3.35-orange) ![Human Time](https://img.shields.io/badge/Human%20Time-25.7h-blue) ![Model](https://img.shields.io/badge/Model-openrouter%2Fqwen%2Fqwen3--coder--next-lightgrey)
+![PyPI](https://img.shields.io/badge/pypi-costs-blue) ![Version](https://img.shields.io/badge/version-0.2.61-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
+![AI Cost](https://img.shields.io/badge/AI%20Cost-$3.49-orange) ![Human Time](https://img.shields.io/badge/Human%20Time-30.7h-blue) ![Model](https://img.shields.io/badge/Model-openrouter%2Fqwen%2Fqwen3--coder--next-lightgrey)
 
-- 🤖 **LLM usage:** $3.3508 (60 commits)
-- 👤 **Human dev:** ~$2573 (25.7h @ $100/h, 30min dedup)
+- 🤖 **LLM usage:** $3.4944 (74 commits)
+- 👤 **Human dev:** ~$3068 (30.7h @ $100/h, 30min dedup)
 
-Generated on 2026-05-24 using [openrouter/qwen/qwen3-coder-next](https://openrouter.ai/qwen/qwen3-coder-next)
+Generated on 2026-05-27 using [openrouter/qwen/qwen3-coder-next](https://openrouter.ai/qwen/qwen3-coder-next)
 
 ---
 
-![PyPI](https://img.shields.io/badge/pypi-wup-blue) ![Version](https://img.shields.io/badge/version-0.2.48-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
+![PyPI](https://img.shields.io/badge/pypi-wup-blue) ![Version](https://img.shields.io/badge/version-0.2.61-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
 
 **WUP (What's Up)** - Intelligent file watcher for regression testing in large projects.
 
@@ -484,6 +484,26 @@ python3 -m pytest tests/test_testql_watcher.py -v
 python3 -m pytest tests/ --cov=wup
 ```
 
+### Goal wrapper (local `.venv`)
+
+When `goal` is installed globally, it may inherit another project's `VIRTUAL_ENV`.
+Use the local wrapper to force `wup/.venv` before running `goal` commands:
+
+```bash
+# Default: runs goal -a
+bash scripts/goal-local
+
+# Explicit arguments
+bash scripts/goal-local -a
+bash scripts/goal-local --dry-run
+```
+
+If needed, point to a specific `goal` binary:
+
+```bash
+GOAL_BIN=/home/tom/.local/bin/goal bash scripts/goal-local -a
+```
+
 ### Examples
 
 ```bash

{wup-0.2.48 → wup-0.2.61}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "wup"
-version = "0.2.48"
+version = "0.2.61"
 description = "WUP (What's Up) - Intelligent file watcher for regression testing in large projects"
 readme = "README.md"
 requires-python = ">=3.9"

wup-0.2.61/tests/test_assistant.py

@@ -0,0 +1,180 @@
+"""Unit tests for WupAssistant."""
+import tempfile
+from pathlib import Path
+import pytest
+
+from wup.assistant import WupAssistant
+from wup.models.config import WupConfig, ProjectConfig, ServiceConfig
+
+
+def test_framework_detection_fastapi():
+    """Test auto-detecting FastAPI project framework."""
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        
+        # Create FastAPI characteristic files and content
+        main_py = root / "main.py"
+        main_py.write_text("from fastapi import FastAPI\napp = FastAPI()", encoding="utf-8")
+        
+        assistant = WupAssistant(str(root))
+        detected = assistant._detect_framework()
+        assert detected == "fastapi"
+
+
+def test_framework_detection_flask():
+    """Test auto-detecting Flask project framework."""
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        
+        # Create Flask characteristic files and content
+        app_py = root / "app.py"
+        app_py.write_text("from flask import Flask\napp = Flask(__name__)", encoding="utf-8")
+        
+        assistant = WupAssistant(str(root))
+        detected = assistant._detect_framework()
+        assert detected == "flask"
+
+
+def test_framework_detection_none():
+    """Test framework detection returns None when no markers match."""
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        assistant = WupAssistant(str(root))
+        detected = assistant._detect_framework()
+        assert detected is None
+
+
+def test_auto_detect_services_fastapi():
+    """Test auto-detecting services for a FastAPI project."""
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        
+        # Create a router folder under app
+        router_dir = root / "app" / "routers"
+        router_dir.mkdir(parents=True)
+        (router_dir / "users.py").write_text("# user routes", encoding="utf-8")
+        
+        assistant = WupAssistant(str(root))
+        services = assistant._auto_detect_services("fastapi")
+        
+        assert len(services) == 1
+        assert services[0].name == "users"
+        assert services[0].type == "auto"
+        assert services[0].paths[0].endswith("app/routers/users.py")
+
+
+def test_detect_service_type():
+    """Test service type detection based on name and path."""
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        assistant = WupAssistant(str(root))
+        
+        # Web service detection
+        assert assistant._detect_service_type("my-api", root) == "web"
+        assert assistant._detect_service_type("http-server", root) == "web"
+        
+        # Shell service detection
+        assert assistant._detect_service_type("cli-tool", root) == "shell"
+        assert assistant._detect_service_type("run-command", root) == "shell"
+        
+        # Default fallback
+        assert assistant._detect_service_type("unknown", root) == "auto"
+
+
+def test_validate_config_success():
+    """Test config validation passes on a valid config."""
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        
+        # Create a watch path to satisfy validator
+        src_dir = root / "src"
+        src_dir.mkdir()
+        
+        # Create scenario dir to satisfy validator
+        scenario_dir = root / "scenarios"
+        scenario_dir.mkdir()
+        
+        assistant = WupAssistant(str(root))
+        assistant.config = WupConfig(
+            project=ProjectConfig(
+                name="MyProj",
+                description="Valid project"
+            ),
+            services=[
+                ServiceConfig(name="web-service", type="web", paths=["src"])
+            ]
+        )
+        assistant.config.watch.paths = ["src/**"]
+        assistant.config.testql.scenario_dir = "scenarios"
+        
+        issues = assistant._validate_config()
+        assert len(issues) == 0
+
+
+def test_validate_config_issues():
+    """Test config validation detects common errors."""
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        assistant = WupAssistant(str(root))
+        
+        # 1. Empty/Missing project name
+        assistant.config.project.name = ""
+        # 2. No services configured
+        assistant.config.services = []
+        # 3. Non-existent watch path
+        assistant.config.watch.paths = ["non_existent_dir/**"]
+        # 4. Non-existent scenario directory
+        assistant.config.testql.scenario_dir = "missing_scenarios"
+        
+        issues = assistant._validate_config()
+        assert len(issues) == 4
+        assert "Project name is required" in issues
+        assert "No services configured" in issues
+        assert "Watch path does not exist" in issues[2]
+        assert "TestQL scenario directory not found" in issues[3]
+
+
+def test_generate_suggestions():
+    """Test assistant suggestions generation."""
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        assistant = WupAssistant(str(root))
+        
+        # Empty watch file types, single service, web dashboard disabled
+        assistant.config.services = [ServiceConfig(name="single", type="web")]
+        assistant.config.watch.file_types = []
+        assistant.config.web.enabled = False
+        assistant.config.testql.scenario_dir = "scenarios"
+        assistant.config.testql.smoke_scenario = ""
+        
+        suggestions = assistant._generate_suggestions()
+        assert len(suggestions) == 4
+        assert "Consider splitting into multiple services" in suggestions[0]
+        assert "Specify file types to avoid watching" in suggestions[1]
+        assert "Enable web dashboard for real-time monitoring" in suggestions[2]
+        assert "Set a smoke test scenario for quick health checks" in suggestions[3]
+
+
+def test_quick_setup():
+    """Test quick non-interactive setup."""
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        
+        # Create FastAPI marker to let quick setup detect it
+        main_py = root / "main.py"
+        main_py.write_text("from fastapi import FastAPI", encoding="utf-8")
+        
+        assistant = WupAssistant(str(root))
+        assistant.run(quick=True)
+        
+        # Should save wup.yaml
+        config_path = root / "wup.yaml"
+        assert config_path.exists()
+        
+        # Verify content of saved config
+        import yaml
+        saved_data = yaml.safe_load(config_path.read_text(encoding="utf-8"))
+        assert saved_data["project"]["name"] == root.name
+        assert "fastapi" in saved_data["project"]["description"].lower()
+        assert len(saved_data["services"]) > 0
+        assert saved_data["web"]["enabled"] is True

wup-0.2.61/tests/test_health_summary_passed.py

@@ -0,0 +1,53 @@
+"""Regression: fleet health treats all-pass summaries as success in non-strict mode."""
+
+from __future__ import annotations
+
+import asyncio
+import json
+import tempfile
+from pathlib import Path
+from subprocess import CompletedProcess
+
+from wup.models.config import ProjectConfig, TestQLConfig, WatchConfig, WupConfig
+from wup.testql_watcher import TestQLWatcher
+
+
+def test_health_summary_all_passed_parser() -> None:
+    assert TestQLWatcher._health_summary_all_passed("17/17 passed, 0 failed")
+    assert not TestQLWatcher._health_summary_all_passed("16/17 passed, 1 failed")
+
+
+def test_fleet_health_nonzero_exit_all_passed_counts_as_up() -> None:
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        scenario_dir = root / "testql-scenarios"
+        scenario_dir.mkdir(parents=True, exist_ok=True)
+        (scenario_dir / "fleet.testql.toon.yaml").write_text("name: fleet\n", encoding="utf-8")
+        (root / ".wup").mkdir(parents=True, exist_ok=True)
+
+        cfg = WupConfig(
+            project=ProjectConfig(name="demo"),
+            watch=WatchConfig(),
+            testql=TestQLConfig(
+                scenario_dir="testql-scenarios",
+                health_scenario="fleet.testql.toon.yaml",
+                health_scenario_strict=False,
+            ),
+        )
+        watcher = TestQLWatcher(
+            project_root=str(root),
+            deps_file=str(root / "deps.json"),
+            scenarios_dir="testql-scenarios",
+            track_dir=".wup/tracks",
+            config=cfg,
+        )
+        watcher._run_testql = lambda args, timeout: CompletedProcess(  # type: ignore[method-assign]
+            args=args,
+            returncode=1,
+            stdout='{"passed": 17, "failed": 0}',
+            stderr="",
+        )
+        assert asyncio.run(watcher._run_fleet_health_scenario()) is True
+        state = json.loads((root / ".wup" / "service-health.json").read_text(encoding="utf-8"))
+        assert state["demo"]["status"] == "up"
+        assert state["demo"]["stage"] == "health_scenario"

wup-0.2.61/tests/test_probe_mutex.py

@@ -0,0 +1,38 @@
+"""Regression: periodic probes do not overlap file-change quick/visual cycles."""
+
+from __future__ import annotations
+
+import io
+import threading
+from unittest.mock import patch
+
+from rich.console import Console
+
+from wup.models.config import ProjectConfig, ServiceConfig, WupConfig
+from wup.testql_watcher import TestQLWatcher
+
+
+def _minimal_watcher() -> TestQLWatcher:
+    watcher = TestQLWatcher.__new__(TestQLWatcher)
+    watcher.console = Console(file=io.StringIO(), width=120)
+    watcher.config = WupConfig(
+        project=ProjectConfig(name="t"),
+        services=[ServiceConfig(name="frontend", paths=["frontend"])],
+    )
+    watcher._watch_work_lock = threading.Lock()
+    watcher._periodic_probe_in_progress = False
+    return watcher
+
+
+def test_periodic_probe_skipped_when_watch_lock_held() -> None:
+    watcher = _minimal_watcher()
+    watcher._watch_work_lock.acquire()
+    try:
+        with patch("asyncio.run"):
+            watcher._run_periodic_probes_once()
+    finally:
+        watcher._watch_work_lock.release()
+
+    output = watcher.console.file.getvalue()
+    assert "skipped" in output.lower()
+    assert watcher._periodic_probe_in_progress is False

{wup-0.2.48 → wup-0.2.61}/tests/test_visual_diff_progress.py

@@ -2,8 +2,11 @@
 
 from __future__ import annotations
 
+import io
 import os
 
+from rich.console import Console
+
 from wup.models.config import VisualDiffConfig
 from wup.visual_diff import VisualDiffer
 
@@ -37,3 +40,17 @@ def test_progress_can_be_disabled_via_env(tmp_path, monkeypatch) -> None:
     monkeypatch.setenv("WUP_VISUAL_DIFF_PROGRESS", "0")
     differ = _make_differ(tmp_path)
     assert differ._build_progress("frontend", total=50) is None
+
+
+def test_progress_uses_injected_console(tmp_path) -> None:
+    custom = Console(file=io.StringIO(), width=80)
+    cfg = VisualDiffConfig(
+        enabled=True,
+        base_url="http://localhost:8100",
+        snapshot_dir=str(tmp_path / "snap"),
+        diff_dir=str(tmp_path / "diff"),
+    )
+    differ = VisualDiffer(str(tmp_path), cfg, console=custom)
+    progress = differ._build_progress("frontend", total=10)
+    assert progress is not None
+    assert progress.console is custom

wup-0.2.61/tests/test_watch_exclude.py

@@ -0,0 +1,34 @@
+"""Regression: nested test directories are ignored by the file watcher."""
+
+from __future__ import annotations
+
+from pathlib import Path
+
+from wup.core import WupWatcher
+from wup.models.config import ProjectConfig, WatchConfig, WupConfig
+
+
+def _watcher() -> WupWatcher:
+    cfg = WupConfig(
+        project=ProjectConfig(name="t"),
+        watch=WatchConfig(
+            exclude_patterns=["**/tests/**", "*.md"],
+            file_types=[".ts", ".py"],
+        ),
+    )
+    return WupWatcher(project_root=".", config=cfg)
+
+
+def test_nested_tests_directory_ignored() -> None:
+    watcher = _watcher()
+    assert watcher._is_file_ignored(Path("frontend/src/tests/setup.ts"))
+
+
+def test_src_file_not_ignored() -> None:
+    watcher = _watcher()
+    assert not watcher._is_file_ignored(Path("frontend/src/services/foo.ts"))
+
+
+def test_glob_exclude_pattern() -> None:
+    watcher = _watcher()
+    assert watcher._path_matches_exclude_pattern(Path("docs/readme.md"), "*.md")

{wup-0.2.48 → wup-0.2.61}/wup/__init__.py

@@ -7,7 +7,7 @@ WUP monitors file changes and runs intelligent regression tests using a 3-layer
 3. Detail Layer: Full tests with blame reports (only on failure)
 """
 
-__version__ = "0.2.48"
+__version__ = "0.2.61"
 __author__ = "Tom Sapletta"
 
 from .config import load_config, save_config, get_default_config

{wup-0.2.48 → wup-0.2.61}/wup/assistant.py

@@ -39,6 +39,16 @@ from .models.config import (
     WebConfig,
     WupConfig,
 )
+from .assistant_discovery import (
+    FRAMEWORK_PATTERNS,
+    detect_framework,
+    auto_detect_services,
+    detect_service_type,
+)
+from .assistant_validator import (
+    validate_config,
+    generate_suggestions,
+)
 
 # Import ServiceType for type checking
 if False:  # TYPE_CHECKING
@@ -50,33 +60,7 @@ console = Console()
 class WupAssistant:
     """Interactive configuration assistant."""
     
-    # Framework detection patterns
-    FRAMEWORK_PATTERNS = {
-        'fastapi': {
-            'files': ['main.py', 'app/main.py'],
-            'content': ['FastAPI', 'from fastapi', 'app = FastAPI'],
-            'services': ['app/routers/*', 'app/routes/*', 'routes/*'],
-            'default_services': ['web', 'api'],
-        },
-        'flask': {
-            'files': ['app.py', 'wsgi.py', 'application.py'],
-            'content': ['Flask', 'from flask', 'app = Flask'],
-            'services': ['app/*/__init__.py', 'blueprints/*'],
-            'default_services': ['web', 'admin'],
-        },
-        'django': {
-            'files': ['manage.py', 'settings.py'],
-            'content': ['Django', 'from django', 'INSTALLED_APPS'],
-            'services': ['*/apps.py', '*/models.py'],
-            'default_services': ['models', 'views', 'tasks'],
-        },
-        'express': {
-            'files': ['server.js', 'app.js'],
-            'content': ['express', 'require("express")', "require('express')"],
-            'services': ['routes/*', 'controllers/*'],
-            'default_services': ['api', 'web'],
-        },
-    }
+    FRAMEWORK_PATTERNS = FRAMEWORK_PATTERNS
     
     def __init__(self, project_root: str = "."):
         self.project_root = Path(project_root).resolve()
@@ -192,59 +176,15 @@ class WupAssistant:
     
     def _detect_framework(self) -> Optional[str]:
         """Auto-detect project framework."""
-        for framework, patterns in self.FRAMEWORK_PATTERNS.items():
-            # Check for characteristic files
-            for file in patterns['files']:
-                if (self.project_root / file).exists():
-                    # Verify content
-                    content = (self.project_root / file).read_text()
-                    if any(marker in content for marker in patterns['content']):
-                        return framework
-        return None
+        return detect_framework(self.project_root)
     
     def _auto_detect_services(self, framework: str) -> List[ServiceConfig]:
         """Auto-detect services based on framework patterns."""
-        services = []
-        patterns = self.FRAMEWORK_PATTERNS.get(framework, {})
-        
-        for pattern in patterns.get('services', []):
-            for path in self.project_root.rglob(pattern):
-                if path.is_dir() or path.is_file():
-                    service_name = path.parent.name if path.name == '__init__.py' else path.stem
-                    
-                    # Detect service type
-                    svc_type = self._detect_service_type(service_name, path)
-                    
-                    services.append(ServiceConfig(
-                        name=service_name,
-                        type=svc_type,
-                        paths=[str(path.parent if path.name == '__init__.py' else path)],
-                    ))
-        
-        return services
+        return auto_detect_services(self.project_root, framework)
     
     def _detect_service_type(self, name: str, path: Path) -> ServiceType:
         """Detect service type from name and path."""
-        name_lower = name.lower()
-        
-        # Web indicators
-        if any(x in name_lower for x in ['web', 'api', 'http', 'rest', 'router', 'route']):
-            return 'web'
-        
-        # Shell indicators
-        if any(x in name_lower for x in ['shell', 'cli', 'cmd', 'command']):
-            return 'shell'
-        
-        # Check directory contents
-        if path.is_dir():
-            files = list(path.iterdir())
-            has_html = any(f.suffix in ['.html', '.htm'] for f in files)
-            has_routes = any('route' in f.name.lower() for f in files)
-            
-            if has_html or has_routes:
-                return 'web'
-        
-        return 'auto'
+        return detect_service_type(name, path)
     
     def _configure_services(self):
         """Interactive service configuration."""
@@ -549,51 +489,11 @@ class WupAssistant:
     
     def _validate_config(self) -> List[str]:
         """Validate current configuration."""
-        issues = []
-        
-        # Check project name
-        if not self.config.project.name:
-            issues.append("Project name is required")
-        
-        # Check services
-        if not self.config.services:
-            issues.append("No services configured")
-        
-        for svc in self.config.services:
-            if not svc.name:
-                issues.append("Service with empty name found")
-        
-        # Check watch paths exist
-        for path in self.config.watch.paths:
-            resolved = self.project_root / path.replace('/**', '').replace('/*', '')
-            if not resolved.exists():
-                issues.append(f"Watch path does not exist: {path}")
-        
-        # Check TestQL
-        if self.config.testql.scenario_dir:
-            scenario_path = self.project_root / self.config.testql.scenario_dir
-            if not scenario_path.exists():
-                issues.append(f"TestQL scenario directory not found: {self.config.testql.scenario_dir}")
-        
-        return issues
+        return validate_config(self.config, self.project_root)
     
     def _generate_suggestions(self) -> List[str]:
         """Generate helpful suggestions."""
-        suggestions = []
-        
-        if len(self.config.services) == 1:
-            suggestions.append("Consider splitting into multiple services for better granularity")
-        
-        if not self.config.watch.file_types:
-            suggestions.append("Specify file types to avoid watching unnecessary files")
-        
-        if not self.config.web.enabled:
-            suggestions.append("Enable web dashboard for real-time monitoring and notifications")
-        
-        if self.config.testql.scenario_dir and not self.config.testql.smoke_scenario:
-            suggestions.append("Set a smoke test scenario for quick health checks")
-        
-        return suggestions
+        return generate_suggestions(self.config)
     
     def _save_configuration(self):
         """Save configuration to wup.yaml."""