fix: use onAnyAction rather than onAction to fix undo/redo bug

v3/src/models/shared/shared-case-metadata.ts

@@ -1,5 +1,6 @@
 import { observable } from "mobx"
 import { getSnapshot, getType, Instance, ISerializedActionCall, onAction, types } from "mobx-state-tree"
+import { onAnyAction } from "../../utilities/mst-utils"
 import { CategorySet, createProvisionalCategorySet, ICategorySet } from "../data/category-set"
 import { DataSet, IDataSet } from "../data/data-set"
 import { ISharedModel, SharedModel } from "./shared-model"
@@ -121,12 +122,12 @@ export const SharedCaseMetadata = SharedModel
           self.removeCategorySet(invalidAttrId)
         })
         const userActionNames = categorySet.userActionNames
-        onAction(categorySet, action => {
+        onAnyAction(categorySet, action => {
           // when a category set is changed by the user, it is promoted to a regular CategorySet
           if (categorySet && userActionNames.includes(action.name)) {
             self.promoteProvisionalCategorySet(categorySet)
           }
-        }, true)
+        })
       }
       return categorySet
     }