Skip to content

Wire Kanon api-review alias#1277

Open
kelos-bot[bot] wants to merge 1 commit into
mainfrom
kelos-task-1265
Open

Wire Kanon api-review alias#1277
kelos-bot[bot] wants to merge 1 commit into
mainfrom
kelos-task-1265

Conversation

@kelos-bot
Copy link
Copy Markdown

@kelos-bot kelos-bot Bot commented Jun 3, 2026

What type of PR is this?

/kind cleanup

What this PR does / why we need it:

Wires /kelos api-review as an alias for the existing Kanon reviewer workflow by broadening both reviewer webhook body filters. Updates the Kanon workflow README so it no longer claims Kanon has no API surface, and documents that CLI/config API review is handled by kanon-reviewer.

Adds example coverage for the Kanon reviewer webhook filter so /kelos api-review works on PR comments and submitted PR reviews while /kelos review continues to work.

Which issue(s) this PR is related to:

Fixes #1265

Special notes for your reviewer:

Verified locally with make test and make verify.

Does this PR introduce a user-facing change?

NONE

Summary by cubic

Enable /kelos api-review as an alias for the Kanon reviewer so maintainers can trigger reviews via PR comments or submitted reviews using either /kelos review or /kelos api-review. Updates filters, docs, and tests; bot exclusion remains unchanged.

  • New Features
    • Expand kanon-reviewer.yaml filters to bodyPattern: /kelos (api-)?review for issue_comment and pull_request_review.
    • Update kanon-development/README.md to document the alias and clarify Kanon’s API surface (CLI/config) is reviewed by the same reviewer.
    • Add tests for the alias on PR comments and reviews, verify bot exclusion, and factor readDevelopmentTaskSpawner helper.

Written for commit 2487073. Summary will update on new commits.

Review in cubic

@github-actions github-actions Bot added needs-triage kind/cleanup needs-kind Indicates an issue or PR lacks a kind/* label needs-priority needs-actor release-note-none and removed needs-kind Indicates an issue or PR lacks a kind/* label labels Jun 3, 2026
@kelos-bot
Copy link
Copy Markdown
Author

kelos-bot Bot commented Jun 3, 2026

🤖 Kelos Worker Agent @gjkim42

/kelos review

@kelos-bot
Copy link
Copy Markdown
Author

kelos-bot Bot commented Jun 3, 2026

🤖 Kelos Task Status

Task kelos-reviewer-issue-comment-c221762f74e2 has succeeded. ✅

Copy link
Copy Markdown
Author

@kelos-bot kelos-bot Bot left a comment

Choose a reason for hiding this comment

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

🤖 Kelos Reviewer Agent @gjkim42

Review Summary

Verdict: COMMENT
Overall correctness: patch is correct
Scope: Wires /kelos api-review as a Kanon reviewer alias for PR comments/reviews, updates the Kanon README, and adds webhook filter coverage.

Findings Overview

Priority Count File:Line Summary
P0 0 none
P1 0 none
P2 0 none
P3 0 none

Findings

No qualifying findings.

Key takeaways

  • The two Kanon reviewer webhook filters now cover both /kelos review and /kelos api-review, including submitted PR reviews.
  • The README now accurately states that Kanon's CLI/config API surface is reviewed by kanon-reviewer and that both commands run the same reviewer.
  • The added example test exercises the alias, the existing review command, and the retained bot exclusion.

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

Projects

None yet

1 participant