docs: update AGENTS.md and README.md to match current architecture#22
docs: update AGENTS.md and README.md to match current architecture#22starknetdev merged 1 commit intonextfrom
Conversation
- Fix build/test commands: replace non-existent `packages/test_starknet` with correct `snforge test -p <package>` syntax - Fix CI matrix fuzzer runs: token=32, minigame=64 (not 50/256) - Add callback system to architecture diagrams and lifecycle docs - Add missing packages to README: leaderboard, registry, tokenomics, presets, interfaces, testing infrastructure - Add deploy_stream_token_factory.sh, remove empty create_objectives.sh from scripts section - Add Registry to MinigameToken architecture (multi-game mode) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the
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 |
|
Claude finished @starknetdev's task —— View job PR Review: Documentation UpdatesTodo List:
Review SummaryThis is a well-structured documentation update that addresses several important issues. The changes are accurate and improve the overall quality of the documentation. ✅ Positive Changes1. Corrected Build/Test Commands (AGENTS.md:32-34, README.md:74-78)
2. Accurate CI Matrix Information (AGENTS.md:107-116)
3. Enhanced Architecture Documentation
4. Comprehensive Package Documentation (README.md:147-197)
5. Updated Deployment Scripts Section (README.md:255-270)
🔍 Technical Accuracy VerificationCI Configuration Consistency
Script References
📝 Minor Suggestions
🎯 Overall Assessment
Recommendation: APPROVE ✅This PR significantly improves the documentation quality and removes several pain points for developers trying to use the documented commands. All technical details have been verified for accuracy. |
Summary of ChangesHello @starknetdev, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request focuses on comprehensively updating the project's documentation, specifically Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request provides a comprehensive update to the project's documentation, primarily in AGENTS.md and README.md. The changes accurately reflect the current architecture, including the new callback system, updated package structure, and revised build/test commands. The documentation is now more complete and aligned with the codebase. I have one suggestion to improve the consistency of the package descriptions in the README.md file.
| - **`testing`**: Shared test constants and addresses | ||
| - **`test_common`**: Shared mock contracts and example implementations |
There was a problem hiding this comment.
For consistency with other package descriptions in this section (e.g., Utils), consider adding the package paths to the testing and test_common list items. This would make it clearer where to find these packages in the repository structure.
| - **`testing`**: Shared test constants and addresses | |
| - **`test_common`**: Shared mock contracts and example implementations | |
| - **`testing`** (`packages/testing/`): Shared test constants and addresses | |
| - **`test_common`** (`packages/test_common/`): Shared mock contracts and example implementations |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary
packages/test_starknetpath with correctsnforge test -p <package>syntaxIMetagameCallback) to architecture diagrams and game lifecycle docsdeploy_stream_token_factory.sh, remove emptycreate_objectives.shTest plan
🤖 Generated with Claude Code