Kanban worker handling of unresolved skills, plus one unrelated Gemini fix

Fixes on record

Merged to main. A stamp counts the issues a fix is linked to.

• #55585 fix(kanban): unknown skill warns instead of crashing the worker

  merged to main

  files: cli.py, scripts/release.py, tui_gateway/server.py

  Show the diff

  • cli.py

    @@ -15359,7 +15359,21 @@ def main(
             )
             if missing_skills:
                 missing_display = ", ".join(missing_skills)
    -            raise ValueError(f"Unknown skill(s): {missing_display}")
    +            # If at least one skill loaded, degrade gracefully: skip the
    +            # unknown ones and continue. A typo'd skill name should not crash
    +            # the worker (which auto-blocks the Kanban task after retries).
    +            # Only when EVERY requested skill is missing do we hard-fail, so a
    +            # fully-misconfigured worker fails loudly instead of running blind.
    +            if loaded_skills:
    +                logger.warning(
    +                    "Unknown skill(s) requested, skipping: %s. "
    +                    "Continuing with: %s. "
    +                    "List available skills with `hermes skills list`.",
    +                    missing_display,
    +                    ", ".join(loaded_skills),
    +                )
    +            else:
    +                raise ValueError(f"Unknown skill(s): {missing_display}")
             if skills_prompt:
                 cli.system_prompt = "\n\n".join(
                     part for part in (cli.system_prompt, skills_prompt) if part

  • scripts/release.py

    @@ -45,6 +45,7 @@
     
     # Auto-extracted from noreply emails + manual overrides
     AUTHOR_MAP = {
    +    "[email protected]": "jimmyjohansson84",  # PR #27123 salvage (Kanban unknown-skill warn-instead-of-crash; #27136)
         "[redacted:email]": "theAgenticBuilder",  # PR #14180 salvage (route delegate_task progress lines through _safe_print so ACP stdio JSON-RPC frames stay clean)
         "[redacted:email]": "huangxudong663-sys",  # PR #15157 salvage (isinstance(dict) guard on tool-call model_extra; NVIDIA NIM non-dict crash)
         "[redacted:email]": "Nickperillo",  # PR #8008 salvage (Discord channel-name matching + flush pending sends on shutdown)

  • tui_gateway/server.py

    @@ -4212,12 +4212,25 @@ def _make_agent(
         if startup_skills:
             from agent.skill_commands import build_preloaded_skills_prompt
     
    -        skills_prompt, _loaded_skills, missing_skills = build_preloaded_skills_prompt(
    +        skills_prompt, loaded_skills, missing_skills = build_preloaded_skills_prompt(
                 startup_skills,
                 task_id=session_id or key,
             )
             if missing_skills:
    -            raise ValueError(f"Unknown skill(s): {', '.join(missing_skills)}")
    +            missing_display = ", ".join(missing_skills)
    +            # Degrade gracefully when some skills loaded; only hard-fail when
    +            # every requested skill is missing. Mirrors cli.py — a typo'd skill
    +            # name should not crash the worker and auto-block the Kanban task.
    +            if loaded_skills:
    +                logger.warning(
    +                    "Unknown skill(s) requested, skipping: %s. "
    +                    "Continuing with: %s. "
    +                    "List available skills with `hermes skills list`.",
    +                    missing_display,
    +                    ", ".join(loaded_skills),
    +                )
    +            else:
    +                raise ValueError(f"Unknown skill(s): {missing_display}")
             if skills_prompt:
                 system_prompt = "\n\n".join(
                     part for part in (system_prompt, skills_prompt) if part