Skip to content

[Docs] Add AGENTS.md - #634

Merged
chr-hertel merged 3 commits into
mainfrom
agents-md
Nov 2, 2026
Merged

chr-hertel merged 3 commits into
mainfrom
agents-md

Conversation

@chr-hertel

Copy link
Copy Markdown
Member

Add an AGENTS.md with commands, architecture overview and coding conventions for AI coding agents.

🤖 Generated with Claude Code

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.

🟡 Changes recommended

AGENTS.md points OAuth work to the wrong namespace and incorrectly claims that supported modern-only result fields are not modeled.

2 open issues
What changed in this PR

Adds repository guidance for AI coding agents working on the PHP MCP SDK. This documentation-only change covers development workflows, architecture, and contribution conventions.

Changes:

  • Documents setup, testing, analysis, and documentation commands.
  • Maps server, client, and shared protocol components across both protocol eras.
  • Defines coding, specification-reference, and review communication guidelines.
File Description
AGENTS.md Adds commands, architecture guidance, and contribution conventions for AI coding agents.

Review effort: Balanced

Comment thread AGENTS.md Resolved
Comment thread AGENTS.md Resolved
chr-hertel and others added 2 commits November 2, 2026 23:03
Clarify the role of codecs in adapting results to wire shapes and handling modern fields.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Clarified OAuth resource-server-only sections in documentation.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@chr-hertel
chr-hertel merged commit 21c47c8 into main Nov 2, 2026
30 checks passed
@chr-hertel
chr-hertel deleted the agents-md branch November 2, 2026 22: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.

2 participants