chore: sprint skill + self-review gate in the dev loop #48

Merged

forgejo_admin merged 2 commits from chore/sprint-skill into main

2026-07-01 13:23:41 +00:00

forgejo_admin commented

2026-07-01 13:19:47 +00:00

Owner

Makes "start sprint N" a deterministic trigger and adds the self-review gate you asked for.

Changes

CLAUDE.md dev loop — after opening the PR: wait for CI green → self-review the diff (/code-review / superpowers:requesting-code-review; fix blocking findings, re-confirm green if I push fixes) → only then say it's ready. Never claim ready before CI-green + self-review; still never self-merge.
.claude/skills/sprint/ — a thin project skill. In any session, "start sprint N" / "groom sprint N" / "continue the sprint" / /sprint runs: groom (just-in-time, plan + current state + prior retro) → per-issue dev loop (worktree → TDD → PR → CI green → self-review → stop for your merge) → retro. It sequences the existing rules and enforces the guardrails (plan is source of truth, groom-first, human merges); composes with the superpowers skills rather than replacing them.
docs/sprints/README.md — the CI-green + self-review gate folded into the per-sprint Definition of Done.

Why

CLAUDE.md already loads every session, so "start sprint 0" is mostly understood today — the skill makes it reliable across context compaction and is the durable home for the self-review step so it can't get dropped.

No feature code. After merge, "start sprint 0" (or /sprint 0) is the entry point.

🤖 Generated with Claude Code

Makes "start sprint N" a deterministic trigger and adds the self-review gate you asked for. ## Changes - **`CLAUDE.md` dev loop** — after opening the PR: **wait for CI green → self-review the diff** (`/code-review` / superpowers:requesting-code-review; fix blocking findings, re-confirm green if I push fixes) → **only then say it's ready**. Never claim ready before CI-green + self-review; still never self-merge. - **`.claude/skills/sprint/`** — a thin project skill. In any session, "start sprint N" / "groom sprint N" / "continue the sprint" / `/sprint` runs: **groom** (just-in-time, plan + current state + prior retro) → **per-issue dev loop** (worktree → TDD → PR → CI green → self-review → stop for your merge) → **retro**. It sequences the existing rules and enforces the guardrails (plan is source of truth, groom-first, human merges); composes with the superpowers skills rather than replacing them. - **`docs/sprints/README.md`** — the CI-green + self-review gate folded into the per-sprint Definition of Done. ## Why `CLAUDE.md` already loads every session, so "start sprint 0" is *mostly* understood today — the skill makes it reliable across context compaction and is the durable home for the self-review step so it can't get dropped. No feature code. After merge, "start sprint 0" (or `/sprint 0`) is the entry point. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

forgejo_admin added 1 commit 2026-07-01 13:19:48 +00:00

chore: add sprint skill + self-review gate to the dev loop 5500c111d9

- CLAUDE.md dev loop: after opening the PR, wait for CI green, then
  self-review the diff (/code-review) before saying it's ready; never
  claim ready before CI-green + self-review; still never self-merge.
- .claude/skills/sprint: thin per-project skill so "start sprint N" /
  "groom sprint N" / "continue the sprint" deterministically runs the
  groom -> per-issue dev-loop -> retro flow, enforcing the guardrails
  (plan is source of truth, groom-first, CI-green + self-review before
  ready, human merges). Composes with the superpowers skills.
- docs/sprints/README.md: fold the CI-green + self-review gate into the
  per-sprint Definition of Done.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

forgejo_admin added 1 commit 2026-07-01 13:21:43 +00:00

chore: add mid-sprint resume procedure + live groom-doc status cd9c710026

- sprint skill: explicit "Resume — session closed mid-sprint" section
  that reconstructs the frontier from groom doc -> Forgejo -> git, and a
  rule to keep per-issue Status live (todo -> in progress -> PR open ->
  merged) and commit/push per TDD step so state is never archaeology.
- docs/sprints/README.md: add a Status column to the groom template.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

forgejo_admin merged commit 8adc70b727 into main

2026-07-01 13:23:41 +00:00

forgejo_admin deleted branch chore/sprint-skill

2026-07-01 13:23:41 +00:00

forgejo_admin referenced this pull request from a commit

2026-07-01 13:23:42 +00:00

Merge pull request 'chore: sprint skill + self-review gate in the dev loop' (#48) from chore/sprint-skill into main

No reviewers

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: forgejo_admin/digger#48

No description provided.

Rows
Columns