@atlaskit/media-viewer 54.1.1 → 54.2.1

package/CHANGELOG.md

@@ -1,5 +1,30 @@
 # @atlaskit/media-viewer
 
+## 54.2.1
+
+### Patch Changes
+
+- [`ee28cf33718b0`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/ee28cf33718b0) -
+  Add @atlassian/react-compiler-gating as a runtime dependency to enable React Compiler platform
+  gating.
+- Updated dependencies
+
+## 54.2.0
+
+### Minor Changes
+
+- [`a56e76c8f33af`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/a56e76c8f33af) -
+  Add processingFailReason to media-viewer loadFailed analytics for failed-processing root-cause
+  analysis (BMPT-8000)
+- [`758e07b536b13`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/758e07b536b13) -
+  [ux] Show a clear "Unsupported file format" message for non-ZIP archives (e.g. 7z, tar, gzip) in
+  the media viewer instead of a generic error, and remove the empty sidebar gap. Behind feature gate
+  platform_media_archive_zip_guard.
+
+### Patch Changes
+
+- Updated dependencies
+
 ## 54.1.1
 
 ### Patch Changes

package/dist/cjs/analytics/events/operational/loadFailed.js

@@ -7,11 +7,13 @@ exports.createLoadFailedEvent = void 0;
 var _ = require("../..");
 var _errors = require("../../../errors");
 var createLoadFailedEvent = exports.createLoadFailedEvent = function createLoadFailedEvent(fileId, error, fileState, traceContext) {
+  var _failReason;
   var _getFileAttributes = (0, _.getFileAttributes)(fileState),
     fileMediatype = _getFileAttributes.fileMediatype,
     fileMimetype = _getFileAttributes.fileMimetype,
     fileSize = _getFileAttributes.fileSize;
   var requestMetadata = (0, _errors.getRequestMetadata)(error);
+  var processingFailReason = (fileState === null || fileState === void 0 ? void 0 : fileState.status) === 'failed-processing' ? (_failReason = fileState.failReason) !== null && _failReason !== void 0 ? _failReason : 'not-available' : undefined;
   return {
     eventType: 'operational',
     actionSubject: 'mediaFile',
@@ -23,6 +25,7 @@ var createLoadFailedEvent = exports.createLoadFailedEvent = function createLoadF
       errorDetail: (0, _errors.getErrorDetail)(error),
       statusCode: requestMetadata === null || requestMetadata === void 0 ? void 0 : requestMetadata.statusCode,
       request: requestMetadata,
+      processingFailReason: processingFailReason,
       fileMimetype: fileMimetype,
       fileAttributes: {
         fileId: fileId,

package/dist/cjs/analytics/index.js

@@ -10,7 +10,7 @@ exports.packageVersion = exports.packageName = void 0;
 var _analytics = require("@atlaskit/media-common/analytics");
 var componentName = exports.component = exports.componentName = 'mediaViewer';
 var packageName = exports.packageName = "@atlaskit/media-viewer";
-var packageVersion = exports.packageVersion = "54.1.0";
+var packageVersion = exports.packageVersion = "54.2.0";
 function getFileAttributes(fileState) {
   if (!fileState) {
     return {

package/dist/cjs/analytics/ufoExperiences.js

@@ -13,7 +13,7 @@ var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 var packageName = "@atlaskit/media-viewer";
-var packageVersion = "54.1.0";
+var packageVersion = "54.2.0";
 var ufoExperience;
 var getExperience = function getExperience() {
   if (!ufoExperience) {

package/dist/cjs/errorMessage.js

@@ -51,6 +51,16 @@ var ErrorMessage = exports.ErrorMessage = /*#__PURE__*/function (_React$Componen
       var _this$props = this.props,
         formatMessage = _this$props.intl.formatMessage,
         error = _this$props.error;
+      // Non-ZIP archives (e.g. RAR, TAR, 7z) aren't a load failure - the
+      // browser-based viewer simply can't preview them. Show a dedicated
+      // "Unsupported file format" heading instead of the generic
+      // "Something went wrong" copy.
+      if ((0, _errors.getPrimaryErrorReason)(error) === 'archiveviewer-not-zip') {
+        return {
+          icon: errorLoadingFileImage(formatMessage),
+          messages: [_mediaUi.messages.unsupported_file_format, _mediaUi.messages.archive_format_not_supported]
+        };
+      }
       var errorInfo = {
         icon: errorLoadingFileImage(formatMessage),
         messages: [_mediaUi.messages.something_went_wrong, _mediaUi.messages.couldnt_generate_preview]

package/dist/cjs/header.js

@@ -11,6 +11,8 @@ var _react = _interopRequireWildcard(require("react"));
 var _mediaClient = require("@atlaskit/media-client");
 var _mediaUi = require("@atlaskit/media-ui");
 var _codeViewer = require("@atlaskit/media-ui/codeViewer");
+var _isZipMimeType = require("@atlaskit/media-common/isZipMimeType");
+var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _reactIntl = require("react-intl");
 var _domain = require("./domain");
 var _styleWrappers = require("./styleWrappers");
@@ -83,7 +85,15 @@ var Header = exports.Header = function Header(_ref) {
     }
     if (fileState.status !== 'error') {
       var _onSetArchiveSideBarV;
-      (_onSetArchiveSideBarV = onSetArchiveSideBarVisibleRef.current) === null || _onSetArchiveSideBarV === void 0 || _onSetArchiveSideBarV.call(onSetArchiveSideBarVisibleRef, !(0, _mediaClient.isErrorFileState)(fileState) && fileState.mediaType === 'archive');
+      // Only reserve the archive sidebar space for previewable ZIP archives.
+      // Non-ZIP archives (e.g. RAR, TAR, 7z) render a full-width "unsupported
+      // file format" error in the body and have no sidebar, so the header
+      // must not leave an empty 300px gap for them.
+      (_onSetArchiveSideBarV = onSetArchiveSideBarVisibleRef.current) === null || _onSetArchiveSideBarV === void 0 || _onSetArchiveSideBarV.call(onSetArchiveSideBarVisibleRef, !(0, _mediaClient.isErrorFileState)(fileState) && fileState.mediaType === 'archive' && (
+      // When the zip guard is on, non-ZIP archives show a full-width
+      // "unsupported file format" error with no sidebar, so the header
+      // must not reserve the 300px sidebar space for them.
+      !(0, _platformFeatureFlags.fg)('platform_media_archive_zip_guard') || (0, _isZipMimeType.isZipMimeType)(fileState.mimeType)));
       setItem(_domain.Outcome.successful(fileState));
     } else {
       setItem(_domain.Outcome.failed(new _errors.MediaViewerError('header-fetch-metadata', (0, _mediaClient.toCommonMediaClientError)(fileState))));

package/dist/cjs/viewers/archiveSidebar/archive-sidebar-renderer.js

@@ -68,7 +68,14 @@ var ArchiveSidebarRenderer = exports.default = /*#__PURE__*/function (_Component
               this.setState({
                 status: 'loaded'
               });
-              onError(new _errors.ArchiveViewerError('archiveviewer-read-binary', _t instanceof Error ? _t : undefined));
+              // Preserve archiveviewer-not-zip as-is so the generic error boundary
+              // can show a clear "format not supported" message instead of a
+              // generic read error.
+              if (_t instanceof _errors.ArchiveViewerError && _t.primaryReason === 'archiveviewer-not-zip') {
+                onError(_t);
+              } else {
+                onError(new _errors.ArchiveViewerError('archiveviewer-read-binary', _t instanceof Error ? _t : undefined));
+              }
             case 4:
             case "end":
               return _context.stop();

package/dist/cjs/viewers/archiveSidebar/archive.js

@@ -16,6 +16,8 @@ var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/
 var _react = _interopRequireDefault(require("react"));
 var _unzipit = require("unzipit");
 var _reactIntl = require("react-intl");
+var _mediaClient = require("@atlaskit/media-client");
+var _isZipMimeType = require("@atlaskit/media-common/isZipMimeType");
 var _mediaUi = require("@atlaskit/media-ui");
 var _codeViewer = require("@atlaskit/media-ui/codeViewer");
 var _compiled = require("@atlaskit/primitives/compiled");
@@ -35,6 +37,7 @@ var _analytics = require("../../analytics");
 var _errors = require("../../errors");
 var _zipEntryLoadSucceeded = require("../../analytics/events/operational/zipEntryLoadSucceeded");
 var _zipEntryLoadFailed = require("../../analytics/events/operational/zipEntryLoadFailed");
+var _previewUnsupported = require("../../analytics/events/operational/previewUnsupported");
 var _codeViewerRenderer = require("../codeViewer/codeViewerRenderer");
 var _util = require("../codeViewer/util");
 var _itemViewer = require("../../item-viewer");
@@ -50,19 +53,25 @@ var getArchiveEntriesFromFileState = exports.getArchiveEntriesFromFileState = /*
     return _regenerator.default.wrap(function (_context) {
       while (1) switch (_context.prev = _context.next) {
         case 0:
-          _context.next = 1;
-          return mediaClient.file.getFileBinaryURL(fileState.id, collectionName);
+          if (!(!(0, _mediaClient.isErrorFileState)(fileState) && !(0, _isZipMimeType.isZipMimeType)(fileState.mimeType) && (0, _platformFeatureFlags.fg)('platform_media_archive_zip_guard'))) {
+            _context.next = 1;
+            break;
+          }
+          throw new _errors.ArchiveViewerError('archiveviewer-not-zip');
         case 1:
+          _context.next = 2;
+          return mediaClient.file.getFileBinaryURL(fileState.id, collectionName);
+        case 2:
           url = _context.sent;
           reader = new _unzipit.HTTPRangeReader(url);
-          _context.next = 2;
+          _context.next = 3;
           return (0, _utils.rejectAfter)(function () {
             return (0, _unzipit.unzip)(reader);
           });
-        case 2:
+        case 3:
           archive = _context.sent;
           return _context.abrupt("return", archive);
-        case 3:
+        case 4:
         case "end":
           return _context.stop();
       }
@@ -81,6 +90,25 @@ var ArchiveViewerBase = exports.ArchiveViewerBase = /*#__PURE__*/function (_Base
     }
     _this = _callSuper(this, ArchiveViewerBase, [].concat(args));
     (0, _defineProperty2.default)(_this, "onError", function (error, entry) {
+      // Non-ZIP archives (e.g. RAR, TAR, 7z) are a known limitation of the
+      // browser-based `unzipit` viewer, not a per-entry failure. Render a
+      // full-width "format not supported" message via BaseViewer's failed
+      // outcome (no sidebar, no 300px offset) and do NOT escalate to the
+      // parent item-viewer error boundary, which would double-render the
+      // message inside the offset archive layout.
+      if (error.primaryReason === 'archiveviewer-not-zip') {
+        // Fire a dedicated, non-SLI `previewUnsupported` event (not a
+        // `loadFailed`) so unsupported archives don't pollute error
+        // dashboards. The event's fileAttributes distinguish the two cases:
+        // `fileMediatype: 'unknown'` (browser-unrecognised, e.g. RAR) vs
+        // `fileMediatype: 'archive'` with a non-ZIP `fileMimetype` (unzipit
+        // limitation, e.g. 7z/tar/gzip).
+        (0, _analytics.fireAnalytics)((0, _previewUnsupported.createPreviewUnsupportedEvent)(_this.props.item), _this.props.createAnalyticsEvent);
+        _this.setState({
+          content: _domain.Outcome.failed(error)
+        });
+        return;
+      }
       _this.props.onError(error);
       _this.setState({
         content: _domain.Outcome.successful(_objectSpread(_objectSpread({}, _this.state.content.data), {}, {

package/dist/es2019/analytics/events/operational/loadFailed.js

@@ -1,12 +1,14 @@
 import { getFileAttributes } from '../..';
 import { getPrimaryErrorReason, getSecondaryErrorReason, getErrorDetail, getRequestMetadata } from '../../../errors';
 export const createLoadFailedEvent = (fileId, error, fileState, traceContext) => {
+  var _failReason;
   const {
     fileMediatype,
     fileMimetype,
     fileSize
   } = getFileAttributes(fileState);
   const requestMetadata = getRequestMetadata(error);
+  const processingFailReason = (fileState === null || fileState === void 0 ? void 0 : fileState.status) === 'failed-processing' ? (_failReason = fileState.failReason) !== null && _failReason !== void 0 ? _failReason : 'not-available' : undefined;
   return {
     eventType: 'operational',
     actionSubject: 'mediaFile',
@@ -18,6 +20,7 @@ export const createLoadFailedEvent = (fileId, error, fileState, traceContext) =>
       errorDetail: getErrorDetail(error),
       statusCode: requestMetadata === null || requestMetadata === void 0 ? void 0 : requestMetadata.statusCode,
       request: requestMetadata,
+      processingFailReason,
       fileMimetype,
       fileAttributes: {
         fileId,

package/dist/es2019/analytics/index.js

@@ -1,7 +1,7 @@
 import { ANALYTICS_MEDIA_CHANNEL, sanitiseAnalyticsPayload } from '@atlaskit/media-common/analytics';
 const componentName = 'mediaViewer';
 const packageName = "@atlaskit/media-viewer";
-const packageVersion = "54.1.0";
+const packageVersion = "54.2.0";
 export { packageName, packageVersion, componentName, componentName as component };
 export function getFileAttributes(fileState) {
   if (!fileState) {

package/dist/es2019/analytics/ufoExperiences.js

@@ -3,7 +3,7 @@ import { getMediaEnvironment, getMediaRegion } from '@atlaskit/media-client';
 import { getFeatureFlagKeysAllProducts } from '@atlaskit/media-common';
 import { fg } from '@atlaskit/platform-feature-flags';
 const packageName = "@atlaskit/media-viewer";
-const packageVersion = "54.1.0";
+const packageVersion = "54.2.0";
 let ufoExperience;
 const getExperience = () => {
   if (!ufoExperience) {

package/dist/es2019/errorMessage.js

@@ -26,6 +26,16 @@ export class ErrorMessage extends React.Component {
       },
       error
     } = this.props;
+    // Non-ZIP archives (e.g. RAR, TAR, 7z) aren't a load failure - the
+    // browser-based viewer simply can't preview them. Show a dedicated
+    // "Unsupported file format" heading instead of the generic
+    // "Something went wrong" copy.
+    if (getPrimaryErrorReason(error) === 'archiveviewer-not-zip') {
+      return {
+        icon: errorLoadingFileImage(formatMessage),
+        messages: [i18nMessages.unsupported_file_format, i18nMessages.archive_format_not_supported]
+      };
+    }
     const errorInfo = {
       icon: errorLoadingFileImage(formatMessage),
       messages: [i18nMessages.something_went_wrong, i18nMessages.couldnt_generate_preview]

package/dist/es2019/header.js

@@ -2,6 +2,8 @@ import React, { useEffect, useRef, useState } from 'react';
 import { isExternalImageIdentifier, isErrorFileState, toCommonMediaClientError } from '@atlaskit/media-client';
 import { hideControlsClassName, messages, toHumanReadableMediaSize, MediaButton } from '@atlaskit/media-ui';
 import { getLanguageType, getExtension, isCodeViewerItem } from '@atlaskit/media-ui/codeViewer';
+import { isZipMimeType } from '@atlaskit/media-common/isZipMimeType';
+import { fg } from '@atlaskit/platform-feature-flags';
 import { FormattedMessage, injectIntl } from 'react-intl';
 import { Outcome } from './domain';
 import { Header as HeaderWrapper, LeftHeader, RightHeader, MetadataWrapper, MetadataSubText, MedatadataTextWrapper, MetadataIconWrapper, MetadataFileName, FormattedMessageWrapper } from './styleWrappers';
@@ -70,7 +72,15 @@ export const Header = ({
     }
     if (fileState.status !== 'error') {
       var _onSetArchiveSideBarV;
-      (_onSetArchiveSideBarV = onSetArchiveSideBarVisibleRef.current) === null || _onSetArchiveSideBarV === void 0 ? void 0 : _onSetArchiveSideBarV.call(onSetArchiveSideBarVisibleRef, !isErrorFileState(fileState) && fileState.mediaType === 'archive');
+      // Only reserve the archive sidebar space for previewable ZIP archives.
+      // Non-ZIP archives (e.g. RAR, TAR, 7z) render a full-width "unsupported
+      // file format" error in the body and have no sidebar, so the header
+      // must not leave an empty 300px gap for them.
+      (_onSetArchiveSideBarV = onSetArchiveSideBarVisibleRef.current) === null || _onSetArchiveSideBarV === void 0 ? void 0 : _onSetArchiveSideBarV.call(onSetArchiveSideBarVisibleRef, !isErrorFileState(fileState) && fileState.mediaType === 'archive' && (
+      // When the zip guard is on, non-ZIP archives show a full-width
+      // "unsupported file format" error with no sidebar, so the header
+      // must not reserve the 300px sidebar space for them.
+      !fg('platform_media_archive_zip_guard') || isZipMimeType(fileState.mimeType)));
       setItem(Outcome.successful(fileState));
     } else {
       setItem(Outcome.failed(new MediaViewerError('header-fetch-metadata', toCommonMediaClientError(fileState))));

package/dist/es2019/viewers/archiveSidebar/archive-sidebar-renderer.js

@@ -35,7 +35,14 @@ export default class ArchiveSidebarRenderer extends Component {
       this.setState({
         status: 'loaded'
       });
-      onError(new ArchiveViewerError('archiveviewer-read-binary', error instanceof Error ? error : undefined));
+      // Preserve archiveviewer-not-zip as-is so the generic error boundary
+      // can show a clear "format not supported" message instead of a
+      // generic read error.
+      if (error instanceof ArchiveViewerError && error.primaryReason === 'archiveviewer-not-zip') {
+        onError(error);
+      } else {
+        onError(new ArchiveViewerError('archiveviewer-read-binary', error instanceof Error ? error : undefined));
+      }
     }
   }
   render() {

package/dist/es2019/viewers/archiveSidebar/archive.js

@@ -3,6 +3,8 @@ import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import React from 'react';
 import { unzip, HTTPRangeReader } from 'unzipit';
 import { FormattedMessage } from 'react-intl';
+import { isErrorFileState } from '@atlaskit/media-client';
+import { isZipMimeType } from '@atlaskit/media-common/isZipMimeType';
 import { CustomMediaPlayer, messages } from '@atlaskit/media-ui';
 import { getLanguageType, isCodeViewerItem } from '@atlaskit/media-ui/codeViewer';
 import { Text } from '@atlaskit/primitives/compiled';
@@ -22,12 +24,23 @@ import { fireAnalytics } from '../../analytics';
 import { ArchiveViewerError, isMediaViewerError } from '../../errors';
 import { createZipEntryLoadSucceededEvent } from '../../analytics/events/operational/zipEntryLoadSucceeded';
 import { createZipEntryLoadFailedEvent } from '../../analytics/events/operational/zipEntryLoadFailed';
+import { createPreviewUnsupportedEvent } from '../../analytics/events/operational/previewUnsupported';
 import { CodeViewRenderer } from '../codeViewer/codeViewerRenderer';
 import { DEFAULT_LANGUAGE } from '../codeViewer/util';
 import { MAX_FILE_SIZE_SUPPORTED_BY_CODEVIEWER } from '../../item-viewer';
 import { NativePdfViewer } from './nativePdfViewer';
 import { fg } from '@atlaskit/platform-feature-flags';
 export const getArchiveEntriesFromFileState = async (fileState, mediaClient, collectionName) => {
+  // The browser-based archive viewer relies on `unzipit`, which only supports
+  // the ZIP container format. Media classifies 40+ archive formats (.tar, .gz,
+  // .rar, .7z, etc.) as `archive` and routes them all through this code path,
+  // causing unzipit to fail with a cryptic "could not find end of central
+  // directory" error. We short-circuit non-ZIP archives up-front so the UI
+  // can show a clear "format not supported" message instead. We rely on the
+  // file's mime type (provided by the media backend).
+  if (!isErrorFileState(fileState) && !isZipMimeType(fileState.mimeType) && fg('platform_media_archive_zip_guard')) {
+    throw new ArchiveViewerError('archiveviewer-not-zip');
+  }
   const url = await mediaClient.file.getFileBinaryURL(fileState.id, collectionName);
   const reader = new HTTPRangeReader(url);
   const archive = await rejectAfter(() => unzip(reader));
@@ -37,6 +50,25 @@ export class ArchiveViewerBase extends BaseViewer {
   constructor(...args) {
     super(...args);
     _defineProperty(this, "onError", (error, entry) => {
+      // Non-ZIP archives (e.g. RAR, TAR, 7z) are a known limitation of the
+      // browser-based `unzipit` viewer, not a per-entry failure. Render a
+      // full-width "format not supported" message via BaseViewer's failed
+      // outcome (no sidebar, no 300px offset) and do NOT escalate to the
+      // parent item-viewer error boundary, which would double-render the
+      // message inside the offset archive layout.
+      if (error.primaryReason === 'archiveviewer-not-zip') {
+        // Fire a dedicated, non-SLI `previewUnsupported` event (not a
+        // `loadFailed`) so unsupported archives don't pollute error
+        // dashboards. The event's fileAttributes distinguish the two cases:
+        // `fileMediatype: 'unknown'` (browser-unrecognised, e.g. RAR) vs
+        // `fileMediatype: 'archive'` with a non-ZIP `fileMimetype` (unzipit
+        // limitation, e.g. 7z/tar/gzip).
+        fireAnalytics(createPreviewUnsupportedEvent(this.props.item), this.props.createAnalyticsEvent);
+        this.setState({
+          content: Outcome.failed(error)
+        });
+        return;
+      }
       this.props.onError(error);
       this.setState({
         content: Outcome.successful({

package/dist/esm/analytics/events/operational/loadFailed.js

@@ -1,11 +1,13 @@
 import { getFileAttributes } from '../..';
 import { getPrimaryErrorReason, getSecondaryErrorReason, getErrorDetail, getRequestMetadata } from '../../../errors';
 export var createLoadFailedEvent = function createLoadFailedEvent(fileId, error, fileState, traceContext) {
+  var _failReason;
   var _getFileAttributes = getFileAttributes(fileState),
     fileMediatype = _getFileAttributes.fileMediatype,
     fileMimetype = _getFileAttributes.fileMimetype,
     fileSize = _getFileAttributes.fileSize;
   var requestMetadata = getRequestMetadata(error);
+  var processingFailReason = (fileState === null || fileState === void 0 ? void 0 : fileState.status) === 'failed-processing' ? (_failReason = fileState.failReason) !== null && _failReason !== void 0 ? _failReason : 'not-available' : undefined;
   return {
     eventType: 'operational',
     actionSubject: 'mediaFile',
@@ -17,6 +19,7 @@ export var createLoadFailedEvent = function createLoadFailedEvent(fileId, error,
       errorDetail: getErrorDetail(error),
       statusCode: requestMetadata === null || requestMetadata === void 0 ? void 0 : requestMetadata.statusCode,
       request: requestMetadata,
+      processingFailReason: processingFailReason,
       fileMimetype: fileMimetype,
       fileAttributes: {
         fileId: fileId,

package/dist/esm/analytics/index.js

@@ -1,7 +1,7 @@
 import { ANALYTICS_MEDIA_CHANNEL, sanitiseAnalyticsPayload } from '@atlaskit/media-common/analytics';
 var componentName = 'mediaViewer';
 var packageName = "@atlaskit/media-viewer";
-var packageVersion = "54.1.0";
+var packageVersion = "54.2.0";
 export { packageName, packageVersion, componentName, componentName as component };
 export function getFileAttributes(fileState) {
   if (!fileState) {

package/dist/esm/analytics/ufoExperiences.js

@@ -6,7 +6,7 @@ import { getMediaEnvironment, getMediaRegion } from '@atlaskit/media-client';
 import { getFeatureFlagKeysAllProducts } from '@atlaskit/media-common';
 import { fg } from '@atlaskit/platform-feature-flags';
 var packageName = "@atlaskit/media-viewer";
-var packageVersion = "54.1.0";
+var packageVersion = "54.2.0";
 var ufoExperience;
 var getExperience = function getExperience() {
   if (!ufoExperience) {

package/dist/esm/errorMessage.js

@@ -44,6 +44,16 @@ export var ErrorMessage = /*#__PURE__*/function (_React$Component) {
       var _this$props = this.props,
         formatMessage = _this$props.intl.formatMessage,
         error = _this$props.error;
+      // Non-ZIP archives (e.g. RAR, TAR, 7z) aren't a load failure - the
+      // browser-based viewer simply can't preview them. Show a dedicated
+      // "Unsupported file format" heading instead of the generic
+      // "Something went wrong" copy.
+      if (getPrimaryErrorReason(error) === 'archiveviewer-not-zip') {
+        return {
+          icon: errorLoadingFileImage(formatMessage),
+          messages: [i18nMessages.unsupported_file_format, i18nMessages.archive_format_not_supported]
+        };
+      }
       var errorInfo = {
         icon: errorLoadingFileImage(formatMessage),
         messages: [i18nMessages.something_went_wrong, i18nMessages.couldnt_generate_preview]

package/dist/esm/header.js

@@ -3,6 +3,8 @@ import React, { useEffect, useRef, useState } from 'react';
 import { isExternalImageIdentifier, isErrorFileState, toCommonMediaClientError } from '@atlaskit/media-client';
 import { hideControlsClassName, messages, toHumanReadableMediaSize, MediaButton } from '@atlaskit/media-ui';
 import { getLanguageType, getExtension, isCodeViewerItem } from '@atlaskit/media-ui/codeViewer';
+import { isZipMimeType } from '@atlaskit/media-common/isZipMimeType';
+import { fg } from '@atlaskit/platform-feature-flags';
 import { FormattedMessage, injectIntl } from 'react-intl';
 import { Outcome } from './domain';
 import { Header as HeaderWrapper, LeftHeader, RightHeader, MetadataWrapper, MetadataSubText, MedatadataTextWrapper, MetadataIconWrapper, MetadataFileName, FormattedMessageWrapper } from './styleWrappers';
@@ -74,7 +76,15 @@ export var Header = function Header(_ref) {
     }
     if (fileState.status !== 'error') {
       var _onSetArchiveSideBarV;
-      (_onSetArchiveSideBarV = onSetArchiveSideBarVisibleRef.current) === null || _onSetArchiveSideBarV === void 0 || _onSetArchiveSideBarV.call(onSetArchiveSideBarVisibleRef, !isErrorFileState(fileState) && fileState.mediaType === 'archive');
+      // Only reserve the archive sidebar space for previewable ZIP archives.
+      // Non-ZIP archives (e.g. RAR, TAR, 7z) render a full-width "unsupported
+      // file format" error in the body and have no sidebar, so the header
+      // must not leave an empty 300px gap for them.
+      (_onSetArchiveSideBarV = onSetArchiveSideBarVisibleRef.current) === null || _onSetArchiveSideBarV === void 0 || _onSetArchiveSideBarV.call(onSetArchiveSideBarVisibleRef, !isErrorFileState(fileState) && fileState.mediaType === 'archive' && (
+      // When the zip guard is on, non-ZIP archives show a full-width
+      // "unsupported file format" error with no sidebar, so the header
+      // must not reserve the 300px sidebar space for them.
+      !fg('platform_media_archive_zip_guard') || isZipMimeType(fileState.mimeType)));
       setItem(Outcome.successful(fileState));
     } else {
       setItem(Outcome.failed(new MediaViewerError('header-fetch-metadata', toCommonMediaClientError(fileState))));

package/dist/esm/viewers/archiveSidebar/archive-sidebar-renderer.js

@@ -59,7 +59,14 @@ var ArchiveSidebarRenderer = /*#__PURE__*/function (_Component) {
               this.setState({
                 status: 'loaded'
               });
-              onError(new ArchiveViewerError('archiveviewer-read-binary', _t instanceof Error ? _t : undefined));
+              // Preserve archiveviewer-not-zip as-is so the generic error boundary
+              // can show a clear "format not supported" message instead of a
+              // generic read error.
+              if (_t instanceof ArchiveViewerError && _t.primaryReason === 'archiveviewer-not-zip') {
+                onError(_t);
+              } else {
+                onError(new ArchiveViewerError('archiveviewer-read-binary', _t instanceof Error ? _t : undefined));
+              }
             case 4:
             case "end":
               return _context.stop();

package/dist/esm/viewers/archiveSidebar/archive.js

@@ -14,6 +14,8 @@ import _regeneratorRuntime from "@babel/runtime/regenerator";
 import React from 'react';
 import { unzip, HTTPRangeReader } from 'unzipit';
 import { FormattedMessage } from 'react-intl';
+import { isErrorFileState } from '@atlaskit/media-client';
+import { isZipMimeType } from '@atlaskit/media-common/isZipMimeType';
 import { CustomMediaPlayer, messages } from '@atlaskit/media-ui';
 import { getLanguageType, isCodeViewerItem } from '@atlaskit/media-ui/codeViewer';
 import { Text } from '@atlaskit/primitives/compiled';
@@ -33,6 +35,7 @@ import { fireAnalytics } from '../../analytics';
 import { ArchiveViewerError, isMediaViewerError } from '../../errors';
 import { createZipEntryLoadSucceededEvent } from '../../analytics/events/operational/zipEntryLoadSucceeded';
 import { createZipEntryLoadFailedEvent } from '../../analytics/events/operational/zipEntryLoadFailed';
+import { createPreviewUnsupportedEvent } from '../../analytics/events/operational/previewUnsupported';
 import { CodeViewRenderer } from '../codeViewer/codeViewerRenderer';
 import { DEFAULT_LANGUAGE } from '../codeViewer/util';
 import { MAX_FILE_SIZE_SUPPORTED_BY_CODEVIEWER } from '../../item-viewer';
@@ -44,19 +47,25 @@ export var getArchiveEntriesFromFileState = /*#__PURE__*/function () {
     return _regeneratorRuntime.wrap(function (_context) {
       while (1) switch (_context.prev = _context.next) {
         case 0:
-          _context.next = 1;
-          return mediaClient.file.getFileBinaryURL(fileState.id, collectionName);
+          if (!(!isErrorFileState(fileState) && !isZipMimeType(fileState.mimeType) && fg('platform_media_archive_zip_guard'))) {
+            _context.next = 1;
+            break;
+          }
+          throw new ArchiveViewerError('archiveviewer-not-zip');
         case 1:
+          _context.next = 2;
+          return mediaClient.file.getFileBinaryURL(fileState.id, collectionName);
+        case 2:
           url = _context.sent;
           reader = new HTTPRangeReader(url);
-          _context.next = 2;
+          _context.next = 3;
           return rejectAfter(function () {
             return unzip(reader);
           });
-        case 2:
+        case 3:
           archive = _context.sent;
           return _context.abrupt("return", archive);
-        case 3:
+        case 4:
         case "end":
           return _context.stop();
       }
@@ -75,6 +84,25 @@ export var ArchiveViewerBase = /*#__PURE__*/function (_BaseViewer) {
     }
     _this = _callSuper(this, ArchiveViewerBase, [].concat(args));
     _defineProperty(_this, "onError", function (error, entry) {
+      // Non-ZIP archives (e.g. RAR, TAR, 7z) are a known limitation of the
+      // browser-based `unzipit` viewer, not a per-entry failure. Render a
+      // full-width "format not supported" message via BaseViewer's failed
+      // outcome (no sidebar, no 300px offset) and do NOT escalate to the
+      // parent item-viewer error boundary, which would double-render the
+      // message inside the offset archive layout.
+      if (error.primaryReason === 'archiveviewer-not-zip') {
+        // Fire a dedicated, non-SLI `previewUnsupported` event (not a
+        // `loadFailed`) so unsupported archives don't pollute error
+        // dashboards. The event's fileAttributes distinguish the two cases:
+        // `fileMediatype: 'unknown'` (browser-unrecognised, e.g. RAR) vs
+        // `fileMediatype: 'archive'` with a non-ZIP `fileMimetype` (unzipit
+        // limitation, e.g. 7z/tar/gzip).
+        fireAnalytics(createPreviewUnsupportedEvent(_this.props.item), _this.props.createAnalyticsEvent);
+        _this.setState({
+          content: Outcome.failed(error)
+        });
+        return;
+      }
       _this.props.onError(error);
       _this.setState({
         content: Outcome.successful(_objectSpread(_objectSpread({}, _this.state.content.data), {}, {

package/dist/types/analytics/index.d.ts

@@ -1,6 +1,7 @@
 import { type FileState, type RequestMetadata } from '@atlaskit/media-client';
 import { type FileAttributes, type WithFileAttributes, type FailureAttributes, type WithTraceContext } from '@atlaskit/media-common/analytics';
 import { type CreateUIAnalyticsEvent } from '@atlaskit/analytics-next';
+import { type ProcessingFailReason } from '@atlaskit/media-state';
 import { type PrimaryErrorReason } from '../errors';
 import { type MediaViewerEventPayload } from './events';
 declare const componentName = "mediaViewer";
@@ -13,5 +14,6 @@ export type MediaViewerFailureAttributes = Omit<FailureAttributes, 'failReason'>
     failReason: PrimaryErrorReason;
     statusCode?: number;
     request?: RequestMetadata;
+    processingFailReason?: ProcessingFailReason | 'not-available';
 } & WithFileAttributes & WithTraceContext;
 export declare function fireAnalytics(payload: MediaViewerEventPayload, createAnalyticsEvent?: CreateUIAnalyticsEvent): void;

package/dist/types/errors.d.ts

@@ -14,7 +14,7 @@ export declare class ArchiveViewerError extends MediaViewerError {
 }
 export declare function isArchiveViewerError(err: Error): err is ArchiveViewerError;
 export type MediaViewerErrorReason = 'collection-fetch-metadata' | 'header-fetch-metadata' | 'itemviewer-onerror' | 'itemviewer-fetch-metadata' | 'itemviewer-file-error-status' | 'itemviewer-file-failed-processing-status' | 'imageviewer-external-onerror' | 'imageviewer-fetch-url' | 'imageviewer-src-onerror' | 'audioviewer-fetch-url' | 'audioviewer-missing-artefact' | 'audioviewer-playback' | 'videoviewer-fetch-url' | 'videoviewer-missing-artefact' | 'videoviewer-playback' | 'docviewer-fetch-url' | 'docviewer-content-fetch-failed' | 'docviewer-fetch-pdf' | 'codeviewer-fetch-src' | 'codeviewer-load-src' | 'codeviewer-file-size-exceeds' | 'codeviewer-parse-email' | 'svg-img-error' | 'svg-binary-fetch' | 'svg-unknown-error' | 'svg-blob-to-datauri' | 'unsupported' | 'custom-viewer-error' | 'download';
-export type ArchiveViewerErrorReason = 'archiveviewer-bundle-loader' | 'archiveviewer-read-binary' | 'archiveviewer-create-url' | 'archiveviewer-imageviewer-onerror' | 'archiveviewer-videoviewer-onerror' | 'archiveviewer-audioviewer-onerror' | 'archiveviewer-docviewer-onerror' | 'archiveviewer-codeviewer-onerror' | 'archiveviewer-codeviewer-file-size-exceeds' | 'archiveviewer-missing-name-src' | 'archiveviewer-unsupported' | 'archiveviewer-encrypted-entry' | 'archiveviewer-customrenderer-onerror';
+export type ArchiveViewerErrorReason = 'archiveviewer-bundle-loader' | 'archiveviewer-read-binary' | 'archiveviewer-not-zip' | 'archiveviewer-create-url' | 'archiveviewer-imageviewer-onerror' | 'archiveviewer-videoviewer-onerror' | 'archiveviewer-audioviewer-onerror' | 'archiveviewer-docviewer-onerror' | 'archiveviewer-codeviewer-onerror' | 'archiveviewer-codeviewer-file-size-exceeds' | 'archiveviewer-missing-name-src' | 'archiveviewer-unsupported' | 'archiveviewer-encrypted-entry' | 'archiveviewer-customrenderer-onerror';
 export type PrimaryErrorReason = MediaViewerErrorReason | ArchiveViewerErrorReason;
 export type SecondaryErrorReason = MediaClientErrorReason | 'unknown' | 'nativeError' | undefined;
 export declare function getPrimaryErrorReason(error: MediaViewerError): PrimaryErrorReason;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/media-viewer",
-	"version": "54.1.1",
+	"version": "54.2.1",
 	"description": "MediaViewer is Atlassian's powerful solution for viewing files on the web. It's both powerful and extendable yet easy-to-integrate",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -38,13 +38,14 @@
 		"@atlaskit/heading": "^6.0.0",
 		"@atlaskit/icon": "^36.0.0",
 		"@atlaskit/icon-file-type": "^8.0.0",
-		"@atlaskit/icon-lab": "^7.0.0",
-		"@atlaskit/media-client": "^37.0.0",
+		"@atlaskit/icon-lab": "^7.1.0",
+		"@atlaskit/media-client": "^37.1.0",
 		"@atlaskit/media-client-react": "^6.0.0",
-		"@atlaskit/media-common": "^14.0.0",
+		"@atlaskit/media-common": "^14.1.0",
 		"@atlaskit/media-document-viewer": "^1.0.0",
+		"@atlaskit/media-state": "^3.0.0",
 		"@atlaskit/media-svg": "^3.0.0",
-		"@atlaskit/media-ui": "^30.1.0",
+		"@atlaskit/media-ui": "^30.3.0",
 		"@atlaskit/platform-feature-flags": "^2.0.0",
 		"@atlaskit/portal": "^6.0.0",
 		"@atlaskit/primitives": "^20.0.0",
@@ -52,10 +53,11 @@
 		"@atlaskit/spinner": "^20.0.0",
 		"@atlaskit/textfield": "^9.0.0",
 		"@atlaskit/theme": "^26.0.0",
-		"@atlaskit/tmp-editor-statsig": "^109.0.0",
-		"@atlaskit/tokens": "^14.0.0",
+		"@atlaskit/tmp-editor-statsig": "^110.0.0",
+		"@atlaskit/tokens": "^15.0.0",
 		"@atlaskit/tooltip": "^23.0.0",
 		"@atlaskit/ufo": "^1.0.0",
+		"@atlassian/react-compiler-gating": "^0.2.0",
 		"@babel/runtime": "^7.0.0",
 		"@codemirror/language": "6.10.8",
 		"@codemirror/language-data": "6.5.1",
@@ -86,14 +88,12 @@
 		"@af/visual-regression": "workspace:^",
 		"@atlaskit/media-core": "^38.0.0",
 		"@atlaskit/media-integration-test-helpers": "workspace:^",
-		"@atlaskit/media-state": "^3.0.0",
 		"@atlaskit/media-test-data": "^4.0.0",
 		"@atlaskit/media-test-helpers": "^42.0.0",
 		"@atlaskit/ssr": "workspace:^",
 		"@atlaskit/toggle": "^17.0.0",
 		"@atlassian/a11y-jest-testing": "^0.12.0",
 		"@atlassian/feature-flags-test-utils": "^1.1.0",
-		"@atlassian/react-compiler-gating": "workspace:^",
 		"@atlassian/testing-library": "^0.6.0",
 		"@atlassian/ufo": "^0.8.0",
 		"@testing-library/dom": "^10.1.0",
@@ -106,7 +106,7 @@
 		"jsverify": "^0.8.3",
 		"react": "^18.2.0",
 		"react-dom": "^18.2.0",
-		"react-intl": "^6.6.2",
+		"react-intl": "^7.0.0",
 		"rxjs": "^5.5.0",
 		"wait-for-expect": "^1.2.0"
 	},
@@ -140,6 +140,9 @@
 		},
 		"platform_media_excel_lazy_load": {
 			"type": "boolean"
+		},
+		"platform_media_archive_zip_guard": {
+			"type": "boolean"
 		}
 	},
 	"techstack": {