agentleFS
syangasyanga/astack/skills/open-pr/SKILL.md

open-pr

Skill0 starsChanged 53 days agoMITView on GitHub

Opening a pull request. Use when work is ready for review, or when asked to open a PR.

62 other projects use a version of this.

Adopted by

Most recently taken up by

Used with

Projects with it also use review-change, commit-push and 8 more.

Other files people use alongside this one.

Files found in the same public projects as a version of this one, more often than they turn up anywhere else. Each project counts once, large collections count for less, and this file's own repository is left out.
  • thlaureSkill

    review-change

    Use this skill when the user asks for a code review, a correctness pass, a check before PR, or asks whether a Signalist change follows project conventions.

    Also in 4 projects that use this

  • DataCoveySkill

    commit-push

    Commit and push the changes made on this chat session to the current branch. Use best practices from Conventional Commits.

    Also in 3 projects that use this

  • Codagent-AISkill

    implement-change

    Implements an approved change in one continuous session, with tiered test runs. Use when asked to implement a change from its approved definition, or to invoke codagent:implement-change.

    Also in 3 projects that use this

  • saffron-healthSkill

    address-review

    Address PR review comments systematically. Use when responding to code review feedback on a pull request.

    Also in 3 projects that use this

  • SentrySkill

    write-tests

    Write tests following project conventions. Use when adding new tests or modifying existing ones.

    Also in 4 projects that use this

  • SentrySkill

    commit

    Create commit messages following Sentry conventions. Use when committing code changes, writing commit messages, or formatting git history. Follows conventional commits with Sentry-specific issue references.

    Also in 17 projects that use this

The file

We split this skill into the rules it gives an agent

5 rules. 0 are common, 2 are distinctive, and 3 are very specific to this project.

  • 0

    Common

    Most of the 61 edited versions we read keep these.

  • 2

    Distinctive

    This author's own choices. Worth a look.

  • 3

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  1. Distinctive:

    Before pushing, install the Gitleaks hook from the repository root:

    line 14
  2. Very specific:

    Use these sections in this order by default.

    line 28
  3. Very specific:

    When screenshots or recordings demonstrate the change, upload them as GitHub attachments using gh pr edit --attach or GitHub's editor.

    line 36
  4. Distinctive:

    Keep logs, SHA lists, exhaustive file inventories, and bulky evidence in linked artifacts.

    line 38
  5. Very specific:

    Create or update the PR using a file for the body.

    line 40

“Common” is measured: kept by at least 6 in 10 of the 61 edited versions of this file we could read. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

Open a PR

Check whether the branch already has an open PR and update it when it does. Review the diff against the intended base and make sure its contents match the user's goal. Keep PRs narrow and unpushed commits coherent. Preserve commits already under review and add new commits for updates.

Before opening a new PR, rebase onto the latest default branch, or its parent's current tip for a stack child. When an orchestrate brief forbids rebasing, skip the rebase and state in your report how far the branch trails its target. Independent work and stack roots target the default branch. Each child targets its parent branch. Start substantial work on a new stack from the latest default branch.

Apply no-comments to the code diff before review. Follow test results to reuse verification, run the suites that cover the change, and publish the results on the PR.

Before pushing, install the Gitleaks hook from the repository root:

python3 "<this skill's directory>/scripts/git_hooks.py" install --repo .

The installer checks an existing installation and installs one when missing. If it refuses, fix the cause before pushing or stop and report it.

Apply technical-writing to the title, description, and commit messages.

Follow the repository's title convention, using recent merged PRs and Git history as examples. Where the repository uses Conventional Commits, use type(scope): subject, with a type such as feat, fix, docs, refactor, test, chore, or perf, and the changed area as the scope. Keep the subject short and imperative, with no trailing period. Prefer a title that explains the outcome, such as fix(web): honor worktree defaults for new threads. Name a real symbol when it carries the change.

The description is a briefing for a reviewer who has the diff but has not seen this conversation. Explain why the change exists, what is out of scope when that matters, and how you verified it. Keep it readable in under a minute and within about 40 lines, since it may become the squash commit body.

Use these sections in this order by default. Include ## Verification in every PR. Omit other sections that have nothing useful to say. A small PR can put the rest in brief prose without headings.

  • ## Why. State the problem or goal and the approach in one or two short paragraphs. For a bug fix, explain what went wrong and what happens now.
  • ## Scope. Name the symbols and paths that define the boundary, including both sides of a rename or retarget. State what is in and out when that distinction matters.
  • ## Tradeoffs. Explain rejected alternatives a reviewer would reasonably ask about. Omit this section when there was no real choice.
  • ## Blast radius. In one to three sentences, name who or what the change affects and why it is safe or risky. Include the cost of leaving the problem unresolved when it helps assess the change.
  • ## Verification. Name the commands or real run paths and their results. For a performance change, give a primary measured result with its unit, before and after.

When screenshots or recordings demonstrate the change, upload them as GitHub attachments using gh pr edit --attach or GitHub's editor.

Keep logs, SHA lists, exhaustive file inventories, and bulky evidence in linked artifacts. End with the model and harness attribution. Use commit bodies to explain what the subject leaves unsaid.

Create or update the PR using a file for the body. Confirm that it targets the intended base and is open for review, not a draft. In T3 Code, link it with link_pull_request when available.

Finish the requested phase or stack first. Then run review-pr on each PR you opened or pushed to, unless the user declines review. Use fix-and-review mode when an implementation workflow invoked open-pr or when the user requested repairs. Otherwise, use report-only mode. When the PR's last complete review-pr record reviewed a commit the branch still contains, pass that commit to review-pr as the fixed point. When the user explicitly requests hardening, run harden-pr instead, unless a brief assigns hardening to another owner.

Return the URL, a brief verification summary, the review outcome, and any unresolved findings or limitations.

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.