Skip to content

Fix NotificationCenter DB path on newer macOS#820

Open
franklinho wants to merge 1 commit into
BlueBubblesApp:masterfrom
franklinho:codex/notificationcenter-usernoted-path
Open

Fix NotificationCenter DB path on newer macOS#820
franklinho wants to merge 1 commit into
BlueBubblesApp:masterfrom
franklinho:codex/notificationcenter-usernoted-path

Conversation

@franklinho

Copy link
Copy Markdown

Summary

  • Prefer the newer macOS group.com.apple.usernoted NotificationCenter database path when it exists.
  • Fall back to the legacy com.apple.notificationcenter database path.
  • Await the existing async DB existence check so startup reports missing notification DBs correctly.

Testing

  • Validated locally on macOS 26 with the installed BlueBubbles server ASAR using the usernoted DB path.
  • Ran git diff --check on the patch branch.

@zlshames zlshames left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Edit: nvm

@zlshames

Copy link
Copy Markdown
Member

Wait actually... can you verify one thing for me. Is the schema of the new location the same as the old one? Also, why is this still a draft?

@franklinho

franklinho commented Jul 19, 2026

Copy link
Copy Markdown
Author

yes, the new path has the same schema the server expects.
New DB path checked:
~/Library/Group Containers/group.com.apple.usernoted/db2/db
It contains:
CREATE TABLE record (
rec_id INTEGER PRIMARY KEY,
app_id INTEGER,
uuid BLOB,
data BLOB,
request_date REAL,
request_last_date REAL,
delivered_date REAL,
presented Bool,
style INTEGER,
snooze_fire_date REAL
);

CREATE TABLE app (
app_id INTEGER PRIMARY KEY,
identifier VARCHAR,
badge INTEGER NULL
);
That matches the server’s TypeORM entities:
Record.ts maps rec_id, app_id, uuid, data, request_date, request_last_date, delivered_date, presented, style, snooze_fire_date
App.ts maps app_id, identifier, badge

@franklinho
franklinho marked this pull request as ready for review July 19, 2026 00:19
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.

2 participants