npx skills add ...
npx skills add nvidia/openshell --skill create-github-pr
Create GitHub pull requests using the gh CLI. Use when the user wants to create a new PR, submit code for review, or open a pull request. Trigger keywords - create PR, pull request, new PR, submit for review, code review.
npx skills add nvidia/openshell --skill create-github-pr
Create pull requests on GitHub using the gh CLI.
gh CLI must be authenticated (gh auth status)<issue-number>-<description>/<username>. Exempt issue-less changes may use <description>/<username>.If the branch changes gateway TOML parsing, [openshell.gateway] fields,
[openshell.drivers.<name>] fields, driver config defaults, or Helm rendering
of gateway.toml, verify that docs/reference/gateway-config.mdx is updated
in the same branch. If the change affects user-facing compute-driver setup,
also update docs/reference/sandbox-compute-drivers.mdx or the relevant
deployment docs.
Use the sync-agent-infra skill's maintenance map to identify related skill updates when the branch changes behavior, commands, or development workflows. Run its full consistency check when the branch adds, removes, or renames skills or crates; changes workflow relationships or skill coverage; modifies issue or PR templates; or changes agent cross-references. Resolve any drift before creating the PR.
Run the local pre-commit task before opening a PR:
Before creating a PR, verify:
You're not on main - Never create PRs directly from main:
Branch follows naming convention - Use <issue-number>-<description>/<initials> for issue-backed work or <description>/<initials> for an exempt issue-less change.
Consider squashing commits - For cleaner history, squash related commits before pushing:
Ensure your branch is pushed to the remote:
Basic PR creation (opens editor for description):
With title and body:
PR titles must follow the conventional commit format:
Types:
feat - New featurefix - Bug fixdocs - Documentation onlyrefactor - Code change that neither fixes a bug nor adds a featuretest - Adding or updating testschore - Maintenance tasks (CI, build, dependencies)perf - Performance improvementScope is typically the component name (e.g., evaluator, cli, sdk, jobs).
Examples:
feat(evaluator): add support for custom rubricsfix(jobs): handle timeout errors gracefullydocs(sdk): update authentication examplesrefactor(models): simplify deployment logicchore(ci): update Python version in pipelineFeatures, user-visible behavior changes, public API changes, architecture changes, and multi-PR efforts must link an accepted issue. Use Closes #<issue-number> in the body to auto-close the issue when merged:
Small documentation fixes, mechanical maintenance, and obvious localized bug fixes may omit a separate issue when the PR contains enough context to review the decision and implementation together. In that case, write No issue required: <brief reason> in the Related Issue section. Do not use this exception for security fixes; follow SECURITY.md.
For work-in-progress that's not ready for review:
Default target is main. To target a different branch:
PR descriptions must follow the project's PR template structure:
Populate the testing checklist based on what was actually run. Check boxes for steps that were completed.
| Option | Description |
|---|---|
--title, -t | PR title (use conventional commit format) |
--body, -b | PR description |
--reviewer, -r | Request review from user |
--draft | Create as draft (WIP) |
--label, -l | Add label (can use multiple times) |
--base, -B | Target branch (default: main) |
--head, -H | Source branch (default: current) |
--web | Open in browser after creation |
The command outputs the PR URL and number.
Display the URL using markdown link syntax so it's easily clickable:
If the user asks to wait for a green CI before posting the RFR, use this snippet to monitor the workflow run:
Or poll manually: