git-review

SKILL.md preview

---
name: git-review
description: >-
  Make the current workspace healthy: read-dependencies-discover → install-dependencies → configuration →
  evaluate (format, lint, tests; prefer fixing failing code and unformatted sources) → write-quality-documentation (§8a).
  No git add, commit, or push. Command matrix in write-quality-evaluate + read-config-configuration.
---

# `@git-review` (workspace health → report)

Normative fences / full matrix: [`read-dependencies-discover`](../../internal/read/dependencies/discover/SKILL.md), [`write-quality-evaluate`](../../internal/write/quality/evaluate/SKILL.md).


**Public.** **Goal:** the tree **as it is now** works — dependencies installed, **sources formatted**, **lint clean**, **tests passing**, then **§8a** **light** polish of **existing** **`docs/**`** + minimal root **`README.md`** (parity with repo + CI—**not** bulk regen). **Not** staging, **commit**, or **publish**—use **`@git-commit`** and **`@git-push`** for savepoints and publish; **`@gh-pr`** runs this skill before **`@git-push`** when opening or updating a PR.

**Route here when:** the user wants **verify / format / lint / test** (or “make it green”). **Not for:** branch cleanup (**`@git-branch`**), sync/merge (**`@git-pull`**), publish (**`@git-push`**), issue/PR text (**`@gh-*`**), history rewrite (**`@git-rebase`** / **`@git-reset`** / **`@git-revert`** / **`@git-cherry-pick`**).

Run each **internal** skill below **in full**, in order. Details live in linked files—**not** duplicated here.

| Step | Internal skill |
| --- | --- |
| 1–2 | [`read-dependencies-discover`](../../internal/read/dependencies/discover/SKILL.md) |
| 3 | [`write-dependencies-install`](../../internal/write/dependencies/install/SKILL.md) |
| — | [`read-config-configuration`](../../internal/read/config/configuration/SKILL.md) |
| 4–7 | [`write-quality-evaluate`](../../internal/write/quality/evaluate/SKILL.md) |
| 8a | [`write-quality-documentation`](../../internal/write/quality/documentation/SKILL.md) |
| 9 | Report (this skill only) |

```mermaid
flowchart LR
  classDef intH fill:#004d40,color:#fff
  classDef int1 fill:#00695c,color:#fff
  classDef int2 fill:#00897b,color:#fff
  classDef int3 fill:#4db6ac,color:#111
  classDef intT fill:#b2dfdb,color:#111
  D[discover-deps] --> I[install-deps] --> C[configuration] --> E[evaluate]
  E --> DOC[documentation 8a]
  class D intT
  class I int3
  class C int2
  class E intH
  class DOC int1
```

**Failure:** Stop on first failure; **skip step 8a** if Evaluate failed.

## Language interaction policy

Always apply [`read-safety-language-interaction-rules`](../../internal/read/safety/language-interaction-rules/SKILL.md) first. Use English by default for all assistant output, including AskQuestion prompts/options, unless the user explicitly requests another full-language response.

## Before batch

**Always run first** (sequential). Complete every applicable row in this table before execution steps in this skill. Skip a row only when its **When** column does not apply.

| Kind | Skill / library | When |
| --- | --- | --- |
| **Recommended** | [`@git-pull`](../pull/SKILL.md) | Branch may be behind upstream or canonical main |
| **Recommended** | [`@git-docs`](../docs/SKILL.md) | Broader post-green doc pass after this skill succeeds |
| **Tooling** | `git` | Repo root; stack tools per discovery |




## Required internal skills

Run after **Before batch** completes (names only; normative fences in linked libraries).

| Library | Role |
| --- | --- |
| `read-dependencies-discover` | See linked library |
| `write-dependencies-install` | See linked library |
| `read-config-configuration` | See linked library |
| `write-quality-evaluate` | See linked library |
| `write-quality-documentation` | See linked library |
| `read-workflow-git` | See linked library |
| `write-workflow-git` | See linked library |
| `read-repo-github` | See linked library |
| `internal-write` | See linked library |
| `read-safety-skill-safety`

…