Skip to content

Conversation

@wesm
Copy link
Collaborator

@wesm wesm commented Jan 30, 2026

Summary

  • Cursor agent: New cursor agent backed by the Cursor agent CLI. Uses --output-format stream-json (same format as Claude Code), --mode plan for read-only review, --force for agentic mode. Defaults to --model auto.
  • roborev check-agents: Smoke-tests every registered agent by sending a real prompt and verifying a response. Shows pass/fail per agent with full error output on failure. Supports --agent to test one and --timeout to adjust.
  • roborev show --prompt: Displays the prompt that was sent to the agent instead of the review output.
  • Default models: Cursor defaults to auto, OpenCode defaults to opencode/minimax-m2.1-free.
  • No-agents error: When no agents are found, the daemon now returns a clear error instead of silently falling back to the test agent. The error suggests running roborev daemon restart from a shell with agents on PATH.

Test plan

  • roborev check-agents passes for all installed agents
  • roborev check-agents --agent cursor works with Cursor CLI installed
  • roborev show --prompt <job_id> displays the prompt
  • go test ./... passes

🤖 Generated with Claude Code

wesm and others added 12 commits January 29, 2026 21:15
Uses --output-format stream-json (same format as Claude Code),
--mode plan for read-only review, --force for agentic mode,
and prompt as positional argument.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
check-agents runs a smoke-test prompt against each agent to verify
they are installed and actually responding, not just on PATH.

show --prompt displays the prompt sent to the agent instead of the
review output.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Print all lines of the error (including stderr) indented, instead of
truncating to the first line.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Avoids premium model errors when no model is explicitly configured.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Avoids auth errors when no model is explicitly configured.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Fix import sort order in main.go (sort after os/signal)
- Document -p flag purpose in cursor.go buildArgs
- Document WithReasoning as accepted but not mapped to CLI flag

Note: CursorCmd config not being wired to the agent is pre-existing
for all agents (CodexCmd, ClaudeCodeCmd are equally unwired).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Cobra prints usage text when RunE returns an error. Set SilenceUsage
to prevent that since agent failures are not usage errors.

Remove cursor-small default model — it's not in the agent CLI's model
list. Let the CLI pick its own default instead.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
When no real agents are on PATH, GetAvailable was falling back to the
test agent (which is always 'available') and producing fake reviews.
Skip it in the last-resort loop so the daemon returns a clear error.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Lets the agent CLI pick a model based on the task, avoiding premium
model errors on free plans.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The test agent is now excluded from GetAvailable fallback, so with an
empty PATH no agents are available. Update the test to expect an error
instead of a test agent fallback.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@wesm wesm merged commit e7988b9 into main Jan 30, 2026
7 checks passed
@wesm wesm deleted the cursor-agent branch January 30, 2026 04:23
@wesm wesm mentioned this pull request Jan 30, 2026
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