refactor: remove virtual execution lifecycle aliases

This commit is contained in:
Adam Weidman
2026-03-08 23:31:18 -04:00
parent 0fc4247287
commit fff69df93e
3 changed files with 18 additions and 64 deletions
@@ -91,7 +91,7 @@ type ManagedExecutionState = VirtualExecutionState | ExternalExecutionState;
*/
export class ExecutionLifecycleService {
private static readonly EXIT_INFO_TTL_MS = 5 * 60 * 1000;
private static nextVirtualExecutionId = 2_000_000_000;
private static nextExecutionId = 2_000_000_000;
private static activeExecutions = new Map<number, ManagedExecutionState>();
private static activeResolvers = new Map<
@@ -121,10 +121,10 @@ export class ExecutionLifecycleService {
}, this.EXIT_INFO_TTL_MS).unref();
}
private static allocateVirtualExecutionId(): number {
let executionId = ++this.nextVirtualExecutionId;
private static allocateExecutionId(): number {
let executionId = ++this.nextExecutionId;
while (this.activeExecutions.has(executionId)) {
executionId = ++this.nextVirtualExecutionId;
executionId = ++this.nextExecutionId;
}
return executionId;
}
@@ -162,7 +162,7 @@ export class ExecutionLifecycleService {
this.activeResolvers.clear();
this.activeListeners.clear();
this.exitedExecutionInfo.clear();
this.nextVirtualExecutionId = 2_000_000_000;
this.nextExecutionId = 2_000_000_000;
}
static registerExecution(
@@ -196,7 +196,7 @@ export class ExecutionLifecycleService {
onKill?: () => void,
executionMethod: ExecutionMethod = 'none',
): ExecutionHandle {
const executionId = this.allocateVirtualExecutionId();
const executionId = this.allocateExecutionId();
this.activeExecutions.set(executionId, {
executionMethod,
@@ -219,17 +219,6 @@ export class ExecutionLifecycleService {
};
}
/**
* @deprecated Use createExecution() for new call sites.
*/
static createVirtualExecution(
initialOutput = '',
onKill?: () => void,
executionMethod: ExecutionMethod = 'none',
): ExecutionHandle {
return this.createExecution(initialOutput, onKill, executionMethod);
}
static appendOutput(executionId: number, chunk: string): void {
const execution = this.activeExecutions.get(executionId);
if (!execution || chunk.length === 0) {
@@ -284,7 +273,7 @@ export class ExecutionLifecycleService {
);
}
static completeVirtualExecution(
static completeExecution(
executionId: number,
options?: ExecutionCompletionOptions,
): void {
@@ -314,16 +303,6 @@ export class ExecutionLifecycleService {
});
}
/**
* @deprecated Use completeVirtualExecution() for new call sites.
*/
static completeExecution(
executionId: number,
options?: ExecutionCompletionOptions,
): void {
this.completeVirtualExecution(executionId, options);
}
static completeWithResult(
executionId: number,
result: ExecutionResult,