fix: extend permissions with BookmarksInfo and improve test coverage

Author: atldays

src/cli/builders/manifest/utils.test.ts

@@ -483,18 +483,18 @@ describe("normalizeDataCollectionPermissions", () => {
         expect(result.optional).toEqual([DataCollectionPermission.SearchTerms]);
     });
 
-    it("should handle mixed string and enum permissions", () => {
+    it("should handle mixed string and enum permissions including new BookmarksInfo", () => {
         const input = {
             required: ["websiteActivity" as any, DataCollectionPermission.SearchTerms],
-            optional: ["searchTerms" as any, "locationInfo" as any],
+            optional: ["searchTerms" as any, DataCollectionPermission.BookmarksInfo],
         };
 
         // "searchTerms" is in both, so it should be removed from optional
         const result = normalizeDataCollectionPermissions(input);
 
         expect(result.required).toContain("websiteActivity");
         expect(result.required).toContain(DataCollectionPermission.SearchTerms);
-        expect(result.optional).toEqual(["locationInfo"]);
+        expect(result.optional).toEqual([DataCollectionPermission.BookmarksInfo]);
     });
 
     it("should filter out invalid permissions not present in the enum", () => {

src/types/browser.ts

@@ -14,17 +14,30 @@ export enum Browser {
  * @see https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/browser_specific_settings#data_collection_permissions
  */
 export enum DataCollectionPermission {
-    AuthenticationInfo = "authenticationInfo",
-    BookmarksInfo = "bookmarksInfo",
-    BrowsingActivity = "browsingActivity",
-    FinancialAndPaymentInfo = "financialAndPaymentInfo",
+    /** Contact and identifying information such as name, address, email, etc. */
+    PersonallyIdentifyingInfo = "personallyIdentifyingInfo",
+    /** Medical information, including medical history, diagnoses and treatments */
     HealthInfo = "healthInfo",
-    LocationInfo = "locationInfo",
+    /** Financial and payment data (cards, transactions, credit scores) */
+    FinancialAndPaymentInfo = "financialAndPaymentInfo",
+    /** Authentication data (usernames, passwords, PIN codes) */
+    AuthenticationInfo = "authenticationInfo",
+    /** Personal communications (emails, chats, social‑media messages) */
     PersonalCommunications = "personalCommunications",
-    PersonallyIdentifyingInfo = "personallyIdentifyingInfo",
+    /** Location data (region, GPS coordinates and similar) */
+    LocationInfo = "locationInfo",
+    /** Information about websites visited and pages viewed */
+    BrowsingActivity = "browsingActivity",
+    /** Content of web pages (text, images, video, links and embedded elements) */
+    WebsiteContent = "websiteContent",
+    /** User activity on websites (scrolling, clicks, typing, downloads) */
+    WebsiteActivity = "websiteActivity",
+    /** Search terms entered in search engines or the address bar */
     SearchTerms = "searchTerms",
+    /** Firefox bookmark data: URLs, titles, folder names */
+    BookmarksInfo = "bookmarksInfo",
+    /** Technical and usage data (device/browser info, extension settings, crash reports) */
     TechnicalAndInteraction = "technicalAndInteraction",
-    WebsiteActivity = "websiteActivity",
 }
 
 export interface DataCollectionPermissions {