CodeCow/clients
1
0
Fork 0
mirror of https://github.com/bitwarden/clients.git synced 2026-06-04 21:04:29 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity
main
clients/apps/browser/CLAUDE.md
Jonathan Prusik aaf239a2c5
[PM-33568] Add Claude guidance on injected content script exceptions (#19559) 
* add Claude guidance on injected content script exceptions

* update CODEOWNERS

* update CODEOWNERS

* remove high-level Claude guidance

* update CODEOWNERS

* execute PR feedback

* include all autofill content files in the rule
2026-03-18 12:04:29 -04:00

1.1 KiB
Raw Permalink Blame History

Browser Extension - Critical Rules

  • NEVER use chrome.* or browser.* APIs directly in business logic

    • Always use BrowserApi abstraction: /apps/browser/src/platform/browser/browser-api.ts
    • Required for cross-browser compatibility (Chrome/Firefox/Safari/Opera)
    • Exception: injected content scripts (see .claude/rules/autofill-content-scripts.md)

  • ALWAYS use BrowserApi.addListener() for event listeners in popup context

    • Safari requires manual cleanup to prevent memory leaks
    • DON'T use native chrome.*.addListener() or browser.*.addListener() directly

  • CRITICAL: Safari has tab query bugs

    • Use BrowserApi.tabsQueryFirstCurrentWindowForSafari() when querying current window tabs
    • Safari can return tabs from multiple windows incorrectly

Manifest V3

  • Extension uses Web Extension API Manifest V3
  • Service workers replace background pages
    • Background context runs as service worker (can be terminated anytime)
    • DON'T assume background page persists indefinitely
    • Use message passing for communication between contexts
    • chrome.extension.getBackgroundPage() returns null in MV3