Skip to content

Copier update: claude config#52

Merged
ejfine merged 2 commits intomainfrom
misc-cop
Mar 24, 2026
Merged

Copier update: claude config#52
ejfine merged 2 commits intomainfrom
misc-cop

Conversation

@ejfine
Copy link
Contributor

@ejfine ejfine commented Mar 24, 2026

Pull in upstream template changes

Summary by CodeRabbit

  • New Features

    • Claude Code integration with Beads-backed issue/workflow commands and comprehensive agent guides
    • New assistant command/workflow templates (ADR, issues, TDD, red/green/refactor, spike, polish, summarize, etc.)
    • Devcontainer with an integrated Dolt-backed task database for local task management
    • CI: added unit-test job and stricter coverage reporting
  • Chores

    • Bumped tooling, extensions and GitHub Action versions
    • Added pre-commit hook to merge local Claude settings and updated permission/config defaults
    • Updated ignore rules for Claude/Dolt artifacts

@ejfine ejfine self-assigned this Mar 24, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 3bfa05de-939c-4b5b-8493-4404fbd4ad04

📥 Commits

Reviewing files that changed from the base of the PR and between f84c3c7 and 6ca7638.

📒 Files selected for processing (1)
  • template/src/aws_organization/lib/central_infra_workload.py

📝 Walkthrough

Walkthrough

Adds Claude Code agent frameworks, Beads/Dolt integration, devcontainer and CI updates, many new command/prompt specs and permission settings, helper scripts for merging Claude settings and removing pre-commit hooks, template mirrors, and unit tests for the new tooling.

Changes

Cohort / File(s) Summary
Claude Code commands & settings
/.claude/commands/*.md, /.claude/settings/*, /.claude/.beads/*, /.claude/package.json, /.claude/helpers/merge-claude-settings.sh
New Claude command specs (TDD, red/green/refactor, ADRs, issues, research, etc.), Claude settings/permissions (bash/read/write), Beads metadata/config, package.json (json5), and a script to merge .claude/settings/*.jsonc deterministically.
Devcontainer & Beads/Dolt runtime
.devcontainer/*, .devcontainer/docker-compose.yml, .devcontainer/on-create-command.sh, .devcontainer/post-start-command.sh
Devcontainer feature/version bumps, install steps for Claude/beads tooling, Yarn GPG key handling, docker-compose addition of a beads-dolt service and persistent volume, and post-start logic to run settings merge and recover/initialize .claude/.beads via bd and git.
Pre-commit hook & merge tooling
.pre-commit-config.yaml, src/copier_tasks/remove_precommit_hooks.py, tests/unit/.../test_remove_precommit_hooks.py
Adds a local pre-commit hook merge-claude-settings, a CLI to remove pre-commit hook blocks by regex, and unit tests validating hook removal and error cases.
Templates mirror
template/.claude/..., template/.devcontainer/..., template/.github/actions/ecr-auth/..., template/.pre-commit-config.yaml
Mirrored Claude/Beads configuration into the template, added an ECR OIDC composite action template, and updated template pre-commit hooks to include merge logic.
CI / GitHub Actions
.github/workflows/*.yaml, .github/reusable_workflows/build-docker-image.yaml, .github/actions/*
Bumped action versions (checkout, setup-node, AWS creds), added unit-test job (py3.12.7 & 3.13.9), introduced full-image-tag output and ECR auth composite action usage in build workflow.
Deps, tooling & context
extensions/context.py, .devcontainer/install-ci-tooling.py, pyproject.toml, .copier-answers.yml, ruff.toml
Bumped many tool/version constraints (uv, pnpm, copier, pytest-cov, ty, Node/Vue/GHA versions) and updated context versions used by templates.
Repo config & docs
.gitignore, .coderabbit.yaml, .coveragerc, copier.yml, AGENTS.md, CLAUDE.md
Updated gitignore for Claude/Dolt, added agent guidelines (AGENTS.md), coverage config, copier tasks, and reviewer guidance changes.
Misc template & scaffolding
template/* (lots of files mirrored)
Template-level mirrors of the above files so new projects receive the same Claude/Beads configuration.

Sequence Diagram(s)

mermaid
sequenceDiagram
participant Dev as Devcontainer (post-start)
participant Merge as merge-claude-settings (pre-commit hook)
participant BD as bd CLI
participant Dolt as beads-dolt (Dolt server)
participant Git as Git

Dev->>Merge: run pre-commit merge-claude-settings
Merge-->>Dev: merged settings.json
Dev->>BD: bd ready
alt bd ready succeeds
BD-->>Dev: ready
else bd ready fails
BD-->>Dev: error
Dev->>Git: rm -rf .claude/.beads
Dev->>BD: bd init --server-host="$BEADS_DOLT_SERVER_HOST" --database="$BEADS_DOLT_SERVER_DATABASE" --skip-hooks --stealth
BD->>Dolt: connect to host (beads-dolt)
Dolt-->>BD: init database
Dev->>Git: git -c core.hooksPath=/dev/null restore .claude/.beads
end

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested reviewers

  • zendern
🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description 'Pull in upstream template changes' is vague and generic. It does not follow the provided template structure, missing all required sections (Link to Issue, Why is this change necessary, How does this address the issue, Side effects, Testing approach, Other). The description provides no meaningful detail about what upstream changes are being pulled in or why. Expand the description to follow the template structure. Explain which upstream template is being integrated, why this update is necessary, what specific changes are included (Claude CLI integration, Beads task tracking, CI/workflow updates), what side effects or breaking changes might result, and how the changes have been tested.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Copier update: claude config' is partially related to the changeset. It describes a Copier template update with Claude configuration, which is accurate for the majority of changes (Claude commands, settings, helpers, and Beads integration), but does not capture the full scope including CI/workflow updates, version bumps across multiple tools, and other infrastructure changes.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ejfine ejfine requested a review from zendern March 24, 2026 13:13
@ejfine ejfine marked this pull request as ready for review March 24, 2026 13:13
@ejfine ejfine merged commit 044c24d into main Mar 24, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant