Skip to content

feat(logs): share every matching log entry as a file - #1334

Merged
everpcpc merged 2 commits into
mainfrom
feat/export-all-logs
Nov 10, 2026
Merged

everpcpc merged 2 commits into
mainfrom
feat/export-all-logs

Conversation

@everpcpc

Copy link
Copy Markdown
Member

Why

Sharing from Settings → Logs only included the rows the list had already paged in, 50 at a time. With tens of thousands of entries, getting a useful export meant scrolling through the whole list first. This came up in #1330, where the attached log stopped minutes before the error we needed to see.

What changes

The Share action on the logs page now exports every entry that matches the current level, category, and search filter, read directly from the log store rather than from the loaded rows.

  • The export is shared as a text file (KMReader-logs-<timestamp>.txt) instead of an inline string.
  • Entries are written oldest first; the previous export was newest first.
  • The file is generated when the share is carried out and written in chunks, so a large log is not held in memory as one string.

No new UI strings.

Validation

  • make build-macos, make build-ios, make build-tvos pass.
  • The share flow itself has not been exercised in the running app yet (the system share picker could not be driven from here); it needs a manual check on iOS and macOS before merging.

Includes build 686.

Related to #1330.

🤖 Generated with Claude Code

Sharing from the logs page only included the rows the list had paged in, 50 at a time, so exporting a large log meant scrolling through all of it first.

The share action now reads every entry matching the current level, category, and search filter from the log store and writes them to a text file, oldest first. The file is produced when the share is carried out and written in chunks.
@everpcpc
everpcpc marked this pull request as ready for review November 10, 2026 18:24
@everpcpc
everpcpc merged commit 14c1c9c into main Nov 10, 2026
3 checks passed
@everpcpc
everpcpc deleted the feat/export-all-logs branch November 10, 2026 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant