@atlaskit/editor-plugin-media 8.6.0 → 8.6.1

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @atlaskit/editor-plugin-media
 
+## 8.6.1
+
+### Patch Changes
+
+- [`b6a737b41d065`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/b6a737b41d065) -
+  Clean up experiment platform_editor_fix_clone_nesting_exp
+- Updated dependencies
+
 ## 8.6.0
 
 ### Minor Changes

package/dist/cjs/pm-plugins/main.js

@@ -485,19 +485,14 @@ var MediaPluginStateImplementation = exports.MediaPluginStateImplementation = /*
   return (0, _createClass2.default)(MediaPluginStateImplementation, [{
     key: "clone",
     value: function clone() {
+      var _originalTarget;
       var clonedAt = (performance || Date).now();
 
       // Prevent double wrapping
       // If clone is repeatedly called, we want to proxy the underlying MediaPluginStateImplementation target, rather than the proxy itself
       // If we proxy the proxy, then calling get in future will need to recursively unwrap proxies to find the original target, which causes performance issues
       // Instead, we check if there is an original target stored on "this", and if so, we use that as the proxy target instead
-      // eslint-disable-next-line @typescript-eslint/no-this-alias -- This is required while this is behind a feature-gate. Once the feature-gate is removed, we can inline proxyTarget as "(this as unknown as { originalTarget?: typeof proxyTarget }).originalTarget ?? this"
-      var proxyTarget = this;
-      var originalTarget = this.originalTarget;
-      if (originalTarget !== undefined && (0, _expValEquals.expValEquals)('platform_editor_fix_clone_nesting_exp', 'isEnabled', true)) {
-        proxyTarget = originalTarget;
-      }
-      return new Proxy(proxyTarget, {
+      return new Proxy((_originalTarget = this.originalTarget) !== null && _originalTarget !== void 0 ? _originalTarget : this, {
         get: function get(target, prop, receiver) {
           if (prop === 'singletonCreatedAt') {
             return clonedAt;

package/dist/es2019/pm-plugins/main.js

@@ -18,10 +18,10 @@ import { isFileIdentifier } from '@atlaskit/media-client';
 import { getMediaFeatureFlag } from '@atlaskit/media-common';
 import { fg } from '@atlaskit/platform-feature-flags';
 import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
-import { updateMediaNodeAttrs } from '../pm-plugins/commands/helpers';
 // Ignored via go/ees005
 // eslint-disable-next-line import/no-namespace
 import * as helpers from '../pm-plugins/commands/helpers';
+import { updateMediaNodeAttrs } from '../pm-plugins/commands/helpers';
 import { getIdentifier, getMediaFromSupportedMediaNodesFromSelection, isNodeDoubleClickSupportedInLivePagesViewMode, removeMediaNode, splitMediaGroup } from '../pm-plugins/utils/media-common';
 import { insertMediaGroupNode, insertMediaInlineNode } from '../pm-plugins/utils/media-files';
 import { getMediaNodeInsertionType } from '../pm-plugins/utils/media-inline';
@@ -462,19 +462,14 @@ export class MediaPluginStateImplementation {
     this.singletonCreatedAt = (performance || Date).now();
   }
   clone() {
+    var _originalTarget;
     const clonedAt = (performance || Date).now();
 
     // Prevent double wrapping
     // If clone is repeatedly called, we want to proxy the underlying MediaPluginStateImplementation target, rather than the proxy itself
     // If we proxy the proxy, then calling get in future will need to recursively unwrap proxies to find the original target, which causes performance issues
     // Instead, we check if there is an original target stored on "this", and if so, we use that as the proxy target instead
-    // eslint-disable-next-line @typescript-eslint/no-this-alias -- This is required while this is behind a feature-gate. Once the feature-gate is removed, we can inline proxyTarget as "(this as unknown as { originalTarget?: typeof proxyTarget }).originalTarget ?? this"
-    let proxyTarget = this;
-    const originalTarget = this.originalTarget;
-    if (originalTarget !== undefined && expValEquals('platform_editor_fix_clone_nesting_exp', 'isEnabled', true)) {
-      proxyTarget = originalTarget;
-    }
-    return new Proxy(proxyTarget, {
+    return new Proxy((_originalTarget = this.originalTarget) !== null && _originalTarget !== void 0 ? _originalTarget : this, {
       get(target, prop, receiver) {
         if (prop === 'singletonCreatedAt') {
           return clonedAt;

package/dist/esm/pm-plugins/main.js

@@ -27,10 +27,10 @@ import { isFileIdentifier } from '@atlaskit/media-client';
 import { getMediaFeatureFlag } from '@atlaskit/media-common';
 import { fg } from '@atlaskit/platform-feature-flags';
 import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
-import { updateMediaNodeAttrs } from '../pm-plugins/commands/helpers';
 // Ignored via go/ees005
 // eslint-disable-next-line import/no-namespace
 import * as helpers from '../pm-plugins/commands/helpers';
+import { updateMediaNodeAttrs } from '../pm-plugins/commands/helpers';
 import { getIdentifier, getMediaFromSupportedMediaNodesFromSelection, isNodeDoubleClickSupportedInLivePagesViewMode, removeMediaNode, splitMediaGroup } from '../pm-plugins/utils/media-common';
 import { insertMediaGroupNode, insertMediaInlineNode } from '../pm-plugins/utils/media-files';
 import { getMediaNodeInsertionType } from '../pm-plugins/utils/media-inline';
@@ -477,19 +477,14 @@ export var MediaPluginStateImplementation = /*#__PURE__*/function () {
   return _createClass(MediaPluginStateImplementation, [{
     key: "clone",
     value: function clone() {
+      var _originalTarget;
       var clonedAt = (performance || Date).now();
 
       // Prevent double wrapping
       // If clone is repeatedly called, we want to proxy the underlying MediaPluginStateImplementation target, rather than the proxy itself
       // If we proxy the proxy, then calling get in future will need to recursively unwrap proxies to find the original target, which causes performance issues
       // Instead, we check if there is an original target stored on "this", and if so, we use that as the proxy target instead
-      // eslint-disable-next-line @typescript-eslint/no-this-alias -- This is required while this is behind a feature-gate. Once the feature-gate is removed, we can inline proxyTarget as "(this as unknown as { originalTarget?: typeof proxyTarget }).originalTarget ?? this"
-      var proxyTarget = this;
-      var originalTarget = this.originalTarget;
-      if (originalTarget !== undefined && expValEquals('platform_editor_fix_clone_nesting_exp', 'isEnabled', true)) {
-        proxyTarget = originalTarget;
-      }
-      return new Proxy(proxyTarget, {
+      return new Proxy((_originalTarget = this.originalTarget) !== null && _originalTarget !== void 0 ? _originalTarget : this, {
         get: function get(target, prop, receiver) {
           if (prop === 'singletonCreatedAt') {
             return clonedAt;

package/dist/types/pm-plugins/main.d.ts

@@ -12,7 +12,7 @@ import type { EditorView } from '@atlaskit/editor-prosemirror/view';
 import { type Identifier } from '@atlaskit/media-client';
 import type { MediaClientConfig } from '@atlaskit/media-core';
 import type { MediaNextEditorPluginType } from '../mediaPluginType';
-import type { MediaOptions, MediaState, MediaStateEventSubscriber, getPosHandlerNode as ProsemirrorGetPosHandler } from '../types';
+import type { getPosHandlerNode as ProsemirrorGetPosHandler, MediaOptions, MediaState, MediaStateEventSubscriber } from '../types';
 import type { MediaPluginOptions } from '../types/media-plugin-options';
 import PickerFacade from './picker-facade';
 import type { MediaNodeWithPosHandler, MediaPluginState } from './types';
@@ -64,7 +64,7 @@ export declare class MediaPluginStateImplementation implements MediaPluginState
     pluginInjectionApi: ExtractInjectionAPI<MediaNextEditorPluginType> | undefined;
     singletonCreatedAt: number;
     constructor(state: EditorState, options: MediaPluginOptions, mediaOptions: MediaOptions | undefined, dispatch: Dispatch | undefined, pluginInjectionApi: ExtractInjectionAPI<MediaNextEditorPluginType> | undefined);
-    clone(): this;
+    clone(): MediaPluginStateImplementation;
     subscribeToUploadInProgressState(fn: (isUploading: boolean) => void): void;
     unsubscribeFromUploadInProgressState(fn: (isUploading: boolean) => void): void;
     private previousMediaProvider;

package/dist/types-ts4.5/pm-plugins/main.d.ts

@@ -12,7 +12,7 @@ import type { EditorView } from '@atlaskit/editor-prosemirror/view';
 import { type Identifier } from '@atlaskit/media-client';
 import type { MediaClientConfig } from '@atlaskit/media-core';
 import type { MediaNextEditorPluginType } from '../mediaPluginType';
-import type { MediaOptions, MediaState, MediaStateEventSubscriber, getPosHandlerNode as ProsemirrorGetPosHandler } from '../types';
+import type { getPosHandlerNode as ProsemirrorGetPosHandler, MediaOptions, MediaState, MediaStateEventSubscriber } from '../types';
 import type { MediaPluginOptions } from '../types/media-plugin-options';
 import PickerFacade from './picker-facade';
 import type { MediaNodeWithPosHandler, MediaPluginState } from './types';
@@ -64,7 +64,7 @@ export declare class MediaPluginStateImplementation implements MediaPluginState
     pluginInjectionApi: ExtractInjectionAPI<MediaNextEditorPluginType> | undefined;
     singletonCreatedAt: number;
     constructor(state: EditorState, options: MediaPluginOptions, mediaOptions: MediaOptions | undefined, dispatch: Dispatch | undefined, pluginInjectionApi: ExtractInjectionAPI<MediaNextEditorPluginType> | undefined);
-    clone(): this;
+    clone(): MediaPluginStateImplementation;
     subscribeToUploadInProgressState(fn: (isUploading: boolean) => void): void;
     unsubscribeFromUploadInProgressState(fn: (isUploading: boolean) => void): void;
     private previousMediaProvider;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-media",
-	"version": "8.6.0",
+	"version": "8.6.1",
 	"description": "Media plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -51,9 +51,9 @@
 		"@atlaskit/editor-prosemirror": "^7.2.0",
 		"@atlaskit/editor-shared-styles": "^3.10.0",
 		"@atlaskit/editor-tables": "^2.9.0",
-		"@atlaskit/form": "^15.0.0",
-		"@atlaskit/icon": "^29.2.0",
-		"@atlaskit/icon-lab": "^5.12.0",
+		"@atlaskit/form": "^15.1.0",
+		"@atlaskit/icon": "^29.3.0",
+		"@atlaskit/icon-lab": "^5.13.0",
 		"@atlaskit/media-card": "^79.11.0",
 		"@atlaskit/media-client": "^35.7.0",
 		"@atlaskit/media-client-react": "^4.1.0",
@@ -66,7 +66,7 @@
 		"@atlaskit/primitives": "^16.4.0",
 		"@atlaskit/textfield": "^8.2.0",
 		"@atlaskit/theme": "^21.0.0",
-		"@atlaskit/tmp-editor-statsig": "^15.12.0",
+		"@atlaskit/tmp-editor-statsig": "^15.13.0",
 		"@atlaskit/tokens": "^8.6.0",
 		"@atlaskit/tooltip": "^20.11.0",
 		"@babel/runtime": "^7.0.0",
@@ -78,7 +78,7 @@
 		"uuid": "^3.1.0"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^110.45.0",
+		"@atlaskit/editor-common": "^110.46.0",
 		"@atlaskit/media-core": "^37.0.0",
 		"react": "^18.2.0",
 		"react-dom": "^18.2.0",