3 Commits

This Branch

This Branch

All Branches

Author	SHA1	Message	Date
Hermes Agent	c430e02117	[grade=B urn:ump:bbfbsfru4nizg75wcxum7mwzhie6txhr22hw6jvz36wiihmgzmlq] P7 people store: dedupe upsert by providerUserId ... 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
Hermes Agent	5b3e637423	[grade=B urn:ump:mao7vadnesmquhmgr4oeg554edt6fn3ykxyiydtfs2psoekd4s7q] P7 tailscale provider: wire coord client via ctx delegation ... Dogfood finding #2: createTailscaleProvider() built with coordClient=null; nothing in prod ever called attachCoordClient() (the wiring the provider docblock promised never landed), so wizard credential validation always failed not_configured even with a valid stored token. Pass a delegating proxy that fetches a fresh coord client per call via the sanctioned ctx.tailscaleCoord.getClient pattern (token re-read from credentialManager each call). Full suite 3413/3413.	2026-10-04 02:51:18 -07:00
Hermes Agent	f7d6fe4a8a	[grade=B urn:ump:2swys44kxvulm5flu4skibjq3yfvtnzrsbgkvm2g65ht55i4jtoa] P7 wizard prod wiring: pass provider instances (dogfood finding) ... The /network/wizard mount in app.js omitted providers, so the wizard built with providers={} and every mode selection returned 503 provider_unavailable in production. Unit tests never caught it because they inject their own provider map. Pass built tailscale + wireguard instances. Full suite: 3413/3413 (152 suites).	2026-10-04 02:41:31 -07:00