Skip to content

automate app-summary article and name-repeat quality checks#6507

Draft
razzeee wants to merge 4 commits intoflathub-infra:mainfrom
razzeee:automate-summary-quality-moderation-checks
Draft

automate app-summary article and name-repeat quality checks#6507
razzeee wants to merge 4 commits intoflathub-infra:mainfrom
razzeee:automate-summary-quality-moderation-checks

Conversation

@razzeee
Copy link
Copy Markdown
Member

@razzeee razzeee commented Apr 15, 2026

No description provided.

razzeee added 4 commits April 15, 2026 16:49
Mark app-summary-dont-start-with-an-article and
app-summary-dont-repeat-app-name as read_only in the DB and evaluate
them automatically in the update_quality_moderation worker.

Check logic lives in a new dependency-free module
(app/quality_moderation_checks.py) so it can be unit-tested without
requiring a broker or database. A new unittests job in backend_ci.yml
runs all unit tests (excluding the Docker-based smoke tests) on every
push/PR.
- Move test_quality_moderation.py to tests/unit/ so the CI job can
  point at that directory directly, with no fragile ignore list that
  breaks when other infra-dependent test files are added
- Add [tool.pytest.ini_options] pythonpath=["."] so 'app' is
  importable without installing the full project deps
- CI runs: uv run --only-group dev pytest tests/unit/ -v
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant