Fix TypeScript declaration of chained batch write() options

Level · Mar 6, 2022 · 392b7f7 · 392b7f7
1 parent 4bdeea9
commit 392b7f7
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/index.d.ts b/index.d.ts
@@ -26,7 +26,8 @@ export {
 export {
   AbstractChainedBatch,
   AbstractChainedBatchPutOptions,
-  AbstractChainedBatchDelOptions
+  AbstractChainedBatchDelOptions,
+  AbstractChainedBatchWriteOptions
 } from './types/abstract-chained-batch'
 
 export {

diff --git a/types/abstract-chained-batch.d.ts b/types/abstract-chained-batch.d.ts
@@ -45,9 +45,9 @@ export class AbstractChainedBatch<TDatabase, KDefault, VDefault> {
    * commits.
    */
   write (): Promise<void>
-  write (options: {}): Promise<void>
+  write (options: AbstractChainedBatchWriteOptions): Promise<void>
   write (callback: NodeCallback<void>): void
-  write (options: {}, callback: NodeCallback<void>): void
+  write (options: AbstractChainedBatchWriteOptions, callback: NodeCallback<void>): void
 
   /**
    * Free up underlying resources. This should be done even if the chained batch has
@@ -113,3 +113,11 @@ export interface AbstractChainedBatchDelOptions<TDatabase, K> {
    */
   sublevel?: AbstractSublevel<TDatabase, any, any, any> | undefined
 }
+
+/**
+ * Options for the {@link AbstractChainedBatch.write} method.
+ */
+// eslint-disable-next-line @typescript-eslint/no-empty-interface
+export interface AbstractChainedBatchWriteOptions {
+  // There are no abstract options but implementations may add theirs.
+}