From 82676080a4280da68b129408f687ea55a8154058 Mon Sep 17 00:00:00 2001 From: Sanjay Santhanam <51058514+Sanjays2402@users.noreply.github.com> Date: Thu, 16 Jul 2026 22:15:59 -0700 Subject: [PATCH] fix(session): describe skill selections accurately Use neutral "set" wording for operator skill markers so re-selecting the current skill does not falsely claim a change. Update the regression expectation for the persisted marker. --- tests/test_session.py | 2 +- turnstone/core/session.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/test_session.py b/tests/test_session.py index 44d2a9f8..0a68d2fb 100644 --- a/tests/test_session.py +++ b/tests/test_session.py @@ -248,7 +248,7 @@ class TestSkillCommand: assert marker["role"] == "system" assert marker["_source"] == "skill_hint" assert marker["content"] == ( - f"Operator changed the active skill from alpha to {expected_target}." + f"Operator set the active skill from alpha to {expected_target}." ) save_message.assert_called_once() diff --git a/turnstone/core/session.py b/turnstone/core/session.py index e7ae2229..675862cd 100644 --- a/turnstone/core/session.py +++ b/turnstone/core/session.py @@ -16929,7 +16929,7 @@ class ChatSession: self.set_skill(None) self._append_system_turn( "skill_hint", - f"Operator changed the active skill from {previous_skill} to defaults.", + f"Operator set the active skill from {previous_skill} to defaults.", ) self.ui.on_info("Skill cleared; using defaults.") else: @@ -16938,7 +16938,7 @@ class ChatSession: self.set_skill(tpl["name"]) self._append_system_turn( "skill_hint", - f"Operator changed the active skill from {previous_skill} to {tpl['name']}.", + f"Operator set the active skill from {previous_skill} to {tpl['name']}.", ) self.ui.on_info(f"Skill set: {tpl['name']}") else: