Topcoder NestJS Autopilot Phase Transition Logic Optimization

Author: billsedison

src/autopilot/dto/autopilot.dto.ts

@@ -6,6 +6,7 @@ import {
   IsDateString,
   IsUUID,
 } from 'class-validator';
+import { AutopilotOperator } from '../interfaces/autopilot.interface';
 
 export class PhaseTransitionDto {
   @IsDateString()
@@ -23,8 +24,8 @@ export class PhaseTransitionDto {
   @IsEnum(['START', 'END'])
   state: 'START' | 'END';
 
-  @IsString()
-  operator: string;
+  @IsEnum(AutopilotOperator)
+  operator: AutopilotOperator | string;
 
   @IsString()
   projectStatus: string;
@@ -40,8 +41,8 @@ export class ChallengeUpdateDto {
   @IsString()
   status: string;
 
-  @IsString()
-  operator: string;
+  @IsEnum(AutopilotOperator)
+  operator: AutopilotOperator | string;
 }
 
 export class CommandDto {
@@ -57,6 +58,6 @@ export class CommandDto {
   @IsObject()
   parameters: Record<string, any>;
 
-  @IsString()
-  operator: string;
+  @IsEnum(AutopilotOperator)
+  operator: AutopilotOperator | string;
 }

src/autopilot/interfaces/autopilot.interface.ts

@@ -1,3 +1,19 @@
+export enum AutopilotOperator {
+  // System operators for internal autopilot operations
+  SYSTEM = 'system',
+  SYSTEM_SCHEDULER = 'system-scheduler',
+  SYSTEM_NEW_CHALLENGE = 'system-new-challenge',
+  SYSTEM_RECOVERY = 'system-recovery',
+  SYSTEM_SYNC = 'system-sync',
+  SYSTEM_PHASE_CHAIN = 'system-phase-chain',
+
+  // Administrative operators
+  ADMIN = 'admin',
+
+  // User operators (when operator comes from external sources)
+  USER = 'user',
+}
+
 export interface BaseMessage {
   topic: string;
   originator: string;
@@ -10,7 +26,7 @@ export interface PhaseTransitionPayload {
   phaseId: string; // Changed from number to string to support UUIDs from the API
   phaseTypeName: string;
   state: 'START' | 'END';
-  operator: string;
+  operator: AutopilotOperator | string; // Allow both enum and string for flexibility
   projectStatus: string;
   date?: string;
   challengeId: string; // Changed to string to support UUIDs
@@ -20,7 +36,7 @@ export interface ChallengeUpdatePayload {
   projectId: number;
   challengeId: string; // Changed to string to support UUIDs
   status: string;
-  operator: string;
+  operator: AutopilotOperator | string; // Allow both enum and string for flexibility
   date?: string;
   // This nested structure may be present in Kafka messages from the API
   phases?: {
@@ -31,7 +47,7 @@ export interface ChallengeUpdatePayload {
 
 export interface CommandPayload {
   command: string;
-  operator: string;
+  operator: AutopilotOperator | string; // Allow both enum and string for flexibility
   projectId?: number;
   challengeId?: string; // Added challengeId for new command handling
   date?: string;