Skip to content

Conversation

@cheolwanpark
Copy link

Problem:
Cursor Agent was streaming 0 messages despite appearing successful:

🔧 [Cursor] Using model: gpt-5
[INFO] [CLI] 🔍 Final success determination: cli_type=CLIType.CURSOR, result_success=None, has_error=False
[INFO] [CLI] Using has_error logic: not False = True
[SUCCESS] [CLI] Streaming completed successfully. Total messages: 0
[INFO] [ACT] Result received: success=True, cli=cursor

Root Cause:
cursor-agent.py:310 was using -m flag, but cursor-agent CLI specification appears to have changed and now only recognizes --model for specifying models. The unknown -m flag caused cursor-agent to fail silently with no output.

Fix:
Changed cmd.extend(["-m", cli_model]) to cmd.extend(["--model", cli_model])

Why This Works:
cursor-agent --help confirms: --model <model> (not -m) Using the correct flag ensures cursor-agent properly receives the model parameter and generates output events.

Problem:
Cursor Agent was streaming 0 messages despite appearing successful:
```
🔧 [Cursor] Using model: gpt-5
[SUCCESS] [CLI] Streaming completed successfully. Total messages: 0
```

Root Cause:
cursor-agent.py:310 was using `-m` flag, but cursor-agent CLI
specification appears to have changed and now only recognizes
`--model` for specifying models. The unknown `-m` flag caused
cursor-agent to fail silently with no output.

Fix:
Changed cmd.extend(["-m", cli_model]) to cmd.extend(["--model", cli_model])

Why This Works:
cursor-agent --help confirms: `--model <model>` (not -m)
Using the correct flag ensures cursor-agent properly receives the
model parameter and generates output events.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Copy link

@mojosolo mojosolo left a comment

Choose a reason for hiding this comment

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

Excellent fix!

This correctly addresses the cursor-agent CLI specification change. The flag was causing silent failures (0 messages streamed), and your diagnosis is spot-on.

Validation:

  • Confirmed cursor-agent uses --model flag (not -m)
  • One-line change, minimal risk
  • Fixes critical functionality issue

Impact: This restores cursor-agent functionality for model selection. Ready to merge.

Copy link

@mojosolo mojosolo left a comment

Choose a reason for hiding this comment

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

Excellent fix! This correctly addresses the cursor-agent CLI specification change. The -m flag was causing silent failures (0 messages streamed), and your diagnosis is spot-on.

Validation:

  • Confirmed cursor-agent uses --model flag (not -m)
  • One-line change, minimal risk
  • Fixes critical functionality issue

Impact: This restores cursor-agent functionality for model selection. Ready to merge.

@mojosolo
Copy link

⚠️ Workflow Approval Required

This PR is from a fork, so GitHub requires manual workflow approval before checks can run.

To enable checks:

  1. Navigate to the Checks tab
  2. Click "Approve and run" for first-time contributor workflows
  3. Wait for docs-consistency check to complete

Current Status:

  • ✅ Code reviewed and approved
  • ⏳ Waiting for workflow approval
  • 📋 Once checks pass, ready to merge

What this PR fixes:

Restores cursor-agent model selection by changing -m--model flag. Critical fix for 0-message streaming issue.


Automated comment from infrastructure monitoring

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants