Updated typings for database methods

Author: rhodgkins

packages/database-types/index.d.ts

@@ -125,31 +125,36 @@ export interface Reference extends Query {
   onDisconnect(): OnDisconnect;
   parent: Reference | null;
   push(value?: any, onComplete?: (a: Error | null) => any): ThenableReference;
-  remove(onComplete?: (a: Error | null) => any): Promise<any>;
+  remove(onComplete?: (a: Error | null) => void): Promise<void>;
   root: Reference;
-  set(value: any, onComplete?: (a: Error | null) => any): Promise<any>;
+  set(value: any, onComplete?: (a: Error | null) => void): Promise<void>;
   setPriority(
     priority: string | number | null,
-    onComplete: (a: Error | null) => any
-  ): Promise<any>;
+    onComplete: (a: Error | null) => void
+  ): Promise<void>;
   setWithPriority(
     newVal: any,
     newPriority: string | number | null,
-    onComplete?: (a: Error | null) => any
-  ): Promise<any>;
+    onComplete?: (a: Error | null) => void
+  ): Promise<void>;
   transaction(
     transactionUpdate: (a: any) => any,
-    onComplete?: (a: Error | null, b: boolean, c: DataSnapshot | null) => any,
+    onComplete?: (a: Error | null, b: boolean, c: DataSnapshot | null) => void,
     applyLocally?: boolean
-  ): Promise<any>;
-  update(values: Object, onComplete?: (a: Error | null) => any): Promise<any>;
+  ): Promise<TransactionResult>;
+  update(values: Object, onComplete?: (a: Error | null) => void): Promise<void>;
 }
 
 export interface ServerValue {
   TIMESTAMP: Object;
   increment(delta: number): Object;
 }
 
+export interface TransactionResult {
+  readonly committed: boolean;
+  readonly snapshot: DataSnapshot;
+}
+
 export interface ThenableReference
   extends Reference,
     Pick<Promise<Reference>, 'then' | 'catch'> {}

packages/firebase/compat/index.d.ts

@@ -7082,7 +7082,7 @@ declare namespace firebase.database {
      *   complete.
      * @return Resolves when remove on server is complete.
      */
-    remove(onComplete?: (a: Error | null) => any): Promise<any>;
+    remove(onComplete?: (a: Error | null) => void): Promise<void>;
     /**
      * The root `Reference` of the Database.
      *
@@ -7160,7 +7160,7 @@ declare namespace firebase.database {
      *   complete.
      * @return Resolves when write to server is complete.
      */
-    set(value: any, onComplete?: (a: Error | null) => any): Promise<any>;
+    set(value: any, onComplete?: (a: Error | null) => void): Promise<void>;
     /**
      * Sets a priority for the data at this Database location.
      *
@@ -7172,8 +7172,8 @@ declare namespace firebase.database {
      */
     setPriority(
       priority: string | number | null,
-      onComplete: (a: Error | null) => any
-    ): Promise<any>;
+      onComplete: (a: Error | null) => void
+    ): Promise<void>;
     /**
      * Writes data the Database location. Like `set()` but also specifies the
      * priority for that data.
@@ -7187,8 +7187,8 @@ declare namespace firebase.database {
     setWithPriority(
       newVal: any,
       newPriority: string | number | null,
-      onComplete?: (a: Error | null) => any
-    ): Promise<any>;
+      onComplete?: (a: Error | null) => void
+    ): Promise<void>;
     /**
      * Atomically modifies the data at this location.
      *
@@ -7280,9 +7280,9 @@ declare namespace firebase.database {
         a: Error | null,
         b: boolean,
         c: firebase.database.DataSnapshot | null
-      ) => any,
+      ) => void,
       applyLocally?: boolean
-    ): Promise<any>;
+    ): Promise<TransactionResult>;
     /**
      * Writes multiple values to the Database at once.
      *
@@ -7329,7 +7329,21 @@ declare namespace firebase.database {
      *   complete.
      * @return Resolves when update on server is complete.
      */
-    update(values: Object, onComplete?: (a: Error | null) => any): Promise<any>;
+    update(
+      values: Object,
+      onComplete?: (a: Error | null) => void
+    ): Promise<void>;
+  }
+
+  interface TransactionResult {
+    /**
+     * Whether the transaction was successfully committed.
+     */
+    readonly committed: boolean;
+    /**
+     * The resulting data snapshot.
+     */
+    readonly snapshot: DataSnapshot;
   }
 
   interface ThenableReference