[milten89]

Closes #177

What

• .claude/skills/triage/SKILL.md: /triage [issue ...] collects the issues in the project's Triage column (or the given issues), checks each against develop (still true, moved, duplicate, blocked, against an ADR), proposes Priority, Phase, Seq, Plan, Size, labels, milestone and Status in one table, asks the maintainer, and writes only the approved values. Guardrails: never maintainer-only; no closing, comments or body edits without approval; Ready only when the maintainer says so.
• A last section records how an agent triages an issue it opens for work the maintainer asked for: at once, with the step 3 rules, Status Ready (or In progress if a PR moved it), and a milestone only when the request names the release.
• CLAUDE.md: the skill in "Skills and agents"; "Issues and the project" points to it for the Ready decision and for sorting follow-ups.
• docs/GITHUB-SETUP.md: Intake mentions /triage; the Plan field mentions F<n> follow-up ids.

Differences from the issue text (maintainer decisions)

• PRs aren't triaged: they arrive without a Status and wait in the Review queue (GITHUB-SETUP Intake), so step 1 collects issues only.
• An item moves to Ready when the maintainer says so in the approval, and an issue opened at the maintainer's request is set Ready directly.

Dry run (steps 1–3, no writes)

The Triage column is empty (0 items), so the dry run replayed the five issues sorted by hand on 2026-10-30 (/triage 159 160 162 168 176):

Issue	Priority	Phase	Seq	Plan	Size	Labels	Milestone	Status	Reason
#160 Remove the unused NbpTableEnumValidator	Low	Phase 6	551	F8	XS	skip-changelog, provider:nbp	—	Backlog	Internal dead code, used only by its own test
#168 Read file_path from tool_input in the format-cs hook	Low	Phase 6	552	F9	XS	ci, area:repo	—	Backlog	Edge cases in a convenience hook; CI still verifies format
#176 Add a display path to the shared executor	Low	Phase 6	553	F10	M	enhancement, area:http	—	Backlog	Not needed for NBP or 1.0; waits for an identifier-carrying provider

Notes: #159 and #162 are closed (fixed by #181 and #180) and were skipped as step 1 says. #160, #168 and #176 were checked against develop (NbpTableEnumValidator used only by its test; format-cs.sh:8 and :11 unchanged; no display path in RestRequestExecutor), with no duplicates or blockers. The run predates the rule that an already-triaged issue keeps its Seq and Plan; with it, these three keep 435/F4, 436/F5 and 560/F6. Its other findings (item id lookup, trimmed gh issue view, Seq fallback, Git Bash path conversion, milestone issue list) are fixed in the skill, and every changed command was re-run.

Checks

• All read-only commands in the skill run against the live project (item list, next F<n> = F8, field ids, labels, milestones, issue view); the write flags (item-edit --date/--number/--text, item-add --url) checked with --help.
• git diff --check: clean. No code changes, so no build or tests.
• reviewer agent: no blocking findings; its milestone guard and the wording points are applied.
• .claude/settings.json needs no change: the read-only calls are already allowed, the writes keep prompting.

🤖 Generated with Claude Code