[marcoarnulfo]

🇬🇧 English

Summary

Closes #37.

--body-file replaces: the new blocks are appended, then every old one is deleted. loadBody's empty check is the only thing between a file with no content and a body that is gone with nothing in its place — and it was strings.TrimSpace, which does not strip U+FEFF (it left Unicode's White_Space in 6.3).

A BOM-only file therefore read as content. The chain from there, reproduced in a test:

GET  /v1/blocks/page1/children     ← snapshot
DELETE /v1/blocks/old1             ← and nothing written between them

goldmark returns zero blocks, ValidateAppendable ranges over an empty slice and returns nil, splitIntoRequests emits no batch so no PATCH goes out at all, and replaceBody deletes every non-sub-page child anyway. Exit 0.

#36 fixed the same bypass on the append path, where it costs a false appended:true. On the replace path it costs the body. isBlank already existed from that fix; this reuses it.

Two more things, same failure

• The twin hole after --expand. loadAppendBody re-checks emptiness after expanding; loadBody did not. A file of nothing but {{ticket}} addressed by --page-id expanded to nothing and took the same path.
• replaceBody now refuses a request with no blocks, whatever the caller passed. Both guards above are argument handling inside one command, and the content they protect is not recoverable — so the property belongs next to the deleting, not two packages away from it.

Also fixes the SKILL.md clause that #36 made stale: the over-limit message names a request size only when it built one, and a floor on the file size when it did not.

Type of change

• Bug fix
• New feature
• Docs
• Refactor / chore

Checklist

• go test ./... -race passes
• gofmt, go vet, staticcheck are clean
• Both README.md and README.it.md updated if behavior changed (no user-facing README claim changed)
• Commit messages follow Conventional Commits with a scope

---

🇮🇹 Italiano

Riepilogo

Chiude la #37.

--body-file sostituisce: scrive i blocchi nuovi e poi cancella tutti i vecchi. Il controllo sul vuoto in loadBody è l'unica cosa fra un file senza contenuto e un corpo sparito senza nulla al suo posto — ed era strings.TrimSpace, che non toglie U+FEFF (uscito da White_Space in Unicode 6.3).

Un file di solo BOM passava quindi per contenuto. La catena, riprodotta in un test: goldmark produce zero blocchi, ValidateAppendable itera su una slice vuota e ritorna nil, splitIntoRequests non emette batch quindi nessuna PATCH parte, e replaceBody cancella comunque ogni figlio che non sia una sub-page. Exit 0.

La #36 ha corretto lo stesso bypass sul path append, dove costa un appended:true falso. Sul path replace costa il corpo della pagina. isBlank esisteva già da quella fix: qui viene riusata.

Altre due cose, stesso guasto

• Il buco gemello dopo --expand. loadAppendBody ricontrolla il vuoto dopo l'espansione, loadBody no. Un file di solo {{ticket}} indirizzato con --page-id si espandeva a nulla e prendeva la stessa strada.
• replaceBody ora rifiuta una richiesta senza blocchi, qualunque cosa passi il chiamante. Le due guardie sopra vivono nella gestione degli argomenti di un solo comando, e il contenuto che proteggono non è recuperabile: la proprietà sta bene accanto al codice che cancella, non due package più in là.

Corretta anche la subordinata di SKILL.md resa obsoleta dalla #36: il messaggio di superamento limite nomina la dimensione della richiesta solo quando l'ha costruita, e un pavimento sulla dimensione del file quando non l'ha fatto.