From 81e6cdf5a5122b99e4a5541b0818f767e76bd6c0 Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Sat, 15 Nov 2025 16:42:32 -0500 Subject: [PATCH] fix: ensure config properties for an app are setup prior to usage --- plugins/config/triggers.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/plugins/config/triggers.go b/plugins/config/triggers.go index 11e962a04..35c1cc920 100644 --- a/plugins/config/triggers.go +++ b/plugins/config/triggers.go @@ -71,8 +71,12 @@ func TriggerInstall() error { return nil } - // migrate all created-at values from app mod-time to property + // migrate all app ENV files to config path for _, appName := range apps { + if err := common.PropertySetupApp("config", appName); err != nil { + return fmt.Errorf("Unable to setup app environment: %s", err.Error()) + } + oldEnvFile := filepath.Join(common.AppRoot(appName) + "ENV") isMigrated := common.PropertyGetDefault("config", appName, "env-migrated", "") // delete the old file on the next install @@ -91,10 +95,6 @@ func TriggerInstall() error { continue } - if err := common.PropertySetupApp("config", appName); err != nil { - return fmt.Errorf("Unable to setup app environment: %s", err.Error()) - } - // merge in the old env into the new env oldEnv, err := loadFromFile(appName, oldEnvFile) if err != nil {