refactor(core): Remove inferral of execution status

Execution status is non-nullable since #9483 and we have been tracking missing edge cases for a month since #9476 and had zero Sentry reports, so it's time to finally remove inferral.

https://n8nio.slack.com/archives/C069HS026UF/p1718267902737659

packages/cli/src/executions/execution.service.ts

@@ -9,7 +9,6 @@ import type {
 	ExecutionStatus,
 } from 'n8n-workflow';
 import {
-	ErrorReporterProxy as EventReporter,
 	ApplicationError,
 	ExecutionStatusList,
 	Workflow,
@@ -27,7 +26,6 @@ import { NodeTypes } from '@/NodeTypes';
 import { Queue } from '@/Queue';
 import type { ExecutionRequest, ExecutionSummaries } from './execution.types';
 import { WorkflowRunner } from '@/WorkflowRunner';
-import { getStatusUsingPreviousExecutionStatusMethod } from './executionHelpers';
 import type { IGetExecutionsQueryFilter } from '@db/repositories/execution.repository';
 import { ExecutionRepository } from '@db/repositories/execution.repository';
 import { WorkflowRepository } from '@db/repositories/workflow.repository';
@@ -111,12 +109,6 @@ export class ExecutionService {
 			return undefined;
 		}
 
-		if (!execution.status) {
-			const { data, workflowData, ...rest } = execution;
-			EventReporter.info('Detected `null` execution status', { extra: { execution: rest } });
-			execution.status = getStatusUsingPreviousExecutionStatusMethod(execution);
-		}
-
 		return execution;
 	}