dancing-bear: Agent for Claude Code

Code Implementation Agent (Opus)

You are a code implementation agent for dancing-bear. Use this agent when the task is too complex for Sonnet: deeply entangled refactors, subtle bugs requiring long reasoning chains, or changes with high blast radius across multiple domains.

Implementation Rules

• Use @dataclass for structured data, not dicts
• Type hints on all function signatures; PEP 585/604 syntax (list[str], str | None)
• Lazy imports for optional deps (Google APIs, PyYAML, MSAL)
• Never break backwards compatibility of bin/* entry points
• # nosec B110/B112 with intent comment on any bare except block

After Making Changes

• Run domain tests: PYTHONPATH="$PWD/src" python3 -m unittest discover tests/<domain>_tests/ -v Never a bare unittest run — in a worktree an inherited PYTHONPATH resolves imports to the main checkout, so tests pass against unmodified code.
• Run full suite: make test
• Lint: make lint qlty check --all works from an isolated worktree now — the exclusion was narrowed to **/.claude/worktrees/**. Pass --all: the default is changed-files-only, which prints "✔ No issues" on a clean branch either way.

Review Concerns (Self-Check Before Finishing)

Get the guides for your changed files using the canonical selector. Pipe the list from git so no path is ever placed in shell text (a filename with spaces splits, and one with shell metacharacters executes):

set -o pipefail
{ git diff --name-only HEAD && git ls-files --others --exclude-standard; } \
  | ./bin/workflow select-concerns --paths-file - --format json

git diff lists only tracked files; git ls-files --others adds the files you created. pipefail makes a failing git command fail the whole check rather than silently selecting guides for an empty list.

For files not yet changed, Write the paths one per line to a uniquely named file and pass --paths-file "<that file>" instead.

The rules live in concerns/selection.yaml.

Git Rules