GrowthExperiments\Mentorship\MentorManager::getMentorshipStateForUserInternal was asked to return mentorship state for a non-named user {user}~ 3,129 in the last 24h, user impact tbd
3,720 hits (Warnings) in the last 7 days https://logstash.wikimedia.org/goto/c137730ff3f4f1271803101cb330b181
It started on `@timestamp
Jan 7, 2026 @ 08:42:57.740` https://logstash.wikimedia.org/goto/166dfb4b6cb24e7f7b746c47e591493f
That was introduced in fix(Mentorship): Do not run MenteeGraduation for temporary accounts for T412858: eswiki: Mentee Graduation is not taking temporary users into account.
I'm a bit surprised to see so many results. We must have made a mistake in that commit. Though the impact is just that log-spam, it should already return the correct result.
Specifically, why is $this->mentorStore->isMentee( $user ) not returning null for a temporary account? Do these temporary accounts somehow do have a mentor? Then that would be a problem that we indeed should fix.
Change #1226927 had a related patch set uploaded (by Michael Große; author: Michael Große):
[mediawiki/extensions/GrowthExperiments@master] fix(PersonalizedPraise): don't try to praise Temporary Accounts
I tried to figure out whether we inadvertently were assigning mentors to temporary accounts, and while doing so, I noticed I had already debugged this to the point of that writing the commits was less effort than documenting my insights in words here and have somebody else look into it again on their own time. The above change maybe needs another test, but otherwise should be done.
Change #1226927 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] fix(PersonalizedPraise): don't try to praise Temporary Accounts