feat(review): add local coderfeli review watcher - #1158
Merged
coderfeli merged 9 commits intoSep 22, 2026
Merged
Conversation
Signed-off-by: jhinpan <47354855+jhinpan@users.noreply.github.com>
Signed-off-by: jhinpan <47354855+jhinpan@users.noreply.github.com>
Signed-off-by: jhinpan <47354855+jhinpan@users.noreply.github.com>
Signed-off-by: jhinpan <47354855+jhinpan@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #1157, which is stacked on #1106. This layer adds only the local operator: immutable coderfeli/repository gates, SQLite claim state keyed by PR head and engine SHA, exact base/head source preparation, a pinned no-GPU review image, timeout/crash recovery, local-only artifacts, publication-disabled canary support, and disabled systemd user templates. The container receives the node model gateway variables by name but no GitHub credential; host-side publication remains identity/OID checked and defaults to dry-run.
The pilot enables
--group-findersand submits one model session at a time. The measured gateway serialized concurrent Opus/max calls: concurrency nine inflated median request latency by about 11.7x and made aggregate completion slower than serial submission. Model sessions expose only Read, Grep, and Glob under the root-deny filesystem policy; no reviewed-tree Bash or code execution is available.A frozen PR #1137 end-to-end canary completed the grouped finder, six independent verifications, two challenges, sweep, artifact reconstruction, and dry-run publisher in 2,154 seconds for $12.89 with zero permission denials. The final artifact contained one confirmed P2 and five plausible risks, so the P1 publication threshold produced no GitHub review.
Independent adjudication of the finder-only sample retained two plausible P2 risks, refuted three candidates, and identified one missed strong P2: ten graph operations overwrite one output while only the final write is observed. The owning test-contract checklist now explicitly audits distinct observed results and overwritten intermediate outputs. This evidence supports pilot use over the non-completing nine-session configuration; it does not establish general recall or equivalence to nine independent finders.