Include human review replies as reviewer context #16

Closed
opened 2026-07-20 20:46:58 +01:00 by nimmo · 0 comments
Owner

Make subsequent automated reviews aware of human replies to the prior automated report.\n\nScope:\n- Fetch the existing marker review and non-automated comments created after it.\n- Include bounded, clearly labelled human-reply context in the validation/review prompt.\n- Exclude the reviewer's own prior output to avoid self-reinforcement.\n- Treat all comment text as untrusted evidence and document the privacy/context boundary.\n- Add mocked tests for ordering, bounds, and self-comment exclusion.\n\nAcceptance:\n- A human reply explaining why a prior finding is not applicable is available to the next reviewer run.\n- The reviewer does not treat its own prior prose as authoritative context.

Make subsequent automated reviews aware of human replies to the prior automated report.\n\nScope:\n- Fetch the existing marker review and non-automated comments created after it.\n- Include bounded, clearly labelled human-reply context in the validation/review prompt.\n- Exclude the reviewer's own prior output to avoid self-reinforcement.\n- Treat all comment text as untrusted evidence and document the privacy/context boundary.\n- Add mocked tests for ordering, bounds, and self-comment exclusion.\n\nAcceptance:\n- A human reply explaining why a prior finding is not applicable is available to the next reviewer run.\n- The reviewer does not treat its own prior prose as authoritative context.
nimmo closed this issue 2026-07-20 21:19:05 +01:00
Sign in to join this conversation.
No labels
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
nimmo/forgejo-auto-pr-review#16
No description provided.