CI gate: enforce 10-line net addition limit on PRs #526

Closed
opened 2026-03-25 17:47:21 +00:00 by perplexity · 1 comment
Member

Context

CONTRIBUTING.md (PR #525) establishes a hard 10-line net addition limit. This ticket adds automated enforcement.

Requirements

  1. Webhook or CI check that runs on every PR:
    • git diff --stat main..HEAD → parse insertions/deletions
    • If net additions > 10, block merge and comment with the violation
    • Exceptions: files matching requirements*.txt, package*.json, *.lock (dependency config bootstrap)
  2. PR template (.gitea/PULL_REQUEST_TEMPLATE.md) requiring:
    • Net line diff section
    • Manual test plan section
    • Automated test results section
  3. Pre-push hook (optional, advisory) that warns locally before push

Acceptance Criteria

  • A PR with net +15 lines is blocked with a clear message
  • A PR with net +8 lines passes
  • Dependency config files are excluded from the count
  • PR template auto-populates on new PRs

Note

PRs #514 and #516 from this session both violated this policy retroactively. This is the bootstrap exception — the rule didn't exist yet. Going forward, zero exceptions.

## Context CONTRIBUTING.md (PR #525) establishes a hard 10-line net addition limit. This ticket adds automated enforcement. ## Requirements 1. **Webhook or CI check** that runs on every PR: - `git diff --stat main..HEAD` → parse insertions/deletions - If net additions > 10, block merge and comment with the violation - Exceptions: files matching `requirements*.txt`, `package*.json`, `*.lock` (dependency config bootstrap) 2. **PR template** (`.gitea/PULL_REQUEST_TEMPLATE.md`) requiring: - Net line diff section - Manual test plan section - Automated test results section 3. **Pre-push hook** (optional, advisory) that warns locally before push ## Acceptance Criteria - A PR with net +15 lines is blocked with a clear message - A PR with net +8 lines passes - Dependency config files are excluded from the count - PR template auto-populates on new PRs ## Note PRs #514 and #516 from this session both violated this policy retroactively. This is the bootstrap exception — the rule didn't exist yet. Going forward, zero exceptions.
perplexity self-assigned this 2026-03-25 17:47:21 +00:00
Owner

Closing during the 2026-03-28 backlog burn-down.

Reason: this issue is being retired as part of a backlog reset toward the current final vision: Heartbeat, Harness, and Portal. If the work still matters after reset, it should return as a narrower, proof-oriented next-step issue rather than stay open as a broad legacy frontier.

Closing during the 2026-03-28 backlog burn-down. Reason: this issue is being retired as part of a backlog reset toward the current final vision: Heartbeat, Harness, and Portal. If the work still matters after reset, it should return as a narrower, proof-oriented next-step issue rather than stay open as a broad legacy frontier.
Timmy closed this issue 2026-03-28 04:52:51 +00:00
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Timmy_Foundation/the-nexus#526