fix: attempt to migrate values on every operator cycle (#2807)

Author: merll

src/operator/apl-operator.test.ts

@@ -212,13 +212,13 @@ describe('AplOperator', () => {
       expect(mockAplOps.applyAsAppsTeams).toHaveBeenCalled()
       expect(mockAplOps.apply).not.toHaveBeenCalled()
     })
-    test('should not run migrate and validate when run is reconcile', async () => {
+    test('should run migrate and but not validate when run is reconcile', async () => {
       Object.defineProperty(aplOperator, 'isApplying', { value: false, configurable: true })
 
       await aplOperator.runApplyIfNotBusy(ApplyTrigger.Reconcile, false)
 
       expect(mockAplOps.apply).toHaveBeenCalled()
-      expect(mockAplOps.migrate).not.toHaveBeenCalled()
+      expect(mockAplOps.migrate).toHaveBeenCalled()
       expect(mockAplOps.validateValues).not.toHaveBeenCalled()
     })
 

src/operator/apl-operator.ts

@@ -75,9 +75,8 @@ export class AplOperator {
       this.d.info('Write default values to env repo')
       await writeValues(defaultValues)
 
+      await this.aplOps.migrate()
       if (trigger === ApplyTrigger.Poll) {
-        await this.aplOps.migrate()
-
         this.d.info(`[${trigger}] Starting validation process`)
         await this.aplOps.validateValues()
         this.d.info(`[${trigger}] Validation process completed`)