Skip to content

feat(cli): submit keyless job feedback - #221

Draft
Max17190 wants to merge 14 commits into
mainfrom
max/enable-keyless-feedback
Draft

feat(cli): submit keyless job feedback#221
Max17190 wants to merge 14 commits into
mainfrom
max/enable-keyless-feedback

Conversation

@Max17190

@Max17190 Max17190 commented Sep 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Why

Let keyless CLI callers submit structured Search, Scrape, and Parse feedback.

Summary

  • Support firecrawl feedback without an API key, with task, assessment, observation JSON/files, and Parse document class.
  • Preserve job references in JSON output. After an eligible call, print the job reference and a one-line feedback request to stderr; the full contract stays in firecrawl feedback --help.
  • Define reason codes and evidence requirements in help, the README, and bundled skills. Keyless feedback is requested in exchange for free keyless use, and each job accepts one submission. Preserve authenticated feedback, Alexandria session feedback, and search-feedback behavior.

Release after API #4616 is deployed and Docs #1414 is published.

Test Plan

  • Passed 84 targeted tests covering the feedback request, feedback help, submission, Search, and Parse, plus the full formatting check. The full suite runs in CI against a fresh install.
  • Covered credential-free submissions, failure references, Parse observations, the short feedback request, and authenticated opt-outs.
  • Real API integration remains a staging check.

@Max17190 Max17190 mentioned this pull request Sep 11, 2026
Open
2 tasks
@Max17190
docs(cli): distinguish keyless and authenticated feedback
521389c
@Max17190
fix(cli): forward feedback invitation preferences
cf929f4
@Max17190
docs(cli): clarify shared keyless feedback daily limit
02baf67
@Max17190
fix(cli): preserve keyless feedback and silence opted-out guidance
2f3855c
@Max17190
fix(cli): retain keyless invitations and optional submissions
aec2df9
@Max17190
feat(cli): support structured keyless feedback categories
80a8808
@Max17190
docs(cli): clarify search feedback source attribution
fc5f679
@Max17190
docs(cli): align keyless feedback evidence guidance
b9a61bb
@Max17190
docs: clarify keyless feedback limits and failure reasons
49c4737
@Max17190
merge: synchronize keyless feedback with main
328a8b8
@Max17190
style(test): format CLI terms refusal test
7a0847d
@Max17190
fix(cli): print a short keyless feedback reminder
dd0af47 
After an eligible call, print the job reference and a one-line optional
reminder pointing to `firecrawl feedback <endpoint> <jobId> --help` instead of
the full server message and flag template. Remove the daily submission rule and
the exact attempt rate from help, the README, and bundled skills, and drop the
retired DAILY_LIMIT_REACHED code.
@Max17190
merge: synchronize keyless feedback with main
5062793 
Resolve conflicts with Alexandria session feedback and the Alexandria-only
search command:

- Send Alexandria session fields for endpoint alexandria and keyless task,
  assessment, document class, and observations for job feedback.
- Keep the keyless feedback help beside the updated search action.
- Keep the Alexandria guidance and relocated completion criteria in the
  bundled skills alongside the keyless feedback contract.
- Describe keyless feedback as requested in exchange for free keyless use in
  the stderr reminder, help, README, and skills.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Max17190