Adriano/PythagorasGoal
1
0
Fork 0
Code Issues 4 Pull Requests Actions Packages Projects Releases Wiki Activity

analista --secco dichiara 'non salva' ma riscrive la voce del giornale nel DB #34

New Issue
Closed
opened 2026-09-27 13:12:14 +00:00 by Adriano · 3 comments
Adriano commented 2026-09-27 13:12:14 +00:00
Owner
Copy Link
Copy Source

Revisione 2026-09-27 (goldrake): rilievo Opus 5.5, verifica avversariale fable. Sola lettura, nessun codice toccato.

Rilievo C6 [bug]

FILE: PG:scripts/live/analista.py:75-76 (vs 98-100 e USO riga 123)
COSA: --secco («stampa e NON salva») esegue J.salva(con, voce) PRIMA del ramo secco: riscrive la riga journal del giorno (mercato/libro/pnl/salute/ts_scritto).
SCENARIO: analista.py --secco lanciato alle 12:00 sul giorno in corso crea/aggiorna nel DB una voce PARZIALE di oggi; su un giorno chiuso ne cambia ts_scritto e i numeri ricalcolati (es. dopo una dichiarazione di movimento) senza riscrivere il .md -> DB e pagina divergono. CLAUDE.md §13 lo presenta come "senza salvare".
PROVA: VERIFICATO (lettura): riga 76 J.salva(con, voce) incondizionata, if secco: … return solo a riga 98.
PROPOSTA: con secco non chiamare salva (la pagina per il prompt si rende gia' da voce; rendi_markdown legge dal DB solo nota/analisi).

Verifica fable C6 VERDETTO: CONFERMATO

GRAVITA': medio
PERCHE': scripts/live/analista.py:75-76 voce = J.costruisci(...); J.salva(con, voce) incondizionato; if secco: ... return a riga 98. journal.salva (src/live/journal.py:400-415) fa INSERT ... ON CONFLICT DO UPDATE di ts_scritto/mercato/libro/pnl/salute (preserva nota e analisi). Con --secco sul giorno in corso crea una riga PARZIALE; su un giorno chiuso aggiorna il DB senza scrivi_file -> DB e .md divergono. USO (riga 123) dice «stampa e NON salva».
CORREZIONE: ok (con secco non chiamare salva; rendi_markdown legge dal DB solo nota/analisi, la pagina si rende da voce).

Revisione 2026-09-27 (goldrake): rilievo Opus 5.5, verifica avversariale fable. Sola lettura, nessun codice toccato. ### Rilievo C6 [bug] FILE: PG:scripts/live/analista.py:75-76 (vs 98-100 e USO riga 123) COSA: `--secco` («stampa e NON salva») esegue `J.salva(con, voce)` PRIMA del ramo secco: riscrive la riga `journal` del giorno (mercato/libro/pnl/salute/ts_scritto). SCENARIO: `analista.py --secco` lanciato alle 12:00 sul giorno in corso crea/aggiorna nel DB una voce PARZIALE di oggi; su un giorno chiuso ne cambia `ts_scritto` e i numeri ricalcolati (es. dopo una dichiarazione di movimento) senza riscrivere il .md -> DB e pagina divergono. CLAUDE.md §13 lo presenta come "senza salvare". PROVA: VERIFICATO (lettura): riga 76 `J.salva(con, voce)` incondizionata, `if secco: … return` solo a riga 98. PROPOSTA: con `secco` non chiamare `salva` (la pagina per il prompt si rende gia' da `voce`; `rendi_markdown` legge dal DB solo nota/analisi). ### Verifica fable C6 VERDETTO: CONFERMATO GRAVITA': medio PERCHE': `scripts/live/analista.py:75-76` `voce = J.costruisci(...); J.salva(con, voce)` incondizionato; `if secco: ... return` a riga 98. `journal.salva` (`src/live/journal.py:400-415`) fa `INSERT ... ON CONFLICT DO UPDATE` di ts_scritto/mercato/libro/pnl/salute (preserva nota e analisi). Con `--secco` sul giorno in corso crea una riga PARZIALE; su un giorno chiuso aggiorna il DB senza `scrivi_file` -> DB e .md divergono. USO (riga 123) dice «stampa e NON salva». CORREZIONE: ok (con `secco` non chiamare `salva`; `rendi_markdown` legge dal DB solo nota/analisi, la pagina si rende da `voce`).
Adriano added the bug label 2026-09-27 13:12:14 +00:00
Adriano commented 2026-09-27 13:17:49 +00:00
Author
Owner
Copy Link
Copy Source

ANALISI (passo 1, Opus) — unita' U4, ramo lavoro/goldrake-u4, repo gitea:Adriano/PythagorasGoal
RIF: #27 (C1), #32 (C2, C10), #34 (C6), #35 (solo C7, C9, C15)
FILE: src/live/journal.py, src/live/tradesdb.py, scripts/live/analista.py, tests/

  1. #27 — journal.py ~246-252 (movimenti dichiarati): gia cerca un evento per ts_dopo senza filtrare fonte, il secondo dichiarato nello stesso intervallo sovrascrive il primo. Nuovo: cercare solo eventi con fonte == "rilevato" per l'aggancio "dichiarato+rilevato"; se esiste gia' un evento DICHIARATO nello stesso intervallo, SOMMARE i delta e concatenare le dichiarazioni (fonte resta "dichiarato"). Test: +25 e +30 dichiarati nello stesso intervallo, trading reale +5 -> trading risulta +5 e TWR coerente; un dichiarato che coincide con un salto rilevato -> "dichiarato+rilevato" come oggi.
  2. #32 C2 — tradesdb.fifo_roundtrips (~264-290): lotti con segno. Un sell senza lotti long apre un lotto SHORT; un buy chiude prima i lotti short (P&L = (entry - exit) * qty) e l'eccedenza apre long; inversione long->short in un solo fill: chiude i long e apre short con il resto. Nessun resto scartato in silenzio. Leggi tutti i chiamanti (reconcile, stato_aperto, report) e adegua dove leggono i lotti (segno/direzione). Test: long->chiuso (invariato, bit-exact con i test esistenti), short aperto e chiuso, inversione in un fill.
    #32 C10 — fill "NON VERIFICATO" a qty 0 (place_order error): nel parser (o subito dopo) i fill con qty <= 0 NON entrano nel FIFO ne' in stato_aperto; restano nel DB marcati (colonna esistente verified=0; se serve un campo nuovo, niente migrazioni distruttive: ALTER TABLE ADD COLUMN idempotente). Test.
    ⚠️ NON cambiare le regex di parse_cron_log ne' il formato che leggono (un'altra unita' lavora su book_execute e ne rispetta il formato).
  3. #34 — analista.py --secco: oggi chiama J.salva prima del return del secco. Nuovo: con --secco non si chiama salva ne' scrivi_file; la pagina si rende da voce (verificare che rendi_markdown funzioni senza la riga in DB; se legge nota/analisi dal DB, passarle vuote). Test: --secco su DB temporaneo -> DB invariato.
  4. #35 C9 — journal.py ~388: atteso = adesso.hour + 1 conta il giro dell'ora corrente prima del :47. Nuovo: adesso.hour + (1 if adesso.minute >= MINUTO_GIRO + 1 else 0) con MINUTO_GIRO = 47 costante di modulo commentata "deve restare d'accordo con la crontab e scripts/cron_book.sh". Test ai minuti 10 e 50.
    #35 C15 — finestra "ultimi 7 giorni" (~594-597) copre 8 giorni di calendario: baseline = ultima lettura <= giorno-7 fine giornata, oppure la finestra dichiarata nel testo come e' davvero: scegli la correzione che rende VERO il testo e dillo. Test.
    #35 C7 — drawdown (~596-615) sulla serie grezza: un prelievo certo diventa drawdown, un versamento alza il picco. Calcolare il DD sulla serie al netto dei movimenti certi (gli stessi che la pagina gia' scorpora per il TWR). Test con versamento e con prelievo.

TEST: nuovi in tests/test_journal_u4.py, tests/test_tradesdb_u4.py; DB sqlite temporanei (tmp_path). Suite completa: cd <worktree> && /opt/docker/PythagorasGoal/.venv/bin/python -m pytest -q -p no:cacheprovider (baseline 1098 passed, 3 skipped).

NON TOCCARE: parse_cron_log e le sue regex; scripts/live/book_execute.py; data/ (nessun DB vero: non esiste nel worktree e non va creato); /opt/docker/PythagorasGoal. Nessuna rete, nessun Telegram.

ANALISI (passo 1, Opus) — unita' U4, ramo lavoro/goldrake-u4, repo gitea:Adriano/PythagorasGoal RIF: #27 (C1), #32 (C2, C10), #34 (C6), #35 (solo C7, C9, C15) FILE: src/live/journal.py, src/live/tradesdb.py, scripts/live/analista.py, tests/ 1. #27 — journal.py ~246-252 (movimenti dichiarati): `gia` cerca un evento per ts_dopo senza filtrare fonte, il secondo dichiarato nello stesso intervallo sovrascrive il primo. Nuovo: cercare solo eventi con fonte == "rilevato" per l'aggancio "dichiarato+rilevato"; se esiste gia' un evento DICHIARATO nello stesso intervallo, SOMMARE i delta e concatenare le dichiarazioni (fonte resta "dichiarato"). Test: +25 e +30 dichiarati nello stesso intervallo, trading reale +5 -> trading risulta +5 e TWR coerente; un dichiarato che coincide con un salto rilevato -> "dichiarato+rilevato" come oggi. 2. #32 C2 — tradesdb.fifo_roundtrips (~264-290): lotti con segno. Un sell senza lotti long apre un lotto SHORT; un buy chiude prima i lotti short (P&L = (entry - exit) * qty) e l'eccedenza apre long; inversione long->short in un solo fill: chiude i long e apre short con il resto. Nessun `resto` scartato in silenzio. Leggi tutti i chiamanti (reconcile, stato_aperto, report) e adegua dove leggono i lotti (segno/direzione). Test: long->chiuso (invariato, bit-exact con i test esistenti), short aperto e chiuso, inversione in un fill. #32 C10 — fill "NON VERIFICATO" a qty 0 (place_order error): nel parser (o subito dopo) i fill con qty <= 0 NON entrano nel FIFO ne' in stato_aperto; restano nel DB marcati (colonna esistente verified=0; se serve un campo nuovo, niente migrazioni distruttive: ALTER TABLE ADD COLUMN idempotente). Test. ⚠️ NON cambiare le regex di parse_cron_log ne' il formato che leggono (un'altra unita' lavora su book_execute e ne rispetta il formato). 3. #34 — analista.py --secco: oggi chiama J.salva prima del return del secco. Nuovo: con --secco non si chiama salva ne' scrivi_file; la pagina si rende da `voce` (verificare che rendi_markdown funzioni senza la riga in DB; se legge nota/analisi dal DB, passarle vuote). Test: --secco su DB temporaneo -> DB invariato. 4. #35 C9 — journal.py ~388: atteso = adesso.hour + 1 conta il giro dell'ora corrente prima del :47. Nuovo: `adesso.hour + (1 if adesso.minute >= MINUTO_GIRO + 1 else 0)` con MINUTO_GIRO = 47 costante di modulo commentata "deve restare d'accordo con la crontab e scripts/cron_book.sh". Test ai minuti 10 e 50. #35 C15 — finestra "ultimi 7 giorni" (~594-597) copre 8 giorni di calendario: baseline = ultima lettura <= giorno-7 fine giornata, oppure la finestra dichiarata nel testo come e' davvero: scegli la correzione che rende VERO il testo e dillo. Test. #35 C7 — drawdown (~596-615) sulla serie grezza: un prelievo certo diventa drawdown, un versamento alza il picco. Calcolare il DD sulla serie al netto dei movimenti certi (gli stessi che la pagina gia' scorpora per il TWR). Test con versamento e con prelievo. TEST: nuovi in tests/test_journal_u4.py, tests/test_tradesdb_u4.py; DB sqlite temporanei (tmp_path). Suite completa: `cd <worktree> && /opt/docker/PythagorasGoal/.venv/bin/python -m pytest -q -p no:cacheprovider` (baseline 1098 passed, 3 skipped). NON TOCCARE: parse_cron_log e le sue regex; scripts/live/book_execute.py; data/ (nessun DB vero: non esiste nel worktree e non va creato); /opt/docker/PythagorasGoal. Nessuna rete, nessun Telegram.
Adriano commented 2026-09-27 14:17:42 +00:00
Author
Owner
Copy Link
Copy Source

ESITO: fatto sul ramo, non in produzione
RIF: gitea:Adriano/PythagorasGoal#34
COSA: correzione su lavoro/goldrake (pushato), analisi Opus, codice Sonnet, controllo finale Opus
FILE: commit 0c19c80
PROVA: /home/adriano/lavoro/PythagorasGoal pytest -q → 1166 passed, 3 skipped
MANCA: merge su main — richiede ok esplicito di Adriano (libro vivo). Chiusura con --soluzione dopo il merge.
PROSSIMO: ok di Adriano → merge lavoro/goldrake su main → git pull in /opt/docker/PythagorasGoal

ESITO: fatto sul ramo, non in produzione RIF: gitea:Adriano/PythagorasGoal#34 COSA: correzione su lavoro/goldrake (pushato), analisi Opus, codice Sonnet, controllo finale Opus FILE: commit 0c19c80 PROVA: /home/adriano/lavoro/PythagorasGoal pytest -q → 1166 passed, 3 skipped MANCA: merge su main — richiede ok esplicito di Adriano (libro vivo). Chiusura con --soluzione dopo il merge. PROSSIMO: ok di Adriano → merge lavoro/goldrake su main → git pull in /opt/docker/PythagorasGoal
Adriano added the attesa:adriano label 2026-09-27 14:17:42 +00:00
Adriano commented 2026-09-27 14:28:00 +00:00
Author
Owner
Copy Link
Copy Source

Soluzione

scripts/live/analista.py: J.salva solo se non --secco.
COMMIT: 0c19c80, su main dal 27/09/2026 (merge con ok di Adriano). Produzione aggiornata.

**Soluzione** scripts/live/analista.py: J.salva solo se non --secco. COMMIT: 0c19c80, su main dal 27/09/2026 (merge con ok di Adriano). Produzione aggiornata.
Adriano removed the attesa:adriano label 2026-09-27 14:28:01 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
attesa:adriano
bug
Difetto trovato in revisione o in uso
 sviluppo
Sviluppo o miglioria da fare
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Adriano
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Adriano/PythagorasGoal#34
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?