From: Dmitry Volyntsev Date: Fri, 22 May 2026 03:58:24 +0000 (-0700) Subject: Updated agent commit message guidance X-Git-Tag: 1.0.0~39 X-Git-Url: http://git.kaiwu.me/postgresql/log/contrib/postgres_fdw/stylesheets/print.css?a=commitdiff_plain;h=9c4cb3fff4109a1e38b92781dafade2f2a743e6f;p=njs.git Updated agent commit message guidance --- diff --git a/AGENTS.md b/AGENTS.md index f1008db0..28af20a3 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -35,10 +35,9 @@ Quick facts: - **Tests:** `make unit_test`, `make lib_test`, `make test262`. NGINX integration tests under `nginx/t/` run with `prove -I nginx/t/`. -- **Code style:** NGINX conventions — 4 spaces (no tabs), 80-column limit, - no trailing whitespace, newline after closing brace, `-Werror` build. -- **Commit subjects:** past tense, prefixed - (`HTTP:`, `Stream:`, `Core:`, `QuickJS:`, `Tests:`, …), ≤67 characters. +- **Code style and commits:** follow + [docs/agent/engine-dev.md](docs/agent/engine-dev.md#code-style-and-commits) + for formatting, warnings, and commit-log style. Full details, sanitizer builds, VM architecture, and object model: [docs/agent/engine-dev.md](docs/agent/engine-dev.md). diff --git a/docs/agent/engine-dev.md b/docs/agent/engine-dev.md index 74227852..f7a764c0 100644 --- a/docs/agent/engine-dev.md +++ b/docs/agent/engine-dev.md @@ -148,9 +148,12 @@ NGINX coding style: Commit messages: - Past tense subject (`Added X`, `Fixed Y`). -- Subject ≤67 chars, body wrapped to ~80 chars. -- Subject prefix: `HTTP:`, `Stream:`, `Core:`, `QuickJS:`, `Tests:`, - `Modules:`, etc. +- Subject ≤72 chars, body wrapped to ~80 chars. +- Use a subject prefix only when local history clearly uses one for the + changed area, such as `HTTP:`, `Stream:`, `QuickJS:`, `Tests:`, or + `Modules:`. Otherwise use no prefix; do not invent generic prefixes. +- Read recent commit history and write commit logs in the established njs + style. - One logical change per commit; rebase/squash before submitting. ## Project layout