janniclas.loosenandClaude Opus 5.5
2f8a9f0c61
Introduces ExposurePolicy (option lit_mcp_exposure) deciding per ability whether it is a direct MCP tool, only reachable via mcp-adapter's execute-ability meta-tool, or disabled entirely (not registered with the Abilities API at all). Ability::isDirect() remains the default when no override applies. lit/guide can never be disabled or made indirect; lit/media-preview and lit/media-download (image/embedded-resource results) can never be made indirect, only direct or disabled. AbilityRegistrar and ServerRegistrar take an optional ExposurePolicy and use it to filter registered abilities / compute the direct tool list. IntegrationRegistry::directAbilityNames() is removed (superseded by ExposurePolicy::directNames()). InstructionsBuilder now also takes the policy and lists the currently direct tools by name (lit-<name>), noting the rest are reachable via mcp-adapter-execute-ability as lit/<name>. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01At3V6SjwbMe6eMzVGW3Hs7
50 lines
1.5 KiB
PHP
50 lines
1.5 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace LIT\MCP\Tests\Unit\Integrations;
|
|
|
|
use LIT\MCP\Abilities\Ability;
|
|
use LIT\MCP\Integrations\Integration;
|
|
use LIT\MCP\Integrations\IntegrationRegistry;
|
|
use LIT\MCP\Tests\Unit\TestCase;
|
|
use Mockery;
|
|
|
|
final class IntegrationRegistryTest extends TestCase
|
|
{
|
|
public function testOnlyAvailableIntegrationsContributeAbilities(): void
|
|
{
|
|
$direct = $this->ability('lit/a', true);
|
|
$indirect = $this->ability('lit/b', false);
|
|
$hidden = $this->ability('lit/c', true);
|
|
|
|
$registry = new IntegrationRegistry(
|
|
$this->integration('core', true, [$direct, $indirect]),
|
|
$this->integration('divi', false, [$hidden])
|
|
);
|
|
|
|
self::assertSame(['core'], array_map(static fn ($i) => $i->id(), $registry->available()));
|
|
self::assertSame([$direct, $indirect], $registry->abilities());
|
|
}
|
|
|
|
private function ability(string $name, bool $direct): Ability
|
|
{
|
|
$ability = Mockery::mock(Ability::class);
|
|
$ability->allows('name')->andReturn($name);
|
|
$ability->allows('isDirect')->andReturn($direct);
|
|
|
|
return $ability;
|
|
}
|
|
|
|
/** @param list<Ability> $abilities */
|
|
private function integration(string $id, bool $available, array $abilities): Integration
|
|
{
|
|
$integration = Mockery::mock(Integration::class);
|
|
$integration->allows('id')->andReturn($id);
|
|
$integration->allows('isAvailable')->andReturn($available);
|
|
$integration->allows('abilities')->andReturn($abilities);
|
|
|
|
return $integration;
|
|
}
|
|
}
|