Skip to content

fix(kanban): tolerate concurrent DB migrations - #22715

Closed
hook-7 wants to merge 1 commit into
NousResearch:mainfrom
hook-7:fix/kanban-db-migration-race
Closed

fix(kanban): tolerate concurrent DB migrations#22715
hook-7 wants to merge 1 commit into
NousResearch:mainfrom
hook-7:fix/kanban-db-migration-race

Conversation

@hook-7

@hook-7 hook-7 commented May 9, 2026

Copy link
Copy Markdown

Summary

  • add an idempotent helper for optional SQLite migration columns
  • treat duplicate-column races from concurrent Hermes processes as no-ops
  • cover stale PRAGMA snapshot behavior in kanban migration tests

Test Plan

  • venv/bin/python -m pytest tests/hermes_cli/test_kanban_core_functionality.py -k 'legacy_migration_both_columns_already_present'\n\nNote: scripts/run_tests.sh ... could not run because the configured venv python has no pip module for installing pytest-split.

@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/plugins Plugin system and bundled plugins duplicate This issue or pull request already exists labels May 9, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #22800 (merged). The kanban concurrent migration fix was already merged as a salvage of #22627.

@hook-7

hook-7 commented May 9, 2026

Copy link
Copy Markdown
Author

Thanks for the heads-up. Confirmed this is covered by #22800, which merged the kanban concurrent migration fix as a salvage of #22627. Closing this PR as a duplicate.

@hook-7

hook-7 commented May 9, 2026

Copy link
Copy Markdown
Author

Closing as duplicate of merged #22800.

@hook-7 hook-7 closed this May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/plugins Plugin system and bundled plugins duplicate This issue or pull request already exists 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.

2 participants