[BUG] [v0.0.7] github run`: treat `@Cortex` and `/Cortex` as mentions (case-insensitive)

[sunlight7777777]

Project

cortex

Description

cortex github run only detects mentions when the comment body contains the exact substrings @cortex or starts with /cortex. GitHub usernames and slash commands are often typed with different casing (e.g. @Cortex). Those comments are skipped with “Skipping: no Cortex mention detected”, and parse_cortex_command never extracts commands from mixed-case forms.

22122.mp4

Error Message

Debug Logs

System Information

Ubuntu 24.04
RAM 32GB

Screenshots

22122.mp4

Steps to Reproduce

1. Save a minimal issue_comment payload as event.json (shape matches GitHub’s webhook; only action, issue, and comment are required for parsing):

   {
     "action": "created",
     "issue": { "number": 1, "title": "PR title" },
     "comment": {
       "id": 999,
       "body": "@Cortex help",
       "user": { "login": "alice" }
     }
   }

2. Run (a dummy token is enough; the handler returns before any HTTP call when the mention is skipped):

   cortex github run --event issue_comment --token dummy --repository owner/repo --event-path event.json

3. Observe stdout: Skipping: no Cortex mention detected and no “Cortex command detected!”.

4. Repeat with "body": "@cortex help" and confirm processing proceeds past the mention check. Note: from that point the handler uses the GitHub API (reactions, comments), so you need a valid token and repository for the command to complete without error.

Expected Behavior

Any case variation of @cortex and /cortex should count as a mention and participate in command parsing, consistent with help text (“Mention @cortex anywhere in your comment”)

Actual Behavior

• Comment: Please @Cortex review this → skipped; no API response.
• parse_cortex_command uses find("@cortex") and find("/cortex"), so it also fails for mixed case

Additional Context

No response

[echobt]

✅ Validation Result: VALID

Reason

The code confirms that parse_cortex_command uses case-sensitive find to detect mentions. The handle_issue_comment function also uses case-sensitive contains and starts_with to check for mentions. This matches the bug report.

Verification Checklist

• ✅ Screenshot/video provided
• ✅ Images accessible
• ✅ Shows CLI/TUI interface
• ✅ Code matches description
• ✅ No duplicates found

[nijanthanvijayakumar]

Opened upstream PR: CortexLM/cortex-code#410

This makes GitHub comment triggers case-insensitive for @Cortex and /Cortex, with focused CLI tests and Docker validation.