iOS: bspatch integration

Author: ofalvai

.github/workflows/ci-test.yml

@@ -104,14 +104,14 @@ jobs:
           arch: x86
           disable-animations: true
           # connectedAndroidTest is included here (rather than a separate job or step) to reuse
-          # this job's already-booted emulator, even though it means it runs once per matrix
-          # variant. The android-emulator-runner action has no post-cleanup step, so a second
-          # step would boot and tear down a second emulator; ::group:: markers keep the two
-          # test runs visually separated in the Actions log instead.
+          # this job's already-booted emulator.
           script: |
-            echo "::group::Instrumented tests"
-            (cd android && ./gradlew :app:connectedAndroidTest)
-            echo "::endgroup::"
+            if [ "${{ matrix.variant }}" = "bare" ]; then
+              # These tests are independent of the bare/expo distinction. Bare tests are slightly faster.
+              echo "::group::Instrumented tests"
+              (cd android && ./gradlew :app:connectedAndroidTest)
+              echo "::endgroup::"
+            fi
             echo "::group::E2E tests"
             npm run ${{ matrix.test-command }}
             echo "::endgroup::"
@@ -141,10 +141,16 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-npm-
 
-      - name: Run iOS Tests
-        run: |
-          npm install
-          npm run ${{ matrix.test-command }}
+      - name: Install dependencies
+        run: npm install
+
+      - name: Run iOS unit tests
+        # These tests are independent of the bare/expo distinction. Bare tests are slightly faster.
+        if: matrix.variant == 'bare'
+        run: npm run test:unit:ios
+
+      - name: Run iOS E2E Tests
+        run: npm run ${{ matrix.test-command }}
 
       - name: Upload iOS Simulator crash reports
         if: failure()

.npmignore

@@ -52,6 +52,8 @@ android/.gradle
 android/**/*.iml
 android/.idea
 
+ios/CodePushDiffPatchTests/
+
 
 # Windows
 windows/.vs/

CLAUDE.md

@@ -10,8 +10,10 @@ React Native CodePush is a native module that enables over-the-air updates for R
 
 #### Unit tests
 
-- `cd android && ./gradlew :app:test`
-- iOS: no unit tests yet.
+- `npm run test:unit:android`
+- `npm run test:unit:ios`
+
+Prefer unit testing what's possible (even though, on iOS, this involves a simulator). Legacy code used E2E tests for everything, which is complex, error-prone, and slow. The existing E2E tests are still useful, but this is not a pattern to follow.
 
 #### E2E Tests
 - `npm run test:android` - Run Android-specific tests
@@ -54,7 +56,7 @@ React Native CodePush is a native module that enables over-the-air updates for R
 - **Custom Test Runner**: TypeScript-based test framework in `test/`
 - **Real App Testing**: Creates actual React Native apps for integration testing
 - **Scenario Testing**: Update, rollback, and error scenarios
-- **No unit test infra for JS/iOS yet**: JS/iOS only have the mocha-based integration suite above. `src/acquisition-sdk/__tests__/` contains tests ported from upstream `microsoft/code-push`, kept for future reference - they are deliberately not wired into `npm test` or any runner. Don't assume they're dead/forgotten code, and don't wire them in without setting up real unit test infra first.
+- **No unit test infra for JS yet**: JS only has the mocha-based integration suite above. `src/acquisition-sdk/__tests__/` contains tests ported from upstream `microsoft/code-push`, kept for future reference - they are deliberately not wired into `npm test` or any runner. Don't assume they're dead/forgotten code, and don't wire them in without setting up real unit test infra first.
 - **Templates**: `test/template/` holds native files (Podfile, AppDelegate, Android app files) and JS scenarios copied over top of a freshly generated RN/Expo app during test setup, overwriting its defaults — edit files here, not the generated project, for changes to persist
 - **`test:ios` vs `test:setup:ios` vs `test:fast:ios`**: `test:ios` is just `test:setup:ios` followed by `test:fast:ios` — the two are meant to be split apart for local iteration.
   - `test:setup:ios` (mocha `--ios --setup`) boots the simulator and provisions the test app once: copies templates, runs `pod install`, patches Info.plist/AppDelegate. It never builds or runs any test scenario.

android/app/src/main/cpp/CMakeLists.txt

@@ -12,12 +12,12 @@ project(codepush_diffpatch C)
 # See third_party/README.md for what we vendor and why.
 add_library(codepush_diffpatch SHARED
         diffpatch_jni.c
-        bspatch_bridge.c
+        ../../../../../shared/diffpatch/bspatch_bridge.c
+
+        ../../../../../shared/third_party/hdiffpatch/libHDiffPatch/HPatch/patch.c
+        ../../../../../shared/third_party/hdiffpatch/bsdiff_wrapper/bspatch_wrapper.c
+        ../../../../../shared/third_party/hdiffpatch/file_for_patch.c
 
-        third_party/hdiffpatch/libHDiffPatch/HPatch/patch.c
-        third_party/hdiffpatch/bsdiff_wrapper/bspatch_wrapper.c
-        third_party/hdiffpatch/file_for_patch.c
-        
         bzip2_error_stub.c
         third_party/bzip2/bzlib.c
         third_party/bzip2/decompress.c
@@ -28,6 +28,8 @@ add_library(codepush_diffpatch SHARED
 
 target_include_directories(codepush_diffpatch PRIVATE
         .
+        ../../../../../shared/diffpatch
+        ../../../../../shared
         third_party/bzip2
 )
 

android/app/src/main/cpp/third_party/README.md

@@ -1,23 +1,6 @@
 # Vendored sources
 
-These directories contain trimmed copies of two upstream libraries, pinned
-to a specific commit. Only the files needed to *apply* a BSDIFF40-style
-patch (the `hdiffz -BSD` producer output) are vendored. `hdiffpatch/` is an
-unmodified-source copy; `bzip2/bzlib.c` carries one small, documented patch
-(see below).
-
-## hdiffpatch/
-
-Source: https://github.com/sisong/HDiffPatch
-Pinned commit: `3b9dca715ca492873bf2c49e22e5d5b7d2a78620` (2026-07-31)
-License: MIT.
-
-Files were chosen by tracing the actual dependency graph of
-`bsdiff_wrapper/bspatch_wrapper.c` (the BSDIFF40-compatible patch applier),
-not by directory boundaries. In particular `libHDiffPatch/HPatch/patch.c`
-(~157KB) is HDiffPatch's own diff-format decoder, but it's still required
-here because `bspatch_wrapper.c` shares its low-level stream-cache helpers
-(`_TOutStreamCache_*`, `getStreamClip`, `_patch_cache_all_old`, etc.).
+Note: `shared/third_party/` also contains vendored sources, shared between Android and iOS.
 
 ## bzip2/
 

android/app/src/main/java/com/microsoft/codepush/react/diffpatch/DiffPatch.kt

@@ -37,7 +37,7 @@ object DiffPatch {
         UNKNOWN;
 
         companion object {
-            // Keep in sync with cpp/bspatch_bridge.h
+            // Keep in sync with shared/diffpatch/bspatch_bridge.h
             fun fromNativeCode(code: Int): PatchResult = when (code) {
                 0 -> OK
                 1 -> BAD_DIFF_HEADER