Refactor JavaScript platform redirect and improve stored platform handling

Co-authored-by: shannon.anahata <[email protected]>

Author: cursoragent

src/components/platformSelector/index.tsx

@@ -87,12 +87,21 @@ export function PlatformSelector({
 
   const router = useRouter();
   const onPlatformChange = (platformKey: string) => {
-    const platform_ = platformsAndGuides.find(
-      platform => platform.key === platformKey.replace('-redirect', '')
-    );
-    if (platform_) {
-      localStorage.setItem('active-platform', platform_.key);
-      router.push(platform_.url);
+    const cleanKey = platformKey.replace('-redirect', '');
+    let targetPlatform = platformsAndGuides.find(platform => platform.key === cleanKey);
+    
+    // Special handling for JavaScript: when platform "javascript" is selected,
+    // redirect to the virtual guide "javascript-platform" instead
+    if (cleanKey === 'javascript' && targetPlatform?.type === 'platform') {
+      const virtualGuide = platformsAndGuides.find(p => p.key === 'javascript-platform');
+      if (virtualGuide) {
+        targetPlatform = virtualGuide;
+      }
+    }
+    
+    if (targetPlatform) {
+      localStorage.setItem('active-platform', targetPlatform.key);
+      router.push(targetPlatform.url);
     }
   };
 
@@ -106,9 +115,18 @@ export function PlatformSelector({
   }, [open]);
 
   const [storedPlatformKey, setStoredPlatformKey] = useState<string | null>(null);
-  const storedPlatform = platformsAndGuides.find(
+  let storedPlatform = platformsAndGuides.find(
     platform => platform.key === storedPlatformKey
   );
+  
+  // Handle stored JavaScript platform: redirect to virtual guide
+  if (storedPlatformKey === 'javascript' && storedPlatform?.type === 'platform') {
+    const virtualGuide = platformsAndGuides.find(p => p.key === 'javascript-platform');
+    if (virtualGuide) {
+      storedPlatform = virtualGuide;
+    }
+  }
+  
   useEffect(() => {
     if (currentPlatformKey) {
       localStorage.setItem('active-platform', currentPlatformKey);
@@ -134,7 +152,7 @@ export function PlatformSelector({
     <div>
       <RadixSelect.Root
         defaultValue={currentPlatformKey}
-        value={showStoredPlatform ? storedPlatformKey : undefined}
+        value={showStoredPlatform ? storedPlatform?.key : undefined}
         onValueChange={onPlatformChange}
         open={open}
         onOpenChange={setOpen}
@@ -234,7 +252,7 @@ export function PlatformSelector({
           </RadixSelect.Content>
         </ComboboxProvider>
       </RadixSelect.Root>
-      {showStoredPlatform && (
+      {showStoredPlatform && storedPlatform && (
         <div className="mt-3">
           <SidebarLink
             href={storedPlatform.url}