Skip to content

fix(kanban): serialize DB initialization (#24410) - #28383

Merged
teknium1 merged 1 commit into
mainfrom
hermes/hermes-de55f5dd
May 19, 2026
Merged

fix(kanban): serialize DB initialization (#24410)#28383
teknium1 merged 1 commit into
mainfrom
hermes/hermes-de55f5dd

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Salvages #24410 by @psionic73.

serializes kanban_db.connect() schema init with reentrant lock; main has no init lock (only threading is in dispatcher); 64 LOC + regression test, well-scoped

Cherry-picked onto current main with original authorship preserved via rebase merge.

@teknium1
teknium1 merged commit ca8126b into main May 19, 2026
@teknium1
teknium1 deleted the hermes/hermes-de55f5dd branch May 19, 2026 03:17
@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/plugins Plugin system and bundled plugins comp/cli CLI entry point, hermes_cli/, setup wizard labels May 19, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔎 Lint report: hermes/hermes-de55f5dd vs origin/main

ruff

Total: 0 on HEAD, 0 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 0 pre-existing issues carried over.

ty (type checker)

Total: 8807 on HEAD, 8807 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 4630 pre-existing issues carried over.

Diagnostics are surfaced as warnings — this check never fails the build.

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