Skip to content

Simplify meme preprocessing and upload failure handling - #396

Merged
ohld merged 5 commits into
productionfrom
codex/untangle-code
Nov 9, 2026
Merged

ohld merged 5 commits into
productionfrom
codex/untangle-code

Conversation

@ohld

@ohld ohld commented Nov 9, 2026

Copy link
Copy Markdown
Member

Meme upload moderation imported image preprocessing through a Prefect ingestion flow, and batch failure handling repeated the same stop rule. This change puts watermarking behind one bytes-returning storage function used by both callers, unifies failure accounting, and removes the single-use upload-source wrapper. Upload source access now imports its focused repository module directly.

Behavior is preserved: JPEG output, non-image passthrough, invalid-image handling, five consecutive failures stopping a batch, reset on success, and uploader source reuse. Production code shrinks by 29 lines; no dependencies or migrations are added.

Validation:

  • Three independent reviews: ponytail-audit, thermo-nuclear-code-quality-review, improve-codebase-architecture; no blockers after simplifications.
  • Full suite: 601 passed, 2 skipped on Python 3.14 with disposable PostgreSQL/Redis.
  • Ruff check/format, public-repo audit, Dockerfile.prod build passed.
  • Synced with current production security updates before validation.

@ohld
ohld merged commit f0dc408 into production Nov 9, 2026
2 checks passed
@ohld
ohld deleted the codex/untangle-code branch November 9, 2026 13:37
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