[opencode] make bonk fix requests implementation-first#12993
Conversation
|
|
Codeowners approval required for this PR:
Show detailed file reviewers |
|
The change is to The restructuring is logical:
No logic bugs, security issues, backward compatibility concerns, or incorrect API behavior. The file is an internal prompt with no runtime impact on any published package. LGTM |
There was a problem hiding this comment.
Clean restructuring that directly addresses the problem described in #12970. The XML-tagged sections, explicit mode precedence, and positive/negative examples should all help anchor behavior toward implementation when action is requested.
Two minor observations (non-blocking):
-
The
<non_negotiable_rules>section packs a lot of behavioral guidance (action bias, PR bias, thread-context bias) alongside scope constraints. If Bonk still drifts, splitting the biases into their own section could make them harder to skip over during prompt processing — but worth seeing how this version performs first. -
The implementation workflow step 4 is doing double duty: on issues it searches for overlaps, on PRs it establishes the current PR as the target. These are quite different actions. If step 4 grows more complex later, splitting it would improve clarity.
Neither of these is worth changing now — the restructure is a clear improvement over the previous version. LGTM.
|
Approved PR #12993. The restructuring is solid — the XML-tagged sections, explicit mode precedence (implementation > review > triage), and concrete positive/negative examples should effectively anchor Bonk toward direct implementation when action verbs are used, which addresses the drift toward suggestion-only comments seen in #12970. Left two minor non-blocking observations about potential future refinements. |
create-cloudflare
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-editor-shared
wrangler
commit: |
This updates the Bonk agent prompt so PR fix requests are treated as implementation work instead of suggestion-only review.
The current prompt gives review behavior more specific guidance than implementation behavior, which makes Bonk drift toward comments like the ones on #12970 instead of directly updating the branch.
.opencode/agents/bonk.mdcc @penalosa