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

Address code review findings: remove unreachable code and fix style issues

Author: Morgy93

src/shared/utils/ddev-utils.ts

@@ -204,7 +204,7 @@ export class DdevUtils {
             }
 
             if (result.status !== 0) {
-                 // Enhance error message with more details
+                // Enhance error message with more details
                 const enhancedError = new Error(result.stderr || 'Command execution failed');
                 enhancedError.name = 'CommandError';
                 (enhancedError as any).status = result.status;
@@ -214,10 +214,8 @@ export class DdevUtils {
                 (enhancedError as any).workspacePath = workspacePath;
                 throw enhancedError;
             }
-
-            return result.stdout;
         } catch (error: any) {
-             // If error was already thrown above, rethrow it
+            // If error was already thrown above, rethrow it
             if (error.name === 'CommandError') {
                 throw error;
             }

src/test/ddev-utils.test.ts

@@ -20,19 +20,17 @@
 import * as assert from 'assert';
 import * as sinon from 'sinon';
 import { afterEach, beforeEach } from 'mocha';
+import * as cp from 'child_process';
+import * as fs from 'fs';
 import { DdevUtils } from '../shared/utils/ddev-utils';
 
-
 suite('DdevUtils Test Suite', () => {
     let sandbox: sinon.SinonSandbox;
     let spawnSyncStub: sinon.SinonStub;
     let existsSyncStub: sinon.SinonStub;
 
     beforeEach(() => {
         sandbox = sinon.createSandbox();
-        // Use require to get the module to ensure we can stub it
-        const cp = require('child_process');
-        const fs = require('fs');
 
         // Try to stub, but handle if it fails (basic check)
         try {