feat: support id_aliases for PluginSlots

Author: brian-smith-tcril

src/plugins/PluginSlot.jsx

@@ -14,6 +14,7 @@ function BasePluginSlot({
   as = React.Fragment,
   children = null,
   id,
+  idAliases = [],
   pluginProps = {},
   slotOptions = {},
   slotErrorFallbackComponent,
@@ -23,7 +24,7 @@ function BasePluginSlot({
   for an example of how PluginSlot is populated, and example/src/index.jsx for a dummy JS config that holds all plugins
   */
 
-  const { keepDefault, plugins } = usePluginSlot(id);
+  const { keepDefault, plugins } = usePluginSlot(id, idAliases);
   const { formatMessage } = useIntl();
 
   const defaultContents = React.useMemo(() => {
@@ -118,6 +119,8 @@ BasePluginSlot.propTypes = {
   children: PropTypes.node,
   /** ID of the PluginSlot configuration */
   id: PropTypes.string.isRequired,
+  /** Aliases (additional IDs for the PluginSlot) */
+  idAliases: PropTypes.arrayOf(PropTypes.string),
   /** Props that are passed down to each Plugin in the Slot */
   pluginProps: PropTypes.shape(),
   /** Options passed to the PluginSlot */

src/plugins/data/hooks.js

@@ -14,12 +14,37 @@ import { PLUGIN_MOUNTED, PLUGIN_READY, PLUGIN_UNMOUNTED } from './constants';
  * @param {String} id - Name of PluginSlot
  * @returns {Object} - JS configuration for the PluginSlot
  */
-export function usePluginSlot(id) {
-  const configSlots = getConfigSlots()?.[id];
-  if (configSlots) {
-    return configSlots;
+export function usePluginSlot(id, idAliases = []) {
+  const notFound = { keepDefault: true, plugins: [] };
+
+  const allConfigSlots = getConfigSlots();
+
+  if (!allConfigSlots) {
+    return notFound;
+  }
+
+  // When defining a JS object with multiple entries
+  // that have the same key, only the last one is kept
+  //
+  // We want to treat all entries in the pluginSlots object
+  // in env.config.jsx that have either "id" or any of the
+  // "idAliases" as if they have the same id
+  //
+  // To do so, we grab the last entry in the object that with
+  // a key that matches either the "id" or any of the "idAliases"
+  const allSlotIds = [id].concat(idAliases);
+  const lastMatchingId = Object.keys(allConfigSlots).reverse().find(slotId => allSlotIds.includes(slotId));
+
+  if (!lastMatchingId) {
+    return notFound;
+  }
+
+  const configSlots = allConfigSlots[lastMatchingId];
+
+  if (!configSlots) {
+    return notFound;
   }
-  return { keepDefault: true, plugins: [] };
+  return configSlots;
 }
 
 /* Listening for events */

src/plugins/data/hooks.test.jsx

@@ -52,4 +52,42 @@ describe('usePluginSlots', () => {
       expect(slotConfig.plugins).toStrictEqual([]);
     });
   });
+
+  describe('when the plugin slot is defined via alias', () => {
+    it('returns keepDefault and plugin changes', () => {
+      getConfig.mockImplementation(() => (
+        {
+          pluginSlots: {
+            example_plugin_slot_alias: {
+              plugins: mockSlotChanges,
+              keepDefault: true,
+            },
+          },
+        }
+      ));
+
+      const slotConfig = usePluginSlot('example_plugin_slot', ['example_plugin_slot_alias']);
+      expect(slotConfig.keepDefault).toBe(true);
+      expect(slotConfig.plugins).toBe(mockSlotChanges);
+    });
+  });
+
+  describe('when no matching plugin slot is found', () => {
+    it('returns true for keepDefault and no plugin changes', () => {
+      getConfig.mockImplementation(() => (
+        {
+          pluginSlots: {
+            wrong_plugin_slot_alias: {
+              plugins: mockSlotChanges,
+              keepDefault: true,
+            },
+          },
+        }
+      ));
+
+      const slotConfig = usePluginSlot('example_plugin_slot', ['example_plugin_slot_alias']);
+      expect(slotConfig.keepDefault).toBe(true);
+      expect(slotConfig.plugins).toStrictEqual([]);
+    });
+  });
 });