fix(V2): stop deep recursion and forward unresolved fields in ownership check

TheLastCicada · TheLastCicada · commit 57f823e4ed48 · 2026-05-28T11:30:27.000-07:00
DB-resolved parent records should return early at their own orgUid
rather than traversing the full transitive ownership graph. Also
forward unresolvedFields from the existing-record ownership check
to assertOwnerOrgUidsAreHome so missing parents are caught rather
than silently treated as ownership-OK.
diff --git a/src/models/v2/staging-v2.model.js b/src/models/v2/staging-v2.model.js
@@ -257,7 +257,7 @@ class StagingV2 extends Model {
           options,
           visited,
           depth + 1,
-          includeParentsForDirectOwner,
+          false,
           unresolvedFields,
           new Set(),
         )),
@@ -331,12 +331,14 @@ class StagingV2 extends Model {
 
       if (existingRecord) {
         const tableExcludedFields = StagingV2.getExcludedFieldsForTable(values.table);
+        const existingUnresolved = [];
         await StagingV2.assertOwnerOrgUidsAreHome(
           await StagingV2.collectOwnerOrgUids(
-            existingRecord, options, new Set(), 0, false, [], tableExcludedFields,
+            existingRecord, options, new Set(), 0, false, existingUnresolved, tableExcludedFields,
           ),
           values.table,
           true,
+          existingUnresolved,
         );
       } else if (values.action !== 'UPDATE') {
         continue;
