Skip to content

Hide Alexandria capability prices by default; add includeCosts to find_tools - #538

Merged
nickscamara merged 4 commits into
mainfrom
capy/hide-alexandria-prices
Oct 15, 2026
Merged

nickscamara merged 4 commits into
mainfrom
capy/hide-alexandria-prices

Conversation

@nickscamara

@nickscamara nickscamara commented Oct 15, 2026 •

Copy link
Copy Markdown
Member

Agents no longer see per-capability credit prices in Alexandria output unless they ask for them.

Default output (no prices)

  • firecrawl_find_tools: drops creditsCost, perRecord, billing, recordsPerUnit and zeroDataRetentionCreditsCost from tool items and relatedTools, along with the call's own zero creditsCost.
  • firecrawl_search and URL-mode firecrawl_scrape: the same fields are dropped from data.tools / tools.
  • firecrawl_scrape in Alexandria mode: each result entry no longer echoes its own creditsCost. The request total data.creditsCost and receipts stay as they are.

Opt-in

  • New includeCosts boolean on firecrawl_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; the nextTool arguments carry it forward so pagination keeps prices visible.

Copy

  • The hosted instructions and the Alexandria-mode scrape copy now say execution is billed per capability and point to includeCosts for 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 includeCosts passthrough and carry-forward. Contract snapshots were regenerated.

Open in Capy


Summary by cubic

Agents no longer see per-capability credit prices in Alexandria output unless they ask for them.

  • Strips creditsCost, perRecord, billing, recordsPerUnit, and zeroDataRetentionCreditsCost from firecrawl_find_tools, firecrawl_search, and Alexandria-mode firecrawl_scrape results; request totals are unchanged.
  • Adds includeCosts to firecrawl_find_tools to opt back in; the flag is handled locally and carried through pagination.
  • Updates hosted instructions and README copy to describe per-capability billing and point to includeCosts.
  • Releases 3.30.1 with version bumps and a changelog entry.

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

View guided diff Turn on auto-fix

…d_tools

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@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 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

Comment thread src/service/search-surface.ts Outdated
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@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.

1 issue found across 4 files (changes from recent commits).

Confidence score: 4/5

  • CHANGELOG.md says data.creditsCost is kept, but the default firecrawl_find_tools discovery 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

Comment thread CHANGELOG.md Outdated
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@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 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
nickscamara merged commit d083a62 into main Oct 15, 2026
1 of 2 checks passed
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>
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