refactor: streamline type definitions and remove unused code

Author: YogendraShelke

src/download-handler/NativeMxDownload.ts

@@ -1,17 +1,6 @@
 import { TurboModuleRegistry, type TurboModule } from 'react-native';
 import type { CodegenTypes } from 'react-native';
 
-type GenericType =
-  | string
-  | number
-  | boolean
-  | null
-  | undefined
-  | { [key: string]: GenericType }
-  | GenericType[];
-
-type GenericMap = { [key: string]: GenericType };
-
 type DownloadConfig = {
   connectionTimeout?: CodegenTypes.Int32;
   mimeType?: string;
@@ -21,10 +10,10 @@ export interface Spec extends TurboModule {
   download(
     url: string,
     downloadPath: string,
-    config: GenericMap
+    config: DownloadConfig
   ): Promise<void>;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>('MxDownload');
 
-export type { GenericMap, DownloadConfig };
+export type { DownloadConfig };

src/download-handler/index.ts

@@ -1,8 +1,5 @@
 import NativeMxDownload from './NativeMxDownload';
 
-// Re-export DownloadConfig type for backward compatibility
-export type { DownloadConfig } from './NativeMxDownload';
-
 export const NativeDownloadHandler = {
   download: (url: string, downloadPath: string, config: Record<string, any>) =>
     NativeMxDownload.download(url, downloadPath, config),

src/file-system/NativeMxFileSystem.ts

@@ -1,4 +1,8 @@
-import { TurboModuleRegistry, type TurboModule } from 'react-native';
+import {
+  TurboModuleRegistry,
+  type TurboModule,
+  type CodegenTypes,
+} from 'react-native';
 
 type BlobData = {
   blobId: string;
@@ -9,33 +13,27 @@ type BlobData = {
   lastModified?: number;
 };
 
-type GenericType =
-  | string
-  | number
-  | boolean
-  | null
-  | undefined
-  | { [key: string]: GenericType }
-  | GenericType[];
-
-type GenericMap = { [key: string]: GenericType };
-type GenericArray = GenericType[];
+type FsConstants = {
+  DocumentDirectoryPath: string;
+  SUPPORTS_DIRECTORY_MOVE: boolean;
+  SUPPORTS_ENCRYPTION: boolean;
+};
 
 export interface Spec extends TurboModule {
-  constants(): GenericMap;
-  save(blob: GenericMap, filePath: string): Promise<void>;
+  constants(): FsConstants;
+  save(blob: BlobData, filePath: string): Promise<void>;
   read(filePath: string): Promise<BlobData>;
   move(filePath: string, newPath: string): Promise<void>;
   remove(filePath: string): Promise<void>;
   list(dirPath: string): Promise<string[]>;
   readAsDataURL(filePath: string): Promise<string>;
   readAsText(filePath: string): Promise<string>;
   fileExists(filePath: string): Promise<boolean>;
-  writeJson(data: GenericMap, filepath: string): Promise<void>;
-  readJson(filepath: string): Promise<GenericMap | GenericArray>;
+  writeJson(data: CodegenTypes.UnsafeObject, filepath: string): Promise<void>;
+  readJson(filepath: string): Promise<CodegenTypes.UnsafeObject | null>;
   setEncryptionEnabled(enabled: boolean): void;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>('MxFileSystem');
 
-export type { BlobData, GenericMap, GenericArray };
+export type { BlobData, FsConstants };

src/ota/NativeMxOta.ts

@@ -1,17 +1,6 @@
 import { TurboModuleRegistry, type TurboModule } from 'react-native';
 import type { CodegenTypes } from 'react-native';
 
-type GenericType =
-  | string
-  | number
-  | boolean
-  | null
-  | undefined
-  | { [key: string]: GenericType }
-  | GenericType[];
-
-type GenericMap = { [key: string]: GenericType };
-
 type OtaDownloadConfig = {
   url: string;
 };
@@ -32,15 +21,14 @@ type DownloadProgress = {
 };
 
 export interface Spec extends TurboModule {
-  download(config: GenericMap): Promise<OtaDownloadResponse>;
-  deploy(config: GenericMap): Promise<void>;
+  download(config: OtaDownloadConfig): Promise<OtaDownloadResponse>;
+  deploy(config: OtaDeployConfig): Promise<void>;
   readonly onDownloadProgress: CodegenTypes.EventEmitter<DownloadProgress>;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>('MxOta');
 
 export type {
-  GenericMap,
   OtaDownloadConfig,
   OtaDeployConfig,
   OtaDownloadResponse,

src/ota/index.ts

@@ -1,13 +1,9 @@
-import NativeMxOta from './NativeMxOta';
-
-// Re-export types for backward compatibility
-export type {
-  OtaDeployConfig,
-  OtaDownloadConfig,
-  OtaDownloadResponse,
+import NativeMxOta, {
+  type OtaDeployConfig,
+  type OtaDownloadConfig,
 } from './NativeMxOta';
 
 export const NativeOta = {
-  download: (config: Record<string, any>) => NativeMxOta.download(config),
-  deploy: (config: Record<string, any>) => NativeMxOta.deploy(config),
+  download: (config: OtaDownloadConfig) => NativeMxOta.download(config),
+  deploy: (config: OtaDeployConfig) => NativeMxOta.deploy(config),
 };