Skip to content

Update CLAUDE.md: restore formatting and document missing classes#7

Merged
AbaSheger merged 1 commit into
mainfrom
feature/demo-spel-flow
May 3, 2026
Merged

Update CLAUDE.md: restore formatting and document missing classes#7
AbaSheger merged 1 commit into
mainfrom
feature/demo-spel-flow

Conversation

@AbaSheger
Copy link
Copy Markdown
Owner

@AbaSheger AbaSheger commented May 3, 2026

Restores original markdown structure (headers, bullets, code fences) and adds SpelExpressionResolver, GuardrailController, GuardrailAction, GuardrailDecision, and UsageRecord to the key classes section. Also clarifies onViolation type and ConditionalOnProperty havingValue attribute.

Summary

Type of change

  • Bug fix
  • New feature
  • Refactor / internal improvement
  • Documentation
  • Build / CI

Test plan

  • mvn clean verify passes locally
  • New behaviour covered by tests
  • Existing tests still pass

Checklist

  • No unrelated files in the diff
  • PR is focused on a single concern
  • Limitations or follow-up work noted in the description if applicable

Summary by CodeRabbit

  • Documentation
    • Updated system architecture documentation to reflect current component interactions and expanded descriptions of key system components for improved clarity.

Restores original markdown structure (headers, bullets, code fences) and
adds SpelExpressionResolver, GuardrailController, GuardrailAction,
GuardrailDecision, and UsageRecord to the key classes section. Also
clarifies onViolation type and ConditionalOnProperty havingValue attribute.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@AbaSheger AbaSheger merged commit 1551dea into main May 3, 2026
2 of 3 checks passed
@AbaSheger AbaSheger deleted the feature/demo-spel-flow branch May 3, 2026 20:24
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 3, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 72d5a383-85c7-4d8f-b25f-f7925a4e41e7

📥 Commits

Reviewing files that changed from the base of the PR and between deea1f0 and 84d5a49.

📒 Files selected for processing (1)
  • CLAUDE.md

📝 Walkthrough

Walkthrough

The CLAUDE.md documentation file was updated to insert SpelExpressionResolver into the architecture data-flow diagram between GuardrailInterceptor and CostEstimator, and the "Key Classes" section was expanded with detailed descriptions of all major components and their models.

Changes

Architecture & Component Documentation

Layer / File(s) Summary
Architecture Diagram Update
CLAUDE.md
SpelExpressionResolver is now shown in the data-flow diagram resolving dynamic annotation fields after GuardrailInterceptor and before CostEstimator.
Key Classes Expansion
CLAUDE.md
The "Key Classes" section replaced a short bullet list with detailed descriptions of @LLMGuarded, GuardrailInterceptor, GuardrailDecisionEngine, CostEstimator, UsageStore/InMemoryUsageStore, GuardrailController, Guardrail4jAutoConfiguration, Guardrail4jProperties, SpelExpressionResolver, and models (GuardrailAction, GuardrailDecision, UsageRecord) with their responsibilities and notable fields.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes


🐰 A diagram grows more complete,
Components dance in rhythm neat,
SpelExpressions find their place,
Documentation shows the race!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/demo-spel-flow

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
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

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