Skip to content

docs(agents): Remove directory AGENTS.md files - #2569

Merged
dcramer merged 1 commit into
mainfrom
docs/remove-directory-agents-md
Nov 2, 2026
Merged

dcramer merged 1 commit into
mainfrom
docs/remove-directory-agents-md

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

The 33 directory AGENTS.md files grew a little with each feature patch. 10 of the last 12 commits that touched them were feature fixes that added rules. Most bullets repeated what the code, types, tests, or existing comments already say.

This PR removes all of them and keeps only the root AGENTS.md. Code defines how its systems work. Comments at the owning code record intent where it is needed.

  • I checked each bullet against the code. Most were already in a JSDoc or a module header, for example the Docker per-Sandbox network, BullMQ runtime imports, Google service-account scope, the TTS no-retry rule, and service-account order. Those bullets are dropped.
  • The few invariants that had no home in the code are now comments at the owning symbol: the scheduler credential binding label, the 0.107.1 migration bridge guard, GitHub Junior-owned writes never falling back to user OAuth, Guardian rejection marker versioning, recording privacy, and the Roach copy's upstream commit and update steps.
  • pnpm file-length:check now fails when an AGENTS.md exists outside the repo root, so agents stop adding new ones.
  • I updated the root AGENTS.md, policies/documentation.md, ast-grep and check-script messages, skills, and contributor docs so they no longer point to deleted files.

Some procedural content is gone and not moved anywhere: eval local setup and Roach recording steps, plus dashboard UI conventions that the style check does not enforce. If we want those back, an evals or dashboard skill is a better home than a directory doc.

via @dcramer.

--

View Session [Sentry]

@vercel

vercel Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
junior-docs Ready Ready Preview Nov 2, 2026 9:56pm UTC

Request Review

@github-actions github-actions Bot added the risk: high PR risk score: high label Nov 2, 2026
Keep only the root AGENTS.md. Move the few invariants that had no code home into comments at the owning code, and fail file-length:check when a directory AGENTS.md comes back.

Co-Authored-By: David Cramer <23610+dcramer@users.noreply.github.com>

Co-Authored-By: David Cramer <23610+dcramer@users.noreply.github.com>
@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

1 screenshot change — 1 changed · 0 added · 0 removed

Review screenshots in Frameshift

Settings Mcp · Desktop
Settings Mcp · Desktop
Changed

@dcramer
dcramer merged commit e26b434 into main Nov 2, 2026
50 of 51 checks passed
@dcramer
dcramer deleted the docs/remove-directory-agents-md branch November 2, 2026 22:05

This branch was successfully deployed

1 active deployment
Preview – junior-docs — 57039017 Deployed Nov 2, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: high PR risk score: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant