Merge pull request #58 from OpenForgeProject/copilot/sub-pr-56

Handle null status in isDdevRunning and use separate args for PHPStan options

Author: Morgy93

src/services/phpstan-service.ts

@@ -70,12 +70,12 @@ export class PhpstanService extends BasePhpToolService {
                 command.push('-c', autoConfigPath);
             } else {
                 // Use individual settings when no config file is found
-                command.push(`--level=${this.config.level}`);
+                command.push('--level', String(this.config.level));
 
                 // Add exclude paths
                 if (this.config.excludePaths && this.config.excludePaths.length > 0) {
                     for (const excludePath of this.config.excludePaths) {
-                        command.push(`--exclude=${excludePath}`);
+                        command.push('--exclude', excludePath);
                     }
                 }
             }

src/shared/utils/ddev-utils.ts

@@ -67,6 +67,18 @@ export class DdevUtils {
                 cwd: workspacePath,
                 encoding: 'utf-8'
             });
+            if (result.error) {
+                vscode.window.showErrorMessage(
+                    'Failed to execute "ddev". Please ensure DDEV is installed and available in your PATH.',
+                );
+                return false;
+            }
+            if (result.status === null) {
+                vscode.window.showErrorMessage(
+                    'The "ddev" command did not exit normally. Please check your DDEV installation.',
+                );
+                return false;
+            }
             return result.status === 0;
         } catch (error) {
             return false;