Skip to content

fix(cli): send documented default search limit - #250

Merged
developersdigest merged 2 commits into
mainfrom
fix/cli-search-default-limit
Sep 17, 2026
Merged

developersdigest merged 2 commits into
mainfrom
fix/cli-search-default-limit

Conversation

@developersdigest

@developersdigest developersdigest commented Sep 17, 2026 •

Copy link
Copy Markdown
Member

Problem

firecrawl search documents a default of 5 results, but when --limit is omitted the CLI omitted the field from the /v2/search body. The API then applied its default of 10, so authenticated and keyless CLI requests returned more results than documented.

Change

Apply the CLI's documented default (limit: 5) in the shared search payload builder. Explicit --limit values continue to pass through unchanged, and both authenticated and keyless paths use the same body.

Validation

  • pnpm build
  • pnpm exec vitest run src/__tests__/commands/search.test.ts (42 passed)
  • pnpm exec prettier --check src/commands/search.ts src/__tests__/commands/search.test.ts
  • git diff --check

Summary by cubic

Fixes firecrawl search to send the documented default of 5 results when --limit is omitted, instead of letting the API default to 10. Explicit --limit values still pass through unchanged for both authenticated and keyless requests.

Written for commit 945d0fa. Summary will update on new commits.

Review in cubic

@developersdigest
developersdigest merged commit 0984400 into main Sep 17, 2026
6 of 7 checks passed
@developersdigest
developersdigest deleted the fix/cli-search-default-limit branch September 17, 2026 15:40
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