Merge pull request #459 from Extra-Chill/fix/workspace-tools-sandbox-mode

Expose workspace tools in sandbox mode

Author: chubes4

inc/Tools/WorkspaceTools.php

@@ -76,8 +76,8 @@ public function check_configuration( $configured, $tool_id )
      */
     public function __construct()
     {
-        $contexts        = array( 'chat', 'pipeline' );
-        $policy_contexts = array( 'chat', 'pipeline' );
+        $contexts        = array( 'chat', 'pipeline', 'sandbox' );
+        $policy_contexts = array( 'chat', 'pipeline', 'sandbox' );
         $policy_meta     = array( 'requires_opt_in' => true );
         $this->registerTool('workspace_path', array( $this, 'getPathDefinition' ), $contexts, array( 'ability' => 'datamachine/workspace-path' ));
         $this->registerTool('workspace_capabilities', array( $this, 'getCapabilitiesDefinition' ), $contexts, array( 'ability' => 'datamachine/workspace-capabilities' ));

inc/Workspace/RemoteWorkspaceBackend.php

@@ -22,7 +22,10 @@ class RemoteWorkspaceBackend {
 	 * Whether the remote backend should handle workspace operations.
 	 */
 	public static function should_handle(): bool {
-		return ! \DataMachineCode\Support\GitRunner::is_available();
+		$should_handle = ! \DataMachineCode\Support\GitRunner::is_available();
+		return function_exists('apply_filters')
+			? (bool) apply_filters('datamachine_code_remote_workspace_backend_should_handle', $should_handle)
+			: $should_handle;
 	}
 
 	/**

tests/smoke-workspace-policy-tools.php

@@ -91,7 +91,7 @@ static function ( array $tools ) use ( $tool_id, $definition_callback, $contexts
     );
 
     foreach ( $default_pipeline_tools as $tool ) {
-        $assert("{$tool} remains default pipeline-visible", array( 'chat', 'pipeline' ) === ( $tools[ $tool ]['modes'] ?? null ));
+        $assert("{$tool} remains default pipeline-visible and sandbox-visible", array( 'chat', 'pipeline', 'sandbox' ) === ( $tools[ $tool ]['modes'] ?? null ));
     }
 
     $show_definition = $workspace_tools->getShowDefinition();
@@ -123,7 +123,7 @@ static function ( array $tools ) use ( $tool_id, $definition_callback, $contexts
     );
 
     foreach ( $policy_tools as $tool ) {
-        $assert("{$tool} is available in real chat and pipeline modes", array( 'chat', 'pipeline' ) === ( $tools[ $tool ]['modes'] ?? null ));
+        $assert("{$tool} is available in real chat, pipeline, and sandbox modes", array( 'chat', 'pipeline', 'sandbox' ) === ( $tools[ $tool ]['modes'] ?? null ));
         $assert("{$tool} requires explicit opt-in for pipeline use", true === ( $tools[ $tool ]['requires_opt_in'] ?? null ));
     }