gabriel / musehub public
fix minor dev
AI Agent gabriel · 74 days ago · Apr 9, 2026 · Diff

fix: backfill per_symbol_intel_json when early-return hits stale index row

Root cause: build_symbol_index's early-return path fired when existing_row.ref == head_commit_id even if per_symbol_intel_json was NULL. This happened when a long-running worker process cached a stale module version (before per_symbol_intel_json computation was added), writing index rows with NULL intel. Every subsequent push hit the early-return and left the NULL indefinitely, causing 'Anchored symbols not yet indexed' on the issue detail page and missing blast radius in the stats strip.

Fix: skip the early-return when per_symbol_intel_json is None; fall through to the full computation path which backfills the column from the existing symbol_history blob (no commit walk needed — op_count=0 but existing_row carries valid history).

Tests: 6 new tests in TestPerSymbolIntelBackfill covering: - early return fires only when intel is populated - stale row (correct ref, NULL intel) triggers backfill - lookup_symbol_intel returns {} for stale rows - per_symbol_intel_json populated on fresh build - all expected metric fields present - lookup returns only requested addresses

sha256:150703b6424cff6fab7cff597b7751fa78b005eafe1b5a347a960a3e5de6afdc sha
sha256:ed930a6a126558b0cffd7cd431c9ac63fdaa5ce68f11361b6ffa0ce8aa350ebf snapshot
← Older Oldest on dev
All commits
Newer → Latest on dev

0 comments

No comments yet. Be the first to start the discussion.

To add a comment, use the Muse CLI: muse hub commit comment sha256:150703b6424cff6fab7cff597b7751fa78b005eafe1b5a347a960a3e5de6afdc --body "your comment"