fix(automation): prioritize demonstrated user pain
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 68/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- github, javascript, python
- Ambito
- developer-experience, testing-qa, tooling
Direzione di ricerca
Start with skills/github-issue-triage/scripts/worker.py and skills/github-pr-reviewer/scripts/worker.py, then read their contract tests in tests/test_github_triage_delivery.py and tests/test_github_reviewer_delivery.py. Run uv run pytest -q tests/test_github_triage_delivery.py tests/test_github_reviewer_delivery.py before inspecting the related README files and bundle build. Done means the specified priority, handoff, invalid-reference, documentation, and regression-test behavior passes, with automations/bundle-index.js regenerated.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
Issue triage currently classifies too many issues as medium or high priority without evidence of user pain, distorting backlog ordering. The reviewer handoff also treats approved low-priority work the same as higher-priority work.
Scope
- Default issue triage to
priority:low. - Require recorded user pain or demonstrated product impact before escalating priority.
- Avoid maintainer handoff after approval when the PR closes a low-priority issue, while preserving handoff for maintainer-decision scope stops.
- Handle invalid linked issue references without failing the reviewer scan.
- Add regression coverage.
Implementation: #681.
This issue was created by an AI agent (OpenHands) on behalf of Graham Neubig.
OpenHands AI triage
The following comments and acceptance criteria were added by the OpenHands AI agent.
Triage
Bounded scope: a policy change to two existing automations in this repository, plus their contract tests. The issue-triage worker prompt in skills/github-issue-triage/scripts/worker.py currently tells the agent to prioritize "high or normal"; the PR reviewer's _finish_completed_review in skills/github-pr-reviewer/scripts/worker.py requests a maintainer after any exact-head approval or maintainer-decision scope stop. No user-pain evidence (a real user report, support or customer evidence, production impact, or a reproduced failure in the actual product or workflow) is recorded on this issue, so it is prioritized priority:low.
Interpretation: "closes a low-priority issue" means at least one same-repository closing reference in the PR body (a close[sd]/fix(e[sd])/resolve[sd] keyword followed by #N) resolves to an issue carrying priority:low, matching the adjacent implementation. A closing reference that cannot be resolved in the repository (a nonexistent number, or a reference to another repository) is skipped and must not abort the scan. The suppression applies to approvals only; a 🛑 MAINTAINER DECISION REQUIRED scope stop still requests a maintainer.
Non-goals: changing the reviewer's required-check gate, per-scan intake bound, review publication, or stale-head handling; changing the triage acceptance-criteria or DECISION NEEDED behavior beyond priority selection; adding any label beyond the existing priority:high/priority:medium/priority:low; and implementing or merging PR #681.
Acceptance Criteria
- The issue-triage worker prompt (
skills/github-issue-triage/scripts/worker.py) instructs the agent to default topriority:low, and to assignpriority:mediumorpriority:highonly when the issue or human discussion records clear user pain or demonstrated product impact, naming the qualifying evidence (a real user report, support or customer evidence, production impact, or a reproduced failure in the actual product or workflow). - The triage prompt states that technical possibility, code inspection, or a synthetic/unit reproduction alone stays
priority:low, and listspriority:high,priority:medium, andpriority:lowamong the labels it may create while preserving unrelated labels, replacing any existing priority label, and keepingready-for-devgated on the scope/criteria standard. - After an exact-head approval of a PR whose same-repository closing issue carries
priority:low, the reviewer (skills/github-pr-reviewer/scripts/worker.py) does not request a maintainer review, and the PR is otherwise completed as today (label cleared / scan not redelivered). - A
🛑 MAINTAINER DECISION REQUIREDscope stop still requests a maintainer review even when the PR's linked issue carriespriority:low(the suppression applies to approvals only, not to scope stops). - A PR whose linked issue is not
priority:low(for examplepriority:medium,priority:high, or no priority label) still requests a maintainer review after approval, preserving existing behavior. - When a PR body contains a closing reference to an issue that cannot be resolved in the repository (a nonexistent number, or a reference to another repository), the reviewer scan skips that reference, does not raise, and continues so the PR's review completion and handoff are unaffected.
- Regression coverage: contract tests in
tests/test_github_triage_delivery.pyandtests/test_github_reviewer_delivery.pyassert the triage prompt's default-to-low and evidence wording, the reviewer handoff suppression for an approval that closes apriority:lowissue, the preserved handoff for a maintainer-decision scope stop and for a non-low-priority linked issue, and that an unresolvable closing reference does not fail the scan. -
uv run pytest -q tests/test_github_triage_delivery.py tests/test_github_reviewer_delivery.pypasses, andnpm run build:automationsregeneratesautomations/bundle-index.jsso the bundled triage and reviewer sources match the edited sources (the automation bundles inline these files). - Per AGENTS.md ("If you change a skill's behavior or scope, update its README.md accordingly"),
skills/github-issue-triage/README.mdandskills/github-pr-reviewer/README.md(and any catalogmanifest.jsondescription of the handoff) are updated to describe the new priority default and the narrowed approval handoff.
- Lingua principale
- Python
- Stelle
- 172
- Fork
- 117
- Merge medio
- 1g 7h
- PR unite (30g)
- 56
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di OpenHands/extensions
-
priority:low
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
OpenHands/extensions#829 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Add Employed by Guaq AI (hosted job-search MCP) to the integrations catalogForse già presa @buddywhitman l’ha presa 12 giorni fa. Apertapriority:low ready-for-dev
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
OpenHands/extensions#769 ·
I maintainer di solito rispondono entro 1 giorno
-
Add Spicrawl web-scraping MCP catalogue entryForse già presa @DibbayajyotiRoy l’ha presa 11 giorni fa. Apertapriority:low ready-for-dev
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
OpenHands/extensions#738 ·
I maintainer di solito rispondono entro 1 giorno
-
Add IsMalicious threat-intelligence MCP catalogue entryForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertapriority:low ready-for-dev
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
OpenHands/extensions#721 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
OpenHands/extensions#605 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di OpenHands/extensions
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
google-agentic-commerce/AP2#388 ·
-
area:codegen backend:cuda backend:hip bug direction:input
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
CrossGL/crosstl#3061 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
rlaope/oh-my-hermes#2304 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
mochan-tk/agentic-dev-kit-for-codex#64 ·
I maintainer di solito rispondono entro 1 giorno
-
Windows ARM memory-leakApertabug docs performance
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
I maintainer di solito rispondono entro 3 giorni