GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — docs: drop stale crate paths from AGENTS.md by leehanjeong · Pull Request #8425 · RustPython/RustPython

Conversation

@leehanjeong

@leehanjeong leehanjeong commented Aug 1, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor
  • Closes AGENTS.md: Repository Structure section predates the move into crates/ #8416
  • This PR follows our AI policy

Summary

AGENTS.md's "Repository Structure" section described the pre-crates/ layout (vm/, compiler/, derive/, etc.), none of which exist anymore. CONTRIBUTING.md already documents the current layout under "Code organization", so this replaces the stale list with a link there instead of maintaining two copies that can drift apart again.

Also generalized the file's title/intro from "GitHub Copilot Instructions" to "AI Agent Instructions". The file was already renamed from .github/copilot-instructions.md to AGENTS.md in #6813 to follow the cross-vendor AGENTS.md standard, but the title text was never updated to match. Happy to revert this part if the intent was Copilot-specific.

Assisted-by: Claude Code:claude-sonnet-5

Summary by CodeRabbit

  • Documentation
    • Updated contributor guidance to support a broader range of AI coding agents.
    • Replaced outdated repository structure details with a link to the current code organization documentation.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@leehanjeong
docs: drop stale crate paths from AGENTS.md
d054e40
The "Repository Structure" section listed pre-`crates/` paths (`vm/`,
`compiler/`, `derive/`, etc.) that no longer exist; `CONTRIBUTING.md`
already documents the current layout under "Code organization". Replace
the stale list with a link to that section.

Also generalized the title and intro from "GitHub Copilot Instructions"
to "AI Agent Instructions" — the file was already renamed from
`.github/copilot-instructions.md` to `AGENTS.md` in RustPython#6813 to follow the
cross-vendor AGENTS.md standard (https://agents.md/), but the title text
was never updated to match.

Assisted-by: Claude Code:claude-sonnet-5
@coderabbitai

coderabbitai Bot commented Aug 1, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

AGENTS.md now provides guidance for AI coding agents generally. It links to the current repository code organization in CONTRIBUTING.md instead of maintaining a static directory listing.

Changes

AI agent guidance

Layer / File(s) Summary
Update agent guidance
AGENTS.md
The document targets AI coding agents instead of GitHub Copilot specifically. The outdated repository structure listing is replaced with a link to CONTRIBUTING.md.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • RustPython/RustPython#7030: Updates AGENTS.md documentation for AI coding agents in different sections.

Suggested labels: skip:ci

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The changes replace the obsolete repository structure with a CONTRIBUTING.md link and generalize the Copilot-specific document title as required by issue #8416.
Out of Scope Changes check ✅ Passed The changes are limited to AGENTS.md documentation updates and align with the linked issue objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main documentation change: removing obsolete crate paths from AGENTS.md.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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
  • X
  • Mastodon
  • Reddit
  • LinkedIn

Comment @coderabbitai help to get the list of available commands.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@moreal moreal added the z-ca-2026 Tag to track Contribution Academy 2026 label Aug 1, 2026
@coderabbitai coderabbitai Bot mentioned this pull request Aug 1, 2026
docs: fix AGENTS.md's broken DEVELOPMENT.md references #8426
Merged
2 tasks done
youknowone
youknowone approved these changes Aug 1, 2026
View reviewed changes

@youknowone youknowone left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Thank you!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@youknowone
youknowone merged commit bd653da into RustPython:main Aug 1, 2026
11 of 12 checks passed

Uh oh!

There was an error while loading. Please reload this page.

youknowone pushed a commit that referenced this pull request Sep 16, 2026
@leehanjeong
docs: drop stale crate paths from AGENTS.md (#8425)
457e195
The "Repository Structure" section listed pre-`crates/` paths (`vm/`,
`compiler/`, `derive/`, etc.) that no longer exist; `CONTRIBUTING.md`
already documents the current layout under "Code organization". Replace
the stale list with a link to that section.

Also generalized the title and intro from "GitHub Copilot Instructions"
to "AI Agent Instructions" — the file was already renamed from
`.github/copilot-instructions.md` to `AGENTS.md` in #6813 to follow the
cross-vendor AGENTS.md standard (https://agents.md/), but the title text
was never updated to match.

Assisted-by: Claude Code:claude-sonnet-5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@youknowone youknowone youknowone approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

z-ca-2026 Tag to track Contribution Academy 2026

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AGENTS.md: Repository Structure section predates the move into crates/

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@leehanjeong @youknowone @moreal
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.