fix(miner-ui): align demo portfolio-queue items with the summary, fleet-wide and per repo#7286
Conversation
…et-wide and per repo DEFAULT_DEMO_PORTFOLIO_QUEUE_ITEMS listed only 4 actionable items while DEMO_PORTFOLIO_QUEUE_SUMMARY declares 3 in_progress + 15 done, so VITE_DEMO_MODE showed status cards above a table that visibly disagreed with them. Expand the fixture to 18 items whose in_progress/done split matches the summary both fleet-wide (3/15) AND per repo (widgets 1/6, api-gateway 1/4, docs-site 0/3, inventory 1/2) -- only the summary's `queued` rows, which an actionable item can't represent, are absent. Uses only the four synthetic demo repos and the existing identifier style. A test asserts the fleet-wide and per-repo counts agree so the fixtures can't drift again. Closes JSONbored#7227
|
🚨 Contributor flagged. Click here for more info: Superagent Dashboard |
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Warning ⏸️ LoopOver review result - manual review recommendedReview updated: 2026-07-19 12:40:44 UTC
Review summary Nits — 4 non-blocking
Flagged checks (non-blocking)
Decision drivers
Context & advisory signals — never blocks the verdict
Linked issue satisfactionAddressed Review context
Contributor next steps
Signal definitions
🧪 Chat with LoopOverAsk LoopOver a question about this PR directly in a comment — grounded only in the same cached, public-safe facts shown above, never a new claim.
Full command reference: https://loopover.ai/docs/loopover-commands 🧪 Experimental — new and may change. 🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by LoopOver, a quiet PR intelligence layer for OSS maintainers.
|
What
In
apps/loopover-miner-ui/src/lib/demo-data.ts(theVITE_DEMO_MODE=1fixture layer), two fixtures describing the same portfolio queue disagreed:DEMO_PORTFOLIO_QUEUE_SUMMARY.byStatusreportsin_progress: 3, done: 15(18 actionable items), butDEFAULT_DEMO_PORTFOLIO_QUEUE_ITEMSlisted only 4. A demo visitor saw status cards claiming "In progress: 3, Done: 15" directly above a table with only 4 rows.How
Expand
DEFAULT_DEMO_PORTFOLIO_QUEUE_ITEMSto 18 items that match the summary both fleet-wide and per repo:Every per-repo
in_progress/donesplit equals that repo'sDEMO_PORTFOLIO_QUEUE_SUMMARY.repos[].byStatus(only the summary'squeuedrows, whichPortfolioQueueActionItemcan't represent, are absent). Entirely synthetic — only the four existing demo repos and the establishedwgt-/gw-/docs-/inv-identifier style.Validation
A test added to
demo-data.test.ts's portfolio-queue block asserts the items'in_progress/donecounts matchDEMO_PORTFOLIO_QUEUE_SUMMARYfleet-wide and for every repo, plus a guard that every item references only the four synthetic repos. Confirmed the check fails against the old 4-item fixture;ui:lint/ui:typecheckand the demo-data suite pass;resetDemoDataForTestcontinues to restore the expanded fixture.Closes #7227