Skip to content

Block OpenCode approval on stale fact evidence#362

Merged
seonghobae merged 1 commit into
developfrom
codex/opencode-fact-gate-20260621
Jun 21, 2026
Merged

Block OpenCode approval on stale fact evidence#362
seonghobae merged 1 commit into
developfrom
codex/opencode-fact-gate-20260621

Conversation

@seonghobae

Copy link
Copy Markdown
Collaborator

Summary

  • add current-head docs tree evidence to OpenCode review input before it judges docs/assets claims
  • re-query unresolved, non-outdated human review threads immediately before publishing APPROVE
  • add a tiny runnable contract smoke test for this OpenCode fact gate

Why

This rolls out the org-level fix proven in ContextualWisdomLab/naruon#664. The failing mode was: OpenCode prepared evidence, a human review thread arrived afterward, and approval still happened without a fresh thread/fact check.

Branch model

  • Default branch: develop
  • Flow: Git Flow

Validation

  • bash scripts/ci/test_opencode_fact_gate_contract.sh
  • git diff --check -- .github/workflows/opencode-review.yml scripts/ci/test_opencode_fact_gate_contract.sh
  • actionlint -shellcheck "" .github/workflows/opencode-review.yml

@opencode-agent

opencode-agent Bot commented Jun 21, 2026

Copy link
Copy Markdown

OpenCode Review Overview

  • Head SHA: b02b02ac2148336a8cf44fcbcbf5758228b7bac2
  • Workflow run: 27892016371
  • Workflow attempt: 1
  • Gate result: APPROVE (approval step)

Pull request overview

The changes in PR #362 include updates to the OpenCode review workflow and a new test script. Structural exploration confirmed no regressions or broken contracts. No failed GitHub Checks were present.

Findings

No blocking findings from OpenCode's independent review.

Verification

  • Review source: independent OpenCode review of the current checkout, focused changed hunks, and current-head GitHub Check evidence.
  • Structural exploration: completed before approval; if structural exploration, changed-file inspection, or evidence completeness is missing, OpenCode must not approve.
  • Result: APPROVE
  • Reason: No source-backed blockers found; structural exploration completed.

Gate evidence

  • Head SHA: b02b02ac2148336a8cf44fcbcbf5758228b7bac2
  • Workflow run: 27892016371
  • Workflow attempt: 1

@opencode-agent opencode-agent Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

The changes in PR #362 include updates to the OpenCode review workflow and a new test script. Structural exploration confirmed no regressions or broken contracts. No failed GitHub Checks were present.

Findings

No blocking findings from OpenCode's independent review.

Verification

  • Review source: independent OpenCode review of the current checkout, focused changed hunks, and current-head GitHub Check evidence.
  • Structural exploration: completed before approval; if structural exploration, changed-file inspection, or evidence completeness is missing, OpenCode must not approve.
  • Result: APPROVE
  • Reason: No source-backed blockers found; structural exploration completed.

Gate evidence

  • Head SHA: b02b02ac2148336a8cf44fcbcbf5758228b7bac2
  • Workflow run: 27892016371
  • Workflow attempt: 1

@seonghobae seonghobae merged commit c31a869 into develop Jun 21, 2026
25 checks passed
@seonghobae seonghobae deleted the codex/opencode-fact-gate-20260621 branch June 21, 2026 03:37
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