From 3d245524dcb4c05adfe2fef5e526cdabab0f5fbd Mon Sep 17 00:00:00 2001 From: kshitijk4poor <82637225+kshitijk4poor@users.noreply.github.com> Date: Sat, 26 Sep 2026 22:49:15 +0530 Subject: [PATCH] refactor(sessions): name the repair-prompts detector after its check The helper now tests a single thing - whether a non-empty stored prompt lacks the Skill Safety heading - so _repair_prompts_missing_skills_markers (plural, "skills markers") misdescribed it. Rename to _repair_prompts_lacks_skill_safety. --- hermes_cli/sessions_cmd.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hermes_cli/sessions_cmd.py b/hermes_cli/sessions_cmd.py index 875073afb5..2da0960c23 100644 --- a/hermes_cli/sessions_cmd.py +++ b/hermes_cli/sessions_cmd.py @@ -1003,7 +1003,7 @@ def _repair_prompts_pin_names(row) -> list[str] | None: return names -def _repair_prompts_missing_skills_markers(row) -> bool: +def _repair_prompts_lacks_skill_safety(row) -> bool: # Only the Skill Safety guidance is a reliable marker: is legitimately # absent when no skills are installed, but the guidance is emitted whenever skill_manage is. from agent.prompt_builder import SKILL_SAFETY_HEADING @@ -1048,7 +1048,7 @@ def _cmd_repair_prompts(db, args): continue seen.add(summary["id"]) row = db.get_session(summary["id"]) - if not row or not _repair_prompts_missing_skills_markers(row): + if not row or not _repair_prompts_lacks_skill_safety(row): continue pin = _repair_prompts_pin_names(row) entry = {"id": row["id"], "prompt_chars": len(row["system_prompt"])}