Block a user
feat: implement GitHub API methods for PR review
test: fix cleanEnv VCS_ leak, add githubAPIURL tests, add GitHub integration test
PR #131: cleanEnv() does not strip VCS_TYPE or VCS_URL env vars
PR #131: integration test not extended for GitHub (issue #130 acceptance criterion)
PR #131: githubAPIURL() function has no unit tests
test: fix cleanEnv VCS_ leak, add githubAPIURL tests, add GitHub integration test
Self-review against 4d48917e3601461d819ee3f0430cc4ea9dd8e139
Assessment: ✅ Clean
No issues found — ready for human review.
Review notes:
- #135 fix: VCS_ prefix strips both VCS_TYPE and…
test: fix cleanEnv VCS_ leak, add githubAPIURL tests, add GitHub integration test
feat: implement GitHub API methods for PR review
Rebase complete, conflict resolved.
This branch had a merge conflict because PR #131 was merged first with an equivalent (but architecturally different) implementation of the same feature (issue…
PR #131: cleanEnv() does not strip VCS_TYPE or VCS_URL env vars
PR #131: githubAPIURL() function has no unit tests
PR #131: integration test not extended for GitHub (issue #130 acceptance criterion)