Merge pull request #5 from VirtualFlyBrain/feature/answer-quality-recipes

Release v3.3.1 — surface uncounted (count -1) term-info queries

Author: Robbie1977

lib/orchestrator.mjs

@@ -434,7 +434,7 @@ async function synthesise(ledger, deps, models) {
   // "no information" — when specific evidence is thin.
   const availableData = Object.values(ledger.terms)
     .filter(t => t.id && t.digest?.queries?.length)
-    .map(t => `${t.digest.name || t.label}: ${t.digest.queries.slice(0, 10).map(q => `${q.label} (${q.count})`).join('; ')}`)
+    .map(t => `${t.digest.name || t.label}: ${t.digest.queries.slice(0, 10).map(q => `${q.label} (${q.count < 0 ? 'available — run this query for the count' : q.count})`).join('; ')}`)
   const availableBlock = availableData.length
     ? `\n\nAVAILABLE VFB DATA for the resolved terms (counts of records VFB holds — use to say what IS available, never claim "no information" when this lists relevant data):\n${availableData.join('\n')}`
     : ''

lib/termInfoDigest.mjs

@@ -183,7 +183,13 @@ export function buildTermInfoDigest(info = {}) {
       // full parsed preview rows (name, id, thumbnail, tags) for result tables
       previewRows: parsed.slice(0, 5)
     }
-  }).filter(q => q.label && (q.count > 0 || q.examples.length > 0))
+  // Keep a query if it has data (count > 0), preview examples, OR an uncounted
+  // count (-1). get_term_info returns count -1 for query types it did not
+  // pre-count (common for large ones like ImagesNeurons on a big region) — that
+  // means the data IS available and the query must be RUN to get it, NOT that
+  // there is no data. Only drop a query that was actually counted as empty
+  // (count === 0) with no preview rows.
+  }).filter(q => q.label && (q.count !== 0 || q.examples.length > 0))
 
   const publications = (Array.isArray(info.Publications) ? info.Publications : [])
     .map(p => {
@@ -214,10 +220,16 @@ export function digestToText(digest, { maxQueries = 16, maxExamples = 5 } = {})
   if (digest.relationships) lines.push(`Relationships: ${digest.relationships}`)
   if (digest.synonyms?.length) lines.push(`Synonyms: ${digest.synonyms.join(', ')}`)
   if (digest.queries?.length) {
-    lines.push('Available VFB data for this term (query result: total count — examples):')
+    lines.push('Available VFB data for this term (query result: total count — examples). "run this query" means the total was not precomputed but the data exists — call vfb_run_query with that query_type to get the count/results:')
     for (const q of digest.queries.slice(0, maxQueries)) {
       const ex = q.examples.slice(0, maxExamples).join(', ')
-      lines.push(`- ${q.label}: ${q.count}${ex ? ` (e.g. ${ex})` : ''}`)
+      if (q.count < 0) {
+        // Count not precomputed by get_term_info — the data is available; the
+        // query must be run to get the count/results (do NOT read this as "no data").
+        lines.push(`- ${q.label}: run this query — call vfb_run_query with query_type ${q.query_type} to get the count/results`)
+      } else {
+        lines.push(`- ${q.label}: ${q.count}${ex ? ` (e.g. ${ex})` : ''}`)
+      }
     }
   }
   if (digest.publications?.length) {