Fix trying to close file twice in strict mode

packages/h5wasm/src/H5WasmProvider.tsx

@@ -1,7 +1,7 @@
 import type { DataProviderApi } from '@h5web/app';
 import { DataProvider } from '@h5web/app';
 import type { PropsWithChildren } from 'react';
-import { useMemo, useState } from 'react';
+import { useMemo, useRef } from 'react';
 
 import { H5WasmApi } from './h5wasm-api';
 import type { Plugin } from './models';
@@ -16,16 +16,16 @@ interface Props {
 function H5WasmProvider(props: PropsWithChildren<Props>) {
   const { filename, buffer, getExportURL, getPlugin, children } = props;
 
-  const api = useMemo(
-    () => new H5WasmApi(filename, buffer, getExportURL, getPlugin),
-    [buffer, filename, getExportURL, getPlugin],
-  );
+  const prevApiRef = useRef<H5WasmApi>();
 
-  const [prevApi, setPrevApi] = useState(api);
-  if (prevApi !== api) {
-    setPrevApi(api);
-    void prevApi.cleanUp(); // https://github.com/silx-kit/h5web/pull/1568
-  }
+  const api = useMemo(() => {
+    const newApi = new H5WasmApi(filename, buffer, getExportURL, getPlugin);
+
+    void prevApiRef.current?.cleanUp();
+    prevApiRef.current = newApi;
+
+    return newApi;
+  }, [buffer, filename, getExportURL, getPlugin]);
 
   return <DataProvider api={api}>{children}</DataProvider>;
 }

packages/h5wasm/src/h5wasm-api.ts

@@ -116,8 +116,10 @@ export class H5WasmApi extends DataProviderApi {
   }
 
   public async cleanUp(): Promise<void> {
+    const module = await h5wasmReady;
     const file = await this.file;
-    file.close();
+    // `file.close()` flushes the file, which is not needed
+    module.ccall('H5Fclose', 'number', ['bigint'], [file.file_id]);
   }
 
   private async initH5Wasm(): Promise<typeof Module> {

packages/h5wasm/src/local/H5WasmLocalFileProvider.tsx

@@ -1,7 +1,7 @@
 import type { DataProviderApi } from '@h5web/app';
 import { DataProvider } from '@h5web/app';
 import type { PropsWithChildren } from 'react';
-import { useMemo, useState } from 'react';
+import { useMemo, useRef } from 'react';
 
 import type { Plugin } from '../models';
 import { H5WasmLocalFileApi } from './h5wasm-local-file-api';
@@ -15,16 +15,16 @@ interface Props {
 function H5WasmLocalFileProvider(props: PropsWithChildren<Props>) {
   const { file, getExportURL, getPlugin, children } = props;
 
-  const api = useMemo(
-    () => new H5WasmLocalFileApi(file, getExportURL, getPlugin),
-    [file, getExportURL, getPlugin],
-  );
+  const prevApiRef = useRef<H5WasmLocalFileApi>();
 
-  const [prevApi, setPrevApi] = useState(api);
-  if (prevApi !== api) {
-    setPrevApi(api);
-    void prevApi.cleanUp(); // https://github.com/silx-kit/h5web/pull/1568
-  }
+  const api = useMemo(() => {
+    const newApi = new H5WasmLocalFileApi(file, getExportURL, getPlugin);
+
+    void prevApiRef.current?.cleanUp();
+    prevApiRef.current = newApi;
+
+    return newApi;
+  }, [file, getExportURL, getPlugin]);
 
   return <DataProvider api={api}>{children}</DataProvider>;
 }

packages/h5wasm/src/local/worker.ts

@@ -38,7 +38,6 @@ async function openFile(file: File): Promise<bigint> {
 
 async function closeFile(fileId: bigint): Promise<number> {
   const h5wasm = await h5wasmReady;
-  h5wasm.flush(fileId);
   return h5wasm.ccall('H5Fclose', 'number', ['bigint'], [fileId]);
 }