Skip to content

feat(review): collect decisions, retain finding history and isolate repository checks - #4

Draft
pmanko wants to merge 1 commit into
feat/ogc-1560-review-harness-v1from
c/ogc-1560-context-execution
Draft

pmanko wants to merge 1 commit into
feat/ogc-1560-review-harness-v1from
c/ogc-1560-context-execution

Conversation

@pmanko

@pmanko pmanko commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

Reviews now retain accepted intent and changed expectations, PR conversations, stable finding identity and explicit re-review dispositions. Pinned source remains review data; discussion association does not automatically confer approval. Adds a generic repository check executor using a disposable container without network, host mounts or credentials.

Validation: 55 tests passed at this slice. A real L1 fixture executed two behavior tests with zero model calls and removed its container. Later stack tests cover all five re-review dispositions. This establishes collection/execution behavior, not model-review quality.

Current implementation checklist · OGC-1560

Stack created with GitHub Stacks CLI • Give Feedback 💬

@pmanko
pmanko added this pull request to stack #3 October 30, 2026 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant