enhancement(logging): Improve rotation retention beyond KeepOne #160

Closed
opened 2026-08-31 00:07:58 +01:00 by gronod · 0 comments
Owner

Description

Rotation is currently RotationStrategy::KeepOne — only one rotated backup. Heavy chartread sessions or multi-day triage can lose history quickly.

Scope

  1. Review tauri-plugin-log v2 options (KeepAll, size limits, retained file count).
  2. Retain several segments (e.g. last 3–5) and/or rotate on size (e.g. 5–10 MB), with a hard cap so disk use stays bounded.
  3. Keep the active file name stable (iccery.log) so Copy Path remains correct.
  4. One-line code comment or Settings/help note describing retention.

Acceptance Criteria

  • More than one historical segment can exist after heavy use.
  • get_log_path still points at the active file.
  • Retention is bounded (no unbounded growth).

Dependencies

  • None.

Filing order

  1. Ticket 1 — bug; Settings control is wrong today.
  2. Ticket 2 — supportability bundle (frontend + redaction + excerpt).
  3. Ticket 3 — operational polish; ship when convenient.

Already done (do not re-file)

Item Reference
tauri-plugin-log + file under app_log_dir #139 / PR #142
Subprocess stdout/stderr with process id process_manager.rs
get_log_path / open_log_dir commands.rs
Settings: level dropdown, open folder, copy path settings.js + index.html
log_level field persistence settings.rs (apply = Ticket 1)
### Description Rotation is currently `RotationStrategy::KeepOne` — only one rotated backup. Heavy `chartread` sessions or multi-day triage can lose history quickly. ### Scope 1. Review `tauri-plugin-log` v2 options (KeepAll, size limits, retained file count). 2. Retain several segments (e.g. last 3–5) and/or rotate on size (e.g. 5–10 MB), with a hard cap so disk use stays bounded. 3. Keep the active file name stable (`iccery.log`) so **Copy Path** remains correct. 4. One-line code comment or Settings/help note describing retention. ### Acceptance Criteria - [ ] More than one historical segment can exist after heavy use. - [ ] `get_log_path` still points at the active file. - [ ] Retention is bounded (no unbounded growth). ### Dependencies - None. --- ## Filing order 1. **Ticket 1** — bug; Settings control is wrong today. 2. **Ticket 2** — supportability bundle (frontend + redaction + excerpt). 3. **Ticket 3** — operational polish; ship when convenient. --- ## Already done (do not re-file) | Item | Reference | |------|-----------| | `tauri-plugin-log` + file under `app_log_dir` | #139 / PR #142 | | Subprocess stdout/stderr with process id | `process_manager.rs` | | `get_log_path` / `open_log_dir` | `commands.rs` | | Settings: level dropdown, open folder, copy path | `settings.js` + `index.html` | | `log_level` field persistence | `settings.rs` (apply = Ticket 1) |
gronod added the Kind/Enhancement
Reviewed
Confirmed
1
Priority
Low
4
labels 2026-08-31 00:07:58 +01:00
gronod added this to the Milestone 10: Production Ready Features (v0.6.0) milestone 2026-08-31 00:08:15 +01:00
Sign in to join this conversation.