CtrlK
BlogDocsLog inGet started
Tessl Logo

work-with-pr

Full PR lifecycle in a fresh task-owned git worktree: implement via the ulw-loop skill with mandatory evidence-bound manual QA → reviewer-readable English PR → verification loop (CI + Cubic, where Cubic is skipped only when its quota is exhausted) → merge by default → worktree cleanup. Decomposes one task into the smallest atomic, independently-mergeable PRs and builds the independent ones concurrently via one worktree per PR driven by parallel subagents or a team. Unbounded loop: any failing gate sends you back to fix-and-re-QA inside that PR's worktree. Use whenever implementation work needs to land as a PR. Triggers: 'create a PR', 'implement and PR', 'work on this and make a PR', 'implement issue', 'land this as a PR', 'split into atomic PRs', 'parallel PRs', 'work-with-pr', 'PR workflow', 'implement end to end', even when user just says 'implement X' if the context implies PR delivery.

69

Quality

84%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

Low

Low-risk findings worth noting

SKILL.md
Quality
Evals
Security

Quality

Content

77%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

An exceptionally actionable and well-sequenced lifecycle skill: executable commands at every step, explicit gates, feedback loops, and failure handling. Its weaknesses are repetition of the same prohibitions (polling, quota-skip) three-plus times each, and a monolithic ~400-line body that inlines templates and monitor recipes which belong in one-level-deep reference files.

Suggestions

Extract the PR body template and the three monitor subscription recipes (CI completion, Cubic review arrival, merge completion) into a single one-level-deep references file, keeping SKILL.md as the lifecycle overview with well-signaled pointers.

State each repeated rule once: the 'never block a model round-trip on --watch / polling loops' guidance and the 'Cubic SKIPPED only on quota exhaustion' rule each appear 3+ times across phases, the verify loop, Gate B, Phase 4, and the anti-patterns table — consolidate to a single authoritative statement and cross-reference it.

Deduplicate the anti-patterns table against in-body rules (e.g., the 200-vs-2000-line atomicity rationale and the scope-creep warning each appear twice) so each violation is explained in one place.

DimensionReasoningScore

Conciseness

The body is mostly efficient — no tutorial filler, no explanation of concepts Claude already knows — but it states the same rules multiple times: 'never block a model round-trip on `gh pr checks --watch`' appears in the Gate A prose, the monitor comment block, and the Cubic section ('never spin a `for _ in $(seq 1 30)` polling loop'), and the Cubic-quota-skip rule is repeated in the Phase 3 intro, the verify-loop pseudocode, Gate B, Phase 4, and the anti-patterns table; the 200-vs-2000-line review rationale is also stated twice. This fits anchor 3 ('mostly efficient… could be tightened') rather than 4, where only minor trimming would be needed.

3 / 5

Actionability

Every phase is backed by copy-paste-ready commands: worktree/branch setup bash, pre-push checks, `gh pr create` with a fully specified PR body template, Cubic review retrieval with a three-branch grep classifier, CI log retrieval, auto-merge commands, and monitor subscription recipes. The only pseudo-code (the `while true` loop and `monitor({...})` blocks) represents harness invocations where that is the native form. This matches anchor 5.

5 / 5

Workflow Clarity

The multi-step process is clearly sequenced (Phase 0–4 with an architecture diagram up front), gates are ordered with stated rationale (CI cheapest, Cubic asynchronous), the failure route back to Phase 1 is explicit, and validation checkpoints abound: evidence-bound manual QA before commit, pre-push local validation, the two-gate verify loop, iteration discipline, failure recovery, and an anti-patterns severity table. This is a textbook match for anchor 5's feedback-loop and checklist profile.

5 / 5

Progressive Disclosure

The body is well-sectioned (phases plus <setup>/<implementation>/<verify_loop> tags), but it is a ~400-line monolithic file with no bundle: the PR body template, the three monitor subscription recipes, and the Cubic parsing/classification logic are content that would sit better in one-level-deep reference files. That fits anchor 3 ('some structure… content that should be separate is inline') rather than 4, which requires most content appropriately placed across files.

3 / 5

Total

16

/

20

Passed

Description

91%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A strong description: comprehensive concrete capabilities, an explicit 'Use when' clause, and a rich natural-language trigger list. Its only deductions are a second-person slip ('sends you back') that costs it specificity, and the intentionally broad 'implement X' fallback that raises mild conflict risk with plain implementation skills.

DimensionReasoningScore

Specificity

The description lists multiple concrete actions covering the whole lifecycle ('implement via the ulw-loop skill with mandatory evidence-bound manual QA → reviewer-readable English PR → verification loop (CI + Cubic…) → merge by default → worktree cleanup', 'builds the independent ones concurrently via one worktree per PR driven by parallel subagents or a team'), matching the comprehensive anchor 5 — but it slips into second person ('any failing gate sends you back to fix-and-re-QA inside that PR's worktree'), which the judging guidelines penalize by one point. It is not a 3 because the action coverage is comprehensive and highly concrete, not merely 1-2 actions.

4 / 5

Completeness

It explicitly answers both questions: what ('Full PR lifecycle in a fresh task-owned git worktree: … merge by default → worktree cleanup') and when ('Use whenever implementation work needs to land as a PR' followed by concrete trigger phrases). This matches anchor 5 exactly; anchor 4 would require the 'when' to be less explicit, which it is not.

5 / 5

Trigger Term Quality

The explicit trigger list is comprehensive and natural: 'create a PR', 'implement and PR', 'work on this and make a PR', 'implement issue', 'land this as a PR', 'split into atomic PRs', 'parallel PRs', 'PR workflow', 'implement end to end', plus the fallback 'even when user just says "implement X" if the context implies PR delivery'. These are phrasings a user would actually say, matching anchor 5; no common variation is missing.

5 / 5

Distinctiveness Conflict Risk

It carves out a clear niche (PR delivery lifecycle with worktree isolation and CI/Cubic gates) with distinct triggers, but the clause 'even when user just says "implement X" if the context implies PR delivery' deliberately broadens it toward generic implementation requests, creating minor overlap with general implementation skills. Anchor 5 requires minimal conflict risk; this is a notch below.

4 / 5

Total

18

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository
code-yeongyu/oh-my-openagent
Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.