fix: address PR review — stale closure in sort, false child indent for external refs

- handleModelSort: pass sortBy directly to sortModels instead of relying on async state
- applyModelDecorations: filter references to only include sibling model names, ignore external table references

Author: wicky-zipstack

frontend/src/ide/explorer/explorer-component.jsx

@@ -234,16 +234,33 @@ const IdeExplorer = ({
   };
 
   const applyModelDecorations = (models) => {
+    // Build set of model names to distinguish from external table references
+    const modelNames = new Set(models.map((m) => m.title));
     models.forEach((m) => {
-      if ((m.references || []).length > 0) {
+      const refs = (m.references || []).filter((r) => modelNames.has(r));
+      if (refs.length > 0) {
         m._isChild = true;
       }
     });
   };
 
-  const handleModelSort = (key) => {
-    setModelSortBy(key);
-    rebuildTree();
+  const handleModelSort = (sortBy) => {
+    setModelSortBy(sortBy);
+    if (rawTreeDataRef.current.length > 0) {
+      const freshData = JSON.parse(JSON.stringify(rawTreeDataRef.current));
+      freshData.forEach((node) => {
+        if (node.title === "models" && node.children) {
+          node.children.forEach((child) => {
+            if (child.title === "no_code" && child.children) {
+              child.children = sortModels(child.children, sortBy);
+              applyModelDecorations(child.children);
+            }
+          });
+        }
+      });
+      transformTree(freshData);
+      setTreeData(freshData, false);
+    }
   };
 
   // Function to map string icons from API to actual icon components