Skip to content

Commit d3408e6

Browse files
mnriemCopilot
andcommitted
docs(agents): align token-resolution comment with output-layout separator rule (review #3415)
Address review #3415 (4725516805). The comment above resolve_command_refs still described the removed state-based behavior ("resolve it from the integration using the project's persisted skills state"). Update it to describe the output-layout rule that register_commands now uses: _sep is derived from the layout this registrar writes (a /SKILL.md scaffold uses the skills separator; a command-layout file uses the command separator), not the persisted ai_skills state. Comment-only change; no behavior change. Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 63f93544-a77f-4f01-bf04-c88806a97dbf
1 parent ba6655c commit d3408e6

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

src/specify_cli/agents.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -742,9 +742,12 @@ def register_commands(
742742
# Resolve __SPECKIT_COMMAND_*__ tokens using the agent's invoke separator.
743743
# For dual-layout agents (e.g. Bob) the separator differs between the
744744
# skills and command layouts, so a single static AGENT_CONFIGS value is
745-
# insufficient. Resolve it from the integration using the project's
746-
# persisted skills state; single-layout agents fall back to the static
747-
# AGENT_CONFIGS value unchanged (invoke_separator_for_mode default).
745+
# insufficient. ``_sep`` (resolved above) is derived from the *output
746+
# layout* this registrar writes — a "/SKILL.md" scaffold uses the skills
747+
# separator, any command-layout file uses the command separator — not
748+
# the project's persisted ai_skills state. Single-layout agents fall back
749+
# to the static AGENT_CONFIGS value unchanged (invoke_separator_for_mode
750+
# default).
748751
# Deferred import of IntegrationBase avoids a circular import at module load
749752
# (base.py itself imports CommandRegistrar lazily).
750753
from specify_cli.integrations.base import IntegrationBase # noqa: PLC0415

0 commit comments

Comments
 (0)