fix: correct tier names and group variable convention

Tier names updated per Cal's spec:
  T0=Base Card, T1=Base Chrome, T2=Refractor, T3=Gold Refractor, T4=Superfractor

Also renames refractor_group → group_refractor per project convention.
All 39 tests pass.

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

cogs/refractor.py

@@ -22,10 +22,10 @@ logger = logging.getLogger("discord_app")
 PAGE_SIZE = 10
 
 TIER_NAMES = {
-    0: "Base Chrome",
-    1: "Refractor",
-    2: "Gold Refractor",
-    3: "Superfractor",
+    0: "Base Card",
+    1: "Base Chrome",
+    2: "Refractor",
+    3: "Gold Refractor",
     4: "Superfractor",
 }
 
@@ -122,11 +122,11 @@ class Refractor(commands.Cog):
     def __init__(self, bot):
         self.bot = bot
 
-    refractor_group = app_commands.Group(
+    group_refractor = app_commands.Group(
         name="refractor", description="Refractor tracking commands"
     )
 
-    @refractor_group.command(
+    @group_refractor.command(
         name="status", description="Show your team's refractor progress"
     )
     @app_commands.describe(

tests/test_refractor_commands.py

@@ -136,9 +136,9 @@ class TestFormatRefractorEntry:
         assert "**Mike Trout**" in result
 
     def test_tier_label_in_output(self, batter_state):
-        """Current tier name (Refractor for T1) appears in output."""
+        """Current tier name (Base Chrome for T1) appears in output."""
         result = format_refractor_entry(batter_state)
-        assert "(Refractor)" in result
+        assert "(Base Chrome)" in result
 
     def test_progress_values_in_output(self, batter_state):
         """current/threshold values appear in output."""
@@ -307,20 +307,20 @@ class TestTierNames:
     """
     Verify all tier display names are correctly defined.
 
-    T0=Base Chrome, T1=Refractor, T2=Gold Refractor, T3=Superfractor, T4=Superfractor
+    T0=Base Card, T1=Base Chrome, T2=Refractor, T3=Gold Refractor, T4=Superfractor
     """
 
-    def test_t0_base_chrome(self):
-        assert TIER_NAMES[0] == "Base Chrome"
+    def test_t0_base_card(self):
+        assert TIER_NAMES[0] == "Base Card"
 
-    def test_t1_refractor(self):
-        assert TIER_NAMES[1] == "Refractor"
+    def test_t1_base_chrome(self):
+        assert TIER_NAMES[1] == "Base Chrome"
 
-    def test_t2_gold_refractor(self):
-        assert TIER_NAMES[2] == "Gold Refractor"
+    def test_t2_refractor(self):
+        assert TIER_NAMES[2] == "Refractor"
 
-    def test_t3_superfractor(self):
-        assert TIER_NAMES[3] == "Superfractor"
+    def test_t3_gold_refractor(self):
+        assert TIER_NAMES[3] == "Gold Refractor"
 
     def test_t4_superfractor(self):
         assert TIER_NAMES[4] == "Superfractor"