Repository navigation
Conversation
There was a problem hiding this comment.
🤖 AI Review
Both independent reviews reported no findings. Inspection of the checked-out code found no actionable issues: the setting is consistently wired through schemas, hosted config mapping, and both local Auth paths, with backward-compatible defaults. Tests were inspected but not executed.
Findings
No issues found.
Stats
Claude findings: 0 · Codex findings: 0 · Confirmed: 0 · Refuted: 0 · Uncertain: 0
Models: claude-opus-5-5 + gpt-6.1-sol · Trigger: auto · Workflow run
This review runs once per PR. A maintainer can request another with a /ai-review comment.
CLI benchmark (Lightweight Commands): no changes foundCommit
API calls: unchanged for all 7 commands. Binary size: 134.0 MB, up 4.0 KB from 134.0 MB. Details: timing ranges, API calls per command, slowest phasesTiming ranges
API calls per command
Slowest phasesNo command moved beyond noise, so there are no phase deltas. Comment |
TL;DR
adds
require_current_passwordunder[auth.email]so local stacks and linked projects can require the current password for password changes.whats introduced?
supabase startnow passes the value to auth asGOTRUE_SECURITY_UPDATE_PASSWORD_REQUIRE_CURRENT_PASSWORD.config push,config pullandconfig diffmap it to the linked project'ssecurity_update_password_require_current_password.it defaults to off and
supabase initwrites it commented out.config pushleaves the project toggle alone whenconfig.tomldoes not set it.a project with the toggle on and no key in
config.tomlnow shows up inconfig diff, soconfig diff --exit-codeexits 2 untilconfig.tomlsets it totrue.visual rep:
ref: