docs: aggiungi temp.md con TODO per completamento suite di test

La suite era allineata a una vecchia forma dello stato (sp.punt/sp.set/
sp.servHome) e a una vecchia architettura e2e (controller su :3001).
Baseline iniziale: 77/170 test Vitest falliti.

Vitest (ora 212/212 verdi):
- gameState.test.js: riscritto con helper che derivano punteggio/set/
  servizio dalla striscia; aggiunto blocco formInizio
- server-utils.js: getNetworkIPs accetta interfacce iniettabili e
  printServerInfo accetta gli IP iniettabili (deterministico anche su WSL);
  filtro LAN unificato (esclude 127./169.254./172.)
- websocket + stress: punteggio letto via punteggio(striscia)
- ControllerPage/DisplayPage: forzato layout mobile (viewport portrait),
  punteggi impostati via striscia; aggiunto test bottone REFERTO
- nuovi: referto.test.js, persist.test.js (mock fs), wsMixin.test.js,
  integration/server.test.js (routing)

Refactor di supporto:
- referto.js: estratta buildRefertoHtml(state, now) pura; generaReferto
  resta wrapper con window.open/print
- server.js: estratti createApp()/startServer(); avvio solo se entrypoint

e2e (migrazione parziale, NON ancora verificata verde):
- tutti i riferimenti controller :3001 -> :3000/controller
- forzato viewport portrait sul controller prima del goto
- reset helper: chiude il dialog di configurazione che doReset apre
- game-simulation: gestione del dialog SET VINTO automatico a 25

.claude/settings.json

@@ -0,0 +1,7 @@
+{
+  "enabledPlugins": {
+    "claude-md-management@claude-plugins-official": true,
+    "code-simplifier@claude-plugins-official": true,
+    "frontend-design@claude-plugins-official": true
+  }
+}

CLAUDE.md

@@ -2,26 +2,28 @@
 
 This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository.
 
-## Project Purpose
+## Scopo del progetto
 
-**Segnapunti Anto** is a real-time volleyball scoreboard PWA. An Express/WebSocket server hosts the game state; two separate web interfaces — a **display** (public scoreboard) and a **controller** (operator panel) — stay in sync via WebSocket.
+**Segnapunti Anto** è una PWA per il segnapunti pallavolo in tempo reale. Un server Express/WebSocket gestisce lo stato di gioco; due interfacce Vue separate — **display** (tabellone pubblico) e **controller** (pannello operatore) — si sincronizzano via WebSocket.
 
-## Commands
+## Comandi
 
 ```bash
 npm run dev          # Vite dev server — display: :5173/display, controller: :5173/controller
-npm run serve        # Build + run production — display: :3000/display, controller: :3000/controller
+npm run serve        # Build + avvio produzione — display: :3000/display, controller: :3000/controller
 
-npm run test         # Vitest watch mode
+npm run test         # Vitest in modalità watch
-npm run test:all     # All Vitest suites once (unit, integration, component, stress)
+npm run test:all     # Tutte le suite Vitest in una sola esecuzione
-npm run test:unit    # Unit + integration only
+npm run test:unit    # Solo unit + integration
-npm run test:component  # Vue component tests
+npm run test:component  # Test componenti Vue (happy-dom)
-npm run test:stress  # Load tests (50+ concurrent clients)
+npm run test:stress  # Load test (50+ client concorrenti)
-npm run test:e2e     # Playwright E2E (requires servers to be running via npm run serve)
+npm run test:e2e     # Playwright E2E (richiede npm run serve attivo)
-npm run test:e2e:ui  # Playwright with interactive UI
+npm run test:e2e:ui  # Playwright con UI interattiva
 ```
 
-## Architecture
+Per eseguire un singolo file di test: `npx vitest run tests/unit/gameState.test.js`
+
+## Architettura
 
 ```
 Controller (Vue)  ──WebSocket──┐
@@ -30,33 +32,51 @@ Display (Vue)     ──WebSocket──┤── websocket-handler.js ── gam
                                │         └── persist.js ── .segnapunti/state.json
 ```
 
-All game logic lives in `src/gameState.js` as three pure functions:
+**Tutta la logica di gioco** è in `src/gameState.js` come tre funzioni pure esportate:
-- `createInitialState()` — returns the initial state
+- `createInitialState()` — restituisce lo stato iniziale
-- `applyAction(state, action)` — immutable reducer (deep-clones via `structuredClone`)
+- `applyAction(state, action)` — reducer immutabile (deep-clone via `structuredClone`)
-- `checkVittoria(state)` — volleyball win conditions (25-point sets with 2-point margin, 15-point final set)
+- `checkVittoria(state)` — condizioni di vittoria set (25 punti, vantaggio di 2; 15 punti nel set decisivo)
 
-`src/websocket-handler.js` receives actions, validates that the sender is a registered controller (not just a display), calls `applyAction`, broadcasts the new state to all clients, then calls `onStateChange` to persist state to disk.
+`src/websocket-handler.js` riceve i messaggi WebSocket, valida che il mittente sia un `controller` (non un `display`), chiama `applyAction`, fa il broadcast del nuovo stato a tutti i client, poi invoca `onStateChange` per persistere su disco.
 
-`server.js` (Express) serves both interfaces on port 3000: `/display` → `dist/index.html`, `/controller` → `dist/controller.html`. Single WebSocket endpoint at `/ws`.
+`server.js` (Express) serve entrambe le interfacce sulla porta 3000: `/display` → `dist/index.html`, `/controller` → `dist/controller.html`. Singolo endpoint WebSocket su `/ws`.
 
-In development, `vite-plugin-websocket.js` embeds the WebSocket server inside the Vite dev server with URL rewrite middleware for `/display` and `/controller`.
+In sviluppo, `vite-plugin-websocket.js` incorpora il server WebSocket dentro il dev server Vite con middleware di URL rewrite per `/display` e `/controller`.
 
-## Key Design Constraints
+## Vincoli di design
 
-- **All game rules on the server** — clients are pure UI; the server is the source of truth.
+- **Tutta la logica sul server** — i client sono pura UI; il server è l'unica source of truth.
-- **Role-based WebSocket** — clients register as `display` or `controller`; only controllers may send actions.
+- **WebSocket role-based** — i client si registrano come `display` o `controller`; solo i controller possono inviare azioni.
-- **Immutable state** — `applyAction` never mutates; always returns a new state object.
+- **Stato immutabile** — `applyAction` non muta mai lo stato, restituisce sempre un nuovo oggetto.
-- **Single-controller intent** — the design targets one active controller and one display; no conflict resolution exists for simultaneous controllers.
+- **Un solo controller** — il design prevede un controller e un display attivi; non esiste conflict resolution per controller simultanei.
-- **Persistent state** — `src/persist.js` saves state to `.segnapunti/state.json` after every action and loads it on server startup.
+- **Stato persistente** — `src/persist.js` salva lo stato in `.segnapunti/state.json` dopo ogni azione e lo carica all'avvio del server.
 
-## Test Layout
+## Layout dei test
 
-| Suite | Path | Runner |
+| Suite | Percorso | Runner |
-|-------|------|--------|
+|-------|----------|--------|
 | Unit | `tests/unit/` | Vitest + Node |
 | Integration | `tests/integration/` | Vitest + Node |
 | Component | `tests/component/` | Vitest + Happy-DOM |
 | Stress | `tests/stress/` | Vitest + Node |
 | E2E | `tests/e2e/` | Playwright (Chromium, Firefox, Mobile Chrome) |
 
-E2E tests run serially (`workers: 1`) to avoid WebSocket state races. Run `npm run serve` before `npm run test:e2e`.
+I test E2E girano in serie (`workers: 1`) per evitare race condition sullo stato WebSocket. Eseguire `npm run serve` prima di `npm run test:e2e`.
+
+## Deploy
+
+Il progetto si distribuisce tramite Docker. L'immagine è pubblicata sul registry Gitea self-hosted (`santantonio.sytes.net`):
+
+```bash
+docker compose up -d   # Avvia con immagine dal registry privato (porta 3000)
+```
+
+Il volume `./.segnapunti` persiste lo stato tra i riavvii del container.
+
+## Istruzioni operative
+
+- Per ogni nuova funzionalità: analizza come si inserisce nel flusso `action → applyAction → broadcast`, poi decidi se aggiungere un nuovo tipo di action o estendere uno esistente.
+- Qualsiasi modifica alle regole di gioco va fatta esclusivamente in `src/gameState.js`.
+- Qualsiasi modifica al protocollo WebSocket va fatta in `src/websocket-handler.js`.
+- Aggiungere test unit in `tests/unit/gameState.test.js` per ogni nuovo action type o regola di gioco.
+- Il codice deve essere scritto in italiano per commenti e nomi di variabili di dominio (es. `servHome`, `striscia`, `nomi`), ma in inglese per nomi tecnici standard (`state`, `action`, `handler`).

README.md

@@ -14,7 +14,6 @@ Segnapunti digitale in tempo reale per partite di pallavolo. Un server centrale
 - [Architettura](#architettura)
 - [Guida utente](#guida-utente)
 - [Funzionalità](#funzionalità)
-- [Shortcuts tastiera](#shortcuts-tastiera)
 - [Deploy con Docker](#deploy-con-docker)
 - [Sviluppo](#sviluppo)
 - [Test](#test)
@@ -111,25 +110,6 @@ http://<IP-del-server>:3000/controller
 
 ---
 
-## Shortcuts tastiera
-
-> Valide sul controller da browser desktop.
-
-| Tasto | Azione |
-|---|---|
-| `Ctrl + ↑` | Punto casa |
-| `Ctrl + ↓` | Annulla ultimo punto |
-| `Shift + ↑` | Punto ospite |
-| `Ctrl + ←` | Cambia servizio (solo a 0-0) |
-| `Ctrl + M` | Apri configurazione |
-| `Ctrl + C` | Cambi squadra casa |
-| `Shift + C` | Cambi squadra ospite |
-| `Ctrl + Z` | Toggle punteggio / formazioni |
-| `Ctrl + S` | Annuncio vocale punteggio |
-| `Ctrl + B` | Mostra/nascondi barra pulsanti |
-| `Ctrl + F` | Fullscreen |
-
----
 
 ## Deploy con Docker
 

docker-compose.yml

@@ -1,12 +1,9 @@
 services:
   segnapunti:
-    image: santantonio.sytes.net/attilio/segnapunti:1.0.0
+    image: santantonio.sytes.net/attilio/segnapunti:2.0.0
     container_name: segnapunti
     ports:
       - "3000:3000"
     volumes:
-      - segnapunti-state:/app/.segnapunti
+      - ./.segnapunti:/app/.segnapunti
     restart: unless-stopped
-
-volumes:
-  segnapunti-state:

package.json

@@ -4,7 +4,7 @@
   "version": "2.0.0",
   "type": "module",
   "scripts": {
-    "dev": "vite",
+    "dev": "vite --host",
     "build": "vite build",
     "serve": "vite build && node server.js",
     "test": "vitest",

server.js

@@ -1,7 +1,7 @@
 import { createServer } from 'http'
 import express from 'express'
 import { WebSocketServer } from 'ws'
-import { fileURLToPath } from 'url'
+import { fileURLToPath, pathToFileURL } from 'url'
 import { dirname, join } from 'path'
 import { setupWebSocketHandler } from './src/websocket-handler.js'
 import { printServerInfo } from './src/server-utils.js'
@@ -10,26 +10,34 @@ import { loadState, saveState } from './src/persist.js'
 const __filename = fileURLToPath(import.meta.url)
 const __dirname = dirname(__filename)
 
-const PORT = process.env.PORT || 3000
+const DIST_DIR = join(__dirname, 'dist')
-const distDir = join(__dirname, 'dist')
 
-const app = express()
+// Crea l'app Express (asset statici + route display/controller) senza avviare il
+// listen né il WebSocket: così il routing è testabile in isolamento.
+export function createApp(distDir = DIST_DIR) {
+    const app = express()
 
-app.use(express.static(distDir, { index: false }))
+    app.use(express.static(distDir, { index: false }))
 
-app.get(['/', '/display', '/display/*splat'], (_req, res) => {
+    app.get(['/', '/display', '/display/*splat'], (_req, res) => {
         res.sendFile(join(distDir, 'index.html'))
-})
+    })
 
-app.get(['/controller', '/controller/*splat'], (_req, res) => {
+    app.get(['/controller', '/controller/*splat'], (_req, res) => {
         res.sendFile(join(distDir, 'controller.html'))
-})
+    })
 
-const server = createServer(app)
+    return app
-const wss = new WebSocketServer({ noServer: true })
+}
-setupWebSocketHandler(wss, { initialState: loadState(), onStateChange: saveState })
 
-server.on('upgrade', (request, socket, head) => {
+// Avvia HTTP + WebSocket. Lo stato viene caricato da disco e ripersistito ad ogni azione.
+export function startServer(port = process.env.PORT || 3000) {
+    const app = createApp()
+    const server = createServer(app)
+    const wss = new WebSocketServer({ noServer: true })
+    setupWebSocketHandler(wss, { initialState: loadState(), onStateChange: saveState })
+
+    server.on('upgrade', (request, socket, head) => {
         const pathname = new URL(request.url, `http://${request.headers.host}`).pathname
         if (pathname === '/ws') {
             wss.handleUpgrade(request, socket, head, (ws) => {
@@ -38,8 +46,16 @@ server.on('upgrade', (request, socket, head) => {
         } else {
             socket.destroy()
         }
-})
+    })
 
-server.listen(PORT, '0.0.0.0', () => {
+    server.listen(port, '0.0.0.0', () => {
-    printServerInfo(PORT)
+        printServerInfo(port)
-})
+    })
+
+    return server
+}
+
+// Avvia solo se eseguito direttamente (`node server.js`), non quando importato nei test.
+if (process.argv[1] && import.meta.url === pathToFileURL(process.argv[1]).href) {
+    startServer()
+}

src/components/DisplayPage.vue

@@ -7,22 +7,22 @@
           <span :style="{ 'float': 'left' }">
             {{ state.sp.nomi.home }}
             <span class="serv-slot">
-              <img v-show="state.sp.servHome" src="/serv.png" width="25" alt="Servizio" />
+              <img v-show="servHome" src="/serv.png" width="25" alt="Servizio" />
             </span>
-            <span v-if="state.visuForm" class="score-inline">{{ state.sp.punt.home }}</span>
+            <span v-if="state.visuForm" class="score-inline">{{ punt.home }}</span>
           </span>
-          <span class="mr3" :style="{ 'float': 'right' }">set {{ state.sp.set.home }}</span>
+          <span class="mr3" :style="{ 'float': 'right' }">set {{ set.home }}</span>
         </div>
 
         <div class="hea guest">
           <span :style="{ 'float': 'right' }">
-            <span v-if="state.visuForm" class="score-inline">{{ state.sp.punt.guest }}</span>
+            <span v-if="state.visuForm" class="score-inline">{{ punt.guest }}</span>
             <span class="serv-slot">
-              <img v-show="!state.sp.servHome" src="/serv.png" width="25" alt="Servizio" />
+              <img v-show="!servHome" src="/serv.png" width="25" alt="Servizio" />
             </span>
             {{ state.sp.nomi.guest }}
           </span>
-          <span class="ml3" :style="{ 'float': 'left' }">set {{ state.sp.set.guest }}</span>
+          <span class="ml3" :style="{ 'float': 'left' }">set {{ set.guest }}</span>
         </div>
 
         <span v-if="state.visuForm">
@@ -39,8 +39,8 @@
         </span>
         <span v-else>
           <div class="punteggio-container">
-            <div class="col punt home">{{ state.sp.punt.home }}</div>
+            <div class="col punt home">{{ punt.home }}</div>
-            <div class="col punt guest">{{ state.sp.punt.guest }}</div>
+            <div class="col punt guest">{{ punt.guest }}</div>
           </div>
         </span>
       </span>
@@ -51,22 +51,22 @@
           <span :style="{ 'float': 'left' }">
             {{ state.sp.nomi.guest }}
             <span class="serv-slot">
-              <img v-show="!state.sp.servHome" src="/serv.png" width="25" alt="Servizio" />
+              <img v-show="!servHome" src="/serv.png" width="25" alt="Servizio" />
             </span>
-            <span v-if="state.visuForm" class="score-inline">{{ state.sp.punt.guest }}</span>
+            <span v-if="state.visuForm" class="score-inline">{{ punt.guest }}</span>
           </span>
-          <span class="mr3" :style="{ 'float': 'right' }">set {{ state.sp.set.guest }}</span>
+          <span class="mr3" :style="{ 'float': 'right' }">set {{ set.guest }}</span>
         </div>
 
         <div class="hea home">
           <span :style="{ 'float': 'right' }">
-            <span v-if="state.visuForm" class="score-inline">{{ state.sp.punt.home }}</span>
+            <span v-if="state.visuForm" class="score-inline">{{ punt.home }}</span>
             <span class="serv-slot">
-              <img v-show="state.sp.servHome" src="/serv.png" width="25" alt="Servizio" />
+              <img v-show="servHome" src="/serv.png" width="25" alt="Servizio" />
             </span>
             {{ state.sp.nomi.home }}
           </span>
-          <span class="ml3" :style="{ 'float': 'left' }">set {{ state.sp.set.home }}</span>
+          <span class="ml3" :style="{ 'float': 'left' }">set {{ set.home }}</span>
         </div>
 
         <span v-if="state.visuForm">
@@ -83,8 +83,8 @@
         </span>
         <span v-else>
           <div class="punteggio-container">
-            <div class="col punt guest">{{ state.sp.punt.guest }}</div>
+            <div class="col punt guest">{{ punt.guest }}</div>
-            <div class="col punt home">{{ state.sp.punt.home }}</div>
+            <div class="col punt home">{{ punt.home }}</div>
           </div>
         </span>
       </span>
@@ -116,80 +116,15 @@
 </template>
 
 <script>
+import { createWsMixin } from '../wsMixin.js'
+
 export default {
   name: "DisplayPage",
-  data() {
+  mixins: [createWsMixin('display')],
-    return {
-      ws: null,
-      wsConnected: false,
-      isConnecting: false,
-      reconnectTimeout: null,
-      reconnectAttempts: 0,
-      maxReconnectDelay: 30000, // Ritardo massimo di riconnessione: 30 secondi
-      state: {
-        order: true,
-        visuForm: false,
-        visuStriscia: true,
-        modalitaPartita: "3/5",
-        sp: {
-          striscia: [{ serv: 'home', r: [] }],
-          servHome: true,
-          punt: { home: 0, guest: 0 },
-          set: { home: 0, guest: 0 },
-          nomi: { home: "Antoniana", guest: "Guest" },
-          form: {
-            home: ["1", "2", "3", "4", "5", "6"],
-            guest: ["1", "2", "3", "4", "5", "6"],
-          },
-        },
-      },
-    }
-  },
   mounted() {
-    this.connectWebSocket()
-    // Attiva la modalita fullscreen su dispositivi mobili.
     if (this.isMobile()) {
       try { document.documentElement.requestFullscreen() } catch (e) {}
     }
-
-    // Gestisce l'HMR di Vite evitando riconnessioni durante la ricarica a caldo.
-    if (import.meta.hot) {
-      import.meta.hot.on('vite:beforeUpdate', () => {
-        // Annulla eventuali tentativi di riconnessione pianificati.
-        if (this.reconnectTimeout) {
-          clearTimeout(this.reconnectTimeout)
-          this.reconnectTimeout = null
-        }
-      })
-    }
-  },
-  beforeUnmount() {
-    // Pulisce il timeout di riconnessione.
-    if (this.reconnectTimeout) {
-      clearTimeout(this.reconnectTimeout)
-      this.reconnectTimeout = null
-    }
-
-    // Chiude il WebSocket con il codice di chiusura appropriato.
-    if (this.ws) {
-      this.ws.onclose = null // Rimuove il listener per evitare nuove riconnessioni pianificate.
-      this.ws.onerror = null
-      this.ws.onmessage = null
-      this.ws.onopen = null
-
-      // Usa il codice 1000 (chiusura normale) se la connessione e aperta.
-      try {
-        if (this.ws.readyState === WebSocket.OPEN) {
-          this.ws.close(1000, 'Component unmounting')
-        } else if (this.ws.readyState === WebSocket.CONNECTING) {
-          // Se la connessione e ancora in fase di apertura, chiude direttamente.
-          this.ws.close()
-        }
-      } catch (err) {
-        console.error('[Display] Error closing WebSocket:', err)
-      }
-      this.ws = null
-    }
   },
   computed: {
     stricciaStrip() {
@@ -197,8 +132,8 @@ export default {
       if (!currentSet) return { home: [], guest: [] }
       let h = 0, g = 0
       const home = [], guest = []
-      for (const scorer of currentSet.r) {
+      for (const scorer of currentSet.ris) {
-        if (scorer === 'home') { h++; home.push(h); guest.push(' ') }
+        if (scorer === 'h') { h++; home.push(h); guest.push(' ') }
         else               { g++; guest.push(g); home.push(' ') }
       }
       return { home, guest }
@@ -216,154 +151,25 @@ export default {
     },
   },
   methods: {
+    onWsMessage(msg) {
+      if (msg.type === 'speak') this.speakOnDisplay(msg.text)
+      else if (msg.type === 'error') console.error('[Display] Server error:', msg.message)
+    },
     isMobile() {
       return /Android|webOS|iPhone|iPad|iPod|BlackBerry|IEMobile|Opera Mini/i.test(navigator.userAgent)
     },
-    connectWebSocket() {
-      // Evita connessioni simultanee multiple.
-      if (this.isConnecting) {
-        console.log('[Display] Already connecting, skipping...')
-        return
-      }
-
-      // Chiude la connessione precedente, se presente.
-      if (this.ws) {
-        this.ws.onclose = null
-        this.ws.onerror = null
-        this.ws.onmessage = null
-        this.ws.onopen = null
-        try {
-          if (this.ws.readyState === WebSocket.OPEN) {
-            this.ws.close(1000, 'Reconnecting')
-          } else if (this.ws.readyState === WebSocket.CONNECTING) {
-            this.ws.close()
-          }
-        } catch (err) {
-          console.error('[Display] Error closing previous WebSocket:', err)
-        }
-        this.ws = null
-      }
-
-      this.isConnecting = true
-      const protocol = location.protocol === 'https:' ? 'wss:' : 'ws:'
-
-      // Permette di specificare un host WebSocket alternativo via query parameter
-      // Utile per scenari WSL2 o development remoto: ?wsHost=192.168.1.100:5173
-      const params = new URLSearchParams(location.search)
-      const defaultHost = (
-        location.hostname === 'localhost' || location.hostname === '::1'
-      )
-        ? `127.0.0.1${location.port ? `:${location.port}` : ''}`
-        : location.host
-      const wsHost = params.get('wsHost') || defaultHost
-      const wsUrl = `${protocol}//${wsHost}/ws`
-
-      console.log('[Display] Connecting to WebSocket:', wsUrl)
-
-      try {
-        this.ws = new WebSocket(wsUrl)
-      } catch (err) {
-        console.error('[Display] Failed to create WebSocket:', err)
-        this.isConnecting = false
-        this.scheduleReconnect()
-        return
-      }
-
-      this.ws.onopen = () => {
-        this.isConnecting = false
-        this.wsConnected = true
-        this.reconnectAttempts = 0
-        console.log('[Display] Connected to server')
-
-        // Registra il client come display solo con connessione effettivamente aperta.
-        if (this.ws && this.ws.readyState === WebSocket.OPEN) {
-          try {
-            this.ws.send(JSON.stringify({ type: 'register', role: 'display' }))
-          } catch (err) {
-            console.error('[Display] Failed to register:', err)
-          }
-        }
-      }
-
-      this.ws.onmessage = (event) => {
-        try {
-          const msg = JSON.parse(event.data)
-
-          if (msg.type === 'state') {
-            this.state = msg.state
-          } else if (msg.type === 'speak') {
-            this.speakOnDisplay(msg.text)
-          } else if (msg.type === 'error') {
-            console.error('[Display] Server error:', msg.message)
-          }
-        } catch (e) {
-          console.error('[Display] Error parsing message:', e)
-        }
-      }
-
-      this.ws.onclose = (event) => {
-        this.isConnecting = false
-        this.wsConnected = false
-        console.log('[Display] Disconnected from server', event.code, event.reason)
-
-        // Non riconnette durante HMR (codice 1001, "going away")
-        // ne in caso di chiusura pulita (codice 1000).
-        if (event.code === 1000 || event.code === 1001) {
-          console.log('[Display] Clean close, not reconnecting')
-          return
-        }
-
-        this.scheduleReconnect()
-      }
-
-      this.ws.onerror = (err) => {
-        console.error('[Display] WebSocket error:', err)
-        this.isConnecting = false
-        this.wsConnected = false
-      }
-    },
-    scheduleReconnect() {
-      // Evita pianificazioni multiple della riconnessione.
-      if (this.reconnectTimeout) {
-        return
-      }
-
-      // Applica backoff esponenziale: 1s, 2s, 4s, 8s, 16s, fino a 30s.
-      const delay = Math.min(
-        1000 * Math.pow(2, this.reconnectAttempts),
-        this.maxReconnectDelay
-      )
-      this.reconnectAttempts++
-
-      console.log(`[Display] Reconnecting in ${delay}ms (attempt ${this.reconnectAttempts})`)
-
-      this.reconnectTimeout = setTimeout(() => {
-        this.reconnectTimeout = null
-        this.connectWebSocket()
-      }, delay)
-    },
     speakOnDisplay(text) {
-      if (typeof text !== 'string' || !text.trim()) {
+      if (typeof text !== 'string' || !text.trim() || !('speechSynthesis' in window)) return
-        return
-      }
-      if (!('speechSynthesis' in window)) {
-        console.warn('[Display] speechSynthesis not supported')
-        return
-      }
-
       const utterance = new SpeechSynthesisUtterance(text.trim())
       const voices = window.speechSynthesis.getVoices()
-      const preferredVoice = voices.find((v) => v.name === 'Google italiano')
+      utterance.voice = voices.find(v => v.name === 'Google italiano')
-        || voices.find((v) => v.lang && v.lang.toLowerCase().startsWith('it'))
+        || voices.find(v => v.lang?.toLowerCase().startsWith('it'))
-      if (preferredVoice) {
+        || null
-        utterance.voice = preferredVoice
-      }
       utterance.lang = 'it-IT'
-
       window.speechSynthesis.cancel()
       window.speechSynthesis.speak(utterance)
-    }
+    },
-  }
+  },
 }
 </script>
 

src/gameState.js

@@ -1,3 +1,21 @@
+export function punteggio(striscia) {
+    let home = 0, guest = 0
+    for (const c of striscia.at(-1).ris) c === 'h' ? home++ : guest++
+    return { home, guest }
+}
+
+export function servizio(striscia) {
+    const set = striscia.at(-1)
+    return set.ris.length === 0 ? set.serv === 'h' : set.ris.at(-1) === 'h'
+}
+
+export function setVinti(striscia) {
+    return {
+        home:  striscia.filter(s => s.vinc === 'h').length,
+        guest: striscia.filter(s => s.vinc === 'g').length,
+    }
+}
+
 export function createInitialState() {
     return {
         order: true,
@@ -5,10 +23,7 @@ export function createInitialState() {
         visuStriscia: true,
         modalitaPartita: "3/5",
         sp: {
-            striscia: [{ serv: 'home', r: [] }],
+            striscia: [{ serv: 'h', ris: '', vinc: null }],
-            servHome: true,
-            punt: { home: 0, guest: 0 },
-            set: { home: 0, guest: 0 },
             nomi: { home: "Antoniana", guest: "Guest" },
             form: {
                 home: ["1", "2", "3", "4", "5", "6"],
@@ -19,12 +34,9 @@ export function createInitialState() {
 }
 
 export function checkVittoria(state) {
-    const puntHome = state.sp.punt.home
+    const { home: puntHome, guest: puntGuest } = punteggio(state.sp.striscia)
-    const puntGuest = state.sp.punt.guest
+    const sv = setVinti(state.sp.striscia)
-    const setHome = state.sp.set.home
+    const totSet = sv.home + sv.guest
-    const setGuest = state.sp.set.guest
-    const totSet = setHome + setGuest
-
     const isSetDecisivo = state.modalitaPartita === "2/3" ? totSet >= 2 : totSet >= 4
     const punteggioVittoria = isSetDecisivo ? 15 : 25
 
@@ -33,6 +45,13 @@ export function checkVittoria(state) {
     return false
 }
 
+export function checkVittoriaPartita(state) {
+    if (state.modalitaPartita === 'amichevole') return false
+    const setsToWin = state.modalitaPartita === "2/3" ? 2 : 3
+    const sv = setVinti(state.sp.striscia)
+    return sv.home >= setsToWin || sv.guest >= setsToWin
+}
+
 export function applyAction(state, action) {
     const s = structuredClone(state)
 
@@ -41,32 +60,38 @@ export function applyAction(state, action) {
             const team = action.team
             if (checkVittoria(s)) break
 
-            const cambioPalla = (team === "home") !== s.sp.servHome
+            const servHome = servizio(s.sp.striscia)
-            s.sp.punt[team]++
+            const cambioPalla = (team === "home") !== servHome
-            s.sp.striscia.at(-1).r.push(team)
+            const setCorrente = s.sp.striscia.at(-1)
+            if (setCorrente.ris === '') {
+                setCorrente.formInizio = {
+                    home: [...s.sp.form.home],
+                    guest: [...s.sp.form.guest],
+                }
+            }
+            s.sp.striscia.at(-1).ris += team === 'home' ? 'h' : 'g'
 
             if (cambioPalla) {
                 s.sp.form[team].push(s.sp.form[team].shift())
             }
-
-            s.sp.servHome = team === "home"
             break
         }
 
         case "decPunt": {
             const currentSet = s.sp.striscia.at(-1)
-            if (currentSet.r.length === 0) break
+            if (currentSet.ris.length === 0) break
 
-            const lastScorer = currentSet.r[currentSet.r.length - 1]
+            const lastScorerShort = currentSet.ris.at(-1)
-            const prevServer = currentSet.r.length >= 2
+            const prevServerShort = currentSet.ris.length >= 2
-                ? currentSet.r[currentSet.r.length - 2]
+                ? currentSet.ris.at(-2)
                 : currentSet.serv
 
-            const wasCambioPalla = lastScorer !== prevServer
+            const wasCambioPalla = lastScorerShort !== prevServerShort
 
-            currentSet.r.pop()
+            currentSet.ris = currentSet.ris.slice(0, -1)
-            s.sp.punt[lastScorer]--
+            if (currentSet.ris === '') delete currentSet.formInizio
-            s.sp.servHome = prevServer === 'home'
+
+            const lastScorer = lastScorerShort === 'h' ? 'home' : 'guest'
 
             if (wasCambioPalla) {
                 s.sp.form[lastScorer].unshift(s.sp.form[lastScorer].pop())
@@ -76,10 +101,17 @@ export function applyAction(state, action) {
 
         case "incSet": {
             const team = action.team
-            if (s.sp.set[team] === 2) {
+            const sv = setVinti(s.sp.striscia)
-                s.sp.set[team] = 0
+            const count = sv[team]
+            const teamShort = team === 'home' ? 'h' : 'g'
+            if (count >= 2) {
+                // cicla a 0: rimuove le voci fantasma per questo team
+                s.sp.striscia = s.sp.striscia.filter(
+                    entry => !(entry._phantom && entry.vinc === teamShort)
+                )
             } else {
-                s.sp.set[team]++
+                // inserisce una voce fantasma prima dell'ultimo set (quello in corso)
+                s.sp.striscia.splice(-1, 0, { serv: teamShort, ris: '', vinc: teamShort, _phantom: true })
             }
             break
         }
@@ -87,11 +119,10 @@ export function applyAction(state, action) {
         case "nuovoSet": {
             const team = action.team
             if (team !== 'home' && team !== 'guest') break
-            s.sp.set[team]++
+            if (checkVittoriaPartita(s)) break
-            s.sp.punt.home = 0
+            s.sp.striscia.at(-1).vinc = team === 'home' ? 'h' : 'g'
-            s.sp.punt.guest = 0
+            if (checkVittoriaPartita(s)) break
-            s.sp.servHome = team === 'home'
+            s.sp.striscia.push({ serv: team === 'home' ? 'h' : 'g', ris: '', vinc: null })
-            s.sp.striscia.push({ serv: team, r: [] })
             s.sp.form = {
                 home: ["1", "2", "3", "4", "5", "6"],
                 guest: ["1", "2", "3", "4", "5", "6"],
@@ -100,24 +131,21 @@ export function applyAction(state, action) {
         }
 
         case "cambiaPalla": {
-            if (s.sp.punt.home === 0 && s.sp.punt.guest === 0) {
+            const currentSet = s.sp.striscia.at(-1)
-                s.sp.servHome = !s.sp.servHome
+            if (currentSet.ris.length === 0) {
-                s.sp.striscia.at(-1).serv = s.sp.servHome ? 'home' : 'guest'
+                currentSet.serv = currentSet.serv === 'h' ? 'g' : 'h'
             }
             break
         }
 
         case "resetta": {
             s.visuForm = false
-            s.sp.punt.home = 0
+            const servIniziale = s.sp.striscia[0]?.serv ?? 'h'
-            s.sp.punt.guest = 0
+            s.sp.striscia = [{ serv: servIniziale, ris: '', vinc: null }]
-            s.sp.set.home = 0
-            s.sp.set.guest = 0
             s.sp.form = {
                 home: ["1", "2", "3", "4", "5", "6"],
                 guest: ["1", "2", "3", "4", "5", "6"],
             }
-            s.sp.striscia = [{ serv: s.sp.servHome ? 'home' : 'guest', r: [] }]
             break
         }
 

src/referto.js

@@ -0,0 +1,141 @@
+function vincitoreSet(s) {
+    if (s.vinc === 'h' || s.vinc === 'g') return s.vinc
+    let h = 0, g = 0
+    for (const c of s.ris) c === 'h' ? h++ : g++
+    if (h > g) return 'h'
+    if (g > h) return 'g'
+    return null
+}
+
+// Costruisce l'HTML del referto (funzione pura, testabile).
+// `now` è iniettabile per rendere deterministica la data nei test.
+export function buildRefertoHtml(state, now = new Date()) {
+    const { sp, modalitaPartita } = state
+    const { nomi, striscia } = sp
+
+    const setReali = striscia.filter(s => !s._phantom)
+
+    const setVinti = { home: 0, guest: 0 }
+    for (const s of setReali) {
+        const v = vincitoreSet(s)
+        if (v === 'h') setVinti.home++
+        else if (v === 'g') setVinti.guest++
+    }
+
+    const dataOra = now.toLocaleString('it-IT', {
+        day: '2-digit', month: '2-digit', year: 'numeric',
+        hour: '2-digit', minute: '2-digit',
+    })
+
+    const giocatoreHtml = (n) => `<div class="giocatore">${n}</div>`
+
+    const formazioneHtml = (formazioneSet) => {
+        if (!formazioneSet) return '<em style="color:#999;font-size:11px">non disponibile</em>'
+        return `
+          <div class="form-row">
+            <div class="form-team">
+              <div class="form-team-name">${nomi.home}</div>
+              <div class="giocatori">${formazioneSet.home.map(giocatoreHtml).join('')}</div>
+            </div>
+            <div class="form-team">
+              <div class="form-team-name">${nomi.guest}</div>
+              <div class="giocatori">${formazioneSet.guest.map(giocatoreHtml).join('')}</div>
+            </div>
+          </div>`
+    }
+
+    const setsHtml = setReali.map((s, i) => {
+        let h = 0, g = 0
+        const punti = []
+        for (const c of s.ris) {
+            c === 'h' ? h++ : g++
+            punti.push({ chi: c, h, g })
+        }
+
+        const vinc = vincitoreSet(s)
+        const nomeVinc = vinc === 'h' ? nomi.home : vinc === 'g' ? nomi.guest : ''
+        const etichettaVinc = nomeVinc ? ` &nbsp;·&nbsp; vinto da <strong>${nomeVinc}</strong>` : ''
+
+        const puntiHtml = punti.map(p =>
+            `<span class="punto punto-${p.chi}">${p.h}-${p.g}</span>`
+        ).join('')
+
+        return `
+        <div class="set-section">
+          <div class="set-header">
+            Set ${i + 1} &nbsp;—&nbsp; ${nomi.home} <strong>${h}</strong> · <strong>${g}</strong> ${nomi.guest}${etichettaVinc}
+          </div>
+          <div class="form-inizio">
+            <div class="form-inizio-label">Formazione di partenza</div>
+            ${formazioneHtml(s.formInizio)}
+          </div>
+          <div class="punti-grid">${puntiHtml || '<em style="color:#999;font-size:11px">Nessun punto registrato</em>'}</div>
+        </div>`
+    }).join('')
+
+    const html = `<!DOCTYPE html>
+<html lang="it">
+<head>
+  <meta charset="UTF-8">
+  <title>Referto — ${nomi.home} vs ${nomi.guest}</title>
+  <style>
+    * { box-sizing: border-box; margin: 0; padding: 0; }
+    body { font-family: Arial, Helvetica, sans-serif; color: #111; background: #fff; padding: 28px; font-size: 14px; }
+    @media print { body { padding: 12px; } }
+
+    .header { text-align: center; border-bottom: 2px solid #111; padding-bottom: 12px; margin-bottom: 20px; }
+    .titolo { font-size: 18px; font-weight: bold; letter-spacing: 3px; text-transform: uppercase; }
+    .meta { font-size: 12px; color: #666; margin-top: 5px; }
+
+    .squadre-row { display: flex; justify-content: space-around; align-items: center; margin: 18px 0 6px; }
+    .nome-sq { font-size: 22px; font-weight: bold; text-align: center; max-width: 40%; }
+    .vs { font-size: 16px; color: #999; }
+    .risultato { text-align: center; font-size: 36px; font-weight: bold; letter-spacing: 6px; margin-bottom: 4px; }
+    .modalita-label { text-align: center; font-size: 12px; color: #888; margin-bottom: 22px; }
+
+    .set-section { margin-bottom: 12px; border: 1px solid #ddd; border-radius: 4px; overflow: hidden; }
+    .set-header { background: #f5f5f5; padding: 7px 12px; font-size: 13px; border-bottom: 1px solid #ddd; }
+    .punti-grid { display: flex; flex-wrap: wrap; gap: 3px; padding: 8px 10px; }
+    .punto { display: inline-block; padding: 2px 5px; border-radius: 3px; font-size: 11px; font-family: 'Courier New', monospace; white-space: nowrap; }
+    .punto-h { background: #d0e8ff; color: #003a6e; }
+    .punto-g { background: #ffddc0; color: #6e2700; }
+
+    .form-inizio { padding: 8px 10px; border-bottom: 1px solid #eee; background: #fafafa; }
+    .form-inizio-label { font-size: 11px; font-weight: bold; letter-spacing: 0.5px; text-transform: uppercase; color: #888; margin-bottom: 7px; }
+    .form-row { display: flex; gap: 40px; }
+    .form-team { flex: 1; }
+    .form-team-name { font-weight: bold; font-size: 13px; margin-bottom: 6px; }
+    .giocatori { display: flex; gap: 5px; flex-wrap: wrap; }
+    .giocatore { background: #f0f0f0; border-radius: 50%; width: 30px; height: 30px; display: flex; align-items: center; justify-content: center; font-weight: bold; font-size: 12px; border: 1px solid #ccc; }
+  </style>
+</head>
+<body>
+  <div class="header">
+    <div class="titolo">Referto di Gara</div>
+    <div class="meta">${dataOra} &nbsp;·&nbsp; Modalità: ${modalitaPartita}</div>
+  </div>
+
+  <div class="squadre-row">
+    <div class="nome-sq">${nomi.home}</div>
+    <div class="vs">vs</div>
+    <div class="nome-sq">${nomi.guest}</div>
+  </div>
+  <div class="risultato">${setVinti.home} – ${setVinti.guest}</div>
+  <div class="modalita-label">set vinti</div>
+
+  ${setsHtml}
+</body>
+</html>`
+
+    return html
+}
+
+// Apre il referto in una nuova scheda e avvia la stampa (effetto collaterale,
+// solo browser). La generazione dell'HTML è delegata a buildRefertoHtml.
+export function generaReferto(state) {
+    const html = buildRefertoHtml(state)
+    const w = window.open('', '_blank')
+    w.document.write(html)
+    w.document.close()
+    w.print()
+}

src/server-utils.js

@@ -1,34 +1,68 @@
 import { networkInterfaces } from 'os'
+import { readFileSync, existsSync } from 'fs'
+import { execSync } from 'child_process'
 
-export function getNetworkIPs() {
+function isWSL() {
-  const nets = networkInterfaces()
+  try {
-  const networkIPs = []
+    return existsSync('/proc/sys/kernel/osrelease') &&
-
+      readFileSync('/proc/sys/kernel/osrelease', 'utf8').toLowerCase().includes('microsoft')
-  for (const name of Object.keys(nets)) {
+  } catch { return false }
-    for (const net of nets[name]) {
-      if (net.family === 'IPv4' &&
-        !net.internal &&
-        !net.address.startsWith('172.17.') &&
-        !net.address.startsWith('172.18.')) {
-        networkIPs.push(net.address)
-      }
-    }
-  }
-
-  return networkIPs
 }
 
-export function printServerInfo(port = 3000) {
+// Un IPv4 è "pubblicabile" in LAN se non è loopback, link-local o bridge Docker.
-  const networkIPs = getNetworkIPs()
+function isLanIPv4(address) {
+  return !!address
+    && !address.startsWith('127.')
+    && !address.startsWith('169.254.')
+    && !address.startsWith('172.')
+}
 
+// Estrae gli IP LAN da un oggetto in stile os.networkInterfaces().
+// Esportata per poter essere testata in isolamento, senza dipendere dalla piattaforma.
+export function collectIPs(nets) {
+  const out = []
+  for (const name of Object.keys(nets || {})) {
+    for (const net of nets[name]) {
+      if (net.family === 'IPv4' && !net.internal && isLanIPv4(net.address)) {
+        out.push(net.address)
+      }
+    }
+  }
+  return out
+}
+
+// Restituisce gli IP di rete della macchina.
+// Passando `nets` (oggetto in stile os.networkInterfaces) si forza il percorso
+// deterministico, scavalcando il rilevamento WSL/PowerShell: utile nei test.
+export function getNetworkIPs(nets) {
+  if (nets) return collectIPs(nets)
+
+  if (isWSL()) {
+    try {
+      const out = execSync(
+        'powershell.exe -NoProfile -Command "Get-NetIPAddress -AddressFamily IPv4 | Select-Object -ExpandProperty IPAddress"',
+        { timeout: 3000 }
+      )
+      return out.toString().trim().split('\n')
+        .map(s => s.trim())
+        .filter(isLanIPv4)
+    } catch { return [] }
+  }
+
+  return collectIPs(networkInterfaces())
+}
+
+// `networkIPs` è iniettabile per rendere la stampa testabile in modo deterministico.
+export function printServerInfo(port = 3000, networkIPs = getNetworkIPs()) {
   console.log(`\nSegnapunti Server`)
   console.log(`   Display:    http://127.0.0.1:${port}/display`)
   console.log(`   Controller: http://127.0.0.1:${port}/controller`)
 
   if (networkIPs.length > 0) {
-    console.log(`\n   Controller da dispositivi remoti:`)
+    console.log(`\n   Da dispositivi remoti:`)
     networkIPs.forEach(ip => {
-      console.log(`   http://${ip}:${port}/controller`)
+      console.log(`   Display:    http://${ip}:${port}/display`)
+      console.log(`   Controller: http://${ip}:${port}/controller`)
     })
   }
 

src/wsMixin.js

@@ -0,0 +1,146 @@
+import { createInitialState, punteggio, servizio, setVinti } from './gameState.js'
+
+export function createWsMixin(role) {
+    return {
+        data() {
+            return {
+                ws: null,
+                wsConnected: false,
+                isConnecting: false,
+                reconnectTimeout: null,
+                reconnectAttempts: 0,
+                maxReconnectDelay: 30000,
+                state: createInitialState(),
+            }
+        },
+        computed: {
+            punt()     { return punteggio(this.state.sp.striscia) },
+            servHome() { return servizio(this.state.sp.striscia) },
+            set()      { return setVinti(this.state.sp.striscia) },
+        },
+        mounted() {
+            this.connectWebSocket()
+            if (import.meta.hot) {
+                import.meta.hot.on('vite:beforeUpdate', () => {
+                    if (this.reconnectTimeout) {
+                        clearTimeout(this.reconnectTimeout)
+                        this.reconnectTimeout = null
+                    }
+                })
+            }
+        },
+        beforeUnmount() {
+            if (this.reconnectTimeout) {
+                clearTimeout(this.reconnectTimeout)
+                this.reconnectTimeout = null
+            }
+            if (this.ws) {
+                this.ws.onclose = null
+                this.ws.onerror = null
+                this.ws.onmessage = null
+                this.ws.onopen = null
+                try {
+                    if (this.ws.readyState === WebSocket.OPEN) {
+                        this.ws.close(1000, 'Component unmounting')
+                    } else if (this.ws.readyState === WebSocket.CONNECTING) {
+                        this.ws.close()
+                    }
+                } catch (err) {
+                    console.error(`[${role}] Error closing WebSocket:`, err)
+                }
+                this.ws = null
+            }
+        },
+        methods: {
+            connectWebSocket() {
+                if (this.isConnecting) return
+                if (this.ws) {
+                    this.ws.onclose = null
+                    this.ws.onerror = null
+                    this.ws.onmessage = null
+                    this.ws.onopen = null
+                    try {
+                        if (this.ws.readyState === WebSocket.OPEN) {
+                            this.ws.close(1000, 'Reconnecting')
+                        } else if (this.ws.readyState === WebSocket.CONNECTING) {
+                            this.ws.close()
+                        }
+                    } catch (err) {
+                        console.error(`[${role}] Error closing previous WebSocket:`, err)
+                    }
+                    this.ws = null
+                }
+
+                this.isConnecting = true
+                const protocol = location.protocol === 'https:' ? 'wss:' : 'ws:'
+                const params = new URLSearchParams(location.search)
+                const defaultHost = (location.hostname === 'localhost' || location.hostname === '::1')
+                    ? `127.0.0.1${location.port ? `:${location.port}` : ''}`
+                    : location.host
+                const wsUrl = `${protocol}//${params.get('wsHost') || defaultHost}/ws`
+
+                try {
+                    this.ws = new WebSocket(wsUrl)
+                } catch (err) {
+                    console.error(`[${role}] Failed to create WebSocket:`, err)
+                    this.isConnecting = false
+                    this.scheduleReconnect()
+                    return
+                }
+
+                this.ws.onopen = () => {
+                    this.isConnecting = false
+                    this.wsConnected = true
+                    this.reconnectAttempts = 0
+                    try {
+                        this.ws?.send(JSON.stringify({ type: 'register', role }))
+                    } catch (err) {
+                        console.error(`[${role}] Failed to register:`, err)
+                    }
+                }
+
+                this.ws.onmessage = (event) => {
+                    try {
+                        const msg = JSON.parse(event.data)
+                        if (msg.type === 'state') this.state = msg.state
+                        else this.onWsMessage?.(msg)
+                    } catch (e) {
+                        console.error(`[${role}] Error parsing message:`, e)
+                    }
+                }
+
+                this.ws.onclose = (event) => {
+                    this.isConnecting = false
+                    this.wsConnected = false
+                    if (event.code !== 1000 && event.code !== 1001) this.scheduleReconnect()
+                }
+
+                this.ws.onerror = () => {
+                    this.isConnecting = false
+                    this.wsConnected = false
+                }
+            },
+
+            scheduleReconnect() {
+                if (this.reconnectTimeout) return
+                const delay = Math.min(1000 * Math.pow(2, this.reconnectAttempts), this.maxReconnectDelay)
+                this.reconnectAttempts++
+                this.reconnectTimeout = setTimeout(() => {
+                    this.reconnectTimeout = null
+                    this.connectWebSocket()
+                }, delay)
+            },
+
+            sendWs(msg) {
+                if (!this.wsConnected || this.ws?.readyState !== WebSocket.OPEN) return false
+                try {
+                    this.ws.send(JSON.stringify(msg))
+                    return true
+                } catch (err) {
+                    console.error(`[${role}] Failed to send:`, err)
+                    return false
+                }
+            },
+        },
+    }
+}

temp.md

@@ -0,0 +1,108 @@
+# TODO — completamento suite di test
+
+Checkpoint: branch `test-suite-repair`, commit `bf9a6f8`.
+
+## Contesto
+
+La suite era allineata a una **vecchia forma dello stato** (`sp.punt` / `sp.set` /
+`sp.servHome`) e a una **vecchia architettura e2e** (controller su `:3001`).
+Baseline iniziale: **77/170 test Vitest falliti**, **tutti gli e2e rotti**.
+
+Nel commit `bf9a6f8`:
+- Vitest riportato a **212/212 verde** (verificato con `npx vitest run`).
+- e2e migrati **parzialmente** (porte + viewport + reset→config), **NON verificati verdi**.
+
+---
+
+## ✅ Fatto (verificato)
+
+- `tests/unit/gameState.test.js` — riscritto con helper che derivano
+  punteggio/set/servizio dalla striscia; aggiunto blocco `formInizio`.
+- `src/server-utils.js` — `getNetworkIPs(nets)` con interfacce iniettabili +
+  `printServerInfo(port, ips)` con IP iniettabili (deterministico anche su WSL);
+  filtro LAN unificato (esclude `127.` / `169.254.` / `172.`). Test riscritto.
+- `tests/integration/websocket.test.js` + `tests/stress/websocket-load.test.js`
+  — punteggio letto via `punteggio(striscia)`.
+- `tests/component/ControllerPage.test.js` + `DisplayPage.test.js` — forzato
+  layout mobile (viewport portrait), punteggi via striscia; aggiunto test REFERTO.
+- Nuovi unit/integration: `referto.test.js`, `persist.test.js` (mock `fs`),
+  `wsMixin.test.js`, `integration/server.test.js` (routing).
+- `src/referto.js` — estratta `buildRefertoHtml(state, now)` pura; `generaReferto`
+  resta wrapper con `window.open`/`print`.
+- `server.js` — estratti `createApp()` / `startServer()`; avvio solo se entrypoint.
+
+---
+
+## ⚠️ Da finire / verificare — e2e Playwright
+
+Gli e2e NON sono stati eseguiti fino a verde (run lenti). Punto di ripartenza.
+
+### Fix già applicati nei 6 spec
+1. `:3001` → `:3000/controller`.
+2. `setViewportSize({ width: 390, height: 844 })` prima di ogni `goto` controller
+   (su desktop renderizza la dashboard landscape `.e-dash`, ma i test cercano il
+   markup mobile `.team-score` / `.team-pts` / `.btn-set`).
+3. Il reset ora chiude il dialog di configurazione che `doReset()` apre in automatico.
+4. `game-simulation`: a 25 gestito il modal automatico "SET VINTO" con
+   "VAI AL SET SUCCESSIVO".
+
+### Da verificare / probabili fix residui
+
+- [ ] **`full-match.spec.cjs`** — RISCHIO ALTO. I test arrivano a 25/15 → scatta
+      il modal automatico (`squadraVincente`). Le asserzioni che cliccano
+      `.btn-set` o `.team-score` DOPO i 25 punti vengono bloccate dall'overlay del
+      modal. Da rivedere il flusso (usare i bottoni del modal).
+- [ ] **`game-simulation.spec.cjs`** — verificare il nuovo finale col modal.
+- [ ] **`game-operations.spec.cjs`** — verificare flusso cambi/toggle/config dopo
+      `resetGame` (config ora chiuso).
+- [ ] **`basic-flow.spec.cjs`** — probabile OK, confermare.
+- [ ] **`accessibility.spec.cjs`** — rieseguire axe sul markup attuale.
+- [ ] **`visual-regression.spec.cjs`** — FALLIRÀ: snapshot baseline della vecchia
+      UI. Rigenerare con `npm run test:e2e -- --update-snapshots` DOPO aver
+      sistemato il resto.
+
+### Note importanti
+- Stato e2e **condiviso e persistente** (`.segnapunti/state.json`): i test
+  dipendono dall'ordine e dal reset. Partire da stato pulito.
+- I 3 progetti Playwright (chromium, firefox, Mobile Chrome) girano in serie
+  (`workers: 1`).
+
+---
+
+## ⬜ Non ancora iniziato
+
+- [ ] **`tests/e2e/referto.spec.cjs`** (nuovo) — portare una partita 2/3 a fine
+      match, intercettare il popup con `page.waitForEvent('popup')` dopo il click
+      su REFERTO, verificare nomi squadre + punteggi. Stubbare `window.print`
+      (via `addInitScript`) perché può bloccare.
+- [ ] **Documentazione**:
+  - `README.md` — sezione Controller: aggiungere il referto (PARTITA FINITA →
+    referto stampabile/PDF, prototipo). Sezione Test: aggiungere `test:ui`.
+  - `CLAUDE.md` — aggiungere `src/referto.js` (`buildRefertoHtml` + `generaReferto`),
+    menzionare `wsMixin.js` / `persist.js` / `server-utils.js`; nota su `formInizio`.
+  - `tests/README.md` — correggere i conteggi obsoleti ("159 passed (159)",
+    "6 files"); aggiungere referto / persist / wsMixin / routing server / spec
+    referto.
+
+---
+
+## Comandi utili
+
+```bash
+# Vitest (deve restare verde: 212/212)
+npx vitest run
+
+# e2e — partire da stato pulito + server attivo
+rm -f .segnapunti/state.json
+npm run serve            # in un terminale a parte
+
+# e2e mirati (chromium, dai più rischiosi)
+npx playwright test --config=playwright.config.cjs --project=chromium tests/e2e/full-match.spec.cjs
+npx playwright test --config=playwright.config.cjs --project=chromium tests/e2e/game-simulation.spec.cjs
+
+# rigenerare gli snapshot visual DOPO aver sistemato la UI dei test
+npm run test:e2e -- --update-snapshots
+
+# suite e2e completa (3 browser, lenta)
+npm run test:e2e
+```

tests/component/ControllerPage.test.js

@@ -2,6 +2,10 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
 import { mount } from '@vue/test-utils'
 import ControllerPage from '../../src/components/ControllerPage.vue'
+import { generaReferto } from '../../src/referto.js'
+
+// Il referto apre una finestra/print: lo mockiamo per testarne solo l'invocazione.
+vi.mock('../../src/referto.js', () => ({ generaReferto: vi.fn() }))
 
 // Mock globale WebSocket per jsdom
 class MockWebSocket {
@@ -25,6 +29,21 @@ class MockWebSocket {
 
 vi.stubGlobal('WebSocket', MockWebSocket)
 
+// Forza l'orientamento portrait → il controller usa il layout "mobile"
+// (con .team-pts, .btn-ctrl, ecc.) su cui questi test fanno asserzioni.
+Object.defineProperty(window, 'innerWidth', { value: 400, writable: true, configurable: true })
+Object.defineProperty(window, 'innerHeight', { value: 800, writable: true, configurable: true })
+
+// Imposta il punteggio del set in corso costruendo una ris coerente.
+// `serv` ('h'|'g') controlla l'ultimo punto, quindi chi risulta al servizio.
+function setScore(wrapper, home, guest, serv = 'h') {
+    const altro = serv === 'h' ? 'g' : 'h'
+    const nAltro = serv === 'h' ? guest : home
+    const nServ = serv === 'h' ? home : guest
+    // mette per ultimo il carattere del battitore desiderato
+    wrapper.vm.state.sp.striscia.at(-1).ris = altro.repeat(nAltro) + serv.repeat(nServ)
+}
+
 // Helper per creare il componente con stato personalizzato
 function mountController(stateOverrides = {}) {
     const wrapper = mount(ControllerPage, {
@@ -105,7 +124,7 @@ describe('ControllerPage.vue', () => {
 
         it('dovrebbe essere disabilitato se il punteggio non è 0-0', async () => {
             const wrapper = mountController()
-            wrapper.vm.state.sp.punt.home = 5
+            setScore(wrapper, 5, 0)
             await wrapper.vm.$nextTick()
             const btn = wrapper.findAll('.btn-ctrl').find(b => b.text().includes('Cambio Palla'))
             expect(btn.attributes('disabled')).toBeDefined()
@@ -194,8 +213,7 @@ describe('ControllerPage.vue', () => {
 
         it('dovrebbe generare "N pari" a punteggio uguale', () => {
             const wrapper = mountController()
-            wrapper.vm.state.sp.punt.home = 5
+            setScore(wrapper, 5, 5)
-            wrapper.vm.state.sp.punt.guest = 5
             wrapper.vm.wsConnected = true
             wrapper.vm.ws = { readyState: 1, send: vi.fn() }
             wrapper.vm.speak()
@@ -205,9 +223,7 @@ describe('ControllerPage.vue', () => {
 
         it('dovrebbe annunciare prima il punteggio di chi batte (home serve)', () => {
             const wrapper = mountController()
-            wrapper.vm.state.sp.punt.home = 15
+            setScore(wrapper, 15, 10, 'h')
-            wrapper.vm.state.sp.punt.guest = 10
-            wrapper.vm.state.sp.servHome = true
             wrapper.vm.wsConnected = true
             wrapper.vm.ws = { readyState: 1, send: vi.fn() }
             wrapper.vm.speak()
@@ -217,9 +233,7 @@ describe('ControllerPage.vue', () => {
 
         it('dovrebbe annunciare prima il punteggio di chi batte (guest serve)', () => {
             const wrapper = mountController()
-            wrapper.vm.state.sp.punt.home = 10
+            setScore(wrapper, 10, 15, 'g')
-            wrapper.vm.state.sp.punt.guest = 15
-            wrapper.vm.state.sp.servHome = false
             wrapper.vm.wsConnected = true
             wrapper.vm.ws = { readyState: 1, send: vi.fn() }
             wrapper.vm.speak()
@@ -228,6 +242,51 @@ describe('ControllerPage.vue', () => {
         })
     })
 
+    // =============================================
+    // REFERTO (modal PARTITA FINITA)
+    // =============================================
+    describe('Referto', () => {
+        // Porta il componente allo stato "partita finita" per home in 2/3
+        function setPartitaFinita(wrapper) {
+            wrapper.vm.state.modalitaPartita = '2/3'
+            wrapper.vm.state.sp.striscia = [
+                { serv: 'h', ris: '', vinc: 'h' },
+                { serv: 'h', ris: '', vinc: null },
+            ]
+            wrapper.vm.setVintoTeam = 'home'
+            wrapper.vm.showSetVinto = true
+        }
+
+        it('mostra il bottone REFERTO quando la partita è finita', async () => {
+            const wrapper = mountController()
+            setPartitaFinita(wrapper)
+            await wrapper.vm.$nextTick()
+            expect(wrapper.vm.isPartitaFinita).toBe(true)
+            const btn = wrapper.findAll('.btn-secondary').find(b => b.text().includes('REFERTO'))
+            expect(btn).toBeDefined()
+        })
+
+        it('il click su REFERTO invoca generaReferto con lo stato', async () => {
+            const wrapper = mountController()
+            setPartitaFinita(wrapper)
+            await wrapper.vm.$nextTick()
+            const btn = wrapper.findAll('.btn-secondary').find(b => b.text().includes('REFERTO'))
+            await btn.trigger('click')
+            expect(generaReferto).toHaveBeenCalledWith(wrapper.vm.state)
+        })
+
+        it('NON mostra il bottone REFERTO a set vinto (partita non finita)', async () => {
+            const wrapper = mountController()
+            wrapper.vm.state.modalitaPartita = '3/5'
+            wrapper.vm.setVintoTeam = 'home'
+            wrapper.vm.showSetVinto = true
+            await wrapper.vm.$nextTick()
+            expect(wrapper.vm.isPartitaFinita).toBe(false)
+            const btn = wrapper.findAll('.btn-secondary').find(b => b.text().includes('REFERTO'))
+            expect(btn).toBeUndefined()
+        })
+    })
+
     // =============================================
     // BARRA CONNESSIONE
     // =============================================

tests/component/DisplayPage.test.js

@@ -75,8 +75,8 @@ describe('DisplayPage.vue', () => {
 
         it('dovrebbe aggiornare il punteggio quando lo stato cambia', async () => {
             const wrapper = mountDisplay()
-            wrapper.vm.state.sp.punt.home = 15
+            // il punteggio si ricava dalla striscia: 15 punti home + 12 guest
-            wrapper.vm.state.sp.punt.guest = 12
+            wrapper.vm.state.sp.striscia.at(-1).ris = 'h'.repeat(15) + 'g'.repeat(12)
             await wrapper.vm.$nextTick()
             const punti = wrapper.findAll('.punt')
             expect(punti[0].text()).toBe('15')

tests/component/wsMixin.test.js

@@ -0,0 +1,174 @@
+// @vitest-environment happy-dom
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { mount } from '@vue/test-utils'
+import { createWsMixin } from '../../src/wsMixin.js'
+import { createInitialState } from '../../src/gameState.js'
+
+// WebSocket mock controllabile: i gestori (onopen/onmessage/onclose) vengono
+// assegnati dal mixin e li invochiamo manualmente nei test.
+class MockWebSocket {
+    static CONNECTING = 0
+    static OPEN = 1
+    static CLOSING = 2
+    static CLOSED = 3
+    constructor(url) {
+        this.url = url
+        this.readyState = MockWebSocket.CONNECTING
+        this.send = vi.fn()
+        this.close = vi.fn()
+        MockWebSocket.instances.push(this)
+    }
+}
+MockWebSocket.instances = []
+
+vi.stubGlobal('WebSocket', MockWebSocket)
+
+// Monta un componente che usa il mixin. `extra` permette di aggiungere hook.
+function mountWith(role = 'controller', extra = {}) {
+    const Comp = {
+        mixins: [createWsMixin(role)],
+        template: '<div></div>',
+        ...extra,
+    }
+    return mount(Comp)
+}
+
+function ultimaWs() {
+    return MockWebSocket.instances.at(-1)
+}
+
+describe('createWsMixin (wsMixin.js)', () => {
+    beforeEach(() => {
+        MockWebSocket.instances = []
+    })
+
+    afterEach(() => {
+        vi.restoreAllMocks()
+    })
+
+    describe('computed derivati', () => {
+        it('punt/servHome/set delegano alle funzioni pure sulla striscia', async () => {
+            const wrapper = mountWith()
+            wrapper.vm.state.sp.striscia = [
+                { serv: 'h', ris: 'h', vinc: 'h' },
+                { serv: 'h', ris: 'h'.repeat(10) + 'g'.repeat(8), vinc: null },
+            ]
+            await wrapper.vm.$nextTick()
+            expect(wrapper.vm.punt).toEqual({ home: 10, guest: 8 })
+            expect(wrapper.vm.set).toEqual({ home: 1, guest: 0 })
+            // ultimo punto 'g' → serve guest
+            expect(wrapper.vm.servHome).toBe(false)
+        })
+    })
+
+    describe('connessione', () => {
+        it('apre una WebSocket verso /ws al mount', () => {
+            mountWith()
+            const ws = ultimaWs()
+            expect(ws).toBeDefined()
+            expect(ws.url).toMatch(/^ws:\/\/.+\/ws$/)
+        })
+
+        it('invia il messaggio register all\'apertura', () => {
+            const wrapper = mountWith('controller')
+            const ws = ultimaWs()
+            ws.readyState = MockWebSocket.OPEN
+            ws.onopen()
+            expect(ws.send).toHaveBeenCalledTimes(1)
+            const msg = JSON.parse(ws.send.mock.calls[0][0])
+            expect(msg).toEqual({ type: 'register', role: 'controller' })
+            expect(wrapper.vm.wsConnected).toBe(true)
+        })
+
+        it('un messaggio "state" aggiorna lo stato locale', () => {
+            const wrapper = mountWith()
+            const ws = ultimaWs()
+            const nuovo = createInitialState()
+            nuovo.sp.nomi.home = 'Nuova Squadra'
+            ws.onmessage({ data: JSON.stringify({ type: 'state', state: nuovo }) })
+            expect(wrapper.vm.state.sp.nomi.home).toBe('Nuova Squadra')
+        })
+
+        it('un messaggio non-state invoca l\'hook onWsMessage', () => {
+            const onWsMessage = vi.fn()
+            const wrapper = mountWith('display', { methods: { onWsMessage } })
+            const ws = ultimaWs()
+            ws.onmessage({ data: JSON.stringify({ type: 'speak', text: 'ciao' }) })
+            expect(onWsMessage).toHaveBeenCalledWith({ type: 'speak', text: 'ciao' })
+        })
+    })
+
+    describe('riconnessione', () => {
+        it('scheduleReconnect usa backoff esponenziale con cap a 30s', () => {
+            const wrapper = mountWith()
+            const delays = []
+            const spy = vi.spyOn(globalThis, 'setTimeout').mockImplementation(() => 123)
+            // intercetta i delay leggendoli dalle chiamate
+            spy.mockImplementation((_fn, d) => { delays.push(d); return 123 })
+
+            wrapper.vm.reconnectAttempts = 0
+            wrapper.vm.reconnectTimeout = null
+            wrapper.vm.scheduleReconnect() // 1000
+            wrapper.vm.reconnectTimeout = null
+            wrapper.vm.scheduleReconnect() // 2000
+            wrapper.vm.reconnectTimeout = null
+            wrapper.vm.scheduleReconnect() // 4000
+            expect(delays).toEqual([1000, 2000, 4000])
+
+            // attempts alto → cap a 30000
+            delays.length = 0
+            wrapper.vm.reconnectAttempts = 20
+            wrapper.vm.reconnectTimeout = null
+            wrapper.vm.scheduleReconnect()
+            expect(delays[0]).toBe(30000)
+
+            spy.mockRestore()
+        })
+
+        it('non riconnette su chiusura pulita (1000/1001)', () => {
+            const wrapper = mountWith()
+            const ws = ultimaWs()
+            const spy = vi.spyOn(wrapper.vm, 'scheduleReconnect')
+            ws.onclose({ code: 1000 })
+            ws.onclose({ code: 1001 })
+            expect(spy).not.toHaveBeenCalled()
+            expect(wrapper.vm.wsConnected).toBe(false)
+        })
+
+        it('riconnette su chiusura anomala (es. 1006)', () => {
+            const wrapper = mountWith()
+            const ws = ultimaWs()
+            const spy = vi.spyOn(wrapper.vm, 'scheduleReconnect')
+            ws.onclose({ code: 1006 })
+            expect(spy).toHaveBeenCalled()
+        })
+    })
+
+    describe('sendWs', () => {
+        it('ritorna false se non connesso', () => {
+            const wrapper = mountWith()
+            wrapper.vm.wsConnected = false
+            expect(wrapper.vm.sendWs({ type: 'action' })).toBe(false)
+        })
+
+        it('serializza e invia se connesso e aperto', () => {
+            const wrapper = mountWith()
+            const ws = ultimaWs()
+            ws.readyState = MockWebSocket.OPEN
+            wrapper.vm.wsConnected = true
+            const ok = wrapper.vm.sendWs({ type: 'action', action: { type: 'incPunt' } })
+            expect(ok).toBe(true)
+            const inviato = JSON.parse(ws.send.mock.calls.at(-1)[0])
+            expect(inviato.type).toBe('action')
+        })
+    })
+
+    describe('cleanup', () => {
+        it('beforeUnmount chiude la WebSocket', () => {
+            const wrapper = mountWith()
+            const ws = ultimaWs()
+            wrapper.unmount()
+            expect(ws.close).toHaveBeenCalled()
+        })
+    })
+})

tests/e2e/accessibility.spec.cjs

@@ -17,7 +17,8 @@ test.describe('Accessibility (a11y)', () => {
     });
 
     test('Controller: non dovrebbe avere violazioni critiche a11y', async ({ page }) => {
-        await page.goto('http://localhost:3001');
+        await page.setViewportSize({ width: 390, height: 844 });
+        await page.goto('http://localhost:3000/controller');
         await page.waitForTimeout(500);
 
         const results = await new AxeBuilder({ page })
@@ -42,7 +43,8 @@ test.describe('Accessibility (a11y)', () => {
     });
 
     test('Controller: i touch target dovrebbero avere dimensione minima', async ({ page }) => {
-        await page.goto('http://localhost:3001');
+        await page.setViewportSize({ width: 390, height: 844 });
+        await page.goto('http://localhost:3000/controller');
         await page.waitForSelector('.conn-bar.connected');
 
         // Controlla che i bottoni principali abbiano dimensione minima 44x44px
@@ -57,7 +59,8 @@ test.describe('Accessibility (a11y)', () => {
     });
 
     test('Controller: i bottoni punteggio dovrebbero avere dimensione adeguata', async ({ page }) => {
-        await page.goto('http://localhost:3001');
+        await page.setViewportSize({ width: 390, height: 844 });
+        await page.goto('http://localhost:3000/controller');
         await page.waitForSelector('.conn-bar.connected');
 
         const scoreButtons = page.locator('.team-score');

tests/e2e/basic-flow.spec.cjs

@@ -7,7 +7,8 @@ test.describe('Basic Flow: Controller ↔ Display', () => {
         const controllerPage = await context.newPage();
 
         await displayPage.goto('http://localhost:3000');
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
 
         await expect(displayPage).toHaveTitle(/Segnapunti/);
         await expect(controllerPage).toHaveTitle(/Controller/);
@@ -15,7 +16,8 @@ test.describe('Basic Flow: Controller ↔ Display', () => {
 
     test('il punteggio iniziale dovrebbe essere 0-0', async ({ context }) => {
         const controllerPage = await context.newPage();
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
 
         // Attende la connessione WebSocket
         await controllerPage.waitForSelector('.conn-bar.connected');
@@ -31,7 +33,8 @@ test.describe('Basic Flow: Controller ↔ Display', () => {
         const controllerPage = await context.newPage();
 
         await displayPage.goto('http://localhost:3000');
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
 
         // Attende la connessione WebSocket del controller
         await controllerPage.waitForSelector('.conn-bar.connected');
@@ -42,6 +45,11 @@ test.describe('Basic Flow: Controller ↔ Display', () => {
         if (await btnConfirm.isVisible()) {
             await btnConfirm.click();
         }
+        // doReset apre automaticamente il dialog di configurazione: chiudilo
+        const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel');
+        if (await cfgCancel.isVisible()) {
+            await cfgCancel.click();
+        }
         await controllerPage.waitForTimeout(200);
 
         // Click +1 Home
@@ -60,7 +68,8 @@ test.describe('Basic Flow: Controller ↔ Display', () => {
         const controllerPage = await context.newPage();
 
         await displayPage.goto('http://localhost:3000');
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
 
         await controllerPage.waitForSelector('.conn-bar.connected');
 
@@ -70,6 +79,11 @@ test.describe('Basic Flow: Controller ↔ Display', () => {
         if (await btnConfirm.isVisible()) {
             await btnConfirm.click();
         }
+        // doReset apre automaticamente il dialog di configurazione: chiudilo
+        const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel');
+        if (await cfgCancel.isVisible()) {
+            await cfgCancel.click();
+        }
         await controllerPage.waitForTimeout(200);
 
         // Click +1 Guest
@@ -88,7 +102,8 @@ test.describe('Basic Flow: Controller ↔ Display', () => {
         const controllerPage = await context.newPage();
 
         await displayPage.goto('http://localhost:3000');
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
 
         await controllerPage.waitForSelector('.conn-bar.connected');
 
@@ -98,6 +113,11 @@ test.describe('Basic Flow: Controller ↔ Display', () => {
         if (await btnConfirm.isVisible()) {
             await btnConfirm.click();
         }
+        // doReset apre automaticamente il dialog di configurazione: chiudilo
+        const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel');
+        if (await cfgCancel.isVisible()) {
+            await cfgCancel.click();
+        }
         await controllerPage.waitForTimeout(200);
 
         // Home +1, Guest +1, Home +1

tests/e2e/full-match.spec.cjs

@@ -7,6 +7,11 @@ async function resetGame(controllerPage) {
     if (await btnConfirm.isVisible()) {
         await btnConfirm.click();
     }
+    // doReset apre automaticamente il dialog di configurazione: chiudilo
+    const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel');
+    if (await cfgCancel.isVisible()) {
+        await cfgCancel.click();
+    }
     await controllerPage.waitForTimeout(300);
 }
 
@@ -36,7 +41,8 @@ test.describe('Full Match Simulation', () => {
 
     test('Partita 2/3: Home vince 2 set a 0', async ({ context }) => {
         const controllerPage = await context.newPage();
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         await resetGame(controllerPage);
@@ -64,7 +70,8 @@ test.describe('Full Match Simulation', () => {
 
     test('Set decisivo 2/3: vittoria a 15 punti', async ({ context }) => {
         const controllerPage = await context.newPage();
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         await resetGame(controllerPage);
@@ -101,7 +108,8 @@ test.describe('Full Match Simulation', () => {
 
     test('Set normale: punti oltre 25 fino ai vantaggi', async ({ context }) => {
         const controllerPage = await context.newPage();
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         await resetGame(controllerPage);

tests/e2e/game-operations.spec.cjs

@@ -7,6 +7,11 @@ async function resetGame(controllerPage) {
     if (await btnConfirm.isVisible()) {
         await btnConfirm.click();
     }
+    // doReset apre automaticamente il dialog di configurazione: chiudilo
+    const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel');
+    if (await cfgCancel.isVisible()) {
+        await cfgCancel.click();
+    }
     await controllerPage.waitForTimeout(300);
 }
 
@@ -14,7 +19,8 @@ test.describe('Game Operations', () => {
 
     test('Undo: dovrebbe annullare l\'ultimo punto', async ({ context }) => {
         const controllerPage = await context.newPage();
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         await resetGame(controllerPage);
@@ -32,7 +38,8 @@ test.describe('Game Operations', () => {
 
     test('Reset: dovrebbe azzerare tutto dopo conferma', async ({ context }) => {
         const controllerPage = await context.newPage();
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         // Imposta qualche punto
@@ -54,7 +61,8 @@ test.describe('Game Operations', () => {
         const controllerPage = await context.newPage();
 
         await displayPage.goto('http://localhost:3000');
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         // Apri config
@@ -84,7 +92,8 @@ test.describe('Game Operations', () => {
         const controllerPage = await context.newPage();
 
         await displayPage.goto('http://localhost:3000');
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         // Inizialmente mostra punteggio, non formazione
@@ -103,7 +112,8 @@ test.describe('Game Operations', () => {
         const controllerPage = await context.newPage();
 
         await displayPage.goto('http://localhost:3000');
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         // Inizialmente la striscia è visibile
@@ -125,7 +135,8 @@ test.describe('Game Operations', () => {
         const controllerPage = await context.newPage();
 
         await displayPage.goto('http://localhost:3000');
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         await resetGame(controllerPage);
@@ -157,7 +168,8 @@ test.describe('Game Operations', () => {
 
     test('Cambi: dovrebbe mostrare errore per giocatore non in formazione', async ({ context }) => {
         const controllerPage = await context.newPage();
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         await resetGame(controllerPage);

tests/e2e/game-simulation.spec.cjs

@@ -7,7 +7,8 @@ test.describe('Game Simulation', () => {
         const controllerPage = await context.newPage();
 
         await displayPage.goto('http://localhost:3000');
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
 
         // Selettori (basati su ID ipotetici o classi, adattali al tuo HTML reale)
         // Assumo che nel DOM ci siano elementi con ID o classi riconoscibili
@@ -23,6 +24,12 @@ test.describe('Game Simulation', () => {
                 await btnConfirmReset.click();
             }
         }
+        // doReset apre automaticamente il dialog di configurazione: chiudilo
+        const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel');
+        if (await cfgCancel.isVisible()) {
+            await cfgCancel.click();
+        }
+        await controllerPage.waitForTimeout(200);
 
         // 2. Loop per vincere il primo set (25 punti)
         // In ControllerPage.vue, il click su .team-score.home-bg incrementa i punti home
@@ -55,15 +62,15 @@ test.describe('Game Simulation', () => {
         // 2. Cliccare "SET HOME".
         // 3. Verificare che Set Home = 1.
 
-        // Verifica che siamo a 25
+        // A 25-0 compare automaticamente il dialog "SET VINTO"
         await expect(controllerPage.locator('.team-score.home-bg .team-pts')).toHaveText('25');
+        await controllerPage.waitForSelector('.dialog-winner');
 
-        // Clicca bottone SET
+        // Procedi al set successivo: registra il set vinto da Home
-        const btnSetHome = controllerPage.locator('.btn-set.home-bg');
+        await controllerPage.getByText('VAI AL SET SUCCESSIVO').click();
-        await btnSetHome.click();
+        await controllerPage.waitForTimeout(300);
 
-        // Verifica che il set sia incrementato
+        // Verifica che il set Home sia incrementato a 1
-        // Nota: display potrebbe chiamarsi diversamente, controlliamo Controller per coerenza
         await expect(controllerPage.locator('.team-score.home-bg .team-set')).toContainText('SET 1');
     });
 });

tests/e2e/visual-regression.spec.cjs

@@ -7,6 +7,11 @@ async function resetGame(controllerPage) {
     if (await btnConfirm.isVisible()) {
         await btnConfirm.click();
     }
+    // doReset apre automaticamente il dialog di configurazione: chiudilo
+    const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel');
+    if (await cfgCancel.isVisible()) {
+        await cfgCancel.click();
+    }
     await controllerPage.waitForTimeout(300);
 }
 
@@ -16,7 +21,8 @@ test.describe('Visual Regression', () => {
         const controllerPage = await context.newPage();
         const displayPage = await context.newPage();
 
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await displayPage.goto('http://localhost:3000');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
@@ -35,7 +41,8 @@ test.describe('Visual Regression', () => {
         const controllerPage = await context.newPage();
         const displayPage = await context.newPage();
 
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await displayPage.goto('http://localhost:3000');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
@@ -59,7 +66,8 @@ test.describe('Visual Regression', () => {
 
     test('Controller: screenshot stato iniziale', async ({ context }) => {
         const controllerPage = await context.newPage();
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         await resetGame(controllerPage);
@@ -71,7 +79,8 @@ test.describe('Visual Regression', () => {
 
     test('Controller: screenshot con modal config aperta', async ({ context }) => {
         const controllerPage = await context.newPage();
-        await controllerPage.goto('http://localhost:3001');
+        await controllerPage.setViewportSize({ width: 390, height: 844 });
+        await controllerPage.goto('http://localhost:3000/controller');
         await controllerPage.waitForSelector('.conn-bar.connected');
 
         // Apri config

tests/integration/server.test.js

@@ -0,0 +1,66 @@
+import { describe, it, expect, beforeAll, afterAll } from 'vitest'
+import { mkdtempSync, writeFileSync, rmSync } from 'fs'
+import { tmpdir } from 'os'
+import { join } from 'path'
+import { createApp } from '../../server.js'
+
+// Usiamo una dist temporanea con file marcatori, così il test non dipende da
+// una build reale ed è deterministico.
+let server
+let baseUrl
+let distDir
+
+beforeAll(async () => {
+    distDir = mkdtempSync(join(tmpdir(), 'segnapunti-dist-'))
+    writeFileSync(join(distDir, 'index.html'), 'DISPLAY_MARKER')
+    writeFileSync(join(distDir, 'controller.html'), 'CONTROLLER_MARKER')
+    writeFileSync(join(distDir, 'app.js'), 'ASSET_MARKER')
+
+    const app = createApp(distDir)
+    await new Promise((resolve) => {
+        server = app.listen(0, () => {
+            baseUrl = `http://127.0.0.1:${server.address().port}`
+            resolve()
+        })
+    })
+})
+
+afterAll(async () => {
+    await new Promise((resolve) => server.close(resolve))
+    rmSync(distDir, { recursive: true, force: true })
+})
+
+async function get(path) {
+    const res = await fetch(baseUrl + path)
+    return { status: res.status, body: await res.text() }
+}
+
+describe('Routing server (server.js)', () => {
+    it('GET / serve la pagina display', async () => {
+        const { status, body } = await get('/')
+        expect(status).toBe(200)
+        expect(body).toBe('DISPLAY_MARKER')
+    })
+
+    it('GET /display serve la pagina display', async () => {
+        expect((await get('/display')).body).toBe('DISPLAY_MARKER')
+    })
+
+    it('GET /display/qualsiasi serve comunque la pagina display (SPA)', async () => {
+        expect((await get('/display/foo/bar')).body).toBe('DISPLAY_MARKER')
+    })
+
+    it('GET /controller serve la pagina controller', async () => {
+        expect((await get('/controller')).body).toBe('CONTROLLER_MARKER')
+    })
+
+    it('GET /controller/qualsiasi serve comunque la pagina controller', async () => {
+        expect((await get('/controller/foo')).body).toBe('CONTROLLER_MARKER')
+    })
+
+    it('serve gli asset statici dalla dist', async () => {
+        const { status, body } = await get('/app.js')
+        expect(status).toBe(200)
+        expect(body).toBe('ASSET_MARKER')
+    })
+})

tests/integration/websocket.test.js

@@ -1,7 +1,11 @@
 import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest'
 import { setupWebSocketHandler } from '../../src/websocket-handler.js'
+import { punteggio } from '../../src/gameState.js'
 import { EventEmitter } from 'events'
 
+// Il punteggio non è memorizzato nello stato: si ricava dalla striscia.
+const puntHome = (state) => punteggio(state.sp.striscia).home
+
 // Mock parziale di una WebSocket e del Server
 class MockWebSocket extends EventEmitter {
     constructor() {
@@ -103,7 +107,7 @@ describe('WebSocket Integration (websocket-handler.js)', () => {
             expect(controller.send).toHaveBeenCalled()
             const sentMsg = lastSent(controller)
             expect(sentMsg.type).toBe('state')
-            expect(sentMsg.state.sp.punt.home).toBe(1)
+            expect(puntHome(sentMsg.state)).toBe(1)
         })
 
         it('dovrebbe impedire al display di inviare azioni', () => {
@@ -182,8 +186,8 @@ describe('WebSocket Integration (websocket-handler.js)', () => {
             const msg1 = lastSent(display1)
             const msg2 = lastSent(display2)
             expect(msg1.type).toBe('state')
-            expect(msg1.state.sp.punt.home).toBe(1)
+            expect(puntHome(msg1.state)).toBe(1)
-            expect(msg2.state.sp.punt.home).toBe(1)
+            expect(puntHome(msg2.state)).toBe(1)
         })
 
         it('non dovrebbe inviare a client con readyState != OPEN', () => {
@@ -302,7 +306,7 @@ describe('WebSocket Integration (websocket-handler.js)', () => {
 
             const msg = lastSent(controller)
             expect(msg.type).toBe('state')
-            expect(msg.state.sp.punt.home).toBe(1)
+            expect(puntHome(msg.state)).toBe(1)
         })
     })
 
@@ -374,15 +378,15 @@ describe('WebSocket Integration (websocket-handler.js)', () => {
     describe('API pubblica', () => {
         it('getState dovrebbe restituire lo stato corrente', () => {
             const state = handler.getState()
-            expect(state.sp.punt.home).toBe(0)
+            expect(puntHome(state)).toBe(0)
-            expect(state.sp.punt.guest).toBe(0)
+            expect(punteggio(state.sp.striscia).guest).toBe(0)
         })
 
         it('setState dovrebbe sovrascrivere lo stato', () => {
             const newState = handler.getState()
-            newState.sp.punt.home = 99
+            newState.sp.striscia.at(-1).ris = 'hh'
             handler.setState(newState)
-            expect(handler.getState().sp.punt.home).toBe(99)
+            expect(puntHome(handler.getState())).toBe(2)
         })
 
         it('broadcastState dovrebbe inviare a tutti i client', () => {

tests/stress/websocket-load.test.js

@@ -1,7 +1,11 @@
 import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest'
 import { setupWebSocketHandler } from '../../src/websocket-handler.js'
+import { punteggio } from '../../src/gameState.js'
 import { EventEmitter } from 'events'
 
+// Il punteggio si ricava dalla striscia, non è memorizzato nello stato.
+const punt = (state) => punteggio(state.sp.striscia)
+
 class MockWebSocket extends EventEmitter {
     constructor() {
         super()
@@ -57,7 +61,7 @@ describe('Stress Test WebSocket', () => {
             expect(display.send).toHaveBeenCalled()
             const msg = JSON.parse(display.send.mock.calls[display.send.mock.calls.length - 1][0])
             expect(msg.type).toBe('state')
-            expect(msg.state.sp.punt.home).toBe(1)
+            expect(punt(msg.state).home).toBe(1)
         }
     })
 
@@ -81,11 +85,11 @@ describe('Stress Test WebSocket', () => {
         // Lo stato finale dipende da checkVittoria che blocca a 25+2
         // Home arriva a 25-0 → vittoria → blocca. Quindi punti home = 25
         const state = handler.getState()
-        expect(state.sp.punt.home).toBe(25)
+        expect(punt(state).home).toBe(25)
         // Guest: non può segnare dopo vittoria? No, checkVittoria blocca solo il team che ha vinto?
         // Controlliamo: checkVittoria controlla ENTRAMBI i team.
         // A 25-0 → vittoria=true → incPunt per guest è anche bloccato
-        expect(state.sp.punt.guest).toBe(0)
+        expect(punt(state).guest).toBe(0)
     })
 
     it('dovrebbe garantire che tutti i display ricevano ogni update sotto carico', () => {
@@ -112,7 +116,7 @@ describe('Stress Test WebSocket', () => {
         // Verifica stato finale su tutti i display
         for (const display of displays) {
             const lastMsg = JSON.parse(display.send.mock.calls[4][0])
-            expect(lastMsg.state.sp.punt.home).toBe(5)
+            expect(punt(lastMsg.state).home).toBe(5)
         }
     })
 })

tests/unit/gameState.test.js

@@ -1,5 +1,39 @@
 import { describe, it, expect, beforeEach } from 'vitest'
-import { createInitialState, applyAction, checkVittoria } from '../../src/gameState.js'
+import {
+    createInitialState, applyAction, checkVittoria, checkVittoriaPartita,
+    punteggio, setVinti, servizio,
+} from '../../src/gameState.js'
+
+// =============================================
+// HELPER
+// Lo stato non memorizza più punteggio/set/servizio direttamente:
+// si ricavano dalla striscia tramite le funzioni pure esportate.
+// =============================================
+
+// Punteggio del set in corso
+const puntDi = (s) => punteggio(s.sp.striscia)
+// Set vinti nel match
+const setDi = (s) => setVinti(s.sp.striscia)
+// true se serve Home
+const servDi = (s) => servizio(s.sp.striscia)
+
+// Imposta chi serve a inizio set (set in corso a 0-0)
+function setServizioIniziale(state, team) {
+    state.sp.striscia.at(-1).serv = team === 'home' ? 'h' : 'g'
+}
+
+// Imposta il punteggio del set in corso costruendo una ris coerente
+function setPunteggio(state, home, guest) {
+    state.sp.striscia.at(-1).ris = 'h'.repeat(home) + 'g'.repeat(guest)
+}
+
+// Aggiunge set già conclusi (vinti) PRIMA del set in corso
+function setSetVinti(state, home, guest) {
+    const conclusi = []
+    for (let i = 0; i < home; i++) conclusi.push({ serv: 'h', ris: '', vinc: 'h' })
+    for (let i = 0; i < guest; i++) conclusi.push({ serv: 'g', ris: '', vinc: 'g' })
+    state.sp.striscia = [...conclusi, state.sp.striscia.at(-1)]
+}
 
 describe('Game Logic (gameState.js)', () => {
     let state
@@ -13,17 +47,17 @@ describe('Game Logic (gameState.js)', () => {
     // =============================================
     describe('Stato iniziale', () => {
         it('dovrebbe iniziare con 0-0', () => {
-            expect(state.sp.punt.home).toBe(0)
+            expect(puntDi(state).home).toBe(0)
-            expect(state.sp.punt.guest).toBe(0)
+            expect(puntDi(state).guest).toBe(0)
         })
 
         it('dovrebbe avere i set a 0', () => {
-            expect(state.sp.set.home).toBe(0)
+            expect(setDi(state).home).toBe(0)
-            expect(state.sp.set.guest).toBe(0)
+            expect(setDi(state).guest).toBe(0)
         })
 
         it('dovrebbe avere servizio Home', () => {
-            expect(state.sp.servHome).toBe(true)
+            expect(servDi(state)).toBe(true)
         })
 
         it('dovrebbe avere formazione di default [1-6]', () => {
@@ -33,8 +67,8 @@ describe('Game Logic (gameState.js)', () => {
 
         it('dovrebbe avere la striscia iniziale con un set vuoto', () => {
             expect(state.sp.striscia).toHaveLength(1)
-            expect(state.sp.striscia[0].serv).toBe('home')
+            expect(state.sp.striscia[0].serv).toBe('h')
-            expect(state.sp.striscia[0].r).toEqual([])
+            expect(state.sp.striscia[0].ris).toBe('')
         })
 
         it('dovrebbe avere modalità 3/5 di default', () => {
@@ -69,43 +103,43 @@ describe('Game Logic (gameState.js)', () => {
     describe('incPunt', () => {
         it('dovrebbe incrementare i punti Home', () => {
             const newState = applyAction(state, { type: 'incPunt', team: 'home' })
-            expect(newState.sp.punt.home).toBe(1)
+            expect(puntDi(newState).home).toBe(1)
-            expect(newState.sp.punt.guest).toBe(0)
+            expect(puntDi(newState).guest).toBe(0)
         })
 
         it('dovrebbe incrementare i punti Guest', () => {
             const newState = applyAction(state, { type: 'incPunt', team: 'guest' })
-            expect(newState.sp.punt.guest).toBe(1)
+            expect(puntDi(newState).guest).toBe(1)
-            expect(newState.sp.punt.home).toBe(0)
+            expect(puntDi(newState).home).toBe(0)
         })
 
         it('dovrebbe gestire il cambio palla (Guest segna, batteva Home)', () => {
-            state.sp.servHome = true
+            setServizioIniziale(state, 'home')
             const s1 = applyAction(state, { type: 'incPunt', team: 'guest' })
-            expect(s1.sp.servHome).toBe(false)
+            expect(servDi(s1)).toBe(false)
         })
 
         it('dovrebbe gestire il cambio palla (Home segna, batteva Guest)', () => {
-            state.sp.servHome = false
+            setServizioIniziale(state, 'guest')
             const s1 = applyAction(state, { type: 'incPunt', team: 'home' })
-            expect(s1.sp.servHome).toBe(true)
+            expect(servDi(s1)).toBe(true)
         })
 
         it('non dovrebbe cambiare palla se segna chi batte', () => {
-            state.sp.servHome = true
+            setServizioIniziale(state, 'home')
             const s1 = applyAction(state, { type: 'incPunt', team: 'home' })
-            expect(s1.sp.servHome).toBe(true)
+            expect(servDi(s1)).toBe(true)
         })
 
         it('dovrebbe ruotare la formazione al cambio palla', () => {
-            state.sp.servHome = true
+            setServizioIniziale(state, 'home')
             state.sp.form.guest = ["1", "2", "3", "4", "5", "6"]
             const newState = applyAction(state, { type: 'incPunt', team: 'guest' })
             expect(newState.sp.form.guest).toEqual(["2", "3", "4", "5", "6", "1"])
         })
 
         it('non dovrebbe ruotare la formazione se non c\'è cambio palla', () => {
-            state.sp.servHome = true
+            setServizioIniziale(state, 'home')
             state.sp.form.home = ["1", "2", "3", "4", "5", "6"]
             const newState = applyAction(state, { type: 'incPunt', team: 'home' })
             expect(newState.sp.form.home).toEqual(["1", "2", "3", "4", "5", "6"])
@@ -113,26 +147,25 @@ describe('Game Logic (gameState.js)', () => {
 
         it('dovrebbe aggiornare la striscia per punto Home', () => {
             const s = applyAction(state, { type: 'incPunt', team: 'home' })
-            expect(s.sp.striscia.at(-1).r).toEqual(['home'])
+            expect(s.sp.striscia.at(-1).ris).toBe('h')
         })
 
         it('dovrebbe aggiornare la striscia per punto Guest', () => {
             const s = applyAction(state, { type: 'incPunt', team: 'guest' })
-            expect(s.sp.striscia.at(-1).r).toEqual(['guest'])
+            expect(s.sp.striscia.at(-1).ris).toBe('g')
         })
 
         it('dovrebbe registrare scorer nella striscia', () => {
             let s = applyAction(state, { type: 'incPunt', team: 'home' })
             s = applyAction(s, { type: 'incPunt', team: 'guest' })
             s = applyAction(s, { type: 'incPunt', team: 'home' })
-            expect(s.sp.striscia.at(-1).r).toEqual(['home', 'guest', 'home'])
+            expect(s.sp.striscia.at(-1).ris).toBe('hgh')
         })
 
         it('non dovrebbe incrementare i punti dopo vittoria', () => {
-            state.sp.punt.home = 25
+            setPunteggio(state, 25, 23)
-            state.sp.punt.guest = 23
             const s = applyAction(state, { type: 'incPunt', team: 'home' })
-            expect(s.sp.punt.home).toBe(25)
+            expect(puntDi(s).home).toBe(25)
         })
     })
 
@@ -143,33 +176,33 @@ describe('Game Logic (gameState.js)', () => {
         it('dovrebbe annullare l\'ultimo punto Home', () => {
             const s1 = applyAction(state, { type: 'incPunt', team: 'home' })
             const s2 = applyAction(s1, { type: 'decPunt' })
-            expect(s2.sp.punt.home).toBe(0)
+            expect(puntDi(s2).home).toBe(0)
-            expect(s2.sp.punt.guest).toBe(0)
+            expect(puntDi(s2).guest).toBe(0)
         })
 
         it('dovrebbe annullare l\'ultimo punto Guest', () => {
             const s1 = applyAction(state, { type: 'incPunt', team: 'guest' })
             const s2 = applyAction(s1, { type: 'decPunt' })
-            expect(s2.sp.punt.home).toBe(0)
+            expect(puntDi(s2).home).toBe(0)
-            expect(s2.sp.punt.guest).toBe(0)
+            expect(puntDi(s2).guest).toBe(0)
         })
 
         it('non dovrebbe fare nulla sullo stato iniziale', () => {
             const s = applyAction(state, { type: 'decPunt' })
-            expect(s.sp.punt.home).toBe(0)
+            expect(puntDi(s).home).toBe(0)
-            expect(s.sp.punt.guest).toBe(0)
+            expect(puntDi(s).guest).toBe(0)
         })
 
         it('dovrebbe ripristinare il servizio dopo undo con cambio palla', () => {
-            state.sp.servHome = true
+            setServizioIniziale(state, 'home')
             const s1 = applyAction(state, { type: 'incPunt', team: 'guest' })
-            expect(s1.sp.servHome).toBe(false)
+            expect(servDi(s1)).toBe(false)
             const s2 = applyAction(s1, { type: 'decPunt' })
-            expect(s2.sp.servHome).toBe(true)
+            expect(servDi(s2)).toBe(true)
         })
 
         it('dovrebbe invertire la rotazione dopo undo con cambio palla', () => {
-            state.sp.servHome = true
+            setServizioIniziale(state, 'home')
             state.sp.form.guest = ["1", "2", "3", "4", "5", "6"]
             const s1 = applyAction(state, { type: 'incPunt', team: 'guest' })
             expect(s1.sp.form.guest).toEqual(["2", "3", "4", "5", "6", "1"])
@@ -180,7 +213,7 @@ describe('Game Logic (gameState.js)', () => {
         it('dovrebbe ripristinare la striscia', () => {
             const s1 = applyAction(state, { type: 'incPunt', team: 'home' })
             const s2 = applyAction(s1, { type: 'decPunt' })
-            expect(s2.sp.striscia.at(-1).r).toEqual([])
+            expect(s2.sp.striscia.at(-1).ris).toBe('')
         })
 
         it('dovrebbe gestire undo multipli in sequenza', () => {
@@ -188,14 +221,66 @@ describe('Game Logic (gameState.js)', () => {
             s = applyAction(s, { type: 'incPunt', team: 'home' })
             s = applyAction(s, { type: 'incPunt', team: 'guest' })
             s = applyAction(s, { type: 'incPunt', team: 'home' })
-            expect(s.sp.punt.home).toBe(2)
+            expect(puntDi(s).home).toBe(2)
-            expect(s.sp.punt.guest).toBe(1)
+            expect(puntDi(s).guest).toBe(1)
             s = applyAction(s, { type: 'decPunt' })
-            expect(s.sp.punt.home).toBe(1)
+            expect(puntDi(s).home).toBe(1)
             s = applyAction(s, { type: 'decPunt' })
-            expect(s.sp.punt.guest).toBe(0)
+            expect(puntDi(s).guest).toBe(0)
             s = applyAction(s, { type: 'decPunt' })
-            expect(s.sp.punt.home).toBe(0)
+            expect(puntDi(s).home).toBe(0)
+        })
+    })
+
+    // =============================================
+    // FORMAZIONE DI PARTENZA (formInizio)
+    // =============================================
+    describe('formInizio', () => {
+        it('dovrebbe salvare la formazione corrente al primo punto del set', () => {
+            const s = applyAction(state, { type: 'incPunt', team: 'home' })
+            expect(s.sp.striscia.at(-1).formInizio).toEqual({
+                home: ["1", "2", "3", "4", "5", "6"],
+                guest: ["1", "2", "3", "4", "5", "6"],
+            })
+        })
+
+        it('formInizio è uno snapshot: la rotazione successiva non lo modifica', () => {
+            setServizioIniziale(state, 'home')
+            // 1° punto Home: nessun cambio palla, salva formInizio
+            let s = applyAction(state, { type: 'incPunt', team: 'home' })
+            // 2° punto Guest: cambio palla → ruota la formazione guest
+            s = applyAction(s, { type: 'incPunt', team: 'guest' })
+            expect(s.sp.form.guest).toEqual(["2", "3", "4", "5", "6", "1"])
+            // lo snapshot resta quello iniziale
+            expect(s.sp.striscia.at(-1).formInizio.guest).toEqual(["1", "2", "3", "4", "5", "6"])
+        })
+
+        it('decPunt che riporta il set a 0-0 cancella formInizio', () => {
+            const s1 = applyAction(state, { type: 'incPunt', team: 'home' })
+            expect(s1.sp.striscia.at(-1).formInizio).toBeDefined()
+            const s2 = applyAction(s1, { type: 'decPunt' })
+            expect(s2.sp.striscia.at(-1).formInizio).toBeUndefined()
+        })
+
+        it('decPunt con punti ancora presenti NON cancella formInizio', () => {
+            let s = applyAction(state, { type: 'incPunt', team: 'home' })
+            s = applyAction(s, { type: 'incPunt', team: 'home' })
+            s = applyAction(s, { type: 'decPunt' })
+            expect(s.sp.striscia.at(-1).ris).toBe('h')
+            expect(s.sp.striscia.at(-1).formInizio).toBeDefined()
+        })
+
+        it('ogni set mantiene la propria formInizio', () => {
+            const custom = ['7', '8', '9', '10', '11', '12']
+            let s = applyAction(state, { type: 'setFormazione', team: 'home', form: custom })
+            // primo punto del set 1: salva formInizio custom
+            s = applyAction(s, { type: 'incPunt', team: 'home' })
+            // chiude il set 1 e ne apre uno nuovo (formazioni resettate a default)
+            s = applyAction(s, { type: 'nuovoSet', team: 'home' })
+            // primo punto del set 2: salva formInizio default
+            s = applyAction(s, { type: 'incPunt', team: 'home' })
+            expect(s.sp.striscia[0].formInizio.home).toEqual(custom)
+            expect(s.sp.striscia.at(-1).formInizio.home).toEqual(['1', '2', '3', '4', '5', '6'])
         })
     })
 
@@ -205,24 +290,27 @@ describe('Game Logic (gameState.js)', () => {
     describe('incSet', () => {
         it('dovrebbe incrementare il set Home', () => {
             const s = applyAction(state, { type: 'incSet', team: 'home' })
-            expect(s.sp.set.home).toBe(1)
+            expect(setDi(s).home).toBe(1)
         })
 
         it('dovrebbe incrementare il set Guest', () => {
             const s = applyAction(state, { type: 'incSet', team: 'guest' })
-            expect(s.sp.set.guest).toBe(1)
+            expect(setDi(s).guest).toBe(1)
         })
 
         it('dovrebbe fare wrap da 2 a 0', () => {
-            state.sp.set.home = 2
+            let s = applyAction(state, { type: 'incSet', team: 'home' })
-            const s = applyAction(state, { type: 'incSet', team: 'home' })
+            s = applyAction(s, { type: 'incSet', team: 'home' })
-            expect(s.sp.set.home).toBe(0)
+            expect(setDi(s).home).toBe(2)
+            s = applyAction(s, { type: 'incSet', team: 'home' })
+            expect(setDi(s).home).toBe(0)
         })
 
         it('dovrebbe incrementare da 1 a 2', () => {
-            state.sp.set.home = 1
+            let s = applyAction(state, { type: 'incSet', team: 'home' })
-            const s = applyAction(state, { type: 'incSet', team: 'home' })
+            expect(setDi(s).home).toBe(1)
-            expect(s.sp.set.home).toBe(2)
+            s = applyAction(s, { type: 'incSet', team: 'home' })
+            expect(setDi(s).home).toBe(2)
         })
     })
 
@@ -231,31 +319,29 @@ describe('Game Logic (gameState.js)', () => {
     // =============================================
     describe('nuovoSet', () => {
         it('dovrebbe incrementare il set della squadra vincente', () => {
-            state.sp.punt.home = 25
             const s = applyAction(state, { type: 'nuovoSet', team: 'home' })
-            expect(s.sp.set.home).toBe(1)
+            expect(setDi(s).home).toBe(1)
-            expect(s.sp.set.guest).toBe(0)
+            expect(setDi(s).guest).toBe(0)
         })
 
-        it('dovrebbe azzerare i punti', () => {
+        it('dovrebbe azzerare i punti nel nuovo set', () => {
-            state.sp.punt.home = 25
+            setPunteggio(state, 25, 10)
-            state.sp.punt.guest = 10
             const s = applyAction(state, { type: 'nuovoSet', team: 'home' })
-            expect(s.sp.punt.home).toBe(0)
+            expect(puntDi(s).home).toBe(0)
-            expect(s.sp.punt.guest).toBe(0)
+            expect(puntDi(s).guest).toBe(0)
         })
 
         it('dovrebbe aggiungere un nuovo set vuoto alla striscia', () => {
             const s = applyAction(state, { type: 'nuovoSet', team: 'home' })
             expect(s.sp.striscia).toHaveLength(2)
-            expect(s.sp.striscia.at(-1).r).toEqual([])
+            expect(s.sp.striscia.at(-1).ris).toBe('')
-            expect(s.sp.striscia.at(-1).serv).toBe('home')
+            expect(s.sp.striscia.at(-1).serv).toBe('h')
         })
 
         it('dovrebbe conservare il set precedente nella striscia', () => {
-            state.sp.striscia[0].r = ['home', 'guest', 'home']
+            state.sp.striscia[0].ris = 'hgh'
             const s = applyAction(state, { type: 'nuovoSet', team: 'home' })
-            expect(s.sp.striscia[0].r).toEqual(['home', 'guest', 'home'])
+            expect(s.sp.striscia[0].ris).toBe('hgh')
         })
 
         it('dovrebbe resettare le formazioni', () => {
@@ -268,8 +354,70 @@ describe('Game Logic (gameState.js)', () => {
 
         it('dovrebbe ignorare team non valido', () => {
             const s = applyAction(state, { type: 'nuovoSet', team: 'invalid' })
-            expect(s.sp.set.home).toBe(0)
+            expect(setDi(s).home).toBe(0)
-            expect(s.sp.set.guest).toBe(0)
+            expect(setDi(s).guest).toBe(0)
+        })
+
+        it('in 2/3 alla palla match registra il set vincente senza aprirne uno nuovo', () => {
+            state.modalitaPartita = '2/3'
+            setSetVinti(state, 1, 0)
+            setPunteggio(state, 25, 18)
+            const s = applyAction(state, { type: 'nuovoSet', team: 'home' })
+            expect(setDi(s).home).toBe(2)
+            expect(puntDi(s).home).toBe(25)
+            expect(puntDi(s).guest).toBe(18)
+        })
+
+        it('in 3/5 alla palla match registra il set vincente senza aprirne uno nuovo', () => {
+            state.modalitaPartita = '3/5'
+            setSetVinti(state, 2, 0)
+            setPunteggio(state, 25, 20)
+            const s = applyAction(state, { type: 'nuovoSet', team: 'home' })
+            expect(setDi(s).home).toBe(3)
+            expect(puntDi(s).home).toBe(25)
+            expect(puntDi(s).guest).toBe(20)
+        })
+
+        it('dovrebbe ignorare nuovoSet se la partita è già finita', () => {
+            state.modalitaPartita = '2/3'
+            setSetVinti(state, 2, 0)
+            const s = applyAction(state, { type: 'nuovoSet', team: 'home' })
+            expect(setDi(s).home).toBe(2)
+        })
+    })
+
+    // =============================================
+    // VITTORIA PARTITA (checkVittoriaPartita)
+    // =============================================
+    describe('checkVittoriaPartita', () => {
+        it('in 2/3 restituisce false se nessuno ha 2 set', () => {
+            state.modalitaPartita = '2/3'
+            setSetVinti(state, 1, 0)
+            expect(checkVittoriaPartita(state)).toBe(false)
+        })
+
+        it('in 2/3 restituisce true se home ha 2 set', () => {
+            state.modalitaPartita = '2/3'
+            setSetVinti(state, 2, 0)
+            expect(checkVittoriaPartita(state)).toBe(true)
+        })
+
+        it('in 3/5 restituisce false se nessuno ha 3 set', () => {
+            state.modalitaPartita = '3/5'
+            setSetVinti(state, 2, 2)
+            expect(checkVittoriaPartita(state)).toBe(false)
+        })
+
+        it('in 3/5 restituisce true se guest ha 3 set', () => {
+            state.modalitaPartita = '3/5'
+            setSetVinti(state, 0, 3)
+            expect(checkVittoriaPartita(state)).toBe(true)
+        })
+
+        it('in amichevole restituisce sempre false', () => {
+            state.modalitaPartita = 'amichevole'
+            setSetVinti(state, 5, 0)
+            expect(checkVittoriaPartita(state)).toBe(false)
         })
     })
 
@@ -278,27 +426,29 @@ describe('Game Logic (gameState.js)', () => {
     // =============================================
     describe('cambiaPalla', () => {
         it('dovrebbe invertire il servizio a 0-0', () => {
-            expect(state.sp.servHome).toBe(true)
+            expect(servDi(state)).toBe(true)
             const s = applyAction(state, { type: 'cambiaPalla' })
-            expect(s.sp.servHome).toBe(false)
+            expect(servDi(s)).toBe(false)
         })
 
         it('dovrebbe tornare a Home con doppio toggle', () => {
             let s = applyAction(state, { type: 'cambiaPalla' })
             s = applyAction(s, { type: 'cambiaPalla' })
-            expect(s.sp.servHome).toBe(true)
+            expect(servDi(s)).toBe(true)
         })
 
         it('non dovrebbe cambiare palla se il punteggio non è 0-0', () => {
-            state.sp.punt.home = 1
+            setPunteggio(state, 1, 0)
+            const prima = servDi(state)
             const s = applyAction(state, { type: 'cambiaPalla' })
-            expect(s.sp.servHome).toBe(true)
+            expect(servDi(s)).toBe(prima)
         })
 
         it('non dovrebbe cambiare palla se Guest ha punti', () => {
-            state.sp.punt.guest = 3
+            setPunteggio(state, 0, 3)
+            const prima = servDi(state)
             const s = applyAction(state, { type: 'cambiaPalla' })
-            expect(s.sp.servHome).toBe(true)
+            expect(servDi(s)).toBe(prima)
         })
     })
 
@@ -509,44 +659,37 @@ describe('Game Logic (gameState.js)', () => {
     // =============================================
     describe('checkVittoria', () => {
         it('non dovrebbe dare vittoria a 24-24', () => {
-            state.sp.punt.home = 24
+            setPunteggio(state, 24, 24)
-            state.sp.punt.guest = 24
             expect(checkVittoria(state)).toBe(false)
         })
 
         it('dovrebbe dare vittoria a 25-23', () => {
-            state.sp.punt.home = 25
+            setPunteggio(state, 25, 23)
-            state.sp.punt.guest = 23
             expect(checkVittoria(state)).toBe(true)
         })
 
         it('non dovrebbe dare vittoria a 25-24 (serve 2 punti di scarto)', () => {
-            state.sp.punt.home = 25
+            setPunteggio(state, 25, 24)
-            state.sp.punt.guest = 24
             expect(checkVittoria(state)).toBe(false)
         })
 
         it('dovrebbe dare vittoria a 26-24', () => {
-            state.sp.punt.home = 26
+            setPunteggio(state, 26, 24)
-            state.sp.punt.guest = 24
             expect(checkVittoria(state)).toBe(true)
         })
 
         it('dovrebbe dare vittoria Guest a 25-20', () => {
-            state.sp.punt.home = 20
+            setPunteggio(state, 20, 25)
-            state.sp.punt.guest = 25
             expect(checkVittoria(state)).toBe(true)
         })
 
         it('dovrebbe dare vittoria ai vantaggi (30-28)', () => {
-            state.sp.punt.home = 30
+            setPunteggio(state, 30, 28)
-            state.sp.punt.guest = 28
             expect(checkVittoria(state)).toBe(true)
         })
 
         it('non dovrebbe dare vittoria ai vantaggi senza scarto (28-27)', () => {
-            state.sp.punt.home = 28
+            setPunteggio(state, 28, 27)
-            state.sp.punt.guest = 27
             expect(checkVittoria(state)).toBe(false)
         })
     })
@@ -557,82 +700,64 @@ describe('Game Logic (gameState.js)', () => {
     describe('Set decisivo', () => {
         it('modalità 3/5: set decisivo dopo 4 set totali → vittoria a 15', () => {
             state.modalitaPartita = "3/5"
-            state.sp.set.home = 2
+            setSetVinti(state, 2, 2)
-            state.sp.set.guest = 2
+            setPunteggio(state, 15, 10)
-            state.sp.punt.home = 15
-            state.sp.punt.guest = 10
             expect(checkVittoria(state)).toBe(true)
         })
 
         it('modalità 3/5: non vittoria a 14-10 nel set decisivo', () => {
             state.modalitaPartita = "3/5"
-            state.sp.set.home = 2
+            setSetVinti(state, 2, 2)
-            state.sp.set.guest = 2
+            setPunteggio(state, 14, 10)
-            state.sp.punt.home = 14
-            state.sp.punt.guest = 10
             expect(checkVittoria(state)).toBe(false)
         })
 
         it('modalità 3/5: vittoria a 15-13 nel set decisivo', () => {
             state.modalitaPartita = "3/5"
-            state.sp.set.home = 2
+            setSetVinti(state, 2, 2)
-            state.sp.set.guest = 2
+            setPunteggio(state, 15, 13)
-            state.sp.punt.home = 15
-            state.sp.punt.guest = 13
             expect(checkVittoria(state)).toBe(true)
         })
 
         it('modalità 3/5: non vittoria a 15-14 nel set decisivo (serve scarto)', () => {
             state.modalitaPartita = "3/5"
-            state.sp.set.home = 2
+            setSetVinti(state, 2, 2)
-            state.sp.set.guest = 2
+            setPunteggio(state, 15, 14)
-            state.sp.punt.home = 15
-            state.sp.punt.guest = 14
             expect(checkVittoria(state)).toBe(false)
         })
 
         it('modalità 3/5: vittoria a 16-14 nel set decisivo', () => {
             state.modalitaPartita = "3/5"
-            state.sp.set.home = 2
+            setSetVinti(state, 2, 2)
-            state.sp.set.guest = 2
+            setPunteggio(state, 16, 14)
-            state.sp.punt.home = 16
-            state.sp.punt.guest = 14
             expect(checkVittoria(state)).toBe(true)
         })
 
         it('modalità 2/3: set decisivo dopo 2 set totali → vittoria a 15', () => {
             state.modalitaPartita = "2/3"
-            state.sp.set.home = 1
+            setSetVinti(state, 1, 1)
-            state.sp.set.guest = 1
+            setPunteggio(state, 15, 10)
-            state.sp.punt.home = 15
-            state.sp.punt.guest = 10
             expect(checkVittoria(state)).toBe(true)
         })
 
-        it('modalità 2/3: non vittoria a 24-20 nel set decisivo (soglia 15)', () => {
+        it('modalità 2/3: non vittoria a 14-10 nel set decisivo (soglia 15)', () => {
             state.modalitaPartita = "2/3"
-            state.sp.set.home = 1
+            setSetVinti(state, 1, 1)
-            state.sp.set.guest = 1
+            setPunteggio(state, 14, 10)
-            state.sp.punt.home = 14
-            state.sp.punt.guest = 10
             expect(checkVittoria(state)).toBe(false)
         })
 
         it('modalità 2/3: set non decisivo (1-0) → soglia 25', () => {
             state.modalitaPartita = "2/3"
-            state.sp.set.home = 1
+            setSetVinti(state, 1, 0)
-            state.sp.set.guest = 0
+            setPunteggio(state, 15, 10)
-            state.sp.punt.home = 15
-            state.sp.punt.guest = 10
             expect(checkVittoria(state)).toBe(false)
         })
 
         it('modalità 3/5: set non decisivo (2-1) → soglia 25', () => {
             state.modalitaPartita = "3/5"
-            state.sp.set.home = 2
+            setSetVinti(state, 2, 1)
-            state.sp.set.guest = 1
+            setPunteggio(state, 15, 10)
-            state.sp.punt.home = 15
-            state.sp.punt.guest = 10
             expect(checkVittoria(state)).toBe(false)
         })
     })
@@ -642,15 +767,13 @@ describe('Game Logic (gameState.js)', () => {
     // =============================================
     describe('Reset', () => {
         it('dovrebbe resettare punti e set a zero', () => {
-            state.sp.punt.home = 10
+            setSetVinti(state, 1, 1)
-            state.sp.punt.guest = 8
+            setPunteggio(state, 10, 8)
-            state.sp.set.home = 1
-            state.sp.set.guest = 1
             const s = applyAction(state, { type: 'resetta' })
-            expect(s.sp.punt.home).toBe(0)
+            expect(puntDi(s).home).toBe(0)
-            expect(s.sp.punt.guest).toBe(0)
+            expect(puntDi(s).guest).toBe(0)
-            expect(s.sp.set.home).toBe(0)
+            expect(setDi(s).home).toBe(0)
-            expect(s.sp.set.guest).toBe(0)
+            expect(setDi(s).guest).toBe(0)
         })
 
         it('dovrebbe resettare formazioni a default', () => {
@@ -661,10 +784,10 @@ describe('Game Logic (gameState.js)', () => {
         })
 
         it('dovrebbe resettare la striscia a un set vuoto', () => {
-            state.sp.striscia = [{ serv: 'home', r: ['home', 'guest', 'home'] }, { serv: 'home', r: ['guest'] }]
+            state.sp.striscia = [{ serv: 'h', ris: 'hgh', vinc: 'h' }, { serv: 'h', ris: 'g', vinc: null }]
             const s = applyAction(state, { type: 'resetta' })
             expect(s.sp.striscia).toHaveLength(1)
-            expect(s.sp.striscia[0].r).toEqual([])
+            expect(s.sp.striscia[0].ris).toBe('')
         })
 
         it('dovrebbe impostare visuForm a false', () => {
@@ -688,8 +811,8 @@ describe('Game Logic (gameState.js)', () => {
     describe('Azione sconosciuta', () => {
         it('dovrebbe restituire lo stato invariato con azione non riconosciuta', () => {
             const s = applyAction(state, { type: 'azioneInesistente' })
-            expect(s.sp.punt.home).toBe(0)
+            expect(puntDi(s).home).toBe(0)
-            expect(s.sp.punt.guest).toBe(0)
+            expect(puntDi(s).guest).toBe(0)
         })
     })
 })

tests/unit/persist.test.js

@@ -0,0 +1,71 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+
+// Mock dell'I/O su disco: i test non toccano il filesystem reale né dipendono
+// dal path relativo a src/.
+vi.mock('fs', () => ({
+    readFileSync: vi.fn(),
+    writeFileSync: vi.fn(),
+    mkdirSync: vi.fn(),
+    existsSync: vi.fn(),
+}))
+
+import * as fs from 'fs'
+import { loadState, saveState } from '../../src/persist.js'
+import { createInitialState } from '../../src/gameState.js'
+
+describe('Persistenza stato (persist.js)', () => {
+    beforeEach(() => {
+        vi.clearAllMocks()
+    })
+
+    afterEach(() => {
+        vi.restoreAllMocks()
+    })
+
+    describe('saveState', () => {
+        it('crea la directory e scrive lo stato serializzato', () => {
+            const state = createInitialState()
+            saveState(state)
+
+            expect(fs.mkdirSync).toHaveBeenCalledWith(expect.any(String), { recursive: true })
+            expect(fs.writeFileSync).toHaveBeenCalled()
+
+            const [, contenuto, encoding] = fs.writeFileSync.mock.calls[0]
+            expect(JSON.parse(contenuto)).toEqual(state)
+            expect(encoding).toBe('utf8')
+        })
+
+        it('non lancia eccezioni se la scrittura fallisce', () => {
+            fs.mkdirSync.mockImplementation(() => { throw new Error('EACCES') })
+            const errSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
+
+            expect(() => saveState(createInitialState())).not.toThrow()
+            expect(errSpy).toHaveBeenCalled()
+        })
+    })
+
+    describe('loadState', () => {
+        it('legge e fa il parse di un file valido', () => {
+            const salvato = createInitialState()
+            salvato.sp.nomi.home = 'Squadra X'
+            fs.existsSync.mockReturnValue(true)
+            fs.readFileSync.mockReturnValue(JSON.stringify(salvato))
+
+            expect(loadState()).toEqual(salvato)
+        })
+
+        it('ritorna lo stato iniziale se il file non esiste', () => {
+            fs.existsSync.mockReturnValue(false)
+            expect(loadState()).toEqual(createInitialState())
+        })
+
+        it('ritorna lo stato iniziale se il JSON è corrotto', () => {
+            fs.existsSync.mockReturnValue(true)
+            fs.readFileSync.mockReturnValue('{ questo non è json')
+            const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
+
+            expect(loadState()).toEqual(createInitialState())
+            expect(warnSpy).toHaveBeenCalled()
+        })
+    })
+})

tests/unit/referto.test.js

@@ -0,0 +1,110 @@
+import { describe, it, expect } from 'vitest'
+import { buildRefertoHtml } from '../../src/referto.js'
+import { createInitialState } from '../../src/gameState.js'
+
+// Data fissa per asserzioni deterministiche
+const NOW = new Date('2026-03-14T20:30:00')
+
+// Costruisce uno stato con una striscia di set arbitraria
+function statoConSet(striscia, extra = {}) {
+    const state = createInitialState()
+    state.sp.striscia = striscia
+    state.sp.nomi = { home: 'Antoniana', guest: 'Rivali' }
+    return { ...state, ...extra }
+}
+
+describe('buildRefertoHtml (referto.js)', () => {
+    it('esclude i set _phantom dal referto', () => {
+        const striscia = [
+            { serv: 'h', ris: 'h'.repeat(25) + 'g'.repeat(20), vinc: 'h' },
+            { serv: 'g', ris: '', vinc: 'g', _phantom: true },
+            { serv: 'h', ris: 'h'.repeat(25) + 'g'.repeat(18), vinc: 'h' },
+        ]
+        const html = buildRefertoHtml(statoConSet(striscia), NOW)
+        // due set reali → "Set 1" e "Set 2", mai "Set 3"
+        expect(html).toContain('Set 1')
+        expect(html).toContain('Set 2')
+        expect(html).not.toContain('Set 3')
+    })
+
+    it('calcola il punteggio finale di ogni set dalla ris', () => {
+        const striscia = [
+            { serv: 'h', ris: 'h'.repeat(25) + 'g'.repeat(20), vinc: 'h' },
+            { serv: 'h', ris: '', vinc: null },
+        ]
+        const html = buildRefertoHtml(statoConSet(striscia), NOW)
+        // header del set: "Antoniana 25 · 20 Rivali"
+        expect(html).toContain('<strong>25</strong>')
+        expect(html).toContain('<strong>20</strong>')
+    })
+
+    it('conta i set vinti usando vinc', () => {
+        const striscia = [
+            { serv: 'h', ris: '', vinc: 'h' },
+            { serv: 'g', ris: '', vinc: 'g' },
+            { serv: 'h', ris: '', vinc: 'h' },
+            { serv: 'h', ris: '', vinc: null },
+        ]
+        const html = buildRefertoHtml(statoConSet(striscia), NOW)
+        // risultato 2 – 1
+        expect(html).toContain('2 – 1')
+    })
+
+    it('ricava il vincitore dal conteggio punti se vinc è nullo', () => {
+        const striscia = [
+            { serv: 'h', ris: 'h'.repeat(25) + 'g'.repeat(23), vinc: null },
+            { serv: 'h', ris: '', vinc: null },
+        ]
+        const html = buildRefertoHtml(statoConSet(striscia), NOW)
+        // il primo set, pur con vinc null, conta come vinto da home → 1 – 0
+        expect(html).toContain('1 – 0')
+    })
+
+    it('include la progressione punto-punto con classi per squadra', () => {
+        const striscia = [
+            { serv: 'h', ris: 'hhg', vinc: null },
+        ]
+        const html = buildRefertoHtml(statoConSet(striscia), NOW)
+        expect(html).toContain('punto-h')
+        expect(html).toContain('punto-g')
+        expect(html).toContain('1-0')
+        expect(html).toContain('2-0')
+        expect(html).toContain('2-1')
+    })
+
+    it('rende la formazione di partenza quando presente', () => {
+        const striscia = [
+            {
+                serv: 'h', ris: 'h', vinc: null,
+                formInizio: { home: ['4', '8', '15'], guest: ['16', '23', '42'] },
+            },
+        ]
+        const html = buildRefertoHtml(statoConSet(striscia), NOW)
+        expect(html).toContain('Formazione di partenza')
+        expect(html).toContain('>4<')
+        expect(html).toContain('>42<')
+    })
+
+    it('mostra "non disponibile" se manca formInizio', () => {
+        const striscia = [{ serv: 'h', ris: 'h', vinc: null }]
+        const html = buildRefertoHtml(statoConSet(striscia), NOW)
+        expect(html).toContain('non disponibile')
+    })
+
+    it('mostra "Nessun punto registrato" per un set senza punti', () => {
+        const striscia = [{ serv: 'h', ris: '', vinc: null }]
+        const html = buildRefertoHtml(statoConSet(striscia), NOW)
+        expect(html).toContain('Nessun punto registrato')
+    })
+
+    it('header contiene nomi squadre, modalità e data iniettata', () => {
+        const striscia = [{ serv: 'h', ris: '', vinc: null }]
+        const state = statoConSet(striscia)
+        state.modalitaPartita = '2/3'
+        const html = buildRefertoHtml(state, NOW)
+        expect(html).toContain('Antoniana')
+        expect(html).toContain('Rivali')
+        expect(html).toContain('Modalità: 2/3')
+        expect(html).toContain('14/03/2026')
+    })
+})

tests/unit/server-utils.test.js

@@ -1,14 +1,9 @@
 import { describe, it, expect, vi, afterEach } from 'vitest'
-import * as os from 'os'
+import { getNetworkIPs, collectIPs, printServerInfo } from '../../src/server-utils.js'
 
-vi.mock('os', async (importOriginal) => {
+// Nota: gli IP vengono iniettati nei test (oggetto stile os.networkInterfaces o
-    return {
+// array di IP), così i risultati sono deterministici su qualsiasi piattaforma —
-        ...await importOriginal(),
+// incluso WSL, dove getNetworkIPs() userebbe altrimenti PowerShell.
-        networkInterfaces: vi.fn(() => ({}))
-    }
-})
-
-import { getNetworkIPs, printServerInfo } from '../../src/server-utils.js'
 
 describe('Server Utils', () => {
 
@@ -17,85 +12,73 @@ describe('Server Utils', () => {
     })
 
     // =============================================
-    // getNetworkIPs
+    // getNetworkIPs / collectIPs
     // =============================================
     describe('getNetworkIPs', () => {
         it('dovrebbe restituire indirizzi IPv4 non-loopback', () => {
-            os.networkInterfaces.mockReturnValue({
+            expect(getNetworkIPs({
-                eth0: [
+                eth0: [{ family: 'IPv4', internal: false, address: '192.168.1.100' }]
-                    { family: 'IPv4', internal: false, address: '192.168.1.100' }
+            })).toEqual(['192.168.1.100'])
-                ]
-            })
-            expect(getNetworkIPs()).toEqual(['192.168.1.100'])
         })
 
         it('dovrebbe escludere indirizzi loopback (internal)', () => {
-            os.networkInterfaces.mockReturnValue({
+            const ips = getNetworkIPs({
-                lo: [
+                lo: [{ family: 'IPv4', internal: true, address: '127.0.0.1' }],
-                    { family: 'IPv4', internal: true, address: '127.0.0.1' }
+                eth0: [{ family: 'IPv4', internal: false, address: '192.168.1.100' }]
-                ],
-                eth0: [
-                    { family: 'IPv4', internal: false, address: '192.168.1.100' }
-                ]
             })
-            const ips = getNetworkIPs()
             expect(ips).not.toContain('127.0.0.1')
             expect(ips).toContain('192.168.1.100')
         })
 
         it('dovrebbe escludere indirizzi IPv6', () => {
-            os.networkInterfaces.mockReturnValue({
+            const ips = getNetworkIPs({
                 eth0: [
                     { family: 'IPv6', internal: false, address: 'fe80::1' },
                     { family: 'IPv4', internal: false, address: '192.168.1.100' }
                 ]
             })
-            const ips = getNetworkIPs()
             expect(ips).toEqual(['192.168.1.100'])
         })
 
         it('dovrebbe escludere bridge Docker 172.17.x.x', () => {
-            os.networkInterfaces.mockReturnValue({
+            const ips = getNetworkIPs({
-                docker0: [
+                docker0: [{ family: 'IPv4', internal: false, address: '172.17.0.1' }],
-                    { family: 'IPv4', internal: false, address: '172.17.0.1' }
+                eth0: [{ family: 'IPv4', internal: false, address: '10.0.0.5' }]
-                ],
-                eth0: [
-                    { family: 'IPv4', internal: false, address: '10.0.0.5' }
-                ]
             })
-            const ips = getNetworkIPs()
             expect(ips).not.toContain('172.17.0.1')
             expect(ips).toContain('10.0.0.5')
         })
 
         it('dovrebbe escludere bridge Docker 172.18.x.x', () => {
-            os.networkInterfaces.mockReturnValue({
+            expect(getNetworkIPs({
-                br0: [
+                br0: [{ family: 'IPv4', internal: false, address: '172.18.0.1' }]
-                    { family: 'IPv4', internal: false, address: '172.18.0.1' }
+            })).toEqual([])
-                ]
         })
-            expect(getNetworkIPs()).toEqual([])
+
+        it('dovrebbe escludere indirizzi link-local 169.254.x.x', () => {
+            expect(getNetworkIPs({
+                eth0: [{ family: 'IPv4', internal: false, address: '169.254.1.1' }]
+            })).toEqual([])
         })
 
         it('dovrebbe restituire array vuoto se nessuna interfaccia disponibile', () => {
-            os.networkInterfaces.mockReturnValue({})
+            expect(getNetworkIPs({})).toEqual([])
-            expect(getNetworkIPs()).toEqual([])
         })
 
         it('dovrebbe restituire più indirizzi da interfacce diverse', () => {
-            os.networkInterfaces.mockReturnValue({
+            const ips = getNetworkIPs({
-                eth0: [
+                eth0: [{ family: 'IPv4', internal: false, address: '192.168.1.100' }],
-                    { family: 'IPv4', internal: false, address: '192.168.1.100' }
+                wlan0: [{ family: 'IPv4', internal: false, address: '192.168.1.101' }]
-                ],
-                wlan0: [
-                    { family: 'IPv4', internal: false, address: '192.168.1.101' }
-                ]
             })
-            const ips = getNetworkIPs()
             expect(ips).toHaveLength(2)
             expect(ips).toContain('192.168.1.100')
             expect(ips).toContain('192.168.1.101')
         })
+
+        it('collectIPs gestisce input vuoto/undefined senza errori', () => {
+            expect(collectIPs()).toEqual([])
+            expect(collectIPs({})).toEqual([])
+        })
     })
 
     // =============================================
@@ -103,9 +86,8 @@ describe('Server Utils', () => {
     // =============================================
     describe('printServerInfo', () => {
         it('dovrebbe stampare la porta di default (3000)', () => {
-            os.networkInterfaces.mockReturnValue({})
             const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
-            printServerInfo()
+            printServerInfo(3000, [])
             const allLogs = consoleSpy.mock.calls.map(c => c[0]).join('\n')
             expect(allLogs).toContain('3000')
             expect(allLogs).toContain('/display')
@@ -114,22 +96,16 @@ describe('Server Utils', () => {
         })
 
         it('dovrebbe stampare la porta personalizzata', () => {
-            os.networkInterfaces.mockReturnValue({})
             const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
-            printServerInfo(8080)
+            printServerInfo(8080, [])
             const allLogs = consoleSpy.mock.calls.map(c => c[0]).join('\n')
             expect(allLogs).toContain('8080')
             consoleSpy.mockRestore()
         })
 
         it('dovrebbe mostrare gli URL remoti se ci sono IP di rete', () => {
-            os.networkInterfaces.mockReturnValue({
-                eth0: [
-                    { family: 'IPv4', internal: false, address: '192.168.1.50' }
-                ]
-            })
             const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
-            printServerInfo(3000)
+            printServerInfo(3000, ['192.168.1.50'])
             const allLogs = consoleSpy.mock.calls.map(c => c[0]).join('\n')
             expect(allLogs).toContain('192.168.1.50')
             expect(allLogs).toContain('remoti')
@@ -137,9 +113,8 @@ describe('Server Utils', () => {
         })
 
         it('non dovrebbe mostrare sezione remoti se nessun IP di rete', () => {
-            os.networkInterfaces.mockReturnValue({})
             const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
-            printServerInfo(3000)
+            printServerInfo(3000, [])
             const allLogs = consoleSpy.mock.calls.map(c => c[0]).join('\n')
             expect(allLogs).not.toContain('remoti')
             consoleSpy.mockRestore()