Skip to content

feat: import and map native browser bookmarks - #10

Merged
WangEn merged 14 commits into
mainfrom
feat/native-bookmark-import
Sep 12, 2026
Merged

feat: import and map native browser bookmarks#10
WangEn merged 14 commits into
mainfrom
feat/native-bookmark-import

Conversation

@WangEn

@WangEn WangEn commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

What changed

  • move Chromium bookmarks access from required install-time permission to optional runtime permission
  • bump Dockmark Extension to 0.2.0
  • expose native bookmark capability + granted-permission state in the Browser Bridge handshake
  • read the browser bookmark tree through the trusted Web/Extension bridge, preserving folder paths
  • add reviewed native bookmark import to the Transfer page
  • de-duplicate same-URL browser bookmarks into one Dockmark bookmark
  • map native bookmark IDs to Dockmark bookmark IDs in extension-local storage only
  • automatically prune mappings when a native bookmark is removed or its URL changes
  • allow already-existing Dockmark URLs to be linked without creating duplicates
  • keep native bookmark import read-only: no browser bookmark create/update/move/delete operations
  • update Chromium smoke coverage for extension 0.2.0 and optional bookmark permission semantics

Mapping boundary

browserBookmarkId ↔ dockmarkBookmarkId lives only in browser.storage.local; browser-internal IDs are not written to D1. This PR intentionally does not implement two-way sync or destructive native bookmark mutation.

@WangEn
WangEn merged commit a9a6d2e into main Sep 12, 2026
2 checks passed
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