Docs: Remove Cursor from CLI check list in README#3184
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the README to accurately describe which agent tools are probed during the CLI’s agent-tool check, aligning the docs with how integrations are classified (requires_cli: True/False) in the codebase.
Changes:
- Removed “Cursor CLI” from the README list of tools checked by the CLI (Cursor is an IDE-first integration and is not prechecked as a required CLI tool).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
mnriem
left a comment
mnriem
left a comment
There was a problem hiding this comment.
Please address Copilot feedback
DyanGalih
force-pushed
the
docs/3178-cursor-cli-check
branch
from
cedf298 to
53e84d3
Compare
|
Please address Copilot feedback |
|
Checked the latest two feedback items, and they appear addressed in the current head commit. The README wording now explicitly says the CLI tool must be installed, and the earlier conditional wording has already been reworded. Thanks! |
mnriem
left a comment
mnriem
left a comment
There was a problem hiding this comment.
Please address Copilot feedback
|
Follow-up update: I pushed commit cb289fb which clarifies README wording to match implementation behavior. The text now states that specify init checks the selected integration tool is installed only when requires_cli is true, and the ignore-agent-tools guidance remains unchanged. |
|
Thank you! |
Upstream commits merged (7): - 94c7ec2 escape TOML control chars so generated command files parse (github#3341) - 5a901a6 fall through to grep/sed when python3 is a broken stub (github#3312) - 295eb22 port update-agent-context to Python (github#3387) - ba1ce36 remove Cursor from CLI check list in README (github#3184) - 13d2cca document missing CLI flags and integrations (github#3182) - 0da969d add LLM Wiki extension to community catalog (github#3361) - a7b4391 release 0.12.8, begin 0.12.9.dev0 (github#3410) New file: src/specify_cli/_toml_string.py Conflicts: pyproject.toml (trivial version); base.py (import block — combined fork accent() with upstream _toml_string imports). Assisted-by: OpenCode (model: glm-5.2, autonomous)
Fixes #3178
This PR removes Cursor CLI from the list of agents checked by
specify checkin the README, as Cursor is an IDE-based integration and is not checked by the CLI tool.