Skip to content

Commit 3b228c1

Browse files
committed
fix(skills): use invocation-neutral hook guidance
Describe hook-derived references as command invocations so dollar-prefixed skills do not receive contradictory slash-command terminology. Assisted-by: GitHub Copilot (model: GPT-5.6 Sol, autonomous) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02f9e138-da58-4a60-93b9-eae659d2aa19
1 parent ed0d0de commit 3b228c1

3 files changed

Lines changed: 4 additions & 2 deletions

File tree

src/specify_cli/integrations/base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
from .manifest import IntegrationManifest
3636

3737
_HOOK_COMMAND_NOTE = (
38-
"- When constructing slash commands from hook command names, "
38+
"- When constructing command invocations from hook command names, "
3939
"replace dots (`.`) with hyphens (`-`). "
4040
"For example, `speckit.git.commit` → `/speckit-git-commit`.\n"
4141
)

tests/integrations/test_integration_base_skills.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ def test_hook_note_injected_for_each_instruction_independently(self):
191191
"---\n"
192192
"name: test\n"
193193
"---\n\n"
194-
"- When constructing slash commands from hook command names, "
194+
"- When constructing command invocations from hook command names, "
195195
"replace dots (`.`) with hyphens (`-`). "
196196
"For example, `speckit.git.commit` → `/speckit-git-commit`.\n"
197197
"- For each executable hook, output the following first block:\n"

tests/integrations/test_integration_codex.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,8 @@ def test_hook_note_injected_in_skills_with_hooks(self, tmp_path):
108108
assert "replace dots" in content, (
109109
"speckit-specify should have dot-to-hyphen hook note"
110110
)
111+
assert "constructing command invocations" in content
112+
assert "constructing slash commands" not in content
111113

112114
def test_hook_note_not_in_skills_without_hooks(self):
113115
"""Skills without hook sections should not get the note."""

0 commit comments

Comments
 (0)