optimize(harmony): enable LTO, strip symbols, and conditionally skip ohpm install on empty packages

Author: sunnylqm

harmony/pushy/src/main/cpp/CMakeLists.txt

@@ -68,3 +68,19 @@ if(TARGET rnoh)
     target_include_directories(rnoh_pushy PUBLIC ${CMAKE_CURRENT_SOURCE_DIR})
     target_link_libraries(rnoh_pushy PUBLIC rnoh)
 endif()
+
+# Optimize binary size and performance for Release builds
+if(NOT CMAKE_BUILD_TYPE STREQUAL "Debug")
+    # Enable Link Time Optimization (LTO / IPO)
+    include(CheckIPOSupported)
+    check_ipo_supported(RESULT ipo_supported OUTPUT error)
+    if(ipo_supported)
+        set_property(TARGET rnupdate PROPERTY INTERPROCEDURAL_OPTIMIZATION TRUE)
+        if(TARGET rnoh_pushy)
+            set_property(TARGET rnoh_pushy PROPERTY INTERPROCEDURAL_OPTIMIZATION TRUE)
+        endif()
+    endif()
+
+    # Strip debug symbols and perform garbage collection of unused code sections
+    set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -Wl,--gc-sections -s")
+endif()

scripts/build-harmony-har.js

@@ -134,23 +134,30 @@ function buildHar() {
   }
 
   if (!skipInstall) {
-    runCommand(ohpmPath, ['install'], {
-      cwd: harmonyModuleDir,
-      env,
-      label: 'Install Harmony dependencies',
-    });
+    if (hasDependencies(harmonyModuleDir)) {
+      runCommand(ohpmPath, ['install'], {
+        cwd: harmonyModuleDir,
+        env,
+        label: 'Install Harmony dependencies',
+      });
+    }
 
-    runCommand(ohpmPath, ['install'], {
-      cwd: wrapperProjectDir,
-      env,
-      label: 'Install wrapper project dependencies',
-    });
+    if (hasDependencies(wrapperProjectDir)) {
+      runCommand(ohpmPath, ['install'], {
+        cwd: wrapperProjectDir,
+        env,
+        label: 'Install wrapper project dependencies',
+      });
+    }
   }
 
   const hvigorArgs = ['assembleHar'];
   if (buildMode !== 'debug') {
     hvigorArgs.push('-p', `buildMode=${buildMode}`);
   }
+  if (process.env.CI === 'true') {
+    hvigorArgs.push('--no-daemon');
+  }
 
   runCommand(hvigorwPath, hvigorArgs, {
     cwd: wrapperProjectDir,
@@ -437,3 +444,20 @@ function relativeToProject(filePath) {
 function fail(message) {
   throw new Error(message);
 }
+
+function hasDependencies(dir) {
+  try {
+    const pkgPath = path.join(dir, 'oh-package.json5');
+    if (!fs.existsSync(pkgPath)) {
+      return false;
+    }
+    const pkgContent = fs.readFileSync(pkgPath, 'utf8');
+    const cleanJson = pkgContent.replace(/\/\*[\s\S]*?\*\/|([^\\:]|^)\/\/.*$/gm, '');
+    const pkg = eval('(' + cleanJson + ')');
+    const hasDeps = pkg.dependencies && Object.keys(pkg.dependencies).length > 0;
+    const hasDevDeps = pkg.devDependencies && Object.keys(pkg.devDependencies).length > 0;
+    return !!(hasDeps || hasDevDeps);
+  } catch (e) {
+    return true;
+  }
+}