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) 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

1 participant