Repository navigation
feat: controlli di coerenza 2 — scaletta, stessa versione, allegati richiesti, clausole del rider (S14) - #82
Merged
Merged
Conversation
… stessa versione cambiata, allegati richiesti, clausole del rider cambiate nel «Cosa cambia» Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…agno); test e mutazioni
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Controlli di coerenza 2 — specifica S14
Regole nuove, tutte non bloccanti, nella striscia «Prima di esportare» (stesso meccanismo di S1/#61):
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-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-da-chiedere): elenco libero nell'Esporta (nome + stato «da chiedere» / «ricevuto», clic per alternare, ✕ per togliere, max 30), campoallegatiRichiestidel documento, solo se l'utente lo usa; se resta qualcosa «da chiedere» l'avviso lo elenca.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
--aggiornae media avvisi invariata a 0,17 (soglia < 1). Mutazioninote/mutazioni_S14_coerenza2.py: 44 prove, tutte rosse tranne 3 equivalenti per costruzione (stringa vsString()di una clausola già testuale;if(_vN)e_voce.rcoperti datrye dah). 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