Repository navigation
Conversation
…at works Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
All reported issues were addressed across 4 files
Shadow auto-approve: would not auto-approve because issues were found.
View guided diff | Turn on auto-fix | Re-trigger cubic
…eld, reset exitCode in tests Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
0 issues found across 4 files (changes from recent commits).
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would auto-approve. Improves CLI error guidance for mistyped Alexandria/find-tools calls and unknown subcommands, with tests covering the new messages. The change is confined to error paths and is a clearly beneficial, bounded fix.
Tip: Get a Slack DM when cubic finishes reviewing your PRs. Connect Slack
View guided diff | Turn on auto-fix | Re-trigger cubic
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
All reported issues were addressed across 4 files (changes from recent commits).
Shadow auto-approve: would auto-approve with 1 open P3 issue. Improves CLI error handling: alexandria browse now reports an unknown command or category instead of a misleading API message, and all command groups show their subcommands after an unknown-command error. Focused, user-facing UX fix.
Tip: Review your code locally with the cubic CLI to iterate faster.
View guided diff | Turn on auto-fix | Re-trigger cubic
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would auto-approve. Improves CLI error messaging: unknown Alexandria categories get a clear 'unknown command or category' message and command groups list their subcommands after unknown-command errors. Bounded to error paths with no behavioral or API change.
View guided diff | Turn on auto-fix | Re-trigger cubic
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Shadow auto-approve: would not auto-approve because issues were found.
View guided diff | Turn on auto-fix | Re-trigger cubic
…uote it Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would require human review. Improves unknown-command and unknown-category errors, but the dismissed P2 finding that alexandria search bypasses --json/--pretty/--output is unresolved in the diff, so a human must sign off.
View guided diff | Turn on auto-fix | Re-trigger cubic
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Shadow auto-approve: would not auto-approve because issues were found.
View guided diff | Turn on auto-fix | Re-trigger cubic
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would require human review. Adds alexandria search unknown-command error and lists subcommands after unknown-command errors, but the search error still bypasses --json/--pretty/--output handling, contrary to the PR description.
View guided diff | Turn on auto-fix | Re-trigger cubic
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Shadow auto-approve: would not auto-approve because issues were found.
View guided diff | Turn on auto-fix | Re-trigger cubic
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Shadow auto-approve: would require human review. This CLI error-message fix adds an unconditional guard that treats any category path starting with search as a mistyped search command; a dismissed finding that this blocks a legitimate search category remains unaddressed and needs a scoping decision.
View guided diff | Turn on auto-fix | Re-trigger cubic
Two CLI-only errors that the API can't help with, because they're about CLI command shapes:
firecrawl alexandria search <query>was sent to Find Tools ascategories: ["search"]. It now fails without making a request:unknown command 'search'. Did you mean firecrawl search alexandria '<query>'?. The message respects--json,--prettyand--output, and the query is shell-quoted.firecrawl research searchaddsCommands of firecrawl research: search-papers, inspect-paper, related-papers, read-paper. Before, "Did you mean" appeared only for near-spellings.Other unknown categories (
firecrawl alexandria 3d) show the API's message unchanged. Since firecrawl/exchange#1207, that message names the rejected ID, lists the valid categories and points toquery.