docs(repo): require safe session Git synchronization - #37
antongulin wants to merge 1 commit into
Conversation
🏹 Robin✅ Review done. I flagged 2 things worth a look. Push fixes whenever you like, then comment |
There was a problem hiding this comment.
🏹 Robin
Heads up: this is a point-in-time review. Push fixes freely, then comment
/robinwhenever you want another pass.
🔵 1 Low | 💡 1 Suggestions
Summary
This is a documentation-only change adding git-safety rules for agents to AGENTS.md. The rules are sensible and defensive: they explicitly prohibit destructive operations (reset, discard, auto-stash, force-push), require preserving unfinished work, and are correctly placed above the canonical DOX block so they won't be overwritten by DOX sync. The main issue is mild ambiguity in 'resolve the blocker,' which an agent could interpret as license for destructive resolution; the dense single-paragraph format also makes the discrete rules harder for an agent to follow precisely.
Robin — the Robin Hood of code review. Free for every PR.
Adds the required session-start Git synchronization and preservation rule to the root agent instructions. Existing local work and stashes remain preserved.
Validation: git diff --check passed; only the root agent instructions changed. Local main matches origin/main after fast-forward synchronization. No application, runtime maintenance, or deployment commands were run.