Set 13 reveal season, preview-card parser, and Temporary Shift support

CLAUDE.md

@@ -160,6 +160,8 @@ See `packages/synergy-engine/REMOVED_RULES.md` for archived rules (Evasive, Trib
 
 Shift cards find same-named base characters; base characters find Shift cards. Both directions use the same scoring. Scores 3-10 based on curve gap, inkwell flexibility, free Shift cost tiers, and condition activation.
 
+**Variants** (`getShiftType` in `utils/cardHelpers.ts`): `Shift N` and `Temporary Shift N` target same-named characters (standard); `X Shift N` (e.g. `Puppy Shift`, `Madrigal Shift`, `Temporary Red Panda Shift`) targets characters with classification `X` — the prefix may be multiple words, and a leading `Temporary` modifier is stripped before classifying; `Universal Shift N` targets any character. **Team** cards print a compound name ("Belle & Beast", "Sulley & Boo") and shift onto a character named either half, whatever the keyword's flavor label (`Shift`, `Combo Shift`, `Duo Shift`). The `&` in the name is the team signal — checked before the classification branch so `Combo`/`Duo` aren't misread as classifications; `standard` matching routes through `getShiftBaseNames`, which splits the base name on `&` so each component is a valid target (atomic names with no `&` pass through unchanged).
+
 **Full documentation**: See [`packages/synergy-engine/SHIFT_TARGET_RULE.md`](packages/synergy-engine/SHIFT_TARGET_RULE.md) for detailed score tables, examples, condition matchers, and design rationale.
 
 ### Rule 2: Named Companions (direct, forward-only matching)

apps/web/e2e/E2E_TESTS.md

@@ -196,10 +196,10 @@ Regression guard for issue #268 (skeleton-loading UI). Each test intercepts `/da
 
 | Test | What it verifies |
 |---|---|
-| renders hero and franchise tiers at /reveals | All 4 tier headings (Toy Story, Incredibles, Brave, Returning) render |
+| renders hero and franchise tiers at /reveals | All 4 tier headings (Monsters, Inc., Up, Turning Red, Returning) render |
 | desktop nav shows Reveals entry with NEW badge | `/` has a Reveals link with a "NEW" badge child |
-| mobile nav shows Reveals tab | On mobile, `/browse`'s bottom nav has a "Set 12 reveals" link |
-| promo modal appears on landing page and not on /reveals | `role="complementary" name=/Set 12 reveals/` visible on `/`, absent on `/reveals` |
+| mobile nav shows Reveals tab | On mobile, `/browse`'s bottom nav has a "Set 13 reveals" link |
+| promo modal appears on landing page and not on /reveals | `role="complementary" name=/Set 13 reveals/` visible on `/`, absent on `/reveals` |
 | tier card click opens the card overview modal | Clicking a card tile on `/reveals` opens the modal; URL stays `/reveals` |
 
 ## Patterns

apps/web/e2e/tests/reveals-page.spec.ts

@@ -40,9 +40,9 @@ test.describe('Reveals page (flag on)', () => {
 
     await page.goto('/reveals');
 
-    await expect(page.getByRole('heading', {name: /Toy Story/i})).toBeVisible();
-    await expect(page.getByRole('heading', {name: /The Incredibles/i})).toBeVisible();
-    await expect(page.getByRole('heading', {name: /Brave/i})).toBeVisible();
+    await expect(page.getByRole('heading', {name: 'Monsters, Inc.', exact: true})).toBeVisible();
+    await expect(page.getByRole('heading', {name: 'Up', exact: true})).toBeVisible();
+    await expect(page.getByRole('heading', {name: 'Turning Red', exact: true})).toBeVisible();
     await expect(page.getByRole('heading', {name: /Returning franchises/i})).toBeVisible();
   });
 
@@ -64,7 +64,7 @@ test.describe('Reveals page (flag on)', () => {
     await page.goto('/browse');
     const mobileNav = page.getByRole('navigation', {name: 'Mobile navigation'});
     // aria-label is the descriptive form after the Option B accessible-name refactor.
-    const revealsLink = mobileNav.getByRole('link', {name: 'Set 12 reveals', exact: true});
+    const revealsLink = mobileNav.getByRole('link', {name: 'Set 13 reveals', exact: true});
     await expect(revealsLink).toBeVisible();
   });
 
@@ -73,19 +73,24 @@ test.describe('Reveals page (flag on)', () => {
 
     await page.goto('/');
     await expect(
-      page.getByRole('complementary', {name: /Set 12 reveals/i}),
+      page.getByRole('complementary', {name: /Set 13 reveals/i}),
     ).toBeVisible();
 
     await page.goto('/reveals');
     await expect(
-      page.getByRole('complementary', {name: /Set 12 reveals/i}),
+      page.getByRole('complementary', {name: /Set 13 reveals/i}),
     ).toHaveCount(0);
   });
 
   test('tier card click opens the card overview modal', async ({page}, testInfo) => {
     if (testInfo.project.name.startsWith('mobile-')) test.skip();
 
     await page.goto('/reveals');
+    // Early reveal season: previewCards.json may still have cards: [] (no tiles to
+    // click). Skip until at least one Set 13 card is curated, mirroring the
+    // season-ended skip in beforeEach so the suite stays green across the lifecycle.
+    const tileCount = await page.getByTestId('card-tile').count();
+    test.skip(tileCount === 0, 'No reveal cards curated yet (previewCards.json cards: []).');
     const firstTile = page.getByTestId('card-tile').first();
     await expect(firstTile).toBeVisible();
     await firstTile.click();

apps/web/e2e/tests/synergy-detail-modal.spec.ts

@@ -155,10 +155,12 @@ test.describe('Synergy comparison — Mobile', () => {
     // queried on `page`, not scoped to the modal.
     await appPage.cardOverviewModal.getByRole('button', {name: /^Enlarge /}).first().click();
     const lightbox = page.getByRole('dialog', {name: /enlarged/i});
-    await expect(lightbox).toBeVisible({timeout: 3000});
+    // The mobile-safari binary runs the lightbox open/close transition slower than the
+    // 3s default; widen both assertions to 15s for determinism (same fix as #376's modal).
+    await expect(lightbox).toBeVisible({timeout: 15000});
 
     await lightbox.getByRole('button', {name: /close enlarged card/i}).click();
-    await expect(lightbox).toHaveCount(0, {timeout: 3000});
+    await expect(lightbox).toHaveCount(0, {timeout: 15000});
   });
 
   test('should exit comparison mode via BACK on mobile', async ({appPage}) => {