pr-classify

分类: 工具与效率 | 上传者: hifi-phil | 下载: 0 | 版本: v1.0（最新）

Classify open PRs on the Umbraco-CMS repo by testability and help pick which PR(s) to test with /pr-test. Use this skill whenever the user mentions PRs, pull requests, reviewing PRs, triaging PRs, wants to see what's open, wants to pick a PR, or asks "what should I test?" — even if they don't explicitly say "classify". Also triggers for questions like "which PRs can be automated?", "show me testable PRs", or "what's open?".

更新日志: Source: GitHub https://github.com/hifi-phil/Umbraco_CC_Plugins

目录结构

当前层级: plugins/umb-cms-reviews/skills/pr-classify/

📁 references/
- 📄 pr-classification-heuristics.md 3.5 KB
- 📄 pr-test-extraction.md 4.9 KB
📄 SKILL.md 4.3 KB

SKILL.md

---
name: pr-classify
description: Classify open PRs on the Umbraco-CMS repo by testability and help pick which PR(s) to test with /pr-test. Use this skill whenever the user mentions PRs, pull requests, reviewing PRs, triaging PRs, wants to see what's open, wants to pick a PR, or asks "what should I test?" — even if they don't explicitly say "classify". Also triggers for questions like "which PRs can be automated?", "show me testable PRs", or "what's open?".
---

# PR Classifier

You help the developer understand which open PRs on the Umbraco-CMS repository can be automatically tested via browser automation, and which ones aren't suitable. Your tone is conversational — you're a colleague helping triage PR reviews.

## Arguments

- No arguments: classify all open PRs
- A PR number (e.g., `21887`): classify and extract test plan for just that PR

## Step 1: Fetch Open PRs

Run this command to get all open PRs with their details:

```bash
gh pr list --repo umbraco/Umbraco-CMS --state open --limit 50 --json number,title,labels,body,headRefName,baseRefName,changedFiles,additions,deletions,isDraft
```

If the user provided a specific PR number, fetch just that one:

```bash
gh pr view {PR_NUMBER} --repo umbraco/Umbraco-CMS --json number,title,labels,body,headRefName,baseRefName,changedFiles,additions,deletions,isDraft
```

## Step 2: Classify Each PR

Read `references/pr-classification-heuristics.md` for the detailed classification rules.

Classify each PR into one of three categories:

### BROWSER_TESTABLE
The PR has changes that can be verified by navigating the Umbraco backoffice and interacting with the UI. Indicators:
- Has explicit UI test steps in the description (checkbox items with action words like "Go to", "Click", "Verify", "Navigate")
- Changed files include frontend code (`.ts`, `.js` files in `src/Umbraco.Web.UI.Client/`)
- Changed files include controllers or views
- Labels include `area/frontend` or `area/backoffice`

### API_TESTABLE
The PR modifies backend behavior that can be verified through API calls but has no UI test plan. Indicators:
- Changes to API controllers, services, or repositories
- Has behavioral expectations described in the PR body
- No UI test steps but has clear before/after behavior

### NOT_TESTABLE
The PR has no user-facing behavioral change that can be tested. Indicators:
- Dependency bumps (title starts with "Bump", labels include `dependencies`)
- Draft or WIP PRs
- Pure test infrastructure changes (files only in `tests/`)
- Static asset changes with no behavioral impact
- Build/template fixes
- Pure refactoring with no behavioral change

## Step 3: Extract Test Plans

For each BROWSER_TESTABLE PR, read `references/pr-test-extraction.md` and extract:

1. **Preconditions** — what needs to exist before testing (document types, content, media, users, languages, specific configuration)
2. **Test Steps** — the specific actions to perform in the browser
3. **Expected Outcomes** — what should be visible or verifiable after each step
4. **Complexity** — rate as LOW / MEDIUM / HIGH based on how much setup and how many steps are involved

## Step 4: Present Results

Present a summary table like this:

```
| # | Title | Classification | Complexity | Notes |
|---|-------|---------------|------------|-------|
| 21887 | Upload Field: Show filename after upload | BROWSER_TESTABLE | LOW | Simple UI change, clear test steps |
| 21860 | Dependencies: Update server-side deps | NOT_TESTABLE | - | NuGet package updates only |
```

Then for each BROWSER_TESTABLE PR, provide a brief summary:

```
### PR #21887: Upload Field: Show filename after file upload
**Classification:** BROWSER_TESTABLE | **Complexity:** LOW
**What it does:** After uploading a file, the filename now appears as plain text (pre-save) or as a clickable link (post-save).
**Test steps:** Upload a file → verify filename shows → save → verify filename is now a link
**Preconditions:** Need a document type with an upload field property
```

## Step 5: Help Pick

After presenting the results, ask the developer which PR(s) they'd like to test. Suggest starting with LOW complexity PRs if this is their first time using the system. Mention they can run `/pr-test {number}` to test a specific PR.

If there are particularly interesting or important PRs (high impact, many changed files, or critical bugfixes), call those out specifically.

登录后下载/点赞/收藏 ❤ 2 | ★ 0

pr-classify

目录结构

SKILL.md

举报内容

提示