fix: prevent duplicate MSO after table (#1267)

* fix: prevent duplicate MSO after table

* chore: always stop stages in catch

Author: mdonnalley

src/commands/project/deploy/start.ts

@@ -279,6 +279,7 @@ export default class DeployMetadata extends SfCommand<DeployResultJson> {
 
     const result = await deploy.pollStatus({ timeout: flags.wait });
     process.exitCode = determineExitCode(result);
+    this.stages.stop();
     const formatter = new DeployResultFormatter(result, flags);
 
     if (!this.jsonEnabled()) {
@@ -292,10 +293,11 @@ export default class DeployMetadata extends SfCommand<DeployResultJson> {
   }
 
   protected catch(error: Error | SfError): Promise<never> {
+    this.stages?.update({ status: 'Failed' });
+    this.stages?.error();
+
     if (error instanceof SourceConflictError && error.data) {
       if (!this.jsonEnabled()) {
-        this.stages?.update({ status: 'Failed' });
-        this.stages?.error();
         writeConflictTable(error.data);
         // set the message and add plugin-specific actions
         return super.catch({