fix: Strip the 'disabledReasons' property when saving blocks to the backpack.

Disabled blocks that are saved to the backpack retain their 'disabledReasons' property, which makes them impossible to take. Add 'disabledReasons' to the list of stripped keys and apply the stripping to blocks added via the 'Backpackable' interface ('addBackpackables').

Author: B-head

plugins/workspace-backpack/src/backpack.ts

@@ -487,7 +487,14 @@ export class Backpack
    */
   private cleanFlyoutInfo(dirtyInfo: Blockly.utils.toolbox.FlyoutItemInfo) {
     // The keys to remove.
-    const removeKeys = ['id', 'height', 'width', 'pinned', 'enabled'];
+    const removeKeys = [
+      'id',
+      'height',
+      'width',
+      'pinned',
+      'enabled',
+      'disabledReasons',
+    ];
 
     // Traverse the object recursively.
     const traverseClean = function (
@@ -615,7 +622,10 @@ export class Backpack
       backpackables
         .map((b) => b.toFlyoutInfo())
         .reduce((acc, curr) => [...acc, ...curr])
-        .map((info) => JSON.stringify(info)),
+        .map((info) => {
+          this.cleanFlyoutInfo(info);
+          return JSON.stringify(info);
+        }),
     );
   }