simplify: Remove black, mypy, and bandit from linting

Per request, simplified the linting setup by removing:
- black (code formatter)
- mypy (type checker)
- bandit (Python security linter)

Kept:
- ruff (fast Python linter for basic checks)
- ansible-lint
- yamllint
- shellcheck
- safety (dependency vulnerability scanner)

This provides a good balance of code quality checks without
being overly restrictive or requiring code style changes.

Author: dguido

.github/workflows/lint.yml

@@ -56,23 +56,13 @@ jobs:
       - name: Install Python linters
         run: |
           python -m pip install --upgrade pip
-          pip install ruff black mypy
+          pip install ruff
 
       - name: Run ruff
         run: |
           # Fast Python linter
           ruff check . || true  # Start with warnings only
 
-      - name: Check Python formatting with black
-        run: |
-          # Check formatting (don't modify)
-          black --check --diff . || true  # Start with warnings only
-
-      - name: Run mypy
-        run: |
-          # Type checking for Python
-          mypy library/ --ignore-missing-imports || true  # Start with warnings only
-
   shellcheck:
     name: Shell script linting
     runs-on: ubuntu-22.04
@@ -102,12 +92,7 @@ jobs:
       - name: Install security tools
         run: |
           python -m pip install --upgrade pip
-          pip install bandit safety
-
-      - name: Run bandit
-        run: |
-          # Security linter for Python
-          bandit -r library/ || true  # Start with warnings only
+          pip install safety
 
       - name: Check dependencies with safety
         run: |

docs/linting.md

@@ -22,13 +22,10 @@ The project uses multiple linters to ensure code quality across different file t
   - `partial-become`: Avoid unnecessary privilege escalation
 
 ### 2. Python Linting
-- **Tools**: 
-  - `ruff` - Fast Python linter (replaces flake8, isort, etc.)
-  - `black` - Code formatter
-  - `mypy` - Type checker
-  - `bandit` - Security linter
+- **Tool**: `ruff` - Fast Python linter (replaces flake8, isort, etc.)
 - **Config**: `pyproject.toml`
 - **Style**: 120 character line length, Python 3.10+
+- **Checks**: Syntax errors, imports, code style
 
 ### 3. Shell Script Linting
 - **Tool**: `shellcheck`
@@ -42,7 +39,6 @@ The project uses multiple linters to ensure code quality across different file t
 
 ### 5. Security Scanning
 - **Tools**:
-  - `bandit` - Python security issues
   - `safety` - Known vulnerabilities in dependencies
   - `zizmor` - GitHub Actions security (run separately)
 
@@ -68,8 +64,6 @@ ansible-lint -v *.yml roles/{local,cloud-*}/*/*.yml
 
 # Python
 ruff check .
-black --check .
-mypy library/
 
 # Shell
 find . -name "*.sh" -exec shellcheck {} \;
@@ -78,7 +72,6 @@ find . -name "*.sh" -exec shellcheck {} \;
 yamllint .
 
 # Security
-bandit -r library/
 safety check
 ```
 

pyproject.toml

@@ -13,31 +13,4 @@ select = [
 ]
 ignore = [
     "E501", # line too long (handled by formatter)
-]
-
-[tool.black]
-line-length = 120
-target-version = ['py310']
-include = '\.pyi?$'
-extend-exclude = '''
-(
-  /(
-      \.eggs
-    | \.git
-    | \.mypy_cache
-    | \.tox
-    | \.venv
-    | _build
-    | buck-out
-    | build
-    | dist
-  )/
-)
-'''
-
-[tool.mypy]
-python_version = "3.10"
-warn_return_any = true
-warn_unused_configs = true
-disallow_untyped_defs = false
-ignore_missing_imports = true
+]