Conversation
- reuse a shared align-pr-branch component in PR fix and review commands - register the new component in config and schema so it is available consistently - restrict review workflows to trusted same-repo PRs and add comment-triggered review support
- add a debug gate job to surface PR review trigger inputs - replace the JSON contains check with explicit OWNER or MEMBER conditions - make review workflow behavior easier to diagnose when automation does not start
- remove the temporary debug gate job from the PR review workflow - allow automated reviews for same-repo branches without author association checks - keep draft and forked pull requests excluded from the review run
![kompassdev[bot]](https://avatars.githubusercontent.com/in/3075751?s=60&v=4){kind=small}
There was a problem hiding this comment.
★★★★☆
Well-structured PR. The shared branch-alignment component is clean, the workflow split between automatic and comment-triggered reviews is sensible, and the same-repo guard on the automatic workflow is a good security tightening.
Two sync gaps need attention before or shortly after merge:
1. Missing align-pr-branch in bundled config files – All three kompass.jsonc files (root, packages/core/, packages/opencode/) list every other DEFAULT_COMPONENT_NAMES entry in their components section but omit "align-pr-branch": { "enabled": true }. Runtime defaults still register it, so nothing breaks, but per AGENTS.md these surfaces must stay in sync when adding a component.
2. Missing align-pr-branch in web docs – packages/web/src/content/docs/docs/reference/components/index.mdx does not list align-pr-branch and there is no align-pr-branch.mdx doc page. Per AGENTS.md, user-facing docs that describe the changed surface should be updated in the same change.
- add align-pr-branch to the bundled component config in the root, core, and opencode manifests - document align-pr-branch in the component index and add a dedicated reference page - keep the new shared component aligned across runtime, config, and docs surfaces
|
Addressed the review follow-up in
This covers the unresolved config sync thread and the missing docs noted in review. |
1 participant
Ticket
SKIPPED
Description
Improve the PR review flows so review and fix commands operate against the actual PR branch, and limit automated review runs to safe repository and author contexts.
Checklist
Review execution
/reviewissue comments to trigger PR reviews against the PR head branchBranch safety
Configuration coverage
Validation
/reviewcomments check out the PR head branch before running review logic