PPL: Code Review [Sprint 1, Week 3]

Fri, 13 Mar 2026 00:00:00 +0700

What I Worked On

This week I reviewed 10 teammates’ merge requests, leaving substantive comments with code samples, performance suggestions, and architectural fixes. Three reviews included fix commits that I pushed directly to the teammate’s branch to unblock merging.

Review Summary

MR	Author	Feature	Review Type
!75	soydoradesu	Client management pages	Duplicate code + missing field
!72	hl	Dashboard stale data fix	TanStack Query invalidation gap
!64	soydoradesu	Frontend foundation	Comprehensive review + fix commits
!49	qenthm	Overdue detection worker	Consistency + naming review
!45	froklax	Staff management	Auth/DB desync risk
!46	adipppp	Auth deadlock fix	Correctness verification
!50	dafandikri	Dev environment fix	Review + 4 fix commits pushed
!40	soydoradesu	Frontend foundation	Cleanup commits pushed
!52	adipppp	Payment seeder	FK resolution review
!63	adipppp	JWT alg:none hardening	Security review

How I Conduct Code Reviews

Reviewing 10 MRs in a single week is only feasible because of a structured AI-assisted workflow. I use two Claude Code plugins depending on the MR type, then manually filter and post the findings.

PPL: Code Review Across Architecture and Runtime Bugs [Sprint 1, Week 2]

Wed, 04 Mar 2026 00:00:00 +0700

Overview

MR	Feature	Issues
!10 - SIRA-31	Payment recording (dafandikri)	2
!12 - SIRA-30	Invoice management (froklax)	7
!17 - SIRA-29	Client management (fadhli)	1
!20 - SIRA-27	Layout and dashboard (haliza)	1

11 issues total. Each review included specific file and line references.

MR !10 - Payment Recording (2 issues)

Issue 1: .single() causes an unhandled 500 on missing records.

get_payment_by_id used .single() which throws PostgRESTError (PGRST116) when no row is found, making the if payment is None guard below it dead code. The fix is .maybe_single(), which returns None cleanly. The same fix was already applied in MR !23 for app_users.py.

Collaboration on Daffa Abhipraya

PPL: Code Review [Sprint 1, Week 3]

# What I Worked On

# Review Summary

# How I Conduct Code Reviews