docs: Add CLAUDE.md and UNITY_CONTEXT.md for AI development guidance#11
docs: Add CLAUDE.md and UNITY_CONTEXT.md for AI development guidance#11
Conversation
- Add comprehensive CLAUDE.md with payment gateway integration context - Add UNITY_CONTEXT.md with educational domain business context - Include EaseBuzz payment processing for educational fee collection - Provide Unity-specific development patterns for payment gateways - Reference educational payment workflows and fee management 🤖 Generated with Claude Code (claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
WalkthroughTwo new documentation files, Changes
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
UNITY_CONTEXT.md (3)
22-24: Use a heading instead of bold text for the lifecycle flow.Bold emphasis (
**Lead Generation → … Alumni**) is triggering MD036 (“no-emphasis-as-heading”).
Convert this to a proper Markdown heading for consistency with the rest of the document.-**Lead Generation → Student Applicant → Student → Program Enrollment → Alumni** +#### Lead Generation → Student Applicant → Student → Program Enrollment → Alumni
160-162: Add the missing article “a”.LanguageTool flags the phrase “using common admission process”.
Insert the article “a” for grammatical correctness.-**Centralized Admissions**: Multiple schools using common admission process +**Centralized Admissions**: Multiple schools using a common admission process
255-259: Replace hyphens with en dashes in numeric ranges.Typographical nit: ranges such as “500-5000” and “50-500” should use an en dash (–) rather than a hyphen (-).
-**Student Population**: 500-5000 students per entity -**Staff Size**: 50-500 employees depending on institution size +**Student Population**: 500–5000 students per entity +**Staff Size**: 50–500 employees depending on institution sizeCLAUDE.md (1)
60-60: Clarify wording of document-event description.“Processes settlement logs on save” reads awkwardly. Consider re-phrasing for clarity.
-**Easebuzz Settlement Log**: Processes settlement logs on save via `process_log` function +**Easebuzz Settlement Log**: Processes settlement logs when the document is saved via the `process_log` function
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
CLAUDE.md(1 hunks)UNITY_CONTEXT.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
UNITY_CONTEXT.md
[uncategorized] ~161-~161: You might be missing the article “a” here.
Context: ...ed Admissions**: Multiple schools using common admission process - Staff Mobility:...
(AI_EN_LECTOR_MISSING_DETERMINER_A)
[uncategorized] ~192-~192: Did you mean “regulation”? If following ‘per’, nouns are often singular.
Context: ...**: Educational record retention as per regulations - Anonymization: Research and analy...
(CONFUSION_OF_NNS_NN_UN)
[typographical] ~256-~256: If specifying a range, consider using an en dash instead of a hyphen.
Context: ...itution Scale - Student Population: 500-5000 students per entity - Staff Size: 5...
(HYPHEN_TO_EN)
[typographical] ~257-~257: If specifying a range, consider using an en dash instead of a hyphen.
Context: ...0 students per entity - Staff Size: 50-500 employees depending on institution size...
(HYPHEN_TO_EN)
CLAUDE.md
[uncategorized] ~60-~60: The preposition ‘to’ seems more likely in this position.
Context: ...lement Log**: Processes settlement logs on save via process_log function ## Dev...
(AI_HYDRA_LEO_REPLACE_ON_TO)
🪛 markdownlint-cli2 (0.17.2)
UNITY_CONTEXT.md
22-22: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
🔇 Additional comments (2)
UNITY_CONTEXT.md (1)
1-266: Overall document is comprehensive and well-structured.The added business-context documentation significantly improves onboarding for new contributors working on education-domain features. Nice work.
CLAUDE.md (1)
1-137: Documentation looks solid.The architecture, commands, and security guidance are clear and actionable.
1 participant
AI Development Documentation for Payment Gateway Integration
This PR adds documentation files to support AI-assisted development in Unity ERP's payment gateway module:
Files Added
Key Features
Benefits
🤖 Generated with Claude Code Documentation System
Summary by CodeRabbit