diff --git a/.agents/skills/openclaw-landable-bug-sweep/SKILL.md b/.agents/skills/openclaw-landable-bug-sweep/SKILL.md index 52a24194af00..398b2871b6f9 100644 --- a/.agents/skills/openclaw-landable-bug-sweep/SKILL.md +++ b/.agents/skills/openclaw-landable-bug-sweep/SKILL.md @@ -88,7 +88,7 @@ Reject: - include PRs linked from issues and duplicates 3. For each cluster: - read issue/PR body, comments, labels, linked refs, current source, adjacent tests - - suppress maintainer-owned queue noise unless it is the best fix path + - exclude PRs authored by wide-access maintainers until `created_at` is at least 14 days old; only a named PR or explicit maintainer-work request overrides - identify opener/author and preserve credit - decide: `repair-existing-pr`, `create-new-pr`, `close-fixed-on-main`, `close-duplicate`, or `reject` 4. Prove before patching: diff --git a/.agents/skills/openclaw-pr-maintainer/SKILL.md b/.agents/skills/openclaw-pr-maintainer/SKILL.md index dbf980319eda..9e6e9ab58d92 100644 --- a/.agents/skills/openclaw-pr-maintainer/SKILL.md +++ b/.agents/skills/openclaw-pr-maintainer/SKILL.md @@ -76,9 +76,9 @@ gh api -X POST "repos/openclaw/openclaw/issues//assignees" -f 'assignees - If `name` is empty, use the login only. If profile lookup is rate-limited or unavailable, say `account age unknown` rather than omitting the opener. - Use identity and activity as triage signal, not proof by itself: new, low-activity, or bot-like accounts can raise review caution, but code, repro, and CI evidence still decide. -## Suppress top-maintainer items in issue triage +## Suppress recent wide-access maintainer PRs in triage -When asked for issue triage, hot issues, pressing bugs, Discord-correlated issues, or "what is still open", do not surface issues or PRs authored by top maintainers by default. Prefer external/user-reported hot issues and external PRs, not maintainer-owned work queues. +In generic issue/PR triage, hot queues, landable shortlists, or "what is still open", exclude PRs authored by maintainers with broad repository access until 14 days after `created_at`. Prefer external contributors' PRs. An ordinary request for landing candidates does not override the age gate. Continue suppressing maintainer-authored issues by default. Suppress by default when the opener/author is one of: @@ -107,8 +107,9 @@ Also suppress lower-priority maintainer-owned noise from the broader keep/top-ma Exceptions: -- Show maintainer-authored items when the requester explicitly asks for maintainer PRs/issues, PR landing candidates, release-blocking maintainer work, or a specific PR/issue number. -- Show a maintainer-authored item when it is the canonical fix for an external hot issue, but frame it as the fix path rather than as a user-facing issue candidate. +- Once a maintainer-authored PR is at least 14 days old, triage it normally. +- A specific PR/issue number or an explicit request for maintainer-owned work overrides suppression. +- When a recent maintainer PR is the canonical fix for an external hot issue, mention it only as the fix path; do not count it as a triage or landing candidate. - Do not close, label, or deprioritize solely because an item is maintainer-authored; this section only controls what appears in triage shortlists. ## Apply close and triage labels correctly diff --git a/AGENTS.md b/AGENTS.md index e3ab71c23c1e..21625ef5605b 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -159,6 +159,7 @@ Skills own workflows; root owns hard policy and routing. - PR superseded by `main`: if code proof shows `main` already has same-or-better behavior, comment canonical commit/PR + focused proof, then close. Bar high: inspect PR diff, current code/tests, linked issue, caller/sibling path. If unsure, leave open. - Issue/PR numbers need a short summary every time; assume the reader has not opened or read them. - Before presenting a batch of issues/PRs, use smart subagents to verify live state and current `main`; omit closed/fixed items, and comment+close items already fixed on `main` when maintainer action is authorized. +- Generic triage and landing shortlists: exclude PRs authored by maintainers with broad repository access until 14 days after creation. An ordinary request for landing candidates does not override this gate; only a named PR or explicit request for maintainer-owned work does. - PR review answer: bug/behavior, URL(s), affected surface, provenance for regressions when traceable, best-fix judgment, evidence from code/tests/CI/current or shipped behavior. - PR reviewable findings: post them on the PR, not chat-only, so author sees actionable feedback. - Issue/PR final answer: last line is the full GitHub URL.