refactor(install-dynamic-plugins): address remaining Sonar findings

- `merger.ts isEqual` (complexity 20 → ~6): extract `isArrayEqual` and
  `isObjectEqual` helpers. Each branch now dispatches to a single-
  purpose function so the main `isEqual` is a flat type-dispatch
  instead of nested loops.

- `plugin-hash.ts compareCodePoint`: replace the nested ternary
  `a < b ? -1 : a > b ? 1 : 0` with three early returns. Same
  behaviour, no nested conditional.

115 tests still pass.

Author: gustavolira

scripts/install-dynamic-plugins/dist/install-dynamic-plugins.cjs

@@ -253,21 +253,21 @@ function copyPluginFields(src: Plugin, dst: Plugin, skip: ReadonlyArray<string>)
 function isEqual(a: unknown, b: unknown): boolean {
   if (a === b) return true;
   if (typeof a !== typeof b) return false;
-  if (Array.isArray(a) && Array.isArray(b)) {
-    if (a.length !== b.length) return false;
-    for (let i = 0; i < a.length; i++) {
-      if (!isEqual(a[i], b[i])) return false;
-    }
-    return true;
-  }
-  if (isPlainObject(a) && isPlainObject(b)) {
-    const keysA = Object.keys(a);
-    const keysB = Object.keys(b);
-    if (keysA.length !== keysB.length) return false;
-    for (const k of keysA) {
-      if (!isEqual(a[k], b[k])) return false;
-    }
-    return true;
-  }
+  if (Array.isArray(a) && Array.isArray(b)) return isArrayEqual(a, b);
+  if (isPlainObject(a) && isPlainObject(b)) return isObjectEqual(a, b);
   return false;
 }
+
+function isArrayEqual(a: readonly unknown[], b: readonly unknown[]): boolean {
+  if (a.length !== b.length) return false;
+  return a.every((v, i) => isEqual(v, b[i]));
+}
+
+function isObjectEqual(
+  a: Record<string, unknown>,
+  b: Record<string, unknown>,
+): boolean {
+  const keysA = Object.keys(a);
+  if (keysA.length !== Object.keys(b).length) return false;
+  return keysA.every(k => isEqual(a[k], b[k]));
+}

scripts/install-dynamic-plugins/src/merger.ts

@@ -68,7 +68,9 @@ function localPackageInfo(pkgPath: string): LocalPackageInfo {
  * without pulling in `String.localeCompare` which varies per-locale).
  */
 function compareCodePoint(a: string, b: string): number {
-  return a < b ? -1 : a > b ? 1 : 0;
+  if (a < b) return -1;
+  if (a > b) return 1;
+  return 0;
 }
 
 function stableStringify(value: unknown): string {