From ee5b06f9fe99439d1722252b0441bfd6cb052dd5 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 11 May 2026 14:26:42 +0100 Subject: [PATCH] docs: clarify contributor changelog ownership --- AGENTS.md | 1 + CONTRIBUTING.md | 1 + 2 files changed, 2 insertions(+) diff --git a/AGENTS.md b/AGENTS.md index 46de681ebf7..bc0dddea7ff 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -116,6 +116,7 @@ Skills own workflows; root owns hard policy and routing. - Codex harness upgrade (`extensions/codex/package.json` `@openai/codex`): refresh `docs/plugins/codex-harness.md` model snapshot from the new harness `model/list`. - Docs final answers: include relevant full `https://docs.openclaw.ai/...` URL(s). If issue/PR work too, GitHub URL last. - Changelog entries: active version `### Changes`/`### Fixes`; single-line bullets only. +- Contributor PR authors should not edit `CHANGELOG.md`; maintainer/AI adds entries during landing/merge. - Contributor-facing changelog entries thank credited human `@author`. Never thank bots, `@openclaw`, `@clawsweeper`, or `@steipete`; if unknown, omit thanks. ## Git diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 8f489e609f5..a52186bfb80 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -107,6 +107,7 @@ For coordinated change sets that genuinely need more than 20 PRs, join the **#cl - Test locally with your OpenClaw instance - External PRs must include a filled **Real behavior proof** section in the PR body. Show the real setup you tested, the exact command or steps you ran after the patch, after-fix evidence, the observed result, and anything you did not test. Screenshots, recordings, terminal screenshots, console output, copied live output, linked artifacts, and redacted runtime logs all count. Unit tests, mocks, snapshots, lint, typechecks, and CI are useful but do not satisfy this requirement by themselves. Maintainers may apply `proof: override` only when the proof gate should not apply. +- Do not edit `CHANGELOG.md` in contributor PRs. Maintainers or ClawSweeper add the changelog entry when landing user-facing changes. - Run tests: `pnpm build && pnpm check && pnpm test` - For iterative local commits, `scripts/committer --fast "message" ` passes `FAST_COMMIT=1` through to the pre-commit hook so it skips the repo-wide `pnpm check`. Only use it when you've already run equivalent targeted validation for the touched surface. - For extension/plugin changes, run the fast local lane first: