Skip to content

Stop framing breaking changes as a workflow in AGENTS.md - #3286

Merged
maxisbey merged 1 commit into
mainfrom
docs/agents-api-stability
Aug 16, 2026
Merged

Stop framing breaking changes as a workflow in AGENTS.md#3286
maxisbey merged 1 commit into
mainfrom
docs/agents-api-stability

Conversation

@maxisbey

@maxisbey maxisbey commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Rewords the two places in AGENTS.md that still framed breaking changes as a documented workflow, now that 2.x is the released stable line.

Motivation and Context

The Branching Model bullets ("removing or replacing an API must be intentional… changes that break code written against v1 must be documented in docs/migration.md") and the ## Breaking Changes section were written while v2 was being assembled. Read today they nudge an agent toward "breaking public API is fine as long as it's written up in the migration guide", and treat docs/migration.md as a live document to append to.

This PR states the 2.x compatibility contract in one bullet (changes to existing public API are an explicit maintainer decision and generally avoided), marks docs/migration.md as the finished v1 → v2 record that's closed to new entries, and drops the ## Breaking Changes section, which only existed to describe how to add to that file.

How Has This Been Tested?

pre-commit run --files AGENTS.md (markdownlint) passes. No code changes.

Breaking Changes

None.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

CONTRIBUTING.md still tells human contributors that main takes "New APIs, refactors" with no stability caveat; left for a follow-up so this stays scoped to agent guidance.

The Branching Model bullets and the "Breaking Changes" section were written
while v2 was still being assembled, and read as instructions for making a
breaking change: be intentional about it, then write it up in
docs/migration.md. With 2.x released that nudges the wrong way.

State the 2.x compatibility contract in one bullet, mark docs/migration.md
as closed to new entries, and drop the Breaking Changes section, which only
existed to describe how to add to that file.

No-Verification-Needed: contributor-guidance doc only
@maxisbey maxisbey changed the title Close the door on breaking changes in AGENTS.md Stop framing breaking changes as a workflow in AGENTS.md Aug 16, 2026
@maxisbey
maxisbey force-pushed the docs/agents-api-stability branch from e5b1d03 to eb43bb6 Compare August 16, 2026 12:49
@maxisbey
maxisbey marked this pull request as ready for review August 16, 2026 17:09
@maxisbey
maxisbey merged commit 37b3cb1 into main Aug 16, 2026
33 checks passed
@maxisbey
maxisbey deleted the docs/agents-api-stability branch August 16, 2026 17:10

@claude claude 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.

Code review found no issues

No high-confidence issues detected in this change.

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