Merge pull request #401 from apollographql/fix/remove-verbose-logging

fix: remove verbose logging

Author: swcollard

.changesets/fix_fix_remove_verbose_logging.md

@@ -0,0 +1,5 @@
+### fix: remove verbose logging - @swcollard PR #401
+
+The tracing-subscriber crate we are using to create logs does not have a configuration to exclude the span name and attributes from the log line. This led to rather verbose logs on app startup which would dump the full operation object into the logs before the actual log line. 
+
+This change strips the attributes from the top level spans so that we still have telemetry and tracing during this important work the server is doing, but they don't make it into the logs. The relevant details are provided in child spans after the operation has been parsed so we aren't losing any information other than a large json blob in the top level trace of generating Tools from GraphQL Operations.

crates/apollo-mcp-server/src/operations/operation.rs

@@ -48,7 +48,7 @@ impl Operation {
         self.inner
     }
 
-    #[tracing::instrument(skip(graphql_schema, custom_scalar_map))]
+    #[tracing::instrument(skip_all, name = "load_tool")]
     pub fn from_document(
         raw_operation: RawOperation,
         graphql_schema: &GraphqlSchema,

crates/apollo-schema-index/src/lib.rs

@@ -119,6 +119,7 @@ pub struct SchemaIndex {
 }
 
 impl SchemaIndex {
+    #[tracing::instrument(skip_all, name = "schema_index")]
     pub fn new(
         schema: &Valid<Schema>,
         root_types: EnumSet<OperationType>,