Skip to content

[Klaud Cold] AGENTS.md: sync codeowner-signoff-verify.yml with PR_REVIEW_CHECKLIST.md updates#2024

Merged
functionstackx merged 1 commit into
mainfrom
klaud/agents-signoff-sync-rule
Jul 4, 2026
Merged

[Klaud Cold] AGENTS.md: sync codeowner-signoff-verify.yml with PR_REVIEW_CHECKLIST.md updates#2024
functionstackx merged 1 commit into
mainfrom
klaud/agents-signoff-sync-rule

Conversation

@functionstackx

Copy link
Copy Markdown
Collaborator

Summary

  • Adds a "Checklist ↔ sign-off verifier sync" rule to AGENTS.md: docs/PR_REVIEW_CHECKLIST.md is the source of truth for the merge standard, and whenever its policy items change (added/removed/materially reworded), agents are allowed and expected to update the checks in codeowner-signoff-verify.yml to match — ideally in the same PR.
  • Notes the boundaries: cosmetic/formatting/_zh-sync edits need no verifier change, and Check 5 already catches stale sign-off templates automatically; only new/removed policy items need check-logic changes.
  • Documents the validation recipe used in [Klaud Cold] Align codeowner-signoff-verify with latest PR_REVIEW_CHECKLIST.md #2015: merge, throwaway [DO NOT MERGE] test PR, sign-off comment containing the exact trigger phrase, read the verdict, close.

Test plan

  • Docs-only (agent guidance); no runtime surface.

🤖 Generated with Claude Code

…HECKLIST.md

Agents are allowed and expected to update the sign-off verifier workflow
whenever the checklist's policy items change, and to validate the change
via a throwaway sign-off test PR.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@functionstackx functionstackx requested a review from a team July 4, 2026 02:28
@functionstackx functionstackx merged commit 5b0a22c into main Jul 4, 2026
3 checks passed
@functionstackx functionstackx deleted the klaud/agents-signoff-sync-rule branch July 4, 2026 02:28

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM — docs-only agent-guidance addition to AGENTS.md documenting the checklist ↔ verifier sync rule.

Extended reasoning...

Overview

This PR adds a single paragraph to AGENTS.md documenting a "Checklist ↔ sign-off verifier sync" rule — declaring docs/PR_REVIEW_CHECKLIST.md as the source of truth for the merge standard, and instructing agents to update .github/workflows/codeowner-signoff-verify.yml when checklist policy items are added/removed/materially reworded. It also notes cosmetic-edit exceptions and documents the validation recipe (throwaway [DO NOT MERGE] test PR).

Security risks

None. This is a text-only edit to an agent-instruction file. No runtime code, no workflows, no permissions, no secrets. AGENTS.md is an explicit _zh-exemption per the file's own docs rule, so no bilingual mirror is required.

Level of scrutiny

Very low. The change is a docs-only, self-contained addition of guidance for future agents. There is no logic, no configuration, no CI wiring modified. The paragraph is internally consistent with the surrounding AGENTS.md conventions (bilingual rules, _zh exceptions, Check 5 semantics).

Other factors

The added guidance is accurate about the workflow's behavior (Check 5 dynamically loads the checklist, trigger phrase is required) and references the correct validation approach used in PR #2015. Content is coherent, no contradictions with existing sections.

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

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant