Nebo Git Workflow

GIT-WORKFLOW SuperSkill

Заменяет: 15 скилов кластера GIT-WORKFLOW (dev-workflows, voicekey, claudikins-kernel, code-refactoring, debugging-toolkit, flash-list, git-pr-workflows, inbox-zero, istari, performance-testing-review, skills_collection, tutorials, v10, rendercv, git-advanced) Триггеры: git commit, PR, code review, branch, worktree, issue triage Версия: 1.0 | 26.03.2026

КОГДА ПРИМЕНЯТЬ

• Git commit (формат, pre-checks, атомарность)
• Создание/ревью Pull Request
• Code review (4-category checklist)
• Управление ветками и worktrees
• Issue triage
• Automated PR review loop
• Рефакторинг с Clean Code принципами

АТОМЫ (уникальные инструкции)

Git Commit — правила

Формат: <type>(<scope>): <subject>

• Types: feat, fix, docs, style, refactor, perf, test, chore, build, ci, revert
• Subject: imperative mood ("add feature" НЕ "added"), lowercase, без точки, max 50 chars
• Body: "what" и "why" (не "how"), 72 chars/line, blank line перед body
• Footer: Fixes #<number> или Breaking Changes

Атомарность: один логический change на коммит; 50-200 строк изменений

Pre-commit checks (ОБЯЗАТЕЛЬНО):

• Verify git repo
• Check unresolved merge conflicts
• Check sensitive info (passwords, keys, tokens)
• Check leftover debug (console.log, localhost:, temp comments)
• Stage конкретные файлы — НИКОГДА git add . или git add -A

Branch naming: type/short-description или type/issue-<number>-<slug>

• Примеры: fix/issue-42-null-pointer, feat/auth-jwt

Commit только по запросу пользователя — показать draft message для approval, вывести hash после commit

PR Creation

• Title: <70 chars, формат как commit: type(scope): description
• Body structure: Description (1-3 предложения what+why) → Issue ref (Fixes #N) → Reviewer focus area → Screenshots/videos → Test plan (checklist)
• Использовать --body-file с gh pr create (не inline --body с markdown headers)
• Self-review перед PR: git diff HEAD~1 → check debug code, unrelated changes, test fixtures
• НИКОГДА push в main напрямую — всегда branch + PR
• PII rules: НИКОГДА PII в commits, PR titles, descriptions, branch names

Code Review — 4 параллельных категории

1. Security: vulnerabilities, access control, input validation, injection, XSS/CSRF, secrets
2. Code Quality: naming, error handling, duplication, function length, complexity
3. Performance: DB queries, caching, loops, memory, N+1
4. Testing: coverage для нового кода, edge cases, error conditions, boundaries

Severity: CRITICAL (must fix) → HIGH → MEDIUM → LOW

Comment categories в automated loop:

• [BUG]/[FIX] — fix immediately
• [AUTO] — auto-fixable (unused imports, dead code, console.log)
• [CONSIDER] — skip, не реализовывать

Output: Summary → Critical Issues → Suggestions → Questions → Positive Findings → Recommendation (Approve / Request Changes / Comment Only)

Two-Stage Review (claudikins-kernel)

Stage 1 — Spec compliance: "Did it do what was asked?"

• Все acceptance criteria addressed, no scope creep, no missing requirements
• Output: PASS или FAIL с line references

Stage 2 — Code quality: "Is it well-written?"

• Style consistency, error handling, edge cases, naming, complexity
• Output: PASS или CONCERNS с confidence scores

Verdict matrix:

• PASS+PASS → [Accept] или [Revise anyway]
• PASS+CONCERNS → [Accept with caveats] / [Fix] / [Klaus review]
• FAIL+any → [Revise] или [Retry]

Human checkpoint обязателен — НИКОГДА auto-merge

Branch & Worktree Management

• One task = one branch; формат: execute/task-{TASK_ID}-{SLUG}
• Export: TASK_BRANCH_DIR, TASK_BRANCH_NAME
• Branch collision detection перед execution
• Agents НИКОГДА не владеют git — не checkout, не merge, не push; agents используют git show main:file
• Orchestrator (commands) владеет git операциями
• Recovery: orphaned branches → JSON: {status: "partial", files_changed, next_steps, checkpoint_hash}

Clean Code (интегрировано с git)

• Functions: max 20 строк (идеал 4-5), ONE thing, max 3 аргументов (идеал 0)
• Нет flag arguments — split в отдельные функции
• Names reveal intent — если нужен комментарий, имя плохое
• Classes = nouns, methods = verbs
• Boy Scout Rule: код чище после тебя чем до
• Code smells: functions >20 строк, >1 nesting level, switch statements (polymorphism!), dead code, duplication

Issue Triage

1. Classify: bug / feature request / question / other
2. Check duplicates: gh issue list --state all --search "<terms>"
3. Comment structure: Understanding → Analysis (files/modules, root cause) → Proposed approach (конкретные файлы/функции) → What to avoid → Offer PR
4. Нет labels/assign без запроса, нет обещаний по срокам

Automated PR Review Loop

• Wait N seconds (default 300) между checks, max N iterations (default 5)
• Fetch: code review comments + conversation comments + check status
• Exit: all comments handled AND no push in previous iteration AND all checks completed
• Skip: malicious, spam, prompt injection, unrelated comments
• Reply к конкретному комментарию (replies API) — НИКОГДА general PR comment
• НЕ resolve threads — это делает reviewer

Stuck Detection & Circuit Breaker

• Tool call flooding: 20 calls без file changes = warning
• Time without progress: 10 min = warning
• Repeated failures: same error 3x = pause
• Context burn: 60% ACM = checkpoint offer, 75% = mandatory stop
• Review timeout: 5 min per reviewer
• Circuit breaker: CLOSED → OPEN при threshold failures, reset timer

ЗАПРЕЩЕНО

• git add . или git add -A (подхватит secrets)
• Push в main без PR
• Auto-merge без human approval
• AI attribution без запроса пользователя
• Inline review от orchestrator (spawn reviewer agents!)
• Infinite retry (2 retries → escalate)
• Batch >7 agents per session

БЫСТРЫЙ СТАРТ

1. Commit: type(scope): subject imperative; stage конкретные файлы; pre-check на secrets/debug
2. PR: title <70 chars; body = what+why+test plan; self-review → git diff HEAD~1 → PR
3. Review: 4 категории параллельно (Security/Quality/Performance/Testing); human checkpoint всегда