[jbtronics]

Summary

This PR improves the extension's host permission handling by refactoring permission requests to be batch-processed, adding better error detection for missing permissions, and providing more actionable error messages to users.

Key Changes

• Refactored permission request logic: Renamed requestHostPermission() to requestHostPermissions() to accept multiple URLs at once, reducing redundant permission requests when saving multiple hosts. The function now deduplicates origins and requests them in a single chrome.permissions.request() call.

• Added host permission utilities: Created hostPermissionPattern() and hasHostPermission() functions in host-storage.js to centralize host permission pattern generation and checking. The pattern deliberately omits ports since Firefox doesn't support them in match patterns.

• Improved error detection: Added networkErrorResult() helper in the service worker that detects when network errors are caused by missing host permissions (rather than actual network failures) and returns a specific missing_host_permission error code.

• Better error messages:

  • Added user-friendly error message when host permissions are missing, guiding users to re-grant permissions via settings
  • Added error message for permission request failures with the underlying error details
  • Updated popup error handling to display the new missing permission error message

• Updated documentation: Clarified in README that host permissions are scoped to hosts (all ports) rather than specific URLs, explaining Firefox's port limitation.

Implementation Details

• Permission requests are now synchronized from user input handlers before any await statements, as required by Firefox's permission API
• Origins are deduplicated using Set to avoid redundant permission requests
• Error handling includes try-catch for the chrome.permissions.request() call to gracefully handle permission API failures
• The batch permission request in saveAllSettings() only executes if there are new hosts to add