feat: add /rebase command to cloudflare-docs-bot#32121
Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
|
Nimbus Preview URL: https://nimbus-fea0f6b8.preview.developers.cloudflare.com |
|
Preview URL: https://fea0f6b8.preview.developers.cloudflare.com |
c7fa096 to
1cffd2a
Compare
1cffd2a to
9dd8749
Compare
|
/full-review |
Review✅ No outstanding issues in commit Code ReviewThis code review is in beta and may not always be helpful — use your judgment. No code review issues found. ConventionsNo convention issues found. Style Guide ReviewNo style-guide issues found. Acknowledged by author (1)
CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
|
/bonk please review |
Review triage
Fix appliedThe tool was allowing abbreviated SHAs ( characters) even though the GitHub endpoint requires a full 40-character SHA. Updated:
This prevents unnecessary HTTP 422 errors when the model passes a short SHA. Overall PR assessment: LGTM. The rebase workflow is well-structured with clear guard rails (fork detection, wrong-base detection, file-size caps, production-head-race guards, completeness checks) and good failure transparency. |
Review triage
Fix appliedThe
This prevents unnecessary HTTP 422 errors when the model passes a short SHA. Overall PR assessment: LGTM. The rebase workflow is well-structured with clear guard rails (fork detection, wrong-base detection, file-size caps, production-head-race guards, completeness checks) and good failure transparency. |
|
Done. Triage:
Fix applied to
This prevents the tool from passing abbreviated SHAs to the GitHub I also left a review comment on PR #32121 with these details. |
Co-authored-by: mvvmm <mvvmm@users.noreply.github.com>
|
/rebase |
…n, fork detection, dead code)
…t, commit msg, mode, confidence)
…d tools and skill
…progress error handling, backtick strip, groot-preview, stale JSDoc
…lidation, confidence reason
Co-authored-by: mvvmm <mvvmm@users.noreply.github.com>
d306617 to
44352c7
Compare
|
CR-0fc9cd37e19c — skipping. The design is intentional. |
|
CR-18ed11a65d12 — skipping for now. Per-file confidence is a legitimate improvement but requires changing the output schema, |
|
CR-8513f9f82eee — skipping.
|
Summary
Adds a
/rebaseslash command to the cloudflare-docs-bot (.flue/). The command is codeowner-only and posts status updates at the top of the bot's singleton comment throughout the operation./rebase— Rebases the PR branch againstproduction. Stops with an explanation if:productionOn a clean rebase (no conflicts), posts a success status and triggers a
/full-reviewautomatically since the head SHA changed.On conflict, attempts AI-assisted resolution:
kimi-k2.7-code) with the file contents + PR intent to produce resolved versions and a confidence score/full-reviewOther changes:
postOrUpdateCommentextracted from the two workflow files that duplicated it intolib/code-review-render.tsas a shared exportrenderRebaseStatusUpdate()added to the render lib — injects/replaces the rebase status block at the top of the bot comment, preserving existing review content belowFlueRebaseWorkflowDurable Object class added (wrangler.jsonc v9 migration)Images