Update feature capabilities matrix

Author: LC43

docs/MCP.md

@@ -131,11 +131,14 @@ Each individual framework capability can be gated in the model's `config` array.
 ### Resolution Rules
 
 For each feature/capability configuration section:
-1. **Omitted (Null):** If a capability config section is omitted from the model configuration, the feature defaults to **enabled** for both REST and MCP.
+1. **Omitted (Null):** If a capability config section is omitted from the model configuration, the feature's availability defaults to the master model option:
+   - For **REST API**: falls back to `show_in_rest` (which itself defaults to `true`).
+   - For **MCP Tools**: falls back to `mcp_tools` (which itself defaults to `false` if omitted).
+   - If the respective master option is omitted/false, the feature is **disabled**. If the master option is `true`, the feature is **enabled**.
 2. **Boolean Value:** If defined as a simple boolean (e.g., `'meta' => false` or `'features' => ['duplicate' => false]`), it acts as a joint gate. A value of `false` disables both REST and MCP for that capability; a value of `true` enables both.
 3. **Array Value:** If defined as an array, REST and MCP gating can be configured independently:
-   - **REST Route Gating:** Governed by the `show_in_rest` key in the section array. If the key is omitted, REST is **enabled** (`true`). If present, it resolves to its boolean value.
-   - **MCP Tool Gating:** Governed by the `show_in_mcp` key in the section array. If the key is omitted, MCP is **enabled** (`true`). If present, it resolves to its boolean value.
+   - **REST Route Gating:** Governed by the `show_in_rest` key in the section array. If present, it resolves to its boolean value. If omitted, it falls back to matching the master model `show_in_rest` option.
+   - **MCP Tool Gating:** Governed by the `show_in_mcp` key in the section array. If present, it resolves to its boolean value. If omitted, it falls back to matching the master model `mcp_tools` option.
 
 Enable all Saltus REST-backed and MCP capabilities for a model:
 
@@ -170,12 +173,12 @@ return [
 		'settings' => false,
 
 		'features' => [
-			// 3. Array style: enabled for REST, and defaults to enabled for MCP
+			// 3. Array style: enabled for REST, and defaults to matching master options (enabled) for MCP
 			'duplicate'     => [
 				'show_in_rest' => true,
 			],
 			// 4. Omitted config for single_export and drag_and_drop:
-			// both default to enabled for REST and MCP
+			// both default to matching the master options (enabled here because show_in_rest & mcp_tools are true)
 		],
 	],
 ];

docs/ROADMAP.md

@@ -481,3 +481,21 @@ AI write -> draft/pending/revision -> human approval -> publish
 ---
 
 **Exit criteria (Phase 6 overall):** AI governance is configurable per model via `ai_context`. Mutating MCP tools respect context rules and default to review-queue creation. Inside-admin assistants are operational for configured models. All features are tested.
+
+---
+
+### Phase 7: Advanced Dependency Injection & Container Hardening (v2.3+)
+
+**Theme:** Upgrade the framework's dependency injection container to support reflection-based parameter resolution (autowiring) for third-party services, avoiding standard constructor mapping errors.
+
+| Item | Status |
+|------|--------|
+| `ReflectionInstantiator` class implementing `Instantiator` | ○ Pending |
+| Positional constructor parameter resolution and dependency matching | ○ Pending |
+| Constructor parameter default value fallbacks | ○ Pending |
+| Clean validation and exception flow for unresolved parameters | ○ Pending |
+| Remove requirement for `Assembly::make` boilerplate on custom services | ○ Pending |
+| Container autowiring unit tests (`tests/Unit/Infrastructure/Container/`) | ○ Pending |
+| Developer documentation update for custom service constructors | ○ Pending |
+
+**Exit criteria:** Developers can register custom services in the container with standard typed/positional constructor arguments. The container uses PHP Reflection to map parameter names to container keys, falling back to default arguments or throwing descriptive runtime exceptions when dependencies cannot be resolved.

src/MCP/McpPolicy.php

@@ -57,19 +57,28 @@ public function has_capability( string $capability, ?string $model_type = null )
 	 * @return bool
 	 */
 	public function is_enabled( Model $model, string $capability ): bool {
-		$options = $model->get_options();
-
-		if ( empty( $options['mcp_tools'] ) ) {
-			return false;
+		$options    = $model->get_options();
+		$global_val = null;
+		if ( array_key_exists( 'mcp_tools', $options ) ) {
+			$global_val = (bool) $options['mcp_tools'];
 		}
 
-		if ( $capability === ModelRestPolicy::CAPABILITY_HEALTH || $capability === ModelRestPolicy::CAPABILITY_MODELS ) {
+		if ( $capability === ModelRestPolicy::CAPABILITY_HEALTH ) {
 			return true;
 		}
 
-		$config = $model->get_config();
+		if ( $capability === ModelRestPolicy::CAPABILITY_MODELS ) {
+			return $global_val === true;
+		}
+
+		$config      = $model->get_config();
+		$feature_val = $this->resolve_feature_value( $config, $capability );
+
+		if ( $feature_val !== null ) {
+			return $feature_val;
+		}
 
-		return $this->resolve_show_in_mcp( $config, $capability );
+		return $global_val === true;
 	}
 
 	/**
@@ -107,13 +116,13 @@ private function get_capability_config( array $config, string $capability ) {
 	 * Resolve whether a capability should be shown in MCP.
 	 *
 	 * @param array<string, mixed> $config
-	 *
-	 * @return bool
+	 * @param string               $capability
+	 * @return bool|null
 	 */
-	private function resolve_show_in_mcp( array $config, string $capability ): bool {
+	private function resolve_feature_value( array $config, string $capability ): ?bool {
 		$section = $this->get_capability_config( $config, $capability );
 		if ( $section === null ) {
-			return true;
+			return null;
 		}
 
 		if ( ! is_array( $section ) ) {

src/Rest/ModelRestPolicy.php

@@ -40,19 +40,28 @@ public function has_capability( string $capability, ?string $model_type = null )
 	}
 
 	public function is_enabled( Model $model, string $capability ): bool {
-		$options = $this->get_model_options( $model );
-
-		if ( array_key_exists( 'show_in_rest', $options ) && $options['show_in_rest'] === false ) {
-			return false;
+		$options    = $this->get_model_options( $model );
+		$global_val = null;
+		if ( array_key_exists( 'show_in_rest', $options ) ) {
+			$global_val = (bool) $options['show_in_rest'];
 		}
 
-		if ( $capability === self::CAPABILITY_HEALTH || $capability === self::CAPABILITY_MODELS ) {
+		if ( $capability === self::CAPABILITY_HEALTH ) {
 			return true;
 		}
 
-		$config = $model->get_config();
+		if ( $capability === self::CAPABILITY_MODELS ) {
+			return $global_val !== false;
+		}
+
+		$config      = $model->get_config();
+		$feature_val = $this->resolve_feature_value( $config, $capability );
 
-		return $this->resolve_show_in_rest( $config, $capability );
+		if ( $feature_val !== null ) {
+			return $feature_val;
+		}
+
+		return $global_val === true;
 	}
 
 	/**
@@ -87,12 +96,16 @@ private function get_capability_config( array $config, string $capability ) {
 	}
 
 	/**
+	 * Resolve the capability value from the feature configuration.
+	 *
 	 * @param array<string, mixed> $config
+	 * @param string               $capability
+	 * @return bool|null
 	 */
-	private function resolve_show_in_rest( array $config, string $capability ): bool {
+	private function resolve_feature_value( array $config, string $capability ): ?bool {
 		$section = $this->get_capability_config( $config, $capability );
 		if ( $section === null ) {
-			return true;
+			return null;
 		}
 
 		if ( ! is_array( $section ) ) {