chore: remove Android targets\n\n- Remove Android from build and test matrices\n- Remove Android from target triples\n- Update README to remove Android from platform list

Author: kevinmichaelchen

.github/workflows/CI.yml

@@ -57,18 +57,7 @@ jobs:
               npm install &&
               npx @napi-rs/cli build --target aarch64-unknown-linux-gnu &&
               aarch64-unknown-linux-gnu-strip *.node
-          - host: ubuntu-latest
-            target: aarch64-linux-android
-            build: |
-              npm install
-              npx @napi-rs/cli build --target aarch64-linux-android
-              ${ANDROID_NDK_LATEST_HOME}/toolchains/llvm/prebuilt/linux-x86_64/bin/llvm-strip *.node
-          - host: ubuntu-latest
-            target: armv7-linux-androideabi
-            build: |
-              npm install
-              npx @napi-rs/cli build --target armv7-linux-androideabi
-              ${ANDROID_NDK_LATEST_HOME}/toolchains/llvm/prebuilt/linux-x86_64/bin/llvm-strip *.node
+
 
     name: stable - ${{ matrix.settings.target }} - node@18
     runs-on: ${{ matrix.settings.host }}
@@ -204,20 +193,7 @@ jobs:
               name: macOS ARM64
               runner: macos-latest
               target: aarch64-apple-darwin
-          - if: startsWith(github.ref, 'refs/tags/v')
-            node: 18
-            platform:
-              name: Android ARM64
-              runner: ubuntu-latest
-              target: aarch64-linux-android
-              android: true
-          - if: startsWith(github.ref, 'refs/tags/v')
-            node: 18
-            platform:
-              name: Android ARM v7
-              runner: ubuntu-latest
-              target: armv7-linux-androideabi
-              android: true
+
     runs-on: ${{ matrix.platform.runner }}
 
     steps:
@@ -360,9 +336,11 @@ jobs:
           ls -la
           echo "\nNode files:"
           find . -maxdepth 1 -name '*.node'
-          find . -name "*.node"
         shell: bash
 
+      - name: Link local package
+        run: npm link
+
       - name: Build TypeScript
         run: npm run build:ts
 

README.md

@@ -21,14 +21,14 @@ npm install cel-typescript
 ```
 
 **Requirements:**
+
 - Node.js 18 or later
 
 This package includes pre-compiled native binaries for multiple platforms:
 
 - macOS (x64, arm64)
 - Linux (x64, arm64)
 - Windows (x64)
-- Android (arm64, armv7)
 
 The appropriate binary for your platform will be automatically loaded at
 runtime.
@@ -156,6 +156,40 @@ The native module is built using NAPI-RS and provides cross-platform support:
 - This structure allows for seamless cross-platform distribution while
   maintaining platform-specific optimizations
 
+### Package Size and Platform Support
+
+The npm package is relatively large (~37 MB unpacked) because it includes
+pre-compiled native binaries for all supported platforms:
+
+- macOS (x64, arm64)
+- Linux (x64, arm64)
+- Windows (x64)
+
+However, when you install this package, npm will only extract the `.node` file
+for your platform. For example:
+
+- On an M1/M2 Mac, only `cel-typescript.darwin-arm64.node` (~7.4 MB) is used
+- On Windows, only `cel-typescript.win32-x64.node` is used
+- On Linux, only `cel-typescript.linux-x64.node` or
+  `cel-typescript.linux-arm64.node` is used
+
+This is a common pattern for packages with native bindings. For comparison:
+
+- `sharp` (image processing): 39.7 MB unpacked
+- `better-sqlite3`: 12.8 MB unpacked
+- `canvas`: 8.9 MB unpacked
+
+#### A Note on Tree-Shaking
+
+Tree-shaking (dead code elimination) primarily works with JavaScript modules and
+doesn't affect native binaries. The `.node` files are loaded dynamically at
+runtime based on the platform, so tree-shaking can't eliminate unused platform
+binaries during build time.
+
+However, the JavaScript/TypeScript portion of this package is tree-shakeable.
+For example, if you only use `evaluate()` and not `CelProgram`, a bundler like
+webpack or Rollup can exclude the unused code.
+
 ### How it Works
 
 When you build this project:

package.json

@@ -47,10 +47,8 @@
       "defaults": true,
       "additional": [
         "aarch64-apple-darwin",
-        "aarch64-linux-android",
         "aarch64-unknown-linux-gnu",
         "aarch64-pc-windows-msvc",
-        "armv7-linux-androideabi",
         "x86_64-unknown-linux-gnu",
         "x86_64-apple-darwin",
         "x86_64-pc-windows-msvc"