Skip to content

Condense AGENTS.md - #9814

Merged
youknowone merged 1 commit into
RustPython:mainfrom
youknowone:agents-md-trim
Nov 12, 2026
Merged

youknowone merged 1 commit into
RustPython:mainfrom
youknowone:agents-md-trim

Conversation

@youknowone

Copy link
Copy Markdown
Member

One of checkbox below must be checked.

  • I did not use AI to write the code of this patch.
  • This PR follows our AI policy

Summary

AGENTS.md goes from 306 lines to 109. Supersedes #9703.

Merged

  • The Lib/test marker rules and the test-modification rules were stated in two sections. They are now one section.
  • The test commands were listed under both "Commit Hooks" and "Testing". They are now only under "Testing".
  • Python code rules are folded into "Working with the Lib Directory".

Changed

  • Required checks are scoped by change type in a table. Documentation-only changes need no test run. Clippy is listed with the Rust checks.
  • The marker spelling is TODO: RUSTPYTHON; <reason> everywhere.
  • The grep example is replaced with rg.
  • The clean-build command uses find . -name '*.pyc' -delete.
  • scripts/whats_left.py is noted as running under CPython 3.15 or newer.

Removed

  • "Integration Between Rust and Python" and "Common Patterns" (pymodule example, add_native_module example).
  • The Bad/Good code example for branch deduplication; the rule itself stays as one sentence.
  • "Follow Rust best practices", the PEP 695 example, REPL and JIT run examples, cargo doc, and the docs.rs link.

Unchanged

  • AI policy, git operation, and commit hook rules.
  • Marker forms and the expectedFailure vs skip criteria.
  • Comment rules, including the /// docstring rule.
  • venvlauncher, zizmor, WASM build command.

Written by Claude Code (claude-fable-5-2) under maintainer direction.

🤖 Generated with Claude Code

Merge the duplicated Lib/test marker and test-modification rules into
one section, and the duplicated test commands into the Testing section.
Scope the required checks by change type; documentation-only changes
need none.

Remove the pymodule examples, the Rust/Python integration list, the
branch-deduplication code example, and the cargo doc and docs.rs
references. Replace the grep example with rg, drop the PEP 695 example,
and use one marker spelling. Note that scripts/whats_left.py runs under
CPython 3.15 or newer.

Assisted-by: Claude Code:claude-fable-5-2
@chatgpt-codex-connector

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-12T03:08:47.475932Z
We couldn’t complete this review. Comment @codex review to try again.
f1cdf5f PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

Caution

Review failed

The pull request is closed.

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@youknowone youknowone added the skip:ci Skip running the ci label Nov 12, 2026
@youknowone
youknowone merged commit 2933db2 into RustPython:main Nov 12, 2026
7 checks passed
@youknowone
youknowone deleted the agents-md-trim branch November 12, 2026 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip:ci Skip running the ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AGENTS.md causes instruction overload and context rot

1 participant