v0.5.2 (#568)

## Description
Resource Fetcher changes:
- Fix an issue which prevented models from loading when device was not
connected to the internet.
- Fix an issue which crashed IOS apps in release mode using bundled
model sources.

### Introduces a breaking change?

- [ ] Yes
- [x] No

### Type of change

- [x] Bug fix (change which fixes an issue)
- [ ] New feature (change which adds functionality)
- [ ] Documentation update (improves or adds clarity to existing
documentation)
- [ ] Other (chores, tests, code style improvements etc.)

### Tested on

- [x] iOS
- [x] Android

### Testing instructions

<!-- Provide step-by-step instructions on how to test your changes.
Include setup details if necessary. -->

### Screenshots

<!-- Add screenshots here, if applicable -->

### Related issues

<!-- Link related issues here using #issue-number -->

### Checklist

- [x] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [x] My changes generate no new warnings

### Additional notes

<!-- Include any additional information, assumptions, or context that
reviewers might need to understand this PR. -->

Author: mlodyjesienin

packages/react-native-executorch/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-executorch",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "An easy way to run AI models in React Native with ExecuTorch",
   "source": "./src/index.ts",
   "main": "./lib/module/index.js",

packages/react-native-executorch/src/utils/ResourceFetcher.ts

@@ -29,6 +29,7 @@
  */
 import {
   cacheDirectory,
+  copyAsync,
   createDownloadResumable,
   moveAsync,
   FileSystemSessionType,
@@ -38,6 +39,7 @@ import {
   readDirectoryAsync,
 } from 'expo-file-system';
 import { Asset } from 'expo-asset';
+import { Platform } from 'react-native';
 import { RNEDirectory } from '../constants/directories';
 import { ResourceSource } from '../types/common';
 import {
@@ -61,7 +63,6 @@ export class ResourceFetcher {
     }
     const { results: info, totalLength } =
       await ResourceFetcherUtils.getFilesSizes(sources);
-
     const head: ResourceSourceExtended = {
       source: info[0]!.source,
       sourceType: info[0]!.type,
@@ -316,16 +317,17 @@ export class ResourceFetcher {
     const uri = asset.uri;
     const filename = ResourceFetcherUtils.getFilenameFromUri(uri);
     const fileUri = `${RNEDirectory}${filename}`;
-    const fileUriWithType = `${fileUri}.${asset.type}`;
+    // On Android, file uri does not contain file extension, so we add it manually
+    const fileUriWithType =
+      Platform.OS === 'android' ? `${fileUri}.${asset.type}` : fileUri;
     if (await ResourceFetcherUtils.checkFileExists(fileUri)) {
       return ResourceFetcherUtils.removeFilePrefix(fileUri);
     }
     await ResourceFetcherUtils.createDirectoryIfNoExists();
-    await asset.downloadAsync();
-    if (!asset.localUri) {
-      throw new Error(`Asset local URI is not available for ${source}`);
-    }
-    await moveAsync({ from: asset.localUri, to: fileUriWithType });
+    await copyAsync({
+      from: asset.uri,
+      to: fileUriWithType,
+    });
     return ResourceFetcherUtils.removeFilePrefix(fileUriWithType);
   }
 

packages/react-native-executorch/src/utils/ResourceFetcherUtils.ts

@@ -53,17 +53,16 @@ export namespace ResourceFetcherUtils {
       return SourceType.OBJECT;
     } else if (typeof source === 'number') {
       const uri = Asset.fromModule(source).uri;
-      if (!uri.includes('://')) {
-        return SourceType.RELEASE_MODE_FILE;
+      if (uri.startsWith('http')) {
+        return SourceType.DEV_MODE_FILE;
       }
-      return SourceType.DEV_MODE_FILE;
-    } else {
-      // typeof source == 'string'
-      if (source.startsWith('file://')) {
-        return SourceType.LOCAL_FILE;
-      }
-      return SourceType.REMOTE_FILE;
+      return SourceType.RELEASE_MODE_FILE;
+    }
+    // typeof source == 'string'
+    if (source.startsWith('file://')) {
+      return SourceType.LOCAL_FILE;
     }
+    return SourceType.REMOTE_FILE;
   }
 
   export async function getFilesSizes(sources: ResourceSource[]) {
@@ -78,9 +77,8 @@ export namespace ResourceFetcherUtils {
     for (const source of sources) {
       const type = await ResourceFetcherUtils.getType(source);
       let length = 0;
-
-      if (type === SourceType.REMOTE_FILE && typeof source === 'string') {
-        try {
+      try {
+        if (type === SourceType.REMOTE_FILE && typeof source === 'string') {
           const response = await fetch(source, { method: 'HEAD' });
           if (!response.ok) {
             Logger.warn(
@@ -97,14 +95,14 @@ export namespace ResourceFetcherUtils {
           length = contentLength ? parseInt(contentLength, 10) : 0;
           previousFilesTotalLength = totalLength;
           totalLength += length;
-        } catch (error) {
-          Logger.warn(`Error fetching HEAD for ${source}:`, error);
-          continue;
         }
+      } catch (error) {
+        Logger.warn(`Error fetching HEAD for ${source}:`, error);
+        continue;
+      } finally {
+        results.push({ source, type, length, previousFilesTotalLength });
       }
-      results.push({ source, type, length, previousFilesTotalLength });
     }
-
     return { results, totalLength };
   }