Skip to content

Commit a03cecc

Browse files
authored
chore: Add instructions for handling merge conflicts (#1941)
Should help agents be a bit more careful with merge conflict resolution.
1 parent 134f1dc commit a03cecc

1 file changed

Lines changed: 21 additions & 0 deletions

File tree

AGENTS.md

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,27 @@ skip hooks. If the pre-commit hook fails (e.g. due to husky not being set up in
152152
a worktree), run `npx lint-staged` manually before committing to ensure lint and
153153
formatting checks pass. Fix any issues before creating the commit.
154154

155+
## Merge Conflict Resolution
156+
157+
1. **Never blindly pick a side.** Read both sides of every conflict to
158+
understand the intent of each change before choosing a resolution.
159+
160+
2. **Refactor/move conflicts require extra verification.** When one side
161+
refactored, moved, or extracted code (e.g., inline components to separate
162+
files), always diff the discarded side against the destination files before
163+
declaring the conflict resolved. Code can diverge after extraction — the
164+
other branch may have made fixes or additions that the extracting branch
165+
never picked up. A naive "keep ours" resolution silently drops those changes.
166+
167+
3. **Verify the result compiles.** After resolving, check for missing imports,
168+
broken references, or type errors introduced by the resolution — especially
169+
when discarding a side that added new dependencies or exports.
170+
171+
4. **Ask for help when uncertain.** If you are not 100% confident about which
172+
side to keep, or whether a change can be safely discarded, stop and ask for
173+
manual intervention rather than guessing. A wrong guess silently breaks
174+
things; asking is always cheaper than debugging later.
175+
155176
---
156177

157178
_Need more details? Check the `agent_docs/` directory or ask which documentation

0 commit comments

Comments
 (0)