From 0667adbf72234b8ff359b09036fc0397b5fea4de Mon Sep 17 00:00:00 2001 From: Andrew Ridgway Date: Tue, 18 Aug 2026 22:08:16 +1000 Subject: [PATCH] Revert "docs: document pr_reviewer manual-trigger workflow in AGENTS.md" This reverts commit e2ba3dd0990d31167f48bfd0f33f273f0e426514. --- AGENTS.md | 44 -------------------------------------------- 1 file changed, 44 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 5feeec8..05be289 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -341,50 +341,6 @@ All three must pass for merges to main. - Look at test examples in `tests/` for testing patterns - Check `.github/workflows/ci.yml` for what CI expects -## PR Review Workflow (pr_reviewer) - -The `pr_reviewer` service (deployed in the `pr-reviewer` namespace, NodePort 30001) reviews pull requests on `git.aridgwayweb.com`. **Important:** the webhook only fires on PR *open* — it does **not** re-trigger on branch updates or recall. To iterate on a review, you must **manually trigger** the review via the API, then write both the review output and your response to the PR as comments. - -### Manual trigger - -```bash -# 1. Gather the changed file contents -# 2. POST to the review endpoint with the files -curl -X POST http://192.168.178.160:30001/api/v1/review \ - -H "Content-Type: application/json" \ - -d '{ - "pr_id": "1", - "title": "", - "repo": {"name": "armistace/steward_mirror", "url": "https://git.aridgwayweb.com/armistace/steward_mirror"}, - "source": {"branch": "feat/matrix-appservice"}, - "target": {"branch": "main"}, - "files": [{"path": "steward/bot/core.py", "status": "modified", "content": ""}] - }' -``` - -The response contains `results.code_review`, `results.security_review`, `results.infra_review`, and `results.summary`. - -### Posting results to the PR - -The webhook does not post the manual review to the PR. You must post it yourself via the gitea API: - -```bash -TOKEN="" -# Post the review output -curl -X POST -H "Authorization: token $TOKEN" -H "Content-Type: application/json" \ - -d "{\"body\": \"\"}" \ - "https://git.aridgwayweb.com/api/v1/repos/armistace/steward_mirror/issues/1/comments" -# Post your response addressing each finding -``` - -### Iteration loop - -1. Manually trigger the review (above) with the current branch's file contents. -2. Post the review output to the PR as a comment. -3. Fix the actionable findings, commit, and push to the branch. -4. Post a response comment addressing each finding (fixed / already handled / deferred). -5. Re-trigger the review manually and repeat until the review passes. - ## Key Files to Know | File | Purpose |