docs: collision cleanup no longer deletes — excluded rows age to inactive - #11
Merged
Merged
Conversation
…tive
Follows server 26df9c9, which removed Logic 1 (the prefix-overlap
duplicate delete) from repeaters_cleanup.php and widened Logic 3 to
enabled IN (1,2).
Collisions are now handled entirely by the collision system at ingest;
nothing is deleted on a 3× stale timer any more. Instead a colliding
repeater that goes silent is marked Inactive at repeater_inactive_days
(default 30), which drops it out of the recompute's live set — the
comparison only considers enabled 1|2 — so its partner is restored on its
next advert rather than being pinned at excluded forever.
Two user-visible changes, both worth being explicit about: the timeline
moves from 72 hours to 30 days, and the outcome from deleted to Inactive.
admins.md
- dropped the 72-hour row from the lifecycle table and the whole
"Duplicate collision cleanup" subsection, including the survivor
restore that no longer exists
- 3× stale now attributed to pending resolution alone
- Repeater Inactive After covers Excluded rows, and says why
- Excluded status and Bypass Auto Delete corrected — the latter no
longer skips a stale-duplicate delete, but does now hold a partner
excluded while it's away
duplicaterepeaterid.md
- Automatic Cleanup rewritten: nothing is deleted, the survivor returns
on its next advert, and a returning repeater is re-excluded if the ID
is still ambiguous
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KojjSrgSQQJW7Z8b3eakEA
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follows server
26df9c9, which removed Logic 1 (the prefix-overlap duplicate delete) fromrepeaters_cleanup.phpand widened Logic 3 toenabled IN (1, 2).What changed on the server
Collisions are now handled entirely by the collision system at ingest (
repeater_collision.php+backfill_repeater_collisions.php/fold_corrupt_twins.php). Nothing is deleted on a 3× stale timer any more.Instead, a colliding repeater that goes silent is marked Inactive at
repeater_inactive_days(default 30). That drops it out of the collision recompute's live set — which only considersenabled IN (1,2)— so its partner is restored on its next advert instead of being pinned at Excluded forever.Two user-visible changes: the timeline moves from 72 hours → 30 days, and the outcome from deleted → Inactive.
admins.mdduplicaterepeaterid.mdAutomatic Cleanup rewritten. It previously described deletion at 72 hours as the primary resolution path, which is now wrong end to end. Neither repeater is deleted; the survivor returns on its next advert; a returning repeater is re-excluded if the ID is still ambiguous.
🤖 Generated with Claude Code
https://claude.ai/code/session_01KojjSrgSQQJW7Z8b3eakEA