ewilliams28/turnstone
1
0
Fork 0
mirror of https://github.com/turnstonelabs/turnstone.git synced 2026-08-12 23:12:23 -06:00
Code Issues Packages Projects Releases Wiki Activity

fix(session): describe skill selections accurately

Browse Source 
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.
This commit is contained in:
Sanjay Santhanam
2026-07-16 22:15:59 -07:00
committed by Patrick Buckley
parent a64cd25807
commit 82676080a4
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/test_session.py
+1 -1
View File
@@ -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()
turnstone/core/session.py
+2 -2
View File
@@ -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: