gnosisguild / zodiac-modifier-roles / 28596149191

96%

push

github

feat(app): legacy-flow sunset modal with migration steps (#488)

* feat(app): warn on legacy Roles-app permission pushes

Pushing permission updates through the Roles app is the legacy flow; the
permissions starter kit now guides users through the Zodiac app instead. When a
user pushes to the Roles app and lands on the role diff page, show a hint to
update their starter kit.

- Stamp a server-assigned `createdAt` on each permissions post (kept out of the
  content hash, so dedup is unaffected).
- On the role diff page, show the hint only for posts created at/after the
  sunset date (2026-07-03). Earlier posts — and older posts with no timestamp —
  render exactly as before.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012my5oZkF1tFnhsXcNzSkKV

* feat(app): make legacy-flow warning a blocking modal with migration steps

Replace the inline banner with a modal that carries the migration
instructions directly: update to zodiac-roles-sdk v4, or — for projects based
off the permissions starter kit — bootstrap and run the template sync script.

The modal is dismissable ("Continue anyway") until August 15, 2026. From then
on the legacy flow is switched off: the modal can no longer be closed and
blocks the page. Posts from before July 3, 2026 (and older posts without a
timestamp) still render entirely unchanged.

The modal sits at z-index 100 (the app's overlay tier) so the diff view's
condition-grouping connectors (z-index 10) can't bleed through.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012my5oZkF1tFnhsXcNzSkKV

* feat(app): secret unlock param keeps legacy flow usable past the cutoff

Appending ?unlock=<token> to the diff page URL makes the sunset modal
dismissable even after LEGACY_FLOW_END, for teams that coordinated staying on
the legacy flow. The token is read from the LEGACY_FLOW_UNLOCK env var
server-side;... (continued)

401 of 414 branches covered (96.86%)

Branch coverage included in aggregate %.

695 of 732 relevant lines covered (94.95%)

234.66 hits per line