docs: E01 wireframe states and deterministic generator#45
Merged
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
📝 WalkthroughWalkthroughThis PR adds and updates Excalidraw wireframes and linked feature docs: it introduces new registration and organization-settings wireframes (including a verify-email rate-limit state), updates F01/F02 feature docs to reference them, refines layout for email verification screens, and regenerates metadata across many existing diagram assets. ChangesWireframes and Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/epics/E01-platform-foundation/features/F02-organization-management.md (1)
1-4:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winReplace the custom back-link with the required feature navigation callout.
Line 3 should use the standard feature-file navigation format and appear as the first thing after the title.
As per coding guidelines, feature docs must include:
> **Navigation**: [← Epic README](README.md)as the first thing after the title.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/epics/E01-platform-foundation/features/F02-organization-management.md` around lines 1 - 4, Replace the custom backlink "[← Back to E01](../README.md)" that currently follows the title "F02 — Organization Management" with the standard feature navigation callout; remove the existing link and insert the exact line "> **Navigation**: [← Epic README](README.md)" immediately after the title so the navigation callout is the first thing after the header.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@docs/epics/E01-platform-foundation/features/F02-organization-management.md`:
- Around line 1-4: Replace the custom backlink "[← Back to E01](../README.md)"
that currently follows the title "F02 — Organization Management" with the
standard feature navigation callout; remove the existing link and insert the
exact line "> **Navigation**: [← Epic README](README.md)" immediately after the
title so the navigation callout is the first thing after the header.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: df978c03-d2d4-4a5b-ba03-5b5ee06786c5
⛔ Files ignored due to path filters (30)
docs/epics/E01-platform-foundation/wireframes/email-confirmation.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/pricing.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/register-org.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/settings-org-delete-modal.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/settings-org-deletion-scheduled.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/settings-org-upload-states.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/settings-org-usage-error.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/settings-org.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/verify-email-rate-limit.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/verify-email.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/workspace-provisioning.svgis excluded by!**/*.svgdocs/epics/E02-identity-access/wireframes/accept-invitation.svgis excluded by!**/*.svgdocs/epics/E02-identity-access/wireframes/change-password.svgis excluded by!**/*.svgdocs/epics/E02-identity-access/wireframes/forgot-password.svgis excluded by!**/*.svgdocs/epics/E02-identity-access/wireframes/login.svgis excluded by!**/*.svgdocs/epics/E02-identity-access/wireframes/register.svgis excluded by!**/*.svgdocs/epics/E02-identity-access/wireframes/settings-roles.svgis excluded by!**/*.svgdocs/epics/E02-identity-access/wireframes/settings-security.svgis excluded by!**/*.svgdocs/epics/E02-identity-access/wireframes/settings-users.svgis excluded by!**/*.svgdocs/epics/E03-data-modeling/wireframes/data-classes.svgis excluded by!**/*.svgdocs/epics/E03-data-modeling/wireframes/data-models.svgis excluded by!**/*.svgdocs/epics/E03-data-modeling/wireframes/records.svgis excluded by!**/*.svgdocs/epics/E04-workflow-builder/wireframes/workflow-editor.svgis excluded by!**/*.svgdocs/epics/E04-workflow-builder/wireframes/workflows.svgis excluded by!**/*.svgdocs/epics/E05-form-builder/wireframes/form-editor.svgis excluded by!**/*.svgdocs/epics/E05-form-builder/wireframes/form-submission.svgis excluded by!**/*.svgdocs/epics/E05-form-builder/wireframes/forms.svgis excluded by!**/*.svgdocs/epics/E06-workflow-engine/wireframes/execution-detail.svgis excluded by!**/*.svgdocs/epics/E06-workflow-engine/wireframes/executions.svgis excluded by!**/*.svgdocs/wireframes/app-shell.svgis excluded by!**/*.svg
📒 Files selected for processing (36)
docs/epics/E01-platform-foundation/features/F01-tenant-registration.mddocs/epics/E01-platform-foundation/features/F02-organization-management.mddocs/epics/E01-platform-foundation/wireframes/email-confirmation.excalidrawdocs/epics/E01-platform-foundation/wireframes/pricing.excalidrawdocs/epics/E01-platform-foundation/wireframes/register-org.excalidrawdocs/epics/E01-platform-foundation/wireframes/settings-org-delete-modal.excalidrawdocs/epics/E01-platform-foundation/wireframes/settings-org-deletion-scheduled.excalidrawdocs/epics/E01-platform-foundation/wireframes/settings-org-upload-states.excalidrawdocs/epics/E01-platform-foundation/wireframes/settings-org-usage-error.excalidrawdocs/epics/E01-platform-foundation/wireframes/settings-org.excalidrawdocs/epics/E01-platform-foundation/wireframes/verify-email-rate-limit.excalidrawdocs/epics/E01-platform-foundation/wireframes/verify-email.excalidrawdocs/epics/E01-platform-foundation/wireframes/workspace-provisioning.excalidrawdocs/epics/E02-identity-access/wireframes/accept-invitation.excalidrawdocs/epics/E02-identity-access/wireframes/change-password.excalidrawdocs/epics/E02-identity-access/wireframes/forgot-password.excalidrawdocs/epics/E02-identity-access/wireframes/login.excalidrawdocs/epics/E02-identity-access/wireframes/register.excalidrawdocs/epics/E02-identity-access/wireframes/settings-roles.excalidrawdocs/epics/E02-identity-access/wireframes/settings-security.excalidrawdocs/epics/E02-identity-access/wireframes/settings-users.excalidrawdocs/epics/E03-data-modeling/wireframes/data-classes.excalidrawdocs/epics/E03-data-modeling/wireframes/data-models.excalidrawdocs/epics/E03-data-modeling/wireframes/records.excalidrawdocs/epics/E04-workflow-builder/wireframes/workflow-editor.excalidrawdocs/epics/E04-workflow-builder/wireframes/workflows.excalidrawdocs/epics/E05-form-builder/wireframes/form-editor.excalidrawdocs/epics/E05-form-builder/wireframes/form-submission.excalidrawdocs/epics/E05-form-builder/wireframes/forms.excalidrawdocs/epics/E06-workflow-engine/wireframes/execution-detail.excalidrawdocs/epics/E06-workflow-engine/wireframes/executions.excalidrawdocs/playbooks/patterns.mddocs/playbooks/wireframes.mddocs/wireframes/app-shell.excalidrawdocs/wireframes/components.mjsdocs/wireframes/generate-screens.mjs
Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/epics/E01-platform-foundation/features/F01-tenant-registration.md`:
- Line 3: Replace the current navigation link line "[← Back to
E01](../README.md)" with the standard blockquote navigation format; edit the
markdown in F01-tenant-registration.md so the top navigation becomes ">
**Navigation**: [← Epic README](README.md)" to match the project's coding
guidelines and other feature files.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: b39f66a1-ec1c-4d6b-a150-6ec0c39b849a
⛔ Files ignored due to path filters (6)
docs/epics/E01-platform-foundation/wireframes/register-org-states.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/settings-org-access-denied.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/settings-org-delete-states.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/settings-org-free-plan.svgis excluded by!**/*.svgdocs/epics/E01-platform-foundation/wireframes/settings-org-profile-states.svgis excluded by!**/*.svgdocs/epics/E02-identity-access/wireframes/login-unverified.svgis excluded by!**/*.svg
📒 Files selected for processing (11)
docs/epics/E01-platform-foundation/features/F01-tenant-registration.mddocs/epics/E01-platform-foundation/features/F02-organization-management.mddocs/epics/E01-platform-foundation/wireframes/register-org-states.excalidrawdocs/epics/E01-platform-foundation/wireframes/settings-org-access-denied.excalidrawdocs/epics/E01-platform-foundation/wireframes/settings-org-delete-states.excalidrawdocs/epics/E01-platform-foundation/wireframes/settings-org-free-plan.excalidrawdocs/epics/E01-platform-foundation/wireframes/settings-org-profile-states.excalidrawdocs/epics/E02-identity-access/features/F01-authentication.mddocs/epics/E02-identity-access/wireframes/login-unverified.excalidrawdocs/playbooks/wireframes.mddocs/wireframes/generate-screens.mjs
✅ Files skipped from review due to trivial changes (5)
- docs/epics/E01-platform-foundation/wireframes/settings-org-delete-states.excalidraw
- docs/epics/E01-platform-foundation/wireframes/register-org-states.excalidraw
- docs/epics/E01-platform-foundation/wireframes/settings-org-access-denied.excalidraw
- docs/epics/E01-platform-foundation/features/F02-organization-management.md
- docs/epics/E01-platform-foundation/wireframes/settings-org-profile-states.excalidraw
| @@ -3,8 +3,11 @@ | |||
| [← Back to E01](../README.md) | |||
There was a problem hiding this comment.
Update navigation format to match coding guidelines.
The navigation link should follow the standard blockquote format with a "Navigation:" label. As per coding guidelines, feature files must use: > **Navigation**: [← Epic README](README.md)
📝 Proposed fix
-[← Back to E01](../README.md)
+> **Navigation**: [← E01 Epic README](../README.md)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| [← Back to E01](../README.md) | |
| > **Navigation**: [← E01 Epic README](../README.md) |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/epics/E01-platform-foundation/features/F01-tenant-registration.md` at
line 3, Replace the current navigation link line "[← Back to E01](../README.md)"
with the standard blockquote navigation format; edit the markdown in
F01-tenant-registration.md so the top navigation becomes "> **Navigation**: [←
Epic README](README.md)" to match the project's coding guidelines and other
feature files.
8 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
verify-email-rate-limit,settings-org-upload-states,settings-org-usage-error,settings-org-deletion-scheduled.generate-screens.mjsfor full E01 coverage; update F01/F02 wireframe callouts and screen inventory inwireframes.md.runScreen) so editing one screen does not churn unrelated epics.stateHeadlinelayout (icon + semantic title + short underline); compact auth cards; semantic text colors only.wireframes.mdandpatterns.md.Gate 1
Docs-only PR —
dotnet test/npm run cinot required (nosrc/,tests/, orfrontend/changes).Gate 2
patterns.md(deterministic seed pitfall);wireframes.md(deterministic seed,stateHeadline, exports)generate-screens.mjs+generate-wireframes.ps1run; F01/F02 callouts updatedGate 3
wireframes.mdTest plan
verify-email,email-confirmation,verify-email-rate-limit, new settings-org state screensnode docs/wireframes/generate-screens.mjstwice; confirm no file changes on second rundocs/scripts/generate-wireframes.ps1if.excalidrawedited locallySummary by CodeRabbit