PR summaries: 150-word limit, checked on GitHub, required to merge

[EdbertChan]

Summary

A checker reads every PR write-up before it goes out. It makes sure a busy reader can follow the short summary at the top.

It had no length limit. A summary could run on and still pass.

It also only ran when Claude wrote the PR by hand. Other write-ups skipped it. One recent PR went out long and full of file names.

Now a summary over 150 words fails, and the message says how many words to cut. The writing guide says to use plain, short words.

A new check on GitHub runs on every PR, even after edits on the website. A PR that fails it cannot merge.

Review Claim

Approve a 150-word limit on the PR summary, and a GitHub check that runs the same PR checker on every PR and must pass before merging.

Review Lane

behavior

Review Unit

engine-runtime

Safety Invariant

The merge queue now needs the new check to pass, next to lint and test; nothing else in the queue rules changes. The limit counts words in the Summary section only; later sections stay unlimited. A missing Summary is reported as "unchecked", never as clean.

Slice Rationale

The word limit, the guide text, and the GitHub check are one rule: the guide states it, the checker enforces it, and CI makes sure the checker actually runs. Without the CI piece, the limit only applies to PRs written through the local skill, which is the gap that let PR #518 out.

Non-goals

• Does not rewrite the 17 open PRs whose bodies already fail the existing rules (scan below). Each one now needs its body fixed before it can merge.
• Does not change Invoker's own copy of the PR skill.

Test Plan

Test Plan

Fail-before: new tests added, checker unchanged.

FAIL: test_summary_over_150_words_fails_and_says_how_many_to_cut
AssertionError: 0 != 1 : PR body validation passed.
Ran 3 tests in 2.804s
FAILED (failures=1)
exit=1

Pass-after, whole draft-pr suite:

Ran 16 tests in 32.059s
OK
exit=0

Preflight:

declare Review Unit: engine-runtime
ok      preflight passed
preflight exit=0

Impact scan: validate-pr-body.mjs run on every open PR's live body with its changed files. 12 pass, 17 fail, 0 unchecked. No PR fails on the new word limit; every failure is an existing rule (code names, reading grade, missing sections, bad review unit).

• python3 -m unittest test_draft_pr_scripts (engine/skills/draft-pr/tests)
• python3 engine/skills/make-pr/scripts/preflight.py --base origin/main
• .mergify.yml parses; merge conditions read back as ['check-success = lint', 'check-success = test', 'check-success = validate']
• The PR body workflow on this PR's real runs, both ways:

run 34736161445 (PR opened, body = commit message only)
PR body validation failed:
- Missing required section: ## Summary
##[error]Process completed with exit code 1.

run 34736169352 (after the body edit)
PR body validation passed.

Revert Plan

Revert Plan

• Safe to revert? Yes
• Revert command: git revert <sha>
• Post-revert steps: None. The GitHub check and its merge requirement disappear, and summaries of any length pass again.
• Data migration? No

🤖 Generated with Claude Code

https://claude.ai/code/session_014uqkMWYjsKGenUSA1CqBVv

[cursor]

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_305bee43-03ed-469e-9792-53a8863ef6a4)

[mergify]

Queued — the merge queue status continues in this comment ↓.

[EdbertChan]

Revision history

#	Type	Changes	Reason	Date
1	initial	b8daef1		2026-09-13 03:52 UTC
2	content	b8daef1 → f83499e (raw)		2026-09-13 03:52 UTC

[cursor]

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_53be56a5-d7c9-4b9c-a50b-dbef8c586a31)

[EdbertChan]

@Mergifyio queue

[mergify]

Merge Queue Status

• ✅ Entered queue — 2026-09-13 03:58 UTC · Rule: admin-bypass · triggered by @EdbertChan with the @mergifyio queue command
• ✅ Checks passed · on draft merge queue: checking #524 on main (0dcf8ec) #526
• ✅ Merged — 2026-09-13 04:04 UTC · at 1343d9548d9da12d8fea23ea336e43af78788d83 · squash

This pull request spent 5 minutes 50 seconds in the queue, including 5 minutes 6 seconds running CI.

Required conditions to merge

• check-success = lint
• check-success = test