feat(admin): provision a user with an emailed password reset instead of a set password - #6328
Conversation
…of a set password
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview The admin users table adds a per-row Reset password action (with sending/sent feedback) and widens the actions column; create-time reset failures show a
Reviewed by Cursor Bugbot for commit 6d04bce. Configure here. |
Greptile SummaryThe PR adds password-reset-link provisioning and per-user reset actions to the admin interface, while introducing a reusable masked password treatment for chip-modal inputs.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/app/workspace/[workspaceId]/settings/components/admin/add-user-modal.tsx | Adds credential-mode selection and propagates partial reset-email failures after successful user creation. |
| apps/sim/app/workspace/[workspaceId]/settings/components/admin/admin.tsx | Adds row-level password-reset actions and clearly surfaces provisioning-email failures. |
| apps/sim/hooks/queries/admin-users.ts | Supports passwordless provisioning followed by a reset-email request while preserving the created-user result on delivery failure. |
| packages/emcn/src/components/chip-modal/chip-modal.tsx | Adds the masked password control and contains the corrected mouse focus handling and dedicated props interface. |
| packages/emcn/src/components/chip-modal/chip-modal.test.tsx | Covers masking, focus transitions, mouse activation, and keyboard toggle behavior. |
Sequence Diagram
sequenceDiagram
participant Admin
participant Modal as Add-user modal
participant Auth as Better Auth
participant Reset as Password reset endpoint
Admin->>Modal: Choose Email a reset link
Modal->>Auth: Create user without password
Auth-->>Modal: Created user
Modal->>Reset: Request reset email
alt Email sent
Reset-->>Modal: Success
Modal-->>Admin: Show created user
else Email failed
Reset-->>Modal: Error
Modal-->>Admin: Show user and retry warning
end
Reviews (4): Last reviewed commit: "test(emcn): cover keyboard activation of..." | Re-trigger Greptile
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
1 issue from previous review remains unresolved.
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 8dadbec. Configure here.
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 29fa87a. Configure here.
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 6d04bce. Configure here.

Summary
Type of Change
Testing
ChipModalField inputType='password'unit tests cover mask-on-blur, reveal-on-focus, the read-only-until-focus anti-autofill guard, and the eye toggleaddUserunit tests cover both paths: password omitted → reset email requested; create failure → no email sent; email failure → error names the row actionChecklist