Skip to content

docs(claude): PR→issue hygiene standing rule - #3119

Merged
lsm merged 1 commit into
devfrom
docs/pr-issue-hygiene-rule
Sep 16, 2026
Merged

lsm merged 1 commit into
devfrom
docs/pr-issue-hygiene-rule

Conversation

@lsm

@lsm lsm commented Aug 27, 2026 •

Copy link
Copy Markdown
Owner

What

Adds one bullet to the CLAUDE.md decomposition standing rules:

  • Every PR description carries Closes #<issue> for its mapped issue; the post-approval procedure verifies the issue actually closed after squash-merge (close manually with the squash-commit reference if auto-close missed).
  • A post-merge base mismatch from linear dev advancement is informational, not a blocker — the gate binds to the PR head; record it in the merge audit and proceed.

Why

A merge audit on 2026-08-27 found 31 merged PRs had silently left their issues open (no closing keywords in PR bodies), and two coders escalated base mismatches as blockers that were normal sequential-merge behavior. Fleet-wide directive already delivered to all live sessions; this makes it durable for every future coder (CLAUDE.md is read from each task worktree at session start — no daemon restart needed).

Docs-only: one bullet in CLAUDE.md. No code, no migrations. (Body corrected: an earlier draft carried a stray Closes reference to an unrelated issue — this PR has no mapped issue, like #3064.)

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

…body, post-approval issue-close backstop, base mismatch is informational
@lsm
lsm force-pushed the docs/pr-issue-hygiene-rule branch from d884dfe to 5c5c1d4 Compare September 16, 2026 00:53
@hyperneo-ai-test

hyperneo-ai-test Bot commented Sep 16, 2026 •

Copy link
Copy Markdown

Review status

Started 2026-09-15T20:53:20-04:00
Finished 2026-09-15T20:59:40-04:00
Head 5c5c1d4
Engine claude (glm-5.3-flash)
Verdict no issues found

@lsm
lsm merged commit 8b64b8c into dev Sep 16, 2026
12 checks passed
@lsm
lsm deleted the docs/pr-issue-hygiene-rule branch September 16, 2026 00:56

@hyperneo-ai-test hyperneo-ai-test Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

Engine claude (glm-5.3-flash) · Reviewed commit 5c5c1d4 · Trigger: push

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant