fix(registry): canonicalize prediction-market category to prediction-markets#6464
fix(registry): canonicalize prediction-market category to prediction-markets#6464glorydavid03023 wants to merge 1 commit into
Conversation
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
1 similar comment
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6464 +/- ##
=======================================
Coverage 97.86% 97.86%
=======================================
Files 176 176
Lines 22330 22330
Branches 8838 8838
=======================================
Hits 21853 21853
Misses 79 79
Partials 398 398 🚀 New features to boost your workflow:
|
|
Caution 🛑 LoopOver review result - reject/close recommendedReview updated: 2026-07-17 10:52:12 UTC
Review summary Nits — 2 non-blocking
Why this is blocked
📋 Copy for AI agents — paste into your coding agentDecision 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.
|
|
LoopOver is closing this pull request on the maintainer's behalf (Registry surface review). This is an automated maintenance action — to pursue this change, please open a new pull request with the issues resolved. Closed PRs may be analyzed later to improve review accuracy, but they are not automatically reopened or re-reviewed. |
Summary
Fixes #6332. The registry used two spellings for the same concept —
prediction-market(8-ball.jsonSN125,sparket.jsonSN57) andprediction-markets(almanac.jsonSN41,degenbrain.jsonSN90) — so a category filter/facet for one spelling silently excluded the other two subnets.This adopts the plural
prediction-marketsas canonical (per the issue's recommendation) and updates the two singular files.registry/subnets/8-ball.json:prediction-market→prediction-marketsregistry/subnets/sparket.json:prediction-market→prediction-marketsPlural matches the dominant convention elsewhere in the corpus (
language-models,capital-markets,trading-strategies,developer-tools). Verified registry-wide that these were the only two files using the singular form; existing alphabetical ordering ofcategories[]is preserved in both files.Per the issue this is a pure data fix — no
categoriesenum is added (explicitly out of scope).Expected outcome
A category filter for "prediction markets" now surfaces all 4 relevant subnets (SN41, SN57, SN90, SN125) instead of 2.
Validation
npm run validate:surface -- registry/subnets/8-ball.json→ passed (5 surfaces)npm run validate:surface -- registry/subnets/sparket.json→ passed (5 surfaces)npx prettier --check→ cleanCloses #6332