Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add CLAUDE.md for Claude Code assistance with Mondo Ontology #8868

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add CLAUDE.md for Claude Code assistance with Mondo Ontology #8868

wants to merge 2 commits into from

Conversation

dragon-ai-agent
Copy link
Collaborator

This PR adds a CLAUDE.md file to provide instructions and guidelines for Claude Code when working with the Mondo Ontology. The file includes:

  - Project layout information
  - How to query the ontology effectively
  - Best practices for making edits
  - OBO format guidelines
  - GitHub contribution process
  - Common build commands

  This will help Claude Code assist users with automated tasks, term edits, and issue resolution in a way that follows project conventions.

  🤖 Generated with [Claude Code](https://claude.ai/code)

  Co-Authored-By: Claude <[email protected]>

@twhetzel twhetzel self-assigned this Mar 24, 2025
twhetzel
twhetzel reviewed Mar 24, 2025
View reviewed changes
CLAUDE.md
- For terms that are compositional, check `src/patterns/dosdp-patterns/*yaml`

## Edits
- Do not edit the edit file directly, it's large
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not edit the edit file directly, it's large --> should the second "edit" be mondo-edit.obo?

twhetzel
twhetzel reviewed Mar 24, 2025
View reviewed changes
CLAUDE.md

## Project Layout
- Main development file is `src/ontology/mondo-edit.obo`
- This file is LARGE, never Search or grep this directly EXCEPT using the tools below
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW, when I tried Claude code on a PR (#8843) without this information here in this file, the greps Claude did seemed fine.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, interesting, I haven't done any kind of systematic eval...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@twhetzel twhetzel twhetzel left review comments

@sabrinatoro sabrinatoro Awaiting requested review from sabrinatoro sabrinatoro is a code owner

@nicolevasilevsky nicolevasilevsky Awaiting requested review from nicolevasilevsky nicolevasilevsky is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@twhetzel twhetzel

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dragon-ai-agent @twhetzel