refactor: unified error logging (#2045)

Author: andriimredocly

__tests__/respect/local-json-server/__snapshots__/local-json-server.test.ts.snap

@@ -72,7 +72,7 @@ exports[`local-json-server > local-json-server test case 1`] = `
 └───────────────────────────────────────────────────────────────────────┴────────────┴─────────┴─────────┴──────────┘
 
 
-
  Tests exited with error 
+
 "
 `;

__tests__/respect/max-steps/__snapshots__/max-steps.test.ts.snap

@@ -159,7 +159,7 @@ exports[`should quit an infinite loop on RESPECT_MAX_STEPS 1`] = `
 └─────────────────────────────────────────────────────┴────────────┴─────────┴─────────┴──────────┘
 
 
-
  Tests exited with error 
+
 "
 `;

__tests__/respect/reusable-components/__snapshots__/reusable-components.test.ts.snap

@@ -504,7 +504,7 @@ exports[`should use inputs from CLI and env to map with resolved refs 1`] = `
 └─────────────────────────────────────────────────────────────────────────┴────────────┴─────────┴─────────┴──────────┘
 
 
-
  Tests exited with error 
+
 "
 `;

__tests__/respect/server-override-with-console-parameters/__snapshots__/server-override-with-console-parameters.test.ts.snap

@@ -61,7 +61,7 @@ exports[`should use server override from CLI and env 1`] = `
 └─────────────────────────────────────────────────────────────────────────────────────────────┴────────────┴─────────┴─────────┴──────────┘
 
 
-
  Tests exited with error 
+
 "
 `;

__tests__/respect/severity-error-level/__snapshots__/severity-error-level.test.ts.snap

@@ -188,7 +188,7 @@ exports[`should use error severity level 1`] = `
 └────────────────────────────────────────────────────────────────────┴────────────┴─────────┴─────────┴──────────┘
 
 
-
  Tests exited with error 
+
 "
 `;

__tests__/respect/step-on-failure-type-end-action/__snapshots__/step-on-failure-type-end-action.test.ts.snap

@@ -45,7 +45,7 @@ exports[`should end workflow execution, context returns to the caller with appli
 └─────────────────────────────────────────────────────────────────────────────────────┴────────────┴─────────┴─────────┴──────────┘
 
 
-
  Tests exited with error 
+
 "
 `;

__tests__/respect/workflow-failure-actions/__snapshots__/workflow-failure-actions.test.ts.snap

@@ -71,7 +71,7 @@ exports[`should execute successActions for each workflow step if it does not hav
 └──────────────────────────────────────────────────────────────────────────────┴────────────┴─────────┴─────────┴──────────┘
 
 
-
  Tests exited with error 
+
 "
 `;

packages/cli/src/wrapper.ts

@@ -70,16 +70,7 @@ export function commandWrapper<T extends CommandOptions>(
       if (err instanceof AbortFlowError) {
         // do nothing
       } else if (err instanceof HandledError) {
-        // FIXME: remove differentiator between respect and openapi-core error logging
-        if (
-          process.env.REDOCLY_CLI_COMMAND === 'respect' ||
-          process.env.REDOCLY_CLI_COMMAND === 'generate-arazzo'
-        ) {
-          logger.error(err.message + '\n');
-          logger.output('\n');
-        } else {
-          logger.error(err.message + '\n\n');
-        }
+        logger.error(err.message + '\n\n');
       } else {
         logger.error(
           'An unexpected error occurred. This is likely a bug that should be reported.\n'