Skip to content

fix: migrate legacy kanban session indexes - #28797

Closed
mikebrownlee wants to merge 1 commit into
NousResearch:mainfrom
mikebrownlee:fix/kanban-session-indexes
Closed

fix: migrate legacy kanban session indexes#28797
mikebrownlee wants to merge 1 commit into
NousResearch:mainfrom
mikebrownlee:fix/kanban-session-indexes

Conversation

@mikebrownlee

Copy link
Copy Markdown

Summary

  • Migrate legacy kanban session indexes so older session records remain compatible.
  • Adds regression coverage for kanban DB initialisation/migration paths.

Test plan

  • Not run in this push-only step; this PR contains the existing local commit 9e8dfb7 only.

Safety notes

  • Pushed to fork branch mikebrownlee:fix/kanban-session-indexes.
  • No force push.
  • No new commits or code changes made during PR creation.

@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/plugins Plugin system and bundled plugins labels May 19, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #28461 — same kanban legacy DB migration fix (session_id index created before column exists). See #28464 for the root issue. Multiple prior duplicate PRs: #28620, #28693, #28741, #28754, #28776.

@teknium1

Copy link
Copy Markdown
Contributor

This has already been implemented on main by the broader Kanban migration salvage. Thanks for sending the fix — the root issue and duplicate PR discussion were useful signal.

Evidence from this automated hermes-sweeper review:

That shipped in v2026.5.28, so this duplicate PR can be closed.

@teknium1 teknium1 closed this Jun 15, 2026
@teknium1 teknium1 added the sweeper:implemented-on-main Sweeper: behavior already present on current main label Jun 15, 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 P3 Low — cosmetic, nice to have sweeper:implemented-on-main Sweeper: behavior already present on current main type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants