don't dispose of previousCommandMarker

Author: meganrogge

src/vs/workbench/contrib/terminal/browser/capabilities/commandDetectionCapability.ts

@@ -213,7 +213,6 @@ export class CommandDetectionCapability implements ICommandDetectionCapability {
 			this._logService.debug('CommandDetectionCapability#onCommandFinished', newCommand);
 			this._onCommandFinished.fire(newCommand);
 		}
-		this._currentCommand.previousCommandMarker?.dispose();
 		this._currentCommand.previousCommandMarker = this._currentCommand.commandStartMarker;
 		this._currentCommand = {};
 	}

src/vs/workbench/contrib/terminal/browser/terminalInstance.ts

@@ -815,7 +815,7 @@ export class TerminalInstance extends Disposable implements ITerminalInstance {
 						description,
 						id: entry.timestamp.toString(),
 						command: entry,
-						buttons: (!entry.endMarker?.isDisposed && !entry.marker?.isDisposed && (entry.endMarker!.line - entry.marker!.line > 0)) ? buttons : undefined
+						buttons: entry.hasOutput ? buttons : undefined
 					});
 					commandMap.add(label);
 				}