Repository navigation
feat: Phase 0 — install and initialize gh-aw in .github-private #276
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 6 commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
8456717
feat: Phase 0 — install and initialize gh-aw in .github-private (clos…
788f6cd
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 8d5ac05
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 2136990
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 0472e86
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 6d3902c
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry b0707af
chore: apply manual instructions [skip ci-relay]
donpetry-bot f3c344c
chore: apply manual instructions [skip ci-relay]
donpetry-bot 7af2259
fix(ci): auto-fix for SonarCloud Code Analysis [skip ci-relay]
donpetry-bot af09072
fix(ci): auto-fix for SonarCloud Code Analysis [skip ci-relay]
donpetry-bot c3d3eaa
chore: apply manual instructions [skip ci-relay]
donpetry-bot 4d5637c
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry c8080c0
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 59b0e8f
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 9d2f609
fix(reviews): address review comments [skip ci-relay]
donpetry-bot 9edb44f
fix(reviews): address review comments [skip ci-relay]
donpetry-bot 8c12622
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 3db690c
fix(reviews): address review comments [skip ci-relay]
donpetry-bot ace0631
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 04ee301
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 21cc4c4
fix(reviews): address review comments [skip ci-relay]
donpetry-bot 13479f7
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 23556eb
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry e16e7ca
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry ff2fc97
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry cba0449
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry a7bb213
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 5e2b544
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 8bf73b4
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry a900c25
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry bf14278
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 2d51e10
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry a74527e
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 760d1f9
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry e5fd179
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 3c94ddb
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 6c5d13a
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 72b5dd2
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 08568df
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 210931e
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 2ba36c7
Merge branch 'main' into dev-lead/issue-230-20260518-1642
don-petry 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
Some comments aren't visible on the classic Files Changed page.
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,36 @@ | ||
| --- | ||
|
don-petry marked this conversation as resolved.
Resolved
|
||
| name: agentic-workflows | ||
|
don-petry marked this conversation as resolved.
Resolved
|
||
| description: > | ||
| Dispatcher for gh-aw agentic workflows in petry-projects/.github-private. | ||
| Routes GitHub events to the appropriate workflow definition and executes | ||
| them using Claude as the engine. | ||
| engine: claude | ||
| tools: ["read", "search", "execute"] | ||
|
don-petry marked this conversation as resolved.
Resolved
don-petry marked this conversation as resolved.
Resolved
don-petry marked this conversation as resolved.
Resolved
|
||
| --- | ||
|
|
||
| You are the agentic workflow dispatcher for `petry-projects/.github-private`. | ||
|
|
||
| ## Engine | ||
|
|
||
| This dispatcher uses Claude via `CLAUDE_CODE_OAUTH_TOKEN` as the execution engine. | ||
|
don-petry marked this conversation as resolved.
Resolved
|
||
|
|
||
| ## Workflows | ||
|
|
||
| Dispatch to the appropriate workflow based on the GitHub event context: | ||
|
don-petry marked this conversation as resolved.
Resolved
|
||
|
|
||
| - `issue-triage` — triages newly opened issues, applies labels, and routes to the right team | ||
| - `ci-failure-analyst` — analyzes CI failures on PRs and posts a root-cause summary | ||
|
|
||
| ## Dispatch rules | ||
|
|
||
| 1. Read the incoming event payload from context | ||
| 2. Match against available workflow definitions in `.github/workflows/*.md` | ||
| 3. Execute the matched workflow with the event payload | ||
| 4. Post results back to the triggering issue, PR, or workflow run | ||
|
|
||
| ## Constraints | ||
|
|
||
| - Never modify `.github/workflows/agent-shield.yml` | ||
| - Follow org standards in `AGENTS.md` and `petry-projects/.github/AGENTS.md` | ||
| - Use `GITHUB_TOKEN` for GitHub API calls; use `CLAUDE_CODE_OAUTH_TOKEN` for Claude invocations | ||
| - Do not store or log secrets | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,70 @@ | ||
| # Agentic Workflow Tests | ||
|
|
||
| This directory contains scenario specs and smoke tests for `gh-aw` agentic workflows. | ||
|
|
||
| ## Test methodology | ||
|
|
||
| Tests follow a three-level approach: | ||
|
|
||
| 1. **Compile validation (CI gate)** — `gh aw compile` runs on every PR that touches `.github/workflows/*.md`. A compile failure blocks merge. No runtime required. | ||
|
|
||
| 2. **Staged smoke tests (pre-merge)** — Run a workflow against a fixture payload without posting real GitHub API side-effects. Use `--staged` to dry-run: | ||
| ``` | ||
| gh aw run <workflow-name> --staged --payload tests/aw/<workflow-name>/fixtures/<fixture>.json | ||
|
don-petry marked this conversation as resolved.
Resolved
|
||
| ``` | ||
| Smoke tests assert that the workflow reaches the expected terminal state and produces the expected output structure. | ||
|
don-petry marked this conversation as resolved.
Resolved
|
||
|
|
||
| 3. **Integration tests (manual / scheduled)** — Execute against a real GitHub event in a sandbox repo. Reserved for release-gate validation; not required on every PR. | ||
|
|
||
| ## Scenario spec format | ||
|
|
||
| Write scenario specs **before** the workflow markdown. Each spec lives in `tests/aw/<workflow-name>/` as a plain markdown file: | ||
|
|
||
| ``` | ||
| tests/aw/<workflow-name>/ | ||
| README.md ← scenario specs (this pattern) | ||
| fixtures/ ← JSON payloads matching GitHub webhook schemas | ||
| smoke.sh ← staged smoke test runner (optional) | ||
| ``` | ||
|
|
||
| A scenario spec block: | ||
|
|
||
| ```markdown | ||
| ### Scenario: <name> | ||
|
|
||
| **Given** <precondition> | ||
| **When** <event / trigger> | ||
| **Then** <expected outcome> | ||
|
|
||
| **Fixture:** `fixtures/<name>.json` | ||
| ``` | ||
|
|
||
| ## Running compile validation locally | ||
|
|
||
| ```bash | ||
| curl -sL https://raw.githubusercontent.com/github/gh-aw/main/install-gh-aw.sh | bash | ||
|
don-petry marked this conversation as resolved.
Resolved
|
||
| gh aw compile | ||
| ``` | ||
|
|
||
| The compile step validates all `*.md` files in `.github/workflows/` against the gh-aw schema. It does not invoke Claude or make API calls. | ||
|
don-petry marked this conversation as resolved.
Resolved
|
||
|
|
||
| ## Running staged smoke tests | ||
|
|
||
| ```bash | ||
| # Run a single workflow against a fixture payload | ||
| gh aw run issue-triage --staged --payload tests/aw/issue-triage/fixtures/new-bug.json | ||
|
|
||
| # Run all smoke tests for a workflow | ||
| bash tests/aw/issue-triage/smoke.sh | ||
| ``` | ||
|
|
||
| ## Definition of Done checklist | ||
|
|
||
| A workflow is ready to merge when: | ||
|
|
||
| - [ ] Scenario specs written in `tests/aw/<name>/README.md` before implementation starts | ||
| - [ ] `gh aw compile` passes with no errors | ||
| - [ ] At least one fixture payload exists under `tests/aw/<name>/fixtures/` | ||
| - [ ] Staged smoke test passes: `gh aw run <name> --staged` exits 0 | ||
| - [ ] CI lint job (`gh-aw-compile`) is green on the PR | ||
| - [ ] Workflow markdown reviewed by a second author or the pr-reviewer agent | ||
Empty file.
Empty file.
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.
Uh oh!
There was an error while loading. Please reload this page.