Compare commits

...
2 Commits
Author SHA1 Message Date
sami7777 ecdb2fd8ee P7 wizard prod wiring fix [grade=B] (PR #2)
CI / Test & Lint (push) Canceled after 0s
CI / Security audit (push) Canceled after 0s
Judge B r1 urn:ump:2swys44kxvulm5flu4skibjq3yfvtnzrsbgkvm2g65ht55i4jtoa. Dogfood finding: wizard mount omitted providers. 3413/3413.
2026-10-04 11:41:45 +02:00
Hermes Agent f7d6fe4a8a [grade=B urn:ump:2swys44kxvulm5flu4skibjq3yfvtnzrsbgkvm2g65ht55i4jtoa] P7 wizard prod wiring: pass provider instances (dogfood finding)
CI / Test & Lint (pull_request) Canceled after 0s
CI / Security audit (pull_request) Canceled after 0s
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
+6
View File
@@ -776,8 +776,14 @@ async function createApp() {
asyncHandler: ctx.asyncHandler,
log: ctx.log,
}));
// Private Network wizard (DC-152). Dogfood finding 2026-10-04: the wizard
// factory needs INSTANCES keyed by mode id; without them every mode
// selection 503s provider_unavailable (unit tests inject their own map).
const p7TailscaleProvider = require('./network/providers/tailscale').createTailscaleProvider(); // eslint-disable-line global-require
const p7WireGuardProvider = require('./network/providers/wireguard').createWireGuardProvider(); // eslint-disable-line global-require
apiRouter.use('/network/wizard', networkWizardRoutes({
registry: { getActiveProvider },
providers: { tailscale: p7TailscaleProvider, wireguard: p7WireGuardProvider },
peopleStore: p7PeopleStore,
accessStore: p7AccessStore,
asyncHandler: ctx.asyncHandler,