Skip to content

fix(doctor): correct ℹ️ icon and generalize hook example from PR #56 review #59

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
TechNickAI wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(doctor): correct ℹ️ icon and generalize hook example from PR #56 review #59

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions plugins/core/commands/ai-coding-config.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -659,9 +659,9 @@ Report a single summary line per skill (not per field) to keep output scannable.
Hook permissions are the only thing doctor auto-fixes (with user confirmation), and
only in source-repo context where the script files are present:

"⚠️ todo-persist.sh is not executable. Fix now? [y/N]"
"⚠️ <hook-name>.sh is not executable. Fix now? [y/N]"

If yes: `chmod +x plugins/core/hooks/todo-persist.sh`
If yes: `chmod +x plugins/core/hooks/<hook-name>.sh`

For all other issues, report and direct to the appropriate fix command. Don't modify
JSON files, settings, or symlinks without explicit user instruction.
Expand Down Expand Up @@ -701,7 +701,7 @@ Context: source repo (plugins/core/ detected)
### Skill Frontmatter
✅ brainstorming — name, description, triggers, next-skill: brainstorm-synthesis (found)
✅ systematic-debugging — name, description, triggers, next-skill: verify-fix (found)
⚠️ mcp-debug — triggers field present, stability: experimental
ℹ️ mcp-debug — triggers field present, stability: experimental
Copy link
Copy Markdown

@cursor cursor Bot May 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary warning count inconsistent after icon change

Medium Severity

The example summary line still says 2 warnings ⚠️, but changing the mcp-debug line from ⚠️ to ℹ️ means there's now only 1 warning (todo-persist.sh) in the example output. The count wasn't updated to match, so the example is internally inconsistent. Since this is a spec/prompt that guides LLM behavior, the mismatched count could teach models to produce incorrect summary tallies at runtime.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 0bc0588. Configure here.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot May 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Align summary counts with info-level mcp-debug example

Changing mcp-debug from ⚠️ to ℹ️ makes the sample output internally inconsistent because the summary later still reports 2 warnings. In this command file, the example output is behavioral guidance for the doctor response format, so this mismatch can lead the generated report to present incorrect warning totals (e.g., claiming two warnings when only todo-persist.sh is a warning).

Useful? React with 👍 / 👎.


---
## Summary
Expand Down