-
Notifications
You must be signed in to change notification settings - Fork 0
chore(agents): upgrade DWP to 2.17.0 + AI Diff Reviewer Flow B [skip release] #73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
4 commits
Select commit
Hold shift + click to select a range
bdf7c03
chore(agents): upgrade DWP to 2.17.0 and add AI Diff Reviewer Flow B
xergioalex 2ee7e71
docs(ci): address AI review findings — command delegators + label-gat…
xergioalex 00ce17e
fix(ci): skip AI review gate on fork heads + pin action to v2.0.0
xergioalex 36e93b5
docs(release): codify the vendored agent-tooling [skip release] use case
xergioalex File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| --- | ||
| description: Read the CI review on the current PR and walk through findings (provided by the vendored `ai-diff-reviewer` skill) | ||
| --- | ||
|
|
||
| # /ai-diff-reviewer-apply-review — provided by the `ai-diff-reviewer` skill | ||
|
|
||
| > Thin alias. The flow lives in the vendored `ai-diff-reviewer` skill — this | ||
| > file only routes to it, so there is a single source of truth and no drift. | ||
|
|
||
| ## What to do | ||
|
|
||
| Route this invocation to the **apply-review** sub-skill of the vendored | ||
| `ai-diff-reviewer` skill and follow it: read | ||
| `.agents/skills/ai-diff-reviewer/apply-review/SKILL.md` and execute its flow | ||
| (anchor on the latest `<!-- ai-pr-reviewer-marker -->` comment, skip minimized | ||
| comments, per-finding apply / defer / skip consent; never commits or pushes). | ||
| This is the executable counterpart of `docs/PR_REVIEW_WORKFLOW.md`. | ||
|
|
||
| > Other agents: invoke the sub-skill directly | ||
| > (`#ai-diff-reviewer-apply-review` in Cursor/Codex/Gemini). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,19 @@ | ||
| --- | ||
| description: Regenerate `.review/extension.md` from repo evidence (provided by the vendored `ai-diff-reviewer` skill) | ||
| --- | ||
|
|
||
| # /ai-diff-reviewer-generate-extension — provided by the `ai-diff-reviewer` skill | ||
|
|
||
| > Thin alias. The flow lives in the vendored `ai-diff-reviewer` skill — this | ||
| > file only routes to it, so there is a single source of truth and no drift. | ||
|
|
||
| ## What to do | ||
|
|
||
| Route this invocation to the **generate-extension** sub-skill of the vendored | ||
| `ai-diff-reviewer` skill and follow it: read | ||
| `.agents/skills/ai-diff-reviewer/generate-extension/SKILL.md` and execute its | ||
| flow. The output lands in `.review/extension.md` — the same file CI's | ||
| `prompt-extension-file:` input reads, keeping local and CI reviews in parity. | ||
|
|
||
| > Other agents: invoke the sub-skill directly | ||
| > (`#ai-diff-reviewer-generate-extension` in Cursor/Codex/Gemini). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| --- | ||
| description: Draft the PR title + body from the current branch's diff (provided by the vendored `ai-diff-reviewer` skill) | ||
| --- | ||
|
|
||
| # /ai-diff-reviewer-open-pr — provided by the `ai-diff-reviewer` skill | ||
|
|
||
| > Thin alias. The flow lives in the vendored `ai-diff-reviewer` skill — this | ||
| > file only routes to it, so there is a single source of truth and no drift. | ||
|
|
||
| ## What to do | ||
|
|
||
| Route this invocation to the **open-pr** sub-skill of the vendored | ||
| `ai-diff-reviewer` skill and follow it: read | ||
| `.agents/skills/ai-diff-reviewer/open-pr/SKILL.md` and execute its flow | ||
| (Conventional-Commits title inference, structured body, `gh pr create`/`edit`). | ||
| Remember this repo's rule: every change ships through a branch + PR — never a | ||
| direct push to `main`. | ||
|
|
||
| > Other agents: invoke the sub-skill directly (`#ai-diff-reviewer-open-pr` in | ||
| > Cursor/Codex/Gemini). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| --- | ||
| description: Re-run the AI Diff Reviewer CI workflow wizard (provided by the vendored `ai-diff-reviewer` skill) | ||
| --- | ||
|
|
||
| # /ai-diff-reviewer-setup — provided by the `ai-diff-reviewer` skill | ||
|
|
||
| > Thin alias. The flow lives in the vendored `ai-diff-reviewer` skill — this | ||
| > file only routes to it, so there is a single source of truth and no drift. | ||
|
|
||
| ## What to do | ||
|
|
||
| Route this invocation to the **setup** sub-skill of the vendored | ||
| `ai-diff-reviewer` skill and follow it: read | ||
| `.agents/skills/ai-diff-reviewer/setup/SKILL.md` and execute its flow. In this | ||
| repo the workflow already exists at `.github/workflows/pr-review.yml` (Flow B, | ||
| label-gated on `Ready`) — setup runs are reconfiguration passes, and | ||
| `.agents/skills/ai-diff-reviewer/setup/reference.md` doubles as the reference | ||
| manual for every `action.yml` input. | ||
|
|
||
| > Other agents: invoke the sub-skill directly (`#ai-diff-reviewer-setup` in | ||
| > Cursor/Codex/Gemini). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,19 @@ | ||
| --- | ||
| description: Run a local AI review of the current branch's diff (provided by the vendored `ai-diff-reviewer` skill) | ||
| --- | ||
|
|
||
| # /ai-diff-reviewer — provided by the `ai-diff-reviewer` skill | ||
|
|
||
| > Thin alias. The flow lives in the vendored `ai-diff-reviewer` skill — this | ||
| > file only routes to it, so there is a single source of truth and no drift. | ||
|
|
||
| ## What to do | ||
|
|
||
| Route this invocation to the vendored `ai-diff-reviewer` skill's default | ||
| local-review flow: read `.agents/skills/ai-diff-reviewer/SKILL.md` and execute | ||
| it (diff vs `main`, base prompt + `.review/extension.md`, findings table in the | ||
| CI-parity format). This is the same methodology CI runs on `Ready`-labeled PRs | ||
| via `.github/workflows/pr-review.yml`. | ||
|
|
||
| > Other agents: invoke the skill directly (`#ai-diff-reviewer` in | ||
| > Cursor/Codex/Gemini). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This comment was marked as outdated.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.