Skip to content

fix: suggest search alexandria for alexandria search, and list subcommands after unknown-command errors - #371

Open
Ben2W wants to merge 10 commits into
mainfrom
capy/point-mistyped-alexandria-and
Open

Ben2W wants to merge 10 commits into
mainfrom
capy/point-mistyped-alexandria-and

Conversation

@Ben2W

@Ben2W Ben2W commented Nov 4, 2026 •

Copy link
Copy Markdown

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 as categories: ["search"]. It now fails without making a request: unknown command 'search'. Did you mean firecrawl search alexandria '<query>'?. The message respects --json, --pretty and --output, and the query is shell-quoted.
  • Every command group now lists its subcommands after commander's unknown-command error. For example, firecrawl research search adds Commands 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 to query.

…at works

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

Comment thread src/commands/list.test.ts Resolved
Comment thread src/commands/alexandria.ts Resolved
Comment thread src/commands/alexandria.ts Resolved
Comment thread src/commands/list.ts Resolved
…eld, reset exitCode in tests

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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>
@capy-ai capy-ai Bot changed the title fix: point mistyped Alexandria and subcommand calls at the command that works fix: say 'unknown command' when a subcommand doesn't exist Nov 4, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

Comment thread src/commands/list.ts Resolved
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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>

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

Comment thread src/commands/list.ts Resolved
Comment thread src/commands/list.ts Resolved
…uote it

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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>
@capy-ai capy-ai Bot changed the title fix: say 'unknown command' when a subcommand doesn't exist fix: suggest search alexandria for alexandria search, and list subcommands after unknown-command errors Nov 5, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

Comment thread src/commands/list.ts Resolved
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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>

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

Comment thread src/commands/list.ts Resolved
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

Comment thread src/commands/list.ts Resolved

This branch has not been deployed

No deployments
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.

1 participant