@atlaskit/jql-editor 5.0.7 → 5.0.8

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @atlaskit/jql-editor
 
+## 5.0.8
+
+### Patch Changes
+
+- [#129314](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/129314)
+  [`439b876d25b73`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/439b876d25b73) -
+  Improved error logging for editor transactions.
+
 ## 5.0.7
 
 ### Patch Changes

package/dist/cjs/analytics/util.js

@@ -6,5 +6,5 @@ Object.defineProperty(exports, "__esModule", {
 exports.useJqlEditorAnalytics = void 0;
 var _jqlEditorCommon = require("@atlaskit/jql-editor-common");
 var useJqlEditorAnalytics = exports.useJqlEditorAnalytics = function useJqlEditorAnalytics(analyticsSource) {
-  return (0, _jqlEditorCommon.useJqlPackageAnalytics)(analyticsSource, "@atlaskit/jql-editor", "5.0.7", _jqlEditorCommon.ANALYTICS_CHANNEL);
+  return (0, _jqlEditorCommon.useJqlPackageAnalytics)(analyticsSource, "@atlaskit/jql-editor", "5.0.8", _jqlEditorCommon.ANALYTICS_CHANNEL);
 };

package/dist/cjs/state/index.js

@@ -452,10 +452,25 @@ var actions = exports.actions = {
         query = _getState7.query,
         editorState = _getState7.editorState,
         editorView = _getState7.editorView,
-        enableRichInlineNodes = _getState7.enableRichInlineNodes;
+        enableRichInlineNodes = _getState7.enableRichInlineNodes,
+        onDebugUnsafeMessage = _getState7.onDebugUnsafeMessage;
       var oldSelection = editorState.selection;
       var updatedQuery = (0, _documentText.getNodeText)(transaction.doc, 0, transaction.doc.content.size);
-      var updatedEditorState = editorState.apply(transaction);
+      var updatedEditorState;
+      try {
+        updatedEditorState = editorState.apply(transaction);
+      } catch (error) {
+        // We've observed several errors in Splunk from this step but we're unsure how to reproduce it. It seems to be some type of
+        // race condition where the transaction is applied to the editor state but the editor state is being updated in another transaction.
+        if (error instanceof RangeError && editorView) {
+          var message = "Error occurred trying to update editor state with the message: ".concat(error.message);
+          (0, _util.sendDebugMessage)(message, editorView, editorState, onDebugUnsafeMessage, {
+            stack: error.stack,
+            transaction: JSON.stringify(transaction)
+          });
+        }
+        throw error;
+      }
 
       // Update state in our editor view
       if (editorView) {

package/dist/es2019/analytics/util.js

@@ -1,4 +1,4 @@
 import { ANALYTICS_CHANNEL, useJqlPackageAnalytics } from '@atlaskit/jql-editor-common';
 export const useJqlEditorAnalytics = analyticsSource => {
-  return useJqlPackageAnalytics(analyticsSource, "@atlaskit/jql-editor", "5.0.7", ANALYTICS_CHANNEL);
+  return useJqlPackageAnalytics(analyticsSource, "@atlaskit/jql-editor", "5.0.8", ANALYTICS_CHANNEL);
 };

package/dist/es2019/state/index.js

@@ -463,11 +463,26 @@ export const actions = {
       query,
       editorState,
       editorView,
-      enableRichInlineNodes
+      enableRichInlineNodes,
+      onDebugUnsafeMessage
     } = getState();
     const oldSelection = editorState.selection;
     const updatedQuery = getNodeText(transaction.doc, 0, transaction.doc.content.size);
-    const updatedEditorState = editorState.apply(transaction);
+    let updatedEditorState;
+    try {
+      updatedEditorState = editorState.apply(transaction);
+    } catch (error) {
+      // We've observed several errors in Splunk from this step but we're unsure how to reproduce it. It seems to be some type of
+      // race condition where the transaction is applied to the editor state but the editor state is being updated in another transaction.
+      if (error instanceof RangeError && editorView) {
+        const message = `Error occurred trying to update editor state with the message: ${error.message}`;
+        sendDebugMessage(message, editorView, editorState, onDebugUnsafeMessage, {
+          stack: error.stack,
+          transaction: JSON.stringify(transaction)
+        });
+      }
+      throw error;
+    }
 
     // Update state in our editor view
     if (editorView) {

package/dist/esm/analytics/util.js

@@ -1,4 +1,4 @@
 import { ANALYTICS_CHANNEL, useJqlPackageAnalytics } from '@atlaskit/jql-editor-common';
 export var useJqlEditorAnalytics = function useJqlEditorAnalytics(analyticsSource) {
-  return useJqlPackageAnalytics(analyticsSource, "@atlaskit/jql-editor", "5.0.7", ANALYTICS_CHANNEL);
+  return useJqlPackageAnalytics(analyticsSource, "@atlaskit/jql-editor", "5.0.8", ANALYTICS_CHANNEL);
 };

package/dist/esm/state/index.js

@@ -445,10 +445,25 @@ export var actions = {
         query = _getState7.query,
         editorState = _getState7.editorState,
         editorView = _getState7.editorView,
-        enableRichInlineNodes = _getState7.enableRichInlineNodes;
+        enableRichInlineNodes = _getState7.enableRichInlineNodes,
+        onDebugUnsafeMessage = _getState7.onDebugUnsafeMessage;
       var oldSelection = editorState.selection;
       var updatedQuery = getNodeText(transaction.doc, 0, transaction.doc.content.size);
-      var updatedEditorState = editorState.apply(transaction);
+      var updatedEditorState;
+      try {
+        updatedEditorState = editorState.apply(transaction);
+      } catch (error) {
+        // We've observed several errors in Splunk from this step but we're unsure how to reproduce it. It seems to be some type of
+        // race condition where the transaction is applied to the editor state but the editor state is being updated in another transaction.
+        if (error instanceof RangeError && editorView) {
+          var message = "Error occurred trying to update editor state with the message: ".concat(error.message);
+          sendDebugMessage(message, editorView, editorState, onDebugUnsafeMessage, {
+            stack: error.stack,
+            transaction: JSON.stringify(transaction)
+          });
+        }
+        throw error;
+      }
 
       // Update state in our editor view
       if (editorView) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/jql-editor",
-	"version": "5.0.7",
+	"version": "5.0.8",
 	"description": "This package allows consumers to render an advanced JQL editor component to enable autocomplete-assisted authoring and validation of JQL queries.",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -50,10 +50,10 @@
 		"@atlaskit/jql-parser": "^2.0.0",
 		"@atlaskit/legacy-custom-icons": "^0.22.0",
 		"@atlaskit/platform-feature-flags": "^1.1.0",
-		"@atlaskit/primitives": "^14.1.0",
+		"@atlaskit/primitives": "^14.2.0",
 		"@atlaskit/spinner": "^18.0.0",
 		"@atlaskit/theme": "^18.0.0",
-		"@atlaskit/tokens": "^4.4.0",
+		"@atlaskit/tokens": "^4.5.0",
 		"@atlaskit/tooltip": "^20.0.0",
 		"@babel/runtime": "^7.0.0",
 		"@emotion/react": "^11.7.1",