-
-
Notifications
You must be signed in to change notification settings - Fork 90
feat(miner-manage-ui): manage-phase dashboard panel for run portfolio #4279
Copy link
Copy link
Closed
Labels
gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.gittensor:priorityMaintainer-selected Gittensor priority — scores a 1.5x multiplier.Maintainer-selected Gittensor priority — scores a 1.5x multiplier.help wantedExtra attention is neededExtra attention is needed
Description
Metadata
Metadata
Assignees
Labels
gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.gittensor:priorityMaintainer-selected Gittensor priority — scores a 1.5x multiplier.Maintainer-selected Gittensor priority — scores a 1.5x multiplier.help wantedExtra attention is neededExtra attention is needed
Projects
StatusShow more project fields
Done
A basic textual panel for managed PRs already exists:
manage-status.js(#2325/#3070) implementscollectManageStatus(manage-status.js:59-107, joining the portfolio queue and event ledger),renderManageStatusTable(manage-status.js:114-141), and a wired CLI command —gittensory-miner manage status [--json]is dispatched inbin/gittensory-miner.js(cliArgs[0] === "manage" && cliArgs[1] === "status"). So this issue is not "build a panel from nothing"; it's narrower than the title suggests once the existing coverage is accounted for.Two gaps remain. First,
collectManageStatusis PR-scoped only — it never readsrun-state.js, so there's no view of a repo's own discover→plan→prepare lifecycle (idle/discovering/planning/preparing) alongside its tracked PRs; a "run portfolio" panel implies both. Second, an actual GUI dashboard surface doesn't exist in this repo yet:apps/gittensory-miner-ui/is explicitly Phase 6 scope in the same roadmap tracker this issue comes from (tracker #2353:feat(miner-ui): scaffold apps/gittensory-miner-ui/ dashboard shell), and Phase 6 has not been released as filed issues yet (only Phase 4 is being released now) — so building a real dashboard app here would be building ahead of its own scaffold.Deliverables
collectManageStatus(or add a sibling aggregator) to also readrun-state.js's per-repo state and fold it into the panel — a "run portfolio" row per tracked repo showing its current discover/plan/prepare state alongside its managed PR rows, not PRs alonerenderManageStatusTable/add a new render function for the combined view; keep the existingmanage statusoutput and--jsonshape backward compatible (additive fields, not a breaking rename)gittensory-miner manage statuscommand for now — do not scaffoldapps/gittensory-miner-ui/here; note the dependency on the (not-yet-filed) Phase 6 dashboard-shell item for an eventual GUI surfacetest/unit/miner-manage-status.test.ts's existing style for the run-state-inclusive aggregation and render outputpackages/gittensory-miner/README.md's## Commandssection, which currently doesn't listmanage status/manage pollat all despite both being wired inbin/gittensory-miner.jsReferences
packages/gittensory-miner/lib/manage-status.js:59-141(collectManageStatus,renderManageStatusTable— existing PR-only panel)packages/gittensory-miner/bin/gittensory-miner.js:37(existingmanage statusCLI dispatch)packages/gittensory-miner/lib/run-state.js:6,100-102(RUN_STATES,getRunState— the per-repo signal not yet folded in)packages/gittensory-miner/README.md:68-78(## Commandssection missingmanage status/manage poll)#2353, Phase 6:feat(miner-ui): scaffold apps/gittensory-miner-ui/ dashboard shell(not yet filed/released — the dependency this issue should note, not preempt)