Repository navigation
RFC 000: deprecate before removing, in line with INVARIANTS.md - #1295
Conversation
RFC 000 said there would be no soft deprecation before 1.0, which contradicts the Deprecations policy added to INVARIANTS.md in #1277.
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
There was a problem hiding this comment.
Alignment Review Report
Automated Checks
- Lint: N/A — the diff touches only
rfcs/000-project-phases.md(Markdown, no Python), sousort/ruff(.claude/hooks/lint.sh) don't apply.uvwasn't available in this execution environment to confirm, but there is nothing lintable in this change regardless. - Debug code: CLEAN —
.claude/hooks/check-debug.shonly surfaces pre-existingprint/TODOoccurrences elsewhere insrc/; none originate from this diff.
Open RFCs Context
- RFC 000 (
rfcs/000-project-phases.md, Status: In Review) is the only RFC touched. The edit amends its "Our approach towards breaking changes" section and adds an Amendment History entry. - No other open RFCs (001–012) reference deprecation or are otherwise affected by this docs-only change.
Tier 1: Fixes Required
None. This is a 6-line, Markdown-only amendment (2 insertions, swap of one sentence, one amendment-history bullet, and an updated Amended date). No syntax errors, no debug code, no lint surface.
Tier 2: Alignment Discussion
Principle Conflicts
None identified. The change doesn't touch the Gymnasium API, client/server boundary, reward placement, or the WebSocket/MCP dual-API model from PRINCIPLES.md — it's purely about release/removal process.
RFC Conflicts
None identified — and worth calling out explicitly why: this PR is itself the resolution of a previously-flagged conflict, not a new one. RFC 000 used to say "we will not put soft deprecation in place until 1.0", which contradicted the Deprecations policy merged into .claude/docs/INVARIANTS.md in #1277 (same author). That exact contradiction was flagged by this automation's review on #1277, and the PR description here explicitly references that flag and cc's @Darktex (RFC 000's author of this section, confirmed via git blame). The new RFC 000 text now matches INVARIANTS.md on every substantive point: FutureWarning, naming the removal release + replacement, and "at least two minor releases" pre-1.0 — the post-1.0 sentence is left untouched. I verified the two documents are now textually consistent, so no residual conflict remains.
One light process note rather than a blocking flag: RFC 000 lists four authors (@Darktex, @pankit-eng, @jspisak, @zkwentz) and is still "In Review" — the author has already looped in @Darktex, which covers the substantive sign-off; no further action needed from my side.
Summary
- 0 mechanical issues to fix
- 0 principle-alignment points for human review
- 0 open RFC conflicts — this PR resolves the one previously flagged on #1277
Small, well-scoped docs fix that reconciles RFC 000 with the already-merged INVARIANTS.md deprecation policy. No concerns blocking merge.
Sent by Cursor Automation: Pre-review


Summary
RFC 000 says "we will not put soft deprecation in place until 1.0", but the Deprecations policy added to
.claude/docs/INVARIANTS.mdin #1277 (and first used in #1276) requires deprecating before removing. Cursor's alignment review on #1277 flagged this. This updates the "breaking changes" paragraph of RFC 000 to match: removals of APIs, CLI flags and environments get aFutureWarningnaming the removal release and the replacement, for at least two minor releases, with a pointer to INVARIANTS.md. It also adds an amendment history entry. The post-1.0 sentence stays as it was.cc @Darktex, since you're one of the RFC's authors.
Type of Change
Alignment Checklist
Before submitting, verify:
.claude/docs/PRINCIPLES.mdand this PR aligns with our principles.claude/docs/INVARIANTS.mdand no invariants are violated/pre-submit-pr(orbash .claude/hooks/lint.shand tests) and addressed all issuesRFC Status
rfcs/000-project-phases.md), amended hereTest Plan
Docs only. RFC 000 and INVARIANTS.md now describe the same rule.
Claude Code Review
N/A.
This PR was prepared with AI assistance.
Note
Low Risk
Documentation-only RFC amendment; no runtime or API behavior changes.
Overview
RFC 000 is amended so its pre-1.0 breaking-change stance matches Deprecations in
.claude/docs/INVARIANTS.md, instead of saying soft deprecation would not happen until 1.0.The Amended date is October 1, 2026, with a new amendment-history note that API, CLI flag, and environment removals must be deprecated first. The Our approach towards breaking changes section now states that removals keep working with a
FutureWarning(removal release + replacement) for at least two minor releases, with a pointer to INVARIANTS. Fast iteration and release-note documentation for other breaks are unchanged; the post-1.0 guarantees sentence is unchanged.Reviewed by Cursor Bugbot for commit 346435c. Bugbot is set up for automated code reviews on this repo. Configure here.