Repository navigation
Conversation
The pull request template asks for the issue link, the atc overlap result, the lean checker's base and head counts and the paths and platforms reached. Its receipt and inference checklist now say "n/a" for changes that cannot reach inference, the kernel contract is a section only kernel changes fill, and new weights or drafters state their licence. Issue forms for bugs, performance regressions, feature requests and the issue-first proposal for a model family, backend or arithmetic change ask for the version or commit, the checkpoint and revision, the hardware and the exact command. Blank issues are off; contact links point to CONTRIBUTING, the agent radar, the known issues and the runbook.
3 of 4 tasks
Add docs/pr/SKILL.md, a pre-flight an agent runs before opening a pull request: scope and issue link, ATC overlap, rebase, the receipt, kernel and arithmetic gates, tests and lean delta, hygiene and a final check. Point to it from AGENTS.md and the README agent workflow.
jayleaton
force-pushed
the
repo/pr-skill
branch
from
October 29, 2026 11:09
a4fe45d to
542857c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this changes
Before: contributor agents had no procedure for preparing a pull request, and maintainers kept asking for the same missing pieces. After:
docs/pr/SKILL.md(tensorfold-pull-request) is a pre-flight an agent runs beforegh pr create. Its §3 and §7 map each step to a line of the revised template. AGENTS.md and README#contributing-with-agents point to it.Each rule comes from CONTRIBUTING.md, the kernel contract or the ATC skill, plus a recent survey of 100 PRs. This change doesn't touch
.github/.8ae199036). This PR adds one commit,542857cfe. The base staysmainbecause Issue forms and a sharper pull request template #1596's branch is on a fork, and GitHub only accepts a base branch in this repository. The diff therefore also shows Issue forms and a sharper pull request template #1596's.github/files until Issue forms and a sharper pull request template #1596 lands. Review8ae199036..542857cfefor this change alone.atc overlap --json(ATC 0.7.0, 2026-10-29, againstmain):complete: true.unknown.conflictson README.md. Each has the same single conflict hunk in the model table againstmainwithout this change, so the conflict is inherited and this line adds no new hunk.clean.Checks on this head
542857cfe226228b9768c2e2c8a37bdb32224661lean: 425 problems, 425 on the base, 0 new. The 425 come from the base tree.docs/pr/SKILL.md, AGENTS.md and README.md resolves (script check).git diff --checkis clean.tf-cuda-test kernel-contract-limits DIR(zig/tests/cuda/main.zig), andhip-kernel-contract-limitsis azig buildstep (zig/build/hip_contract.zig).fmtpasses; the other workflows have not started (fork workflows wait for approval).zig fmt, because no Zig source changed. No workflow runs a markdown check on docs.Receipt
n/a: this is documentation and cannot reach inference.
Checklist