CS

Author: LC43

src/MCP/McpPolicy.php

@@ -7,13 +7,29 @@
 use Saltus\WP\Framework\Rest\ModelRestPolicy;
 
 class McpPolicy {
-
+	/**
+	 * Modeler instance.
+	 * It is responsible for providing models and model types.
+	 *
+	 * @var Modeler
+	 */
 	private Modeler $modeler;
 
+	/**
+	 * @param Modeler $modeler
+	 */
 	public function __construct( Modeler $modeler ) {
 		$this->modeler = $modeler;
 	}
 
+	/**
+	 * Check if a capability is enabled.
+	 *
+	 * @param string $capability      The capability.
+	 * @param string|null $model_type The model type.
+	 *
+	 * @return bool
+	 */
 	public function has_capability( string $capability, ?string $model_type = null ): bool {
 		if ( $capability === ModelRestPolicy::CAPABILITY_HEALTH ) {
 			return true;
@@ -32,6 +48,14 @@ public function has_capability( string $capability, ?string $model_type = null )
 		return false;
 	}
 
+	/**
+	 * Check if a capability is enabled for a model.
+	 *
+	 * @param Model  $model      The model.
+	 * @param string $capability The capability.
+	 *
+	 * @return bool
+	 */
 	public function is_enabled( Model $model, string $capability ): bool {
 		$options = $model->get_options();
 
@@ -49,7 +73,11 @@ public function is_enabled( Model $model, string $capability ): bool {
 	}
 
 	/**
+	 * Resolve whether a capability should be shown in MCP.
+	 *
 	 * @param array<string, mixed> $config
+	 *
+	 * @return bool
 	 */
 	private function resolve_show_in_mcp( array $config, string $capability ): bool {
 		$map = [
@@ -73,7 +101,10 @@ private function resolve_show_in_mcp( array $config, string $capability ): bool
 	}
 
 	/**
-	 * @param string $name
+	 * Get a model by name.
+	 *
+	 * @param string $name The model name.
+	 * @return Model|null
 	 */
 	public function get_model( string $name ): ?Model {
 		$models = $this->modeler->get_models();

src/Modeler.php

@@ -35,11 +35,23 @@ class Modeler implements RestRouteProvider, ToolContributor {
 	/** @var array<string, Model> */
 	protected array $model_list = [];
 
+
+
+	/**
+	 * Construct the modeler.
+
+	 * @param ModelFactory $model_factory
+	 */
 	public function __construct( ModelFactory $model_factory ) {
 		$this->model_factory = $model_factory;
 		// should contain a list of loaded models
 	}
 
+	/**
+	 * Initialize the modeler.
+	 *
+	 * @param string $project_path The project path.
+	 */
 	public function init( string $project_path ): void {
 		$path = $this->get_path( $project_path );
 		if ( ! $path ) {
@@ -50,6 +62,10 @@ public function init( string $project_path ): void {
 
 	/**
 	 * Get custom path
+	 *
+	 * @param string $project_path The project path.
+	 *
+	 * @return string|null The path.
 	 */
 	protected function get_path( string $project_path ): ?string {
 
@@ -69,7 +85,9 @@ protected function get_path( string $project_path ): ?string {
 	}
 
 	/**
-	 * Load Models
+	 * Load Models.
+	 *
+	 * @param string $path The path to the model
 	 */
 	protected function load( string $path ): void {
 		if ( file_exists( $path ) ) {
@@ -176,6 +194,8 @@ protected function create( AbstractConfig $config ): void {
 
 	/**
 	 * Adds the model to a list
+	*
+	 * @param Model $model The model.
 	 */
 	protected function add( Model $model ): void {
 		$this->model_list[ $model->get_name() ] = $model;
@@ -184,13 +204,17 @@ protected function add( Model $model ): void {
 	/**
 	 * Return all loaded models.
 	 *
-	 * @return array<string, \Saltus\WP\Framework\Models\Model> Associative array keyed by model name.
+	 * @return array<string, Model> Associative array keyed by model name.
 	 */
 	public function get_models(): array {
 		return $this->model_list;
 	}
 
 	/**
+	 * Get rest routes.
+	 *
+	 * @param Modeler $modeler
+	 * @param ModelRestPolicy $policy
 	 * @return list<RestRouteDefinition>
 	 */
 	public function get_rest_routes( Modeler $modeler, ModelRestPolicy $policy ): array {
@@ -203,6 +227,10 @@ public function get_rest_routes( Modeler $modeler, ModelRestPolicy $policy ): ar
 	}
 
 	/**
+	 * Get MCP tools.
+	 *
+	 * @param Modeler $modeler
+	 * @param ModelRestPolicy|null $policy
 	 * @return list<ToolInterface>
 	 */
 	public function get_mcp_tools( Modeler $modeler, ?ModelRestPolicy $policy = null ): array {

tests/MCP/MCPConfigTest.php

@@ -97,6 +97,18 @@ function ( array $category ): array {
 		$this->assertSame( 'Overridden Label', $category['label'] );
 	}
 
+	public function testGetAbilityCategoryFilterReturnsIncompleteArrayMergesWithDefaults(): void {
+		global $wp_filter_values;
+		$wp_filter_values['saltus/framework/mcp/ability_category'] = [
+			'id' => 'custom-id',
+		];
+
+		$category = MCPConfig::get_ability_category();
+		$this->assertSame( 'custom-id', $category['id'] );
+		$this->assertSame( 'Saltus Framework', $category['label'] );
+		$this->assertSame( 'Saltus Framework content modeling and administration abilities.', $category['description'] );
+	}
+
 	public function testGetAbilityPrefixReturnsDefault(): void {
 		$this->assertSame( 'saltus/', MCPConfig::get_ability_prefix() );
 	}