Greptile Summary

This PR fixes an unintended side effect in PersonalData::isValid() where calling the method in non-strict mode permanently lowercased $this->userId, $this->email, $this->name, and $this->phone, causing validator instances reused across multiple calls to silently lose their original case. The fix captures each property into a local variable before applying strtolower, leaving instance state untouched.

• isValid() now introduces four local variables ($userId, $email, $name, $phone) initialized from the instance properties at the top of the method, then lowercased conditionally — instance properties are never mutated.
• All comparison logic is unchanged; only the variable names were updated from $this->* to the local copies.

Confidence Score: 5/5

Safe to merge — the change is a clean, narrowly scoped fix with no impact on the validation logic itself.

The diff is a straightforward refactor of four $this->property = strtolower(...) assignments into local variable assignments, with all downstream comparisons updated to match. The fix is correct: local variables are initialized before the strict-mode branch so both code paths work identically to the original, and no validation rule is altered. No new conditional paths, no changes to error handling, and no risk of introducing a regression.

No files require special attention — the single changed file has a well-contained, mechanical fix.

Important Files Changed

_{Reviews (1): Last reviewed commit: "fix: prevent state mutation in PersonalD..." | Re-trigger Greptile}