Commit 7975232
fix(github-issues): ban conventional commit prefixes in issue titles (#739)
## What failed
Junior was creating GitHub issues with conventional commit-style
prefixes in their titles — `fix(attachments): ...`, `feat(sandbox):
...`, `ref(task-execution): ...` — which is wrong. That format belongs
in commit messages and PR titles only.
## Evidence
Several real issues in `getsentry/junior` with the wrong format:
- #737: `fix(attachments): durable file attachment pipeline…`
- #735: `feat: first-class agents.toml / dotagents skill support`
- #715: `feat(sandbox): make snapshot sandbox vcpus/memory configurable
via env var`
- #705: `ref(task-execution): centralize wake scheduling with
ensureConversationWake()`
- #679: `feat(memory): add recall-layer relevance gate to filter
low-relevance vector matches`
## Root cause
The `github-issues` skill had no explicit rule against conventional
commit prefixes in issue titles. The `pr-writer` skill heavily uses
`fix(scope):` for PR titles, and without a constraint in the issue skill
the model generalizes the pattern to issues too.
## Fix
Added a single prescriptive hard constraint to `SKILL.md` § Draft issue
content:
> **Issue title format: plain language, no type prefix.** Do not use
`fix(x):`, `feat(x):`, `chore:`, `ref(x):`, or any other type-scope
prefix — those belong in commit messages and PR titles, not issues.
## Verification
Content review — no automated checks applicable for skill text changes.
Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>1 parent 43e61aa commit 7975232
1 file changed
Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
| 55 | + | |
55 | 56 | | |
56 | 57 | | |
57 | 58 | | |
| |||
0 commit comments