fix: use slug-aware matching in MergeDeep.processArrays for teams

Teams in YAML use slug format (e.g. name: "my-team") while the GitHub
API returns display name (e.g. name: "My Team") with a separate slug
field. This caused phantom diffs in NOP mode: teams reported as
additions + deletions when no actual changes exist.

The fix adds slug-aware matching in processArrays addition/deletion
filters. When a target item has a slug field, it's compared against
the source name (which is the slug in safe-settings convention).

This is backward compatible: items without slug (labels, rulesets)
continue to match by name as before.

Author: fgenois-coveo

lib/mergeDeep.js

@@ -184,7 +184,7 @@ class MergeDeep {
     if (source.length < target.length) {
       const dels = target.filter(item => {
         if (this.isObjectNotArray(item)) {
-          return !source.some(sourceItem => GET_NAME_USERNAME_PROPERTY(item) === GET_NAME_USERNAME_PROPERTY(sourceItem))
+          return !source.some(sourceItem => { const s = GET_NAME_USERNAME_PROPERTY(sourceItem); const t = GET_NAME_USERNAME_PROPERTY(item); if (s === t) return true; if (item.slug && typeof s === "string") return item.slug === s || item.slug === s.toLowerCase(); return false; })
         } else {
           return !source.includes(item)
         }
@@ -222,7 +222,7 @@ class MergeDeep {
       // Elements that are not in target are additions
       additions[key] = combined.filter(item => {
         if (this.isObjectNotArray(item)) {
-          return !target.some(targetItem => GET_NAME_USERNAME_PROPERTY(item) === GET_NAME_USERNAME_PROPERTY(targetItem))
+          return !target.some(targetItem => { const s = GET_NAME_USERNAME_PROPERTY(item); const t = GET_NAME_USERNAME_PROPERTY(targetItem); if (s === t) return true; if (targetItem.slug && typeof s === "string") return targetItem.slug === s || targetItem.slug === s.toLowerCase(); return false; })
         } else {
           return !target.includes(item)
         }
@@ -233,7 +233,7 @@ class MergeDeep {
       // Elements that not in source are deletions
       deletions[key] = combined.filter(item => {
         if (this.isObjectNotArray(item)) {
-          return !source.some(sourceItem => GET_NAME_USERNAME_PROPERTY(item) === GET_NAME_USERNAME_PROPERTY(sourceItem))
+          return !source.some(sourceItem => { const s = GET_NAME_USERNAME_PROPERTY(sourceItem); const t = GET_NAME_USERNAME_PROPERTY(item); if (s === t) return true; if (item.slug && typeof s === "string") return item.slug === s || item.slug === s.toLowerCase(); return false; })
         } else {
           return !source.includes(item)
         }