diff --git a/.github/workflows/quality-gate.yml b/.github/workflows/quality-gate.yml index af251ab..2ef089c 100644 --- a/.github/workflows/quality-gate.yml +++ b/.github/workflows/quality-gate.yml @@ -57,11 +57,38 @@ jobs: restore-keys: | ${{ runner.os }}-pip- - - name: ๐Ÿ“ฆ Install Dependencies + - name: ๐Ÿ“ฆ Install Development Dependencies run: | python -m pip install --upgrade pip + pip install -r requirements-dev.txt + + - name: ๐Ÿ“ฆ Install Runtime Dependencies + run: | pip install -r ${{ steps.prepare-requirements.outputs.PROCESSED_REQUIREMENTS_FILE }} - pip install pytest pytest-cov pytest-asyncio + + - name: ๐Ÿ“ Code Formatting Check (Ruff) + run: | + echo "Checking code formatting..." + ruff format --check laborious/ tests/ + continue-on-error: false + + - name: ๐Ÿ”Ž Code Linting (Ruff) + run: | + echo "Running linting checks..." + ruff check laborious/ tests/ + continue-on-error: false + + - name: ๐Ÿท๏ธ Type Checking (mypy) + run: | + echo "Running type checks..." + mypy laborious/ + continue-on-error: true + + - name: ๐Ÿ”’ Security Analysis (Bandit) + run: | + echo "Running security analysis..." + bandit -r laborious/ -ll -q + continue-on-error: true - name: ๐Ÿงช Run Tests with Pytest run: | diff --git a/validate.sh b/validate.sh index 1cd4474..2199a3b 100755 --- a/validate.sh +++ b/validate.sh @@ -11,6 +11,27 @@ YELLOW='\033[1;33m' BLUE='\033[0;34m' NC='\033[0m' # No Color +# Args +FIX_MODE=false +while [[ $# -gt 0 ]]; do + case "$1" in + --fix) + FIX_MODE=true + shift + ;; + -h|--help) + echo "Usage: $0 [--fix]" + echo " --fix Apply Ruff auto-fixes (format and lint fixes)." + exit 0 + ;; + *) + echo -e "${RED}Unknown option: $1${NC}" + echo "Usage: $0 [--fix]" + exit 2 + ;; + esac +done + echo -e "${BLUE}โ•”โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•—${NC}" echo -e "${BLUE}โ•‘ Model Manager - Code Validation Suite โ•‘${NC}" echo -e "${BLUE}โ•šโ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•โ•${NC}" @@ -47,12 +68,16 @@ run_step() { FAILED_STEPS=() # Step 1: Code Formatting Check (Ruff) -if ! run_step "1. Code Formatting (Ruff)" "ruff format orchestrator/ tests/ && ruff format --check orchestrator/ tests/"; then +# - default: check only +# - --fix: write changes +if ! run_step "1. Code Formatting (Ruff)" "if \$FIX_MODE; then ruff format orchestrator/ tests/; else ruff format --check orchestrator/ tests/; fi"; then FAILED_STEPS+=("Code Formatting") fi # Step 2: Linting (Ruff) -if ! run_step "2. Code Linting (Ruff)" "ruff check --fix orchestrator/ tests/"; then +# - default: check only +# - --fix: apply autofixes +if ! run_step "2. Code Linting (Ruff)" "if \$FIX_MODE; then ruff check --fix orchestrator/ tests/; else ruff check orchestrator/ tests/; fi"; then FAILED_STEPS+=("Linting") fi