Skip to content

fix: avoid kanban session_id index migration crash - #28816

Closed
liowald wants to merge 1 commit into
NousResearch:mainfrom
liowald:fix/kanban-session-id-migration
Closed

fix: avoid kanban session_id index migration crash#28816
liowald wants to merge 1 commit into
NousResearch:mainfrom
liowald:fix/kanban-session-id-migration

Conversation

@liowald

@liowald liowald commented May 19, 2026

Copy link
Copy Markdown

Summary

  • move idx_tasks_session_id creation out of the static schema script
  • create the index after the additive session_id migration has run
  • preserves the index for fresh databases and legacy migrated databases

Test Plan

  • python -m pytest tests/hermes_cli/test_kanban_db.py tests/hermes_cli/test_kanban_db_init.py tests/hermes_cli/test_kanban_boards.py -q -o 'addopts='

@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard labels May 19, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #28461 — same kanban session_id index migration fix. Already resolved by the comprehensive salvage in #28781 (merged). See also #28776, #28562, #28597, #28693 (all duplicates).

@teknium1

Copy link
Copy Markdown
Contributor

Closing as superseded by #28781.

Triage notes (high confidence):
Same idx_tasks_session_id-out-of-SCHEMA_SQL fix was merged via PR #28781 (kanban_db.py:1369 on main); PR is CONFLICTING and redundant.

Thanks for the contribution — the underlying problem this PR addresses has been resolved by the linked PR on current main. If you believe this was closed in error, please comment and we'll reopen.

(Bulk-closed during a CLI PR triage sweep.)

@teknium1 teknium1 closed this May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard 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