fix(collab): de feed-loop per team stopt nu met zijn team - #7
Merged
Merged
Conversation
…ls wees collab-launch.sh startte per team een inline `while true` die messages.jsonl naar feed.txt kopieert, zonder enige stopconditie. disbandTeam schreef wel de .finished-marker maar liet die loop met rust, en collab-cleanup.sh haalt alleen mappen weg. Op 01-09-2026 draaiden er zestien, de oudste elf dagen, voor teams die de service niet meer kende; ze zijn toen met de hand gekild. De loop staat nu in scripts/collab-poller.sh en houdt zelf op zodra zijn team voorbij is: bij de .finished-marker (na een laatste flush), bij een verdwenen runtime-map, wanneer de service 404 geeft of het team disbanded meldt, en na COLLAB_POLLER_MAX_API_FAILURES mislukte checks op rij als de service weg is. Hij schrijft zijn eigen poller.pid en ruimt die bij vertrek op. disbandTeam stuurt hem daarnaast meteen een SIGTERM, zodat een disband nooit een loop achterlaat, ook niet tot de volgende tick. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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
1 participant
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.
Wat
collab-launch.shstartte per team een inlinewhile truediemessages.jsonlnaarfeed.txtkopieert, zonder stopconditie.disbandTeamschreef wel de.finished-marker maar liet die loop met rust, encollab-cleanup.shhaalt alleen mappen weg. Op 01-09-2026 draaiden er zestien, de oudste elf dagen, voor teams die de service niet meer kende.De loop staat nu in
scripts/collab-poller.shen houdt zelf op zodra zijn team voorbij is:.finished-marker, na een laatste flush zodat de afscheidsberichten nog in de feed komen;disbandedmeldt (eens per minuut gevraagd);COLLAB_POLLER_MAX_API_FAILURESmislukte checks op rij (standaard 10) als de service weg is.Hij schrijft zijn eigen
poller.piden ruimt die bij vertrek op.disbandTeamstuurt hem daarnaast meteen een SIGTERM, zodat een disband nooit een loop achterlaat tot de volgende tick. De intervallen zijn instelbaar viaCOLLAB_POLL_SECSenCOLLAB_POLLER_CHECK_EVERY, vooral voor de tests.Test
tests/collab-poller.test.tsdraait het echte script tegen een nep-service en dekt: kopiëren naar de feed, pid-bestand schrijven en opruimen, en elk van de stopcondities hierboven, plus dat een kort onbereikbare service hem niet laat stoppen. Een aparte test controleert datdisbandTeamhet proces uitpoller.pidbeëindigt, en een statische test datcollab-launch.shhet script gebruikt en de inline loop weg is.npm test(103 tests),npm run typecheckennpm run lintzijn groen.🤖 Generated with Claude Code