fixes from staging deploy

Author: osfameron

preview-src/ui-model.yml

@@ -9,7 +9,7 @@ site:
       [
         { "title": "Server", "startPage": "home::server.adoc", "components": ["server", "*-connector"], "url": "/index.html", "color": "#c41016" },
         { "title": "Develop",
-          "subPages": [
+          "subGroups": [
             {
               "title": "Mobile",
               "startPage": "home::mobile.adoc",
@@ -19,8 +19,8 @@ site:
             {
               "title": "Cloud",
               "startPage": "home::cloud.adoc",
-              "components": ["operator"],
-              "url": "#"
+              "components": ["cloud"],
+              "url": "/cloud/index.html"
             }
           ]
         }
@@ -167,6 +167,15 @@ site:
       url: /couchbase-lite/2.1/index.html
     sync-gateway:
       url: /sync-gateway/2.1/index.html
+    cloud:
+      # continuously-deployed/unversioned component -- exercises the search
+      # catalog's handling of a genuine empty-string version (e.g. Capella)
+      versions:
+      - &latest_cloud
+        version: ''
+        title: Capella
+        url: /cloud/index.html
+      latest: *latest_cloud
     home:
       latest: &home_latest
         version: ''

src/helpers/search-catalog.js

@@ -87,12 +87,18 @@ module.exports = (
   // playbook) -- subGroups/components deliberately carry none of their own
   // here, and the client is what walks this tree inheriting a parent's color
   // down, same as this data looked when it was still a hand-maintained mock.
+  //
+  // components is only set when the group actually has direct components
+  // (never an empty array) -- the client tells "leaf group" apart from
+  // "subGroups-holding group" with a plain truthiness check, and an empty
+  // array is truthy in JS, so a subGroups-only group like "Develop" would
+  // otherwise wrongly look like an (empty) leaf and never recurse.
   function serializeGroup (group) {
     return {
       title: group.title,
       url: group.url,
       color: group.color,
-      components: (group.components || []).map(serializeComponent),
+      components: group.components && group.components.length ? group.components.map(serializeComponent) : undefined,
       latestVersions: group.latestVersions,
       subGroups: group.subGroups ? group.subGroups.map(serializeGroup) : undefined,
     }

src/js/13-docsearch.js

@@ -123,17 +123,34 @@
     return undefined
   }
 
+  // A continuously-deployed/unversioned component (Capella, AI Data Plane,
+  // etc.) has a genuine version of "" in Antora, not a missing one -- a
+  // plain truthiness check would silently drop it (same reasoning as the
+  // ['', 'master'].includes(...) checks elsewhere in this file, just
+  // inverted: there, '' means "don't show a redundant version picker";
+  // here, '' must still count as "this component has a real, includable
+  // version").
+  function hasVersion (versionMap, name) {
+    return versionMap && Object.prototype.hasOwnProperty.call(versionMap, name)
+  }
+
   function computeDefaultRefinement () {
     var pageUrl = metaContent('page-url')
     var landingGroup = pageUrl ? findGroupByUrl(componentCatalog.navGroups, pageUrl) : undefined
     if (landingGroup) {
-      return landingGroup.components
-        .filter(function (c) { return landingGroup.latestVersions && landingGroup.latestVersions[c.name] })
+      return (landingGroup.components || [])
+        .filter(function (c) { return hasVersion(landingGroup.latestVersions, c.name) })
         .map(function (c) { return c.name + '@' + landingGroup.latestVersions[c.name] })
     }
     var component = metaContent('docsearch:component')
     var version = metaContent('docsearch:cversion')
-    return component && version ? [component + '@' + version] : []
+    // "home" pages are landing/informational content (the site index, and
+    // every home::*.adoc category landing page not already caught above by
+    // its own group's url match) -- nav-group-for-page.js already treats
+    // these as belonging to no specific group for the same reason, so don't
+    // default search to "home"'s own (largely meaningless) component here.
+    if (!component || component === 'home' || version === undefined) return []
+    return [component + '@' + version]
   }
 
   // The refinementList widget only ever exposes component_version values that