Skip to content

Broaden Step 2 to find the PR template under every accepted spelling #108

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.

Sign up for GitHub

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

Open
dmccoystephenson wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Broaden Step 2 to find the PR template under every accepted spelling #108

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
ffe59ac
Broaden Step 2 to find the PR template under every accepted spelling
dmccoystephenson Aug 7, 2026
282849c
Record PR #108 as an Implementations entry under RESEARCH.md finding 3
dmccoystephenson Aug 7, 2026
469649b
Lead the Step 2 PR-template bullet with a path to match its siblings
dmccoystephenson Aug 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ Generic AI coding assistants treat every repo the same. A tailored dev loop enco
| Generic assistant | Tailored dev loop |
|---|---|
| Guesses your test command | Knows `./mvnw test` vs `cargo test` vs `pytest` |
| Ignores your PR template | Fills it in from `.github/pull_request_template.md` |
| Ignores your PR template | Fills it in from your PR template, whichever spelling and location it uses |
| Doesn't know your branch convention | Enforces `feature/` or `fix/` based on your `CONTRIBUTING.md` |
| Skips your linter | Runs `ruff check` / `eslint` / `golangci-lint` before every push |
| Opens PRs with placeholder reviewers | Requests the right person from `CODEOWNERS` or adds `Copilot` |
Expand Down
1 change: 1 addition & 0 deletions RESEARCH.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,7 @@ Last reviewed: 2026-08-04.
**Implementations.**
- PR #31 (Localization + regression gates): shipped 2026-05-25. Observed effect: pending — needs N cycles of data. Adds a Phase 3 "Localization verification" step (enumerate files, `test -f`, grep for the named surface area) and a Phase 8 regression gate (bug-fix `Closes #N` requires a new/modified test or validation step). Generalized to projects whose external anchor is manual validation rather than tests.
- PR #52 (Add documentation-accuracy and unit-test-expansion work-mode stages to Phase 2): shipped 2026-06-08. Observed effect: pending — needs N cycles of data. Adds Stage B, a characterization-test work-mode that locks in current behavior and builds the regression guards this finding argues are the only way to distinguish a real fix from a coincidental patch — and forbids changing production code under a test-only cycle, so an apparent bug becomes a filed issue rather than an unverified patch.
- PR #108 (Broaden Step 2 to find the PR template under every accepted spelling): shipped 2026-08-07. Observed effect: pending — needs N cycles of data. Widens Step 2's PR-template bullet from a single lowercase path to every spelling and location GitHub accepts. Applies the finding one level earlier than PR #31 does: before a generated skill can localize within its repo, Step 2 must localize the repo's own convention files, and a silently under-matching file list loses that input with no trace.

---

Expand Down
2 changes: 1 addition & 1 deletion create-dev-loop.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ Read the following files if they exist (skip silently if absent):

**Issue and PR patterns**
- `.github/CODEOWNERS` — who reviews PRs
- `.github/pull_request_template.md` — PR body requirements
- `.github/PULL_REQUEST_TEMPLATE.md` or `.github/pull_request_template.md` (either spelling is also accepted at the repo root or under `docs/`, and a `.github/PULL_REQUEST_TEMPLATE/` directory holds a repo's multiple templates) — PR body requirements. On a case-sensitive filesystem only the exact spelling present will match, so check each rather than the lowercase `.github/` path alone
- Recent closed PRs: `gh pr list --state closed --limit 5` — infer title style, squash vs merge, reviewer patterns

**Code conventions**
Expand Down
Loading