revert: fix(cloudinary): keep widget open for multi-insert on multi-asset fields [] (#8394)

Author: primeinteger

apps/cloudinary2/src/locations/AssetPickerDialog.tsx

@@ -1,49 +1,23 @@
 import { DialogAppSDK } from '@contentful/app-sdk';
-import { useAutoResizer, useSDK } from '@contentful/react-apps-toolkit';
+import { useSDK } from '@contentful/react-apps-toolkit';
 import { injectGlobal } from '@emotion/css';
 import { css } from '@emotion/react';
-import { Button } from '@contentful/f36-components';
-import { useCallback, useRef, useState } from 'react';
+import { useCallback } from 'react';
 import { APP_ENV, APP_VERSION } from '../constants';
-import { AppInstallationParameters, MediaLibraryResultAsset } from '../types';
+import { AppInstallationParameters } from '../types';
 import { loadScript } from '../utils';
 import { ShowOptions } from './types';
 
 const styles = {
   container: css({
     height: '100%',
   }),
-  doneBar: css({
-    position: 'fixed',
-    bottom: 0,
-    left: 0,
-    right: 0,
-    display: 'flex',
-    alignItems: 'center',
-    justifyContent: 'flex-end',
-    gap: '8px',
-    padding: '12px 16px',
-    background: 'white',
-    borderTop: '1px solid #e5ebed',
-    zIndex: 10,
-  }),
-  doneLabel: css({
-    fontSize: '14px',
-    color: '#536171',
-  }),
 };
 
 const AssetPickerDialog = () => {
   const sdk = useSDK<DialogAppSDK<AppInstallationParameters>>();
-  useAutoResizer();
   const invocationParams = sdk.parameters.invocation as Record<string, unknown>;
   const expression = invocationParams.expression as string;
-  const [pendingAssets, setPendingAssets] = useState<MediaLibraryResultAsset[]>([]);
-  const pendingRef = useRef<MediaLibraryResultAsset[]>([]);
-
-  const handleDone = useCallback(() => {
-    sdk.close(pendingRef.current.length > 0 ? { assets: pendingRef.current, mlId: '' } : undefined);
-  }, [sdk]);
 
   /**
    * Initialization is triggered using the div's ref to ensure the container exists in the DOM
@@ -100,19 +74,7 @@ const AssetPickerDialog = () => {
         };
 
         const instance = window.cloudinary.createMediaLibrary(options, {
-          insertHandler: (data) => {
-            if (maxFiles <= 1) {
-              // Single-asset field: close immediately (original behavior)
-              sdk.close(data);
-            } else {
-              // Multi-asset field: accumulate up to maxFiles limit and keep widget open
-              const remaining = maxFiles - pendingRef.current.length;
-              if (remaining <= 0) return;
-              const next = [...pendingRef.current, ...data.assets.slice(0, remaining)];
-              pendingRef.current = next;
-              setPendingAssets([...next]);
-            }
-          },
+          insertHandler: (data) => sdk.close(data),
         });
 
         const showOptions: ShowOptions = {
@@ -124,35 +86,10 @@ const AssetPickerDialog = () => {
         instance.show(showOptions);
       })();
     },
-    [sdk, invocationParams, expression],
+    [sdk],
   );
 
-  const maxFiles = 'maxFiles' in invocationParams ? Number(invocationParams.maxFiles) : sdk.parameters.installation.maxFiles;
-  const isMulti = maxFiles > 1;
-
-  return (
-    <>
-      <div ref={init} id="container" css={styles.container} />
-      {isMulti && (
-        <div css={styles.doneBar}>
-          {pendingAssets.length > 0 && (
-            <span css={styles.doneLabel}>{pendingAssets.length} selected</span>
-          )}
-          <Button variant="secondary" size="small" onClick={() => sdk.close(undefined)}>
-            Cancel
-          </Button>
-          <Button
-            variant="primary"
-            size="small"
-            isDisabled={pendingAssets.length === 0}
-            onClick={handleDone}
-          >
-            Add to field {pendingAssets.length > 0 ? `(${pendingAssets.length})` : ''}
-          </Button>
-        </div>
-      )}
-    </>
-  );
+  return <div ref={init} id="container" css={styles.container} />;
 };
 
 export default AssetPickerDialog;

apps/flexfields/src/locations/EntryEditor.tsx

@@ -1,5 +1,5 @@
 import React, { useCallback, useEffect, useRef, useState } from "react";
-import { EditorAppSDK, EditorLocaleSettings, ConfigAppSDK } from "@contentful/app-sdk";
+import { EditorAppSDK, EditorLocaleSettings } from "@contentful/app-sdk";
 import { useSDK } from "@contentful/react-apps-toolkit";
 import { getDefaultWidgetId } from "@contentful/default-field-editors";
 import { Stack, Form, Heading, Text } from "@contentful/f36-components";
@@ -63,14 +63,6 @@ const EntryEditor = () => {
     };
   }, [handlePageHide]);
 
-  useEffect(() => {
-    // onConfigurationCompleted fires after the config screen saves new installation parameters.
-    // EditorAppSDK doesn't expose `app` in its type but the runtime SDK always has it.
-    (sdk as unknown as ConfigAppSDK).app.onConfigurationCompleted(() => {
-      window.location.reload();
-    });
-  }, [sdk]);
-
   useEffect(() => {
     sdk.editor.onLocaleSettingsChanged((localeSetings) =>
       setLocaleSetings(localeSetings)

apps/flexfields/test/mocks/mockSdk.ts

@@ -4,7 +4,6 @@ const mockSdk: any = {
     getParameters: vi.fn().mockReturnValueOnce({}),
     setReady: vi.fn(),
     getCurrentState: vi.fn(),
-    onConfigurationCompleted: vi.fn(),
   },
   ids: {
     app: 'test-app',