Found during the two-axis review of #88 (Standards axis, hard violation).
AGENTS.md → "Svelte Development Guide > Component Philosophy" states: "Single Responsibility: One thing per component. Decompose if exceeding 150 lines." Five new components exceed it:
ZoneRenderer.svelte — 646TabletopV2.svelte — 522ComponentSidebar.svelte — 320PileRenderer.svelte — 263ShuffleAnimation.svelte — 162
ZoneRenderer.svelte is the worst case and breaches a second documented rule — "A third mode/variant prop means refactor into base + sub-components" / "Prefer Snippets over Conditionals." It renders all four zone types through if-cascades in one file ({#if spread}, {#if grid}, group via class:group, freeform background/nesting, plus a per-type edit toolbar). This is the exact "base + per-type sub-components" case the rule names — e.g. split into ZoneSpread/ZoneGrid/ZoneGroup behind a base renderer.
Fix: decompose each of the above; prioritise ZoneRenderer.svelte's per-type split.
Found during the two-axis review of #88 (Standards axis, hard violation).
AGENTS.md → "Svelte Development Guide > Component Philosophy" states: "Single Responsibility: One thing per component. Decompose if exceeding 150 lines." Five new components exceed it:
ZoneRenderer.svelte— 646TabletopV2.svelte— 522ComponentSidebar.svelte— 320PileRenderer.svelte— 263ShuffleAnimation.svelte— 162ZoneRenderer.svelteis the worst case and breaches a second documented rule — "A third mode/variant prop means refactor into base + sub-components" / "Prefer Snippets over Conditionals." It renders all four zone types through if-cascades in one file ({#if spread},{#if grid}, group viaclass:group, freeform background/nesting, plus a per-type edit toolbar). This is the exact "base + per-type sub-components" case the rule names — e.g. split intoZoneSpread/ZoneGrid/ZoneGroupbehind a base renderer.Fix: decompose each of the above; prioritise
ZoneRenderer.svelte's per-type split.