Install
$ agentstack add skill-njoyimpeguy-augments-finishing-a-branch ✓ scanned · ✓ verified, works with Claude Code, Cursor, and more.
Security review
✓ PassedNo issues found. Passed automated security review. · v0.1.0 How review works →
- ✓ Prompt-injection patterns
- ✓ Secret / credential exfiltration
- ✓ Dangerous shell & filesystem operations
- ✓ Untrusted network calls
- ✓ Known-malicious package signatures
What it can access
- ✓ Network access No
- ✓ Filesystem access No
- ✓ Shell / process execution No
- ✓ Environment & secrets No
- ✓ Dynamic code execution No
From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.
Verified badge
Passed review? Show it. Paste this badge into your README, it links to the public security report.
Reliability & compatibility
Declared compatibility
Compatibility is declared by the source manifest. End-to-end runtime verification is coming, see below.
We're building live execution health for every listing: tool-call success rate, median latency, uptime, and last-checked timestamps, measured, not self-reported. It isn't live yet, so we don't show numbers we can't stand behind.
How agent discovery & health will work →About
Finishing a Branch
Take a working branch to a merge-ready state. The order is fixed — gate, tidy, describe, decide, clean up — and nothing destructive happens before the thing it depends on is confirmed.
When to use
- A change is complete and its checks have been run and pass (not merely claimed), and you're ready to merge, open a PR, or set the branch aside.
- Skip mid-development — this is the wrap-up, not a checkpoint.
Procedure
- Gate on green. Run the tests and checks fresh; read the output. If anything fails, STOP and fix it before any step below. A merge-ready branch has passing checks, not "should be passing".
- Tidy the history. One logical commit per coherent change. Imperative subject ≤ ~72 chars; the body says why, not what (the diff is the what). Squash fixup noise. Do not force-push a branch with shared history unless explicitly asked.
- Write the PR description — two sections, no more:
## Summary— 2–3 bullets: what changed and why.## Test Plan— how it was verified.
A reader should grasp the change without opening the diff.
- Decide, explicitly — surface the choice once; don't open-end it:
- Merge it — only after green checks and review (
requesting-code-review; its self-review path covers a trivial mechanical diff). If it ships to a running system,release-readinessis the next gate before deploy. - Open a PR and leave the branch for review.
- Keep it as-is, or discard it.
- Clean up only what you created, and only after the merge is confirmed. Remove a workspace/worktree before deleting its branch, never the reverse (see
using-task-branches). Never delete a branch or workspace you didn't create, and never discard without explicit confirmation.
Common mistakes
- Presenting merge options while checks are red — the gate comes first.
- A force-push that rewrites shared history nobody asked you to rewrite.
- A PR body that restates the diff instead of explaining the why and the test plan.
- Deleting a branch before the worktree that references it (the delete fails), or cleaning up a workspace the harness owns.
Source & license
This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.
- Author: NjoyimPeguy
- Source: NjoyimPeguy/augments
- License: MIT
Install and usage instructions live in the source repository linked above.
Reviews
No reviews yet, be the first.
Write a review
Versions
- v0.1.0 Imported from the upstream source.