chore: AGENTS.md

Author: fbosch

AGENTS.md

@@ -14,6 +14,18 @@ pnpm.
 - Test: `pnpm test`
 - Typecheck: `pnpm typecheck`
 
+## Git workflow
+
+**IMPORTANT**: AI agents should NEVER commit or push changes without explicit user permission.
+
+- **DO NOT** run `git commit` or `git push` automatically
+- **DO NOT** create commits as part of completing a task
+- **ALWAYS** ask the user before committing or pushing
+- **ONLY** commit when the user explicitly requests it (e.g., "commit these changes", "push this to git")
+- After making changes, inform the user what was changed and let them decide when to commit
+
+The user maintains full control over git operations and commit history.
+
 ## Testing expectations
 
 - Add or update tests for behavior changes and bug fixes.
@@ -49,18 +61,3 @@ pnpm.
 - Files use kebab-case (e.g. `parse-args.ts`).
 - Types/interfaces use PascalCase; functions/variables use camelCase.
 - Use `index.ts` barrels for public entrypoints.
-
-## Review guidelines
-
-When implementing PR feedback or making changes for review:
-
-- **Run all checks before pushing**: `pnpm build && pnpm test && pnpm typecheck && pnpm lint`
-- **Test cross-platform compatibility**: Windows CI often catches issues Linux doesn't
-- **Keep commits focused**: One logical change per commit with clear messages
-- **Avoid premature extraction**: Only extract shared code when you have 2+ actual uses
-- **Document edge cases**: Add comments explaining non-obvious behavior or workarounds
-- **Test git operations carefully**: Use `DOCS_CACHE_GIT_COMMAND` env var to override git path in tests
-- **Preserve existing test patterns**: Follow the style and structure of existing test files
-- **Check for regex gotchas**: Use non-greedy `.*?` instead of greedy `.*` when appropriate
-- **Validate input limits**: Add safety checks (like `MAX_BRACE_EXPANSIONS`) for user-controlled expansion
-- **Update documentation**: Ensure README reflects new features and limitations