From de47d0f8a4d0c2e717e10f6f5c29f94b706c5f36 Mon Sep 17 00:00:00 2001 From: SSC-ENG Date: Thu, 30 Jul 2026 03:37:11 -0700 Subject: [PATCH] docs(kanban): fix stale skipped_nonspawnable docstring event reference The DispatchResult.skipped_nonspawnable docstring still named the removed dispatch_nonspawnable_assignee event. Since PR #6 (HEL-3114) removed the superseded record_nonspawnable helper, the durable signal for a nonspawnable-assignee skip is the pre_dispatch_validation_failed event (error_code=missing_assignee_profile) emitted via reject_pre_dispatch. Update the docstring to name the actual event. Follow-up to TRC low-severity finding on PR #6 review at head 93c6e87. Linear: HEL-3114 --- hermes_cli/kanban_db.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/hermes_cli/kanban_db.py b/hermes_cli/kanban_db.py index f3616fec0a8f..d01e1f43a1b4 100644 --- a/hermes_cli/kanban_db.py +++ b/hermes_cli/kanban_db.py @@ -6837,8 +6837,10 @@ class DispatchResult: skipped_nonspawnable: list[str] = field(default_factory=list) """Ready/review task ids skipped because their assignee is not an installed Hermes profile. Each real (non-dry-run) skip also emits a - deduplicated ``dispatch_nonspawnable_assignee`` event so the failure is - durable and machine-readable instead of silently leaving work queued.""" + deduplicated ``pre_dispatch_validation_failed`` event + (``error_code=missing_assignee_profile``, via ``reject_pre_dispatch``) + so the failure is durable and machine-readable instead of silently + leaving work queued.""" pre_dispatch_failed: list[tuple[str, str]] = field(default_factory=list) """Task ids rejected by the pre-claim capability gate, paired with the stable failure code. A rejected task is blocked before a run is created