Skip to content

fix(kanban): persist worker session metadata on completion (#25579) - #28386

Closed
teknium1 wants to merge 1 commit into
mainfrom
hermes/hermes-de55f5dd
Closed

fix(kanban): persist worker session metadata on completion (#25579)#28386
teknium1 wants to merge 1 commit into
mainfrom
hermes/hermes-de55f5dd

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Salvages #25579 by @wesleysimplicio.

Stamps task_runs.metadata.worker_session_id from HERMES_SESSION_ID on kanban_complete; not in main and clean focused diff to tools/kanban_tools.py.

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

@teknium1
teknium1 force-pushed the hermes/hermes-de55f5dd branch 12 times, most recently from ead1072 to 7a6f1c3 Compare May 19, 2026 03:20
Co-Authored-By: Paperclip <noreply@paperclip.ing>
@teknium1
teknium1 force-pushed the hermes/hermes-de55f5dd branch from 7a6f1c3 to 025f146 Compare May 19, 2026 03:21
@teknium1

Copy link
Copy Markdown
Contributor Author

Wrong commit picked up (AUTHOR_MAP fixup tip instead of substantive fix). Reopening as a new PR with the correct commit.

@teknium1 teknium1 closed this May 19, 2026
@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 19, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #28387 — same salvage of #25579 (persist worker session metadata). This PR was superseded by #28387 which targets the correct file (tools/kanban_tools.py).

@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: 8812 on HEAD, 8810 on base (🆕 +2)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 4626 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/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.

3 participants