Skip to content

Comments

Add Claude Code configuration (pilot/experiment)#91

Draft
em3s wants to merge 33 commits intomainfrom
exp/claude-code-setup
Draft

Add Claude Code configuration (pilot/experiment)#91
em3s wants to merge 33 commits intomainfrom
exp/claude-code-setup

Conversation

@em3s
Copy link
Contributor

@em3s em3s commented Jan 21, 2026

Summary

Pilot/Experiment: Add Claude Code configuration for Actionbase, fine-tuned from the everything-claude-code recipe.

Discussion #90

Status

This is an experimental configuration on exp/claude-code-setup branch. Not for immediate merge - gathering feedback first.

How to Use

Apply Configuration

git fetch origin
git restore --source=origin/exp/claude-code-setup -- .claude/ CLAUDE.md

Files are ignored by .gitignore and will not be committed. Run often to get latest updates.

Available Commands

Command Description
/plan Create implementation plan for a feature
/build Run Gradle or Go build
/build-fix Fix build errors automatically
/test Run test suite
/test-coverage Run tests with coverage report
/tdd Test-driven development workflow
/e2e Run end-to-end tests
/code-review Review code for quality and security
/review Quick code review
/refactor-clean Clean up dead code
/update-docs Update documentation
/update-codemaps Update codebase maps
/oss-triage <issue#> Triage GitHub issue
/oss-pr-summary <pr#> Summarize PR changes
/oss-pr-review <pr#> Review PR as maintainer

Return to Development Branch

If you are developing this configuration:

git restore -- .claude/ CLAUDE.md
git checkout exp/claude-code-setup

Or keep local changes:

git stash push -m "local claude config" -- .claude/ CLAUDE.md
git checkout exp/claude-code-setup
git stash drop

Feedback

Please share feedback in Discussion #90:

  • What works well?
  • What is missing or incorrect?
  • Suggestions for improvement?

Practice Log

Issues completed using this configuration:

Add project-specific Claude Code setup based on everything-claude-code recipe,
fine-tuned for Actionbase's Kotlin/Java + Go tech stack.

Includes:
- CLAUDE.md with project overview and critical rules
- 9 agents (planner, architect, code-reviewer, etc.)
- 10 commands (/plan, /build, /tdd, /review, etc.)
- 8 rules (security, testing, coding-style, etc.)
- 4 skills (backend-patterns, cli-patterns, etc.)
- Shared hooks in settings.json
- Permission template (local changes allowed, remote changes require approval)

References #90

Co-Authored-By: Claude <noreply@anthropic.com>
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. enhancement New feature or request labels Jan 21, 2026
@em3s em3s self-assigned this Jan 21, 2026
em3s added a commit that referenced this pull request Jan 21, 2026
Currently ignored to allow testing from experiment branches (#91).
To test: git checkout exp/claude-code-setup -- .claude/ CLAUDE.md

Note: .claude/CLAUDE.md and .claude/README.md were previously allowed,
now the entire .claude/ directory is ignored.

Co-Authored-By: Claude <noreply@anthropic.com>
em3s and others added 24 commits January 21, 2026 12:45
Remote changes always require explicit approval per session,
even if user accidentally allows all.

Denied: git push, gh pr/issue create/merge/close, gh release create, docker push

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
- Storage/Metastore/Messaging abstraction
- Architecture: Backend/DX/Documentation structure
- Mono/Flux (not coroutines), spotlessApply
- Testing: Given/When/Then + ParameterizedTest
- Conventional commits with scope: type(scope): description
- Remove deployment-specific config

Co-Authored-By: Claude <noreply@anthropic.com>
- security.md: generalize HBase to Storage
- testing.md: add Kotest(legacy)/JUnit 5(new), reference CLAUDE.md

Co-Authored-By: Claude <noreply@anthropic.com>
- New code should be written in Kotlin
- Existing Java maintained as-is or migrated when appropriate
- Java exceptions: performance-critical code, HBase/Hadoop interop

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add Kotlin patterns: data class, sealed class, enum with abstract method
- Add Java patterns: immutable class, enum with abstract methods, fluent interface
- Add Go patterns: generics, struct embedding, command pattern, table-driven tests
- Reference CLAUDE.md for Backend Language policy

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- CLAUDE.md: REWRITE (complete)
- Rules: security, testing, coding-style (complete)
- Rules: agents, hooks (KEEP - no changes needed)
- Rules: git-workflow, performance, patterns (pending EDIT)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Remove duplicate commit format (now in CLAUDE.md)
- Remove duplicate PR template (now in CLAUDE.md)
- Keep branch naming, workflow, rules

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- HBase -> Storage abstraction
- Kafka -> Messaging abstraction
- Reference CLAUDE.md for reactive patterns

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Remove duplicate code patterns (now in CLAUDE.md, coding-style.md)
- HBase/Kafka -> Storage/Messaging abstraction
- Keep architecture patterns (CQRS, Repository, Service Layer)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Updated 7 agents:
- planner.md
- architect.md
- code-reviewer.md
- security-reviewer.md
- e2e-runner.md
- refactor-cleaner.md
- doc-updater.md

HBase/Kafka -> Storage/Messaging abstraction

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Updated 5 commands:
- plan.md
- tdd.md
- code-review.md
- refactor-clean.md
- e2e.md

HBase/Kafka -> Storage/Messaging abstraction

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- backend-patterns.md: Replace HBase/Kafka with Storage/Messaging
- actionbase-concepts.md: Update architecture diagram and glossary

Skills review complete (4/4). Recipe conversion 32/32 done.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Row key design is finalized. Updated Skills to reference
the authoritative encoding documentation instead of
duplicating design details.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Recipe conversion complete (32/32 files reviewed).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- /test-coverage: JaCoCo + Go coverage analysis
- /update-codemaps: Architecture documentation updates

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- .claude/codemaps/architecture.md - overall architecture
- .claude/codemaps/core.md - core module
- .claude/codemaps/engine.md - engine module
- .claude/codemaps/server.md - server module
- .claude/codemaps/cli.md - Go CLI
- .claude/codemaps/data.md - data model

Updated /update-codemaps command to reference new location.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@em3s em3s changed the title chore: add Claude Code configuration chore: add Claude Code configuration (pilot/experiment) Jan 21, 2026
em3s and others added 6 commits January 21, 2026 14:52
Commands:
- /oss-triage - Issue classification and labeling
- /oss-pr-summary - PR change summaries

Agents:
- oss-maintainer.md - GitHub issue/PR management

Skills:
- oss-maintenance.md - Issue templates, labels, releases

Rules:
- oss-community.md - Tone, response times, CoC

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Updated oss-triage.md and oss-maintenance.md to use
actual labels from the repository:
- enhancement (not feature)
- module:xxx (colon, not slash)
- Removed priority/* labels (not in use)
- Added size:* labels (automated)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Maintainer-focused PR review that:
- Checks PR process and guidelines
- Calls /code-review for code quality
- Provides approve/request-changes recommendation

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- backend-patterns.md: 337 -> 100 lines
- cli-patterns.md: 571 -> 117 lines
- e2e-runner.md: 412 -> 97 lines
- coding-standards.md: 399 -> 62 lines

Reference CLAUDE.md and rules for detailed patterns instead of
duplicating content across files.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add negation patterns (!CLAUDE.md, !.claude/) to enable git add
without -f flag. This preserves main branch compatibility since
later rules override earlier ones in .gitignore.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Keep local settings untracked even with !.claude/ negation.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@em3s em3s marked this pull request as draft January 21, 2026 11:22
- Add git worktree section to git-workflow.md for parallel agent work
- Add README.md explaining how to contribute changes to exp branch

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@em3s em3s changed the title chore: add Claude Code configuration (pilot/experiment) Add Claude Code configuration (pilot/experiment) Feb 12, 2026
…idelines

- Merge agents/claude/.claude (SSOT) into exp/claude-code-setup
- Add rules/refactoring.md: one chain one read, no 3-jump splits,
  minimal diff, derived from #195 retrospective
- Add rules/coding-standards.md, security-rules.md, testing-guide.md
- Add codemap skills (architecture, cli, core, data, engine, server)
- Migrate skills to subdirectory convention (skill-name/SKILL.md)
- Update README with structure overview and key rules table

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant