[MCP Bundle] Fix services when monolog is not installed

[WebMamba]

Q	A
Bug fix?	yes
New feature?	no
Docs?	no
Issues	Fix #830
License	MIT

As said on the issue, when monolog is not installed, the container can't compile the services.
The solution here is to register the service only if the bundle is installed

[WebMamba]

@chr-hertel tell me what you think about this solution, but while working on it I was thinking that just require monolog could be even better. I don't see any good reason to not use it.

[chr-hertel]

I think that solution works for me - is this the only place where we'd need that tho?

I also wonder about the implementation in the McpBundle class:

ai/src/mcp-bundle/src/McpBundle.php

Lines 113 to 134 in 806d6be

	if ($transports['stdio']) {
	$container->register('mcp.server.command', McpCommand::class)
	->setArguments([
	new Reference('mcp.server'),
	new Reference('logger'),
	])
	->addTag('console.command');
	}
	if ($transports['http']) {
	$container->register('mcp.server.controller', McpController::class)
	->setArguments([
	new Reference('mcp.server'),
	new Reference('mcp.psr_http_factory'),
	new Reference('mcp.http_foundation_factory'),
	new Reference('mcp.psr17_factory'),
	new Reference('mcp.psr17_factory'),
	new Reference('monolog.logger.mcp', ContainerInterface::NULL_ON_INVALID_REFERENCE),
	])
	->setPublic(true)
	->addTag('controller.service_arguments');
	}

@camilleislasse do you recall why it's Reference('logger') with McpCommand, but McpController is with Reference('monolog.logger.mcp')?

[chr-hertel]

please use in import here instead the FCQN:

Suggested change

	if (class_exists(\Symfony\Bundle\MonologBundle\MonologBundle::class)) {
	if (class_exists(MonologBundle::class)) {

[camilleislasse]

@camilleislasse do you recall why it's Reference('logger') with McpCommand, but McpController is with Reference('monolog.logger.mcp')?

@chr-hertel I don't recall why, probably a mistake on my part

[OskarStark]

Closing in favor of #864

Thanks

[camilleislasse]

thank's @WebMamba