Cleanup/code quality #1

2026-07-18T17:23:44+02:00

froxxxy commented

2026-07-18 17:23:44 +02:00

No description provided.

froxxxy added 9 commits 2026-07-18 17:23:44 +02:00

docs: add cleanup plan from phase-0 analysis 90094cf8f8

Phase 0 inventory: ~7 Python modules, no tests/CI/lint, pipeline.py
concentration. Roadmap for incremental cleanup without behavior changes.

Co-authored-by: Cursor <cursoragent@cursor.com>

chore: add pyproject.toml with ruff and pytest config e9f494ed9d

Pin Python 3.11 for parity with the Docker image. Dev deps live in
requirements-dev.txt for local lint/test without rembg/onnxruntime.

Co-authored-by: Cursor <cursoragent@cursor.com>

style: apply ruff format and extend gitignore 0963297815

Ruff auto-fixes import order and line wrapping. Gitignore covers
build artifacts and coverage output from future test runs.

Co-authored-by: Cursor <cursoragent@cursor.com>

refactor: extract atomic JSON helpers to io_utils b63a796907

Centralize read_json/write_json_atomic used by pipeline meta I/O and
worker processed.json bookkeeping. Preserves corrupt-file warning in
load_processed().

Co-authored-by: Cursor <cursoragent@cursor.com>

refactor: share env-bool parsing and logging setup 5a3118f590

env_utils.env_bool replaces duplicated truthy parsing in config and
rembg_cli. logging_config.configure_logging unifies entry-point setup.

Co-authored-by: Cursor <cursoragent@cursor.com>

chore: deduplicate compose environment blocks 86989c1a4c

Use YAML anchor x-app-env for the shared worker/web environment vars.

Co-authored-by: Cursor <cursoragent@cursor.com>

refactor: named preprocess timeout and single meta read 1cf4ea0f07

Add PREPROCESS_TIMEOUT config (default 120s). Expose read_meta() and use
one read per job in list_jobs() instead of read_status + read_manifest.

Co-authored-by: Cursor <cursoragent@cursor.com>

test: add characterization tests for core modules c29186a227

33 tests covering env/io helpers, config parsers, pipeline meta I/O,
worker inbox logic, and FastAPI smoke routes. No gmic/rembg subprocesses.

Co-authored-by: Cursor <cursoragent@cursor.com>

docs: update README and add ARCHITECTURE

CI / lint-and-test (pull_request) Successful in 2m3s

Details

fd024d31e9

Dev setup (venv, ruff, pytest), config table entries, architecture
overview, and cleanup completion report.

Co-authored-by: Cursor <cursoragent@cursor.com>

froxxxy merged commit 9a9b143f08 into main

2026-07-18 17:26:22 +02:00

froxxxy deleted branch cleanup/code-quality

2026-07-18 17:28:00 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: froxxxy/livef12rocks#1