Skip to content

Remove Alexandria access warning from MCP instructions - #582

Merged
erikengervall merged 2 commits into
mainfrom
capy/remove-alexandria-access-warning
Oct 22, 2026
Merged

erikengervall merged 2 commits into
mainfrom
capy/remove-alexandria-access-warning

Conversation

@erikengervall

@erikengervall erikengervall commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Removes the API-key and team-enabled warning from MCP initialization and tool descriptions while leaving runtime credential and entitlement checks unchanged. Provider-term responses continue to direct users to the Firecrawl dashboard URL returned with the requirement.

Refreshes contract snapshots for the affected MCP surfaces.

Open in Capy


Summary by cubic

Removes the API-key and team-enabled Alexandria access warning from MCP initialization instructions and tool descriptions, leaving runtime credential and entitlement checks unchanged. Keyless sessions keep a note that Alexandria provider calls require a Firecrawl API key and are not included in the hosted keyless tier.

  • Provider-term guidance now directs users to the Firecrawl dashboard URL returned with a terms requirement.
  • Refreshes contract snapshots for the affected MCP surfaces.

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

View guided diff Turn on auto-fix

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

Reply with feedback, questions, or to request a fix.

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

Comment thread tests/contract-snapshots/stdio-keyless.json Outdated
Comment thread src/server.ts Outdated

@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 10 files (changes from recent commits).

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Auto-approved: Bounded documentation/messaging update: removes the Alexandria API-key/team-enabled warning from MCP instructions and tool descriptions, keeps runtime checks and the keyless API-key note, and refreshes contract snapshots.

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

@erikengervall
erikengervall force-pushed the capy/remove-alexandria-access-warning branch from e35cca1 to d652c0e Compare October 21, 2026 03:46
@erikengervall
erikengervall merged commit 2078454 into main Oct 22, 2026
2 checks passed
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