Skip to content

ci: un-bonk bonk - #3138

Merged
anish-sahoo merged 6 commits into
mainfrom
debug/bonk-reviewer-hang
Aug 5, 2026
Merged

ci: un-bonk bonk#3138
anish-sahoo merged 6 commits into
mainfrom
debug/bonk-reviewer-hang

Conversation

@anish-sahoo

@anish-sahoo anish-sahoo commented Aug 5, 2026

Copy link
Copy Markdown
Member
  • Switch Bonk from the deprecated, rate-limited kimi-k2.5 to paid kimi-k2.7-code.
  • Update opencode to 1.18.13 and set its timeout below the 30-minute job limit.
  • Print opencode's INFO logs in the Actions output so provider errors are visible.

Every reviewer run since 2026-07-28 has hung at "Sending message to
opencode..." and been killed at the 30 minute job timeout with no
output. Last good run was 2026-07-27; every run since has failed.

Three changes:

- Swap cf-gateway/kimi-k2.5 for cf-gateway/nemotron-3-120b.
  @cf/moonshotai/kimi-k2.5 was deprecated on 2026-05-30 and now aliases
  to @cf/moonshotai/kimi-k2.6, which changed where reasoning content
  lands in the response (reasoning_content -> reasoning) and became
  Workers Paid only on 2026-07-28. @cf/nvidia/nemotron-3-120b-a12b keeps
  the 256k context, supports function calling, and is neither deprecated
  nor plan-gated.

- Set OPENCODE_TIMEOUT to 25m. The harness defaults to 45m while both
  jobs cap at timeout-minutes: 30, so opencode's own timeout and retry
  handling could never fire and the runner killed it with no diagnostic.

- Add a temporary Probe AI Gateway step that curls the gateway directly
  before opencode starts, logging status, latency, the model the gateway
  actually resolves to, and the raw stream shape for both models. Remove
  once the cause is confirmed.

The kimi-k2.5 entry stays in opencode.json so the probe can compare the
two side by side.
Copilot AI review requested due to automatic review settings August 5, 2026 20:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ask-bonk

ask-bonk Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

LGTM

github run

Replaces the throwaway gateway probe with opencode's own logging.

The reviewer hang was diagnosable in seconds once we saw the response:
the gateway returned 429 "inference request per min rate reached" for
the deprecated @cf/moonshotai/kimi-k2.5. opencode retried it silently
and printed nothing, so the job just sat there until the runner killed
it. The error existed the whole time, it was only ever written to
opencode's log file inside the runner.

--print-logs and --log-level are plain sugar over OPENCODE_PRINT_LOGS
and OPENCODE_LOG_LEVEL (packages/opencode/src/index.ts:67-68), and the
harness spawns opencode with the ambient environment, so setting the two
variables streams the logs to stderr and into the job log. No flags to
plumb through and nothing to dump after the fact.

Also bumps the pinned opencode from 1.2.27, which is roughly 11,700
versions behind, to the current 1.18.13.

Drops the kimi-k2.5 entry from opencode.json now that nothing references
it.
Copilot AI review requested due to automatic review settings August 5, 2026 20:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ask-bonk

ask-bonk Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

LGTM

github run

DEBUG is more noise than the reviewer job needs. INFO still carries
provider errors, which is the whole point of turning the logs on.
Copilot AI review requested due to automatic review settings August 5, 2026 20:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@anish-sahoo anish-sahoo changed the title ci: debug bonk reviewer hangs, move off deprecated kimi-k2.5 ci: fix bonk reviewer hangs, surface opencode logs Aug 5, 2026
@anish-sahoo
anish-sahoo marked this pull request as ready for review August 5, 2026 20:18
@anish-sahoo
anish-sahoo requested a review from a team as a code owner August 5, 2026 20:18
@ask-bonk

ask-bonk Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

LGTM

github run

Updated OPENCODE_TIMEOUT to 25m for better retry handling.

Signed-off-by: Anish Sahoo <anishsahoo2005@gmail.com>
Copilot AI review requested due to automatic review settings August 5, 2026 20:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ask-bonk

ask-bonk Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

LGTM

github run

Updated OPENCODE_TIMEOUT to provide more retry time.

Signed-off-by: Anish Sahoo <anishsahoo2005@gmail.com>
Copilot AI review requested due to automatic review settings August 5, 2026 20:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ask-bonk

ask-bonk Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

LGTM

github run

@anish-sahoo anish-sahoo changed the title ci: fix bonk reviewer hangs, surface opencode logs ci: un-bonk bonk Aug 5, 2026
Copilot AI review requested due to automatic review settings August 5, 2026 20:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@anish-sahoo
anish-sahoo enabled auto-merge August 5, 2026 20:57
@anish-sahoo
anish-sahoo added this pull request to the merge queue Aug 5, 2026
Merged via the queue into main with commit 9a62f0c Aug 5, 2026
43 of 44 checks passed
@anish-sahoo
anish-sahoo deleted the debug/bonk-reviewer-hang branch August 5, 2026 21:10
@ask-bonk

ask-bonk Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

@anish-sahoo Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

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.

3 participants