docs: state the principles in CLAUDE.md and shorten REVIEW.md

[claude]

Requested by Felix Weinberger · Slack thread

CLAUDE.md is now eight principles and one sentence saying that style belongs to the formatter and linter; REVIEW.md is one sentence pointing at those principles plus the style line; CONTRIBUTING.md gains a short "Good to know" list under Development.

The previous files carried style rules the formatter and linter contradict and an architecture walkthrough that had gone stale, so automated review kept quoting them. Principles over process: one sentence each, nothing a linter can check.

No package changes, so no changeset.

🤖 Generated with Claude Code

https://claude.ai/code/session_012VRbFCp41otcScXE1YY3es

---

Generated by Claude Code

[changeset-bot]

⚠️ No Changeset found

Latest commit: eb7d661

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[pkg-pr-new]

Open in StackBlitz

@modelcontextprotocol/client

npm i https://pkg.pr.new/@modelcontextprotocol/client@2938

@modelcontextprotocol/codemod

npm i https://pkg.pr.new/@modelcontextprotocol/codemod@2938

@modelcontextprotocol/core

npm i https://pkg.pr.new/@modelcontextprotocol/core@2938

@modelcontextprotocol/server

npm i https://pkg.pr.new/@modelcontextprotocol/server@2938

@modelcontextprotocol/server-legacy

npm i https://pkg.pr.new/@modelcontextprotocol/server-legacy@2938

@modelcontextprotocol/express

npm i https://pkg.pr.new/@modelcontextprotocol/express@2938

@modelcontextprotocol/fastify

npm i https://pkg.pr.new/@modelcontextprotocol/fastify@2938

@modelcontextprotocol/hono

npm i https://pkg.pr.new/@modelcontextprotocol/hono@2938

@modelcontextprotocol/node

npm i https://pkg.pr.new/@modelcontextprotocol/node@2938

commit: eb7d661

[claude]

Nothing blocking. The comments below are optional suggestions. There is no need to push a fix for them before merging.

Beyond the inline findings, I checked that every path the new text points at resolves — test/e2e/requirements.ts, packages/codemod/src/migrations/v1-to-v2/mappings/, and the per-package test/ layout all exist — and that packages/client and packages/server still carry barrelClean tests, so the browser-safe root-entry rule remains machine-checked for those two packages even with the written rule gone.

Extended reasoning...

Docs-only change rewriting CLAUDE.md to eight principles, cutting REVIEW.md to a pointer, and adding a short list to CONTRIBUTING.md; no code or security-sensitive surface. Not approved because the inline findings concern deleted guidance (the auto-maintained Recurring Catches section and the runtime-neutral export rule) and the removal of a review-convention file is a policy decision a maintainer should weigh.

[claude]

Code review found no issues

No high-confidence issues detected in this change.