Skip to content

test: verify visible automatic bonk wrapper#360

Closed
geoquant wants to merge 6 commits intocloudflare:mainfrom
geoquant:geoquant/auto-bonk-pr-smoke-3
Closed

test: verify visible automatic bonk wrapper#360
geoquant wants to merge 6 commits intocloudflare:mainfrom
geoquant:geoquant/auto-bonk-pr-smoke-3

Conversation

@geoquant
Copy link
Copy Markdown
Collaborator

@geoquant geoquant commented Apr 2, 2026

Third smoke-test PR for the automatic Bonk workflow.

This PR targets main so the smoke test reflects the real protected-branch flow. The wrapper workflow will be triggered manually from the feature branch workflow definition to verify the intended visible UX before merge.

Expected behavior for this smoke test:

  • a visible automatic /bonk review comment appears on the PR
  • Bonk runs and leaves a review without a human typing /bonk
  • the PR remains unmergeable under the current main branch protections until the legacy required checks are addressed

  • Reviews
    • bonk has reviewed the change
    • automated review not possible because: this PR exists only to verify the Bonk workflow changes themselves
  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because: this PR only adds a markdown smoke-test file to exercise the workflow behavior under test

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

/bonk review

Triggered automatically for this PR.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 2, 2026

npm i https://pkg.pr.new/@cloudflare/kumo@360

commit: 8a567bf

Copy link
Copy Markdown
Contributor

@ask-bonk ask-bonk bot left a comment

Choose a reason for hiding this comment

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

LGTM. The automatic Bonk workflow changes are correctly implemented:

Security: Proper use of pull_request_target with NO_PUSH token permissions for auto-reviews; draft PRs are skipped.

Architecture: The wrapper pattern (bonk-check.yml → bonk.yml via workflow_dispatch) cleanly separates event handling from review execution.

UX: Visible /bonk comment provides transparency about automated activity.

Test file: The smoke test markdown is appropriate for this workflow validation PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Docs Preview

View docs preview

Commit: 8a567bf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant