Skip to content

fix: handle a matching comment only after Reddit's unmarked-edit window - #104

Closed
Toldry wants to merge 1 commit into
masterfrom
claude/project-thread-ujqmzo
Closed

Toldry wants to merge 1 commit into
masterfrom
claude/project-thread-ujqmzo

Conversation

@Toldry

@Toldry Toldry commented Oct 19, 2026

Copy link
Copy Markdown
Owner

Requested by Querent · project thread

Before: a comment that was exactly r/name got an auxiliary-bot reply within seconds. Reddit lets authors edit a comment for about 3 minutes without marking it as edited, so it could become r/trucklodytes makes me so mad... with same_subreddit_bot's "Yes, that's where we are." under it, as if the bot had ignored the whole-body rule.

After: Phase 1 handles a matching comment 4 minutes after it was written, and checks its text as it is then. Auxiliary replies come about 4 minutes later than before.

The matching rule itself was never loose: every version of subreddit_regex since 2020 requires the whole body to be the reference, so neither reported comment can have matched in its final form.

How: Phase1Workers.submit() takes a not_before timestamp (created_utc + EDIT_GRACE_SECONDS), and a worker waits for it with shutdown.stop_requested_within() before fetching the comment, so shutdown is still immediate. Comments arrive roughly in creation order, so waiting on the oldest never holds back a due one. AGENTS.md and a changelog fragment are updated.


Generated by Claude Code

Reddit lets an author edit a comment for about 3 minutes without marking
it as edited. A bare "r/name" was answered within seconds and then often
extended ("r/trucklodytes makes me so mad..."), leaving a reply that the
final text would never have earned. Phase 1 workers now wait until 4
minutes after the comment was written, then fetch and check it.

Co-Authored-By: Toldry <toldry@gmail.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WLkGZFH1VQLrPQ59LmMepx
@Toldry Toldry self-assigned this Oct 19, 2026
@Toldry
Toldry marked this pull request as ready for review October 19, 2026 19:57
@Toldry Toldry closed this Oct 19, 2026
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.

2 participants