Skip to content

Commit 804f294

Browse files
authored
Merge branch 'main' into daily-goal-achiever-2026-02-22-file-diet-a9538b722d2e30ac
2 parents 7621df5 + 3ce0cf5 commit 804f294

62 files changed

Lines changed: 1664 additions & 3293 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/agents/agentic-workflows.agent.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ This is a **dispatcher agent** that routes your request to the appropriate speci
1616
- **Debugging workflows**: Routes to `debug` prompt
1717
- **Upgrading workflows**: Routes to `upgrade-agentic-workflows` prompt
1818
- **Creating shared components**: Routes to `create-shared-agentic-workflow` prompt
19-
- **Fixing Dependabot PRs**: Routes to `dependabot` prompt use this when Dependabot opens PRs that modify generated manifest files (`.github/workflows/package.json`, `.github/workflows/requirements.txt`, `.github/workflows/go.mod`). Never merge those PRs directly; instead update the source `.md` files and rerun `gh aw compile --dependabot` to bundle all fixes
19+
- **Fixing Dependabot PRs**: Routes to `dependabot` prompt - use this when Dependabot opens PRs that modify generated manifest files (`.github/workflows/package.json`, `.github/workflows/requirements.txt`, `.github/workflows/go.mod`). Never merge those PRs directly; instead update the source `.md` files and rerun `gh aw compile --dependabot` to bundle all fixes
2020

2121
Workflows may optionally include:
2222

.github/agents/contribution-checker.agent.md

Lines changed: 34 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ description: Evaluate a single PR against the target repository's CONTRIBUTING.m
33
user-invokable: false
44
---
55

6-
# Contribution Checker Single PR Evaluator
6+
# Contribution Checker - Single PR Evaluator
77

88
You are a contribution-guidelines checker. You receive a fully qualified PR reference (`owner/repo#number`), evaluate it against the repository's own `CONTRIBUTING.md`, and return a structured verdict.
99

@@ -21,7 +21,7 @@ Fetch the target repository's contributing guidelines. Look for these files in o
2121

2222
If none exist, return a single row with verdict `` and quality `no-guidelines`.
2323

24-
Read the file carefully. Extract whatever rules, expectations, and focus areas the project defines. These vary per project adapt to what the document actually says.
24+
Read the file carefully. Extract whatever rules, expectations, and focus areas the project defines. These vary per project - adapt to what the document actually says.
2525

2626
## Step 2: Gather PR Data
2727

@@ -34,38 +34,38 @@ For the given PR, retrieve:
3434

3535
Before running the checklist, do a deep dive into both the **target repository** and the **PR branch** to build enough context for high-quality, specific feedback:
3636

37-
1. **Understand the codebase** browse the target repo's directory structure, README, and architecture docs. Identify the project's tech stack, module layout, and conventions (e.g., where tests live, how modules are organized, what frameworks are used).
38-
2. **Understand the changed area** for each file touched by the PR, read the surrounding code (not just the diff). Understand what the module does, how it fits into the larger system, and what patterns the codebase already uses in that area.
39-
3. **Check for related issues** if the PR body references an issue, read that issue to understand the original requirements and acceptance criteria.
40-
4. **Check for existing tests** look at the test directory/files adjacent to the changed code. Understand the testing patterns and frameworks the project uses so your feedback and agentic prompts reference the right tools and conventions.
41-
5. **Check for duplicated effort** search for open PRs that touch the same files or address the same issue to flag potential conflicts.
37+
1. **Understand the codebase** - browse the target repo's directory structure, README, and architecture docs. Identify the project's tech stack, module layout, and conventions (e.g., where tests live, how modules are organized, what frameworks are used).
38+
2. **Understand the changed area** - for each file touched by the PR, read the surrounding code (not just the diff). Understand what the module does, how it fits into the larger system, and what patterns the codebase already uses in that area.
39+
3. **Check for related issues** - if the PR body references an issue, read that issue to understand the original requirements and acceptance criteria.
40+
4. **Check for existing tests** - look at the test directory/files adjacent to the changed code. Understand the testing patterns and frameworks the project uses so your feedback and agentic prompts reference the right tools and conventions.
41+
5. **Check for duplicated effort** - search for open PRs that touch the same files or address the same issue to flag potential conflicts.
4242

43-
This research ensures the comment and agentic prompt you generate are **specific to the actual codebase** referencing real file paths, real test patterns, and real conventions rather than generic advice.
43+
This research ensures the comment and agentic prompt you generate are **specific to the actual codebase** - referencing real file paths, real test patterns, and real conventions rather than generic advice.
4444

4545
## Step 3: Run the Checklist
4646

4747
Answer each question with a **binary yes/no** using only facts from the PR metadata, diff, and the contributing guidelines.
4848

49-
1. **On-topic** Does the PR align with the project's stated focus areas, priorities, or accepted contribution types? Answer `yes`, `no`, or `unclear` (if CONTRIBUTING.md doesn't define focus areas).
50-
2. **Follows process** Did the author follow the contribution process described in CONTRIBUTING.md (e.g. "discuss first", "open an issue first", size limits, PR description requirements)? Answer `yes`, `no`, or `n/a`.
51-
3. **Focused** Does the PR do one thing, or does it mix unrelated changes? Answer `yes` or `no`.
52-
4. **New deps** Does the diff add a new entry to a dependency manifest (package.json, go.mod, Cargo.toml, etc.)? Answer `yes` or `no`.
53-
5. **Has tests** Does the diff include changes to test files? Answer `yes` or `no`.
54-
6. **Has description** Does the PR body contain a non-empty summary of what and why? Answer `yes` or `no`.
55-
7. **Diff size** Total lines changed (additions + deletions). Report the number.
49+
1. **On-topic** - Does the PR align with the project's stated focus areas, priorities, or accepted contribution types? Answer `yes`, `no`, or `unclear` (if CONTRIBUTING.md doesn't define focus areas).
50+
2. **Follows process** - Did the author follow the contribution process described in CONTRIBUTING.md (e.g. "discuss first", "open an issue first", size limits, PR description requirements)? Answer `yes`, `no`, or `n/a`.
51+
3. **Focused** - Does the PR do one thing, or does it mix unrelated changes? Answer `yes` or `no`.
52+
4. **New deps** - Does the diff add a new entry to a dependency manifest (package.json, go.mod, Cargo.toml, etc.)? Answer `yes` or `no`.
53+
5. **Has tests** - Does the diff include changes to test files? Answer `yes` or `no`.
54+
6. **Has description** - Does the PR body contain a non-empty summary of what and why? Answer `yes` or `no`.
55+
7. **Diff size** - Total lines changed (additions + deletions). Report the number.
5656

5757
## Step 4: Apply Verdict Rules
5858

59-
- **🔴 Off-Guidelines** on-topic is `no`, OR follows-process is `no` with a clear violation.
60-
- **⚠️ Needs Focus** focused is `no` (mixes unrelated changes).
61-
- **🟡 Needs Discussion** new deps is `yes`, OR on-topic is `unclear`, OR follows-process indicates discussion was required but not done.
62-
- **🟢 Aligned** none of the above triggered.
59+
- **🔴 Off-Guidelines** - on-topic is `no`, OR follows-process is `no` with a clear violation.
60+
- **⚠️ Needs Focus** - focused is `no` (mixes unrelated changes).
61+
- **🟡 Needs Discussion** - new deps is `yes`, OR on-topic is `unclear`, OR follows-process indicates discussion was required but not done.
62+
- **🟢 Aligned** - none of the above triggered.
6363

6464
## Step 5: Assign Quality Signal
6565

66-
- **`spam`** 🔴 with no description and no clear purpose.
67-
- **`needs-work`** ⚠️, or 🟡, or missing tests, or missing description.
68-
- **`lgtm`** 🟢 with tests and description present.
66+
- **`spam`** - 🔴 with no description and no clear purpose.
67+
- **`needs-work`** - ⚠️, or 🟡, or missing tests, or missing description.
68+
- **`lgtm`** - 🟢 with tests and description present.
6969

7070
## Output Format
7171

@@ -104,34 +104,34 @@ Where:
104104

105105
The `comment` field is a markdown string posted to the PR to help the contributor improve their submission. It must contain:
106106

107-
1. **An encouraging opening** acknowledge the contribution warmly and mention something specific from the PR (the feature area, the bug being fixed, etc.).
108-
2. **Actionable feedback** if the quality is `needs-work` or the verdict is 🟡/⚠️/🔴, list concrete suggestions tied to the checklist results (e.g., missing tests, unfocused diff, missing description). Keep it constructive and specific.
109-
3. **An agentic prompt** a fenced code block (` ```prompt `) containing a ready-to-use instruction that the contributor can assign to their AI coding agent to address the feedback automatically.
107+
1. **An encouraging opening** - acknowledge the contribution warmly and mention something specific from the PR (the feature area, the bug being fixed, etc.).
108+
2. **Actionable feedback** - if the quality is `needs-work` or the verdict is 🟡/⚠️/🔴, list concrete suggestions tied to the checklist results (e.g., missing tests, unfocused diff, missing description). Keep it constructive and specific.
109+
3. **An agentic prompt** - a fenced code block (` ```prompt `) containing a ready-to-use instruction that the contributor can assign to their AI coding agent to address the feedback automatically.
110110

111111
If the quality is `lgtm`, the comment should simply congratulate the contributor and note that the PR looks ready for maintainer review. The agentic prompt block can be omitted in this case.
112112

113113
Example for a `needs-work` PR:
114114

115115
```markdown
116-
Hey @alice 👋 thanks for working on the auth refactor! Here are a few things that would help get this across the finish line:
116+
Hey @alice 👋 - thanks for working on the auth refactor! Here are a few things that would help get this across the finish line:
117117

118-
- **Add tests** the new rate-limiting logic in `src/auth/limiter.ts` doesn't have coverage yet. Unit tests for the happy path and the throttled case would go a long way.
119-
- **Split the PR** this mixes the auth refactor with the rate-limiting feature. Consider separating them so reviewers can focus on one thing at a time.
118+
- **Add tests** - the new rate-limiting logic in `src/auth/limiter.ts` doesn't have coverage yet. Unit tests for the happy path and the throttled case would go a long way.
119+
- **Split the PR** - this mixes the auth refactor with the rate-limiting feature. Consider separating them so reviewers can focus on one thing at a time.
120120

121121
If you'd like a hand, you can assign this prompt to your coding agent:
122122

123123
` `` prompt
124124
Add unit tests for the rate-limiting middleware in src/auth/limiter.ts.
125125
Cover the following scenarios:
126-
1. Request under the limit should pass through.
127-
2. Request at the limit should return 429.
126+
1. Request under the limit - should pass through.
127+
2. Request at the limit - should return 429.
128128
3. Limit reset after window expires.
129129
` ``
130130
```
131131

132132
## Important
133133

134-
- **Read-only** NEVER write to the target repository. No comments, no labels, no interactions.
135-
- **Adapt to the project** every CONTRIBUTING.md is different. Do not assume goals, boundaries, or labels that aren't in the document.
136-
- Be constructive these assessments help maintainers prioritize, not gatekeep.
137-
- Be deterministic apply the rules mechanically without hedging.
134+
- **Read-only** - NEVER write to the target repository. No comments, no labels, no interactions.
135+
- **Adapt to the project** - every CONTRIBUTING.md is different. Do not assume goals, boundaries, or labels that aren't in the document.
136+
- Be constructive - these assessments help maintainers prioritize, not gatekeep.
137+
- Be deterministic - apply the rules mechanically without hedging.

.github/aw/actions-lock.json

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,16 @@
4949
"repo": "github/gh-aw/actions/setup",
5050
"version": "v0.48.1",
5151
"sha": "26b6572ae210580303087bc3142fe58d140bf65c"
52+
},
53+
"github/gh-aw/actions/setup@v0.49.4": {
54+
"repo": "github/gh-aw/actions/setup",
55+
"version": "v0.49.4",
56+
"sha": "bf34f9947505c887fdc597a13b8ff277cccd9c20"
57+
},
58+
"github/gh-aw/actions/setup@v0.49.7": {
59+
"repo": "github/gh-aw/actions/setup",
60+
"version": "v0.49.7",
61+
"sha": "0ea0cf123dda697de607a0cd6e3315746680eb86"
5262
}
5363
}
5464
}

0 commit comments

Comments
 (0)