Fix filter hidden site spaces

Author: nolannbiron

packages/gitbook/src/lib/context.ts

@@ -259,7 +259,7 @@ export async function fetchSiteContextByIds(
     };
 
     const sections = ids.siteSection
-        ? parseSiteSectionsAndGroups(siteStructure, ids.siteSection, ids.siteSpace)
+        ? parseSiteSectionsAndGroups(siteStructure, ids.siteSection)
         : null;
 
     // Parse the current siteSpace and siteSpaces based on the site structure type.
@@ -274,9 +274,7 @@ export async function fetchSiteContextByIds(
                 );
             }
 
-            const visibleSiteSpaces = filterHiddenSiteSpaces(siteSpaces, ids.siteSpace);
-
-            return { siteSpaces: visibleSiteSpaces, siteSpace };
+            return { siteSpaces: filterHiddenSiteSpaces(siteSpaces), siteSpace };
         }
 
         if (siteStructure.type === 'sections') {
@@ -432,51 +430,41 @@ export function checkIsRootSiteContext(context: GitBookSiteContext): boolean {
 /**
  * Filter out hidden site spaces from a list of site spaces.
  */
-function filterHiddenSiteSpaces(siteSpaces: SiteSpace[], currentSiteSpaceId?: string): SiteSpace[] {
-    return siteSpaces.filter(
-        (siteSpace) => !siteSpace.hidden || siteSpace.id === currentSiteSpaceId
-    );
+function filterHiddenSiteSpaces(siteSpaces: SiteSpace[]): SiteSpace[] {
+    return siteSpaces.filter((siteSpace) => !siteSpace.hidden);
 }
 
 /**
  * Filter out hidden site spaces from sections and groups recursively.
  */
 function filterHiddenSiteSpacesFromSections(
-    sections: (SiteSection | SiteSectionGroup)[],
-    currentSiteSpaceId?: string
+    sections: (SiteSection | SiteSectionGroup)[]
 ): (SiteSection | SiteSectionGroup)[] {
     return sections.map((item) => {
         if (item.object === 'site-section') {
             return {
                 ...item,
-                siteSpaces: filterHiddenSiteSpaces(item.siteSpaces, currentSiteSpaceId),
+                siteSpaces: filterHiddenSiteSpaces(item.siteSpaces),
             };
         }
         if (item.object === 'site-section-group') {
             return {
                 ...item,
-                children: filterHiddenSiteSpacesFromSections(item.children, currentSiteSpaceId),
+                children: filterHiddenSiteSpacesFromSections(item.children),
             };
         }
         return item;
     });
 }
 
-function parseSiteSectionsAndGroups(
-    structure: SiteStructure,
-    siteSectionId: string,
-    currentSiteSpaceId?: string
-) {
+function parseSiteSectionsAndGroups(structure: SiteStructure, siteSectionId: string) {
     const sectionsAndGroups = getSiteStructureSections(structure, { ignoreGroups: false });
-    const filteredSectionsAndGroups = filterHiddenSiteSpacesFromSections(
-        sectionsAndGroups,
-        currentSiteSpaceId
-    );
+    const filteredSectionsAndGroups = filterHiddenSiteSpacesFromSections(sectionsAndGroups);
     const section = parseCurrentSection(structure, siteSectionId);
     assert(section, `couldn't find section "${siteSectionId}" in site structure`);
     const filteredSection = {
         ...section,
-        siteSpaces: filterHiddenSiteSpaces(section.siteSpaces, currentSiteSpaceId),
+        siteSpaces: filterHiddenSiteSpaces(section.siteSpaces),
     };
     return {
         list: filteredSectionsAndGroups,