[Kitenite]

What & why

Add contributor responsibility guidance inspired by MLX's AGENTS.md. Contributors must understand, review, and verify submitted changes, including agent-generated code.

Explicitly allow authorized agent assistance with commit messages, PR descriptions, commits, pushes, PR creation, and review feedback. Require explicit user authorization before posting reviewer replies and accurate reporting of validation. No blanket prohibition on agent-created PRs or contributor-ban language.

Also copy MLX's code standards while preserving the existing Superset instructions.

How I tested it

• git diff --check passed.
• bun run lint:fix and bun run lint passed; no fixes were needed.
• bun run typecheck passed (42 tasks).
• bun run test stopped on a five-second timeout in the existing i18n display-only test. Running bun test packages/i18n/test/display-only.test.ts separately passed all 10 tests.

Checklist

• PR title follows conventional commits (type(scope): subject)
• bun run lint and bun run typecheck pass (CI fails on lint warnings too)
• "Allow edits from maintainers" is checked on fork PRs (not applicable: same-repository branch)

Summary by CodeRabbit

• Documentation
  • Updated contributor guidance with expectations for reviewing and validating changes, clarifying requests, explaining work, and responding to reviews.
  • Added guidance on authorized agent actions and accurately reporting completed checks.
  • Added standards and examples for concise comments and clear technical writing. These updates provide clearer guidance for preparing and reviewing contributions.

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 8e532d51-738b-4dc3-9ec0-734b4b80943d

📥 Commits

Reviewing files that changed from the base of the PR and between 3fe2c36 and 528a795.

📒 Files selected for processing (1)

• AGENTS.md

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review.

---

📝 Walkthrough

Walkthrough

AGENTS.md adds contributor responsibilities for reviewing and verifying changes, handling underspecified requests, and describing agent actions and validation. It also adds code standards for concise comments and Simplified Technical English, with C++ comment examples.

Changes

Contributor guidance

Layer / File(s)	Summary
Contributor responsibilities and code standards AGENTS.md	Adds guidance for review, verification, unclear requests, authorized agent actions, reviewer replies, and accurate claims about checks. Adds comment-writing standards and C++ examples.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: saddlepaddle

Merge Risk: ⚪ Minimal · up to 528a7

The guidance preserves useful explanations of non-obvious traps while discouraging overly detailed comments. No material merge-readiness concern is established.

Architecture Summary

Architecture risk: 🔵 Low · up to 528a7

The change affects 1 system.

Changed systems: AGENTS.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

• observed — AGENTS.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

• observed — Modified behavior in AGENTS.md: Adds contributor responsibilities and code standards, including limits on agent actions and claims, plus examples contrasting uncommented self-describing code with excessive commentary.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title uses the docs commit type and clearly identifies the main change: contributor guidance for agent-assisted changes.
Description check	✅ Passed	The description includes the required What & why, How I tested it, and Checklist sections. It explains the documentation changes and reports validation results, including the test timeout and separate…

✨ Finishing Touches 💡 1

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

🛠️ Fix failing CI checks 💡

• Commit to this branch
• Create a new PR

---

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.}

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

• ✅ Neon database branch

Thank you for your contribution! 🎉