[kevin9327]

Fixes #5097

Problem

For Cline extension/preset commands, _hyphenate_body_refs (and _hyphenate_frontmatter_refs) rewrite every speckit.<a>.<b> token. A file name such as speckit.git.commit.md matches as a reference, so .specify/extensions/git/commands/speckit.git.commit.md becomes .../speckit-git-commit-md (extension lost).

Fix

Both helpers now share one compiled pattern and a small replacer that leaves a match ending in .md unchanged. Command references such as /speckit.git.commit are still hyphenated.

Evidence

• Added test_cline_body_refs_keep_md_file_names and test_cline_frontmatter_refs_keep_md_file_names in tests/integrations/test_integration_cline.py.
• On main both fail; with this change both pass.
• pytest tests/integrations tests/test_extensions.py: 3632 passed, 8 skipped.

AI disclosure

Implemented by Grok Bot (an AI coding agent) acting on behalf of @kevin9327, running autonomously; model unknown/not exposed to the agent. Extent: code generation, tests, and this description. The commit carries an Assisted-by: trailer. I ran the tests above locally.

[Copilot]

🟡 Changes recommended

A moderate active issue remains because the .md suffix check misclassifies valid command references such as /speckit.foo.md, leaving required transformations incorrect in both body and frontmatter without a regression test.

1 open issue

• Handle .md command references separately from filenames · New

What changed in this PR

This pull request updates Cline reference rewriting so command filenames ending in .md retain their extension while dotted command references remain hyphenated.

Changes:

• Shares compiled matching and replacement logic for body and frontmatter references.
• Preserves .md-suffixed command filenames.
• Adds regression tests for body and frontmatter handling.

File	Summary
tests/​integrations/​test_integration_cline.py	Adds regression coverage for preserving .md command filenames in body and frontmatter references.
src/​specify_cli/​agents.py	Centralizes Cline reference rewriting, but the .md suffix check also leaves valid command references such as /speckit.foo.md unchanged; this requires a context-aware distinction.

_{Review effort: Lite (auto)}

[mnriem]

Please address Copilot feedback

[Copilot]

🟡 Changes recommended

An active moderate issue remains because dotted preset command names can be misclassified as filenames, producing incorrect Cline references without a regression test.

1 open issue

• Handle dotted preset command names ending in md · New

1 resolved since last review

• Handle .md command references separately from filenames

_{Review effort: Lite (auto)}

[kevin9327]

Fixed in a5afd73. I dropped the segment-count rule, since preset names can contain dots and it could not be made reliable. A speckit.*.md token now stays unchanged only when it sits inside a path (presets/foo/commands/speckit.foo.bar.md). Bare and slash-prefixed tokens (speckit.foo.bar.md, /speckit.foo.md) are hyphenated as command references in both the body and frontmatter. Added test_cline_refs_hyphenate_dotted_preset_command_ending_in_md, which fails without the change and passes with it. tests/integrations: 3148 passed, 8 skipped.

[mnriem]

Please address Copilot feedback