Skip to content

docs: add AGENTS.md and CLAUDE.md contributor guides#743

Open
FBumann wants to merge 1 commit into
mainfrom
worktree-docs+add-agent-guides
Open

docs: add AGENTS.md and CLAUDE.md contributor guides#743
FBumann wants to merge 1 commit into
mainfrom
worktree-docs+add-agent-guides

Conversation

@FBumann

@FBumann FBumann commented Jul 23, 2026

Copy link
Copy Markdown
Member

Adds AGENTS.md and CLAUDE.md to the repo so AI agents (and humans) pick up the project's contribution rules and conventions automatically.

Adapted from the linopy project's guides, with linopy-specific details stripped and replaced by flixopt conventions:

  • Dev workflow → pip install -e ".[full,dev]" + pre-commit + pytest/ruff
  • Branch off main, tests under tests/ as test_*.py, shared conftest.py fixtures
  • Architecture paragraph rewritten for flixopt's component-based, xarray-backed design

CLAUDE.md simply imports AGENTS.md via @AGENTS.md. The AI-contribution rules (mark generated content, human writes their own intent, conversations aren't held through an agent) are carried over as-is.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Added contribution guidelines covering development workflows, coding conventions, testing, formatting, linting, branching, and project architecture.
    • Added guidance for AI-assisted contributions, including content attribution and human-authored discussions.
    • Added a quick-reference document directing contributors to the complete guidelines.

Adapted from the linopy project's guides, with linopy-specific details
replaced by flixopt conventions (main branch, pip/pre-commit workflow,
component-based xarray architecture).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Adds repository documentation for AI-assisted contribution rules, development workflows, project conventions, architecture, and guidance for AI agents through AGENTS.md and CLAUDE.md.

Changes

Contribution guidance

Layer / File(s) Summary
Contribution and agent guidance
AGENTS.md, CLAUDE.md
Defines AI-assisted contribution rules, development and testing workflows, branching conventions, flixopt architecture, and a reference from CLAUDE.md to AGENTS.md.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the change, but it does not follow the required template sections or include testing, issues, or checklist items. Add the template headings for Description, Type of Change, Related Issues, Testing, and Checklist, and fill in the missing items.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding AGENTS.md and CLAUDE.md contributor guides.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch worktree-docs+add-agent-guides

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@AGENTS.md`:
- Around line 26-30: In the handwritten explanation near the
AI-generated-content note, update the typo “Thats” to “That’s” while leaving the
surrounding Markdown and wording unchanged.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: cb909ee5-1e68-4dc0-b5d7-ac6f856ec57a

📥 Commits

Reviewing files that changed from the base of the PR and between 6962e5e and 1fd479a.

📒 Files selected for processing (2)
  • AGENTS.md
  • CLAUDE.md

Comment thread AGENTS.md
Comment on lines +26 to +30
```markdown
Thats why this is important for me (handwritten)

> [!NOTE]
> The following content was generated by AI.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Fix the typo in the example.

Change “Thats” to “That’s” so contributors do not copy incorrect prose.

Proposed fix
-   Thats why this is important for me (handwritten)
+   That's why this is important for me (handwritten)
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
```markdown
Thats why this is important for me (handwritten)
> [!NOTE]
> The following content was generated by AI.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@AGENTS.md` around lines 26 - 30, In the handwritten explanation near the
AI-generated-content note, update the typo “Thats” to “That’s” while leaving the
surrounding Markdown and wording unchanged.

@FBumann
FBumann enabled auto-merge (squash) July 23, 2026 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant