Skip to content

bonk code reviews#2892

Merged
michaeldwan merged 4 commits intomainfrom
md/bonk
Mar 30, 2026
Merged

bonk code reviews#2892
michaeldwan merged 4 commits intomainfrom
md/bonk

Conversation

@michaeldwan
Copy link
Copy Markdown
Member

@michaeldwan michaeldwan commented Mar 30, 2026

Wire up CF bonk for code reviews. Needs to be on main to use for real it seems...

Set up ask-bonk with Kimi K2.5 via Cloudflare AI Gateway. Interactive
workflow (bonk.yml) triggers on /bonk and @ask-bonk in issue/PR comments.
Auto-review on new PRs (new-pr-review.yml) is disabled for now while
testing. Includes a cog-tailored review prompt in bonk_reviewer.md.
@michaeldwan
Copy link
Copy Markdown
Member Author

/bonk hello

@michaeldwan michaeldwan changed the title CI config update bonk Mar 30, 2026
@michaeldwan michaeldwan changed the title bonk bonk code reviews Mar 30, 2026
@michaeldwan michaeldwan marked this pull request as ready for review March 30, 2026 20:53
@michaeldwan michaeldwan requested a review from a team as a code owner March 30, 2026 20:53
@michaeldwan michaeldwan merged commit 32cce86 into main Mar 30, 2026
38 of 39 checks passed
@michaeldwan michaeldwan deleted the md/bonk branch March 30, 2026 21:00
markphelps added a commit that referenced this pull request Mar 30, 2026
….com/replicate/cog into mp/fix/dict-list-output-schema-fixes

* 'mp/fix/dict-list-output-schema-fixes' of https://github.com/replicate/cog: (44 commits)
  bonk code reviews (#2892)
  fix: clarify env variable deny-list error message (#2813)
  fix: use atomic rename in setup_subprocess_double_fork test to prevent race condition (#2815)
  chore(deps): bump github.com/docker/cli (#2885)
  chore(deps): bump ureq from 3.2.0 to 3.3.0 in /crates (#2888)
  docs: fix HTTP discovery endpoint response to match Rust implementation (#2883)
  fix: support PEP 604 unions in File/Path coercion detection (#2878)
  fix: remove mise python venv config to silence warnings (#2879)
  refactor: extract homebrew tap into standalone reusable workflow (#2881)
  chore: 0.17.1 (#2877)
  review: replace string tags with FieldKind enum, add Debug derives, remove redundant test assertion (#2874)
  fix: app id in release workflow for homebrew tap (#2875)
  fix: don't coerce URL strings in str-typed inputs (regression #2868) (#2872)
  use GHCR mirror for cog-base images in integration tests (#2866)
  docs: improve documentation for input/output types (#2864)
  add workflow to mirror cog-base images from r8.im to GHCR (#2871)
  fix: bound fuzz tree size to prevent CI timeout in FuzzJSONSchema
  fix: guard against slice panic on single-char quoted strings in parseTypeFromString
  fix: address review issues in static schema generation
  Bump version to 0.17.0
  ...
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.

2 participants