Method updateJob

server/modules/jobModule.js:939–1055

(jobId, updatedFields)

  }

  async updateJob(jobId, updatedFields) {
    logger.debug({ jobId, status: updatedFields.status }, 'updateJob called');
    if (updatedFields.data && updatedFields.data.videos) {
      logger.debug({ jobId, videoCount: updatedFields.data.videos.length }, 'updateJob data contains videos');
    }

    const job = this.jobs[jobId];
    if (!job) {
      logger.warn('Job to update did not exist!');
      return;
    }

    // Non-download jobs (e.g. Import Subscriptions) manage their own output field.
    const jobIsDownload = isDownloadJob(job.jobType);

    if (
      jobIsDownload && (
        updatedFields.status === 'Complete' ||
        updatedFields.status === 'Error' ||
        updatedFields.status === 'Complete with Warnings' ||
        updatedFields.status === 'Terminated'
      )
    ) {
      // downloadModule already sends proper completion messages with finalSummary
      // Only send the downloadComplete event for backwards compatibility
      MessageEmitter.emitMessage(
        'broadcast',
        null,
        'download',
        'downloadComplete',
        { text: 'Download job completed.', videos: updatedFields.data?.videos || [] }
      );

      // Only modify output and status for actual completions, not terminations
      if (updatedFields.status !== 'Terminated') {
        let numVideos = updatedFields.data?.videos?.length || 0;
        updatedFields.output = numVideos + ' videos.';
        if (updatedFields.status !== 'Complete with Warnings') {
          updatedFields.status = 'Complete';
        }
      }
    }

    // Update in-memory job
    for (let field in updatedFields) {
      job[field] = updatedFields[field];
    }

    // Save only THIS job to DB, don't iterate through all jobs
    const isCompletedJob = updatedFields.status === 'Complete' ||
                           updatedFields.status === 'Complete with Warnings' ||
                           updatedFields.status === 'Error' ||
                           updatedFields.status === 'Terminated' ||
                           updatedFields.status === 'Killed';

    if (isCompletedJob && jobIsDownload) {
      // For completed download jobs, reload videos from DB to ensure accurate counts
      // This is especially important for multi-group downloads where each group