Skip to content

Commit 16e8396

Browse files
committed
chore(diagnostics): ledger the background sync registration check and pin its field names
1 parent ae96cc7 commit 16e8396

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

docs/release-notes/diagnostics-ledger.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ Field names are checked against the log sanitizer by `packages/core/src/release-
3333
- **`v1.2.7/webdav-plaintext-degrade`** — both apps already log `WebDAV read returned no strong ETag; using the plaintext compatibility write` with the `etag` the server actually sent (`apps/mobile/lib/sync-service.ts`, `apps/desktop/src/lib/sync-service.ts`, 6ca1b0375).
3434
- **`v1.2.7/local-only-upload`**`packages/core/src/sync-run.ts` already logs `Sync local reconcile` with `reconcile: 'aligned-skip'` (and `'idle-cache'` when the carried idle snapshot spared the full local read).
3535
- **`v1.2.7/suspended-timeout-offline`** — the sync failure path logs `Sync failed` carrying the error text, which ends in `the request was interrupted while the app was suspended` (`SUSPENDED_REQUEST_MESSAGE`, core `http-utils.ts`).
36-
- **`v1.2.7/background-sync-registration`**owned by the background-sync worker, not by this pass. `Mobile background sync registered` with its interval already exists under **General**; the registration-decision line and its guard land in `apps/mobile/lib/background-sync-task.ts` separately. Add its bullet here when it ships, and add its field names to the sanitizer guard test.
36+
- **`v1.2.7/background-sync-registration`**added (67075324c). `apps/mobile/lib/background-sync-task.ts`, message `Mobile background sync registration checked` with `decision` (register | re-register | unchanged | unregister | deferred-until-foreground), `registered`, `storedInterval`, `interval`, `appState`. A tester's log must show `deferred-until-foreground` on a headless cold start and never a re-register there; `Mobile background sync registered` with its interval stays under General.
3737

3838
### Not added
3939

packages/core/src/release-diagnostics-fields.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ const RELEASE_CHECK_FIELD_NAMES = [
1717
'artifact', 'cloudProvider', 'scheme', 'host', 'delivery', 'deduped',
1818
'platform', 'total', 'multiDay', 'allDay', 'spanning',
1919
'presenceDue', 'hasScope', 'check', 'skipped',
20+
// background-sync-registration (apps/mobile/lib/background-sync-task.ts)
2021
'decision', 'registered', 'storedInterval', 'interval', 'appState',
2122
];
2223

0 commit comments

Comments
 (0)