Skip to content

Add Java support and simplify hook delegation#42

Merged
aseembits93 merged 4 commits intomainfrom
add-java-support-simplify-hook
Apr 3, 2026
Merged

Add Java support and simplify hook delegation#42
aseembits93 merged 4 commits intomainfrom
add-java-support-simplify-hook

Conversation

@aseembits93
Copy link
Copy Markdown
Contributor

Summary

  • Add Java language support (.java file detection, dedicated hook message, pom.xml/build.gradle.kts config lookup)
  • Simplify stop hook to delegate all setup/install logic to the codeflash:optimize skill instead of handling it inline
  • Remove oauth-login.sh (auth now handled by codeflash CLI directly)
  • Add new optimize skill flags: --all, --effort, --no-pr
  • Make debug logging conditional on CODEFLASH_DEBUG=1
  • Fix typos in hook messages ("to to" → "to")

Test plan

  • Verify Java file commits trigger the Java-specific hook message
  • Verify Java dedup prevents double triggers
  • Verify Python and JS/TS paths still work as expected
  • Verify debug logging only activates with CODEFLASH_DEBUG=1
  • Run bats tests/ to confirm all tests pass

🤖 Generated with Claude Code

aseembits93 and others added 4 commits April 3, 2026 11:25
- Add Java file detection (.java) in suggest-optimize.sh with dedicated message
- Remove oauth-login.sh (auth now handled by codeflash CLI directly)
- Simplify hook: delegate all install/config logic to codeflash:optimize skill
- Add --all, --effort, --no-pr flags to optimize skill
- Add Java project config support (pom.xml, build.gradle.kts)
- Make debug logging conditional on CODEFLASH_DEBUG=1
- Fix "to to" typos in hook messages
- Update tests: simplify assertions to match skill delegation, add Java tests
- Update README directory structure

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@aseembits93 aseembits93 merged commit a069184 into main Apr 3, 2026
2 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