CI on PR #13 failed at `pnpm lint --max-warnings 0`. Four findings, all
new-in-N16 react-strict checks:
* ThemeToggle.tsx — "Calling setState synchronously within an effect"
Rewrites the theme reader to use `useSyncExternalStore` with a
`MutationObserver` on `<html data-theme>`. SSR snapshot stays "light"
(matches the root layout); the head script and the toggle just write
the attribute, the observer pushes the change into React. Drops the
`mounted` flag because the icon now mirrors the DOM truthfully.
* WorkflowEditor.tsx — "Cannot access refs during render"
`stateRef.current = { pan, zoom }` was a direct ref-mutation in the
component body so the global mousemove handler could read the latest
viewport without re-subscribing. Moves the mirror into a `useEffect`
keyed on `[pan, zoom]` — same semantics, satisfies the rule.
* MockWorker.tsx — drops an unused `eslint-disable-next-line no-console`
(the `no-console` rule isn't enabled).
* public/mockServiceWorker.js — auto-generated by `msw init`; adds it to
the eslint flat-config `ignores` so the lint pass never crosses it.
Local: `pnpm lint` + `pnpm typecheck` both clean.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Next.js 16 + Auth.js v5 skeleton: host→slug middleware, tenant-context layout, OIDC sign-in flow against breakpilot-dev realm. 100% coverage on src/lib. Bumps next to 16.2.6 to clear trivy CVEs in 15.0.3.
Sharang Parnerkar
sharang