# 0xlayerghost Solidity Agent Kit Git Workflow

> Git Collaboration Standards

- Skill: `tomevault-io/0xlayerghost-solidity-agent-kit-git-workflow` (Agent Skill, multi-file: 2 files)
- Install (CLI): `npx skillmds@latest add tomevault-io/0xlayerghost-solidity-agent-kit-git-workflow`
- Raw SKILL.md: https://api.skillmd.com/api/skills/tomevault-io/0xlayerghost-solidity-agent-kit-git-workflow/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: AI & ML
- Author: tomevault-io (https://skillmd.com/u/tomevault-io)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/tomevault-io/0xlayerghost-solidity-agent-kit-git-workflow

---


# Git Collaboration Standards

## Language Rule

- **Always respond in the same language the user is using.** If the user asks in Chinese, respond in Chinese. If in English, respond in English.

## Commit Rules

Use Conventional Commits format: `<type>: <short description>`

| Type | When to use |
|------|------------|
| `feat:` | New feature or contract |
| `fix:` | Bug fix |
| `refactor:` | Code restructure without behavior change |
| `test:` | Add or update tests |
| `docs:` | Documentation changes |
| `chore:` | Build config, dependency updates, toolchain |
| `security:` | Security fix or hardening |

### Commit Workflow

1. Run `git diff` to review all changes before staging
2. Stage specific files — avoid `git add .` to prevent committing `.env` or artifacts
3. Write concise commit message describing the **why**, not the **what**
4. **Never add `Co-Authored-By` lines** — commit messages should only contain the description
5. **Only commit** — never `git push` unless explicitly requested
6. **Never push directly to main/master** — always use feature branches

## Branch Naming

| Pattern | Example |
|---------|---------|
| `feat/<name>` | `feat/staking-pool` |
| `fix/<name>` | `fix/reentrancy-guard` |
| `refactor/<name>` | `refactor/token-structure` |

## PR Requirements

Every PR must include:

| Section | Content |
|---------|---------|
| Change description | What was changed and why |
| Test results | `forge test` output (all pass) |
| Gas impact | `forge test --gas-report` diff for changed functions |
| Deployment impact | Does this affect deployed contracts? Migration needed? |
| Review focus | Specific areas that need careful review |

## Code Review Rules

| Scenario | Requirement |
|----------|------------|
| Standard changes | Minimum 1 maintainer approval |
| Security-related changes | Minimum 2 maintainer approvals |
| AI-generated code | Must pass manual review + `forge test` before merge |
| Contract upgrades | Requires full team review + upgrade simulation on fork |

## AI Assistance Rules

- AI-generated code must pass `forge test` before committing
- Always review AI output for: correct import paths, proper access control, gas implications
- Include relevant file paths and test cases in AI prompts for better results
- Run `forge fmt` after AI generates code to ensure consistent formatting

---
> Converted and distributed by [TomeVault](https://tomevault.io/claim/0xlayerghost) — claim your Tome and manage your conversions.
<!-- tomevault:4.0:skill_md:2026-04-11 -->