conf: ai scaffolding tier1

Author: pawelpaszki

.github/workflows/lint.yml

@@ -0,0 +1,29 @@
+name: Lint
+
+on:
+  pull_request:
+    branches: [ main, ray-jobs-feature ]
+  push:
+    branches: [ main, ray-jobs-feature ]
+
+jobs:
+  ruff:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Set up python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.12'
+
+      - name: Install poetry
+        run: pip install poetry
+
+      - name: Install dependencies
+        run: |
+          poetry config virtualenvs.create false
+          poetry install --only dev
+
+      - name: Run ruff linter
+        run: ruff check .

.pre-commit-config.yaml

@@ -9,8 +9,9 @@ repos:
     -   id: check-yaml
         args: [--allow-multiple-documents]
     -   id: check-added-large-files
--   repo: https://github.com/psf/black
-    rev: 23.3.0
+-   repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.9.0
     hooks:
-      - id: black
-        language_version: python3.9
+      - id: ruff
+        args: [--fix, --exit-non-zero-on-fix]
+      - id: ruff-format

AGENTS.md

@@ -72,10 +72,16 @@ coverage report -m --include="path/to/changed1.py,path/to/changed2.py"
 
 ```sh
 # Format a single file
-black path/to/file.py
+ruff format path/to/file.py
 
 # Check formatting without modifying
-black --check path/to/file.py
+ruff format --check path/to/file.py
+
+# Lint a single file (with auto-fix)
+ruff check --fix path/to/file.py
+
+# Lint a single file (check only, no changes)
+ruff check path/to/file.py
 ```
 
 ### Coverage Requirements
@@ -88,7 +94,8 @@ black --check path/to/file.py
 
 ### Python Style
 
-- **Formatter**: black (via pre-commit)
+- **Formatter**: ruff-format (via pre-commit)
+- **Linter**: ruff (pycodestyle, pyflakes)
 - **Naming**: snake_case for functions/variables/modules, PascalCase for classes
 - **Type hints**: required for function parameters and return types
 - **Docstrings**: Google-style (Args, Returns, Raises sections)
@@ -139,7 +146,7 @@ Pre-commit hooks enforce:
 - end-of-file newline
 - YAML validation
 - Large file checks
-- black formatting
+- ruff linting and formatting
 
 ## Cursor Rules (extended guidance)
 

demo-notebooks/additional-demos/mnist.py

@@ -18,7 +18,6 @@
 import torch
 from pytorch_lightning import LightningModule, Trainer
 from pytorch_lightning.callbacks.progress import TQDMProgressBar
-from pytorch_lightning.loggers import CSVLogger
 from torch import nn
 from torch.nn import functional as F
 from torch.utils.data import DataLoader, random_split

demo-notebooks/guided-demos/mnist.py

@@ -18,7 +18,6 @@
 import torch
 from pytorch_lightning import LightningModule, Trainer
 from pytorch_lightning.callbacks.progress import TQDMProgressBar
-from pytorch_lightning.loggers import CSVLogger
 from torch import nn
 from torch.nn import functional as F
 from torch.utils.data import DataLoader, random_split

demo-notebooks/guided-demos/mnist_disconnected.py

@@ -18,7 +18,6 @@
 import torch
 from pytorch_lightning import LightningModule, Trainer
 from pytorch_lightning.callbacks.progress import TQDMProgressBar
-from pytorch_lightning.loggers import CSVLogger
 from torch import nn
 from torch.nn import functional as F
 from torch.utils.data import DataLoader, random_split

demo-notebooks/guided-demos/notebook-ex-outputs/mnist.py

@@ -18,7 +18,6 @@
 import torch
 from pytorch_lightning import LightningModule, Trainer
 from pytorch_lightning.callbacks.progress import TQDMProgressBar
-from pytorch_lightning.loggers import CSVLogger
 from torch import nn
 from torch.nn import functional as F
 from torch.utils.data import DataLoader, random_split

demo-notebooks/guided-demos/preview_nbs/mnist.py

@@ -18,7 +18,6 @@
 import torch
 from pytorch_lightning import LightningModule, Trainer
 from pytorch_lightning.callbacks.progress import TQDMProgressBar
-from pytorch_lightning.loggers import CSVLogger
 from torch import nn
 from torch.nn import functional as F
 from torch.utils.data import DataLoader, random_split

poetry.lock

@@ -75,6 +75,42 @@ webdriver-manager = "4.0.2"
 diff-cover = "^9.6.0"
 black = ">=25.11,<27.0"
 pre-commit = "^4.5.0"
+ruff = "^0.9.0"
+
+[tool.ruff]
+target-version = "py312"
+line-length = 88  # Match black
+extend-exclude = [
+    "*.ipynb",  # Exclude Jupyter notebooks (different execution context)
+]
+
+[tool.ruff.lint]
+select = [
+    "F",   # pyflakes (undefined names, unused imports)
+    "E",   # pycodestyle errors
+    # "I",   # isort (import sorting) - disabled due to circular import issues
+]
+ignore = [
+    "E501",  # line too long (handled by black/ruff-format)
+    "E402",  # module level import not at top of file
+    "E722",  # bare except (TODO: fix in future PR)
+    "F403",  # star imports (TODO: refactor in future PR)
+    "F405",  # may be undefined from star imports
+]
+
+[tool.ruff.lint.per-file-ignores]
+"src/codeflare_sdk/vendored/**" = ["E", "F"]  # Exclude vendored code
+"**/test_*.py" = ["F401", "F811", "F841"]  # Allow unused imports, redefinitions, and variables in tests
+"**/__init__.py" = ["F401"]  # Allow unused imports in __init__.py (re-exports for public API)
+"demo-notebooks/**/*.py" = ["F841"]  # Allow unused variables in demo notebooks
+"tests/**/*.py" = ["F841"]  # Allow unused variables in test scripts
+"scripts/**/*.py" = ["F841"]  # Allow unused variables in scripts
+"ui-tests/**/*.py" = ["F821"]  # Allow undefined names in UI test config
+
+[tool.ruff.lint.isort]
+known-first-party = ["codeflare_sdk"]
+force-single-line = false
+lines-after-imports = 2
 
 [tool.pytest.ini_options]
 filterwarnings = [