Windows: Copilot integration hardcodes `copilot.cmd`, breaking installs that ship `copilot.exe`
维护者通常 2 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 88/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- python
- 领域
- cli, testing-qa
调研方向
先在 src/specify_cli/integrations/copilot/init.py 中阅读 _copilot_executable() 和 _resolve_executable(),然后运行 tests/integrations/test_extra_args.py 中现有的覆盖测试。为仅包含 copilot.exe 的 Windows PATH 添加回归覆盖测试,并在命令分发能够解析已安装的可执行文件,而不是假定为 copilot.cmd 时,认为该 issue 已完成。
由索引模型根据 Issue 内容生成。
描述
Bug Description
CopilotIntegration._resolve_executable() falls back to a hardcoded, platform-guessed executable name on Windows (copilot.cmd) instead of detecting what's actually installed. When the GitHub Copilot CLI is installed in a way that produces copilot.exe rather than an npm-style .cmd shim, workflow/command dispatch to the copilot integration fails outright on Windows, because Spec Kit never looks for the actual executable on PATH.
Steps to Reproduce
- On Windows, install the Copilot CLI via a distribution that produces
copilot.exeonPATH(not an npm global-install shim producingcopilot.cmd). - Run any Spec Kit workflow/command step with
integration: copilot(e.g.specify workflow run ...). - Dispatch fails because Specify attempts to invoke
copilot.cmd, which does not exist, instead of thecopilot.exethat is actually onPATH.
Expected Behavior
Spec Kit should detect the actual Copilot CLI executable available on the user's PATH on Windows, rather than assuming a single fixed name.
Actual Behavior
Specify attempts to invoke copilot.cmd even when only copilot.exe is installed on PATH, so workflow/command dispatch fails.
Specify CLI Version
1.0.7 (as encountered; likely affects other versions using the same _copilot_executable() logic)
AI Agent
GitHub Copilot
Operating System
Windows (version not specified)
Python Version
N/A (not provided)
Error Logs
N/A (no specific error output provided)
Additional Context
Where: src/specify_cli/integrations/copilot/__init__.py
def _copilot_executable() -> str:
"""Return the executable name for Copilot CLI on this platform.
On Windows, subprocess invocation is reliable with `copilot.cmd`.
"""
if os.name == "nt":
return "copilot.cmd"
return "copilot"
def _resolve_executable(self) -> str:
"""Return the Copilot CLI executable, respecting the env-var override.
Checks ``SPECKIT_INTEGRATION_COPILOT_EXECUTABLE`` first. Falls
back to the platform-specific default from ``_copilot_executable()``
(``copilot.cmd`` on Windows, ``copilot`` elsewhere) so that
existing behaviour is preserved when the env var is unset.
"""
env_name = "SPECKIT_INTEGRATION_COPILOT_EXECUTABLE"
override = os.environ.get(env_name, "").strip()
return override if override else _copilot_executable()
Why this likely wasn't caught earlier:
- Workflow command-step dispatch is a newer code path. Most integration scaffolding (init, command registration, writing
.github//.claude/files) never has to spawn the Copilot CLI as a subprocess._resolve_executable()/build_exec_args()only matter once something actually shells out tocopilot, which is essentially exclusive tospecify workflow runcommand steps — so long-time Spec Kit + Copilot users could easily never exercise this path. - The hardcoded
copilot.cmdguess suggests testing only covered an npm global install (npm install -g @github/copilotor similar), which npm wraps in a.cmdshim on Windows by convention. Installing the Copilot CLI via a different channel (standalone installer,winget,scoop, a signed native binary, etc.) produces a realcopilot.exeonPATHinstead, and hits this bug immediately and reliably. If most Windows testers install via npm, this simply never surfaces for them. - The
SPECKIT_INTEGRATION_COPILOT_EXECUTABLEoverride itself appears recently added (per its own docstring: "so that existing behaviour is preserved when the env var is unset"), suggesting this is a known, actively-being-patched rough edge rather than a long-stable path.
Net: this only surfaces at the intersection of (a) workflows that dispatch to Copilot as a subprocess, (b) Windows, and (c) a non-npm Copilot CLI install — a narrow combination most contributors/testers may not sit in.
Suggested fix:
def _copilot_executable() -> str:
"""Return the executable name for Copilot CLI on this platform."""
if os.name != "nt":
return "copilot"
for candidate in ("copilot.exe", "copilot.cmd", "copilot"):
if shutil.which(candidate):
return candidate
# Nothing found on PATH — keep the historical default so the
# resulting "command not found" error still references the
# previously expected name.
return "copilot.cmd"
shutil is already imported in this module, so this requires no new dependency. Suggest adding a regression test alongside the existing _resolve_executable/build_exec_args coverage in tests/integrations/test_extra_args.py, mocking shutil.which to simulate a Windows install exposing only copilot.exe.
Workaround: SPECKIT_INTEGRATION_COPILOT_EXECUTABLE already exists and can be set to copilot.exe explicitly, scoped to the invoking process, until this is fixed upstream.
AI Disclosure
GitHub Copilot (GPT-6 Sol) organized the user-provided issue text into the repository's bug-report template and filed it. Human-directed; no independent reproduction or code changes. Reasoning-effort setting not provided.
- 主要语言
- Python
- 星标
- 138k
- 派生
- 12.4k
- 平均合并
- 2 天 11 小时
- 30 天内合并 PR
- 163
环境准备
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
github/spec-kit 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 2 天内回复
-
triage-nice-to-have
难度 2/5 1-3 小时 新手友好度 88/100
维护者通常 2 天内回复
-
enhancement needs-triage triage-can-wait
难度 2/5 1-3 小时 新手友好度 65/100
github/spec-kit#4627 · 3 条评论 ·
维护者通常 2 天内回复
-
needs-triage triage-nice-to-have
难度 2/5 1-3 小时 新手友好度 72/100
github/spec-kit#4527 · 1 条评论 ·
维护者通常 2 天内回复
-
bug-assess severity-medium
难度 2/5 1-3 小时 新手友好度 72/100
github/spec-kit#4273 · 3 条评论 ·
维护者通常 2 天内回复
相似的 Issue
-
pydanty:is-working
难度 2/5 1-3 小时 新手友好度 78/100
pydantic/pydantic-ai#8843 ·
维护者通常 1 天内回复
-
breaking change enhancement server
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 88/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
needs-triage
难度 2/5 1-3 小时 新手友好度 85/100
维护者通常 1 天内回复