Skip to content

Add vendor-neutral assistant guide#501

Merged
dsarno merged 3 commits into
hi-godot:mainfrom
Clubhouse1661:add-vendor-neutral-assistant-guide
May 31, 2026
Merged

Add vendor-neutral assistant guide#501
dsarno merged 3 commits into
hi-godot:mainfrom
Clubhouse1661:add-vendor-neutral-assistant-guide

Conversation

@Clubhouse1661

Copy link
Copy Markdown
Contributor

Fixes #486

Summary

Adds a vendor-neutral assistant guide for repository operating context.

  • Adds AGENTS.md as the shared source of truth for AI assistant guidance.
  • Reduces CLAUDE.md to Claude-specific loading notes and a pointer to AGENTS.md.
  • Reduces .claude/skills/godot-ai/skill.md to a thin Claude skill adapter.
  • Points README and CONTRIBUTING assistant guidance at AGENTS.md.

Why

Godot AI supports multiple MCP clients and assistant workflows, but the repo guidance lived mostly in Claude-specific files. That made useful project context easier for Claude Code to consume than for other agents.

This keeps the same general guidance available, but moves it into a vendor-neutral location.

Notes

This is mostly a consolidation, not a new workflow. Some duplicated content from CLAUDE.md and the Claude skill was merged into one shared guide. The full hand-maintained tool inventory was not copied into AGENTS.md; it points to the canonical inventory sources instead (docs/TOOLS.md, tool_catalog.gd, and tests/unit/test_tool_domains.py) to avoid creating another drift-prone table.

Tests

  • Docs-only change.
  • Verified git diff --check origin/main...HEAD passes.

@codecov

codecov Bot commented May 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copilot AI 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.

Pull request overview

Adds a vendor-neutral AGENTS.md assistant guide and converts Claude-specific guidance files into thin adapters pointing to the shared source of truth.

Changes:

  • Adds consolidated repository guidance in AGENTS.md.
  • Replaces most CLAUDE.md and Claude skill content with pointers to AGENTS.md.
  • Updates README and CONTRIBUTING docs to direct AI assistants to the shared guide.

Reviewed changes

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

Show a summary per file
File Description
AGENTS.md Adds the shared assistant operating guide.
CLAUDE.md Reduces Claude-specific repo guidance to loading notes and an AGENTS.md pointer.
.claude/skills/godot-ai/skill.md Converts the Claude skill into a thin adapter.
README.md Adds an assistant-facing pointer to AGENTS.md.
docs/CONTRIBUTING.md Adds guidance for assistants to read AGENTS.md.

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

Comment thread AGENTS.md Outdated
Comment thread AGENTS.md
Clubhouse1661 and others added 2 commits May 30, 2026 20:12
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@dsarno dsarno merged commit e6f7571 into hi-godot:main May 31, 2026
15 checks passed
@dsarno

dsarno commented May 31, 2026

Copy link
Copy Markdown
Contributor

Thank you @Clubhouse1661 !

@Clubhouse1661 Clubhouse1661 deleted the add-vendor-neutral-assistant-guide branch May 31, 2026 01:58
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.

Add vendor-neutral assistant guide and generated tool inventory

3 participants