Skip to content

docs(repo): require safe session Git synchronization - #37

Open
antongulin wants to merge 1 commit into
mainfrom
chore/session-git-sync-rule
Open

antongulin wants to merge 1 commit into
mainfrom
chore/session-git-sync-rule

Conversation

@antongulin

Copy link
Copy Markdown
Owner

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.

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

🏹 Robin

✅ Review done. I flagged 2 things worth a look.

Push fixes whenever you like, then comment /robin for another pass.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🏹 Robin

Heads up: this is a point-in-time review. Push fixes freely, then comment /robin whenever 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.

Comment thread AGENTS.md
Comment thread AGENTS.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant