browser-commander 0.26.2 → 0.26.3

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.26.3
+
+### Patch Changes
+
+- 796ebd8: Accept dedicated macOS application profiles under `~/Library/Application Support` by separating Safari import discovery roots from protected browser data paths. Safari, Technology Preview and Chrome default profiles remain protected.
+
 ## 0.26.2
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "browser-commander",
-  "version": "0.26.2",
+  "version": "0.26.3",
   "description": "Universal browser automation library that supports Playwright, Puppeteer, and Selenium with a unified API",
   "main": "src/index.js",
   "bin": {

package/src/browser/browser-sources.js

@@ -141,6 +141,19 @@ export function resolveBrowserRoots(name, options = {}) {
   return resolveTemplates(templates, options);
 }
 
+/**
+ * Browser-owned paths forbidden as automation profiles. Import discovery may
+ * search broader locations; use explicit protection roots when declared.
+ */
+export function resolveBrowserProtectionRoots(name, options = {}) {
+  const source = normalizeBrowserSource(name);
+  const roots = source.protectionRoots ?? source.roots;
+  return resolveTemplates(
+    roots?.[options.platform ?? process.platform] ?? [],
+    options
+  );
+}
+
 function resolveTemplates(
   templates,
   {

package/src/browser/browser-sources.json

@@ -764,6 +764,13 @@
           "{home}/Library"
         ]
       },
+      "protectionRoots": {
+        "darwin": [
+          "{home}/Library/Safari",
+          "{home}/Library/Containers/com.apple.Safari",
+          "{home}/Library/Cookies"
+        ]
+      },
       "default": {
         "darwin": ["com.apple.safari"]
       },
@@ -781,6 +788,12 @@
           "{home}/Library/Safari Technology Preview"
         ]
       },
+      "protectionRoots": {
+        "darwin": [
+          "{home}/Library/Safari Technology Preview",
+          "{home}/Library/Containers/com.apple.SafariTechnologyPreview"
+        ]
+      },
       "default": {
         "darwin": ["com.apple.safaritechnologypreview"]
       },

package/src/browser/system-browser.js

@@ -2,7 +2,7 @@ import { constants } from 'node:fs';
 import { physicalPath } from './browser-profile-files.js';
 import {
   BROWSER_SOURCES,
-  resolveBrowserRoots,
+  resolveBrowserProtectionRoots,
   resolveBrowserExecutables,
 } from './browser-sources.js';
 import { access } from 'node:fs/promises';
@@ -46,7 +46,7 @@ export function knownDefaultUserDataDirs({
   environment = process.env,
 } = {}) {
   return BROWSER_SOURCES.flatMap((browser) =>
-    resolveBrowserRoots(browser.id, {
+    resolveBrowserProtectionRoots(browser.id, {
       platform,
       homeDir,
       environment,

package/tests/unit/browser/catalogue-launch.test.js

@@ -6,6 +6,7 @@ import { useTempDirectories } from '../../helpers/temp-directory.js';
 import {
   BROWSER_SOURCES,
   resolveBrowserRoots,
+  resolveBrowserProtectionRoots,
 } from '../../../src/browser/browser-sources.js';
 import {
   assertDedicatedUserDataDir,
@@ -13,8 +14,51 @@ import {
 } from '../../../src/browser/system-browser.js';
 
 const temporary = useTempDirectories('bc-catalogue-protection-');
+const macOSOptions = {
+  platform: 'darwin',
+  homeDir: '/users/test',
+  environment: {},
+};
 
 describe('catalogue launch and protection (#121)', () => {
+  it('accepts dedicated macOS application profiles under Library (#132)', () => {
+    const options = macOSOptions;
+    for (const directory of [
+      'Library/Application Support/package-registry-manager/browser-profile',
+      'Library/Safari-backup/profile',
+      'Library/Cookies-backup/profile',
+      'Library/Containers/com.apple.Safari-helper/profile',
+    ]) {
+      assert.doesNotThrow(() =>
+        assertDedicatedUserDataDir(
+          path.posix.join(options.homeDir, directory),
+          options
+        )
+      );
+    }
+  });
+
+  it('protects Safari stores and Chrome profiles on macOS (#132)', () => {
+    const options = macOSOptions;
+    for (const directory of [
+      'Library/Safari',
+      'Library/Cookies',
+      'Library/Containers/com.apple.Safari',
+      'Library/Safari Technology Preview',
+      'Library/Containers/com.apple.SafariTechnologyPreview',
+      'Library/Application Support/Google/Chrome/Default',
+    ]) {
+      const root = path.posix.join(options.homeDir, directory);
+      for (const requested of [root, path.posix.join(root, 'new-profile')]) {
+        assert.throws(
+          () => assertDedicatedUserDataDir(requested, options),
+          /dedicated.*default profile/,
+          requested
+        );
+      }
+    }
+  });
+
   it('protects a new profile beneath a symlink to a default root', async () => {
     const homeDir = await temporary();
     const options = { homeDir, environment: {} };
@@ -47,7 +91,7 @@ describe('catalogue launch and protection (#121)', () => {
     }
   });
 
-  it('protects every catalogue root and descendant profile on every OS', () => {
+  it('protects every catalogue protection root and descendant profile on every OS', () => {
     for (const platform of ['linux', 'darwin', 'win32']) {
       const options = {
         platform,
@@ -55,7 +99,11 @@ describe('catalogue launch and protection (#121)', () => {
         environment: {},
       };
       for (const browser of BROWSER_SOURCES) {
-        for (const root of resolveBrowserRoots(browser.id, options)) {
+        const roots = resolveBrowserProtectionRoots(browser.id, options);
+        if (!browser.protectionRoots) {
+          assert.deepEqual(roots, resolveBrowserRoots(browser.id, options));
+        }
+        for (const root of roots) {
           assert.throws(
             () => assertDedicatedUserDataDir(root, options),
             /dedicated.*default profile/,