Skip to content

fix(kanban): preflight missing Codex profile auth - #19475

Closed
cirwel wants to merge 1 commit into
NousResearch:mainfrom
cirwel:fix/kanban-profile-auth-preflight
Closed

fix(kanban): preflight missing Codex profile auth#19475
cirwel wants to merge 1 commit into
NousResearch:mainfrom
cirwel:fix/kanban-profile-auth-preflight

Conversation

@cirwel

@cirwel cirwel commented May 4, 2026

Copy link
Copy Markdown

Summary

  • Add a Kanban worker spawn preflight for assignee profiles configured with model.provider: openai-codex but missing a profile-local auth.json.
  • Fail before subprocess.Popen with a non-secret diagnostic that points operators to intentional profile auth setup via hermes -p <profile> auth or hermes -p <profile> model.
  • Cover both missing-store and present-store behavior in Kanban DB tests.

Why

While dogfooding Kanban with a cloned worker profile, the profile copied config that selected OpenAI Codex but did not carry OAuth auth state. The dispatcher could still spawn the worker, which then exited immediately with the existing Codex credential error. This catches the obvious missing-profile-store case before process launch without reading, printing, or copying credential contents.

Test plan

  • python3 -m pytest tests/hermes_cli/test_kanban_db.py -q
  • python3 -m compileall -q hermes_cli/kanban_db.py tests/hermes_cli/test_kanban_db.py
  • git diff --check

@alt-glitch alt-glitch added type/bug Something isn't working P2 Medium — degraded but workaround exists comp/cli CLI entry point, hermes_cli/, setup wizard labels May 4, 2026
@cirwel
cirwel force-pushed the fix/kanban-profile-auth-preflight branch from 0d41cb7 to a296c72 Compare May 6, 2026 13:59
@cirwel

cirwel commented May 8, 2026

Copy link
Copy Markdown
Author

Superseded by #20706 — thanks!

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 P2 Medium — degraded but workaround exists type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants