From 2ed5099fd609dfa40ea269c3ed68b17a318a01f2 Mon Sep 17 00:00:00 2001
From: Randall Stillwell <me@randallstillwell.com>
Date: Sat, 13 Jun 2026 00:45:53 -0500
Subject: [PATCH] refactor(deck): extract grouped card list panel (Brief 2)

Move the grouped card list and empty-deck state into DeckDetailCardList;
page header + stats sidebar remain inline for Brief 3.

Co-authored-by: Cursor <cursoragent@cursor.com>
---
 components/DeckDetailCardList.js | 91 +++++++++++++++++++++++++++++++
 pages/deck/[id].js               | 93 +-------------------------------
 2 files changed, 93 insertions(+), 91 deletions(-)
 create mode 100644 components/DeckDetailCardList.js

diff --git a/components/DeckDetailCardList.js b/components/DeckDetailCardList.js
new file mode 100644
index 0000000..217e7fa
--- /dev/null
+++ b/components/DeckDetailCardList.js
@@ -0,0 +1,91 @@
+/* eslint-disable @next/next/no-img-element -- External card image URLs; next/image migration is out of scope. */
+import { ManaCost } from './ManaSymbols';
+
+export default function DeckDetailCardList({ cards, groupedCards }) {
+  const isEmpty = !cards || cards.length === 0;
+
+  return (
+    <div className="lg:col-span-3">
+      <div className="glass-panel rounded-2xl p-6">
+        {isEmpty ? (
+          <div className="text-center py-12">
+            <div className="text-6xl mb-4">🃏</div>
+            <h3 className="text-xl font-semibold mb-2" style={{ color: 'var(--text-primary)' }}>
+              Empty Deck
+            </h3>
+            <p style={{ color: 'var(--text-secondary)' }}>
+              This deck doesn&apos;t have any cards yet
+            </p>
+          </div>
+        ) : (
+          <div className="space-y-6">
+            {Object.entries(groupedCards)
+              .sort(([a], [b]) => a.localeCompare(b))
+              .map(([group, groupCards]) => (
+                <div key={group}>
+                  <h3
+                    className="text-lg font-semibold mb-3 border-b pb-2"
+                    style={{
+                      color: 'var(--text-primary)',
+                      borderColor: 'var(--border)',
+                    }}
+                  >
+                    {group} ({groupCards.reduce((sum, card) => sum + card.quantity, 0)})
+                  </h3>
+                  <div className="grid grid-cols-1 md:grid-cols-2 gap-3">
+                    {groupCards
+                      .sort((a, b) => a.name.localeCompare(b.name))
+                      .map((card) => (
+                        <div
+                          key={`${card.card_id}-${card.id}`}
+                          className="flex items-center space-x-3 p-3 rounded-xl transition-colors nav-item-hover"
+                          style={{ backgroundColor: 'var(--bg-primary)' }}
+                        >
+                          {card.image_url && (
+                            <img
+                              src={card.image_url}
+                              alt={card.name}
+                              className="w-12 h-16 object-cover rounded"
+                            />
+                          )}
+                          <div className="flex-1 min-w-0">
+                            <div className="flex items-center justify-between">
+                              <h4
+                                className="font-medium truncate"
+                                style={{ color: 'var(--text-primary)' }}
+                              >
+                                {card.name}
+                              </h4>
+                              <span
+                                className="font-medium ml-2"
+                                style={{ color: 'var(--text-primary)' }}
+                              >
+                                {card.quantity}x
+                              </span>
+                            </div>
+                            <p className="text-sm" style={{ color: 'var(--text-secondary)' }}>
+                              {card.set_name}
+                            </p>
+                            <div
+                              className="flex items-center space-x-2 text-xs"
+                              style={{ color: 'var(--text-secondary)' }}
+                            >
+                              {card.mana_cost && (
+                                <ManaCost cost={card.mana_cost} size="xs" />
+                              )}
+                              {card.rarity && (
+                                <span className="capitalize">{card.rarity}</span>
+                              )}
+                            </div>
+                          </div>
+                        </div>
+                      ))}
+                  </div>
+                </div>
+              ))}
+          </div>
+        )}
+      </div>
+    </div>
+  );
+}
diff --git a/pages/deck/[id].js b/pages/deck/[id].js
index 54d3808..4e546ee 100644
--- a/pages/deck/[id].js
+++ b/pages/deck/[id].js
@@ -1,10 +1,9 @@
-/* eslint-disable @next/next/no-img-element -- External or generated image URLs; next/image migration is out of scope. */
 import { useState, useEffect } from 'react';
 import { useRouter } from 'next/router';
 import Link from 'next/link';
 import Layout from '../../components/Layout';
+import DeckDetailCardList from '../../components/DeckDetailCardList';
 import DeckDetailStatsSidebar from '../../components/DeckDetailStatsSidebar';
-import { ManaCost } from '../../components/ManaSymbols';
 import { Button } from '../../components/ui';
 import { computeDeckStats } from '../../lib/deck-builder-stats.js';
 import { groupDeckCards } from '../../lib/deck-detail-grouping.js';
@@ -164,95 +163,7 @@ export default function DeckDetail() {
             onGroupByChange={setGroupBy}
           />
 
-          {/* Card List */}
-          <div className="lg:col-span-3">
-            <div className="glass-panel rounded-2xl p-6">
-              {deck.cards && deck.cards.length === 0 ? (
-                <div className="text-center py-12">
-                  <div className="text-6xl mb-4">🃏</div>
-                  <h3
-                    className="text-xl font-semibold mb-2"
-                    style={{ color: 'var(--text-primary)' }}
-                  >
-                    Empty Deck
-                  </h3>
-                  <p style={{ color: 'var(--text-secondary)' }}>
-                    This deck doesn&apos;t have any cards yet
-                  </p>
-                </div>
-              ) : (
-                <div className="space-y-6">
-                  {Object.entries(groupedCards)
-                    .sort(([a], [b]) => a.localeCompare(b))
-                    .map(([group, cards]) => (
-                      <div key={group}>
-                        <h3
-                          className="text-lg font-semibold mb-3 border-b pb-2"
-                          style={{
-                            color: 'var(--text-primary)',
-                            borderColor: 'var(--border)',
-                          }}
-                        >
-                          {group} ({cards.reduce((sum, card) => sum + card.quantity, 0)})
-                        </h3>
-                        <div className="grid grid-cols-1 md:grid-cols-2 gap-3">
-                          {cards
-                            .sort((a, b) => a.name.localeCompare(b.name))
-                            .map((card) => (
-                              <div
-                                key={`${card.card_id}-${card.id}`}
-                                className="flex items-center space-x-3 p-3 rounded-xl transition-colors nav-item-hover"
-                                style={{ backgroundColor: 'var(--bg-primary)' }}
-                              >
-                                {card.image_url && (
-                                  <img
-                                    src={card.image_url}
-                                    alt={card.name}
-                                    className="w-12 h-16 object-cover rounded"
-                                  />
-                                )}
-                                <div className="flex-1 min-w-0">
-                                  <div className="flex items-center justify-between">
-                                    <h4
-                                      className="font-medium truncate"
-                                      style={{ color: 'var(--text-primary)' }}
-                                    >
-                                      {card.name}
-                                    </h4>
-                                    <span
-                                      className="font-medium ml-2"
-                                      style={{ color: 'var(--text-primary)' }}
-                                    >
-                                      {card.quantity}x
-                                    </span>
-                                  </div>
-                                  <p
-                                    className="text-sm"
-                                    style={{ color: 'var(--text-secondary)' }}
-                                  >
-                                    {card.set_name}
-                                  </p>
-                                  <div
-                                    className="flex items-center space-x-2 text-xs"
-                                    style={{ color: 'var(--text-secondary)' }}
-                                  >
-                                    {card.mana_cost && (
-                                      <ManaCost cost={card.mana_cost} size="xs" />
-                                    )}
-                                    {card.rarity && (
-                                      <span className="capitalize">{card.rarity}</span>
-                                    )}
-                                  </div>
-                                </div>
-                              </div>
-                            ))}
-                        </div>
-                      </div>
-                    ))}
-                </div>
-              )}
-            </div>
-          </div>
+          <DeckDetailCardList cards={deck.cards} groupedCards={groupedCards} />
         </div>
       </div>
     </Layout>
-- 
2.45.2