gramene-search 2.15.5 → 2.15.6

package/dist/index.js

@@ -313,7 +313,7 @@ const $9d9aeaf9299e61a1$var$grameneTaxonomy = (0, $gXNCa$reduxbundler.createAsyn
     persist: true,
     staleAfter: 86400000,
     getPromise: ({ store: store })=>{
-        return fetch(`${store.selectGrameneAPI()}/taxonomy?subset=gramene&rows=-1`).then((res)=>res.json()).then((taxNodes)=>{
+        return fetch(`${store.selectGrameneAPI()}/taxonomy?subset=gramene&rows=-1`).then((res)=>$9d9aeaf9299e61a1$var$okJson(res, 'taxonomy')).then((rows)=>$9d9aeaf9299e61a1$var$nonEmptyList(rows, 'taxonomy')).then((taxNodes)=>{
             let taxonomy = (0, ($parcel$interopDefault($gXNCa$lodash))).keyBy(taxNodes, '_id');
             taxNodes.forEach((t)=>{
                 t._id = +t._id; // ensure taxonomy id is a number
@@ -333,13 +333,28 @@ $9d9aeaf9299e61a1$var$grameneTaxonomy.reactGrameneTaxonomy = (0, $gXNCa$reduxbun
         actionCreator: 'doFetchGrameneTaxonomy'
     };
 });
+// A proxy hiccup can answer with a non-2xx status, or with 200 and an empty
+// list, and `res.json()` happily parses both. createAsyncResourceBundle would
+// then record a *success* with empty data and stamp lastSuccess, so with
+// staleAfter of 24h the client keeps believing it has genomes/taxonomy for a
+// whole day. Downstream that surfaces as "No genome found for {}" and a hard
+// "Bin count mismatch!" from gramene-bins-client. Throwing instead lets the
+// bundle record an error and retry.
+function $9d9aeaf9299e61a1$var$okJson(res, what) {
+    if (!res.ok) throw new Error(`${what} failed (${res.status})`);
+    return res.json();
+}
+function $9d9aeaf9299e61a1$var$nonEmptyList(rows, what) {
+    if (!Array.isArray(rows) || rows.length === 0) throw new Error(`${what} returned no rows`);
+    return rows;
+}
 const $9d9aeaf9299e61a1$var$grameneMaps = (0, $gXNCa$reduxbundler.createAsyncResourceBundle)({
     name: 'grameneMaps',
     actionBaseType: 'GRAMENE_MAPS',
     persist: true,
     staleAfter: 86400000,
     getPromise: ({ store: store })=>{
-        return fetch(`${store.selectGrameneAPI()}/maps?rows=-1`).then((res)=>res.json()).then((maps)=>{
+        return fetch(`${store.selectGrameneAPI()}/maps?rows=-1`).then((res)=>$9d9aeaf9299e61a1$var$okJson(res, 'maps')).then((rows)=>$9d9aeaf9299e61a1$var$nonEmptyList(rows, 'maps')).then((maps)=>{
             maps.forEach((m)=>{
                 m.regionLength = {};
                 m.regions.names.forEach((rname, idx)=>{
@@ -640,7 +655,9 @@ const $9d9aeaf9299e61a1$var$grameneTaxDist = {
         // yet, or a non-faceted search is transiently in state). Bail to null
         // until it's there rather than crashing on `a.length`.
         const binFacet = grameneSearch && grameneSearch.facet_counts && grameneSearch.facet_counts.facet_fields && grameneSearch.facet_counts.facet_fields.fixed_1000__bin;
-        if (binFacet && grameneTaxonomy && grameneMaps) {
+        // Object.keys guards matter here: an empty {} is truthy, and building the
+        // bin mapper from one throws "Bin count mismatch!" and takes the page down.
+        if (binFacet && grameneTaxonomy && Object.keys(grameneTaxonomy).length && grameneMaps && Object.keys(grameneMaps).length) {
             (0, ($parcel$interopDefault($gXNCa$lodash))).forIn(grameneMaps, (map, tid)=>{
                 if (grameneTaxonomy[tid]) grameneTaxonomy[tid].name = map.display_name;
             });