- Notifications
You must be signed in to change notification settings - Fork11
Comments
Draft
Conversation
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#90Co-Authored-By: Claude <noreply@anthropic.com>
em3s added a commit that referenced this pull requestJan 21, 2026
Currently ignored to allow testing from experiment branches (#91).To test: git checkout exp/claude-code-setup -- .claude/ CLAUDE.mdNote: .claude/CLAUDE.md and .claude/README.md were previously allowed,now the entire .claude/ directory is ignored.Co-Authored-By: Claude <noreply@anthropic.com>
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 pushCo-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 configCo-Authored-By: Claude <noreply@anthropic.com>
- security.md: generalize HBase to Storage- testing.md: add Kotest(legacy)/JUnit 5(new), reference CLAUDE.mdCo-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 interopCo-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 policyCo-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, rulesCo-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- HBase -> Storage abstraction- Kafka -> Messaging abstraction- Reference CLAUDE.md for reactive patternsCo-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.mdHBase/Kafka -> Storage/Messaging abstractionCo-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.mdHBase/Kafka -> Storage/Messaging abstractionCo-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 glossarySkills 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 referencethe authoritative encoding documentation instead ofduplicating 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 updatesCo-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 modelUpdated /update-codemaps command to reference new location.Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Commands:- /oss-triage - Issue classification and labeling- /oss-pr-summary - PR change summariesAgents:- oss-maintainer.md - GitHub issue/PR managementSkills:- oss-maintenance.md - Issue templates, labels, releasesRules:- oss-community.md - Tone, response times, CoCCo-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Updated oss-triage.md and oss-maintenance.md to useactual 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 recommendationCo-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 linesReference CLAUDE.md and rules for detailed patterns instead ofduplicating content across files.Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add negation patterns (!CLAUDE.md, !.claude/) to enable git addwithout -f flag. This preserves main branch compatibility sincelater 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>
3 tasks
- Add git worktree section to git-workflow.md for parallel agent work- Add README.md explaining how to contribute changes to exp branchCo-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…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 tableCo-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading.Please reload this page.
Summary
Pilot/Experiment: Add Claude Code configuration for Actionbase, fine-tuned from theeverything-claude-code recipe.
Discussion#90
Status
This is anexperimental configuration on
exp/claude-code-setupbranch. Not for immediate merge - gathering feedback first.How to Use
Apply Configuration
Files are ignored by
.gitignoreand will not be committed. Run often to get latest updates.Available Commands
/plan/build/build-fix/test/test-coverage/tdd/e2e/code-review/review/refactor-clean/update-docs/update-codemaps/oss-triage <issue#>/oss-pr-summary <pr#>/oss-pr-review <pr#>Return to Development Branch
If you are developing this configuration:
Or keep local changes:
git stash push -m"local claude config" -- .claude/ CLAUDE.mdgit checkout exp/claude-code-setupgit stash dropFeedback
Please share feedback in Discussion#90:
Practice Log
Issues completed using this configuration: