Skip to content

docs(policy): limit outside pull requests to fix and perf - #1100

Merged
vastsa merged 1 commit into
mainfrom
docs/pr-acceptance-scope
Sep 26, 2026
Merged

vastsa merged 1 commit into
mainfrom
docs/pr-acceptance-scope

Conversation

@vastsa

@vastsa vastsa commented Sep 26, 2026

Copy link
Copy Markdown
Owner

What

Temporarily restricts outside contributions to perf and fix pull requests. feat, refactor, docs, test, chore, build, and ci PRs from contributors without write access are closed without merging, and are not reimplemented as a replacement while the window is in force.

Maintainers (accounts with write access, plus the branches and automated agent work they direct) keep every change type.

Effective 2026-09-26 until removed; lifted by deleting the recorded scope and R6.1.

Files

  • CONTRIBUTING.md — "Pull Request Acceptance Scope (temporary)" section, plus the pointer in "Commit and Pull Request"
  • docs/spec/06-delivery/03-ai-development-workflow.md — new R6.1 under R6, and an acceptance-criteria item
  • docs/zh-CN/spec/06-delivery/03-ai-development-workflow.md — zh-CN mirror
  • AGENTS.md / CLAUDE.md — temporary PR type scope in §15 / "Issue and PR intake", Policy-Sync: 2026-09-26.1 in both

Validation

  • pnpm check:agent-policy — passed
  • pnpm check:pr-base — passed (contains origin/main 79f3698)
  • pnpm --filter @pi-desktop/docs check:locales — passed (81 pairs)
  • pnpm --filter @pi-desktop/docs check:docs — passed (516 pages)
  • pnpm docs:build — passed, no dead links
  • git diff --check — clean

Docs-only change; no runtime behavior, no E2E impact.

Notes

  • No CI enforcement: the gate is the review policy, not a status check.
  • R6.1 keeps the full root-cause and minimality bar for accepted fix / perf PRs, and clarifies that closing an out-of-scope type is not the "do not discard contributor work" violation.

Outside contributions are temporarily restricted to `perf` and `fix`
pull requests, so maintainer review bandwidth stays on regressions
while feature planning remains on the roadmap. Maintainers, including
the branches and agent work they direct, keep every change type.

The rule is recorded in CONTRIBUTING.md and as R6.1 in the delivery
workflow spec plus its zh-CN mirror, and mirrored into AGENTS.md and
CLAUDE.md with a Policy-Sync bump.
Copilot AI lite review requested due to automatic review settings September 26, 2026 13:11

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vastsa
vastsa merged commit 18770f8 into main Sep 26, 2026
2 checks passed
vastsa added a commit that referenced this pull request Sep 26, 2026
Refresh the task candidate against the dead-code cleanup (#1099) and
the outside-PR policy change (#1100) before pushing, so the branch is
not behind main and the checks run on the integrated state.
@vastsa
vastsa deleted the docs/pr-acceptance-scope branch September 27, 2026 18:39
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.

2 participants