summary
Reads pull requests in four of the products here against a checklist of six to eight points, tailored per repository, at 3am, without complaining, and leaves inline comments. Has never asked for a raise. Occasionally suggests a method that does not exist — which is what the human reviewer is still for.
career
RubyCoin · four products · Pull request review
2026–present
CI · on an opened pull request
CI · on an opened pull request
A checklist, not an opinion. It is in the workflow file, and you can read it.
Flags the N+1 a tired human stops seeing, the migration without a down path, the spec that asserts nothing, and the hardcoded string that should have been a translation key. Advisory only — no merge right, and that is on purpose.
stack
reads
RubyTypeScriptSQLYAMLdiffsstack traces
does not have
production accessmerge rightsan opinion about tabs
Catches the N+1 you stopped seeing on pull request #400
Never gets tired, never gets political about a diff
Does not know your production data, and should not
Cheaper than the bug it prevents
worked on · 4
DNA
Automated review of every pull request against an eight-point checklist, inline in the diff.
WardyBot
Review the moment a pull request opens for review — never on a draft, never twice on the same push.
ImageMaker
Reviews the render path, where a leak shows up as an OOM two days later.
RubyCoin
Inline comments on request — an intern types
review on the pull request and gets one before a human spends time.off work
idle
between pushes
reading
the CHANGELOG
weekends
same as weekdays