Fixed issue 1936: brackets sorting

Author: AnushkaNilangekar

src/api/parameters.js

@@ -31,8 +31,10 @@ export function buildParameterTree(parameters) {
         (cumulativePath in parameters && parameters[cumulativePath].label) ||
         key;
       // Transform e.g. "0]" -> 1
+      let bracketIndex = 0;
       if (key.endsWith("]")) {
-        label = `Bracket ${parseInt(key.slice(0, -1)) + 1}`;
+        bracketIndex = parseInt(key.slice(0, -1));
+        label = `Bracket ${bracketIndex + 1}`;
       }
       label = label.replaceAll("_", " ");
       if (!currentNode.children) {
@@ -46,7 +48,7 @@ export function buildParameterTree(parameters) {
         currentNode.children.push({
           label: label,
           name: cumulativePath,
-          index: 0,
+          index: bracketIndex,
           children: [],
         });
       }

src/pages/PolicyPage.jsx

@@ -75,7 +75,14 @@ function PolicyLeftSidebar(props) {
   const sortTreeInPlace = (tree) => {
     if (!Array.isArray(tree)) return [];
 
-    tree.sort((a, b) => a.label.localeCompare(b.label));
+    // Check if all nodes are bracket nodes
+    const allBrackets = tree.every((item) => /^Bracket \d+$/.test(item.label));
+    // If all nodes are bracket nodes, sort numerically by index else sort alphabetically by label
+    if (allBrackets) {
+      tree.sort((a, b) => a.index - b.index);
+    } else {
+      tree.sort((a, b) => a.label.localeCompare(b.label));
+    }
 
     tree.forEach((item) => {
       if (Array.isArray(item.children)) {