P7 people-store providerUserId dedupe #4

Merged

sami7777 merged 1 commits from p7-people-dedupe into main

2026-10-04 12:14:44 +02:00

Author	SHA1	Message	Date
Hermes Agent	c430e02117	[grade=B urn:ump:bbfbsfru4nizg75wcxum7mwzhie6txhr22hw6jvz36wiihmgzmlq] P7 people store: dedupe upsert by providerUserId CI / Security audit (pull_request) Canceled after 0s Details CI / Test & Lint (pull_request) Canceled after 0s Details Dogfood finding #3: upsertPerson resolved existing records by id only. The wizard upserts a bootstrap admin AFTER a provider sync has already created a person for the same Tailscale user (no id passed), so the same providerUserId was duplicated into two records. Resolve existing by providerUserId when no id is given; bootstrap Admin group + protectedAt are preserved on the merge. New regression test; full suite 3414/3414.	2026-10-04 03:12:23 -07:00

Author

SHA1

Message

Date

Hermes Agent

c430e02117

[grade=B urn:ump:bbfbsfru4nizg75wcxum7mwzhie6txhr22hw6jvz36wiihmgzmlq] P7 people store: dedupe upsert by providerUserId

CI / Security audit (pull_request) Canceled after 0s

Details

CI / Test & Lint (pull_request) Canceled after 0s

Details

Dogfood finding #3: upsertPerson resolved existing records by id only.
The wizard upserts a bootstrap admin AFTER a provider sync has already
created a person for the same Tailscale user (no id passed), so the same
providerUserId was duplicated into two records. Resolve existing by
providerUserId when no id is given; bootstrap Admin group + protectedAt are
preserved on the merge. New regression test; full suite 3414/3414.

2026-10-04 03:12:23 -07:00