Rewrite llms-full.txt as practical LLM reference#98
Conversation
Replace tutorial-heavy 2423-line document with dense 937-line reference organized by task. Every binding type, AOP, contextual bindings, multibindings, and performance optimization includes concrete code examples. Designed for AI code generation.
Reviewer's guide (collapsed on small PRs)Reviewer's GuideRewrites llms-full.txt from a long tutorial-style document into a shorter, practical LLM-focused reference with concrete Ray.Di code examples across features, and makes a small wording/diagram tweak in the factory pattern manual to use English runtime/dependency labels. Sequence diagram for LLM Ray.Di code generation using the rewritten llms_full referencesequenceDiagram
actor Developer
participant LLM
participant LlmsFullReference
participant RayDiCodebase
Developer->>LLM: Request RayDi binding or feature example
LLM->>LlmsFullReference: Retrieve practical reference section
LlmsFullReference-->>LLM: Return focused examples and patterns
LLM->>Developer: Propose RayDi code snippet
Developer->>RayDiCodebase: Integrate and run generated code
RayDiCodebase-->>Developer: Execution result and feedback
Flow diagram for the updated Factory pattern runtime params and DI dependencies illustrationflowchart LR
RuntimeParams[Runtime params]
DIDependencies[DI dependencies]
Factory[Factory]
ObjectWithDeps[Object with dependencies]
RuntimeParams --> Factory
DIDependencies --> Factory
Factory --> ObjectWithDeps
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
📝 WalkthroughWalkthroughA documentation file underwent label translation from Japanese to English in diagram descriptions and formatting adjustments. The changes include translating three diagram labels and adding a trailing newline to the file. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- In
factory-pattern.md, the ASCII diagram labels were switched from Japanese to English while the surrounding explanation remains Japanese; consider keeping the terminology consistent (or bilingual) to avoid confusing readers. - Given that
llms-full.txtis intended as a practical LLM reference, it may help to standardize key terms and patterns (e.g., binding types, AOP, contextual bindings) so they match the exact terminology and phrasing used in the existing manuals to reduce ambiguity for code-generation.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- In `factory-pattern.md`, the ASCII diagram labels were switched from Japanese to English while the surrounding explanation remains Japanese; consider keeping the terminology consistent (or bilingual) to avoid confusing readers.
- Given that `llms-full.txt` is intended as a practical LLM reference, it may help to standardize key terms and patterns (e.g., binding types, AOP, contextual bindings) so they match the exact terminology and phrasing used in the existing manuals to reduce ambiguity for code-generation.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Summary
Test plan