Skip to content

feat(skills): add commit-push-pr bundled skill#173

Merged
emal-avala merged 1 commit intomainfrom
feat/skill-commit-push-pr
Apr 23, 2026
Merged

feat(skills): add commit-push-pr bundled skill#173
emal-avala merged 1 commit intomainfrom
feat/skill-commit-push-pr

Conversation

@emal-avala
Copy link
Copy Markdown
Member

Summary

Adds `/commit-push-pr` — one invocation for the full commit-to-PR flow with verification gates between steps.

Flow:

  1. `git status` (stop if clean)
  2. branch check (stop if on base)
  3. lint + test gate (stop if broken — don't commit broken code into a PR)
  4. stage specific files (never `git add -A`)
  5. commit with conventional subject <70 chars + WHY body
  6. push (`-u` if no upstream)
  7. `gh pr create` with Summary + Test Plan body
  8. print PR URL

Guardrails in the prompt:

  • never force-push, never push to base branch, never skip hooks
  • no `Co-Authored-By` or `🤖` trailers
  • if any step fails → stop and report, don't skip the next step

Why

The `commit` and `pr` skills cover each half, but folding them into one invocation with the verification gate between is the common need and also the common place people forget to run tests before opening the PR.

Test plan

  • `cargo fmt --all` clean
  • `cargo check` passes
  • `cargo test -p agent-code-lib --lib skills` (11 pass)
  • Manual: dirty tree with passing tests → `/commit-push-pr` commits + pushes + opens PR, prints URL
  • Manual: dirty tree with failing test → stops before commit, reports failure
  • Manual: clean tree → stops immediately

Chains commit → push → `gh pr create` with verification gates
between each step. Rules baked in:

- stop if the working tree is clean
- stop if on the base branch (main/master/develop)
- run lint + test gate first — don't commit broken code into a PR
- stage specific files, never `git add -A` (credential leak risk)
- conventional-commit subject under 70 chars, body explains WHY
- no Co-Authored-By: Claude / 🤖 trailers
- push with -u when branch has no upstream
- title under 70 chars, body = Summary + Test Plan
- never force-push, never push to base, never skip hooks
- if any step fails, stop and report — don't compensate by skipping
  the next step
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@emal-avala emal-avala merged commit bb33f56 into main Apr 23, 2026
13 of 14 checks passed
@emal-avala emal-avala deleted the feat/skill-commit-push-pr branch April 23, 2026 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant