Freegle / Iznik / 11307

73%

master: 70%

Build Type

Pull #401

circleci

Committed by edwh

Commit Message

feat(batch): migrate chat_process.php to chats:process-incoming

Process pending chat messages (processingrequired=1) in Laravel batch.
Mirrors V1 cron/chat_process.php + ChatMessage::process().

IncomingMailService creates messages with processingrequired=1 (with comment
'Background chat_process.php cron handles visibility, roster, push
notifications'). Without this migration those incoming mail replies were
invisible to ChatNotificationService (which filters on processingrequired=0
AND processingsuccessful=1).

Key operations per message:
- Spam/ban checks for User2User chats (spam_users, users_banned, chatmodstatus)
- Review cascade: hold new message if previous message in chat is under review
- Mark processingrequired=0, processingsuccessful=1 (or 0 on failure)
- Update sender's chat_roster (lastmsgseen/lastmsgemailed)
- Reopen CLOSED roster entries (not BLOCKED) after new activity

Scheduler entry commented out in routes/console.php pending sign-off.
10 tests covering all transitions.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Pull Request Pull Request #401: feat(batch): migrate chat_process.php to chats:process-incoming

Coverage Stats

13699 of 20568 branches covered (66.6%)

Branch coverage included in aggregate %.

230 of 279 new or added lines in 4 files covered. (82.44%)

19 existing lines in 4 files now uncovered.

101296 of 137329 relevant lines covered (73.76%)

22.56 hits per line