Refactor rollout configuration handling in bindVersionToPackages; replace rollout logic with direct package binding and update API call to use POST method.

cli.json

@@ -213,7 +213,7 @@
         "rncli": {
           "default": false
         },
-        "disableHermes": {
+        "hermes": {
           "default": false
         },
         "name": {
@@ -299,15 +299,6 @@
         }
       }
     },
-    "hdiffFromPPK": {
-      "description": "Create hdiff patch from a Prepare package(.ppk)",
-      "options": {
-        "output": {
-          "default": "${tempDir}/output/hdiff-${time}.ppk-patch",
-          "hasValue": true
-        }
-      }
-    },
     "hdiffFromApp": {
       "description": "Create hdiff patch from a Harmony package(.app)",
       "options": {
@@ -343,6 +334,10 @@
           "hasValue": true
         }
       }
+    },
+    "install": {
+      "description": "Install optional dependencies to the CLI",
+      "options": {}
     }
   },
   "globalOptions": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-update-cli",
-  "version": "2.2.3",
+  "version": "2.4.0",
   "description": "command line tool for react-native-update (remote updates for react native)",
   "main": "index.js",
   "bin": {

src/bundle.ts

@@ -16,7 +16,7 @@ import os from 'os';
 const properties = require('properties');
 import { addGitIgnore } from './utils/add-gitignore';
 import { checkLockFiles } from './utils/check-lockfile';
-import { tempDir } from './utils/constants';
+import { isPPKBundleFileName, scriptName, tempDir } from './utils/constants';
 import { depVersions } from './utils/dep-versions';
 import { t } from './utils/i18n';
 import { versionCommands } from './versions';
@@ -42,7 +42,7 @@ async function runReactNativeBundleCommand({
   platform,
   sourcemapOutput,
   config,
-  disableHermes,
+  forceHermes,
   cli,
 }: {
   bundleName: string;
@@ -52,7 +52,7 @@ async function runReactNativeBundleCommand({
   platform: string;
   sourcemapOutput: string;
   config?: string;
-  disableHermes?: boolean;
+  forceHermes?: boolean;
   cli: {
     taro?: boolean;
     expo?: boolean;
@@ -85,14 +85,30 @@ async function runReactNativeBundleCommand({
 
   const getExpoCli = () => {
     try {
-      cliPath = require.resolve('@expo/cli', {
+      const searchPaths = [process.cwd()];
+
+      // 尝试添加 expo 包的路径作为额外的搜索路径
+      try {
+        const expoPath = require.resolve('expo/package.json', {
           paths: [process.cwd()],
         });
+        // 获取 expo 包的目录路径
+        const expoDir = expoPath.replace(/\/package\.json$/, '');
+        searchPaths.push(expoDir);
+      } catch {
+        // expo 包不存在，忽略
+      }
+
+      // 尝试从搜索路径中解析 @expo/cli
+      cliPath = require.resolve('@expo/cli', {
+        paths: searchPaths,
+      });
+
       const expoCliVersion = JSON.parse(
         fs
           .readFileSync(
             require.resolve('@expo/cli/package.json', {
-              paths: [process.cwd()],
+              paths: searchPaths,
             }),
           )
           .toString(),
@@ -163,18 +179,25 @@ async function runReactNativeBundleCommand({
     bundleCommand = 'build';
   }
 
-  reactNativeBundleArgs.push(cliPath, bundleCommand);
+  if (platform === 'harmony') {
+    bundleName = 'bundle.harmony.js';
+    if (forceHermes === undefined) {
+      // enable hermes by default for harmony
+      forceHermes = true;
+    }
+  }
 
-  if (platform !== 'harmony') {
   reactNativeBundleArgs.push(
-      '--platform',
+    cliPath,
-      platform,
+    bundleCommand,
     '--assets-dest',
     outputFolder,
     '--bundle-output',
     path.join(outputFolder, bundleName),
-      '--reset-cache',
   );
+
+  if (platform !== 'harmony') {
+    reactNativeBundleArgs.push('--platform', platform, '--reset-cache');
   }
 
   if (cli.taro) {
@@ -211,9 +234,9 @@ async function runReactNativeBundleCommand({
       } else {
         let hermesEnabled: boolean | undefined = false;
 
-        if (disableHermes) {
+        if (forceHermes) {
-          hermesEnabled = false;
+          hermesEnabled = true;
-          console.log(t('hermesDisabled'));
+          console.log(t('forceHermes'));
         } else if (platform === 'android') {
           const gradlePropeties = await new Promise<{
             hermesEnabled?: boolean;
@@ -240,8 +263,6 @@ async function runReactNativeBundleCommand({
           fs.existsSync('ios/Pods/hermes-engine')
         ) {
           hermesEnabled = true;
-        } else if (platform === 'harmony') {
-          await copyHarmonyBundle(outputFolder);
         }
         if (hermesEnabled) {
           await compileHermesByteCode(
@@ -251,72 +272,25 @@ async function runReactNativeBundleCommand({
             !isSentry,
           );
         }
+        if (platform === 'harmony') {
+          const harmonyRawAssetsPath =
+            'harmony/entry/src/main/resources/rawfile/assets';
+          // copy all files in outputFolder to harmonyRawPath
+          // assets should be in rawfile/assets
+          fs.ensureDirSync(harmonyRawAssetsPath);
+          fs.copySync(outputFolder, harmonyRawAssetsPath, { overwrite: true });
+          fs.moveSync(
+            `${harmonyRawAssetsPath}/bundle.harmony.js`,
+            `${harmonyRawAssetsPath}/../bundle.harmony.js`,
+            { overwrite: true },
+          );
+        }
         resolve(null);
       }
     });
   });
 }
 
-async function copyHarmonyBundle(outputFolder: string) {
-  const harmonyRawPath = 'harmony/entry/src/main/resources/rawfile';
-  try {
-    await fs.ensureDir(harmonyRawPath);
-    try {
-      await fs.access(harmonyRawPath, fs.constants.W_OK);
-    } catch (error) {
-      await fs.chmod(harmonyRawPath, 0o755);
-    }
-    await fs.remove(path.join(harmonyRawPath, 'update.json'));
-    await fs.copy('update.json', path.join(harmonyRawPath, 'update.json'));
-    await fs.ensureDir(outputFolder);
-
-    // Recursively copy files with special handling for assets directory
-    async function copyFilesRecursively(
-      srcDir: string,
-      destDir: string,
-      relativePath = '',
-    ) {
-      const fullSrcPath = path.join(srcDir, relativePath);
-      const items = await fs.readdir(fullSrcPath);
-
-      for (const item of items) {
-        const itemRelativePath = path.join(relativePath, item);
-        const itemSrcPath = path.join(srcDir, itemRelativePath);
-
-        // Skip update.json and meta.json at root level
-        if (!relativePath && (item === 'update.json' || item === 'meta.json')) {
-          continue;
-        }
-
-        const stat = await fs.stat(itemSrcPath);
-
-        if (stat.isFile()) {
-          // Special handling: remove 'assets/' prefix to move files up one level
-          let itemDestPath = itemRelativePath;
-          if (
-            itemDestPath.startsWith('assets/') ||
-            itemDestPath.startsWith('assets\\')
-          ) {
-            itemDestPath = itemDestPath.replace(/^assets[\\/]/, '');
-          }
-
-          const fullDestPath = path.join(destDir, itemDestPath);
-          await fs.ensureDir(path.dirname(fullDestPath));
-          await fs.copy(itemSrcPath, fullDestPath);
-        } else if (stat.isDirectory()) {
-          // Recursively process subdirectories
-          await copyFilesRecursively(srcDir, destDir, itemRelativePath);
-        }
-      }
-    }
-
-    await copyFilesRecursively(harmonyRawPath, outputFolder);
-  } catch (error: any) {
-    console.error(t('copyHarmonyBundleError', { error }));
-    throw new Error(t('copyFileFailed', { error: error.message }));
-  }
-}
-
 function getHermesOSBin() {
   if (os.platform() === 'win32') return 'win64-bin';
   if (os.platform() === 'darwin') return 'osx-bin';
@@ -505,7 +479,12 @@ async function uploadSourcemapForSentry(
   }
 }
 
-const ignorePackingFileNames = ['.', '..', 'index.bundlejs.map'];
+const ignorePackingFileNames = [
+  '.',
+  '..',
+  'index.bundlejs.map',
+  'bundle.harmony.js.map',
+];
 const ignorePackingExtensions = ['DS_Store', 'txt.map'];
 async function pack(dir: string, output: string) {
   console.log(t('packing'));
@@ -587,10 +566,7 @@ async function diffFromPPK(origin: string, next: string, output: string) {
       // isFile
       originMap[entry.crc32] = entry.fileName;
 
-      if (
+      if (isPPKBundleFileName(entry.fileName)) {
-        entry.fileName === 'index.bundlejs' ||
-        entry.fileName === 'bundle.harmony.js'
-      ) {
         // This is source.
         return readEntry(entry, zipFile).then((v) => (originSource = v));
       }
@@ -598,9 +574,7 @@ async function diffFromPPK(origin: string, next: string, output: string) {
   });
 
   if (!originSource) {
-    throw new Error(
+    throw new Error(t('bundleFileNotFound'));
-      'Bundle file not found! Please use default bundle file name and path.',
-    );
   }
 
   const copies = {};
@@ -641,23 +615,13 @@ async function diffFromPPK(origin: string, next: string, output: string) {
       if (!originEntries[entry.fileName]) {
         addEntry(entry.fileName);
       }
-    } else if (entry.fileName === 'index.bundlejs') {
+    } else if (isPPKBundleFileName(entry.fileName)) {
       //console.log('Found bundle');
       return readEntry(entry, nextZipfile).then((newSource) => {
         //console.log('Begin diff');
         zipfile.addBuffer(
           diff(originSource, newSource),
-          'index.bundlejs.patch',
+          `${entry.fileName}.patch`,
-        );
-        //console.log('End diff');
-      });
-    } else if (entry.fileName === 'bundle.harmony.js') {
-      //console.log('Found bundle');
-      return readEntry(entry, nextZipfile).then((newSource) => {
-        //console.log('Begin diff');
-        zipfile.addBuffer(
-          diff(originSource, newSource),
-          'bundle.harmony.js.patch',
         );
         //console.log('End diff');
       });
@@ -750,9 +714,7 @@ async function diffFromPackage(
   });
 
   if (!originSource) {
-    throw new Error(
+    throw new Error(t('bundleFileNotFound'));
-      'Bundle file not found! Please use default bundle file name and path.',
-    );
   }
 
   const copies = {};
@@ -773,23 +735,13 @@ async function diffFromPackage(
     if (/\/$/.test(entry.fileName)) {
       // Directory
       zipfile.addEmptyDirectory(entry.fileName);
-    } else if (entry.fileName === 'index.bundlejs') {
+    } else if (isPPKBundleFileName(entry.fileName)) {
       //console.log('Found bundle');
       return readEntry(entry, nextZipfile).then((newSource) => {
         //console.log('Begin diff');
         zipfile.addBuffer(
           diff(originSource, newSource),
-          'index.bundlejs.patch',
+          `${entry.fileName}.patch`,
-        );
-        //console.log('End diff');
-      });
-    } else if (entry.fileName === 'bundle.harmony.js') {
-      //console.log('Found bundle');
-      return readEntry(entry, nextZipfile).then((newSource) => {
-        //console.log('Begin diff');
-        zipfile.addBuffer(
-          diff(originSource, newSource),
-          'bundle.harmony.js.patch',
         );
         //console.log('End diff');
       });
@@ -906,24 +858,21 @@ function diffArgsCheck(args: string[], options: any, diffFn: string) {
 
   if (diffFn.startsWith('hdiff')) {
     if (!hdiff) {
-      console.error(
+      console.error(t('nodeHdiffpatchRequired', { scriptName }));
-        `This function needs "node-hdiffpatch". 
-        Please run "npm i node-hdiffpatch" to install`,
-      );
       process.exit(1);
     }
     diff = hdiff;
   } else {
     if (!bsdiff) {
-      console.error(
+      console.error(t('nodeBsdiffRequired', { scriptName }));
-        `This function needs "node-bsdiff". 
-        Please run "npm i node-bsdiff" to install`,
-      );
       process.exit(1);
     }
     diff = bsdiff;
   }
-  const { output } = options;
+  const { output } = translateOptions({
+    ...options,
+    tempDir,
+  });
 
   return {
     origin,
@@ -946,7 +895,7 @@ export const bundleCommands = {
       taro,
       expo,
       rncli,
-      disableHermes,
+      hermes,
       name,
       description,
       metaInfo,
@@ -987,7 +936,7 @@ export const bundleCommands = {
       outputFolder: intermediaDir,
       platform,
       sourcemapOutput: sourcemap || sourcemapPlugin ? sourcemapOutput : '',
-      disableHermes: !!disableHermes,
+      forceHermes: hermes as unknown as boolean,
       cli: {
         taro: !!taro,
         expo: !!expo,
@@ -1054,14 +1003,14 @@ export const bundleCommands = {
     const { origin, next, realOutput } = diffArgsCheck(args, options, 'diff');
 
     await diffFromPPK(origin, next, realOutput);
-    console.log(`${realOutput} generated.`);
+    console.log(t('diffPackageGenerated', { output: realOutput }));
   },
 
   async hdiff({ args, options }) {
     const { origin, next, realOutput } = diffArgsCheck(args, options, 'hdiff');
 
     await diffFromPPK(origin, next, realOutput);
-    console.log(`${realOutput} generated.`);
+    console.log(t('diffPackageGenerated', { output: realOutput }));
   },
 
   async diffFromApk({ args, options }) {
@@ -1077,7 +1026,7 @@ export const bundleCommands = {
       realOutput,
       'assets/index.android.bundle',
     );
-    console.log(`${realOutput} generated.`);
+    console.log(t('diffPackageGenerated', { output: realOutput }));
   },
 
   async hdiffFromApk({ args, options }) {
@@ -1093,7 +1042,7 @@ export const bundleCommands = {
       realOutput,
       'assets/index.android.bundle',
     );
-    console.log(`${realOutput} generated.`);
+    console.log(t('diffPackageGenerated', { output: realOutput }));
   },
 
   async diffFromApp({ args, options }) {
@@ -1108,7 +1057,7 @@ export const bundleCommands = {
       realOutput,
       'resources/rawfile/bundle.harmony.js',
     );
-    console.log(`${realOutput} generated.`);
+    console.log(t('diffPackageGenerated', { output: realOutput }));
   },
 
   async hdiffFromApp({ args, options }) {
@@ -1123,7 +1072,7 @@ export const bundleCommands = {
       realOutput,
       'resources/rawfile/bundle.harmony.js',
     );
-    console.log(`${realOutput} generated.`);
+    console.log(t('diffPackageGenerated', { output: realOutput }));
   },
 
   async diffFromIpa({ args, options }) {
@@ -1138,7 +1087,7 @@ export const bundleCommands = {
       return m?.[1];
     });
 
-    console.log(`${realOutput} generated.`);
+    console.log(t('diffPackageGenerated', { output: realOutput }));
   },
 
   async hdiffFromIpa({ args, options }) {
@@ -1153,6 +1102,6 @@ export const bundleCommands = {
       return m?.[1];
     });
 
-    console.log(`${realOutput} generated.`);
+    console.log(t('diffPackageGenerated', { output: realOutput }));
   },
 };

src/index.ts

@@ -3,6 +3,7 @@
 import { loadSession } from './api';
 import { appCommands } from './app';
 import { bundleCommands } from './bundle';
+import { installCommands } from './install';
 import { moduleManager } from './module-manager';
 import { builtinModules } from './modules';
 import { packageCommands } from './package';
@@ -33,6 +34,7 @@ function printUsage() {
     ...appCommands,
     ...packageCommands,
     ...versionCommands,
+    ...installCommands,
   };
 
   for (const [name, handler] of Object.entries(legacyCommands)) {
@@ -76,6 +78,7 @@ const legacyCommands = {
   ...appCommands,
   ...packageCommands,
   ...versionCommands,
+  ...installCommands,
   help: printUsage,
 };
 

src/install.ts

@@ -0,0 +1,19 @@
+import { spawnSync } from 'child_process';
+import path from 'path';
+import type { CommandContext } from './types';
+
+export const installCommands = {
+  install: async ({ args }: CommandContext) => {
+    if (args.length === 0) {
+      return;
+    }
+
+    const cliDir = path.resolve(__dirname, '..');
+
+    spawnSync('npm', ['install', ...args], {
+      cwd: cliDir,
+      stdio: 'inherit',
+      shell: true,
+    });
+  },
+};

src/locales/en.ts

@@ -49,7 +49,7 @@ export default {
   fileGenerated: '{{- file}} generated.',
   fileSizeExceeded:
     'This file size is {{fileSize}} , exceeding the current quota {{maxSize}} . You may consider upgrading to a higher plan to increase this quota. Details can be found at: {{- pricingPageUrl}}',
-  hermesDisabled: 'Hermes disabled',
+  forceHermes: 'Forcing Hermes enabled for this build',
   hermesEnabledCompiling: 'Hermes enabled, now compiling to hermes bytecode:\n',
   ipaUploadSuccess:
     'Successfully uploaded IPA native package (id: {{id}}, version: {{version}}, buildTime: {{buildTime}})',
@@ -137,4 +137,11 @@ This can reduce the risk of inconsistent dependencies and supply chain attacks.
   deletePackageError:
     'Failed to delete native package {{packageId}}: {{error}}',
   usageDeletePackage: 'Usage: cresc deletePackage [packageId] --appId [appId]',
+  bundleFileNotFound:
+    'Bundle file not found! Please use default bundle file name and path.',
+  diffPackageGenerated: '{{- output}} generated.',
+  nodeBsdiffRequired:
+    'This function needs "node-bsdiff". Please run "{{scriptName}} install node-bsdiff" to install',
+  nodeHdiffpatchRequired:
+    'This function needs "node-hdiffpatch". Please run "{{scriptName}} install node-hdiffpatch" to install',
 };

src/locales/zh.ts

@@ -47,7 +47,7 @@ export default {
   fileGenerated: '已生成 {{- file}}',
   fileSizeExceeded:
     '此文件大小 {{fileSize}} , 超出当前额度 {{maxSize}} 。您可以考虑升级付费业务以提升此额度。详情请访问: {{- pricingPageUrl}}',
-  hermesDisabled: 'Hermes 已禁用',
+  forceHermes: '强制启用 Hermes 编译',
   hermesEnabledCompiling: 'Hermes 已启用，正在编译为 hermes 字节码：\n',
   ipaUploadSuccess:
     '已成功上传ipa原生包（id: {{id}}, version: {{version}}, buildTime: {{buildTime}}）',
@@ -129,4 +129,10 @@ export default {
   deletePackageError: '删除原生包 {{packageId}} 失败: {{error}}',
   usageDeletePackage:
     '使用方法: pushy deletePackage [packageId] --appId [appId]',
+  bundleFileNotFound: '未找到 bundle 文件！请使用默认的 bundle 文件名和路径。',
+  diffPackageGenerated: '{{- output}} 已生成。',
+  nodeBsdiffRequired:
+    '此功能需要 "node-bsdiff"。请运行 "{{scriptName}} install node-bsdiff" 来安装',
+  nodeHdiffpatchRequired:
+    '此功能需要 "node-hdiffpatch"。请运行 "{{scriptName}} install node-hdiffpatch" 来安装',
 };

src/modules/bundle-module.ts

@@ -53,7 +53,7 @@ export const bundleModule: CLIModule = {
               taro = false,
               expo = false,
               rncli = false,
-              disableHermes = false,
+              hermes = false,
               output,
             } = context.options;
 
@@ -73,7 +73,7 @@ export const bundleModule: CLIModule = {
                 taro,
                 expo,
                 rncli,
-                disableHermes,
+                hermes,
                 intermediaDir: '${tempDir}/intermedia/${platform}',
                 output: '${tempDir}/output/${platform}.${time}.ppk',
               };
@@ -170,10 +170,10 @@ export const bundleModule: CLIModule = {
           default: false,
           description: 'Use React Native CLI',
         },
-        disableHermes: {
+        hermes: {
           hasValue: false,
           default: false,
-          description: 'Disable Hermes',
+          description: 'Force enable Hermes',
         },
         name: {
           hasValue: true,

src/provider.ts

@@ -42,7 +42,7 @@ export class CLIProviderImpl implements CLIProvider {
           taro: options.taro || false,
           expo: options.expo || false,
           rncli: options.rncli || false,
-          disableHermes: options.disableHermes || false,
+          hermes: options.hermes || false,
         },
       };
 

src/types.ts

@@ -65,7 +65,7 @@ export interface BundleOptions {
   taro?: boolean;
   expo?: boolean;
   rncli?: boolean;
-  disableHermes?: boolean;
+  hermes?: boolean;
 }
 
 export interface PublishOptions {

src/utils/constants.ts

@@ -1,8 +1,12 @@
 import path from 'path';
 
-const scriptName = path.basename(process.argv[1]) as 'cresc' | 'pushy';
+export const scriptName = path.basename(process.argv[1]) as 'cresc' | 'pushy';
 export const IS_CRESC = scriptName === 'cresc';
 
+export const ppkBundleFileNames = ['index.bundlejs', 'bundle.harmony.js'];
+export const isPPKBundleFileName = (fileName: string) =>
+  ppkBundleFileNames.includes(fileName);
+
 export const credentialFile = IS_CRESC ? '.cresc.token' : '.update';
 export const updateJson = IS_CRESC ? 'cresc.config.json' : 'update.json';
 export const tempDir = IS_CRESC ? '.cresc.temp' : '.pushy';

src/versions.ts

@@ -122,17 +122,6 @@ export const bindVersionToPackages = async ({
     console.log(chalk.yellow(t('dryRun')));
   }
   if (rollout !== undefined) {
-    const rolloutConfig: Record<string, number> = {};
-    for (const pkg of pkgs) {
-      rolloutConfig[pkg.name] = rollout;
-    }
-    if (!dryRun) {
-      await put(`/app/${appId}/version/${versionId}`, {
-        config: {
-          rollout: rolloutConfig,
-        },
-      });
-    }
     console.log(
       `${t('rolloutConfigSet', {
         versions: pkgs.map((pkg: Package) => pkg.name).join(', '),
@@ -142,8 +131,10 @@ export const bindVersionToPackages = async ({
   }
   for (const pkg of pkgs) {
     if (!dryRun) {
-      await put(`/app/${appId}/package/${pkg.id}`, {
+      await post(`/app/${appId}/binding`, {
         versionId,
+        rollout,
+        packageId: pkg.id,
       });
     }
     console.log(