fix: prevent phantom team diffs caused by slug vs name mismatch

[fgenois-coveo]

Problem

When running safe-settings in NOP mode (or any diff-reporting path), teams are incorrectly reported as additions + deletions even when nothing has changed.

Root cause: MergeDeep.processArrays() resolves item identity via NAME_FIELDS, but slug was missing from that list. For teams, the GitHub API returns both name (display name like "Dev Tooling") and slug (like "dev-tooling"). YAML configs use the slug value in the name field. Since the API object's name differs from the YAML's name, MergeDeep treats them as different items.

The sync path in teams.js already handles this correctly (existing.slug === attrs.name.toLowerCase()), so the mismatch only affects diff reporting.

Fix

1. Add slug (first position) to NAME_FIELDS — API team objects now resolve identity via slug, while YAML objects (which lack slug) resolve via name. Both produce the same value → correct match.

2. Add title to NAME_FIELDS — milestones use title as identity but it was missing from the list.

3. Remove the hardcoded if (item.slug) return item.slug special case — the priority-ordered NAME_FIELDS handles this uniformly.

4. Guard compareDeepIfVisited against mixed primitive/object arrays — prevents crashes when branch protection compares users: ["test"] against users: [{login: "test", type: "User"}].

5. Strip identity fields per-item — when source uses name and target uses slug, each side strips its own identity field before deep comparison.

Backward compatible — no other plugin returns objects with a slug field, and title is only used by milestones (which have no name field).

Test matrix

Source (YAML)	Target (API response)	Expected	Before	After
{name: "dev-tooling", permission: "admin"}	{name: "Dev Tooling", slug: "dev-tooling", permission: "admin"}	No diff	❌ Add+Delete	✅ No diff
{name: "dev-tooling", permission: "push"}	{name: "Dev Tooling", slug: "dev-tooling", permission: "admin"}	Modification	❌ Add+Delete	✅ Modification
{name: "My Team", permission: "admin"}	{name: "My Team", slug: "my-team", permission: "admin"}	No diff	✅ No diff	✅ No diff
{name: "new-team", permission: "push"}	(empty)	Addition	✅	✅
(empty)	{name: "Old Team", slug: "old-team"}	Deletion	✅	✅