-
Notifications
You must be signed in to change notification settings - Fork 57
Ignore claude files: CLAUDE.md, .claude/
#639
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
We should not exclude these AI agent files. People should share and mentain the same ones. |
|
I'm not sure. I prefer vibe coding with cursor instead of claude code agent, so I haven't seen these files before. |
5000user5000
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a bit undecided, but I'd like to share my experience with this.
I primarily use claude.md as a form of "development memory" for the AI—tracking my progress through different phases and restoring context if the session disconnects. Since this content is dynamic and personal to my workflow, I agree it fits best in .gitignore.
However, I do believe we should commit a dedicated shared document for AI contexts (perhaps under a different name). This acts as a guardrail for new contributors, ensuring the AI is aware of our protocols—such as verifying CI and unit tests before pushing, or avoiding architectural pitfalls. For example, it could prevent mistakes like the one I made previously, where I implemented an overly long function directly inside a class declaration.
Suggestion: We should add claude.md to .gitignore for personal use, but establish a separate, committed file (e.g., docs/AI_GUIDELINES.md) to standardize these rules and reduce error rates.
@tigercosmos Makes sense. Would you please share what you use in a PR? |
@ThreeMonth03 Would you have time to share what you use with cursor in a PR? |
@5000user5000 This also makes sense, but contradicts to what @tigercosmos said. @tigercosmos, I'd like to know what you think? |
No, I will fix the ongoing pull request and share the usage of cursor after the conference. Recently, I also take a lot of time to practice English Speaking with Toko, in order to attend the meeting by in English. |
I have a suggestion. Maybe I could find a quite simple issue like pr #608 , then I fix the issue, record the process and upload to youtube. The length of the video is about half to two hours. |
I didn't use agent file for modmesh. Copilot works well with the default setup. However I recommend to have a common Asking AI to read |
@tigercosmos Having a comment set of files for hinting AI assistants will help align contributors. On the other hand, hard-coding to fixed file names would make it cumbersome when tryting new things, for it is easy to accidentally check in experimental changes to the hinting files. Maybe the hinting files should go to a sub-directory, and contributors copy them manually to use.
@ThreeMonth03 sounds like a plan. |
I am new to vibe coding. @tigercosmos @ThreeMonth03 @5000user5000 should I/we do this?