@algocare/react-native-code-push 12.1.0 → 12.2.0

package/CodePush.js

@@ -97,6 +97,7 @@ async function checkForUpdate(
 
         if (
           !prNumber &&
+          updateRequest.label &&
           updateRequest.label.includes('-') &&
           releaseHistoryVersions.includes(updateRequest.label)
         ) {

package/cli/commands/common.js

@@ -157,7 +157,7 @@ async function getAppVersionValueFromList(
     ])
     appVersion = appVersionAnswer.appVersion
   } else {
-    appVersion = appVersionSelectionAnswer.appVersion
+    appVersion = appVersionSelectionAnswer.appVersionChoice
   }
 
   return appVersion

package/cli/commands/createHistoryCommand/index.js

@@ -13,26 +13,60 @@ const {
 program
   .command('create-history')
   .description('Creates a new release history for the binary app.')
+  .option('-a, --app <string>', 'target app (user/device)')
+  .option('-p, --platform <string>', 'target platform (ios/android)')
+  .option('-i, --identifier <string>', 'target env identifier (dev/stg/prd)')
+  .option('-b, --binary-version <string>', 'target binary version (x.y.z)')
   .option(
     '-c, --config <path>',
     'set config file name (JS/TS)',
     COMPILED_CONFIG_FILE_NAME
   )
   .action(async (options) => {
-    const config = await findAndReadConfigFile(process.cwd(), options.config)
-    // app
-    const app = await getAppValue()
-    // platform
-    const platform = await getPlatformValue(app)
-    // identifier
-    const identifier = await getIdentifierValue()
-    // binaryVersion
-    const binaryVersion = await getBinaryVersionValueFromInput(
-      config.getBinaryVersionList,
-      app,
-      platform,
-      identifier
-    )
+    if (options.app && !['user', 'device'].includes(options.app)) {
+      throw new Error('App must be either "user" or "device"')
+    }
+
+    if (options.platform && !['ios', 'android'].includes(options.platform)) {
+      throw new Error('Platform must be either "ios" or "android"')
+    }
+
+    if (options.app === 'device' && options.platform === 'ios') {
+      throw new Error('Device app is not supported on iOS')
+    }
+
+    if (
+      options.identifier &&
+      !['dev', 'stg', 'prd'].includes(options.identifier)
+    ) {
+      throw new Error('Identifier must be either "dev" or "stg" or "prd"')
+    }
+
+    const semverRegex = /^(0|[1-9]\d*)\.(0|[1-9]\d*)\.(0|[1-9]\d*)$/
+    if (options.binaryVersion && !semverRegex.test(options.binaryVersion)) {
+      throw new Error('Binary version must be a valid semver format (x.y.z)')
+    }
+
+    const { setReleaseHistory, getBinaryVersionList, setBinaryVersionList } =
+      await findAndReadConfigFile(process.cwd(), options.config)
+
+    const app = options.app || (await getAppValue())
+
+    const platform =
+      options.platform ||
+      (app === 'device' && 'android') ||
+      (await getPlatformValue(app))
+
+    const identifier = options.identifier || (await getIdentifierValue())
+
+    const binaryVersion =
+      options.binaryVersion ||
+      (await getBinaryVersionValueFromInput(
+        getBinaryVersionList,
+        app,
+        platform,
+        identifier
+      ))
 
     const answers = {
       app,
@@ -44,7 +78,7 @@ program
     await createReleaseHistory(
       answers.app,
       answers.binaryVersion,
-      config.setReleaseHistory,
+      setReleaseHistory,
       answers.platform
     )
 
@@ -52,7 +86,7 @@ program
       answers.app,
       answers.binaryVersion,
       answers.platform,
-      config.getBinaryVersionList,
-      config.setBinaryVersionList
+      getBinaryVersionList,
+      setBinaryVersionList
     )
   })

package/cli/commands/releaseCommand/index.js

@@ -83,7 +83,6 @@ program
       throw new Error('App version must be a valid semver format (x.y.z)')
     }
 
-    let answers = {}
     const {
       bundleUploader,
       getReleaseHistory,
@@ -91,49 +90,39 @@ program
       getBinaryVersionList,
     } = await findAndReadConfigFile(process.cwd(), options.config)
 
-    if (
-      options.app &&
-      options.identifier &&
-      options.binaryVersion &&
-      options.appVersion &&
-      (options.app === 'device' || options.platform)
-    ) {
-      answers = {
-        app: options.app,
-        platform: options.platform || (options.app === 'device' && 'android'),
-        identifier: options.identifier,
-        binaryVersion: options.binaryVersion,
-        appVersion: options.appVersion,
-      }
-    } else {
-      // app
-      const app = await getAppValue()
-      // platform
-      const platform = await getPlatformValue(app)
-      // identifier
-      const identifier = await getIdentifierValue()
-      // binaryVersion
-      const binaryVersion = await getBinaryVersionValueFromInput(
+    const app = options.app || (await getAppValue())
+
+    const platform =
+      options.platform ||
+      (app === 'device' && 'android') ||
+      (await getPlatformValue(app))
+
+    const identifier = options.identifier || (await getIdentifierValue())
+
+    const binaryVersion =
+      options.binaryVersion ||
+      (await getBinaryVersionValueFromInput(
         getBinaryVersionList,
         app,
         platform,
         identifier
-      )
-      // appVersion
-      const appVersion = await getAppVersionValueFromInput(
+      ))
+
+    const appVersion =
+      options.appVersion ||
+      (await getAppVersionValueFromInput(
         getReleaseHistory,
         binaryVersion,
         app,
         platform
-      )
+      ))
 
-      answers = {
-        app,
-        platform,
-        identifier,
-        binaryVersion,
-        appVersion,
-      }
+    const answers = {
+      app,
+      platform,
+      identifier,
+      binaryVersion,
+      appVersion,
     }
 
     await release(

package/cli/commands/showHistoryCommand/index.js

@@ -13,26 +13,60 @@ program
   .description(
     'Retrieves and prints the release history of a specific binary version.\n`getReleaseHistory` function should be implemented in the config file.'
   )
+  .option('-a, --app <string>', 'target app (user/device)')
+  .option('-p, --platform <string>', 'target platform (ios/android)')
+  .option('-i, --identifier <string>', 'target env identifier (dev/stg/prd)')
+  .option('-b, --binary-version <string>', 'target binary version (x.y.z)')
   .option(
     '-c, --config <path>',
     'configuration file name (JS/TS)',
     COMPILED_CONFIG_FILE_NAME
   )
   .action(async (options) => {
-    const config = await findAndReadConfigFile(process.cwd(), options.config)
-    // app
-    const app = await getAppValue()
-    // platform
-    const platform = await getPlatformValue(app)
-    // identifier
-    const identifier = await getIdentifierValue()
-    // binaryVersion
-    const binaryVersion = await getBinaryVersionValueFromList(
-      config.getBinaryVersionList,
-      app,
-      platform,
-      identifier
-    )
+    if (options.app && !['user', 'device'].includes(options.app)) {
+      throw new Error('App must be either "user" or "device"')
+    }
+
+    if (options.platform && !['ios', 'android'].includes(options.platform)) {
+      throw new Error('Platform must be either "ios" or "android"')
+    }
+
+    if (options.app === 'device' && options.platform === 'ios') {
+      throw new Error('Device app is not supported on iOS')
+    }
+
+    if (
+      options.identifier &&
+      !['dev', 'stg', 'prd'].includes(options.identifier)
+    ) {
+      throw new Error('Identifier must be either "dev" or "stg" or "prd"')
+    }
+
+    const semverRegex = /^(0|[1-9]\d*)\.(0|[1-9]\d*)\.(0|[1-9]\d*)$/
+    if (options.binaryVersion && !semverRegex.test(options.binaryVersion)) {
+      throw new Error('Binary version must be a valid semver format (x.y.z)')
+    }
+
+    const { getBinaryVersionList, getReleaseHistory } =
+      await findAndReadConfigFile(process.cwd(), options.config)
+
+    const app = options.app || (await getAppValue())
+
+    const platform =
+      options.platform ||
+      (app === 'device' && 'android') ||
+      (await getPlatformValue(app))
+
+    const identifier = options.identifier || (await getIdentifierValue())
+
+    const binaryVersion =
+      options.binaryVersion ||
+      (await getBinaryVersionValueFromList(
+        getBinaryVersionList,
+        app,
+        platform,
+        identifier
+      ))
 
     const answers = {
       app,
@@ -41,7 +75,7 @@ program
       binaryVersion,
     }
 
-    const releaseHistory = await config.getReleaseHistory(
+    const releaseHistory = await getReleaseHistory(
       answers.app,
       answers.binaryVersion,
       answers.platform

package/cli/commands/updateHistoryCommand/index.js

@@ -15,6 +15,11 @@ program
   .description(
     'Updates the release history for a specific binary version.\n`getReleaseHistory`, `setReleaseHistory` functions should be implemented in the config file.'
   )
+  .option('-a, --app <string>', 'target app (user/device)')
+  .option('-p, --platform <string>', 'target platform (ios/android)')
+  .option('-i, --identifier <string>', 'target env identifier (dev/stg/prd)')
+  .option('-b, --binary-version <string>', 'target binary version (x.y.z)')
+  .option('-v, --app-version <string>', 'target codepush version (x.y.z)')
   .option(
     '-c, --config <path>',
     'set config file name (JS/TS)',
@@ -28,27 +33,63 @@ program
   )
   .option('--no-enable', 'make the release to be disabled')
   .action(async (options) => {
-    const config = await findAndReadConfigFile(process.cwd(), options.config)
-    // app
-    const app = await getAppValue()
-    // platform
-    const platform = await getPlatformValue(app)
-    // identifier
-    const identifier = await getIdentifierValue()
-    // binaryVersion
-    const binaryVersion = await getBinaryVersionValueFromList(
-      config.getBinaryVersionList,
-      app,
-      platform,
-      identifier
-    )
-    // appVersion
-    const appVersion = await getAppVersionValueFromList(
-      config.getReleaseHistory,
-      binaryVersion,
-      app,
-      platform
-    )
+    if (options.app && !['user', 'device'].includes(options.app)) {
+      throw new Error('App must be either "user" or "device"')
+    }
+
+    if (options.platform && !['ios', 'android'].includes(options.platform)) {
+      throw new Error('Platform must be either "ios" or "android"')
+    }
+
+    if (options.app === 'device' && options.platform === 'ios') {
+      throw new Error('Device app is not supported on iOS')
+    }
+
+    if (
+      options.identifier &&
+      !['dev', 'stg', 'prd'].includes(options.identifier)
+    ) {
+      throw new Error('Identifier must be either "dev" or "stg" or "prd"')
+    }
+
+    const semverRegex = /^(0|[1-9]\d*)\.(0|[1-9]\d*)\.(0|[1-9]\d*)$/
+    if (options.binaryVersion && !semverRegex.test(options.binaryVersion)) {
+      throw new Error('Binary version must be a valid semver format (x.y.z)')
+    }
+
+    if (options.appVersion && !semverRegex.test(options.appVersion)) {
+      throw new Error('App version must be a valid semver format (x.y.z)')
+    }
+
+    const { getBinaryVersionList, getReleaseHistory, setReleaseHistory } =
+      await findAndReadConfigFile(process.cwd(), options.config)
+
+    const app = options.app || (await getAppValue())
+
+    const platform =
+      options.platform ||
+      (app === 'device' && 'android') ||
+      (await getPlatformValue(app))
+
+    const identifier = options.identifier || (await getIdentifierValue())
+
+    const binaryVersion =
+      options.binaryVersion ||
+      (await getBinaryVersionValueFromList(
+        getBinaryVersionList,
+        app,
+        platform,
+        identifier
+      ))
+
+    const appVersion =
+      options.appVersion ||
+      (await getAppVersionValueFromList(
+        getReleaseHistory,
+        binaryVersion,
+        app,
+        platform
+      ))
 
     const answers = {
       app,
@@ -70,8 +111,8 @@ program
       answers.app,
       answers.appVersion,
       answers.binaryVersion,
-      config.getReleaseHistory,
-      config.setReleaseHistory,
+      getReleaseHistory,
+      setReleaseHistory,
       answers.platform,
       options.mandatory,
       options.enable

package/cli/commands/updateHistoryCommand/updateReleaseHistory.js

@@ -35,7 +35,7 @@ async function updateReleaseHistory(
   const releaseHistory = await getReleaseHistory(app, binaryVersion, platform)
 
   const updateInfo = releaseHistory[appVersion]
-  if (!updateInfo) throw new Error(`v${appVersion} is not released`)
+  if (!updateInfo) throw new Error(`${appVersion} is not released`)
 
   if (typeof mandatory === 'boolean') updateInfo.mandatory = mandatory
   if (typeof enable === 'boolean') updateInfo.enabled = enable

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@algocare/react-native-code-push",
-  "version": "12.1.0",
+  "version": "12.2.0",
   "description": "React Native plugin for the CodePush service",
   "main": "CodePush.js",
   "typings": "typings/react-native-code-push.d.ts",