Greptile Summary

This PR fixes two related bugs in the Makefile. First, the python3 detection used python3 -c "import sys" which always produced empty stdout (the command succeeds silently), so ifneq(...,) never matched — meaning PYTHON always fell through to the bare python fallback on non-Windows systems without a venv. Replacing it with command -v python3 correctly captures the interpreter path. Second, the lint, format-check, and format targets now invoke $(PYTHON) -m ruff instead of bare ruff, aligning them with how the rest of the Makefile invokes Python tooling and making them work reliably after a pip install into a venv where ruff may not be on PATH as a standalone binary.

Confidence Score: 5/5

Safe to merge — both changes are correct bug fixes with no functional regressions.

The Python detection fix correctly addresses a real silent bug (empty-output condition that never matched), and the $(PYTHON) -m ruff change is a straightforward alignment with existing Makefile conventions. No P0/P1 issues found.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[make lint / format-check / format] --> B{.venv/bin/python exists?}
    B -- Yes --> C[PYTHON = .venv/bin/python]
    B -- No --> D{OS == Windows_NT?}
    D -- Yes --> E{.venv/Scripts/python.exe exists?}
    E -- Yes --> F[PYTHON = .venv\\Scripts\\python.exe]
    E -- No --> G[PYTHON = python]
    D -- No --> H{"command -v python3\n(NEW: was: python3 -c 'import sys')"}
    H -- Found --> I[PYTHON = python3]
    H -- Not found --> J[PYTHON = python]
    C & F & G & I & J --> K["$(PYTHON) -m ruff check / format app/ tests/\n(NEW: was: bare ruff)"]

_{Reviews (1): Last reviewed commit: "fix(dev): run ruff through configured Py..." | Re-trigger Greptile}

💜 One more reason the project grows. Thanks @Genmin — your contribution just landed!

👋 Join us on Discord - OpenSRE : hang out, contribute, or hunt for features and issues. Everyone's welcome.