Skip to content

feat: controlli di coerenza 2 — scaletta, stessa versione, allegati richiesti, clausole del rider (S14) - #82

Merged
castelsim merged 5 commits into
mainfrom
coerenza2-13-11
Nov 13, 2026
Merged

castelsim merged 5 commits into
mainfrom
coerenza2-13-11

Conversation

@castelsim

Copy link
Copy Markdown
Owner

Controlli di coerenza 2 — specifica S14

Regole nuove, tutte non bloccanti, nella striscia «Prima di esportare» (stesso meccanismo di S1/#61):

  • Data nella scaletta ≠ data dell'evento (data-nella-scaletta): una data completa nel titolo o nelle note di un brano diversa dalla data evento («La scaletta dice 10/11/2024 ma la data dell'evento è 01/12/2026»). La regola sul titolo (data-nel-titolo) esisteva già in S1 (la mia nota in § 9 del piano era sbagliata: corretta). Provata sui progetti veri: l'unico caso è Mastrotto (scaletta di un'edizione precedente, 22/12/2025 su un evento del 30/10/2026), cioè proprio ciò che la regola deve cogliere.
  • Stessa versione già esportata, contenuto cambiato (stessa-versione-cambiata): se la vN è già nello storico (feat: differenze fra versioni copiabili nella mail (S10) #75) e da allora sono cambiati documenti (stage plot, channel list, output list, rider, organico, backline, lista luci, scaletta — via le impronte per documento di feat: «cambiato / invariato da vN-1» accanto a ogni documento del pacchetto #77) lo dice, nominando i documenti: «La v3 è già stata esportata e da allora sono cambiati: stage plot, channel list. Chi ha il file della v3 ha un contenuto diverso con lo stesso nome: aumenta la versione». Niente avviso con versione nuova, senza storico, o per documenti che nell'export precedente non c'erano.
  • Allegati richiesti (allegati-da-chiedere): elenco libero nell'Esporta (nome + stato «da chiedere» / «ricevuto», clic per alternare, ✕ per togliere, max 30), campo allegatiRichiesti del documento, solo se l'utente lo usa; se resta qualcosa «da chiedere» l'avviso lo elenca.
  • Clausole del rider cambiate nel riquadro «Cosa cambia da vN a vM» (S10): il riassunto dello storico porta anche le impronte delle clausole di testo del rider; il riquadro aggiunge «Clausole del rider cambiate: luci (aggiunta), sistema audio» (anche «tolta»); niente se i riassunti vecchi non le hanno.

Non fatto di proposito: il controllo date sul testo del rider. Provato su Valdagno v2: il rider cita volutamente la data dell'edizione precedente («sullo stage plot di Valdagno 22/12/2025») e la regola dava un falso allarme: tolta (la specifica stessa segnava il rischio).

Prove: motori 2118, Orchestre 124, collaudo 30/30 senza --aggiorna e media avvisi invariata a 0,17 (soglia < 1). Mutazioni note/mutazioni_S14_coerenza2.py: 44 prove, tutte rosse tranne 3 equivalenti per costruzione (stringa vs String() di una clausola già testuale; if(_vN) e _voce.r coperti da try e da h). Browser vero (copia Valdagno v2): aggiunta di un allegato dall'interfaccia → avviso «Allegato ancora da chiedere: Planimetria del teatro»; clic sullo stato → avviso sparito; v2 esportata poi modificata → «La v2 è già stata esportata e da allora sono cambiati: stage plot, channel list…».

Domande per Simone: (1) l'avviso «stessa versione cambiata» scatta anche mentre si lavora su una bozza già esportata una volta: va bene, o preferisci che compaia solo dopo l'invio al service? (2) manca la regola «allegato citato ma assente» sul testo libero (non la faccio senza un elenco da cui partire).

Merge: gh pr merge N --repo castelsim/stageplot --admin --merge.

🤖 Generated with Claude Code

simone castellan and others added 5 commits November 13, 2026 16:03
… stessa versione cambiata, allegati richiesti, clausole del rider cambiate nel «Cosa cambia»

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ne e blocchi di test tenuti entrambi

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@castelsim
castelsim merged commit b998427 into main Nov 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant