Conversation
adipascu
force-pushed
the
save-watch-logs
branch
3 times, most recently
from
August 11, 2026 07:05
d7bc89a to
c3e7ea2
Compare
Contributor
|
This button already does this |
adipascu
force-pushed
the
save-watch-logs
branch
from
August 11, 2026 14:24
c3e7ea2 to
23015d2
Compare
adipascu
changed the title
Contributor
Author
|
You're right, dropped it. What's left is the daily auto-save, so logs outlive the watch's small buffer without manual pulls, and a fix for the zip being shared as an audio file. |
PlatformShareLauncher.share() was hardcoded to audio/wav from when recordings were its only payload. It gains a mimeType parameter, so each caller declares what it actually sends.
A developer collecting firmware logs over days has to remember to pull them before the watch's 128 KB log ring wraps, which loses data when they forget. When "Auto-save watch logs" is enabled the background sync gathers logs from the connected watch daily and keeps the newest 21 copies. Copies go to the phone's Downloads folder, under "Pebble watch logs", so they can be opened, shared or copied off without a cable. Android 10 and newer write through MediaStore. Older releases and iOS fall back to app storage. The toggle is a debug setting, off by default, so nothing changes for regular users.
adipascu
force-pushed
the
save-watch-logs
branch
from
September 8, 2026 02:16
23015d2 to
4ff28a6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The watch only keeps a couple of days of logs, and getting them off currently needs a person: submitting a bug report, or sharing the report zip from the Bug Report screen. The pipelines that run on their own carry crash dumps and numeric metrics, not logs. So a firmware contributor collecting log data over a longer stretch, without access to the server side, has to remember to pull logs every couple of days or silently lose the oldest ones.
This adds an Auto-save watch logs toggle to the debug settings. When it is on, the daily background sync also gathers the connected watch's logs and keeps the newest 21 copies in the phone's Downloads folder under
Pebble watch logs. If no watch is reachable it tries again on the next sync instead of skipping the day. Android 10 and newer write throughMediaStore. Older Android versions and iOS fall back to app storage.Off by default and hidden unless debug options are on, so nothing changes for regular users.
The other commit fixes the share sheet labeling the bug report zip as an audio file.
share()was hardcoded toaudio/wavfrom when recordings were its only payload, and now takes the type from the caller.Written with AI assistance (Claude), and exercised on a Pixel 6 with a Pebble Time 2.
Aside: I am currently available for mobile or firmware contracting or full-time work. Contact: adrian@pascu.be.