vault: scaffold auth-password-reset-ui (frontend gap found via user report) - #66
Merged
Conversation
…eport) Backend password reset shipped backend-only (auth-password-reset.md); its frontend deferral to auth-pages.md was never actually picked up there. requestPasswordReset/updatePassword already exist in frontend/src/api/auth.ts (Supabase-backed) but nothing calls them — no link on LoginForm, no /forgot-password or /reset-password route.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
reik
added a commit
that referenced
this pull request
Aug 24, 2026
A separate, already-running session had independently reached the same diagnosis and was 16h further along on auth-password-reset-ui (PR #66), holding for go-ahead before dispatching. User chose to let that one proceed instead. No application code existed on this branch; only vault routing is reverted/marked abandoned here. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
reik
marked this pull request as ready for review
August 25, 2026 01:21
reik
added a commit
that referenced
this pull request
Aug 25, 2026
- Resolved PR #65's merge conflicts against master (kanban/company.md, kanban/engineering.md, log/2026-08-22.md), caused by #66/#67 landing on master after this PR's merge-base. - Corrects the record on the recurring "[Fact-Forcing Gate]" tool error that multiple prior sessions flagged as unexplained/injected: it's a real PreToolUse hook from the everything-claude-code plugin's gateguard skill, not injection. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
reik
added a commit
that referenced
this pull request
Aug 25, 2026
* vault: ship header-nav-redesign (PR #64 merged to master) --ship transition: engineering + design kanban cards moved Review -> Done, company kanban moved Review -> Done, feature status review -> shipped. Post-ship docs/user-guide.md check via content-writer: no edit warranted (guide never described the old nav's visual form). marketing-lead aggregated and confirmed, no marketing kanban card existed to move (marketing was never routed to this feature during build). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * vault: log PR #65 conflict resolution + gateguard correction - Resolved PR #65's merge conflicts against master (kanban/company.md, kanban/engineering.md, log/2026-08-22.md), caused by #66/#67 landing on master after this PR's merge-base. - Corrects the record on the recurring "[Fact-Forcing Gate]" tool error that multiple prior sessions flagged as unexplained/injected: it's a real PreToolUse hook from the everything-claude-code plugin's gateguard skill, not injection. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * vault: log PR #65 second conflict resolution (master moved past #68) --------- Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
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
auth-password-reset.mdshipped backend-only (2026-05-16); its stated frontend deferral toauth-pages.mdwas never actually picked up there (Phase 3 bootstrap shipped login/register/verify-email only).requestPasswordReset/updatePasswordalready exist infrontend/src/api/auth.ts, fully Supabase-backed — but nothing calls them. No link onLoginForm, no/forgot-passwordor/reset-passwordroute.fofafu_vault/features/auth-password-reset-ui.md, both kanban boards, today's log). No code changes. Perprotocols/dispatch.md, actually building this is a human-invoked/dispatch auth-password-reset-ui— holding for go-ahead.Test plan
🤖 Generated with Claude Code