Skip to content

fix: auto-block repeated kanban retries#20881

Closed
mwnickerson wants to merge 1 commit into
NousResearch:mainfrom
mwnickerson:forge/kanban-retry-loop-guard
Closed

fix: auto-block repeated kanban retries#20881
mwnickerson wants to merge 1 commit into
NousResearch:mainfrom
mwnickerson:forge/kanban-retry-loop-guard

Conversation

@mwnickerson

Copy link
Copy Markdown
Contributor

Summary

  • Adds a Kanban retry-loop guard so repeated timeout/spawn-failure patterns stop respawning indefinitely
  • Blocks/gives up after the configured threshold with human-visible event trail and metadata
  • Resets failure counters on reassignment so legitimate retries are preserved

Verification

  • git diff --check
  • python -m py_compile hermes_cli/kanban_db.py hermes_cli/kanban.py gateway/run.py hermes_cli/config.py
  • python -m pytest tests/hermes_cli/test_kanban_core_functionality.py -q -k 'spawn_failure_auto_blocks_after_limit or reassign_resets_failure_counter or repeated_timeouts_auto_block' -o 'addopts='

Note: Forge reported full tests/hermes_cli/test_kanban_core_functionality.py has unrelated existing state-isolation failures: test_detect_crashed_workers_reclaims, test_board_stats, test_cli_create_on_fresh_home_auto_inits.

@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/gateway Gateway runner, session dispatch, delivery comp/cron Cron scheduler and job management labels May 6, 2026
@teknium1

teknium1 commented May 7, 2026

Copy link
Copy Markdown
Contributor

Merged via #21183 with your commit cherry-picked onto current main — authorship preserved in git log via rebase merge. Thanks @mwnickerson! The aggressive default (5 → 2) and config plumbing both land as-is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cron Cron scheduler and job management comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants