sync: ambient directories, receiving rules, and the gates that were red

.claude/settings.json

@@ -3,7 +3,13 @@
     "allow": [
       "Bash(adb -s 192.168.43.7:46187 shell \"run-as ai.offgridmobile.dev cat databases/RKStorage\")",
       "Bash(adb -s 192.168.43.7:46187 shell \"run-as ai.offgridmobile.dev ls -la files/image_models/\")",
-      "Bash(node -e \"const RNFS = { DocumentDirectoryPath: '[RNFS.DocumentDirectoryPath]' }; console.log\\('Android DocumentDirectoryPath maps to?'\\)\")"
+      "Bash(node -e \"const RNFS = { DocumentDirectoryPath: '[RNFS.DocumentDirectoryPath]' }; console.log\\('Android DocumentDirectoryPath maps to?'\\)\")",
+      "Bash(xcrun devicectl device copy from *)",
+      "Bash(adb devices)",
+      "Bash(xcrun devicectl device info *)",
+      "Bash(xcrun devicectl list devices)",
+      "Bash(npm run depcruise)",
+      "Bash(npm run knip)"
     ]
   }
 }

.coderabbit.yaml

@@ -0,0 +1,45 @@
+# CodeRabbit configuration.
+#
+# Why this exists: on the sync release PRs CodeRabbit reported a GREEN check having reviewed nothing at all -
+# "Review skipped: 140 files exceed the limit of 100" (desktop) and "316 files exceed the limit of 300"
+# (mobile). A passing check that means "not reviewed" is worse than a missing one, so the file count is kept
+# honest here: screenshots, generated artefacts, lockfiles and docs are excluded from review, and therefore
+# from the count that trips the limit.
+#
+# This does NOT rescue a release-sized PR - desktop's diff is still 140 code files against a limit of 100.
+# The fix for those is smaller PRs; this keeps ordinary ones reviewable and the noise out.
+language: en
+reviews:
+  request_changes_workflow: false
+  high_level_summary: true
+  poem: false
+  review_status: true
+  collapse_walkthrough: true
+  path_filters:
+    # Binary and generated evidence: a reviewer cannot read these, and 27 PNGs alone pushed desktop over.
+    - '!**/*.png'
+    - '!**/*.jpg'
+    - '!**/*.jpeg'
+    - '!**/*.gif'
+    - '!**/*.pdf'
+    - '!**/e2e/screenshots/**'
+    - '!**/__tests__/device/screenshots/**'
+    # Lockfiles and dependency graphs: reviewed by the install gate, not by reading.
+    - '!**/package-lock.json'
+    - '!**/yarn.lock'
+    - '!**/Podfile.lock'
+    - '!**/Gemfile.lock'
+    # Build output and vendored trees.
+    - '!**/dist/**'
+    - '!**/out/**'
+    - '!**/build/**'
+    - '!**/coverage/**'
+    - '!**/node_modules/**'
+    - '!**/Pods/**'
+    - '!**/*.xcodeproj/**'
+    - '!**/.claude/**'
+  auto_review:
+    enabled: true
+    drafts: false
+chat:
+  auto_reply: true

.dependency-cruiser.js

@@ -64,6 +64,12 @@ module.exports = {
           '(^|/)index\\.(ts|tsx)$', // barrel/entry files
           '^src/types/', // type barrels are legitimately import-only
           '^src/(bootstrap|shims|config)/', // wiring/shim/config shells reached outside the graph
+          // Core utilities whose only importers are in the private pro/ submodule. The cruiser scans src
+          // alone, so a module used exclusively by pro reads as an orphan even though deleting it would
+          // break the build - coalesce is imported by pro/sync/fileTransferService and
+          // pro/sync/ambientShareService. Same reasoning as the @offgrid/pro exception below: not real
+          // debt, so excluded rather than baselined. Confirm with grep in pro/ before adding to this list.
+          '^src/utils/coalesce\\.ts$',
         ],
       },
       to: {},
@@ -106,6 +112,10 @@ module.exports = {
     },
   ],
   options: {
+    // Keep file-linked workspace packages under node_modules so dependency-cruiser can
+    // classify them from this package.json instead of treating their real paths as
+    // undeclared source files outside the mobile project.
+    preserveSymlinks: true,
     doNotFollow: { path: 'node_modules' },
     tsPreCompilationDeps: true,
     tsConfig: { fileName: 'tsconfig.json' },

.eslintrc.js

@@ -71,9 +71,22 @@ module.exports = {
     'sonarjs/no-duplicated-branches': 'warn',
   },
   overrides: [
+    {
+      files: ['scripts/physical-sync/**/*.mjs'],
+      parserOptions: {
+        ecmaVersion: 2022,
+        sourceType: 'module',
+      },
+    },
     {
       // Relax structural rules in test files — large test suites and helpers are acceptable
-      files: ['__tests__/**/*', '*.test.ts', '*.test.tsx', 'jest.setup.ts'],
+      files: [
+        '__tests__/**/*',
+        'scripts/physical-sync/__tests__/**/*.mjs',
+        '*.test.ts',
+        '*.test.tsx',
+        'jest.setup.ts',
+      ],
       rules: {
         'max-lines': 'off',
         'max-lines-per-function': 'off',