Skip to content

Conversation

@benceruleanlu
Copy link
Member

Summary

  • move root CLAUDE.md to AGENTS.md
  • add redirect stubs in CLAUDE.md paths
  • update integration docs to AGENTS.md with redirect

Copilot AI review requested due to automatic review settings December 18, 2025 19:44
@benceruleanlu benceruleanlu requested a review from a team as a code owner December 18, 2025 19:44
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Dec 18, 2025
@coderabbitai
Copy link

coderabbitai bot commented Dec 18, 2025

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Review skipped — only excluded labels are configured. (1)
  • backport

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

📝 Walkthrough

Walkthrough

The pull request introduces a new AGENTS.md documentation structure at both the project root and tests/integration directory. It provides comprehensive development and testing guidance, while existing CLAUDE.md files are redirected to reference the new AGENTS.md documentation.

Changes

Cohort / File(s) Change Summary
Root-level documentation restructuring
AGENTS.md, CLAUDE.md
Added new AGENTS.md with comprehensive project overview, technologies, development commands, testing guidance, structure, setup, and conventions. Updated CLAUDE.md to reference AGENTS.md.
Integration tests documentation
tests/integration/AGENTS.md, tests/integration/CLAUDE.md
Added new integration testing guide in AGENTS.md. Updated CLAUDE.md to reference tests/integration/AGENTS.md instead of README.md.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/agents-md

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 and usage tips.

@dosubot dosubot bot added the documentation Improvements or additions to documentation label Dec 18, 2025
@benceruleanlu benceruleanlu merged commit 070224f into main Dec 18, 2025
19 of 21 checks passed
@benceruleanlu benceruleanlu deleted the chore/agents-md branch December 18, 2025 19:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR renames CLAUDE.md files to AGENTS.md to make the documentation more agent-agnostic, while maintaining backward compatibility through redirect stubs in the old CLAUDE.md locations.

Key changes:

  • Move root CLAUDE.md content to new AGENTS.md file
  • Move tests/integration/CLAUDE.md content to tests/integration/AGENTS.md
  • Replace old CLAUDE.md files with redirect stubs using @ syntax

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
CLAUDE.md Replaced with redirect stub pointing to AGENTS.md
AGENTS.md New file containing the original CLAUDE.md content (agent instructions)
tests/integration/CLAUDE.md Replaced with redirect stub pointing to AGENTS.md
tests/integration/AGENTS.md New file containing the original integration testing guide content

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -0,0 +1,152 @@
# ComfyUI Desktop - Claude Code Instructions
Copy link

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

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

The title still references "Claude Code Instructions" but the file has been renamed from CLAUDE.md to AGENTS.md to be more agent-agnostic. Consider updating the title to be more generic, such as "ComfyUI Desktop - Agent Instructions" or "ComfyUI Desktop - Development Instructions for AI Agents".

Suggested change
# ComfyUI Desktop - Claude Code Instructions
# ComfyUI Desktop - Agent Instructions

Copilot uses AI. Check for mistakes.
# Integration testing guide

@README.md
@tests/integration/AGENTS.md
Copy link

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

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

The redirect path appears to be absolute when it should be relative. Since this CLAUDE.md file is already in the tests/integration/ directory, the redirect should be @AGENTS.md (without the directory prefix) to match the pattern used in the root CLAUDE.md file which redirects to @AGENTS.md in the same directory.

Suggested change
@tests/integration/AGENTS.md
@AGENTS.md

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants