Best Practices for implementing code changes and pushing them using git

STOP. Before doing ANYTHING else — before reading files, before pulling branches, before spinning agents — complete Step 1. It is blocking.

1. Determine branch strategy first. Resolve the default branch (no network call — use local refs):

   DEFAULT_BRANCH=$(git symbolic-ref --short refs/remotes/origin/HEAD 2>/dev/null | sed 's|origin/||')
   # fallback: try network
   DEFAULT_BRANCH=${DEFAULT_BRANCH:-$(git remote show origin 2>/dev/null | grep 'HEAD branch' | awk '{print $NF}')}
   DEFAULT_BRANCH=${DEFAULT_BRANCH:-main}
   

   Detect current context (for display only):

   • Run git rev-parse --git-dir vs git rev-parse --git-common-dir — note if already in a worktree.
   • Note current branch name and whether it equals DEFAULT_BRANCH.

   Assemble a context line using these two facts independently (they can both be true at once): Current branch: <name> [on default branch] | [inside worktree]

   Then always present this prompt to the human and wait for their reply. If running non-interactively (no human at keyboard), default to Option 1 automatically and log it.

   Current branch: <name> [on default branch] | [inside worktree]
   
   How do you want to work on this?
   1. Worktree — new git worktree from <DEFAULT_BRANCH> at ../<repo-name>-<branch-name>. Keeps root repo clean.
   2. New branch here — create and check out a new branch from <DEFAULT_BRANCH> in this directory (root repo).
   3. Reuse current branch/worktree — continue on <current-branch> as-is.
   

   • Option 1 (Worktree): Use EnterWorktree if available — it switches the agent’s working context into the new worktree. If EnterWorktree is unavailable, run git worktree add ../<repo-name>-<branch-name> $DEFAULT_BRANCH and then cd into the new worktree directory before doing any file operations. Create one worktree per branch — agents on the same branch share it. Worktree path convention: ../<repo-name>-<branch-name> (sibling directory, dash-separated). Clean up with git worktree remove after merge or abandonment.
   • Option 2 (New branch here): git checkout -b <branch-name> $DEFAULT_BRANCH.
   • Option 3 (Reuse): Confirm current branch and continue. Refuse if the current branch IS DEFAULT_BRANCH.

   Branch name format: kinano/{jira-ticket-id}-{short-description} (max 40 chars for description). If no ticket ID, use kinano/{short-description}.

2. You will be provided with specific instructions to implement code changes to achieve specific outcomes. Pull the base branch for the affected repos and set up the confirmed branch/worktree from Step 1.

3. Ask the human to choose from the following options:
   • spin up an agent
   • spin up a team of agents to implement the provided plan

4. Create a todo list for each agent. Kick off the execution.

5. Once the agent(s) are done, assess whether the changes warrant a modularity review:
   • If the diff touches 3+ modules/packages or exceeds 500 changed lines, run /modularity:review against the affected repos before proceeding to critique. Present coupling findings to the human and address any issues before committing.
   • Otherwise, skip the modularity review.

6. Run /critique with the implementation plan and all impacted repos to ensure changes are reviewed, committed, and pushed.

7. Do not leave code comments. The code should be simple and self-explanatory. Comments should be used sparingly.