fix: data race writing on setting file

Composer/packages/server/src/models/bot/botProject.ts

@@ -242,9 +242,9 @@ export class BotProject implements IBotProject {
       // if endpointKey has not been set, migrate old key to new key
       if (!settings.qna.endpointKey) {
         settings.qna.endpointKey = settings.qna.endpointkey;
+        delete settings.qna.endpointkey;
+        await this.updateEnvSettings(settings);
       }
-      delete settings.qna.endpointkey;
-      await this.updateEnvSettings(settings);
     }
 
     // set these after migrating qna settings to not write them to storage

Composer/packages/server/src/models/settings/defaultSettingManager.ts

@@ -121,6 +121,7 @@ export class DefaultSettingManager extends FileSettingManager {
         maxImbalanceRatio: -1,
       },
       skill: {},
+      skillConfiguration: {},
       defaultLanguage: 'en-us',
       languages: ['en-us'],
       customFunctions: [],

extensions/packageManager/src/node/index.ts

@@ -386,7 +386,7 @@ export default async (composer: IExtensionRegistration): Promise<void> => {
                 newlyInstalledPlugin &&
                 !currentProject.settings.runtimeSettings?.components?.find((p) => p.name === newlyInstalledPlugin.name)
               ) {
-                const newSettings = currentProject.settings;
+                const newSettings = await currentProject.getEnvSettings();
                 if (!newSettings.runtimeSettings) {
                   newSettings.runtimeSettings = {
                     components: [],
@@ -475,7 +475,7 @@ export default async (composer: IExtensionRegistration): Promise<void> => {
 
           // update the settings.components array
           if (currentProject.settings.runtimeSettings?.components?.find((p) => p.name === packageName)) {
-            const newSettings = currentProject.settings;
+            const newSettings = await currentProject.getEnvSettings();
             newSettings.runtimeSettings.components = newSettings.runtimeSettings.components.filter(
               (p) => p.name !== packageName
             );