Repository navigation
Hide Alexandria capability prices by default; add includeCosts to find_tools - #538
Merged
Merged
Conversation
…d_tools Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
All reported issues were addressed across 18 files
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>
Contributor
There was a problem hiding this comment.
1 issue found across 4 files (changes from recent commits).
Confidence score: 4/5
CHANGELOG.mdsaysdata.creditsCostis kept, but the defaultfirecrawl_find_toolsdiscovery response omits its zero total. Scope the claim to execution results or note the discovery response omission.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="CHANGELOG.md">
<violation number="1" location="CHANGELOG.md:11">
P2: This says `data.creditsCost` is kept, but default `firecrawl_find_tools` output removes it. Scope the claim to execution results or note that the free discovery response omits its zero total.</violation>
</file>
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
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.
Requires human review: Auto-approval blocked by 1 unresolved P0–P2 issue from previous reviews.
View guided diff | Turn on auto-fix | Re-trigger cubic
nickscamara
added a commit
that referenced
this pull request
Oct 15, 2026
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
Ben2W
pushed a commit
to Ben2W/firecrawl-mcp-server
that referenced
this pull request
Oct 15, 2026
Resolve contract snapshot conflicts with firecrawl#538/firecrawl#539 by regenerating them from the merged source. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Agents no longer see per-capability credit prices in Alexandria output unless they ask for them.
Default output (no prices)
firecrawl_find_tools: dropscreditsCost,perRecord,billing,recordsPerUnitandzeroDataRetentionCreditsCostfrom tool items andrelatedTools, along with the call's own zerocreditsCost.firecrawl_searchand URL-modefirecrawl_scrape: the same fields are dropped fromdata.tools/tools.firecrawl_scrapein Alexandria mode: each result entry no longer echoes its owncreditsCost. The request totaldata.creditsCostand receipts stay as they are.Opt-in
includeCostsboolean onfirecrawl_find_tools("Include per-capability credit prices (creditsCost, perRecord) in tool results."). When it's set, output matches what it was before this change. The MCP server handles the flag itself and doesn't send it upstream; thenextToolarguments carry it forward so pagination keeps prices visible.Copy
includeCostsfor prices, replacing the "billed at the capability's listed price" wording. The README is updated to match.Tests cover default stripping for find_tools, search and execution results, plus
includeCostspassthrough and carry-forward. Contract snapshots were regenerated.Summary by cubic
Agents no longer see per-capability credit prices in Alexandria output unless they ask for them.
creditsCost,perRecord,billing,recordsPerUnit, andzeroDataRetentionCreditsCostfromfirecrawl_find_tools,firecrawl_search, and Alexandria-modefirecrawl_scraperesults; request totals are unchanged.includeCoststofirecrawl_find_toolsto opt back in; the flag is handled locally and carried through pagination.includeCosts.Written for commit ff7828d. Summary will update on new commits.