Skip to content

fix(review): preserve full-hour sessions in .github/workflows/opencode-review.yml#566

Open
seonghobae wants to merge 4 commits into
mainfrom
agent/opencode-long-timeout-bootstrap
Open

fix(review): preserve full-hour sessions in .github/workflows/opencode-review.yml#566
seonghobae wants to merge 4 commits into
mainfrom
agent/opencode-long-timeout-bootstrap

Conversation

@seonghobae

@seonghobae seonghobae commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Outcome

Preserve legitimate long-running OpenCode review sessions and retain valid model work without weakening any review, check, or ruleset gate.

Current-head evidence

  • DeepSeek V3 and DeepSeek R1-0528 were each terminated by the protected-base 600-second cap on the previous head.
  • GPT-4.1 returned a 31-line review in 86 seconds, but the protected-base normalizer rejected only the redundant exact path:line citation in adversarial probe evidence.
  • Every cadence class now allows 5,400 seconds per model because changed-file count is not a repository-complexity proxy.
  • The model pool remains bounded at 11,700 seconds inside a 12,000-second watchdog and a 205-minute Actions step.
  • The normalizer may restore only redundant path:line text when one receipt already matches the sealed current-head source line. Mismatched, duplicate, circular, malformed, or unobserved evidence remains rejected.

Verification

  • Python: 622 passed
  • Focused normalizer branch coverage: 100.00 percent, 105 passed
  • Strix/OpenCode integration suite: PASS after approximately 40 minutes
  • Ruff, actionlint syntax, bash syntax, and git diff check: PASS

No branch protection, approval count, required workflow, review identity, or security gate is relaxed.

@cwl-noema-review cwl-noema-review 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.

Noema PydanticAI review

All check and dependency gates pass, with evidence confirming that the PR's behavioral intent (to preserve full-hour model-pool review sessions in OpenCode workflows, robust R package-aware evidence, and reliable recovery from provider timeouts or HTTP 5xx failures) is implemented thoroughly. The diff delivers:

  • Increased model pool and dynamic cadence session/budget timeouts (from 10–60min to 90–195min bounds) everywhere in .github/workflows/opencode-review.yml and run_opencode_review_model_pool.sh, matching approval/test expectations.
  • Improved handling of paid session timeouts: after a hung or killed attempt, the runner attempts to recover persisted partial results via OpenCode session export by title, before falling through, tested and contract-guarded in test_opencode_model_pool_runner.py and test_opencode_agent_contract.py.
  • Improvements for R package evidence and test handling (conditionally awaiting completed package checks, improved result reporting, new output bindings) in opencode-review.yml and the test gate (test_strix_quick_gate.sh confirms workflow behavior).
  • Extra resilience and coverage in error classification, session export, logic to advance/skipping on provider failure signatures.
  • All tests, SAST, dependency, and workflow checks are green; no regressions or behavioral divergences are reported by CI, contract, or regression tests.

All new/changed behavior is protected by tests. No blocking issue is evidenced. Approving.

Findings

  • No blocking findings.

  • Result: APPROVE

  • Verdict: approve

  • Confidence: high

  • Head SHA: df408934021a206be6029838546c72094ca528ef

  • Reviewer credential: noema-github-app

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