DemoSequence: validation inspector + fix obsolete APIs + shrink b/c decks
Three changes that all fall under "make trailer-capture authoring catch
Three changes that all fall under “make trailer-capture authoring catch its own mistakes.”
-
New custom inspector (Assets/Editor/DemoSequenceEditor.cs) When a DemoSequence asset is selected, the inspector now shows:
- ERROR if any card in scriptedInputs PlayCard entries is missing from deckLoadout (the “not in hand” warning at capture time, surfaced at edit time instead). Lists the missing cards by name.
- WARNING if deck size > 5 (starting hand cap). Cards in positions 6+ may not draw turn 1 because the deterministic shuffle decides which subset draws, not deck order.
- INFO when both checks pass.
- Auto-Tune button: replaces deckLoadout with the unique cards referenced in scripted inputs (in order of first appearance). Yields the smallest deck that satisfies every scripted play and guarantees the entire deck draws each turn after reshuffle.
-
Shrink class_b_combat + class_c_combat deck loadouts Both had 11-card decks where the showcase cards (Psionic Slam at position 8 / Eagle Eye at 11 / Snipe at 10) lived past the 5-card hand cap and never drew turn 1. class_a was already fixed manually in Inspector. Now b and c match: deck = exactly the cards the script needs. Captures will resolve every scripted play instead of silently dropping them.
-
Fix obsolete PlayerSettings.{Get,Set}ScriptingDefineSymbolsForGroup warnings in SequenceCaptureWindow.cs (CS0618). Switched to the NamedBuildTarget overload Unity recommends. Behavior unchanged. Added
using UnityEditor.Build;for the NamedBuildTarget type.
Co-Authored-By: Claude Opus 4.7 (1M context) [email protected]
Commit a97cdd5 by astafford8488.