Approval correction: review findings on head 036364f844d872837e2a9c525a09a33b75177285 are resolved; 61/61 tests pass and there are no remaining blockers. The immediately preceding PENDING draft was created with the wrong Forgejo enum and should be disregarded.
Two correctness issues in the new stoke api escape hatch:
Reviewed current head 0ecd935 against updated base d396e36. No blocking findings.
Final review of head 4444703: no blocking findings. Production behavior remains correct, the new STOKE_APT_ETC override preserves default production paths, and the NodeSource bootstrap/no-clobber logic is intact.
Re-reviewed head 5e99006. The previous blockers are resolved:
Three blocking issues in the NodeSource bootstrap:
The runtime fix in src/cli.js:759-764 is correct: trimming is now used only to validate emptiness, and rawBody is sent unchanged. One review item remains before approval:
The two original blockers are fixed: approval now sends APPROVED, and bodyless/whitespace-only REQUEST_CHANGES and COMMENT reviews are rejected locally. One remaining issue:
Retrospective review (this PR was already merged before I began): the 27 tests pass, shell syntax checks pass, and the Debian package builds successfully, but I found three material follow-ups:
1.…