feat(cli): preserve and display API agent hints - #229
Draft
ericciarla wants to merge 1 commit into
Draft
ericciarla wants to merge 1 commit into
ericciarla wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem and result
CLI commands currently drop optional response guidance when rebuilding API envelopes or unwrapping SDK results. Preserve
agent_hintsacross Search, Scrape, Parse, Map and Alexandria so an agent can see the server's next-step and feedback guidance, including on empty results and failures.success:falseresponse remains a failure even when HTTP succeeds.--no-agent-hintssuppresses guidance locally, including JSON. This flag does not add unsupported request-body parameters or claim to disable generation on the server.Dependency and rollout
Companion API/SDK PR: firecrawl/firecrawl#4641.
Search and Parse read raw response envelopes; keyless Scrape also preserves outer metadata directly. Authenticated Scrape, Map and Alexandria depend on a published SDK containing the metadata preservation in that PR. The current pinned
firecrawl@4.40.0drops those fields. Keep this draft until that release can be pinned and validated; this PR does not invent a future version or publish packages.Separate from skill-routing PR #226; no skills or release identity are changed here.
Validation
git diff --checkpassed.Summary by cubic
Previously, the CLI dropped optional
agent_hintsfrom Search, Scrape, Parse, Map, and Alexandria responses. It now preserves them for JSON output and displays them on stderr for readable and raw output, while--no-agent-hintssuppresses them locally without changing requests or executing suggestions.Migration
SdkError; the pinnedfirecrawl@4.40.0does not preserve all of these fields.Written for commit cd6b2ad. Summary will update on new commits.