Skip to content

fix: load default collection even with introspection#756

Merged
DaleSeo merged 1 commit into
mainfrom
AMS-500
May 27, 2026
Merged

fix: load default collection even with introspection#756
DaleSeo merged 1 commit into
mainfrom
AMS-500

Conversation

@DaleSeo
Copy link
Copy Markdown
Member

@DaleSeo DaleSeo commented May 27, 2026

When operations.source was not configured, Apollo MCP Server inferred the source by checking introspection settings first, which meant a configured apollo_graph_ref was silently ignored whenever any introspection tool (execute, introspect, search, validate) was also enabled. Now, when both apollo_graph_ref and apollo_key are available, the default GraphOS operation collection is loaded regardless of introspection settings. Users who have configured both will see operation tools alongside any enabled introspection tools.

@DaleSeo DaleSeo self-assigned this May 27, 2026
@apollo-librarian
Copy link
Copy Markdown
Contributor

apollo-librarian Bot commented May 27, 2026

✅ Docs preview has no changes

The preview was not built because there were no changes.

Build ID: 635abd6ed947d0762ca2cd21
Build Logs: View logs


✅ AI Style Review — No Changes Detected

No MDX files were changed in this pull request.

Review Log: View detailed log

This review is AI-generated. Please use common sense when accepting these suggestions, as they may not always be accurate or appropriate for your specific context.

@DaleSeo DaleSeo marked this pull request as ready for review May 27, 2026 19:54
@DaleSeo DaleSeo requested a review from a team as a code owner May 27, 2026 19:54
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2026

Changeset file added - thank you!

Comment thread crates/apollo-mcp-server/src/main.rs Outdated
@DaleSeo DaleSeo merged commit ef3ac32 into main May 27, 2026
15 of 16 checks passed
@DaleSeo DaleSeo deleted the AMS-500 branch May 27, 2026 20:45
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.

2 participants