Skip to content

feat(claude): add agents and skills for recurring repo work - #708

Merged
ValgulNecron merged 3 commits into
masterfrom
claude/project-thread-lkccu2
Oct 16, 2026
Merged

ValgulNecron merged 3 commits into
masterfrom
claude/project-thread-lkccu2

Conversation

@ValgulNecron

Copy link
Copy Markdown
Collaborator

Before: .claude/ had 3 agents (ci-triager, crd-engineer, spec-auditor) and 5 skills. Nothing covered Pencil design, building React from a design, driving a PR to ready, reviewing a PR, Dependabot fixes, module authoring, spec status, spec archiving, merging master in, submodule bumps or local-agent handoff prompts. ship-branch and crd-engineer told agents to add session links to commits and PR bodies.

After: 6 new agents and 6 new skills (slash commands) cover those jobs. The session-link lines are gone.

Agents (.claude/agents/)

  • pencil-designer: designs in design.pen or website.pen through the Pencil MCP only (local machine), using pen-canvas-layout, pen-componentize and design-export, and waits for maintainer sign-off before any React work.
  • web-builder: builds dashboard React from an approved design export and stops when there is none.
  • pr-steward: drives a PR to ready (merge master in, CI root cause, every CodeRabbit item fixed or settled) and never merges, approves or rebases.
  • pr-reviewer: read-only review against CLAUDE.md core rules and the whole spec folder.
  • dependabot-fixer: fixes real breakages after checking the blocked-upgrades list, with no overrides or peer bypasses.
  • module-author: adds or edits game modules in GameplanePanel/module, then bumps the pointer.

Skills: /spec-status, /spec-archive, /merge-master, /submodule-bump, /local-prompt, /deps-check.

How: these are Markdown files only. .claude is gitignored, so the new files were added with git add -f, the same way the existing ones were. The /spec-status one-liner was run once against origin/master to check its output.

🤖 Generated with Claude Code


Generated by Claude Code

Adds pencil-designer, web-builder, pr-steward, pr-reviewer,
dependabot-fixer and module-author subagents under .claude/agents/.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Signed-off-by: Claude <noreply@anthropic.com>
Adds /spec-status, /spec-archive, /merge-master, /submodule-bump,
/local-prompt and /deps-check.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Signed-off-by: Claude <noreply@anthropic.com>
Commit and PR templates no longer add Claude-Session trailers or
claude.ai session URLs, matching the maintainer's rule.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Signed-off-by: Claude <noreply@anthropic.com>
@ValgulNecron ValgulNecron added area: shared netguard, gameaction, gameproto, svcutil and other shared packages type: chore Maintenance, deps, or housekeeping labels Oct 16, 2026 — with Claude
@ValgulNecron ValgulNecron self-assigned this Oct 16, 2026
@coderabbitai

coderabbitai Bot commented Oct 16, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 9 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: GameplanePanel/Gameplane/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: cd2db072-8117-4891-aa0e-94afd99b6e5c
📥 Commits

Reviewing files that changed from the base of the PR and between 63308ed and acd2c57.

📒 Files selected for processing: 14
  • .claude/agents/crd-engineer.md
  • .claude/agents/dependabot-fixer.md
  • .claude/agents/module-author.md
  • .claude/agents/pencil-designer.md
  • .claude/agents/pr-reviewer.md
  • .claude/agents/pr-steward.md
  • .claude/agents/web-builder.md
  • .claude/skills/deps-check/SKILL.md
  • .claude/skills/local-prompt/SKILL.md
  • .claude/skills/merge-master/SKILL.md
  • .claude/skills/ship-branch/SKILL.md
  • .claude/skills/spec-archive/SKILL.md
  • .claude/skills/spec-status/SKILL.md
  • .claude/skills/submodule-bump/SKILL.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@ValgulNecron
ValgulNecron merged commit cde3202 into master Oct 16, 2026
1 check passed
@ValgulNecron
ValgulNecron deleted the claude/project-thread-lkccu2 branch October 16, 2026 17:32
ValgulNecron pushed a commit that referenced this pull request Oct 16, 2026
Brings in #701 (chart CRDs moved to the gameplane-crds subchart), #619,
#704, #706, #707, #708 and #709. No conflicts; hack/sync-chart-crds.sh
leaves the subchart CRD copies unchanged.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Signed-off-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: shared netguard, gameaction, gameproto, svcutil and other shared packages type: chore Maintenance, deps, or housekeeping

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants