@@ -347,10 +347,10 @@ export default async function satelliteConfigRoute(server: FastifyInstance) {
347347
348348 // Fetch MCP server installations for this satellite
349349 const mcpServerConfigs : Record < string , McpServerConfig > = { } ;
350-
350+
351351 // Get ALL MCP server installations with server details (adapted from gateway logic)
352352 // Global satellites get all teams, team satellites get only their team
353- const whereClause = satelliteData . satellite_type === 'global'
353+ const whereClause = satelliteData . satellite_type === 'global'
354354 ? eq ( mcpServers . status , 'active' )
355355 : and (
356356 eq ( mcpServers . status , 'active' ) ,
@@ -371,7 +371,15 @@ export default async function satelliteConfigRoute(server: FastifyInstance) {
371371
372372 // Process each installation using proven gateway logic
373373 for ( const { installation, server, team_slug, created_by_user_id } of installations ) {
374- if ( ! server || ! team_slug ) continue ;
374+ if ( ! server || ! team_slug ) {
375+ request . log . warn ( {
376+ operation : 'config_installation_skipped_missing_data' ,
377+ installation_id : installation . id ,
378+ has_server : ! ! server ,
379+ has_team_slug : ! ! team_slug
380+ } , 'Skipping installation - missing server or team_slug' ) ;
381+ continue ;
382+ }
375383
376384 try {
377385 // Create unique process identifier: {server_slug}-{team_slug}-{installation_id}
@@ -391,8 +399,11 @@ export default async function satelliteConfigRoute(server: FastifyInstance) {
391399
392400 if ( ! packages || packages . length === 0 ) {
393401 request . log . warn ( {
402+ operation : 'config_installation_skipped_no_packages' ,
403+ installation_id : installation . id ,
394404 serverId : server . id ,
395- serverName : server . name
405+ serverName : server . name ,
406+ transport_type : server . transport_type
396407 } , 'No packages configuration found for stdio transport' ) ;
397408 continue ;
398409 }
@@ -538,6 +549,8 @@ export default async function satelliteConfigRoute(server: FastifyInstance) {
538549
539550 if ( ! remotes || remotes . length === 0 ) {
540551 request . log . warn ( {
552+ operation : 'config_installation_skipped_no_remotes' ,
553+ installation_id : installation . id ,
541554 serverId : server . id ,
542555 serverName : server . name ,
543556 transportType : server . transport_type
@@ -548,6 +561,8 @@ export default async function satelliteConfigRoute(server: FastifyInstance) {
548561 const remoteConfig = remotes [ 0 ] ;
549562 if ( ! remoteConfig || ! remoteConfig . url ) {
550563 request . log . warn ( {
564+ operation : 'config_installation_skipped_no_url' ,
565+ installation_id : installation . id ,
551566 serverId : server . id ,
552567 serverName : server . name ,
553568 hasRemoteConfig : ! ! remoteConfig
0 commit comments