git-safe-workflow
Git Safe Workflow
Core rules (always)
-
Collect repo context first (non-destructive):
- git rev-parse --show-toplevel
- git status --porcelain=v1 -b
- git log -1 --oneline
-
Collect worktree context when relevant (also non-destructive):
- git branch --show-current
- git worktree list --porcelain
Run worktree context when:
- branch name is unexpected
- you are in a nested folder and not sure which checkout you are in
- Git refuses checkout because branch is already checked out elsewhere
- status shows detached HEAD or unusual metadata
-
Never run destructive or high-risk commands unless explicitly requested:
- Do NOT use:
- git reset --hard
- git clean -fd
- git push --force or --force-with-lease
- git worktree prune
- git worktree remove
- git rebase (interactive or not) unless explicitly requested
- If the user requests one:
- restate the exact command you plan to run
- explain why it is risky
- then proceed
- Do NOT use:
-
Avoid interactive prompts and editors unless the user says it is OK:
- Prefer non-interactive commands
- Avoid:
- git add -p
- editor-based rebase
- commit message editor prompts
- Prefer:
- git commit -m "..." -m "..."
Worktree safety rules
-
Confirm you are in the intended worktree and branch before staging or committing:
- git branch --show-current
- git worktree list --porcelain
-
Detached HEAD safety:
- If 'git branch --show-current' prints nothing, you are likely in detached HEAD.
- Default behavior: do not commit immediately.
- Explain the situation and ask whether to create a branch first, for example:
- git switch -c
- Only commit in detached HEAD if the user explicitly wants that and understands the risk.
-
Branch checked out in another worktree:
- If Git refuses because the branch is already checked out elsewhere, do not use force by default.
- Safe options:
- switch that other worktree to a different branch, or
- create a new branch for this worktree (recommended)
-
Worktree lifecycle operations:
- Do not run these unless explicitly requested:
- git worktree add
- git worktree remove
- git worktree move
- git worktree lock or unlock
- git worktree prune
- Do not run these unless explicitly requested:
Make a checkpoint commit (default)
1) Summarize the change
- git diff --stat
- git diff --staged --stat (if anything is staged)
2) Inspect details when needed
- git diff
- git diff --staged
Guidance:
- Always inspect full diff if changes are large, touch security-sensitive code, or involve config or CI.
3) Stage changes safely
Prefer explicit paths when practical:
- git add path/to/file1 path/to/file2
Otherwise stage tracked modifications and deletions:
- git add -u
Avoid staging everything blindly unless user explicitly wants it:
- avoid: git add .
4) Commit message
Use Conventional Commits when reasonable:
- type(scope): summary
Include:
- what behavior changed
- tests run, or explicitly note: tests not run
5) Commit (non-interactive)
- git commit -m "type(scope): summary" -m "Details... Tests: "
6) Verify
- git status
- git show --stat --oneline HEAD
Amend policy (git commit --amend)
Default rule
- Do not use 'git commit --amend' unless it clearly improves the most recent commit AND the commit has not been pushed.
Safe uses
Use amend when:
- You just made the last commit locally and immediately noticed:
- you forgot a file
- you need a tiny fix
- the commit message is wrong
Common safe commands:
- Add changes then amend without changing message:
- git commit --amend --no-edit
- Replace message non-interactively:
- git commit --amend -m "type(scope): summary" -m "Details... Tests: ..."
Avoid
Do not amend when:
- the commit is already pushed to a shared remote branch
- amending would require a force push to reconcile the remote
In that case:
- prefer a new follow-up commit
- only rewrite history if the user explicitly requests it and accepts the risk
Merge conflicts
-
Collect context:
- git status --porcelain=v1 -b
-
Identify conflicted files:
- git diff --name-only --diff-filter=U
-
Resolve conflicts carefully (no automation that discards intent).
- After resolving:
- git add
- After resolving:
-
Continue the operation:
- If merge:
- git commit -m "merge: resolve conflicts" -m "Details... Tests: ..."
- If rebase was explicitly requested and is in progress:
- git rebase --continue
- If merge:
-
Verify:
- git status
- git show --stat --oneline HEAD (if a commit was created)
Push policy
- Only push if the user explicitly asks.
- Preferred push:
- git push -u origin HEAD
Main or master branch rule
- If currently on main or master, do not push directly by default.
- Create a branch first (ask user for branch name if not provided), then push that branch.
Force push rule
- Never force push unless explicitly requested.
- If requested:
- restate the exact force push command
- explain risk (rewriting shared history)
- then proceed
More from regenrek/agent-skills
hard-cut
Enforce a hard-cut cleanup policy: keep one canonical implementation and delete compatibility, migration, fallback, adapter, coercion, and dual-shape code. Use for pre-release or internal-draft refactors where the goal is one final shape, especially when changing schemas, contracts, persisted state, routing, configuration, feature flags, enum/value sets, or architecture.
14root-cause-finder
Performs root-cause-first debugging and review by tracing expected behavior to the first unintended side effect before changing contracts, parsing, or types. Use when debugging protocol errors, deserialization failures, null payloads, missing fields, restore or hydration issues, state-ownership bugs, unexpected requests, background mutations, or reviewing junior-created code where the visible failure may be downstream noise.
7consolidate-test-suites
Decide exactly where bug-fix test coverage belongs. Use before adding, moving, or deleting tests after a bug fix or architectural change. Select one owning layer, reuse existing canonical suites, block redundant or weakly placed tests, and remove weaker duplicates.
7shellck
Run shellcheck on shell scripts after editing scripts or when debugging shell errors. Use for linting scripts in a repo (especially scripts/), catching issues like set -u with unset vars, bad subshell usage, or quoting mistakes.
3codex-analysis
Run Codex CLI for deep code analysis and second-opinion reviews. Use when the user explicitly asks for Codex analysis, Codex help, or wants a second opinion from Codex on code, architecture, or debugging questions.
3pr-commiter
Agentic PR committer with deterministic commits, enforced branch/PR workflow, and explicit paths (no git add .).
3