fix: write default app config on app init

src/commands/app/init.js

@@ -15,7 +15,7 @@ const path = require('path')
 const fs = require('fs-extra')
 const aioLogger = require('@adobe/aio-lib-core-logging')('@adobe/aio-cli-plugin-app:init', { provider: 'debug' })
 const { flags } = require('@oclif/command')
-const { loadAndValidateConfigFile, importConfigJson } = require('../../lib/import')
+const { loadAndValidateConfigFile, importConfigJson, writeDefaultAppConfig } = require('../../lib/import')
 const { getCliInfo } = require('../../lib/app-helper')
 const chalk = require('chalk')
 const { servicesToGeneratorInput } = require('../../lib/app-helper')
@@ -112,6 +112,9 @@ class InitCommand extends BaseCommand {
       }
     }
 
+    // write default app config to .aio file
+    writeDefaultAppConfig(process.cwd(), { interactive, merge })
+
     // finalize configuration data
     this.log('✔ App initialization finished!')
     return res

src/lib/import.js

@@ -95,6 +95,28 @@ function loadAndValidateConfigFile (fileOrBuffer) {
   return res
 }
 
+/**
+ * Writes default app config to .aio file
+ *
+ * @param {string} parentDir the parent folder to write the .aio file to
+ * @param {object} [flags] flags for file writing
+ * @param {boolean} [flags.overwrite=false] set to true to overwrite the existing .env file
+ * @param {boolean} [flags.merge=false] set to true to merge in the existing .env file (takes precedence over overwrite)
+ * @param {boolean} [flags.interactive=false] set to true to prompt the user for file overwrite
+ * @returns {Promise} promise from writeFile call
+ */
+function writeDefaultAppConfig (parentDir, flags) {
+  // write app config to .aio file
+  const appConfig = {
+    app: {
+      actions: 'actions',
+      dist: 'dist',
+      web: 'web-src'
+    }
+  }
+  return writeAio(appConfig, parentDir, flags)
+}
+
 /**
  * Pretty prints the json object as a string.
  * Delimited by 2 spaces.
@@ -425,13 +447,15 @@ async function writeConsoleConfig (json) {
  * @returns {Promise} promise from writeFile call
  */
 async function writeAio (json, parentFolder, flags) {
+  console.log('-----------writting aio-------------------------')
   aioLogger.debug(`writeAio - parentFolder:${parentFolder} flags:${flags}`)
   aioLogger.debug(`writeAio - json: ${prettyPrintJson(json)}`)
 
   const destination = path.join(parentFolder, AIO_FILE)
   aioLogger.debug(`writeAio - destination: ${destination}`)
 
   const data = prettyPrintJson(json)
+  console.log('-----------done-------------------------')
   return writeFile(destination, data, flags)
 }
 
@@ -598,6 +622,7 @@ module.exports = {
   loadAndValidateConfigFile,
   writeConsoleConfig,
   writeAio,
+  writeDefaultAppConfig,
   writeEnv,
   flattenObjectWithSeparator,
   importConfigJson,

test/commands/lib/import.test.js

@@ -10,7 +10,7 @@ OF ANY KIND, either express or implied. See the License for the specific languag
 governing permissions and limitations under the License.
 */
 
-const { importConfigJson, writeAio, writeEnv, mergeEnv, splitEnvLine, flattenObjectWithSeparator, loadConfigFile } = require('../../../src/lib/import')
+const { importConfigJson, writeAio, writeEnv, mergeEnv, splitEnvLine, flattenObjectWithSeparator, loadConfigFile, writeDefaultAppConfig } = require('../../../src/lib/import')
 const fs = require('fs-extra')
 const path = require('path')
 const inquirer = require('inquirer')
@@ -31,6 +31,9 @@ test('exports', () => {
   expect(writeEnv).toBeDefined()
   expect(writeEnv).toBeInstanceOf(Function)
 
+  expect(writeDefaultAppConfig).toBeDefined()
+  expect(writeDefaultAppConfig).toBeInstanceOf(Function)
+
   expect(flattenObjectWithSeparator).toBeDefined()
   expect(flattenObjectWithSeparator).toBeInstanceOf(Function)
 })
@@ -77,6 +80,19 @@ test('writeAio', async () => {
   return expect(fs.writeFile).toHaveBeenCalledTimes(3)
 })
 
+test('writeDefaultAppConfig', async () => {
+  const parentFolder = 'my-parent-folder'
+  const aioPath = path.join(parentFolder, '.aio')
+
+  writeDefaultAppConfig(parentFolder, { overwrite: true })
+  await expect(fs.writeFile.mock.calls[0][0]).toMatch(aioPath)
+
+  writeDefaultAppConfig(parentFolder, { overwrite: false })
+  await expect(fs.writeFile.mock.calls[1][0]).toMatch(aioPath)
+
+  return expect(fs.writeFile).toHaveBeenCalledTimes(2)
+})
+
 test('splitEnvLine', () => {
   expect(splitEnvLine('#comment')).toEqual(['#comment', undefined])
   expect(splitEnvLine('# comment')).toEqual(['# comment', undefined])