Skip to content

feat: add --sort to research search-papers and print row dates - #364

Merged
PhantomInTheWire merged 3 commits into
mainfrom
research-papers-sort-and-row-fields
Nov 4, 2026
Merged

PhantomInTheWire merged 3 commits into
mainfrom
research-papers-sort-and-row-fields

Conversation

@PhantomInTheWire

@PhantomInTheWire PhantomInTheWire commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

firecrawl research search-papers takes --sort relevance|date_desc, prints each row's created and updated dates, and reads row authors as a name list.

firecrawl research search-papers "speculative decoding" --sort date_desc

Open in Capy


Summary by cubic

firecrawl research search-papers now takes --sort relevance|date_desc and prints each paper row's created and updated dates.

  • Sends the sort value on both the API-key and keyless paths.
  • Renders authors as names, capped at 15, with et al. for name lists and +N more for comma-joined strings.
  • Documents --sort in README.md and --sort, --from, and --to in the firecrawl-research-index skill.

Written for commit f8c9f4f. Summary will update on new commits.

View guided diff Turn on auto-fix

search-papers takes --sort relevance or date_desc and sends it on both
the keyed and keyless paths. Rows whose authors are a list of names
print those names, and each hit prints its created and updated dates.
@PhantomInTheWire
PhantomInTheWire marked this pull request as ready for review November 4, 2026 07:33

@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 6 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/research.ts Resolved
Comment thread src/commands/research.ts Resolved

@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 2 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 require human review. Adds --sort/date display to research search-papers with tests, but relies on unverified external claims: API never sends structured authors, and MCP sort support ships in another PR.

View guided diff | Turn on auto-fix | Re-trigger cubic

@PhantomInTheWire
PhantomInTheWire merged commit 8996950 into main Nov 4, 2026
14 of 15 checks passed
@PhantomInTheWire
PhantomInTheWire deleted the research-papers-sort-and-row-fields branch November 4, 2026 13:11
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