7120a354ec4870b3bc2112c384bdb0af837ed44e
braney
  Thu Apr 23 14:12:36 2026 -0700
hgc: make quickLifted NCBI RefSeq / UCSC RefSeq click details work. refs #36125

- trackHub.c isVetted(): accept refGene, ncbiRefSeq* subtracks, and ncbiOrtho
so the generated quickLift hub.txt doesn't emit 'avoidHandler on' for them.
Without this, hgc skips findNameBasedHandler and the detail click falls
through to genericClickHandler, showing none of the rich RefSeq fields.
- hgc.c findNameBasedHandler: when the tdb has 'quickLifted on', strip the
hub_NNN_ prefix from the dispatch table before comparisons so refGene /
ncbiRefSeq* match their native handlers.
- doNcbiRefSeq / doRefGene: route hAllocConn to quickLiftDb (source), use
trackHubSkipHubName for track/table name comparisons, and pass srcDb into
replaceInUrl, hTableExists, AceView / MGIid / jaxOrtholog / hg-prefix
checks -- otherwise these paths errAborted with "Unknown database
hub_NNN_<db>" on a quickLifted click.
- printRefSeqInfo / prRefGeneInfo / gbCdnaGetVersion: use sqlGetDatabase(conn)
rather than the global `database` for host-db checks.
- Predicted Protein / Predicted mRNA links for quickLifted tracks now point
at htcTranslatedPredMRna / htcGeneMrna so the sequences come from the
destination genome at the lifted exon coordinates, rather than the
NCBI-authored refPep / seqNcbiRefSeq extFiles on the source. Native
behavior is unchanged.
- showGenePos: fetch via quickLiftSql + calcLiftOverGenePreds when the track
is quickLifted so the Position line is in destination coordinates. The
swapped chainHash comes from the track's quickLiftUrl bigChain, matching
the pattern getGenePredForPositionSql already uses.
- Suppress the mRNA/Genomic Alignments block and its trailing <hr> for
quickLifted refGene / ncbiRefSeq* clicks. The PSLs come back in source
coords and the htcCdnaAli links don't line up with the destination window.
- getAlignmentsTName: use sqlGetDatabase(conn) rather than global `database`
so split-table resolution looks up the right db.
- printGeneCards: take a db argument; callers pass srcDb so the GeneCards
link renders on quickLifted pages (the "startsWith hg" check was matching
hub_NNN_hs1 as false).
- geneReviewsClick.c prGRShortRefGene: take a struct sqlConnection * rather
than opening its own on `database` (hub_NNN_hs1 -> Unknown database abort).
Callers in prRefGeneInfo and doOmimGene2 pass their existing source-db
conn, so the Related GeneReviews line now shows on both native and
quickLifted refGene pages.

Verified end-to-end on hgwdev-braney with hg38 -> hs1 quickLift of the
refSeqComposite: ncbiRefSeqCurated and refGene clicks on SHH now render
the full doNcbiRefSeq / doRefGene details (RefSeq / Status / Description
/ Synonyms / OMIM / Protein / HGNC / Entrez Gene / GeneCards / AceView /
Summary / Position / Gene Symbol); Predicted mRNA and Predicted Protein
links return destination-derived sequences; Genomic Sequence reaches the
Get-DNA-in-window page; Ctrl/Cmd+drag zoom no longer ends up with an
HTML error page inside the zoom dialog. Non-quickLifted hg38 refSeq
details are unchanged (alignments still shown, GeneReviews still present).

The duplicate-subtracks symptom from the ticket's step (i) is not
addressed here -- it's an architectural consequence of the shared
`refSeqComposite` cart key applying to both the destination's native
composite and the quickLift hub's composite, and needs cart-scoping
work in quickLift v2.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

diff --git src/hg/lib/trackHub.c src/hg/lib/trackHub.c
index 1f947a5a936..2ff11af1d34 100644
--- src/hg/lib/trackHub.c
+++ src/hg/lib/trackHub.c
@@ -1572,30 +1572,36 @@
 "clinvarSubLolly",
 "pubs",
 "pubsBlat",
 "pubsMarkerBand",
 "pubsMarkerSnp",
 "pubsMarkerGene",
 "refSeqComposite",
 //"gtexGeneV8",
 };
 
 static boolean isVetted(char *track)
 /* Is this a track that's been tested with quickLift?  If not we don't want to do the special name handling on the track. */
 {
 if (startsWith("wgEncodeGencode", track))
     return TRUE;
+// NCBI RefSeq composite subtracks (ncbiRefSeqCurated/Predicted/Other/Psl/
+// Select/Hgmd/Historical, ncbiOrtho) and UCSC RefSeq (refGene) have been
+// validated through quickLift; let their native hgc handlers fire.
+if (startsWith("ncbiRefSeq", track) || startsWith("ncbiOrtho", track) ||
+    sameString("refGene", track))
+    return TRUE;
 static bool inited = FALSE;
 static struct hash *vettedHash = NULL;
 
 if (!inited)
     {
     vettedHash = newHash(10);
 
     int ii;
     int len = sizeof(vettedTracks) / sizeof(char *);
 
     for(ii = 0; ii < len; ii++)
         hashStore(vettedHash, vettedTracks[ii]);
     }
 
 return hashLookup(vettedHash, track) != NULL;