Skip to content

fix: unbreak Bonk - #151

Closed
andre-j3sus wants to merge 1 commit into
mainfrom
ajesus/test-bonk-gpt
Closed

fix: unbreak Bonk#151
andre-j3sus wants to merge 1 commit into
mainfrom
ajesus/test-bonk-gpt

Conversation

@andre-j3sus

@andre-j3sus andre-j3sus commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Bonk has posted nothing since 2026-08-18. Three faults stacked, each hiding the
next: OpenCode 1.15.13 reported nothing at all on failure, claude-opus-4-8 is
not a valid model id, and the gateway's upstream Anthropic key is invalid, so
every anthropic/* model fails auth regardless of id.

Moves to gpt-5.6-terra with variant: high, matching workerd. Not the larger
gpt-5.6-sol, which returns 400 for function tools whenever reasoning_effort
is set — fatal for a reviewer that reads files, and why sol has no working run
anywhere in the org.

@andre-j3sus andre-j3sus self-assigned this Aug 24, 2026
Comment thread .github/workflows/bonk.yml Outdated
# 1.15.13 reported nothing at all when the model call failed, which
# hid this breakage for three weeks. The dev build is what workerd
# runs against these models.
opencode_dev: true

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

opencode_dev: true makes the action run bun install -g opencode-ai@dev, so every privileged review run downloads an unpinned moving build with access to the gateway credentials. This negates the SHA pin above and permits a compromised dev publish to exfiltrate those secrets. Use a reviewed immutable OpenCode release instead (or pin the exact package artifact/version) before enabling this workflow.

@ask-bonk

ask-bonk Bot commented Aug 24, 2026

Copy link
Copy Markdown

Changes Bonk to GPT-5.6-terra, pins the action SHA, adds runtime guardrails, and documents the setup.

Submitted 1 inline security finding: the unpinned opencode-ai@dev install can execute moving code with gateway credentials, defeating the action SHA pin.

github run

@andre-j3sus
andre-j3sus force-pushed the ajesus/test-bonk-gpt branch from 319ed03 to 62fefe2 Compare August 25, 2026 07:31
@andre-j3sus andre-j3sus changed the title test: evaluate gpt-5.6-terra for Bonk (do not merge) fix: unbreak Bonk Aug 25, 2026
Bonk has posted nothing since 2026-08-18. Three faults stacked: OpenCode
1.15.13 reported nothing on failure, hiding the rest; claude-opus-4-8 is not a
valid model id; and the gateway's upstream Anthropic key is invalid, so every
anthropic/* model fails auth regardless.

Moves to gpt-5.6-terra with variant: high. Not the larger sol, which returns
400 for function tools whenever reasoning_effort is set — fatal for a reviewer
that reads files, and why sol has no working run anywhere in the org.

Unpins the version. The pin is what let this rot for three weeks: a stale
version fails silently, where latest fails loudly and early.
@andre-j3sus
andre-j3sus force-pushed the ajesus/test-bonk-gpt branch from 62fefe2 to 9aa80a5 Compare August 25, 2026 08:37
@andre-j3sus

Copy link
Copy Markdown
Collaborator Author

Reopening to fire the opened trigger against opencode-ai@latest.

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