[Feature] Say so when a sidecar is skipped, instead of only logging it #564

New issue

Closed

opened 2026-07-22 15:10:01 +00:00 by brenno · 0 comments

brenno commented

2026-07-22 15:10:01 +00:00

Owner

Since #560 the four deck sidecars (.ink.json, .user-notes.json, .miauw.json, .seal.json) are read behind a 16 MiB cap. Above it the layer is skipped and the write path leaves the file untouched, so nothing is destroyed — but the user is told nothing. Their ink simply is not there, and the only trace is a line in the log.

That is the right trade for safety and the wrong one for the person looking at their deck. It came out of the guardian review on #560 and was deliberately not folded into that change: making it visible needs its own warning channel, and the existing one is ChartDataWarning, which would show the wrong sentence.

Cost: a warning channel from openDeckDetailed alongside the chart-data one, plus one l10n string × 31 languages.

Reachability, honestly: 16 MiB of ink is hours of continuous annotation, so this is not a common case. It is filed because "the user loses work silently" should not depend on how unlikely we think it is.

Since #560 the four deck sidecars (`.ink.json`, `.user-notes.json`, `.miauw.json`, `.seal.json`) are read behind a 16 MiB cap. Above it the layer is skipped and the write path leaves the file untouched, so nothing is destroyed — but the user is told nothing. Their ink simply is not there, and the only trace is a line in the log. That is the right trade for safety and the wrong one for the person looking at their deck. It came out of the guardian review on #560 and was deliberately not folded into that change: making it visible needs its own warning channel, and the existing one is `ChartDataWarning`, which would show the wrong sentence. Cost: a warning channel from `openDeckDetailed` alongside the chart-data one, plus one l10n string × 31 languages. Reachability, honestly: 16 MiB of ink is hours of continuous annotation, so this is not a common case. It is filed because "the user loses work silently" should not depend on how unlikely we think it is.

brenno added the

enhancement

label

2026-07-22 15:10:01 +00:00

brenno referenced this issue

2026-07-22 15:10:16 +00:00

[Feature] Security hardening: nine items below the blocking layer #516

brenno referenced this issue from a commit

2026-07-22 16:06:19 +00:00

feat(sidecar): meld een overgeslagen laag aan de gebruiker

brenno referenced this issue from a pull request that will close it,

2026-07-22 16:06:34 +00:00