fix: fix: setup_ops_repo should seed sprints/ directory (#402) #405

Merged

dev-bot merged 1 commit from fix/issue-402 into main

2026-04-08 07:34:02 +00:00

dev-bot commented

2026-04-08 07:26:28 +00:00

Collaborator

Fixes #402

Changes

Fixes #402 ## Changes

dev-bot added 1 commit 2026-04-08 07:26:29 +00:00

fix: fix: setup_ops_repo should seed sprints/ directory (#402 )

ci/woodpecker/push/ci Pipeline was successful

Details

ci/woodpecker/pr/ci Pipeline was successful

Details

6e350c0838

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

review-bot commented

2026-04-08 07:32:39 +00:00

Collaborator

AI Review

Summary\n\nAdds `sprints/` directory seeding to `setup_ops_repo` so it is created and committed on first setup (or upgrade). The change is correct and well-integrated with the existing seeding pattern.\n\n### Review\n\n- The `mkdir -p` call is correctly placed alongside the other directory creations (line 161).\n- The `[ -f .gitkeep ] || { touch .gitkeep; seeded=true; }` idiom is idempotent: re-running setup on an existing ops repo that already has the directory will skip both the touch and the commit.\n- Setting `seeded=true` here ensures the sprints directory is committed and pushed even for existing repos that don't need a new README.md, which is the right behavior for an upgrade path.\n- The README.md template update (line 179) matches the new directory.\n\nNo issues found.

Verdict

APPROVE — Correct, minimal fix: adds sprints/ directory creation and .gitkeep seeding with proper idempotency check.

Reviewed at 6e350c0 | AGENTS.md

## AI Review  ### Summary\n\nAdds `sprints/` directory seeding to `setup_ops_repo` so it is created and committed on first setup (or upgrade). The change is correct and well-integrated with the existing seeding pattern.\n\n### Review\n\n- The `mkdir -p` call is correctly placed alongside the other directory creations (line 161).\n- The `[ -f .gitkeep ] || { touch .gitkeep; seeded=true; }` idiom is idempotent: re-running setup on an existing ops repo that already has the directory will skip both the touch and the commit.\n- Setting `seeded=true` here ensures the sprints directory is committed and pushed even for existing repos that don't need a new README.md, which is the right behavior for an upgrade path.\n- The README.md template update (line 179) matches the new directory.\n\nNo issues found. ### Verdict **APPROVE** — Correct, minimal fix: adds sprints/ directory creation and .gitkeep seeding with proper idempotency check. --- *Reviewed at `6e350c0` | [AGENTS.md](AGENTS.md)*

review-bot approved these changes 2026-04-08 07:32:40 +00:00

review-bot left a comment

Collaborator

AI Review: APPROVE — Correct, minimal fix: adds sprints/ directory creation and .gitkeep seeding with proper idempotency check.

AI Review: **APPROVE** — Correct, minimal fix: adds sprints/ directory creation and .gitkeep seeding with proper idempotency check.