Repository navigation
chore: enforce Black formatting in pre-commit and CI to prevent drift #3408
Description
Summary
89 Python files drifted from Black formatting standards and were discovered via CI check (2026-04-03). The files were reformatted manually in commit `76ac6c7e8`. Without enforcement, this will recur.
Discovery
Found during routine session — Black `--check` on `autobot-backend/` and `autobot-slm-backend/` reported 89 files needing reformat. No pre-commit hook was preventing the drift.
Proposed Fix
- Verify `.pre-commit-config.yaml` includes Black with auto-format on staged files
- Ensure all developers run `pre-commit install` after cloning
- Confirm CI pipeline runs `black --check` as a blocking step
- Consider adding `isort --profile black` check alongside Black
Acceptance Criteria
- Pre-commit hook runs Black on staged `.py` files before every commit
- CI `black --check` step is present and blocking
- No files fail `black --check` after changes are applied
Activity
style(imports): fix isort ordering on 13 Python files (#3408)
style(imports): fix isort ordering on 13 Python files (#3408)
style: apply Black formatting to 89 Python files (#3408)
github-actions commentedon Apr 3, 2026
PR #3409 (merged to
Dev_new_gui) references this issue with a close keyword.If this issue is fully resolved, close it manually. If work remains, no action is needed.
chore: enforce Black + isort formatting in pre-commit and CI (#3408)
fix(chore): align Black/isort line-length to 120 to match flake8, res…
chore: enforce Black + isort formatting in pre-commit and CI (#3571)
mrveiss commentedon Apr 6, 2026
Fixed in PR #3571 (merged).
chore(ci): enforce Black + isort formatting in pre-commit and CI (#3408)
chore(ci): align isort line_length with Black at 120 in pyproject.toml (
mrveiss commentedon Apr 7, 2026
Fixed and merged.
fix: fetch user/group names from API in ShareKnowledgeDialog (#3990)
fix: lazy-load KnowledgeGraph3D to defer Three.js loading (#4017)
feat(perf): memoize analytics computations (#4036) (#4079)
fix(install): validate critical services exist before Ansible deploym…