replit/timmy-tower
2
0
Fork 0
Code Issues 26 Pull Requests 5 Actions Packages Projects Releases Wiki Activity

[claude] Fix moderation infinite re-review loop (#27) #71

Merged
claude merged 1 commits from claude/issue-27 into main 2026-03-23 00:44:58 +00:00
Conversation 0 Commits 1 Files Changed 4 +14 -4
claude commented 2026-03-23 00:44:12 +00:00
Collaborator
Copy Link

Fixes #27

Summary

  • Added flagged status to QUEUE_STATUSES enum in the schema
  • When AI flags an event in autoReview(), status now transitions to flagged instead of staying pending
  • processPending() queries WHERE status = pending, so flagged events are naturally excluded from future poll cycles
  • Admin stats endpoint now includes flagged count
  • Added DB migration with index on relay_event_queue.status for efficient filtering

Root cause

Line 179-183 of moderation.ts set reviewReason and reviewedBy but left status as pending. Every 30s poll cycle would re-send flagged events to Claude Haiku indefinitely.

Test plan

  • Verify flagged events no longer appear in processPending() queries
  • Verify admin panel can list status=flagged events via GET /admin/relay/queue?status=flagged
  • Verify existing approve/reject/auto_approve flows unchanged
  • Run migration on staging DB
Fixes #27 ## Summary - Added `flagged` status to `QUEUE_STATUSES` enum in the schema - When AI flags an event in `autoReview()`, status now transitions to `flagged` instead of staying `pending` - `processPending()` queries `WHERE status = pending`, so flagged events are naturally excluded from future poll cycles - Admin stats endpoint now includes `flagged` count - Added DB migration with index on `relay_event_queue.status` for efficient filtering ## Root cause Line 179-183 of `moderation.ts` set `reviewReason` and `reviewedBy` but left `status` as `pending`. Every 30s poll cycle would re-send flagged events to Claude Haiku indefinitely. ## Test plan - [ ] Verify flagged events no longer appear in `processPending()` queries - [ ] Verify admin panel can list `status=flagged` events via GET `/admin/relay/queue?status=flagged` - [ ] Verify existing approve/reject/auto_approve flows unchanged - [ ] Run migration on staging DB
claude added 1 commit 2026-03-23 00:44:12 +00:00
fix: break moderation infinite re-review loop by adding 'flagged' status
Some checks failed
CI / Typecheck & Lint (pull_request) Failing after 1s
Details
ce3d6ffb4d 
When AI flags an event, transition status to 'flagged' instead of
leaving it as 'pending'. This prevents processPending() from picking
up the same flagged events every 30-second poll cycle and burning
AI tokens indefinitely.

- Add 'flagged' to QUEUE_STATUSES enum in schema
- Set status='flagged' in autoReview() when AI flags an event
- Include flagged count in admin stats endpoint
- Add index on relay_event_queue.status for efficient queries

Fixes #27

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
claude merged commit 5954a2fdc0 into main 2026-03-23 00:44:58 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
ai
backend
frontend
lightning
mobile
mobile
nostr
workshop
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claude gemini replit
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: replit/timmy-tower#71
Delete Branch "claude/issue-27"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?