Skip to content

fix(kanban): migrate session_id before indexing#28597

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

fix(kanban): migrate session_id before indexing#28597
a-beatle wants to merge 1 commit into
NousResearch:mainfrom
a-beatle:fix/kanban-session-id-migration

Conversation

@a-beatle

Copy link
Copy Markdown

Summary

  • Move idx_tasks_session_id creation out of the initial schema script so legacy Kanban boards add the nullable session_id column before indexing it.
  • Add regression coverage for pre-session_id Kanban databases.
  • Keep kanban-worker spawn tests aligned with resolvable bundled skill behavior.

Test Plan

  • python -m pytest tests/hermes_cli/test_kanban_db.py tests/hermes_cli/test_kanban_core_functionality.py -q

Move idx_tasks_session_id creation out of the initial schema script so legacy boards add the nullable session_id column before creating the index. Add regression coverage for pre-session_id databases.
@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 comp/plugins Plugin system and bundled plugins labels May 19, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #28461 — both move session_id index creation from SCHEMA_SQL to after migration. Also competes with #28562. All three fix #28464.

@teknium1

Copy link
Copy Markdown
Contributor

Closing as already fixed on main.

Triage notes (high confidence):
origin/main hermes_cli/kanban_db.py already removes idx_tasks_session_id from SCHEMA_SQL and recreates it after the additive ALTER TABLE migration (kanban_db.py:1369). The migrate-before-index fix is in main.

If you still see this on the latest version, please reopen with reproduction steps.

(Bulk-closed during a CLI 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 comp/plugins Plugin system and bundled plugins 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